| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2024-05-17 | |||
| 12:21:04 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: compute: Add additional server create paramters https://review.opendev.org/c/openstack/openstacksdk/+/918781 | |
| 12:21:04 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: compute: Add support for legacy 'onSharedStorage' param https://review.opendev.org/c/openstack/openstacksdk/+/918745 | |
| 12:21:05 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: compute, block storage: Add support for quota class sets https://review.opendev.org/c/openstack/openstacksdk/+/919884 | |
| 13:18:51 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Always use SDK client to display server https://review.opendev.org/c/openstack/python-openstackclient/+/918797 | |
| 13:18:52 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Migrate 'server rebuild' to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/918798 | |
| 13:18:52 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Avoid third API call during 'server show' https://review.opendev.org/c/openstack/python-openstackclient/+/918800 | |
| 13:18:53 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Migrate 'server evacuate' to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/918801 | |
| 13:18:53 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Migrate remaining server actions to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/918799 | |
| 13:18:54 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: network: Replace use of in-tree API client https://review.opendev.org/c/openstack/python-openstackclient/+/919890 | |
| 13:18:54 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: compute: Migrate 'server create' to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/918802 | |
| 13:18:55 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: WIP: common: Migrate 'quota *' to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/919891 | |
| 13:18:56 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Remove python-novaclient https://review.opendev.org/c/openstack/python-openstackclient/+/918803 | |
| 13:18:56 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: tests: Remove references to novaclient methods https://review.opendev.org/c/openstack/python-openstackclient/+/919892 | |
| 13:18:58 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: WIP: Remove TestServer base class https://review.opendev.org/c/openstack/python-openstackclient/+/919893 | |
| 19:00:30 | opendevreview | Ghanshyam proposed openstack/openstacksdk master: Remove retired project Senlin job https://review.opendev.org/c/openstack/openstacksdk/+/919981 | |
| #openstack-sdks - 2024-05-19 | |||
| 13:53:41 | opendevreview | Takashi Kajinami proposed openstack/osc-lib master: Drop dependency on simplejson https://review.opendev.org/c/openstack/osc-lib/+/920000 | |
| 13:56:02 | opendevreview | Takashi Kajinami proposed openstack/python-openstackclient master: Drop direct dependency on simplejson https://review.opendev.org/c/openstack/python-openstackclient/+/920001 | |
| 13:57:31 | opendevreview | Takashi Kajinami proposed openstack/osc-lib master: Drop dependency on simplejson https://review.opendev.org/c/openstack/osc-lib/+/920000 | |
| 20:19:07 | opendevreview | Tobias Urdin proposed openstack/python-openstackclient master: Show Created At column for volume backups https://review.opendev.org/c/openstack/python-openstackclient/+/920003 | |
| #openstack-sdks - 2024-05-20 | |||
| 00:58:34 | opendevreview | Takashi Kajinami proposed openstack/python-openstackclient master: Drop direct dependency on simplejson https://review.opendev.org/c/openstack/python-openstackclient/+/920001 | |
| 00:59:06 | opendevreview | Takashi Kajinami proposed openstack/python-openstackclient master: Drop direct dependency on simplejson https://review.opendev.org/c/openstack/python-openstackclient/+/920001 | |
| 00:59:50 | opendevreview | Takashi Kajinami proposed openstack/python-openstackclient master: Drop direct dependency on simplejson https://review.opendev.org/c/openstack/python-openstackclient/+/920001 | |
| 01:00:00 | opendevreview | Takashi Kajinami proposed openstack/osc-lib master: Drop dependency on simplejson https://review.opendev.org/c/openstack/osc-lib/+/920000 | |
| 11:27:15 | opendevreview | Merged openstack/python-openstackclient master: Add CreateVolume class to v3 https://review.opendev.org/c/openstack/python-openstackclient/+/917465 | |
| 11:27:16 | opendevreview | Merged openstack/python-openstackclient master: Add support for managing volumes https://review.opendev.org/c/openstack/python-openstackclient/+/895214 | |
| 11:27:18 | opendevreview | Merged openstack/python-openstackclient master: Add DeleteVolume class to v3 https://review.opendev.org/c/openstack/python-openstackclient/+/919729 | |
| 11:45:20 | opendevreview | Merged openstack/python-openstackclient master: Remove clients of retired projects https://review.opendev.org/c/openstack/python-openstackclient/+/919856 | |
| 11:45:21 | opendevreview | Merged openstack/python-openstackclient master: Drop direct dependency on simplejson https://review.opendev.org/c/openstack/python-openstackclient/+/920001 | |
| 14:12:38 | opendevreview | Merged openstack/openstacksdk master: Remove retired project Senlin job https://review.opendev.org/c/openstack/openstacksdk/+/919981 | |
| 14:23:52 | opendevreview | James E. Blair proposed openstack/openstacksdk master: Add docs for common keystoneauth settings https://review.opendev.org/c/openstack/openstacksdk/+/920042 | |
| 19:42:02 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Add support for volume unmanage https://review.opendev.org/c/openstack/python-openstackclient/+/919820 | |
| 21:15:06 | M0weng[m] | stephenfin: is there anything else I need to do for https://review.opendev.org/c/openstack/python-openstackclient/+/910664 ? | |
| #openstack-sdks - 2024-05-21 | |||
| 06:53:56 | opendevreview | Jens Harbott proposed openstack/openstacksdk stable/2024.1: Remove retired project Senlin job https://review.opendev.org/c/openstack/openstacksdk/+/919875 | |
| 06:54:44 | opendevreview | Jens Harbott proposed openstack/openstacksdk stable/2023.2: Remove retired project Senlin job https://review.opendev.org/c/openstack/openstacksdk/+/919876 | |
| 07:40:58 | opendevreview | Jens Harbott proposed openstack/openstacksdk stable/2023.1: Remove retired project Senlin job https://review.opendev.org/c/openstack/openstacksdk/+/920079 | |
| 07:41:51 | opendevreview | Merged openstack/python-openstackclient master: Fix tests on Python 3.12 https://review.opendev.org/c/openstack/python-openstackclient/+/919046 | |
| 10:18:26 | opendevreview | Merged openstack/openstacksdk stable/2024.1: Remove retired project Senlin job https://review.opendev.org/c/openstack/openstacksdk/+/919875 | |
| 10:27:46 | opendevreview | Merged openstack/openstacksdk stable/2023.2: Remove retired project Senlin job https://review.opendev.org/c/openstack/openstacksdk/+/919876 | |
| 10:51:58 | opendevreview | Merged openstack/openstacksdk stable/2023.1: Remove retired project Senlin job https://review.opendev.org/c/openstack/openstacksdk/+/920079 | |
| 13:42:22 | opendevreview | Merged openstack/keystoneauth master: Update master for stable/2024.1 https://review.opendev.org/c/openstack/keystoneauth/+/912179 | |
| #openstack-sdks - 2024-05-22 | |||
| 16:24:22 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate 'service' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/906395 | |
| 17:37:09 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate 'service' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/906395 | |
| 17:48:12 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate 'service' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/906395 | |
| 17:58:48 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate 'service' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/906395 | |
| #openstack-sdks - 2024-05-23 | |||
| 23:14:19 | opendevreview | Oria Weng proposed openstack/python-openstackclient master: Identity: Migrate 'role assignment' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/910664 | |
| #openstack-sdks - 2024-05-24 | |||
| 20:42:08 | opendevreview | Brian Haley proposed openstack/python-openstackclient master: Remove admin only text from 'port create' help text https://review.opendev.org/c/openstack/python-openstackclient/+/920421 | |
| #openstack-sdks - 2024-05-26 | |||
| 09:28:57 | ZonC | yet? | |
| 09:28:57 | ZonC | Hi, I have started working a bit with the openstackSDK and have run into a wall right now and was wondering if anyone could provide some help? The issue I am facing is in my code I want to do a offline migration to a specific host but the compute.migrate_server only take the server to migrate and not any other arguments. Is there anything I am missing in the documentation or do openstackSDK not have support for this as of | |
| 09:30:16 | ZonC | So basically I want to be able to target a host when offline migrate. Like the SDK does in the live_migrate_server. | |
| #openstack-sdks - 2024-05-27 | |||
| 07:03:21 | opendevreview | Rajat Dhasmana proposed openstack/openstacksdk master: WIP: Add support for default volume types https://review.opendev.org/c/openstack/openstacksdk/+/920464 | |
| 07:04:54 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: WIP: Add support for default volume type commands https://review.opendev.org/c/openstack/python-openstackclient/+/920465 | |
| 10:48:16 | dtantsur | I wonder what the chances are that https://opendev.org/openstack/openstacksdk/commit/2b4aeff6d3972761da9c4a6a8b696103664a758b causes memory leaks in the code that worked before | |
| 10:57:56 | frickler | dtantsur: do you actually see some memory leak or is this just a theoretical question? | |
| 10:58:34 | dtantsur | frickler: I do see ironic-inspector leaking memory. Taking periodical snapshows from Python's tracealloc led me here: https://paste.opendev.org/show/b9HRCbkxqtPrZ3yU0TCH/ | |
| 10:58:48 | dtantsur | but after looking at lines, I'm not longer sure it's related to __del__ | |
| 10:59:28 | dtantsur | I'm now wondering if we somehow make CONF grow infinitely | |
| 11:01:22 | dtantsur | At the very least, it looks like we're modifying global defaults.. | |
| 11:01:40 | dtantsur | mmm, or not really, it has copy() everywhere | |
| 11:09:37 | frickler | dtantsur: which version are you using? there was a leak fixed a short time later https://review.opendev.org/c/openstack/openstacksdk/+/890781 | |
| 11:10:09 | dtantsur | frickler: it's possible that I don't have the newest version, let me check | |
| 11:10:54 | dtantsur | alas, I have this change already | |
| 12:45:56 | opendevreview | Jens Harbott proposed openstack/openstackclient master: zuul: Drop retired repos from required-projects https://review.opendev.org/c/openstack/openstackclient/+/920512 | |
| 13:26:28 | opendevreview | Dmitry Tantsur proposed openstack/openstacksdk master: Do not create a class in runtime on a potentially hot path https://review.opendev.org/c/openstack/openstacksdk/+/920541 | |
| 13:26:40 | dtantsur | Okay, this is the first, unfortunately by far the smallest, part of the puzzle ^^ | |
| 14:01:06 | dtantsur | frickler, so, is now calling close() on a Connection mandatory? I don't think it was always the case. | |
| 14:04:57 | dtantsur | Honestly, using atexit smells badly. Are you sure you guys considered long-running users of openstacksdk? | |
| 14:10:06 | frickler | dtantsur: I've so far successfully avoided to dig into performance relevant details of python implementations, will defer to gtema and stephenfin | |
| 14:10:54 | gtema | dtantsur - token caching currently relies on calling close | |
| 14:11:08 | dtantsur | gtema: it's off by default though? | |
| 14:11:12 | dtantsur | I mean, the caching? | |
| 14:11:27 | gtema | well, yes - it is config option | |
| 14:11:34 | dtantsur | At the very least, you need a very, VERY ugly warning in the release notes | |
| 14:11:34 | gtema | and off by default | |
| 14:11:58 | dtantsur | I don't think we're the only people who don't call close() | |
| 14:12:24 | gtema | currently the cli does so (since that is the main user of caching) | |
| 14:13:20 | dtantsur | gtema: related question: are Connections long lived and (green) thread safe? Can I just cache one for the lifetime of ironic-inspector? | |
| 14:13:55 | gtema | what is this lifetime? I have certain usecases with connections living multiple months | |
| 14:14:12 | dtantsur | gtema: it's an OpenStack service. Weeks, months, in a very unlikely case years :) | |
| 14:14:13 | gtema | should be thread safe | |
| 14:15:46 | dtantsur | in any case, it's a pity that the change went in without a release note (or am I not looking carefully enough?) | |
| 14:16:00 | gtema | this is there since years | |
| 14:16:15 | dtantsur | last year, to be precise | |
| 14:17:02 | gtema | ok, could be, time is relative | |
| 14:17:07 | gtema | :) | |
| 14:19:15 | dtantsur | gtema: also for your attention: https://review.opendev.org/c/openstack/openstacksdk/+/920541 | |
| 14:19:37 | gtema | ack | |
| 14:36:21 | opendevreview | Merged openstack/openstackclient master: zuul: Drop retired repos from required-projects https://review.opendev.org/c/openstack/openstackclient/+/920512 | |
| 21:00:16 | opendevreview | Merged openstack/openstackclient master: Remove clients of retired projects https://review.opendev.org/c/openstack/openstackclient/+/919857 | |
| #openstack-sdks - 2024-05-28 | |||
| 12:31:42 | opendevreview | Merged openstack/python-openstackclient master: Remove admin only text from 'port create' help text https://review.opendev.org/c/openstack/python-openstackclient/+/920421 | |
| 14:03:38 | 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 | |
| 18:57:13 | opendevreview | Merged openstack/openstacksdk master: Allow project switching for Designate API https://review.opendev.org/c/openstack/openstacksdk/+/899027 | |
| #openstack-sdks - 2024-05-29 | |||
| 16:25:29 | stephenfin | M0weng[m]: Took a look at that error message: it's not keystoneclient's fault but rather the fault of '_process_identity_and_resource_options' in 'openstackclient/identity/v3/role.py' | |
| 16:26:31 | stephenfin | M0weng[m]: It's an if-else ladder and there's a missing fallthrough/else case | |
| 16:28:57 | stephenfin | M0weng[m]: IMO, it would be better if we started nesting and had three cases: if user: {do stuff}; elif project: {do stuff}; else {fail} | |
| 16:30:37 | stephenfin | and then another one for if system: {do stuff}; elif project: {do stuff}; elif domain; {do stuff}; else {fail} | |
| 16:30:56 | stephenfin | not sure if the else in that second one is necessary or not. I'll let you figure that out | |
| 17:23:16 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: identity: Migrate 'service' commands to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/906395 | |
| 21:46:10 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Add support for default volume type commands https://review.opendev.org/c/openstack/python-openstackclient/+/920465 | |
| #openstack-sdks - 2024-05-30 | |||
| 07:38:48 | mikal | e can do to make this less of a thing? | |
| 07:38:48 | mikal | Hey, twice now openstacksdk has been broken for python3.8 by mypy changes. This seems to be happening because there isn't any CI for that python version upstream, despite python 3.8 being in the support matrix for 2024.1. I get that the universe is imperfect, but I am mostly catching these by pure luck in a downstream project at the moment. Apart from playing whackamole when I find them, is there something w | |