Tuesday, 2023-11-07

clarkblast call ofr meeting agenda items. I'll get that sent out shortly00:04
tonybnothing from me.00:08
fricklerJayF: just noting that if the reply you were talking about is https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/message/YW2IRRVM2T5WMLRNNM7BJTDRUAKVEKWV/ then it didn't go to the OP, so if they aren't subscribed they won't see this05:59
*** maxh1 is now known as maxh09:49
*** gboutry[m] is now known as gboutry11:30
*** maxh1 is now known as maxh11:40
opendevreviewDmitriy Rabotyagov proposed zuul/zuul-jobs master: Add role for uploading Ansible collections to Galaxy  https://review.opendev.org/c/zuul/zuul-jobs/+/89923012:48
JayFfrickler: ... of course it didn't, /o\ thank you, I'll send it directly15:15
yadneshHey folks, is this mirror working https://opendev.org/openstack/tripleo-quickstart-extras/src/branch/master/roles/nodepool-setup/templates/mirror_info.sh.j2#L17 ? 15:27
yadneshIf not, does anyone know another opendev mirror to download fedora img ?15:28
fungiyadnesh: can you elaborate on what you're trying to do? i think the jobs using that script rely on an ansible task that sets the NODEPOOL_MIRROR_HOST envvar16:03
fungiso that script won't work outside the expected job context16:04
clarkband that mirror was shutdown and removed16:04
clarkbthe default one on that line I mean16:04
fungithe fallback entry there? right it's no longer valid and hasn't been for a very long time16:04
fungibut also, if the question is really about downloading fedora packages, we haven't mirrored those for months16:05
fungithe directories under https://static.opendev.org/mirror/fedora/ are essentially empty16:06
slittlehttps://review.opendev.org/c/starlingx/tools/+/894827 seems to be stuck16:15
fungislittle: its parent was revised but it was never rebased onto the updated parent, which then merged. basically the parent commit for 894827 does not exist on the branch so it can never merge in its present state16:16
fungii wish gerrit's webui made that more obvious, but the reddish "(Merged)" next to the parent in the relation chain is supposed to indicate that, i think16:17
clarkbGerrit tries to expose this in two ways (both of which are probably too subtle). If you hover the (i) icon next to the parent sha it says the parent is outdated and the (merged) text is orange next to the parent in the relation chain16:17
slittleI would have expected something like 'merge conflict'16:18
fungiyeah, if you click the "show all" down-arrow next to the change info in the top right, it'll show the parent and you'll see the "i" in a circle next to it16:18
fungislittle: it's technically not a merge conflict16:18
clarkbmore of a dead end16:19
fungiit's essentially detached commit with no parent in the branch16:19
fungiyou can probably just click the "rebase" button in the top-right corner to get it testing again16:19
fungiunless rebasing it requires some manual merge conflict resolution16:20
fricklerkopecmartin: do you still want to pursue https://review.opendev.org/c/opendev/system-config/+/778090 ? (would need a rebase, best on top of https://review.opendev.org/c/opendev/system-config/+/900220 )16:31
kopecmartinfrickler: no, I'll abandon it16:32
clarkbseems like system-config-run-refstack has done a good job of testing things for us16:32
*** d34dh0r5- is now known as d34dh0r5317:03
opendevreviewDmitriy Rabotyagov proposed zuul/zuul-jobs master: Add role for uploading Ansible collections to Galaxy  https://review.opendev.org/c/zuul/zuul-jobs/+/89923017:37
clarkb900220 is approved19:39
clarkbtonyb (and other interested folks) for getting together to help bootstrap things I've got some early morning school stuff tomorrow that should end by 10am (1800 UTC) local time. And then I think I can be free until lets say 2pm (2200 UTC)19:40
clarkbeventually I'll need to go outside and get some exercise since it is supposed to stop raining19:40
clarkbAnd then my thursday is basically wide open. Do any of those time work for you?19:41
clarkbI'm noticing that 900220 didn't run system-config-run-base19:41
tonybYup I think they all do.19:41
clarkbits probably fine but do we want to unapprove it and add that to the file list and make sure that is happy and reapprove?19:42
tonybLets aim for 1800UTC19:42
fungii can help any time through the end of the week as well19:42
clarkbtonyb: sounds good 1800 UTC tomorrow19:42
tonybGreat.19:43
fungiclarkb: probably a good idea to slip an extra change in for that ahead of 90022019:43
clarkbyup I just removed my approval19:43
clarkbbetter to fix this now and be confident than forget and have problems later19:43
tonybI can make add a new change and then rebase 90022019:44
clarkbtonyb: thanks. Its just the file list for that job missing an inventory/ entry 19:44
clarkband we have plenty of time to land that before 1800 UTC so may as well19:45
clarkbtonyb: I tguess for completness we want to do the same edit for the arm64 and ansible devel jobs as well19:45
tonybWill do19:46
tonybI've added '  - inventory/' to system-config-run-base and system-config-run-base-arm64 but system-config-run-base-ansible-devel already has system-config-run-base as a parent so I don't think it's needed there19:50
tonybDoes that sound right?19:51
fungisounds right to me. also system-config-run-base-ansible-devel is non-voting and has been long broken, so not the end of the world if it gets missed on a change19:53
opendevreviewTony Breeds proposed opendev/system-config master: Add Tony Breeds to base_users for all hosts  https://review.opendev.org/c/opendev/system-config/+/90022020:00
opendevreviewTony Breeds proposed opendev/system-config master: Add the inventory dir to files for system-config-run-base{,-arm64}  https://review.opendev.org/c/opendev/system-config/+/90036720:00
opendevreviewMerged opendev/system-config master: Add the inventory dir to files for system-config-run-base{,-arm64}  https://review.opendev.org/c/opendev/system-config/+/90036720:25
fungilooks like that worked, system-config-run-base, system-config-run-base-ansible-devel, and system-config-run-base-arm64 all ran on patchset 2 of 90022020:36
Clark[m]Thanks. I can reapprove shortly. Just finishing up lunch20:39
tonybNice20:42
clarkband the logs in the jobs confirm the user was created (which was the amin reason for ensuring that job ran)20:50
clarkbI've reapproved it20:50
tonyb#MuchExcite21:02
opendevreviewMerged opendev/system-config master: Add Tony Breeds to base_users for all hosts  https://review.opendev.org/c/opendev/system-config/+/90022021:19
*** dhill is now known as Guest619121:20
clarkbnow because that modified the all.yaml inventory file all the jobs are queued up but the first one is the one that does what we need21:20
tonybNice21:21
opendevreviewDmitriy Rabotyagov proposed zuul/zuul-jobs master: Add role for uploading Ansible collections to Galaxy  https://review.opendev.org/c/zuul/zuul-jobs/+/89923021:26
clarkblooks like the base job ran and did what we needed it to21:27
clarkbtonyb: so ya lets jump in meetpad at 1800 UTC tomorrow and take it from there21:28
tonybSounds good.21:28
fungiit's on my calendar21:28
tonybFancy21:29
fungithat's just so that i get a reminder since i tend to get distracted by other work21:29
tonybfair enough21:30
clarkbwe can use https://meetpad.opendev.org/opendev-root-bootstrap I'm taking some notes in the associated etherpad22:19
tonybAweome.22:34
corvusi'm restarting nodepool-launcher on nl0223:42

Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!