| lajoskatona | gtema, stephenfin: Hi, if you have some free time could you please review this one for Neutron trunks: https://review.opendev.org/c/openstack/python-openstackclient/+/869447 , thanks in advance | 08:18 |
|---|---|---|
| lajoskatona | gtema, stephenfin: if it is merged we can delete the trunk command related code from neutronclient :-) | 08:19 |
| gtema | great. Looking now | 08:19 |
| opendevreview | Radosław Piliszek proposed openstack/openstacksdk master: support notification vmoves for masakari https://review.opendev.org/c/openstack/openstacksdk/+/872080 | 08:23 |
| opendevreview | Radosław Piliszek proposed openstack/openstacksdk master: support notification vmoves for masakari https://review.opendev.org/c/openstack/openstacksdk/+/872080 | 08:23 |
| opendevreview | Radosław Piliszek proposed openstack/openstacksdk master: support notification vmoves for masakari https://review.opendev.org/c/openstack/openstacksdk/+/872080 | 10:45 |
| stephenfin | gtema: I'm going to propose 6.1.0 of OSC, if that's okay with you? In the interim, can you look at https://review.opendev.org/c/openstack/releases/+/870732 ? | 11:25 |
| gtema | Yes, go for it | 11:27 |
| stephenfin | lajoskatona: That had been on my list. Done! | 11:35 |
| stephenfin | gtema: Actually, before we cut that we might merge https://review.opendev.org/q/project:openstack/python-openstackclient+topic:add-missing-cinderclient-commands for whoami-rajat ? | 11:35 |
| gtema | looking on those | 11:36 |
| whoami-rajat | stephenfin, we've 2 weeks for client release right? or am i missing something | 11:36 |
| stephenfin | Yeah, but we're cycle with intermediary so we can cut effectively whenever so long as we have *a* release per-cycle | 11:37 |
| stephenfin | there are a lot of changes queued up (especially for glance - coverage is looking sooo much better there now) so let's get the bulk of them out before the panic | 11:38 |
| whoami-rajat | ok, I've 3 more commands to add and then cinder will be fully in sync with OSC, at least for the commands part | 11:38 |
| whoami-rajat | so are we planning to release again then with other client releases? | 11:38 |
| stephenfin | Perfect. Ping me if/when you've those up and I'll happily review them. | 11:38 |
| stephenfin | Yeah. We can release whenever it suits us | 11:39 |
| whoami-rajat | cool, I will work on them then, thanks! | 11:39 |
| stephenfin | Let's aim for a 6.2.0 in the next few weeks if you can get cinder coverage closed out | 11:39 |
| whoami-rajat | stephenfin, do you want me to update this? just noticed your comments https://review.opendev.org/c/openstack/python-openstackclient/+/869811 | 11:39 |
| whoami-rajat | or better as a followup? | 11:40 |
| stephenfin | follow-up would be better | 11:40 |
| stephenfin | they're nits at best | 11:40 |
| whoami-rajat | ok, sounds good | 11:41 |
| opendevreview | suzhengwei proposed openstack/openstacksdk master: support notification vmoves for masakari https://review.opendev.org/c/openstack/openstacksdk/+/872080 | 11:58 |
| *** tkajinam is now known as Guest3003 | 12:02 | |
| lajoskatona | stephenfin, gtema: thanks | 12:18 |
| opendevreview | Merged openstack/python-openstackclient master: Move network trunk commands from python-neutronclient https://review.opendev.org/c/openstack/python-openstackclient/+/869447 | 12:41 |
| opendevreview | Merged openstack/python-openstackclient master: Add volume summary command https://review.opendev.org/c/openstack/python-openstackclient/+/869436 | 12:43 |
| odyssey4me | hi folks - I see https://review.opendev.org/q/project:openstack/python-openstackclient+branch:stable/wallaby has ~10 patches after the last tagged release... is there an intent to release a new tag at some point soon? | 12:48 |
| opendevreview | Mridula Joshi proposed openstack/openstacksdk master: Add support for glance cache list https://review.opendev.org/c/openstack/openstacksdk/+/869710 | 13:42 |
| odyssey4me | never mind my query - wallaby is in extended maintenance, so I guess no more releases there | 14:10 |
| stephenfin | odyssey4me: Yup, no more I'm afraid https://review.opendev.org/c/openstack/releases/+/847937 | 14:39 |
| gtema | I would say, now we can have a real party - https://pypi.org/project/openstacksdk/1.0.0/ is there | 14:40 |
| stephenfin | #success openstacksdk 1.0.0 is out 🎉 | 14:44 |
| opendevstatus | stephenfin: Added success to Success page (https://wiki.openstack.org/wiki/Successes) | 14:44 |
| odyssey4me | w00t! | 14:44 |
| stephenfin | gtema: did you merge more patches for the container orchestration layer? | 14:45 |
| gtema | :) | 14:45 |
| gtema | yes | 14:45 |
| gtema | coe is also complete | 14:45 |
| opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Update README to indicate COE resource/proxy support https://review.opendev.org/c/openstack/openstacksdk/+/872283 | 14:48 |
| stephenfin | gtema: Probably want that so ^ | 14:49 |
| stephenfin | it's trivial | 14:49 |
| gtema | ah, there are too many pages not built from sources. This should be target for R2.0 ;-) | 14:49 |
| opendevreview | Mridula Joshi proposed openstack/openstacksdk master: Add support for glance cache list https://review.opendev.org/c/openstack/openstacksdk/+/869710 | 15:15 |
| opendevreview | Merged openstack/openstacksdk master: Update README to indicate COE resource/proxy support https://review.opendev.org/c/openstack/openstacksdk/+/872283 | 15:29 |
| opendevreview | Alfredo Moralejo proposed openstack/python-openstackclient stable/wallaby: Replace assertItemsEqual with assertCountEqual https://review.opendev.org/c/openstack/python-openstackclient/+/872327 | 15:58 |
| timburke | hi! i've got some concerns about the stable/xena openstacksdk-functional-devstack job: https://zuul.opendev.org/t/openstack/builds?job_name=openstacksdk-functional-devstack&branch=stable%2Fxena | 16:46 |
| timburke | it looks like the recently-merged-to-master https://review.opendev.org/c/openstack/openstacksdk/+/865045 is getting picked up on the old stable branch (presumably because of https://github.com/openstack/openstacksdk/blob/master/.zuul.yaml#L34-L42) | 16:48 |
| timburke | which is causing import errors down in python-openstackclient (which *is* using stable/xena) because it picked up an undeclared dependency on munch in https://opendev.org/openstack/python-openstackclient/commit/8b7a2c8d5931f15b69d1ef6baad994f4b9904579 | 16:49 |
| gmann | yeah, I also noticed this in devstack fix on stable/wallaby and openstacksdk-functional-devstack job is failing 100% on <=stable/wallaby https://zuul.openstack.org/builds?job_name=openstacksdk-functional-devstack&branch=stable%2Fxena&branch=stable%2Fwallaby&skip=0 | 17:13 |
| gmann | but If I am not wrong sdk master supposed to work for all supported stable branch ? | 17:14 |
| gtema | gmann, the issue is apparently not the SDK itself, but rather old OSC which doesn't want to work good with new SDK. This looks bit weird for me | 17:23 |
| gtema | installing munch lib there (with 5.6.1) helps to solve this, so most likely we need to extend requirements in old branches | 17:27 |
| gmann | ok, so osc was relying on the munch being installed from sdks requirements.txt and with that removal from sdks it broke osc | 17:29 |
| gmann | we should have that here https://opendev.org/openstack/python-openstackclient/src/branch/master/requirements.txt | 17:30 |
| gtema | yes, apparently | 17:30 |
| gmann | let me push change | 17:30 |
| gtema | but only for xena, this is not required anywhere else | 17:30 |
| gmann | right | 17:34 |
| gtema | and interestingly stephenfin just today made release of OSC for xena, what a facepalm on a R1.0 day | 17:34 |
| gmann | humm it worked there? | 17:35 |
| gtema | well, as usual it all depend on the constellation | 17:35 |
| opendevreview | Ghanshyam proposed openstack/python-openstackclient stable/xena: [stable-only] Add munch in requirements.txt https://review.opendev.org/c/openstack/python-openstackclient/+/872340 | 17:40 |
| gmann | gtema: stephenfin: ^^ | 17:40 |
| gtema | yes, I see. Thanks a lot gmann | 17:41 |
| stephenfin | thanks | 17:41 |
| opendevreview | Ghanshyam proposed openstack/python-openstackclient stable/wallaby: [stable-only] Add munch in requirements.txt https://review.opendev.org/c/openstack/python-openstackclient/+/872341 | 17:44 |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Added workaround for non-determinism in ansible-inventory output https://review.opendev.org/c/openstack/ansible-collections-openstack/+/872344 | 18:51 |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Bump minimum required openstacksdk release to 1.0.0 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/872066 | 18:52 |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Refactored coe_cluster{,_template} modules again https://review.opendev.org/c/openstack/ansible-collections-openstack/+/872039 | 18:52 |
| opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Publish 2.0.0 release https://review.opendev.org/c/openstack/ansible-collections-openstack/+/872077 | 18:52 |
| opendevreview | Arvid Requate proposed openstack/keystoneauth master: New auth plugin v3oidcdeviceauthz https://review.opendev.org/c/openstack/keystoneauth/+/869876 | 19:42 |
| opendevreview | Merged openstack/python-openstackclient stable/xena: [stable-only] Add munch in requirements.txt https://review.opendev.org/c/openstack/python-openstackclient/+/872340 | 20:30 |
| odyssey4me | stephenfin when you have a moment, please take a look at https://review.opendev.org/c/openstack/python-openstackclient/+/872327 which helps to resolve some tests on CS9 for wallaby | 20:40 |
| opendevreview | Merged openstack/ansible-collections-openstack master: Added workaround for non-determinism in ansible-inventory output https://review.opendev.org/c/openstack/ansible-collections-openstack/+/872344 | 21:39 |
| opendevreview | Merged openstack/ansible-collections-openstack master: Bump minimum required openstacksdk release to 1.0.0 https://review.opendev.org/c/openstack/ansible-collections-openstack/+/872066 | 21:45 |
| opendevreview | Merged openstack/ansible-collections-openstack master: Refactored coe_cluster{,_template} modules again https://review.opendev.org/c/openstack/ansible-collections-openstack/+/872039 | 22:03 |
| opendevreview | Merged openstack/ansible-collections-openstack master: Publish 2.0.0 release https://review.opendev.org/c/openstack/ansible-collections-openstack/+/872077 | 22:03 |
| gmann | stephenfin: please check sdks job fix for stable/wallaby also https://review.opendev.org/c/openstack/python-openstackclient/+/872341 | 22:16 |
| opendevreview | Merged openstack/python-openstackclient stable/wallaby: Replace assertItemsEqual with assertCountEqual https://review.opendev.org/c/openstack/python-openstackclient/+/872327 | 23:59 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!