| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2024-07-15 | |||
| 11:43:21 | opendevreview | Abdulrahman Al-Taweel proposed openstack/openstacksdk master: baremetal: Adds list node history and get history event https://review.opendev.org/c/openstack/openstacksdk/+/922610 | |
| 11:44:50 | opendevreview | Abdulrahman Al-Taweel proposed openstack/openstacksdk master: baremetal: Adds list node history and get history event https://review.opendev.org/c/openstack/openstacksdk/+/922610 | |
| 12:46:12 | opendevreview | Merged openstack/python-openstackclient master: docs: Fix indentation https://review.opendev.org/c/openstack/python-openstackclient/+/923150 | |
| 12:59:53 | opendevreview | Merged openstack/python-openstackclient master: common: Migrate 'limits show' to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/888673 | |
| 12:59:55 | opendevreview | Merged openstack/python-openstackclient master: compute: Migrate 'server add/remove security group' to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/918794 | |
| 12:59:58 | opendevreview | Merged openstack/python-openstackclient master: compute: Migrate 'server migrate' to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/918795 | |
| 13:00:02 | opendevreview | Merged openstack/python-openstackclient master: compute: Migrate 'server set', 'server unset' commands https://review.opendev.org/c/openstack/python-openstackclient/+/918796 | |
| 13:00:07 | opendevreview | Merged openstack/python-openstackclient master: compute: Always use SDK client to display server https://review.opendev.org/c/openstack/python-openstackclient/+/918797 | |
| 13:00:11 | opendevreview | Merged openstack/python-openstackclient master: compute: Avoid third API call during 'server show' https://review.opendev.org/c/openstack/python-openstackclient/+/918800 | |
| 13:04:57 | opendevreview | Merged openstack/python-openstackclient master: compute: Migrate 'server rebuild' to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/918798 | |
| 13:04:59 | opendevreview | Merged openstack/python-openstackclient master: compute: Migrate remaining server actions to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/918799 | |
| 13:05:01 | opendevreview | Merged openstack/python-openstackclient master: compute: Migrate 'server evacuate' to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/918801 | |
| 13:13:06 | opendevreview | Merged openstack/python-openstackclient master: compute: Prevent use of conflicting v*-fixed-ip for 'server create --nic' https://review.opendev.org/c/openstack/python-openstackclient/+/923766 | |
| 15:01:16 | opendevreview | Merged openstack/python-openstackclient master: compute: Migrate 'server create' to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/918802 | |
| 15:01:18 | opendevreview | Merged openstack/python-openstackclient master: network: Replace use of in-tree API client https://review.opendev.org/c/openstack/python-openstackclient/+/919890 | |
| 15:15:04 | opendevreview | Merged openstack/python-openstackclient master: tests: Remove references to novaclient methods https://review.opendev.org/c/openstack/python-openstackclient/+/919892 | |
| 15:40:02 | opendevreview | Artem Goncharov proposed openstack/codegenerator master: fix network.fip.port_XXX schemas https://review.opendev.org/c/openstack/codegenerator/+/924164 | |
| 16:02:25 | opendevreview | Merged openstack/python-openstackclient master: quota: Move nova-network-related quota https://review.opendev.org/c/openstack/python-openstackclient/+/923866 | |
| 16:11:57 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate 'credential' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/923674 | |
| 16:17:48 | opendevreview | Merged openstack/python-openstackclient master: quota: Remove deprecated quota options https://review.opendev.org/c/openstack/python-openstackclient/+/923867 | |
| 17:21:10 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: quota: Migrate 'quota *' to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/919891 | |
| 17:21:10 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Remove python-novaclient https://review.opendev.org/c/openstack/python-openstackclient/+/918803 | |
| 17:21:11 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Remove TestServer base class https://review.opendev.org/c/openstack/python-openstackclient/+/919893 | |
| 18:03:32 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate 'domain' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/923785 | |
| #openstack-sdks - 2024-07-16 | |||
| 01:38:31 | diablo_rojo | When you get some time, can you circle back to reviewing this one stephenfin https://review.opendev.org/c/openstack/openstacksdk/+/921890? | |
| 07:22:55 | opendevreview | Merged openstack/codegenerator master: fix network.fip.port_XXX schemas https://review.opendev.org/c/openstack/codegenerator/+/924164 | |
| 09:58:25 | opendevreview | mbu proposed openstack/openstacksdk master: Allow to override _max_microversion for Volume._action function https://review.opendev.org/c/openstack/openstacksdk/+/924206 | |
| 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: Remove TestServer base class https://review.opendev.org/c/openstack/python-openstackclient/+/919893 | |
| 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: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: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: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: Add Python 3.12 classifier https://review.opendev.org/c/openstack/python-openstackclient/+/924135 | |
| 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:30:10 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Add Python 3.12 classifier https://review.opendev.org/c/openstack/openstacksdk/+/924136 | |
| 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:11 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: mypy: Run under minimum Python version https://review.opendev.org/c/openstack/openstacksdk/+/923663 | |
| 11:30:11 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Remove redundant code https://review.opendev.org/c/openstack/openstacksdk/+/923664 | |
| 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? | |