kozhukalov | The problem is that I can not re-run the promote job. The change is merged but images are not published. So need to suggest another change and ask someone to review and then we merge and try to promote again. I would like to try to run the job and debug it not disturbing others from the team and for this I need to be able to run post-review jobs which can access to secrets. Jobs in the check pipeline are not allowed to use | 00:28 |
---|---|---|
kozhukalov | secrets | 00:28 |
clarkb | kozhukalov: the problem is the promote jobs are not meant to run pre merge. They will promote unmerged code if you do so | 00:48 |
clarkb | I'm wary of using this method to debug as a result | 00:48 |
clarkb | can you land a noop change instead (add a comment to a dockerfile?) and force things to rebuild and repromote and see if the problem persists? | 00:48 |
clarkb | as an alternative we may be able to reenqueue the buildset, but we'd need to check that is safe | 00:49 |
kozhukalov | yes, I agree, I did what you have just suggested | 00:49 |
kozhukalov | trivial change to kick the promotion | 00:49 |
opendevreview | OpenStack Proposal Bot proposed openstack/project-config master: Normalize projects.yaml https://review.opendev.org/c/openstack/project-config/+/897273 | 02:14 |
opendevreview | Merged openstack/project-config master: Revert "Temporarily remove release docs semaphores" https://review.opendev.org/c/openstack/project-config/+/896949 | 02:33 |
opendevreview | Merged openstack/project-config master: Add nested-virt-debian-bookworm https://review.opendev.org/c/openstack/project-config/+/897331 | 02:36 |
frickler | seems this repo retirement has stalled and was neglected for 2y then :( https://review.opendev.org/c/openstack/project-config/+/798529 | 02:51 |
frickler | clarkb: similar for https://review.opendev.org/c/opendev/puppet-openstack_health/+/836711 where your -1 looks outdated? | 02:54 |
opendevreview | Merged openstack/project-config master: Clean up vestigial default-branch https://review.opendev.org/c/openstack/project-config/+/896595 | 07:50 |
ykarel | frickler, hi | 08:01 |
ykarel | please do release the autohold 0000000007 | 08:02 |
frickler | ykarel: done, thx for the update | 08:40 |
fungi | frickler: looks like 798529 depends on 798503 which needs to merge first, and requires at least one update before it's ready. yoctozepto or anyone in openstacksdk-core seems to have approval rights for that repo according to https://review.opendev.org/admin/groups/js-openstack-lib-core,members | 12:35 |
*** blarnath is now known as d34dh0r53 | 13:49 | |
simondodsley | why did granade-skip-level test become voting in the last few days. It never used to pass as non-voting and is now causing every cinder job to fail | 14:59 |
frickler | simondodsley: because of SLURP I'd assume, that's mandatory now, so I guess cinder has some work to do | 15:11 |
frickler | but also I think grenade needs some patches in order to test the right thing first, maybe we merged things in the wrong order | 15:13 |
fungi | simondodsley: frickler: yes, 2024.1 (caracal) will be a slurp release, so users need to be able to safely upgrade directly from 2023.1 (antelope) | 15:13 |
fungi | worth bringing up in #openstack-qa if the job isn't working as expected | 15:14 |
frickler | https://review.opendev.org/c/openstack/grenade/+/896831 needs to merge before grenade will test the correct from_branch | 15:54 |
opendevreview | Dr. Jens Harbott proposed openstack/project-config master: Retire opendev/puppet-openstack_health: remove project from infa https://review.opendev.org/c/openstack/project-config/+/836712 | 19:25 |
frickler | ricolin: could you please revisit https://review.opendev.org/c/openstack/js-openstack-lib/+/798503 so we can finish this retirement? | 19:47 |
*** cloudnull7 is now known as cloudnull | 23:45 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!