*** sfinucan is now known as stephenfin | 11:35 | |
*** sfinucan is now known as stephenfin | 14:00 | |
clarkb | elodilles: following up on yesterday did those errosr go away? | 16:49 |
---|---|---|
frickler | clarkb: looks like it, e.g. https://review.opendev.org/c/openstack/octavia/+/918665 was just merged | 17:03 |
clarkb | oh heh I should've checked the merge timestamp and not just that it merged | 17:04 |
frickler | infra-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%252B2 | 17:05 |
tonyb | frickler: 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 list | 18:41 |
fungi | i 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 |
tonyb | fungi: Oh could be. I'll look again | 18:42 |
fungi | but yeah, i stopped after approving the first phase of that batch of retirement changes for similar reasons | 18:43 |
frickler | iiuc all the related governance changes are merged, they were considered to be elegible for fast-approval as project updates | 18:45 |
fungi | oh, also good ;) | 18:45 |
frickler | also 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 |
fungi | frickler: 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 option | 18:51 |
frickler | it showed up in the query I posted earlier, but yeah, I can also ignore it until next time at least | 19:00 |
elodilles | clarkb: yes, all gone, your patch solved the issue \o/ i started backporting and merging them: https://review.opendev.org/q/I28a39b31f71153a602d41cefe621a216d09a290f | 19:35 |
clarkb | cool thank you for confirming | 19:35 |
elodilles | thanks, too, for the fix! | 19:37 |
opendevreview | Merged openstack/project-config master: Add editHashtags permissions to some governance repositories https://review.opendev.org/c/openstack/project-config/+/919627 | 19:44 |
opendevreview | Merged openstack/project-config master: Retire puppet-panko completely https://review.opendev.org/c/openstack/project-config/+/919414 | 19:46 |
opendevreview | Merged openstack/project-config master: Retire Solum: remove project from infra https://review.opendev.org/c/openstack/project-config/+/919223 | 19:46 |
opendevreview | Merged openstack/project-config master: Retire Murano: remove project from infra https://review.opendev.org/c/openstack/project-config/+/919371 | 19:46 |
opendevreview | Ghanshyam proposed openstack/project-config master: Retire Senlin: remove project from infra https://review.opendev.org/c/openstack/project-config/+/919355 | 23:29 |
opendevreview | Ghanshyam proposed openstack/project-config master: Retire ec2-api: remove project from infra https://review.opendev.org/c/openstack/project-config/+/919397 | 23:34 |
opendevreview | Ghanshyam proposed openstack/project-config master: Retire Sahara: remove project from infra https://review.opendev.org/c/openstack/project-config/+/919391 | 23:40 |
gmann | frickler: 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/!