| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2023-06-15 | |||
| 13:02:34 | opendevreview | Jan Hartkopf proposed openstack/openstacksdk master: Allow resources to be skipped on project cleanup https://review.opendev.org/c/openstack/openstacksdk/+/878646 | |
| #openstack-sdks - 2023-06-16 | |||
| 12:30:50 | opendevreview | Fabian Wiesel proposed openstack/openstacksdk master: Add support for targeting host in a cold-migration https://review.opendev.org/c/openstack/openstacksdk/+/881552 | |
| #openstack-sdks - 2023-06-18 | |||
| 10:34:41 | opendevreview | Artem Goncharov proposed openstack/python-openstackclient master: Adopt sdk_fakes for compute.aggregate https://review.opendev.org/c/openstack/python-openstackclient/+/885491 | |
| 11:11:24 | opendevreview | Artem Goncharov proposed openstack/python-openstackclient master: Adopt sdk_fakes for compute.test_console https://review.opendev.org/c/openstack/python-openstackclient/+/885492 | |
| 11:11:24 | opendevreview | Artem Goncharov proposed openstack/python-openstackclient master: Adopt sdk_fakes for compute.test_keypair https://review.opendev.org/c/openstack/python-openstackclient/+/885494 | |
| 11:11:25 | opendevreview | Artem Goncharov proposed openstack/python-openstackclient master: Adopt sdk_fakes for compute.test_service https://review.opendev.org/c/openstack/python-openstackclient/+/885495 | |
| 11:11:25 | opendevreview | Artem Goncharov proposed openstack/python-openstackclient master: Adopt sdk_fakes for compute.test_usage https://review.opendev.org/c/openstack/python-openstackclient/+/885496 | |
| 11:11:26 | opendevreview | Artem Goncharov proposed openstack/python-openstackclient master: Adopt sdk_fakes for compute.test_hypervisor https://review.opendev.org/c/openstack/python-openstackclient/+/885517 | |
| 11:11:26 | opendevreview | Artem Goncharov proposed openstack/python-openstackclient master: Adopt sdk_fakes for compute.test_server_group https://review.opendev.org/c/openstack/python-openstackclient/+/885766 | |
| 11:11:27 | opendevreview | Artem Goncharov proposed openstack/python-openstackclient master: Adopt sdk_fakes for compute.test_server_volume https://review.opendev.org/c/openstack/python-openstackclient/+/885768 | |
| 14:53:43 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: fix connection.Connection finalizer https://review.opendev.org/c/openstack/openstacksdk/+/886341 | |
| 16:29:53 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: fix connection.Connection finalizer https://review.opendev.org/c/openstack/openstacksdk/+/886341 | |
| #openstack-sdks - 2023-06-19 | |||
| 09:30:37 | opendevreview | Artem Goncharov proposed openstack/python-openstackclient master: Adopt sdk_fakes for compute.test_flavor https://review.opendev.org/c/openstack/python-openstackclient/+/885493 | |
| 13:16:13 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: fix connection.Connection finalizer https://review.opendev.org/c/openstack/openstacksdk/+/886341 | |
| 13:39:28 | opendevreview | Artem Goncharov proposed openstack/openstacksdk master: fix flavor.swap attribute type https://review.opendev.org/c/openstack/openstacksdk/+/885498 | |
| 21:45:04 | opendevreview | Anvi Joshi proposed openstack/openstacksdk master: Edited functional tests and proxy methods for share metadata and changed share.py to use MetadataMixin class https://review.opendev.org/c/openstack/openstacksdk/+/886123 | |
| #openstack-sdks - 2023-06-20 | |||
| 09:31:24 | gtema | stephenfin: would you pls have a look at https://review.opendev.org/c/openstack/openstacksdk/+/886341 I need it for https://review.opendev.org/c/openstack/devstack/+/886339 but somehow the job is not really respecting depends-on for sdk | |
| 11:01:20 | opendevreview | Merged openstack/openstacksdk master: Adds support for node hardware inventory https://review.opendev.org/c/openstack/openstacksdk/+/886008 | |
| 11:30:48 | frickler | gtema: I think that devstack may intentionally for stability reasons be using only released sdk versions except in the sdk/osc jobs. feel free to discuss in the qa channel if you want to change that | |
| 11:38:48 | stephenfin | gtema: done | |
| 11:40:43 | stephenfin | gtema: Can you look at https://review.opendev.org/c/openstack/python-openstackclient/+/885761 I've seen a few failures like that recently | |
| 11:41:00 | stephenfin | e.g. https://review.opendev.org/c/openstack/osc-lib/+/860890 | |
| 12:21:26 | seongsoocho | Hi, I forgot the etherpad link of this forum "Managing OSC and SDK as multi-project deliverables" Does anyone know about this? | |
| 13:13:28 | opendevreview | Merged openstack/openstacksdk master: fix connection.Connection finalizer https://review.opendev.org/c/openstack/openstacksdk/+/886341 | |
| 13:16:26 | gtema | seongsoocho: https://etherpad.opendev.org/p/oscsdk-vancouver-forum-2023 | |
| 13:17:12 | seongsoocho | gtema: thanks | |
| 13:17:46 | gtema | stephenfin: thks a lot. I also processed your change | |
| 14:31:31 | opendevreview | Merged openstack/python-openstackclient master: tests: Use a new project for quota tests https://review.opendev.org/c/openstack/python-openstackclient/+/885761 | |
| 14:55:24 | lajoskatona | gtema, stephenfin: Hi, do you plan a new SDK release in the coming weeks? | |
| 15:17:25 | stephenfin | lajoskatona: Good idea. Done here https://review.opendev.org/c/openstack/releases/+/886508 | |
| 15:17:58 | gtema | stephenfin, can we get fakes generator into the release? that will unblock certain stuff in OSC patches | |
| 15:19:20 | stephenfin | gtema: I need to review that again and won't get to it today. Could we cut 1.3.0 and do 1.4.0 once that merges? Releases are cheap | |
| 15:19:35 | gtema | okay | |
| 15:20:19 | gtema | they are cheap, but not immediate ;-) | |
| 17:15:54 | lajoskatona | stephenfin: thanks | |
| #openstack-sdks - 2023-06-21 | |||
| 13:31:19 | opendevreview | Rodion Gyrbu proposed openstack/openstacksdk master: Add missing `force` parameter https://review.opendev.org/c/openstack/openstacksdk/+/886637 | |
| 14:16:27 | opendevreview | Rodion Gyrbu proposed openstack/openstacksdk master: Add missing `force` parameter https://review.opendev.org/c/openstack/openstacksdk/+/886637 | |
| 15:04:18 | opendevreview | Jeremy Stanley proposed openstack/openstacksdk master: Replace appdirs usage with platformdirs https://review.opendev.org/c/openstack/openstacksdk/+/886643 | |
| 17:43:49 | opendevreview | Stephen Finucane proposed openstack/openstacksdk master: Revert "Temporarily disable volume and os_image functional tests" https://review.opendev.org/c/openstack/openstacksdk/+/886655 | |
| 18:06:22 | opendevreview | Carlos Eduardo proposed openstack/openstacksdk master: Adds share group resource to shared file system https://review.opendev.org/c/openstack/openstacksdk/+/879685 | |
| 18:07:18 | opendevreview | Carlos Eduardo proposed openstack/openstacksdk master: Add share group snapshots to shared file systems. https://review.opendev.org/c/openstack/openstacksdk/+/879686 | |
| 18:38:23 | opendevreview | Anvi Joshi proposed openstack/openstacksdk master: Fixed get, create, and list functional tests, added update and delete functional tests, and edited proxy methods for share metadata https://review.opendev.org/c/openstack/openstacksdk/+/886123 | |
| #openstack-sdks - 2023-06-22 | |||
| 07:32:59 | opendevreview | Rodion Gyrbu proposed openstack/openstacksdk master: Add missing `force` parameter https://review.opendev.org/c/openstack/openstacksdk/+/886637 | |
| 09:53:33 | opendevreview | Rodion Gyrbu proposed openstack/openstacksdk master: Add missing `force` parameter https://review.opendev.org/c/openstack/openstacksdk/+/886637 | |
| 13:27:00 | opendevreview | Jeremy Stanley proposed openstack/openstacksdk master: Replace appdirs usage with platformdirs https://review.opendev.org/c/openstack/openstacksdk/+/886643 | |
| #openstack-sdks - 2023-06-23 | |||
| 07:19:10 | opendevreview | Rodion Gyrbu proposed openstack/openstacksdk master: Add missing `force` parameter https://review.opendev.org/c/openstack/openstacksdk/+/886637 | |
| 08:41:50 | opendevreview | Rodion Gyrbu proposed openstack/openstacksdk master: Add missing `force` parameter https://review.opendev.org/c/openstack/openstacksdk/+/886637 | |
| 09:19:15 | opendevreview | Rodion Gyrbu proposed openstack/openstacksdk master: Add missing `force` parameter https://review.opendev.org/c/openstack/openstacksdk/+/886637 | |
| 09:59:50 | opendevreview | Rodion Gyrbu proposed openstack/openstacksdk master: Add missing `force` parameter https://review.opendev.org/c/openstack/openstacksdk/+/886637 | |
| 10:30:01 | opendevreview | Rodion Gyrbu proposed openstack/openstacksdk master: Add missing `force` parameter https://review.opendev.org/c/openstack/openstacksdk/+/886637 | |
| 13:00:42 | opendevreview | Mridula Joshi proposed openstack/python-openstackclient master: Adds command ``image metadef object list`` https://review.opendev.org/c/openstack/python-openstackclient/+/886821 | |
| 13:17:47 | opendevreview | Lajos Katona proposed openstack/openstacksdk master: VPN: add missing fields to VpnIpsecPolicy https://review.opendev.org/c/openstack/openstacksdk/+/886822 | |
| 14:17:53 | opendevreview | Sofia Enriquez proposed openstack/python-openstackclient master: Add volume manage command https://review.opendev.org/c/openstack/python-openstackclient/+/886828 | |
| 15:06:00 | opendevreview | yugsuo proposed openstack/openstacksdk master: Add tenant_id to network resource https://review.opendev.org/c/openstack/openstacksdk/+/886853 | |
| 20:08:21 | opendevreview | Anvi Joshi proposed openstack/openstacksdk master: Implemented methods for share metadata Change-Id: Ib3c27ec3d0eb102b30a80994d6cc2af4ee78f3dd https://review.opendev.org/c/openstack/openstacksdk/+/886873 | |
| 20:10:20 | opendevreview | Anvi Joshi proposed openstack/openstacksdk master: Implemented methods for share metadata https://review.opendev.org/c/openstack/openstacksdk/+/886123 | |
| #openstack-sdks - 2023-06-24 | |||
| 14:12:06 | opendevreview | Merged openstack/openstacksdk master: update rackspace profile to specify identity version https://review.opendev.org/c/openstack/openstacksdk/+/886109 | |
| #openstack-sdks - 2023-06-26 | |||
| 13:01:00 | opendevreview | Mridula Joshi proposed openstack/openstacksdk master: Add image metadef object operations https://review.opendev.org/c/openstack/openstacksdk/+/858350 | |
| 19:06:43 | opendevreview | Merged openstack/openstacksdk master: Add missing `force` parameter https://review.opendev.org/c/openstack/openstacksdk/+/886637 | |
| #openstack-sdks - 2023-06-27 | |||
| 12:45:24 | opendevreview | Rajat Dhasmana proposed openstack/openstacksdk master: Add volume transfer support https://review.opendev.org/c/openstack/openstacksdk/+/887037 | |
| 12:48:40 | opendevreview | Rajat Dhasmana proposed openstack/openstacksdk master: Add volume transfer support https://review.opendev.org/c/openstack/openstacksdk/+/887037 | |
| 13:40:16 | opendevreview | Mridula Joshi proposed openstack/python-openstackclient master: Adds command ``image metadef object show`` https://review.opendev.org/c/openstack/python-openstackclient/+/887048 | |
| 15:49:47 | opendevreview | Merged openstack/keystoneauth master: Add doc of OAuth 2.0 Mutual-TLS Authenticate https://review.opendev.org/c/openstack/keystoneauth/+/860923 | |
| 18:53:41 | opendevreview | Rajat Dhasmana proposed openstack/openstacksdk master: Add volume transfer support https://review.opendev.org/c/openstack/openstacksdk/+/887037 | |
| #openstack-sdks - 2023-06-28 | |||
| 02:43:31 | opendevreview | Yamato Tanaka proposed openstack/openstacksdk master: Do exact-matching when finding one instance by name https://review.opendev.org/c/openstack/openstacksdk/+/887127 | |
| 06:21:25 | opendevreview | Rico Lin proposed openstack/openstacksdk master: Fix broken python3.6 support https://review.opendev.org/c/openstack/openstacksdk/+/887137 | |
| 07:54:13 | opendevreview | Rico Lin proposed openstack/openstacksdk master: Fix broken python3.6 support https://review.opendev.org/c/openstack/openstacksdk/+/887137 | |
| 09:07:58 | opendevreview | Yamato Tanaka proposed openstack/openstacksdk master: Do exact-matching when finding one instance by name https://review.opendev.org/c/openstack/openstacksdk/+/887127 | |
| 10:07:01 | opendevreview | Mridula Joshi proposed openstack/openstacksdk master: Adding support for image upload https://review.opendev.org/c/openstack/openstacksdk/+/881939 | |
| 12:14:03 | mnaser | stephenfin: could you have a look at https://review.opendev.org/c/openstack/openstacksdk/+/887137 (since that was introduced by https://review.opendev.org/c/openstack/openstacksdk/+/884494) | |
| 12:32:42 | ricolin | ralonsoh: slaweq gtema appreciate if you can help to review on https://review.opendev.org/c/openstack/openstacksdk/+/887137 as it's currently broken for python3.6 | |
| 13:07:48 | ralonsoh | ricolin, I've commented in the patch. I don't think this is the correct fix | |
| 13:09:47 | opendevreview | Rico Lin proposed openstack/openstacksdk master: Revert "utils: Add 'supports_version'" https://review.opendev.org/c/openstack/openstacksdk/+/887068 | |
| 13:18:58 | opendevreview | Rico Lin proposed openstack/openstacksdk master: Revert "utils: Add 'supports_version'" https://review.opendev.org/c/openstack/openstacksdk/+/887068 | |
| 13:51:39 | opendevreview | Rico Lin proposed openstack/openstacksdk master: Fix broken python3.6 support https://review.opendev.org/c/openstack/openstacksdk/+/887137 | |
| 13:52:10 | ricolin | ralonsoh: please review it again, thanks https://review.opendev.org/c/openstack/openstacksdk/+/887137 | |
| 13:59:26 | opendevreview | Rico Lin proposed openstack/openstacksdk master: Fix broken python3.6 support https://review.opendev.org/c/openstack/openstacksdk/+/887137 | |
| 14:08:45 | ricolin | gtema: hey need your review again to the updated one thanks https://review.opendev.org/c/openstack/openstacksdk/+/887137 | |
| 14:09:47 | gtema | sure | |
| 14:16:41 | ralonsoh | ricolin, Stepehn has replied to it | |
| 14:17:35 | stephenfin | I've no issues fixing it now since we do advertise support but can we drop python3.6 and python3.7 in our next major version bump? Both versions are EOL upstream and unsupported by the rest of OpenStack. Also RHEL8 has app streams so there shouldn't be any issues installing a newer Python 3 version. | |
| 14:19:40 | opendevreview | Rico Lin proposed openstack/openstacksdk master: Fix broken python3.6 support https://review.opendev.org/c/openstack/openstacksdk/+/887137 | |
| 14:20:44 | ricolin | stephenfin: updated^^^ | |
| 14:20:49 | ricolin | ralonsoh: ^^^ | |
| 14:20:54 | stephenfin | yup +2 | |
| 14:20:55 | ralonsoh | is reviewed | |
| 14:21:50 | frickler | ricolin: on what platform do you actually need py3.6 support? | |
| 14:22:36 | ricolin | frickler: it was this command triggers this patch https://github.com/openstack/openstacksdk/commit/3cb5bc98573e8eb7d3487121b7b59ebd535035c5 | |
| 14:22:59 | ricolin | s/command/comments/ | |
| 14:24:53 | stephenfin | tbf, if you attempt to install SDK on python3.6 right now you'll get the latest version since python_requires = >=3.6 in setup.cfg. I think we just need to get this fix out the door and in a release and then we should bump 'python_requires' to avoid the issue | |
| 14:29:42 | frickler | I was just wondering whether supporting old clouds would also imply keeping support for old python version for as long as possible | |
| 14:31:39 | stephenfin | We can still support them via stable branches. They just won't get newer features. | |
| 14:37:50 | mnaser | stephenfin, frickler, ricolin: yeah, imho since we advertise py3.6 it makes sense to land this | |
| 14:42:49 | frickler | I'm not questioning that fix, I'm pondering whether we can drop py3.6 support after that or whether it would be better to reintroduce 3.6 testing and keep that running as long as it works, like when opendev drops the images needed for that | |
| 15:07:27 | stephenfin | frickler: What would the advantage be? | |
| 15:11:45 | stephenfin | The two big use cases of sdk are service-driven operations (nova talking to cinder) and user-driven operations (OSC, Ansible, and SDK itself) | |
| 15:11:56 | stephenfin | The former isn't a concern since a released nova will use a release SDK. For the latter, surely people are deploying from environment with more recent toolchains (or the ability to enable them: see RHEL AppStreams)? | |
| 16:27:32 | mnaser | stephenfin: yeah i've just been wondering about people who want to use something like ansible.. but yeah if you're saying rhel appstream is a workaround then sure | |
| 16:28:15 | mnaser | gtema: https://review.opendev.org/c/openstack/openstacksdk/+/887137 round 2 if you dont mind :P | |
| 17:12:37 | ricolin | Hey stephenfin gtema , check job passed now, can you kindly help to give it a +w? thanks https://review.opendev.org/c/openstack/openstacksdk/+/887137 | |