| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2024-07-16 | |||
| 10:17:42 | opendevreview | mbu proposed openstack/openstacksdk master: Allow to override _max_microversion for Volume._action function https://review.opendev.org/c/openstack/openstacksdk/+/924206 | |
| 10:35:39 | edmbold | hi, everyone, sorry if I am on the wrong channel to ask support/report a possible bug: | |
| 10:36:01 | edmbold | in openstacksdk 3.2.0 python module, I am getting this error: | |
| 10:36:11 | edmbold | AttributeError: 'Connection' object has no attribute '_floating_network_by_router'. | |
| 10:36:50 | edmbold | it looks that the _find_floating_network_by_router method in the _floating_ip.py line 307, referring to this not existing _floating_network_by_router method | |
| 10:38:05 | edmbold | I am using the "create_server()" method in the openstackclient connection object, that was working fine, until it was using openstacksdk 3.0.0, but now it looks broken | |
| 10:42:00 | edmbold | ok, I'll try the bug tracker linked here. .https://storyboard.openstack.org/ | |
| 11:23:00 | frickler | edmbold: that issues seems new to me, also better use https://bugs.launchpad.net/openstacksdk/ | |
| 11:25:10 | edmbold | thanks frickler! | |
| 11:31:00 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: baremetal: Enhance VIF attachment with port and portgroup UUIDs https://review.opendev.org/c/openstack/openstacksdk/+/921890 | |
| 11:46:44 | opendevreview | mbu proposed openstack/openstacksdk master: Allow to override _max_microversion for Volume._action function https://review.opendev.org/c/openstack/openstacksdk/+/924206 | |
| 11:58:23 | opendevreview | mbu proposed openstack/openstacksdk master: Allow to override _max_microversion for Volume._action function https://review.opendev.org/c/openstack/openstacksdk/+/924206 | |
| 13:44:52 | opendevreview | mbu proposed openstack/openstacksdk master: Allow to override _max_microversion for Volume._action function https://review.opendev.org/c/openstack/openstacksdk/+/924206 | |
| 14:17:14 | opendevreview | Salman Hajizada proposed openstack/openstacksdk master: Add virtual media attachment and detachment support https://review.opendev.org/c/openstack/openstacksdk/+/921957 | |
| #openstack-sdks - 2024-07-17 | |||
| 07:49:07 | opendevreview | Jake Yip proposed openstack/openstacksdk master: Cast all header values to string https://review.opendev.org/c/openstack/openstacksdk/+/924301 | |
| 14:02:33 | ganso | hi stephenfin ! thanks for the improvements in https://review.opendev.org/c/openstack/openstacksdk/+/922965 ! I left you a question regarding the changes. If you have a minute could you please take a look? I am sure the question will be trivial to you as you are more familiar with it | |
| 15:49:57 | stephenfin | ganso: It's a fair point. I don't have a good answer for how to resolve that, unfortunately. Personally, I tend to ignore the `location` field entirely but that's just me | |
| 15:56:21 | stephenfin | ganso: As I understand it, that the location attribute is either inferred by the auth information or provided explicitly from a `project_id` attribute of a given resource. Unfortunately cinder's '/limits do not expose this attribute | |
| 16:05:37 | ganso | stephenfin: yea I had the same impression as well, that the location attribute is completely unrelated to the resource but more to who is performing the request, as it is in attribute that is included by the framework. I would be fine with that if you're fine with it, I just wanted to raise awareness just to confirm that's the behavior we want | |
| #openstack-sdks - 2024-07-18 | |||
| 10:16:37 | 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 | |
| 10:27:32 | stephenfin | gtema: Waiting on CI results, SDK 3.3.0 is out and included in upper-constraints so I think the last of the novaclient removal series can finally go in \o/ https://review.opendev.org/c/openstack/python-openstackclient/+/923949/ | |
| 10:28:32 | gtema | right, it was too much to be reviewed in one day and then other things popped up | |
| 10:28:36 | gtema | on it after lunch | |
| 11:26:27 | opendevreview | Merged openstack/python-openstackclient master: quota: Add 'quota set --default' option https://review.opendev.org/c/openstack/python-openstackclient/+/923949 | |
| 11:43:25 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Allow adding, removing multiple SGs https://review.opendev.org/c/openstack/python-openstackclient/+/894529 | |
| 11:44:14 | stephenfin | gtema: all good. ty! Got one final one there too, then I think it's time to cut a new release | |
| 12:23:05 | gtema | stephenfin, pls look at comment at https://review.opendev.org/c/openstack/python-openstackclient/+/918803/13 | |
| 12:25:13 | stephenfin | gtema: Good catch. We're not seeing that in tests because everything is currently using 'sdk_connection.compute' instead of 'compute' | |
| 12:25:30 | stephenfin | lemme respin | |
| 12:25:36 | gtema | yes, I assumed that. | |
| 12:25:37 | gtema | sure | |
| 12:45:33 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Remove python-novaclient https://review.opendev.org/c/openstack/python-openstackclient/+/918803 | |
| 12:45:34 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Remove aliasing from extensions test https://review.opendev.org/c/openstack/python-openstackclient/+/924403 | |
| 12:45:34 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Remove TestServer base class https://review.opendev.org/c/openstack/python-openstackclient/+/919893 | |
| 12:45:35 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Rename 'compute_sdk_client' -> 'compute_client' https://review.opendev.org/c/openstack/python-openstackclient/+/924405 | |
| 12:45:35 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Migrate to 'compute' client alias https://review.opendev.org/c/openstack/python-openstackclient/+/924404 | |
| 12:45:39 | stephenfin | gtema: done | |
| 12:46:14 | gtema | 👍️ | |
| 13:35:20 | opendevreview | Merged openstack/python-openstackclient master: quota: Default network quotas to not force https://review.opendev.org/c/openstack/python-openstackclient/+/923868 | |
| 13:35:21 | opendevreview | Merged openstack/python-openstackclient master: quota: Split up 'quota list' command https://review.opendev.org/c/openstack/python-openstackclient/+/923869 | |
| 17:53:24 | opendevreview | Merged openstack/python-openstackclient master: quota: Migrate 'quota *' to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/919891 | |
| 18:03:29 | opendevreview | Merged openstack/python-openstackclient master: Remove python-novaclient https://review.opendev.org/c/openstack/python-openstackclient/+/918803 | |
| #openstack-sdks - 2024-07-19 | |||
| 11:07:36 | stephenfin | #success python-novaclient is no longer a dependency of OSC. It's all SDK from here on out! | |
| 11:07:41 | opendevstatus | stephenfin: Added success to Success page (https://wiki.openstack.org/wiki/Successes) | |
| 11:27:30 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Drop support for Python 3.8 https://review.opendev.org/c/openstack/python-openstackclient/+/924493 | |
| 11:27:30 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Add Python 3.12 classifier https://review.opendev.org/c/openstack/python-openstackclient/+/924135 | |
| 11:30:10 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Drop support for Python 3.7, 3.8 https://review.opendev.org/c/openstack/openstacksdk/+/923662 | |
| 11:30:10 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Add Python 3.12 classifier https://review.opendev.org/c/openstack/openstacksdk/+/924136 | |
| 11:30:11 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Remove redundant code https://review.opendev.org/c/openstack/openstacksdk/+/923664 | |
| 11:30:11 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: mypy: Run under minimum Python version https://review.opendev.org/c/openstack/openstacksdk/+/923663 | |
| 12:48:01 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: block storage: Add 'set_readonly' action to v2 https://review.opendev.org/c/openstack/openstacksdk/+/924507 | |
| 13:18:48 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: volume: Add missing 'find_type' proxy API to v2 https://review.opendev.org/c/openstack/openstacksdk/+/924508 | |
| 13:19:27 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: block storage: Add missing 'find_type' proxy API to v2 https://review.opendev.org/c/openstack/openstacksdk/+/924508 | |
| 15:23:31 | opendevreview | Merged openstack/openstacksdk master: cloud: Fix typo https://review.opendev.org/c/openstack/openstacksdk/+/923212 | |
| 16:49:54 | opendevreview | Merged openstack/openstacksdk master: exceptions: ResourceNotFound -> NotFoundException https://review.opendev.org/c/openstack/openstacksdk/+/921952 | |
| #openstack-sdks - 2024-07-22 | |||
| 09:50:27 | 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:06:31 | iurygregory | hey openstacksdk folks, if any core has time can you please check https://review.opendev.org/c/openstack/openstacksdk/+/921890 ? o/ | |
| 15:31:25 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate 'domain' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/923785 | |
| 16:05:35 | opendevreview | Tim Bishop proposed openstack/python-openstackclient master: Respect --skip-resource when deleting. https://review.opendev.org/c/openstack/python-openstackclient/+/924659 | |
| 18:25:49 | opendevreview | Tim Bishop proposed openstack/python-openstackclient master: Respect --skip-resource when deleting. https://review.opendev.org/c/openstack/python-openstackclient/+/924659 | |
| 20:09:45 | opendevreview | Tim Bishop proposed openstack/python-openstackclient master: Respect --skip-resource when deleting. https://review.opendev.org/c/openstack/python-openstackclient/+/924659 | |
| 21:42:32 | opendevreview | Tim Bishop proposed openstack/python-openstackclient master: Respect --skip-resource when deleting. https://review.opendev.org/c/openstack/python-openstackclient/+/924659 | |
| 21:52:48 | opendevreview | Merged openstack/openstacksdk master: baremetal: Enhance VIF attachment with port and portgroup UUIDs https://review.opendev.org/c/openstack/openstacksdk/+/921890 | |
| #openstack-sdks - 2024-07-23 | |||
| 10:16:02 | frickler | gtema: ready for osc 6.7.0? https://review.opendev.org/c/openstack/releases/+/924134 | |
| 10:17:17 | gtema | good that you ask. I see now top of the huge stack has failed and need a recheck | |
| 10:17:49 | gtema | means - no, not ready | |
| 10:22:47 | frickler | gtema: ok, then please -1 the release patch just to be on the safe side | |
| 10:23:49 | gtema | frickler: thks, done | |
| 12:55:09 | gibi | gtema: hi! I need help using the openstacksdk from nova to talk to manila. So far we have https://review.opendev.org/c/openstack/nova/+/831194/50/nova/utils.py#991 that creates the connection based on nova's service user and password from the config. | |
| 12:55:25 | gibi | and that works | |
| 12:55:45 | gibi | but we need to use the user context instead. | |
| 12:56:16 | gibi | As nova should only attach a share to a VM if the user calling nova has the rights to see the share in manial | |
| 12:56:46 | gtema | which info from user do you have? token? | |
| 12:58:00 | opendevreview | Merged openstack/python-openstackclient master: Followup: Reduce LOC in volume v3 service https://review.opendev.org/c/openstack/python-openstackclient/+/923056 | |
| 12:58:02 | opendevreview | Merged openstack/python-openstackclient master: Add DeleteVolumeSnapshot class to v3 https://review.opendev.org/c/openstack/python-openstackclient/+/919611 | |
| 12:58:04 | opendevreview | Merged openstack/python-openstackclient master: Add volume snapshot unmanage support https://review.opendev.org/c/openstack/python-openstackclient/+/894634 | |
| 12:59:28 | gibi | gtema: yeah we have the token in the context object I believe | |
| 12:59:43 | gibi | context = RequestContext | |
| 13:05:12 | gtema | gibi - gimme some minutes to find the right way | |
| 13:06:12 | gibi | yepp I can print the auth_token from the context so we definitly has it | |
| 13:06:17 | gibi | gtema: thank! | |
| 13:06:21 | gibi | thanks | |
| 13:58:45 | gtema | gibi: https://paste.opendev.org/show/bQRC2yqufkeGY5Eu8bXG/ - in principle it can be done like that | |
| 13:59:18 | gtema | a bit nasty thing is that you "should" reauth with the user's token since only token is not enough | |
| 13:59:41 | gtema | you could do token inspection, but that is still the API call, so you save nothing | |
| 14:20:44 | gibi | gtema: thanks I will try it and report back | |
| 14:20:57 | gtema | cool | |
| 15:25:52 | opendevreview | Merged openstack/python-openstackclient master: Add tests for 'default security group rule create' https://review.opendev.org/c/openstack/python-openstackclient/+/922565 | |
| #openstack-sdks - 2024-07-24 | |||
| 12:10:57 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Replace reference to removed variable https://review.opendev.org/c/openstack/openstacksdk/+/924847 | |
| 12:11:49 | stephenfin | gtema: Bugfix for a bug I introduced there ^ | |
| 12:13:09 | gtema | ack | |
| 13:34:10 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Remove check for nova extensions https://review.opendev.org/c/openstack/openstacksdk/+/923213 | |
| 13:34:10 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: pre-commit: Bump versions https://review.opendev.org/c/openstack/openstacksdk/+/923221 | |
| 13:34:11 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Make _OpenStackCloudMixin subclass ServicesMixin https://review.opendev.org/c/openstack/openstacksdk/+/923214 | |
| 13:34:11 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: mypy: Enable checks for openstack.tests.fixtures https://review.opendev.org/c/openstack/openstacksdk/+/923225 | |
| 13:34:12 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Reorganize _OpenStackCloudMixin, Connection (1/2) https://review.opendev.org/c/openstack/openstacksdk/+/923216 | |
| 13:34:12 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Make service mixins subclass _OpenStackCloudMixin https://review.opendev.org/c/openstack/openstacksdk/+/923215 | |
| 13:34:13 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Reorganize _OpenStackCloudMixin, Connection (2/2) https://review.opendev.org/c/openstack/openstacksdk/+/923217 | |
| 13:34:14 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Combine networking-related classes https://review.opendev.org/c/openstack/openstacksdk/+/923220 | |
| 13:34:14 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Misc type fixes https://review.opendev.org/c/openstack/openstacksdk/+/923218 | |