| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2023-01-31 | |||
| 10:45:39 | opendevreview | Radosław Piliszek proposed openstack/openstacksdk master: support notification vmoves for masakari https://review.opendev.org/c/openstack/openstacksdk/+/872080 | |
| 10:45:39 | opendevreview | Radosław Piliszek proposed openstack/openstacksdk master: support notification vmoves for masakari https://review.opendev.org/c/openstack/openstacksdk/+/872080 | |
| 11:25:08 | 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:27:04 | gtema | Yes, go for it | |
| 11:35:04 | stephenfin | lajoskatona: That had been on my list. Done! | |
| 11:35:50 | 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:36:44 | gtema | looking on those | |
| 11:36:52 | whoami-rajat | stephenfin, we've 2 weeks for client release right? or am i missing something | |
| 11:37:24 | stephenfin | Yeah, but we're cycle with intermediary so we can cut effectively whenever so long as we have *a* release per-cycle | |
| 11:38:00 | 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:13 | 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:40 | whoami-rajat | so are we planning to release again then with other client releases? | |
| 11:38:50 | stephenfin | Perfect. Ping me if/when you've those up and I'll happily review them. | |
| 11:39:01 | stephenfin | Yeah. We can release whenever it suits us | |
| 11:39:17 | whoami-rajat | cool, I will work on them then, thanks! | |
| 11:39:21 | stephenfin | Let's aim for a 6.2.0 in the next few weeks if you can get cinder coverage closed out | |
| 11:39:48 | whoami-rajat | stephenfin, do you want me to update this? just noticed your comments https://review.opendev.org/c/openstack/python-openstackclient/+/869811 | |
| 11:40:03 | whoami-rajat | or better as a followup? | |
| 11:40:11 | stephenfin | follow-up would be better | |
| 11:40:23 | stephenfin | they're nits at best | |
| 11:41:00 | whoami-rajat | ok, sounds good | |
| 11:58:21 | opendevreview | suzhengwei proposed openstack/openstacksdk master: support notification vmoves for masakari https://review.opendev.org/c/openstack/openstacksdk/+/872080 | |
| 12:18:55 | lajoskatona | stephenfin, gtema: thanks | |
| 12:41:00 | opendevreview | Merged openstack/python-openstackclient master: Move network trunk commands from python-neutronclient https://review.opendev.org/c/openstack/python-openstackclient/+/869447 | |
| 12:43:34 | opendevreview | Merged openstack/python-openstackclient master: Add volume summary command https://review.opendev.org/c/openstack/python-openstackclient/+/869436 | |
| 12:48:44 | 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? | |
| 13:42:10 | opendevreview | Mridula Joshi proposed openstack/openstacksdk master: Add support for glance cache list https://review.opendev.org/c/openstack/openstacksdk/+/869710 | |
| 14:10:10 | odyssey4me | never mind my query - wallaby is in extended maintenance, so I guess no more releases there | |
| 14:39:37 | stephenfin | odyssey4me: Yup, no more I'm afraid https://review.opendev.org/c/openstack/releases/+/847937 | |
| 14:40:16 | gtema | I would say, now we can have a real party - https://pypi.org/project/openstacksdk/1.0.0/ is there | |
| 14:44:03 | stephenfin | #success openstacksdk 1.0.0 is out 🎉 | |
| 14:44:03 | stephenfin | #success openstacksdk 1.0.0 is out 🎉 | |
| 14:44:06 | opendevstatus | stephenfin: Added success to Success page (https://wiki.openstack.org/wiki/Successes) | |
| 14:44:33 | odyssey4me | w00t! | |
| 14:45:07 | stephenfin | gtema: did you merge more patches for the container orchestration layer? | |
| 14:45:14 | gtema | :) | |
| 14:45:15 | gtema | yes | |
| 14:45:23 | gtema | coe is also complete | |
| 14:48:49 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Update README to indicate COE resource/proxy support https://review.opendev.org/c/openstack/openstacksdk/+/872283 | |
| 14:49:15 | stephenfin | gtema: Probably want that so ^ | |
| 14:49:22 | stephenfin | it's trivial | |
| 14:49:59 | gtema | ah, there are too many pages not built from sources. This should be target for R2.0 ;-) | |
| 15:15:32 | opendevreview | Mridula Joshi proposed openstack/openstacksdk master: Add support for glance cache list https://review.opendev.org/c/openstack/openstacksdk/+/869710 | |
| 15:29:19 | opendevreview | Merged openstack/openstacksdk master: Update README to indicate COE resource/proxy support https://review.opendev.org/c/openstack/openstacksdk/+/872283 | |
| 15:58:51 | opendevreview | Alfredo Moralejo proposed openstack/python-openstackclient stable/wallaby: Replace assertItemsEqual with assertCountEqual https://review.opendev.org/c/openstack/python-openstackclient/+/872327 | |
| 16:46:52 | 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:48:12 | 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:49:59 | 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 | |
| 17:13:37 | 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:14:06 | gmann | but If I am not wrong sdk master supposed to work for all supported stable branch ? | |
| 17:23:44 | 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:27:56 | 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:29:47 | 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:30:01 | gtema | yes, apparently | |
| 17:30:01 | gmann | we should have that here https://opendev.org/openstack/python-openstackclient/src/branch/master/requirements.txt | |
| 17:30:11 | gmann | let me push change | |
| 17:30:36 | gtema | but only for xena, this is not required anywhere else | |
| 17:34:01 | gmann | right | |
| 17:34:46 | gtema | and interestingly stephenfin just today made release of OSC for xena, what a facepalm on a R1.0 day | |
| 17:35:12 | gmann | humm it worked there? | |
| 17:35:43 | gtema | well, as usual it all depend on the constellation | |
| 17:40:34 | 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:56 | gmann | gtema: stephenfin: ^^ | |
| 17:41:06 | gtema | yes, I see. Thanks a lot gmann | |
| 17:41:18 | stephenfin | thanks | |
| 17:44:48 | opendevreview | Ghanshyam proposed openstack/python-openstackclient stable/wallaby: [stable-only] Add munch in requirements.txt https://review.opendev.org/c/openstack/python-openstackclient/+/872341 | |
| 18:51:23 | 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:52:09 | 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:27 | 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:39 | opendevreview | Jakob Meng proposed openstack/ansible-collections-openstack master: Publish 2.0.0 release https://review.opendev.org/c/openstack/ansible-collections-openstack/+/872077 | |
| 19:42:59 | opendevreview | Arvid Requate proposed openstack/keystoneauth master: New auth plugin v3oidcdeviceauthz https://review.opendev.org/c/openstack/keystoneauth/+/869876 | |
| 20:30:58 | opendevreview | Merged openstack/python-openstackclient stable/xena: [stable-only] Add munch in requirements.txt https://review.opendev.org/c/openstack/python-openstackclient/+/872340 | |
| 20:40:01 | 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 | |
| 21:39:10 | 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:45:26 | 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 | |
| 22:03:03 | 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:04 | opendevreview | Merged openstack/ansible-collections-openstack master: Publish 2.0.0 release https://review.opendev.org/c/openstack/ansible-collections-openstack/+/872077 | |
| 22:16:14 | gmann | stephenfin: please check sdks job fix for stable/wallaby also https://review.opendev.org/c/openstack/python-openstackclient/+/872341 | |
| 23:59:02 | opendevreview | Merged openstack/python-openstackclient stable/wallaby: Replace assertItemsEqual with assertCountEqual https://review.opendev.org/c/openstack/python-openstackclient/+/872327 | |
| #openstack-sdks - 2023-02-01 | |||
| 00:03:56 | timburke | gmann, gtema, thanks for looking at the busted job -- unfortunately, looks like https://review.opendev.org/c/openstack/python-openstackclient/+/872340 wasn't sufficient. as best i can tell, the job is only installing python-openstackclient from a tagged release, not git? | |
| 00:04:41 | timburke | see https://zuul.opendev.org/t/openstack/build/4b92987487b44219953cbe90716f51c6/log/controller/logs/devstacklog.txt#10818 -- i'm assuming that the 5.6.0 (rather than recently-released 5.6.1) is just because of an out-of-date mirror | |
| 00:10:33 | timburke | i'm still trying to figure out just what the value is we're hoping to get out of the job -- do we actually expect anyone to ever run xena python-openstackclient with master openstacksdk? | |
| 00:55:34 | gmann | timburke: yeah, we need to wait until its released and upper constraints are updated. | |
| 00:56:10 | gmann | I think that is the goal of this job but gtema or stephenfin can correct | |
| 00:57:41 | opendevreview | suzhengwei proposed openstack/openstacksdk master: support notification vmoves for masakari https://review.opendev.org/c/openstack/openstacksdk/+/872080 | |
| 01:10:16 | timburke | particularly given the infrequent releases on stable branches, i would've thought it'd make more sense to run those xena tests on openstacksdk's master branch then, no? | |
| 05:24:21 | gtema | timburke, this is alsoy assumption now. It does feel logical that both SDK and OSC are added explicitly though Zuul project requirements, and still released OSC is used | |
| 07:12:55 | opendevreview | Merged openstack/python-openstackclient stable/wallaby: [stable-only] Add munch in requirements.txt https://review.opendev.org/c/openstack/python-openstackclient/+/872341 | |
| 08:25:22 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: Implement acceptance test job https://review.opendev.org/c/openstack/openstacksdk/+/860399 | |
| 09:39:08 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Add block storage work cleanup command https://review.opendev.org/c/openstack/python-openstackclient/+/872390 | |
| 10:03:51 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Fix issue with multiple records in recordset https://review.opendev.org/c/openstack/ansible-collections-openstack/+/872392 | |
| 10:04:34 | opendevreview | Shnaidman Sagi (Sergey) proposed openstack/ansible-collections-openstack master: Fix issue with multiple records in recordset https://review.opendev.org/c/openstack/ansible-collections-openstack/+/872392 | |
| 15:23:47 | opendevreview | Artom Lifshitz proposed openstack/python-openstackclient master: Add func test for compute 2.94 https://review.opendev.org/c/openstack/python-openstackclient/+/872420 | |
| 15:55:56 | gmann | gtema: as you know osc cannot be released in stable/wallaby https://review.opendev.org/c/openstack/releases/+/872382 | |
| 15:56:13 | gtema | yes, I already figured this out | |
| 15:56:42 | gmann | gtema: we have two option to unblock stable/wallaby gate 1. remove the sdk testing 2. test with old sdk removing master branch from here https://github.com/openstack/openstacksdk/blob/stable/wallaby/.zuul.yaml#L40 | |
| 15:56:42 | gtema | don't know what to do though | |
| 15:56:54 | gmann | 2nd one seems conflict with the notes there | |
| 15:59:04 | gtema | 1st option is not good. On the other side if we are not able to have any releases there I see honestly absolutely no reason to keep this testing | |
| 16:02:33 | gmann | gtema: humm, but testing with old sdk version does not make sense if I am not wrong | |