| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2023-06-28 | |||
| 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 | |
| 16:14:59 | opendevreview | Rajat Dhasmana proposed openstack/python-openstackclient master: Migrate volume transfer commands https://review.opendev.org/c/openstack/python-openstackclient/+/887636 | |
| 20:06:26 | opendevreview | Rajat Dhasmana proposed openstack/openstacksdk master: Add volume transfer support [2/2] https://review.opendev.org/c/openstack/openstacksdk/+/887635 | |
| 20:06:26 | opendevreview | Rajat Dhasmana proposed openstack/openstacksdk master: Add volume transfer support [1/2] https://review.opendev.org/c/openstack/openstacksdk/+/887037 | |
| #openstack-sdks - 2023-07-05 | |||
| 09:45:46 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: image: Add support for cache commands https://review.opendev.org/c/openstack/python-openstackclient/+/878631 | |
| 10:04:02 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Migrate 'volume summary' command to SDK https://review.opendev.org/c/openstack/python-openstackclient/+/884398 | |
| 10:05:25 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Migrate volume transfer commands https://review.opendev.org/c/openstack/python-openstackclient/+/887368 | |
| 10:05:30 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Migrate volume transfer commands https://review.opendev.org/c/openstack/python-openstackclient/+/887636 | |
| 11:07:49 | opendevreview | Merged openstack/python-openstackclient master: Allow multiple `--remove-tag` in `project set` https://review.opendev.org/c/openstack/python-openstackclient/+/885566 | |
| 14:19:51 | opendevreview | Lajos Katona proposed openstack/openstacksdk master: Add SFC to SDK https://review.opendev.org/c/openstack/openstacksdk/+/887387 | |
| 14:40:13 | diablo_rojo_phone | gtema & stephenfin I won't be in the meeting with Antonia today - am in AU till the end of the month, then I'll be back stateside and back in the meetings. Let me know if you need anything and I will reply as soon as I can! | |
| 14:40:34 | stephenfin | ack | |
| 14:41:24 | gtema | I am actually also on vacation in Ireland, so I am also not available | |
| 16:47:40 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: Add image metadef resource type association commands 'create', 'list', 'delete' https://review.opendev.org/c/openstack/python-openstackclient/+/885375 | |
| 19:30:18 | opendevreview | Antonia Gaete proposed openstack/python-openstackclient master: Add image metadef resource type command 'list' https://review.opendev.org/c/openstack/python-openstackclient/+/883279 | |
| 21:08:59 | opendevreview | Douglas Mendizábal proposed openstack/python-openstackclient stable/2023.1: Fix "access rule" commands to only use ID https://review.opendev.org/c/openstack/python-openstackclient/+/887742 | |
| 21:08:59 | opendevreview | Douglas Mendizábal proposed openstack/python-openstackclient stable/2023.1: Fix "access rule" commands to only use ID https://review.opendev.org/c/openstack/python-openstackclient/+/887742 | |
| 21:42:37 | opendevreview | Carlos Eduardo proposed openstack/openstacksdk master: Adds share group resource to shared file system https://review.opendev.org/c/openstack/openstacksdk/+/879685 | |
| 21:44:16 | opendevreview | Carlos Eduardo proposed openstack/openstacksdk master: Add share group snapshots to shared file systems. https://review.opendev.org/c/openstack/openstacksdk/+/879686 | |
| 22:08:36 | opendevreview | Carlos Eduardo proposed openstack/openstacksdk master: Adds share group resource to shared file system https://review.opendev.org/c/openstack/openstacksdk/+/879685 | |
| 22:12:36 | opendevreview | Carlos Eduardo proposed openstack/openstacksdk master: Add share group snapshots to shared file systems. https://review.opendev.org/c/openstack/openstacksdk/+/879686 | |
| #openstack-sdks - 2023-07-06 | |||
| 03:06:24 | opendevreview | Anvi Joshi proposed openstack/openstacksdk master: Implemented methods for share metadata https://review.opendev.org/c/openstack/openstacksdk/+/886123 | |
| 13:35:47 | opendevreview | Pavlo Shchelokovskyy proposed openstack/openstacksdk master: Fix list of server migrations https://review.opendev.org/c/openstack/openstacksdk/+/887847 | |
| 14:04:54 | opendevreview | Mridula Joshi proposed openstack/python-openstackclient master: image: Add support for cache commands https://review.opendev.org/c/openstack/python-openstackclient/+/878631 | |
| 14:44:21 | opendevreview | Lajos Katona proposed openstack/openstacksdk master: Add SFC to SDK https://review.opendev.org/c/openstack/openstacksdk/+/887387 | |
| 15:46:03 | opendevreview | Douglas Mendizábal proposed openstack/python-openstackclient stable/2023.1: Fix pep issue in the network service provider https://review.opendev.org/c/openstack/python-openstackclient/+/887812 | |
| 15:46:24 | opendevreview | Douglas Mendizábal proposed openstack/python-openstackclient stable/2023.1: Fix "access rule" commands to only use ID https://review.opendev.org/c/openstack/python-openstackclient/+/887742 | |
| 16:22:17 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Add image metadef resource type command 'list' https://review.opendev.org/c/openstack/python-openstackclient/+/883279 | |
| 17:06:26 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: doc: Split image command documentation https://review.opendev.org/c/openstack/python-openstackclient/+/887863 | |
| 17:06:26 | opendevreview | Stephen Finucane proposed openstack/python-openstackclient master: Add image metadef resource type command 'list' https://review.opendev.org/c/openstack/python-openstackclient/+/883279 | |
| 18:02:47 | opendevreview | Merged openstack/openstacksdk master: volume: Add missing attributes to Extension https://review.opendev.org/c/openstack/openstacksdk/+/885132 | |
| 18:09:52 | andrewbogott_ | I'm very interested in someone's thoughts about whether or not this is a bug. https://storyboard.openstack.org/#!/story/2010784 | |
| 18:10:27 | andrewbogott_ | I changed all my workflows to use clouds.yaml and now a really regret it, thanks to that issue. Maybe clouds.yaml is out of fashion and I should go back to relying on env settings? | |
| 18:10:35 | andrewbogott_ | s/now a/now I/g | |
| 18:54:00 | frickler | andrewbogott_: to me the story looks incomplete, it ends with " | |
| 18:54:06 | frickler | $ source" | |
| 18:54:08 | frickler | oops | |
| 18:54:24 | andrewbogott_ | I don't understand why storyboard truncates the view, if I click the 'edit' button I can see all the text | |
| 18:54:27 | frickler | anyway, I'm not sure what it actually is that is bothering you | |
| 18:55:08 | frickler | can you put the whole text into paste.openstack.org and link it in a comment? | |
| 18:55:23 | andrewbogott_ | sure | |
| 18:55:28 | frickler | also clouds.yaml it the future IMO, env vars should be deprecated | |
| 18:57:00 | andrewbogott_ | ok, that's what I thought. In that case I think that we have quite a few commands that need a --project arg added. | |
| 18:57:20 | andrewbogott_ | Or the parser could load clouds.yaml first and env second (but that would still be relying on env) | |
| 18:58:00 | frickler | oh, I can see the text when I enter edit mode myself. my guess is that the "<script" thing gets parsed as html tag somehow, maybe fungi can have a look? | |
| 18:58:18 | andrewbogott_ | oh good idea, let me try messing with quotes | |
| 18:59:00 | frickler | what about having one cloud definition per project as workaround? you could easily script setting that up as clouds.yaml | |
| 18:59:16 | andrewbogott_ | how's that, better? | |
| 18:59:44 | andrewbogott_ | frickler: that would work but seems a bit silly. I have hundreds of tenants in my cloud. | |
| 19:00:11 | andrewbogott_ | Also, my understanding is that much of the reason why 'scope' was abandoned as a concept was because it broke the 'global cred' workflows that many clouds use. clouds.yaml also breaks them. | |