Thursday, 2024-05-16

*** sfinucan is now known as stephenfin11:35
*** sfinucan is now known as stephenfin14:00
clarkbelodilles: following up on yesterday did those errosr go away?16:49
fricklerclarkb: looks like it, e.g. https://review.opendev.org/c/openstack/octavia/+/918665 was just merged17:03
clarkboh heh I should've checked the merge timestamp and not just that it merged17:04
fricklerinfra-root: (will keep using that highlight since I got no feedback to my comment yesterday) lots of reviews needing just a second look, I'm not comfortable single-approving all of those (mostly talking about those updated yesterday or today) https://review.opendev.org/q/project:openstack/project-config+status:open+label:code-review%3D%252B217:05
tonybfrickler: I did look at a bunch of them but they seemed to ultimately depends-on governance changes which also had no reviews and IIUC are at least 2 weeks away from approval so I dropped them down my to-do list18:41
fungii haven't revisited, but i think i saw chatter in #openstack-tc that the depends-on was in error and gmann may have pushed revisions fixing that now?18:41
tonybfungi: Oh could be.  I'll look again18:42
fungibut yeah, i stopped after approving the first phase of that batch of retirement changes for similar reasons18:43
frickleriiuc all the related governance changes are merged, they were considered to be elegible for fast-approval as project updates18:45
fungioh, also good ;)18:45
frickleralso maybe have another look at https://review.opendev.org/c/openstack/project-config/+/800442 , no response to my concern in 6 months, would it be ok to abandon that one with a comment to restore it if needed?18:47
fungifrickler: yeah, the proposal started almost 3 years ago, i'd say just abandon it suggesting they restore it when they want to revisit, if ignoring it isn't an option18:51
fricklerit showed up in the query I posted earlier, but yeah, I can also ignore it until next time at least19:00
elodillesclarkb: yes, all gone, your patch solved the issue \o/ i started backporting and merging them: https://review.opendev.org/q/I28a39b31f71153a602d41cefe621a216d09a290f19:35
clarkbcool thank you for confirming19:35
elodillesthanks, too, for the fix!19:37
opendevreviewMerged openstack/project-config master: Add editHashtags permissions to some governance repositories  https://review.opendev.org/c/openstack/project-config/+/91962719:44
opendevreviewMerged openstack/project-config master: Retire puppet-panko completely  https://review.opendev.org/c/openstack/project-config/+/91941419:46
opendevreviewMerged openstack/project-config master: Retire Solum: remove project from infra  https://review.opendev.org/c/openstack/project-config/+/91922319:46
opendevreviewMerged openstack/project-config master: Retire Murano: remove project from infra  https://review.opendev.org/c/openstack/project-config/+/91937119:46
opendevreviewGhanshyam proposed openstack/project-config master: Retire Senlin: remove project from infra  https://review.opendev.org/c/openstack/project-config/+/91935523:29
opendevreviewGhanshyam proposed openstack/project-config master: Retire ec2-api: remove project from infra  https://review.opendev.org/c/openstack/project-config/+/91939723:34
opendevreviewGhanshyam proposed openstack/project-config master: Retire Sahara: remove project from infra  https://review.opendev.org/c/openstack/project-config/+/91939123:40
gmannfrickler: clarkb: due to merge conflict, I rebased them ^^  again, please review https://review.opendev.org/c/openstack/project-config/+/919391/   https://review.opendev.org/c/openstack/project-config/+/919397/  https://review.opendev.org/c/openstack/project-config/+/919355/23:43

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