| opendevreview | Merged openstack/project-config master: Add the cinder-lvm charm to Openstack charms https://review.opendev.org/c/openstack/project-config/+/807206 | 01:16 |
|---|---|---|
| opendevreview | OpenStack Proposal Bot proposed openstack/project-config master: Normalize projects.yaml https://review.opendev.org/c/openstack/project-config/+/808148 | 06:14 |
| opendevreview | Brin Zhang proposed openstack/project-config master: Create openstack/*venus* repository https://review.opendev.org/c/openstack/project-config/+/808149 | 06:33 |
| *** jpena|off is now known as jpena | 06:52 | |
| opendevreview | Jiri Podivin proposed zuul/zuul-jobs master: DNM https://review.opendev.org/c/zuul/zuul-jobs/+/807031 | 08:14 |
| opendevreview | Merged openstack/project-config master: Normalize projects.yaml https://review.opendev.org/c/openstack/project-config/+/808148 | 08:36 |
| opendevreview | Brin Zhang proposed openstack/project-config master: reate openstack/*venus* repository https://review.opendev.org/c/openstack/project-config/+/808149 | 09:33 |
| opendevreview | Aurelien Lourot proposed openstack/project-config master: Fix missing charm-interface-ironic-api https://review.opendev.org/c/openstack/project-config/+/808190 | 09:36 |
| opendevreview | Dmitry Tantsur proposed openstack/diskimage-builder master: Fix debian-minimal security repos https://review.opendev.org/c/openstack/diskimage-builder/+/806188 | 09:36 |
| *** elodilles_pto is now known as elodilles | 09:37 | |
| opendevreview | Dmitry Tantsur proposed openstack/diskimage-builder master: Fix debian-minimal security repos https://review.opendev.org/c/openstack/diskimage-builder/+/806188 | 09:41 |
| dtantsur | hi folks! how is nodepool building images for openSUSE 15.2? DIB doesn't work with it. | 10:44 |
| *** dviroel|afk is now known as dviroel | 11:09 | |
| frickler | dtantsur: well then likely our images are broken, let me try to look for logs | 11:26 |
| dtantsur | frickler: this is what I hit: https://zuul.opendev.org/t/openstack/build/4095f54607fd4d029fe430443eca12d3/log/job-output.txt#1009 | 11:26 |
| dtantsur | looks like they don't publish the artefact we expect | 11:26 |
| *** jpena is now known as jpena|lunch | 11:26 | |
| dtantsur | but I'm quite sure we have 15.2 nodes in nodepool, hence my question | 11:27 |
| frickler | dtantsur: from the logs at nb0[12].opendev.org it looks like we stopped buiding tumbleweed 3 months ago. no trace of 15.2 there at all | 11:28 |
| frickler | dtantsur: https://download.opensuse.org/repositories/Cloud:/Images:/Leap_15.2/images/ isn't something we'd ever published anything to I'd say. looks like opensuse stopped publishing those rootfs tars with 15.2 | 11:31 |
| frickler | dtantsur: a quick check in one of our cloud region seems to indicate the latest opensuse-15 was built less than two weeks ago, but I'm not sure yet where that did happen | 11:37 |
| frickler | hmm, possibly someone did that manually? https://opendev.org/openstack/project-config/src/branch/master/nodepool/nodepool.yaml#L293 | 11:39 |
| frickler | ianw might be able to tell more after their weekend | 11:39 |
| dtantsur | ookay, I see, thank you frickler | 11:41 |
| *** jpena|lunch is now known as jpena | 12:19 | |
| opendevreview | chandan kumar proposed openstack/diskimage-builder master: Correct path for CentOS 9 stream base image https://review.opendev.org/c/openstack/diskimage-builder/+/806819 | 12:23 |
| opendevreview | Sorin Sbârnea proposed zuul/zuul-jobs master: Fix historical ansible-lint errors https://review.opendev.org/c/zuul/zuul-jobs/+/807877 | 12:28 |
| opendevreview | Sorin Sbârnea proposed zuul/zuul-jobs master: Fix historical ansible-lint errors https://review.opendev.org/c/zuul/zuul-jobs/+/807877 | 12:28 |
| opendevreview | Sorin Sbârnea proposed zuul/zuul-jobs master: ensure-tox: avoid using unreliable versions of tox https://review.opendev.org/c/zuul/zuul-jobs/+/807867 | 12:44 |
| *** artom_ is now known as artom | 14:06 | |
| opendevreview | Merged openstack/diskimage-builder master: simple-init: support installing Glean from packages https://review.opendev.org/c/openstack/diskimage-builder/+/782299 | 14:54 |
| opendevreview | Merged openstack/diskimage-builder master: simple-init: allow disabling DHCP fallback https://review.opendev.org/c/openstack/diskimage-builder/+/781491 | 14:54 |
| opendevreview | Merged openstack/diskimage-builder master: Check and remove existing image interface configurations https://review.opendev.org/c/openstack/diskimage-builder/+/775762 | 15:01 |
| *** jpena is now known as jpena|off | 15:47 | |
| *** marios is now known as marios|out | 16:14 | |
| opendevreview | Ghanshyam proposed openstack/project-config master: Rename inspur/*venus* project to openstack/*venus* https://review.opendev.org/c/openstack/project-config/+/808149 | 16:22 |
| opendevreview | Ghanshyam proposed openstack/project-config master: Rename inspur/*venus* project to openstack/*venus* https://review.opendev.org/c/openstack/project-config/+/808149 | 16:37 |
| opendevreview | Ghanshyam proposed openstack/project-config master: Rename inspur/*venus* project to openstack/*venus* https://review.opendev.org/c/openstack/project-config/+/808149 | 16:51 |
| clarkb | gmann: ^ it is probably a good idea to have the project itself drive those changes that way they know when to expect the repo name to change | 16:52 |
| gmann | clarkb: sure, they asked for help as they are not aware of renaming stuff. they are in sync with the changes. | 16:53 |
| gmann | I am just updating their patch to get it correct | 16:53 |
| fungi | clarkb: also we discussed this during the last rename, do we want to change the process to separate the zuul.d/projects.yaml change from the gerrit/projects.yaml change so the latter is testable? | 16:54 |
| clarkb | fungi: yes, but I think we realiezd that doing that was tricky for some reason I'm forgetting now | 16:55 |
| fungi | basically we get no linting of the gerrit config because zuul bails early with its config errors | 16:55 |
| fungi | this would probably be a good time to also mix in renaming all the osf/ namespace repos to openinfra/ | 16:59 |
| fungi | i'll try to put together a change for that | 16:59 |
| *** odyssey4me is now known as Guest6896 | 17:10 | |
| opendevreview | Jeremy Stanley proposed openstack/project-config master: Move osf/ namespace repos to openinfra/ namespace https://review.opendev.org/c/openstack/project-config/+/808479 | 17:12 |
| opendevreview | Jeremy Stanley proposed opendev/system-config master: Adjust RefStack build for osf->openinfra rename https://review.opendev.org/c/opendev/system-config/+/808480 | 17:18 |
| clarkb | fungi: have you had a chance to look at the notes I wrote in the comments of that gerrit user audit yaml file? | 17:35 |
| clarkb | curious if anything jumped out at you, maybe better approaches to propose to users etc | 17:36 |
| opendevreview | Sorin Sbârnea proposed zuul/zuul-jobs master: Include tox_extra_args in tox siblings tasks https://review.opendev.org/c/zuul/zuul-jobs/+/806612 | 17:36 |
| opendevreview | Sorin Sbârnea proposed zuul/zuul-jobs master: Include tox_extra_args in tox siblings tasks https://review.opendev.org/c/zuul/zuul-jobs/+/806612 | 17:50 |
| fungi | clarkb: still looking through the entries but so far the comments in each one look like good plans | 18:35 |
| fungi | clarkb: oh, okay starting over, i'm not so sure about the first one in the list... looks like the older account also has more recent changes and reviews, but you're proposing to retire it. when you say "the older openid is no longer valid" what is that evaluation based on/how was it tested? | 18:52 |
| clarkb | fungi: ubuntu one returns a 404 when GETing or HEADing the url | 18:53 |
| clarkb | it isn't valid in the identity provider | 18:53 |
| fungi | aha, perfect thanks | 18:53 |
| fungi | in that case i think the plan there is fine after all ;) | 18:54 |
| fungi | oh, i see now, the invalid_openids field | 18:54 |
| fungi | that's a little clearer | 18:54 |
| clarkb | ya it is recorded there and that is determined by HEADing the url in the audit script | 18:54 |
| fungi | clarkb: the 4th one (line 104), for situations like that we probably need to remind them to change the username they're pushing changes with and make sure the ssh keys are correct | 19:00 |
| fungi | but yeah it's possible they've been pushing from two different machines and so winding up with two different accounts | 19:01 |
| fungi | or i suppose they could be pushing with a mix of ssh and http | 19:02 |
| fungi | and on the 5th one (line 140) i find it interesting that neither account has pushed a change nor reviewed one. in situations like that, if we don't hear back from the user it's probably safe to just go ahead and retire whichever is older. in general i think we need a fallback plan for when we don't get any reply because i have a feeling that's going to be a majority of these | 19:06 |
| Clark[m] | Good point and ya I think if we get no response after a couple of weeks we proceed with a reasonable approach. Once we get these conflicts addressed we can do one off fixes easily which could include fixing those accounts in the future if the users show up | 19:11 |
| fungi | absolutely | 19:11 |
| fungi | Clark[m]: on the entry at line 451 (i've lost count, but that line number is a great bradbury reference), you say to merge the older id into the newer one, and ask the user to confirm that they see the newer id in the webui, but it looks like the newer id has never reviewed anything while the older id has reviewed a change more recently than the creation of the new id. is the recommendation to | 19:16 |
| fungi | ask them for after we merge the accounts or before? | 19:16 |
| fungi | basically it looks to me like a case of they used the newer account to push changes and the older account to leave reviews in the webui | 19:17 |
| fungi | i agree with the suggestion of merging the old id into the new account, but suspect before we merge they'll say the webui gives them the old number not the new one | 19:19 |
| fungi | Clark[m]: for the entry at line 653, the comment says "Ask the user to confirm is the account they get through web ui login." (no account number mentioned, but i expect you meant to have one there, likely the older one) | 19:28 |
| fungi | Clark[m]: okay, got through the list in one go this time and a better understanding of the basis for the recommendations. i'm good with all of them, just expect we'll need fallbacks for a bunch when their users are unresponsive | 19:36 |
| Clark[m] | The idea was to propose and ask those questions of all the users in the initial email | 19:52 |
| Clark[m] | And then use the info they give us to double check the proposed fixup still makes sense. And they may request the opposite which is probably fine in most cases. We just have to kill one account or the other. | 19:53 |
| fungi | yep | 19:53 |
| *** dviroel is now known as dviroel|out | 21:40 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!