| opendevreview | Merged openstack/diskimage-builder master: Add thin provisioning support to growvols https://review.opendev.org/c/openstack/diskimage-builder/+/848688 | 01:31 |
|---|---|---|
| *** darkowlzz6 is now known as darkowlzz | 02:14 | |
| *** darkowlzz4 is now known as darkowlzz | 02:22 | |
| *** darkowlzz5 is now known as darkowlzz | 02:42 | |
| *** darkowlzz9 is now known as darkowlzz | 02:52 | |
| *** darkowlzz2 is now known as darkowlzz | 03:13 | |
| opendevreview | Ian Wienand proposed opendev/system-config master: Revert "Pin version of grafana-oss container" https://review.opendev.org/c/opendev/system-config/+/852056 | 04:20 |
| *** ysandeep|out is now known as ysandeep | 04:25 | |
| opendevreview | Ian Wienand proposed opendev/base-jobs master: Update fedora-latest to Fedora 36 https://review.opendev.org/c/opendev/base-jobs/+/854331 | 04:39 |
| opendevreview | Ian Wienand proposed opendev/base-jobs master: Add Rocky Linux 9 nodeset https://review.opendev.org/c/opendev/base-jobs/+/854332 | 04:39 |
| opendevreview | Ian Wienand proposed zuul/zuul-jobs master: ansible-lint: pin to < 6.5 https://review.opendev.org/c/zuul/zuul-jobs/+/854335 | 05:17 |
| *** soniya29 is now known as soniya29|ruck | 05:50 | |
| soniya29|ruck | ' | 05:51 |
| *** ysandeep is now known as ysandeep|afk | 07:00 | |
| *** jpena|off is now known as jpena | 07:34 | |
| *** soniya29|ruck is now known as soniya29|ruck|lunch | 08:09 | |
| * pojadhav afk | 08:09 | |
| *** soniya29|ruck|lunch is now known as soniya29|ruck | 09:37 | |
| *** ysandeep|afk is now known as ysandeep|break | 10:16 | |
| *** rlandy|out is now known as rlandy | 10:37 | |
| *** ysandeep|break is now known as ysandeep | 11:03 | |
| *** cloudnull1 is now known as cloudnull | 11:06 | |
| *** ShadowJonathan_ is now known as ShadowJonathan | 11:06 | |
| *** benj_4 is now known as benj_ | 11:06 | |
| *** open10k8s_ is now known as open10k8s | 11:06 | |
| *** JpMaxMan__ is now known as JpMaxMan_ | 11:06 | |
| *** simondodsley_ is now known as simondodsley | 11:06 | |
| *** seongsoocho_ is now known as seongsoocho | 11:06 | |
| *** PrinzElvis_ is now known as PrinzElvis | 11:06 | |
| *** knikolla_ is now known as knikolla | 11:06 | |
| *** rpittau_ is now known as rpittau | 11:06 | |
| *** TheJulia_ is now known as TheJulia | 11:06 | |
| *** paladox_ is now known as paladox | 11:06 | |
| *** sfinucan is now known as stephenfin | 11:06 | |
| *** tweining___ is now known as tweining | 11:08 | |
| *** clarkb is now known as Guest1034 | 11:14 | |
| *** pojadhav is now known as pojadhav|afk | 11:14 | |
| *** arne_wiebalck_ is now known as arne_wiebalck | 11:24 | |
| *** dviroel is now known as dviroel|rover | 11:26 | |
| *** pojadhav|afk is now known as pojadhav | 12:35 | |
| *** kopecmartin_ is now known as kopecmartin | 13:10 | |
| mnasiadka_ | good afternoon | 13:21 |
| mnasiadka_ | Would it be possible to get rockylinux-9 nodes with nested virt? just like nested-virt-centos-9-stream? | 13:22 |
| *** dasm|off is now known as dasm | 13:27 | |
| *** soniya29|ruck is now known as soniya29|ruck|afk | 13:27 | |
| *** soniya29|ruck|afk is now known as soniya29|ruck | 13:40 | |
| *** mnasiadka_ is now known as mnasiadka | 14:25 | |
| *** ysandeep is now known as ysandeep|dinner | 14:46 | |
| fungi | mnasiadka: sure, just add it to the nodepool configs in openstack/project-config | 15:05 |
| fungi | in the same providers as nested-virt-centos-9-stream | 15:05 |
| fungi | it's basically identical to the normal label, we just only expose it in the providers where nested virt acceleration is expected to work (for the usual definitions of "work" which apply to nested virt at least) | 15:06 |
| *** pojadhav is now known as pojadhav|out | 15:18 | |
| *** dviroel|rover is now known as dviroel|rover|lunch | 15:24 | |
| opendevreview | Mark Goddard proposed openstack/diskimage-builder master: Fix LVM build when VG exists on build host https://review.opendev.org/c/openstack/diskimage-builder/+/854427 | 15:27 |
| *** ysandeep|dinner is now known as ysandeep | 15:36 | |
| mnasiadka | fungi: thanks, will do | 15:40 |
| opendevreview | Michal Nasiadka proposed openstack/project-config master: Add nested-virt-rockylinux-9 https://review.opendev.org/c/openstack/project-config/+/854431 | 15:47 |
| *** ysandeep is now known as ysandeep|out | 15:58 | |
| fungi | anybody know if the skip_ansible_lint tag has stopped being honored by recent ansiblelint? it's now complaining about a task we set to be skipped back in 2020 | 16:19 |
| *** dviroel|rover|lunch is now known as dviroel|rover | 16:21 | |
| gmann | fungi: can you help me in this problem, it seems some rebasing issue? https://zuul.opendev.org/t/openstack/build/d2b5e173501147a884264cb88611629e | 16:30 |
| gmann | fungi: there is attribute error in this file which is what is am changing in that patch /home/zuul/src/opendev.org/openstack/nova/nova/policies/shelve.py | 16:31 |
| gmann | but it seems old code is taken for this file and new for others | 16:31 |
| gmann | fungi: ah wait, i think it rebase issue from my side. please ignore | 16:32 |
| gmann | second rebase showed the actual diff | 16:33 |
| gmann | it was strange behavior when gerrit diff was not correct in PS4 from what it showed in error | 16:34 |
| gmann | but after rebase again PS5 gerrit diff and test failure were consistent. | 16:35 |
| fungi | gmann: yeah, it's hard to guess what happened locally, but nova/policies/shelve.py seems to have not changed between patchsets 2 through 5 | 16:38 |
| gmann | not sure which side the issue is but gerrit was showing some different code then what zuul used | 16:38 |
| fungi | well, zuul doesn't use what gerrit has directly, it merges the change in gerrit to the current target branch state and then tests the result | 16:39 |
| gmann | fungi: in PS4 it was strange where gerrit showed the changed code and zuul took unchanged one. or may be gerrit showed the old PS changed onw | 16:39 |
| fungi | it's testing your change as it will be merged, rather than as it is displayed by gerrit/locally | 16:39 |
| gmann | yeah, display by gerrit seems issue as in PS5 when i rebased again it displayed correct one | 16:39 |
| *** jpena is now known as jpena|off | 16:42 | |
| gmann | another strange in gerrit display is- 'patchset' drop down box same PS even it is actually PS5 vs PS6. no option in dropdown to select the PS6 :) https://review.opendev.org/c/openstack/nova/+/849209/5..6/nova/policies/shelve.py | 16:43 |
| gmann | *drop down box shows same... | 16:43 |
| fungi | even after refreshing the page? | 16:43 |
| gmann | after refresh it shows correct | 16:44 |
| fungi | the parent of 849209,4 was 848021,2, the parent of 849209,5 and ,6 is 848021,3 though | 16:45 |
| fungi | gmann: so for patchset 4 gerrit was showing you the change relative to the 848021,2 parent but with patchset 5 you updated it to be relative to 848021,3 | 16:49 |
| gmann | fungi: got it. both are correct seems. there is another change merged in between and my patch child of 848021 was not rebased on latest master | 16:49 |
| gmann | yeah, gerrit showed before rebase things and zuul did rebase on master and then tested | 16:49 |
| gmann | and because there was no conflict in code file so gerrit did not complain about merge conflict. | 16:50 |
| fungi | right | 16:51 |
| fungi | and zuul was also able to cleanly merge it to the branch, so it did not complain about merge conflicts either | 16:52 |
| gmann | yeah | 16:52 |
| vsevolod | Hi. Sorry if this is the wrong place to ask. Point me to proper one if it is. | 16:53 |
| vsevolod | I have got https://review.opendev.org/SignInFailure,SIGN_IN,Contact+site+administrator | 16:53 |
| vsevolod | when trying to login to review.opendev.org | 16:53 |
| vsevolod | How do I contact site administrator? | 16:54 |
| fungi | vsevolod: you just did | 16:54 |
| fungi | how are you trying to sign in? | 16:54 |
| vsevolod | via my ubuntu one account | 16:54 |
| vsevolod | at https://login.ubuntu.com/ | 16:54 |
| vsevolod | "You are logging in to https://review.opendev.org/" | 16:55 |
| fungi | i probably won't be able to look at it until i'm done cooking lunch, but i'll check the logs on the server side. it's possible you've got an account conflict we need to clean up | 16:55 |
| fungi | when was the last time you successfully logged in (if ever), and did you make any recent changes to your ubuntuone account? | 16:55 |
| vsevolod | I just created this account | 16:57 |
| vsevolod | But I also have another one and used it on this site. | 16:57 |
| vsevolod | It have same full name | 16:57 |
| vsevolod | May be that's the reason | 16:57 |
| fungi | vsevolod: if the two accounts used the same e-mail address at some point, that will be the conflict. gerrit gets very picky about non-unique e-mail addresses, so we may find that the old account has an address (possibly a secondary address) which is the same as the address used for the new account | 17:00 |
| vsevolod | no, they did not use same email. Not as I can remember. | 17:00 |
| fungi | vsevolod: okay, so to recap, it looks like you had an ubuntuone account using that address, logged into gerrit with it, deleted the ubuntuone account, created a new ubuntuone account with the same e-mail address, and are trying to log into gerrit with that? just making sure i have the order of operations straight since we're sort of keeping a list of ways people can break openid logins for | 17:31 |
| fungi | gerrit | 17:31 |
| fungi | helps to more quickly identify those same sorts of login problems when we see them in the future | 17:32 |
| fungi | looks like the sequence we've used to fix this in the past is to first run the remove-user-external-ids.py script and then the retire-user.sh script here: https://opendev.org/opendev/system-config/src/branch/master/tools/gerrit-account-inconsistencies | 17:36 |
| vsevolod | fungi: yes, looks like this. | 17:41 |
| fungi | thanks for confirming | 17:42 |
| fungi | aha, i had the sequence for the cleanup backwards. need to use the retire script first to set the account inactive, then use the external ids cleanup script to remove traces of the old addresses | 17:45 |
| fungi | #status log Retired and cleaned external refs from abandoned Gerrit account 34566 at the owner's request | 17:54 |
| fungi | mmm, statusbot appears to be on vacation, i'll work on that next | 17:54 |
| fungi | vsevolod: i think it should be cleaned up now. the next time you get an opportunity to try logging in again, let me or others in this channel know how it goes for you. it's possible that conflict could have been hiding another one | 17:55 |
| fungi | 2022-08-24 11:06:27 <-- opendevstatus (~opendevst@eavesdrop01.opendev.org) has quit (synthon.oftc.net larich.oftc.net) | 17:56 |
| fungi | looks like it never came back after that, the running process doesn't seem to have gotten the memo that it's no longer connected to an irc server | 17:57 |
| fungi | its debug log just basically went silent around then because from its perspective, it stopped seeing anything in irc | 17:58 |
| fungi | #status log Restarted the statusbot container on eavesdrop01 after it fell off the network around 11:00 UTC | 18:00 |
| opendevstatus | fungi: finished logging | 18:02 |
| fungi | #status log Retired and cleaned external refs from abandoned Gerrit account 34566 at the owner's request | 18:02 |
| opendevstatus | fungi: finished logging | 18:02 |
| fungi | okay, back to figuring out why ansiblelint has suddenly started to not ignore things it's been told to ignore | 18:12 |
| fungi | looks like ansible-lint 6.5.0 happened on monday | 18:15 |
| fungi | project-config is capping ansible-lint<7 | 18:15 |
| fungi | https://github.com/ansible/ansible-lint/issues/2320 "skip_ansible_lint tag ignored by ansible-lint 6.5.0" | 18:19 |
| fungi | looks like sbarnea expects https://github.com/ansible/ansible-lint/pull/2300 to (eventually) fix it. in the meantime i'll push a change to blacklist that release | 18:21 |
| vsevolod | fungi: Yes, all good now - I was able to login | 18:26 |
| vsevolod | fungi: thank you! | 18:26 |
| opendevreview | Jeremy Stanley proposed openstack/project-config master: Don't use ansible-lint 6.5.0 https://review.opendev.org/c/openstack/project-config/+/854445 | 18:27 |
| fungi | that should address it. i'll self-approve if it passes and then recheck the other things we've been unable to merge | 18:27 |
| cfriesen | hey all...we had a new repo created on review.opendev.org, but the corresponding "members" list is empty. What's the procedure for setting the initial repo cores? | 18:32 |
| cfriesen | I would have expected the person requesting repo creation to be set as the initial core member. | 18:33 |
| cfriesen | repo is https://review.opendev.org/admin/repos/starlingx/app-kubevirt,general | 18:34 |
| fungi | cfriesen: any newly created groups start out empty, but we generally add the project liaison to the group on request once it exists. in the case of starlingx repos that's typically scott little | 18:35 |
| cfriesen | that works. could you add Scott to https://review.opendev.org/admin/groups/c57b96b5346892daa6ff5ea445ac4d8fb3dcc4aa then? | 18:36 |
| fungi | you bet, just a moment | 18:36 |
| fungi | cfriesen: done | 18:38 |
| cfriesen | thx | 18:39 |
| opendevreview | Merged openstack/project-config master: Don't use ansible-lint 6.5.0 https://review.opendev.org/c/openstack/project-config/+/854445 | 18:48 |
| opendevreview | Merged openstack/project-config master: nodepool: add rocky-9 stub https://review.opendev.org/c/openstack/project-config/+/853894 | 19:06 |
| opendevreview | Merged openstack/project-config master: Retire openstack-helm-addons repository https://review.opendev.org/c/openstack/project-config/+/851858 | 19:30 |
| opendevreview | Merged openstack/project-config master: Formally retire puppet-tuskar https://review.opendev.org/c/openstack/project-config/+/834699 | 19:39 |
| opendevreview | Jeremy Stanley proposed openstack/project-config master: Add 16 vcpu flavors https://review.opendev.org/c/openstack/project-config/+/844116 | 19:54 |
| *** dasm is now known as dasm|off | 21:32 | |
| ianw_ | fungi: thanks, we needed a similar cap in zuul-jobs | 22:25 |
| *** dviroel|rover is now known as dviroel|rover|brb | 22:30 | |
| *** dviroel|rover|brb is now known as dviroel|rover | 23:26 | |
| fungi | aha, i couldn't remember where i'd seen something else fly by this week mentioning ansible-lint | 23:52 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!