| dansmith | I seriously doubt it's still needed.. can you tell who created it or pushed to it? | 13:38 |
|---|---|---|
| opendevreview | Bhavana S proposed openstack/glance_store master: Handle 409 Conflict on Swift container delete in multi-tenant store https://review.opendev.org/c/openstack/glance_store/+/993755 | 13:41 |
| frickler | dansmith: good question, turns out it was actually done via release automation https://review.opendev.org/c/openstack/releases/+/557480 , I'll check with the release-team whether we also have a process to shut these down | 13:49 |
| dansmith | hrm, okay | 13:50 |
| frickler | jokke isn't around anymore I think? | 13:50 |
| dansmith | pretty sure it wasn't him | 13:52 |
| dansmith | seven years ago is a little before my time, but abhishek should know when he's around.. we've done a major policy refactor since then though which is why I suspect it's stale | 13:52 |
| stephenfin | frickler: in case it's helpful, cinder used to have `feature/*` branches and I recall seeing configuration in project-config (?) to handle them differently (skip CI jobs or something like that). smcginnis was the main user of the branches there. We deleted the remaining cinder ones about a year ago | 14:27 |
| stephenfin | and now the the thing I actually came here for, croelandt I've acked both https://review.opendev.org/c/openstack/openstacksdk/+/969839 and https://review.opendev.org/c/openstack/openstacksdk/+/883271 on the assumption that you and Pranali don't want to re-review each other's work. If you do though, lemme know and I'll drag it out of the queue | 14:28 |
| opendevreview | huangjs3 proposed openstack/glance master: db: forward delete_time in image_location_delete https://review.opendev.org/c/openstack/glance/+/993774 | 15:09 |
| dansmith | abhishekk: see the conversation above about the old policy branch? | 15:22 |
| abhishekk | dansmith: i need to gon through chat logs, will look asap | 15:35 |
| dansmith | abhishekk: | 15:35 |
| dansmith | [13:16:21] <frickler> does anyone know whether the branch feature/policy-refactor is still needed? it is blocking some job cleanup https://review.opendev.org/c/openstack/openstack-zuul-jobs/+/993669 | 15:35 |
| dansmith | [13:17:28] <frickler> seems it wasn't touched for 7 years, so I would assume it could be deleted | 15:35 |
| abhishekk | it's not needed | 15:36 |
| dansmith | frickler: ^ | 15:36 |
| croelandt | I had no idea it was even there | 15:38 |
| opendevreview | Cyril Roelandt proposed openstack/glance master: Add support for downloading images from preferred stores https://review.opendev.org/c/openstack/glance/+/967878 | 15:49 |
| opendevreview | Merged openstack/glance master: Fix flaky import task failure waits in functional tests https://review.opendev.org/c/openstack/glance/+/993644 | 15:53 |
| frickler | thx for confirming. I also checked (and likely should have done that before), that the branch only contains the bot-generated .gitreview update plus the opendev migration patch. so I'll go ahead and delete it without further ado. thx for the feedback | 16:02 |
| abhishekk | ack | 16:03 |
| opendevreview | Merged openstack/glance master: Add support for downloading images from preferred stores https://review.opendev.org/c/openstack/glance/+/967878 | 20:08 |
| opendevreview | Ghanshyam Maan proposed openstack/glance-tempest-plugin master: Use the new variable to enable the RBAC new defaults https://review.opendev.org/c/openstack/glance-tempest-plugin/+/993834 | 20:22 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!