| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2024-07-18 | |||
| 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? | |
| 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: pre-commit: Bump versions https://review.opendev.org/c/openstack/openstacksdk/+/923221 | |
| 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: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:11 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Make _OpenStackCloudMixin subclass ServicesMixin https://review.opendev.org/c/openstack/openstacksdk/+/923214 | |
| 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:12 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Reorganize _OpenStackCloudMixin, Connection (1/2) https://review.opendev.org/c/openstack/openstacksdk/+/923216 | |
| 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: Misc type fixes https://review.opendev.org/c/openstack/openstacksdk/+/923218 | |
| 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:16 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: mypy: Enable checks for openstack.cloud https://review.opendev.org/c/openstack/openstacksdk/+/923219 | |
| 13:34:16 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: tests: Remove TestClustering functional tests https://review.opendev.org/c/openstack/openstacksdk/+/923223 | |
| 13:34:18 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: mypy: Enable checks for openstack.tests.functional https://review.opendev.org/c/openstack/openstacksdk/+/923224 | |
| 13:34:18 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: mypy: Enable warn_return_any https://review.opendev.org/c/openstack/openstacksdk/+/923226 | |
| 13:34:20 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: WIP: mypy: Full typing of openstack.resource https://review.opendev.org/c/openstack/openstacksdk/+/923227 | |
| 13:34:20 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: cloud: Fix annotations for proxy method https://review.opendev.org/c/openstack/openstacksdk/+/924853 | |
| 14:22:44 | opendevreview | Salman Hajizada proposed openstack/openstacksdk master: Add virtual media attachment and detachment support https://review.opendev.org/c/openstack/openstacksdk/+/921957 | |
| 14:54:11 | iurygregory | did something merged in openstack cli that could break swift in devstack? | |
| 14:54:51 | iurygregory | lib/swift:swift_configure_tempurls:847 : openstack --os-cloud '' --os-region-name RegionOne --os-auth-url https://217.182.140.102/identity --os-username=swift --os-password=secretservice --os-user-domain-name=Default --os-project-name=service --os-project-domain-name=Default object store account set --property Temp-URL-Key=secretkey | |
| 14:54:52 | iurygregory | openstack-cli-server: 'object store account set --property Temp-URL-Key=secretkey' is not an openstack-cli-server command. See 'openstack-cli-server --help'. | |
| 14:55:04 | iurygregory | saw this in some of the ironic jobs | |
| 14:56:44 | stephenfin | nothing's changed in openstackclient/object since mid April | |
| 14:56:47 | stephenfin | so I doubt it | |
| 14:57:07 | stephenfin | I suspect that might be broken for a while. Is it causing failures? | |
| 14:58:31 | opendevreview | Mohamed Waiel Shikfa proposed openstack/openstacksdk master: Add virtual media attachment and detachment support https://review.opendev.org/c/openstack/openstacksdk/+/921957 | |
| 14:59:54 | iurygregory | stephenfin, yeah we started to see in some jobs in ironic.. | |
| 15:00:22 | iurygregory | https://zuul.opendev.org/t/openstack/build/39105e3b5251484caf6253213d9ad773 | |
| 15:03:20 | iurygregory | stephenfin, ty! seems like some change to use openstack-cli-server broke us XD | |
| 15:04:46 | stephenfin | Yeah, If628c4627f7c8b8c2ee9bca16ea6db693cf8526a looks a likely candidate, though only if it's been failing with ~12 days | |
| 17:35:19 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate 'domain' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/923785 | |