| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2023-06-20 | |||
| 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 | |
| 18:41:15 | opendevreview | Danila Balagansky proposed openstack/openstacksdk master: Add missing `return` in `upload_volume_to_image` https://review.opendev.org/c/openstack/openstacksdk/+/887221 | |
| 18:50:14 | opendevreview | Anvi Joshi proposed openstack/openstacksdk master: Implemented methods for share metadata https://review.opendev.org/c/openstack/openstacksdk/+/886123 | |
| 22:41:55 | opendevreview | Merged openstack/openstacksdk master: Fix broken python3.6 support https://review.opendev.org/c/openstack/openstacksdk/+/887137 | |
| #openstack-sdks - 2023-06-29 | |||
| 03:16:32 | ricolin | stephenfin: gtema mnaser ralonsoh thanks for the review, I just propose a release change for it https://review.opendev.org/c/openstack/releases/+/887242 | |
| 08:52:40 | opendevreview | Lajos Katona proposed openstack/openstacksdk master: VPN: add missing fields to VpnIpsecPolicy https://review.opendev.org/c/openstack/openstacksdk/+/886822 | |
| 10:04:18 | opendevreview | Slawek Kaplonski proposed openstack/openstacksdk master: Add support for default security group rules in SDK https://review.opendev.org/c/openstack/openstacksdk/+/887262 | |
| 10:36:42 | opendevreview | Yamato Tanaka proposed openstack/openstacksdk master: Do exact-matching when finding one instance by name https://review.opendev.org/c/openstack/openstacksdk/+/887127 | |
| 12:26:33 | opendevreview | Yamato Tanaka proposed openstack/openstacksdk master: Do exact-matching when finding one instance by name https://review.opendev.org/c/openstack/openstacksdk/+/887127 | |
| 12:30:02 | opendevreview | Yamato Tanaka proposed openstack/openstacksdk master: Do exact-matching when finding one instance by name https://review.opendev.org/c/openstack/openstacksdk/+/887127 | |
| 13:59:35 | opendevreview | Slawek Kaplonski proposed openstack/openstacksdk master: Add support for default security group rules in SDK https://review.opendev.org/c/openstack/openstacksdk/+/887262 | |
| 21:30:04 | opendevreview | Anvi Joshi proposed openstack/openstacksdk master: Implemented methods for share metadata https://review.opendev.org/c/openstack/openstacksdk/+/886123 | |
| #openstack-sdks - 2023-06-30 | |||
| 08:13:50 | 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:13:41 | opendevreview | Slawek Kaplonski proposed openstack/python-openstackclient master: Add support for default security group rule CRUDs https://review.opendev.org/c/openstack/python-openstackclient/+/887364 | |
| 10:25:06 | opendevreview | Slawek Kaplonski proposed openstack/openstacksdk master: Add support for default security group rules in SDK https://review.opendev.org/c/openstack/openstacksdk/+/887262 | |
| 10:27:58 | opendevreview | Slawek Kaplonski proposed openstack/python-openstackclient master: Add support for default security group rule CRUDs https://review.opendev.org/c/openstack/python-openstackclient/+/887364 | |
| 11:44:49 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Migrate volume transfer commands https://review.opendev.org/c/openstack/python-openstackclient/+/887368 | |
| 15:09:33 | opendevreview | Lajos Katona proposed openstack/openstacksdk master: dd SFC to SDK https://review.opendev.org/c/openstack/openstacksdk/+/887387 | |
| 15:11:40 | opendevreview | Lajos Katona proposed openstack/openstacksdk master: Add SFC to SDK https://review.opendev.org/c/openstack/openstacksdk/+/887387 | |
| #openstack-sdks - 2023-07-03 | |||
| 08:00:34 | opendevreview | Mridula Joshi proposed openstack/openstacksdk master: Adding support for image upload https://review.opendev.org/c/openstack/openstacksdk/+/881939 | |
| 09:54:06 | opendevreview | Mridula Joshi proposed openstack/openstacksdk master: Add image metadef object operations https://review.opendev.org/c/openstack/openstacksdk/+/858350 | |
| 13:24:31 | opendevreview | Elod Illes proposed openstack/openstacksdk stable/wallaby: Do not pin openstacksdk to master https://review.opendev.org/c/openstack/openstacksdk/+/885898 | |
| 13:26:41 | opendevreview | Elod Illes proposed openstack/openstacksdk stable/victoria: Do not pin openstacksdk to master https://review.opendev.org/c/openstack/openstacksdk/+/885901 | |
| #openstack-sdks - 2023-07-04 | |||
| 14:37:07 | opendevreview | Mridula Joshi proposed openstack/python-openstackclient master: image: Add support for cache commands https://review.opendev.org/c/openstack/python-openstackclient/+/878631 | |
| 16:14:27 | opendevreview | Rajat Dhasmana proposed openstack/openstacksdk master: Add volume transfer support [2/2] https://review.opendev.org/c/openstack/openstacksdk/+/887635 | |
| 16:14:27 | opendevreview | Rajat Dhasmana proposed openstack/openstacksdk master: Add volume transfer support [1/2] https://review.opendev.org/c/openstack/openstacksdk/+/887037 | |