| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2024-06-20 | |||
| 07:46:33 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Add job to generate rust code https://review.opendev.org/c/openstack/codegenerator/+/922227 | |
| 09:21:02 | fyanac | Hi, I have this patch: https://review.opendev.org/c/openstack/ansible-collections-openstack/+/922309 but the verification jobs are failing, not sure why, but it seems they are failing since ~25May, could someone help? for example: https://zuul.opendev.org/t/openstack/build/18ab55dbf7ca4c9db9ea1320848a500a | |
| 09:21:11 | fyanac | @gtema^ | |
| 09:31:55 | gtema | fyanac: will try to have a look today | |
| 09:32:28 | fyanac | Thank you :) | |
| 10:03:41 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Add static object-store OpenAPI https://review.opendev.org/c/openstack/codegenerator/+/922381 | |
| 10:29:21 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Add static object-store OpenAPI https://review.opendev.org/c/openstack/codegenerator/+/922381 | |
| 12:28:50 | opendevreview | Lajos Katona proposed openstack/openstacksdk master: BGP: return list for get_dragents and get_bgp_speakers_hosted_by_dragent https://review.opendev.org/c/openstack/openstacksdk/+/920670 | |
| 12:47:24 | opendevreview | Riccardo Pittau proposed openstack/openstacksdk master: Move dib test to CentOS Stream 9 https://review.opendev.org/c/openstack/openstacksdk/+/922390 | |
| 12:49:54 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Add static object-store OpenAPI https://review.opendev.org/c/openstack/codegenerator/+/922381 | |
| 13:14:43 | Uggla | gtema, stephenfin can you please have a look at https://review.opendev.org/c/openstack/nova/+/871642/comment/57e278c1_118ba2af and give your opinion. | |
| 13:25:06 | opendevreview | Merged openstack/codegenerator master: Add static object-store OpenAPI https://review.opendev.org/c/openstack/codegenerator/+/922381 | |
| 13:29:16 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Add job to generate rust code https://review.opendev.org/c/openstack/codegenerator/+/922227 | |
| 14:05:46 | opendevreview | Salman Hajizada proposed openstack/openstacksdk master: Add virtual media attachment and detachment support https://review.opendev.org/c/openstack/openstacksdk/+/921957 | |
| 14:24:08 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Add job to generate rust code https://review.opendev.org/c/openstack/codegenerator/+/922227 | |
| 14:50:08 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: Add job to generate rust code https://review.opendev.org/c/openstack/codegenerator/+/922227 | |
| 16:12:08 | 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 | |
| 16:12:25 | opendevreview | Merged openstack/codegenerator master: Add job to generate rust code https://review.opendev.org/c/openstack/codegenerator/+/922227 | |
| 16:12:53 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: pre-commit: Bump versions https://review.opendev.org/c/openstack/openstacksdk/+/922419 | |
| 16:12:53 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: image: Check path is a file before attempting to use it https://review.opendev.org/c/openstack/openstacksdk/+/922418 | |
| 16:20:54 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate 'access rule' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/921882 | |
| 16:26:49 | 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:01:27 | stephenfin | Uggla: Done. gibbi has it right IMO | |
| 17:02:59 | Uggla | Thx stephenfin, I'll have a look tomorrow. | |
| 22:40:02 | opendevreview | Baek Seung Ju proposed openstack/openstacksdk master: Added missing stack documentation and rewrite stack proxy document https://review.opendev.org/c/openstack/openstacksdk/+/921863 | |
| #openstack-sdks - 2024-06-21 | |||
| 10:10:45 | 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 | |
| 10:57:03 | opendevreview | Salman Hajizada proposed openstack/openstacksdk master: Add virtual media attachment and detachment support https://review.opendev.org/c/openstack/openstacksdk/+/921957 | |
| 11:04:31 | opendevreview | Salman Hajizada proposed openstack/openstacksdk master: Enhance VIF attachment with port and portgroup UUIDs https://review.opendev.org/c/openstack/openstacksdk/+/921890 | |
| 11:09:49 | opendevreview | Salman Hajizada proposed openstack/openstacksdk master: Add virtual media attachment and detachment support https://review.opendev.org/c/openstack/openstacksdk/+/921957 | |
| 15:10:03 | opendevreview | Merged openstack/keystoneauth master: reno: Update master for unmaintained/zed https://review.opendev.org/c/openstack/keystoneauth/+/918619 | |
| 16:23:47 | opendevreview | Merged openstack/keystoneauth master: Python 3.12: do not use datetime.utcnow() https://review.opendev.org/c/openstack/keystoneauth/+/903860 | |
| #openstack-sdks - 2024-06-22 | |||
| 23:25:24 | 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-24 | |||
| 05:12:49 | whoami-rajat | stephenfin, gtema hey, can you take a look at these patches? | |
| 05:12:51 | whoami-rajat | https://review.opendev.org/c/openstack/python-openstackclient/+/919611/2 | |
| 05:12:53 | whoami-rajat | https://review.opendev.org/c/openstack/python-openstackclient/+/894634/8 | |
| 09:11:30 | opendevreview | Ebil Jacob proposed openstack/openstacksdk master: baremetal: Adds list node firmware support https://review.opendev.org/c/openstack/openstacksdk/+/922591 | |
| 10:05:27 | opendevreview | Ebil Jacob proposed openstack/openstacksdk master: baremetal: Adds list node firmware support https://review.opendev.org/c/openstack/openstacksdk/+/922591 | |
| 10:53:53 | opendevreview | Rawan El Ghali proposed openstack/openstacksdk master: baremetal: Adds list node history https://review.opendev.org/c/openstack/openstacksdk/+/922610 | |
| 11:08:32 | opendevreview | Abdulrahman Al-Taweel proposed openstack/openstacksdk master: baremetal: Adds list and get node history https://review.opendev.org/c/openstack/openstacksdk/+/922610 | |
| 11:19:33 | opendevreview | Lajos Katona proposed openstack/openstacksdk master: Add port bindings to SDK https://review.opendev.org/c/openstack/openstacksdk/+/915902 | |
| 12:53:51 | opendevreview | Merged openstack/keystoneauth master: tox: Migrate to tox v4 https://review.opendev.org/c/openstack/keystoneauth/+/922008 | |
| 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 | |