| opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: DNM: Test neutron segments fix https://review.opendev.org/c/openstack/python-openstackclient/+/848397 | 09:30 |
|---|---|---|
| frickler | ralonsoh: ^^ I hope my idea to run multiple copies of the job in one patch works | 09:31 |
| ralonsoh | it does, I've done it before | 09:31 |
| stephenfin | diablo_rojo_phone: Sure, I can do that. tl;dr though, I just ran 'cinder --help' with sequential versions of 'OS_VOLUME_API_VERSION' and did a side-by-side comparison. Not particularly clever 😅 | 09:47 |
| stephenfin | gtema: Care to look at https://review.opendev.org/c/openstack/python-openstackclient/+/844268 (and maybe the follow-ups, but they're less important) | 09:48 |
| stephenfin | *? | 09:48 |
| gtema | going to look right now | 09:48 |
| gtema | okay, so upstream Friday can finally go on | 09:49 |
| gtema | we just maybe should fix networking tests first, cause they will bring us to endless rechecks again | 09:50 |
| frickler | gtema: my local testing makes me rather optimistic on ralonsoh's fix in neutron that is being tested in 848397 | 09:55 |
| gtema | yes, that would be awesome | 09:57 |
| gtema | and I see neutron fix is eventually going to land in 1-2 hours | 09:57 |
| opendevreview | Merged openstack/python-openstackclient master: Fix: create image from volume command https://review.opendev.org/c/openstack/python-openstackclient/+/844268 | 11:19 |
| viks__ | hi, i'm using python openstacksdk.. why project object returned by `conn.identity.projects` does not contain custom property key-value pair? | 12:55 |
| opendevreview | Cyril Roelandt proposed openstack/python-openstackclient stable/yoga: Fix: create image from volume command https://review.opendev.org/c/openstack/python-openstackclient/+/848446 | 13:27 |
| *** rcastillo is now known as rcastillo|rover | 13:31 | |
| opendevreview | Cyril Roelandt proposed openstack/python-openstackclient stable/wallaby: openstack image create: honor protection/visibility flags https://review.opendev.org/c/openstack/python-openstackclient/+/848448 | 13:35 |
| opendevreview | Cyril Roelandt proposed openstack/python-openstackclient stable/wallaby: Fix: create image from volume command https://review.opendev.org/c/openstack/python-openstackclient/+/848449 | 13:35 |
| opendevreview | Dr. Jens Harbott proposed openstack/python-openstackclient master: Add support for CRUD operations for QoS minimum packet rate rule https://review.opendev.org/c/openstack/python-openstackclient/+/810559 | 13:55 |
| stephenfin | gtema: Could we include the python-openstackclient-core group in the python-openstackclient-stable-maint group, like we do for openstacksdk? | 14:48 |
| stephenfin | I'd like to be able to review croelandt's changes above ^ | 14:48 |
| gtema | looking | 14:49 |
| stephenfin | You're in the latter, so you should be able to add python-openstackclient-core to Included Groups here https://review.opendev.org/admin/groups/83af131761f12c002cab329fe1a1ea7ffdbaf7cd,members | 14:49 |
| stephenfin | We probably want to drop Steve, Lee and Dean too | 14:50 |
| gtema | I remembered now: only admins can do this. I myself cant change the group | 14:50 |
| stephenfin | gtema: By admins do you mean fungi or clarkb? | 14:51 |
| gtema | yeah, l remember asking fungi to add myself into this group | 14:52 |
| fungi | https://review.opendev.org/admin/groups/python-openstackclient-stable-maint | 14:52 |
| fungi | "owners: stable-maint-core" | 14:53 |
| fungi | https://review.opendev.org/admin/groups/stable-maint-core,members | 14:53 |
| stephenfin | TIL that group owners were a thing in Gerrit. Every day a new adventure | 14:53 |
| fungi | that's the list of people who decide to add/remove anyone from python-openstackclient-stable-maint | 14:54 |
| gtema | hmm, and is this how we want it to be? | 14:54 |
| stephenfin | fungi++ Thanks, I didn't know that | 14:54 |
| fungi | gtema: that's probably something to discuss with the openstack stable maintainers and tc | 14:55 |
| stephenfin | diablo_rojo_phone is in there (via Release Managers). Now is diablo_rojo_phone around? 🤔 | 14:55 |
| fungi | it used to be that the openstack stable maintenance team wanted to be able to confirm people who were added to per-project stable maintenance groups were familiar with the stable backport review policies for openstack before adding them, so the release managers tend to not get involved in that. not sure whether that's still the case | 14:56 |
| stephenfin | gtema: fungi: Yeah, I note that at least octavia-stable-maint and openstacksdk-stable-maint are owned by themselves | 14:56 |
| stephenfin | fungi: I suspect today we'll take stable cores where we can find them :) | 14:57 |
| stephenfin | openstacksdk https://review.opendev.org/admin/groups/1b6ed81aaf8ad4ab52cc54284be7199607dcaa8d | 14:57 |
| fungi | i think it had to do with whether the project asserted stable:follows-policy | 14:57 |
| stephenfin | octavia https://review.opendev.org/admin/groups/e93320d644e6a71c904d06ec6e379efb541f6dd8 | 14:57 |
| stephenfin | oh, aren't those policies gone now? | 14:57 |
| stephenfin | maybe it was another thing to do with badges | 14:57 |
| fungi | right, which is why i don't know what the current approval workflow is, hence my suggestion to check with the tc | 14:58 |
| stephenfin | mistral-stable-maint, neutron-lib-stable-maint, neutron-vpnaas-stable-maint and karbor-stable-maint too. Likely more (those are all I checked) | 14:58 |
| stephenfin | fungi: I'll post to openstack-discuss now | 14:58 |
| fungi | ideally, if stable-maint-core no longer needs to be an oversight body for per-project stable-maint groups, then we'd go through and make all those groups self-owned at the same time rather than dealing with a lot of one-off requests for them | 14:58 |
| johnsom | Octavia asserts stable:follows-policy | 14:59 |
| johnsom | The special group was a hold over from when we were starting the project up from scratch | 14:59 |
| fungi | yeah, far enough back in time, there weren't per-project stable-maint groups and instead the central openstack-stable-maint members handled all backports, then they started delegating some of it to project-specific reviewers as long as they were familiar with and agreed to follow openstack's backport policies | 15:00 |
| johnsom | Yep, that is kind of where we landed | 15:01 |
| fungi | now that there isn't much of a central stable-maint group any longer, i think the idea is to just hand that over to the per-project groups, but it would be nice to have that clearly asserted somewhere and then update all the existing groups to that new model | 15:01 |
| gtema | someone should open discuss thread about it? | 15:03 |
| fungi | stephenfin just said he was going to | 15:03 |
| gtema | oh, right, overseen | 15:04 |
| diablo_rojo_phone | I'm around. What's up? | 15:09 |
| stephenfin | diablo_rojo_phone: ttx beat you to the punch. We were trying to get osc core added to osc stable core and needed a release manager or openstack stable core to do it | 15:10 |
| diablo_rojo_phone | Ohhh got it. Sorry! | 15:10 |
| stephenfin | Took me a while but sent http://lists.openstack.org/pipermail/openstack-discuss/2022-July/029367.html | 15:31 |
| stephenfin | gtema: What do you think about patches like https://review.opendev.org/c/openstack/python-openstackclient/+/848448 ? Technically it's a feature backport. I don't personally mind these for user-facing tool like OSC but wanted to double check before I +2'd | 15:35 |
| gtema | ok for me as well | 15:36 |
| gtema | I feel here pretty similar situation to sdk - we should eventually not having branches for OSC at all. It just should work with any older cloud | 15:37 |
| stephenfin | Agreed | 15:39 |
| opendevreview | Merged openstack/python-openstackclient master: image: Trivial style changes https://review.opendev.org/c/openstack/python-openstackclient/+/848252 | 15:39 |
| opendevreview | Cyril Roelandt proposed openstack/python-openstackclient stable/yoga: Fix: create image from volume command https://review.opendev.org/c/openstack/python-openstackclient/+/848446 | 16:05 |
| opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: The Python 3.6 and Python 3.7 Support has been dropped since zed https://review.opendev.org/c/openstack/python-openstackclient/+/844259 | 16:16 |
| opendevreview | Cyril Roelandt proposed openstack/python-openstackclient stable/wallaby: Fix: create image from volume command https://review.opendev.org/c/openstack/python-openstackclient/+/848449 | 16:23 |
| opendevreview | Merged openstack/python-openstackclient master: image: Make better use of argparse https://review.opendev.org/c/openstack/python-openstackclient/+/848253 | 18:00 |
| opendevreview | Merged openstack/python-openstackclient master: image: Split image creation depending on service https://review.opendev.org/c/openstack/python-openstackclient/+/848254 | 18:01 |
| opendevreview | Merged openstack/python-openstackclient master: Stop testing lower-constraints https://review.opendev.org/c/openstack/python-openstackclient/+/841389 | 18:01 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!