| mrjoshi | Hi gtema, I will add the documentation as a separate patch , could you please have a look at these patches https://review.opendev.org/c/openstack/openstacksdk/+/874584, https://review.opendev.org/c/openstack/openstacksdk/+/874940, https://review.opendev.org/c/openstack/openstacksdk/+/874372 | 07:55 |
|---|---|---|
| jkulik | what would be the next version released for openstacksdk? I'd like to create a review-request that relies on a commit currently only in master | 11:26 |
| frickler | jkulik: with https://review.opendev.org/c/openstack/openstacksdk/+/876134 in (not sure why that wasn't merged yet), it will very likely be 1.1.0, unless there's a reason to go for 2.0.0. but testing unreleased versions may be difficult, are you talking about a review in sdk or some other project? | 12:01 |
| jkulik | frickler: I'm talking about changing python-openstackclient to use a feature from openstacksdk which just got merged yesterday. | 12:02 |
| jkulik | I'll look out for the release of 1.1.0 then and prepare my branch for the review-request already for it. Thank you :) | 12:04 |
| frickler | jkulik: best option would be to assure the code can also work with older sdk, I'm not sure we'd want to immediately bump min reqs to a version that was just released. but maybe gtema has more insight, also in regards on the expected timeframe for a release | 12:06 |
| jkulik | hm ... makes sense. the code in question will start working with that version, but will not break if an older version is present. so I could as well leave bumping the requirements out of the review-request. | 12:09 |
| jkulik | it's a fix for already-not-breaking but also doing-nothing code | 12:10 |
| opendevreview | Merged openstack/openstacksdk master: Update master for stable/2023.1 https://review.opendev.org/c/openstack/openstacksdk/+/876134 | 12:16 |
| frickler | stephenfin: do you want to re-review https://review.opendev.org/c/openstack/python-openstackclient/+/864497 ? would be nice to finally get that in, too | 12:19 |
| opendevreview | Johannes Kulik proposed openstack/python-openstackclient master: Fix --security-group for port list https://review.opendev.org/c/openstack/python-openstackclient/+/877662 | 14:02 |
| jkulik | FYI, ^ is the patch I was talking about earlier | 14:03 |
| *** artom_ is now known as artom | 21:15 | |
| opendevreview | Tim Burke proposed openstack/keystoneauth master: Fix up some packaging metadata https://review.opendev.org/c/openstack/keystoneauth/+/877722 | 23:04 |
| opendevreview | Tim Burke proposed openstack/keystoneauth master: Add project URLs from README as packaging metadata https://review.opendev.org/c/openstack/keystoneauth/+/877723 | 23:04 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!