| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2024-06-24 | |||
| 15:25:23 | opendevreview | Artem Goncharov proposed openstack/ansible-collections-openstack master: Wait for deleted server to disappear from results https://review.opendev.org/c/openstack/ansible-collections-openstack/+/922634 | |
| 15:56:34 | opendevreview | Artem Goncharov proposed openstack/ansible-collections-openstack master: Wait for deleted server to disappear from results https://review.opendev.org/c/openstack/ansible-collections-openstack/+/922634 | |
| 16:51:43 | opendevreview | Salman Hajizada proposed openstack/openstacksdk master: Add virtual media attachment and detachment support https://review.opendev.org/c/openstack/openstacksdk/+/921957 | |
| 16:56:16 | stephenfin | dtantsur: Not for today (I'm just about to disappear for the evening), but there's a couple of ironic-related patches arrived in for SDK recently where the author is adding a new method to the Node resource rather than a wholly new resource. | |
| 16:57:47 | stephenfin | Firstly, is the influx a coincidence or are there people working on it? Secondly, any concerns with what i'm suggesting. For example: https://review.opendev.org/c/openstack/openstacksdk/+/922610/2/openstack/baremetal/v1/node.py | |
| 16:57:54 | stephenfin | *dedicated people | |
| 17:03:38 | stephenfin | whoami-rajat: sorry, missed those pings. Done | |
| 17:47:11 | opendevreview | Artem Goncharov proposed openstack/ansible-collections-openstack master: [WIP] Allow munch results in server_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/922417 | |
| 17:48:02 | opendevreview | Artem Goncharov proposed openstack/ansible-collections-openstack master: [WIP] Allow munch results in server_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/922417 | |
| 19:26:29 | whoami-rajat | thanks stephenfin , will address the comments | |
| 20:22:05 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Add DeleteVolumeSnapshot class to v3 https://review.opendev.org/c/openstack/python-openstackclient/+/919611 | |
| 20:22:06 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Add volume snapshot unmanage support https://review.opendev.org/c/openstack/python-openstackclient/+/894634 | |
| #openstack-sdks - 2024-06-25 | |||
| 06:32:41 | opendevreview | Takashi Kajinami proposed openstack/os-client-config master: Drop usage of six https://review.opendev.org/c/openstack/os-client-config/+/922705 | |
| 07:40:44 | opendevreview | Merged openstack/ansible-collections-openstack master: Wait for deleted server to disappear from results https://review.opendev.org/c/openstack/ansible-collections-openstack/+/922634 | |
| 08:16:50 | opendevreview | Salman Hajizada proposed openstack/openstacksdk master: Add virtual media attachment and detachment support https://review.opendev.org/c/openstack/openstacksdk/+/921957 | |
| 08:43:44 | opendevreview | Salman Hajizada proposed openstack/openstacksdk master: Enhance VIF attachment with port and portgroup UUIDs https://review.opendev.org/c/openstack/openstacksdk/+/921890 | |
| 09:27:32 | opendevreview | Artem Goncharov proposed openstack/ansible-collections-openstack master: Allow munch results in server_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/922417 | |
| 11:04:40 | dtantsur | stephenfin: so yeah, we have a team of students who work on SDK improvements for Ironic | |
| 11:04:52 | dtantsur | iurygregory is their main mentor, I'll ask him to join | |
| 11:05:09 | dtantsur | hmm, he's not in the ironic channel either, might have simply lost his IRC connection | |
| 11:28:06 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: baremetal: Add 'details' parameter to various 'find' proxy methods https://review.opendev.org/c/openstack/openstacksdk/+/869370 | |
| 11:38:06 | frickler | "openstack server stop xyz" returns "cannot access local variable 'server_id' where it is not associated with a value" instead of "No server found" :-/ | |
| 17:41:47 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Remove unnecessary try-except https://review.opendev.org/c/openstack/python-openstackclient/+/922765 | |
| 17:41:48 | stephenfin | frickler: ^ | |
| 19:19:34 | opendevreview | Merged openstack/ansible-collections-openstack master: Allow munch results in server_info module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/922417 | |
| 19:49:25 | opendevreview | Merged openstack/ansible-collections-openstack master: Add insecure_registry property to coe_cluster_templates https://review.opendev.org/c/openstack/ansible-collections-openstack/+/917969 | |
| 23:01:13 | opendevreview | Merged openstack/ansible-collections-openstack master: Add vlan_tranparency for creation networks https://review.opendev.org/c/openstack/ansible-collections-openstack/+/922309 | |
| #openstack-sdks - 2024-06-26 | |||
| 01:57:56 | opendevreview | Steve Baker proposed openstack/ansible-collections-openstack master: Add application_credential module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/910463 | |
| 04:07:54 | opendevreview | Salman Hajizada proposed openstack/openstacksdk master: Enhance VIF attachment with port and portgroup UUIDs https://review.opendev.org/c/openstack/openstacksdk/+/921890 | |
| 04:13:16 | opendevreview | Salman Hajizada proposed openstack/openstacksdk master: Add virtual media attachment and detachment support https://review.opendev.org/c/openstack/openstacksdk/+/921957 | |
| 08:33:25 | opendevreview | Salman Hajizada proposed openstack/openstacksdk master: baremetal: Add virtual media attachment and detachment support https://review.opendev.org/c/openstack/openstacksdk/+/921957 | |
| 08:33:51 | opendevreview | Salman Hajizada proposed openstack/openstacksdk master: baremetal: Enhance VIF attachment with port and portgroup UUIDs https://review.opendev.org/c/openstack/openstacksdk/+/921890 | |
| 12:44:10 | kleini_ | I am trying to use Python openstacksdk for the first time. I created an application credential with access_rules and try to read that now using the API. I see in the response, that access_rules are transferred but they do not appear in the ApplicationCredential object. I get object has no attribute 'access_rules'. What am I doing wrong? Btw. it works, if I run the code in the Python debugger. | |
| 12:47:05 | kleini_ | https://paste.opendev.org/show/b42yPGMYoAA5kAnLSAnz/ | |
| 13:01:27 | kleini | Problem solved. 3.0.0 is broken and 3.1.0 is fixed. | |
| 13:01:55 | kleini | And unfortunately Debugger used venv while production used systemwide installed python. | |
| 14:27:43 | stephenfin | gtema: do you recall the reason for this comment? https://github.com/openstack/openstacksdk/blob/master/openstack/cloud/_identity.py#L1280-L1281 | |
| 14:28:00 | stephenfin | i.e. why you're using 'self.get_user' rather than 'self.identity.find_user' | |
| 14:28:28 | gtema | hmm | |
| 14:29:02 | gtema | I think some test was failing | |
| 14:30:27 | gtema | most likely because cloud layer test was designed the way that was not including call to find without parameters or something like that | |
| 14:30:52 | stephenfin | ah, indeed: 24 failures if you change it | |
| 14:30:59 | gtema | that was quite a pain during that switch to maintain order and count of api calls | |
| 14:31:59 | stephenfin | yeah, at this point most of those URL checks would be better served running against the proxy layer, but that's a huge rework | |
| 14:32:31 | gtema | indeed, and I doubt in any sense of such rework | |
| 15:55:15 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate 'application credential' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/922843 | |
| 18:08:48 | opendevreview | Rajat Dhasmana proposed openstack/openstacksdk master: Add support for default volume types https://review.opendev.org/c/openstack/openstacksdk/+/920464 | |
| 18:56:52 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Add cluster to volume service list https://review.opendev.org/c/openstack/python-openstackclient/+/922865 | |
| 20:37:05 | opendevreview | Brian Haley proposed openstack/python-openstackclient master: Add tests for 'default security group rule create' https://review.opendev.org/c/openstack/python-openstackclient/+/922565 | |
| #openstack-sdks - 2024-06-27 | |||
| 06:07:05 | shajizad | Good morning. In one of the patches I made (https://review.opendev.org/c/openstack/openstacksdk/+/921890) I got a zuul failure on openstacksdk-functional-devstack. However, my patch only includes baremetal changes and this job does not run them. This isn't the first time it happened, and usually after I upload a trivial patchset Zuul succeeds. | |
| 06:07:55 | shajizad | Could somebody check if theres anything in my patch that could be causing this? I would really appreciate it. | |
| 06:30:21 | frickler | shajizad: that failure may point to a regression in neutron, I've notified the team about it and triggered a recheck. this happens occasionally and sadly there's not much you can do about it except mentioning such an issue here | |
| 06:31:51 | shajizad | I see, thank you | |
| 14:07:50 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Add cluster to volume service list https://review.opendev.org/c/openstack/python-openstackclient/+/922865 | |
| 17:26:41 | opendevreview | Rodrigo Barbieri proposed openstack/openstacksdk master: block_storage: Add support for project_id in Limits https://review.opendev.org/c/openstack/openstacksdk/+/922965 | |
| #openstack-sdks - 2024-06-28 | |||
| 07:16:12 | opendevreview | Tobias Urdin proposed openstack/ansible-collections-openstack master: Add Neutron trunk module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/923025 | |
| 08:32:58 | opendevreview | Tobias Urdin proposed openstack/ansible-collections-openstack master: Add Neutron trunk module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/923025 | |
| 08:34:35 | opendevreview | Tobias Urdin proposed openstack/ansible-collections-openstack master: Add Neutron trunk module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/923025 | |
| 09:33:46 | opendevreview | Tobias Urdin proposed openstack/ansible-collections-openstack master: Add Neutron trunk module https://review.opendev.org/c/openstack/ansible-collections-openstack/+/923025 | |
| 11:21:16 | whoami-rajat_ | stephenfin, hey, can you take a look at this patch? it already has a +2 from cinder team https://review.opendev.org/c/openstack/python-openstackclient/+/922865 | |
| 11:24:07 | stephenfin | whoami-rajat_: Done. Got a nit but it can be addressed later | |
| 11:25:23 | whoami-rajat_ | stephenfin, yeah that would be a good improvement to reduce LOC, will address in a followup | |
| 11:25:47 | whoami-rajat_ | thanks | |
| 12:29:00 | opendevreview | Merged openstack/python-openstackclient master: Add cluster to volume service list https://review.opendev.org/c/openstack/python-openstackclient/+/922865 | |
| 15:05:23 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Followup: Reduce LOC in volume v3 service https://review.opendev.org/c/openstack/python-openstackclient/+/923056 | |
| 15:10:19 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Followup: Reduce LOC in volume v3 service https://review.opendev.org/c/openstack/python-openstackclient/+/923056 | |
| 16:47:00 | opendevreview | Merged openstack/keystoneauth master: Do not log sensitive info in OIDC https://review.opendev.org/c/openstack/keystoneauth/+/861906 | |
| #openstack-sdks - 2024-06-30 | |||
| 08:01:10 | opendevreview | Merged openstack/keystoneauth master: Output verification_uri_complete by logger https://review.opendev.org/c/openstack/keystoneauth/+/907775 | |
| #openstack-sdks - 2024-07-01 | |||
| 08:47:52 | opendevreview | Simon Hensel proposed openstack/ansible-collections-openstack master: Allow to specify multiple allocation pools when creating a subnet https://review.opendev.org/c/openstack/ansible-collections-openstack/+/912775 | |
| 11:09:19 | opendevreview | Pavlo Shchelokovskyy proposed openstack/python-openstackclient master: Return nothing from volume backup restore https://review.opendev.org/c/openstack/python-openstackclient/+/916906 | |
| 11:11:02 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Remove unnecessary try-except https://review.opendev.org/c/openstack/python-openstackclient/+/922765 | |
| 11:42:05 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: docs: Fix indentation https://review.opendev.org/c/openstack/python-openstackclient/+/923150 | |
| 13:04:36 | stephenfin | whoami-rajat: fyi https://bugs.launchpad.net/python-openstackclient/+bug/2070080 | |
| 13:37:06 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Migrate tests for ShowServer to SDK objects https://review.opendev.org/c/openstack/python-openstackclient/+/879073 | |
| 13:37:06 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: network: Migrate port list to compute SDK client https://review.opendev.org/c/openstack/python-openstackclient/+/918782 | |
| 13:37:07 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Migrate agent commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/918783 | |
| 13:37:07 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Migrate 'host set' to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/918784 | |
| 13:37:08 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Migrate remaining tests to SDK objects https://review.opendev.org/c/openstack/python-openstackclient/+/918785 | |
| 13:37:08 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: docs: Remove references to novaclient https://review.opendev.org/c/openstack/python-openstackclient/+/918786 | |
| 13:37:09 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: volume: Add v3-specific volume backup module https://review.opendev.org/c/openstack/python-openstackclient/+/918787 | |
| 13:37:10 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: volume: Add v3-specific volume module https://review.opendev.org/c/openstack/python-openstackclient/+/918788 | |
| 13:37:10 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: volume: add v3-specific volume type module https://review.opendev.org/c/openstack/python-openstackclient/+/918789 | |
| 13:37:12 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: volume: Add v3-specific volume transfer module https://review.opendev.org/c/openstack/python-openstackclient/+/918790 | |
| 13:37:12 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Add ability to configure fake server API version https://review.opendev.org/c/openstack/python-openstackclient/+/918791 | |
| 13:37:14 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Migrate to 'set_xxx_api_version' helpers https://review.opendev.org/c/openstack/python-openstackclient/+/918792 | |
| 13:37:14 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: volume: Migrate volume attachment commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/888234 | |
| 13:37:16 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: volume: Migrate 'volume list' command to compute SDK https://review.opendev.org/c/openstack/python-openstackclient/+/918793 | |
| 13:37:16 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: common: Migrate 'limits show' to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/888673 | |
| 13:37:18 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Migrate 'server add/remove security group' to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/918794 | |
| 13:37:18 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Migrate 'server migrate' to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/918795 | |
| 15:42:07 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Fix: incremental volume backup https://review.opendev.org/c/openstack/python-openstackclient/+/923169 | |
| 15:42:43 | whoami-rajat_ | stephenfin, fixed here https://review.opendev.org/c/openstack/python-openstackclient/+/923169 | |
| 15:43:21 | stephenfin | ty! | |
| 16:40:53 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate 'application credential' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/922843 | |
| 17:14:52 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate 'access rule' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/921882 | |
| 17:23:14 | opendevreview | Ebil Jacob proposed openstack/openstacksdk master: baremetal: Adds list node firmware support https://review.opendev.org/c/openstack/openstacksdk/+/922591 | |
| 18:06:39 | whoami-rajat_ | stephenfin, thanks for reporting it, also we've another +2 from cinder team so maybe we can merge/fix the issue | |
| 22:38:56 | opendevreview | Merged openstack/python-openstackclient master: compute: Remove unnecessary try-except https://review.opendev.org/c/openstack/python-openstackclient/+/922765 | |
| #openstack-sdks - 2024-07-02 | |||
| 11:15:47 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Fix typo https://review.opendev.org/c/openstack/openstacksdk/+/923212 | |
| 11:15:48 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Remove check for nova extensions https://review.opendev.org/c/openstack/openstacksdk/+/923213 | |