| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2021-02-18 | |||
| 14:36:30 | gouthamr | erm, why? | |
| 14:36:43 | gtema | as a user you should be abstracted from that | |
| 14:36:50 | gtema | we did lot to make this transparent | |
| 14:37:09 | gtema | user doesn't need to know in which MV function was added | |
| 14:37:29 | gtema | he just says - I want X, and SDK says, sorry, X is not supported in your case | |
| 14:37:49 | gouthamr | i see, yes - that'd be the case for most users | |
| 14:37:51 | gtema | OSC is in it's side also getting MV handling "dropped" | |
| 14:38:35 | gtema | on the resource level you specify which MV it corresponds to and all operations on it automatically get this MV used (unless server doesn't support it) | |
| 14:38:59 | tosky | talking about that (sorry for jumping in)... can we move forward with this, or an equivalent solution? https://review.opendev.org/c/openstack/python-openstackclient/+/590807 | |
| 14:39:06 | gtema | on the functions side you implement a check (or better to say: MV x.y is required for this to work) | |
| 14:39:21 | tosky | (if it's unrelated I'll wait of course; that's a major pain point in "selling" OSC internally in cinder) | |
| 14:39:44 | gtema | we move a bit into different direction, gimme a sec | |
| 14:40:34 | gtema | https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/compute/v2/hypervisor.py#L90 | |
| 14:41:03 | gtema | OSC user just tells: I want this and that. SDK verifies with the server whether it is possible or not | |
| 14:41:55 | gtema | for cinder our NDSU students will complete SDK part and we can move OSC to use SDK | |
| 14:42:08 | gtema | with that MV handling will pretty much be offloaded from users completely | |
| 14:42:31 | gtema | we are doing this currently for nova, albeit with no big progress in last month | |
| 14:44:03 | gtema | another example of tricky MV implementation is https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/compute/v2/_proxy.py#L1807 | |
| 14:44:21 | gtema | this literally was dropped from OSC and handled in SDK | |
| 14:45:15 | gtema | further questions about MV? | |
| 14:45:35 | tosky | I understand that's an internal restructuring, but when it's time to go back to the cinder part of OSC, it would be better to advertise that in the cinder channels as well | |
| 14:45:35 | amotoki | OSC user still needs to specify MV, right? | |
| 14:45:50 | gtema | amotoki - user CAN, but he doesn't need | |
| 14:46:04 | gtema | highest possible will be used | |
| 14:46:16 | amotoki | gtema: so does user need to specify the major version only? | |
| 14:46:30 | tosky | (and see if anything can be fixed there: apart from this supposed "microversions are not supported" claims which pop up from time to time, another complain is about some choices in the parameters which may not match the expectations on the cinder side) | |
| 14:46:42 | gtema | it depends on the service catalog - pretty much it "picks" major version for us | |
| 14:47:22 | gouthamr | i don't fully understand the version negotiation aspect in https://opendev.org/openstack/openstacksdk/src/branch/master/openstack/utils.py#L98 - i'll read up more | |
| 14:47:29 | gtema | tosky, I know this pops up as a complain often | |
| 14:47:39 | gouthamr | in the meanwhile, i'll submit a patch dropping the versions resource | |
| 14:47:45 | gouthamr | from the manila implementation' | |
| 14:47:55 | gtema | and we thought it make not much sense to continue convincing people, we just need to start doing things | |
| 14:48:42 | gtema | I was hoping to complete together with stephenfin nova part this cycle to be a show case for other teams | |
| 14:48:54 | gtema | not sure, however, we will be in time (busy times) | |
| 14:49:09 | amotoki | gtema: thanks. I will try to follow it up. | |
| 14:49:14 | gtema | cool | |
| 14:49:47 | gtema | summary: SDK negotiates it's own max version, max version supported by server and the one optionally selected by the user | |
| 14:49:53 | gtema | and picks the highest possible | |
| 14:50:17 | gouthamr | ^ thanks for that clarification | |
| 14:50:19 | gtema | this gives user still possibility to fall back to the older one | |
| 14:50:44 | gouthamr | one last thing regarding this $topic | |
| 14:50:55 | gouthamr | #link https://review.opendev.org/c/openstack/keystoneauth/+/774042 (Specify manila microversion header) | |
| 14:51:08 | gouthamr | ^ we need this to merge to get things to work for manila | |
| 14:51:25 | gouthamr | so if there are keystoneauth reviewers in the room, i'd love some help :) | |
| 14:52:15 | gtema | sadly nobody | |
| 14:52:19 | diablo_rojo | Lol | |
| 14:52:23 | gouthamr | haha :) | |
| 14:52:35 | gouthamr | okay i'll ask in the keystone channel | |
| 14:52:38 | stephenfin | gouthamr: You probably want to bring it up on #openstack-keystone | |
| 14:52:40 | stephenfin | yeah | |
| 14:52:46 | amotoki | gouthamr: one question | |
| 14:52:50 | gtema | I'm wondering there is nothing for cinder over there | |
| 14:53:04 | amotoki | gouthamr: don't we use OpenStack-API-Version: <service> <version> for such case? | |
| 14:53:14 | stephenfin | fwiw, I've got very little feedback on any of my keystone-related stuff lately, but if you keep prodding hopefully you'll get somewhere | |
| 14:54:01 | gouthamr | amotoki: not with manila unfortunately, i can follow up right after this meeting - but, the gist is that we missed the boat on that standardization, and there's a lot of existing clouds out there to update now | |
| 14:54:01 | amotoki | rather than OpenStack-Compute-API-Version. I need to find the source on what we discussed so far though. | |
| 14:54:14 | gouthamr | stephenfin: ack, will do :) | |
| 14:54:36 | stephenfin | zimmerry, diablo_rojo: https://review.opendev.org/c/openstack/openstacksdk/+/745375 | |
| 14:54:41 | gouthamr | we've six minutes, and i'd like to say hi to other diablo_rojo prodigies :) | |
| 14:55:26 | diablo_rojo | Thanks stephenfin! | |
| 14:55:50 | diablo_rojo | Oh I don't think all the NDSU students are here just wanted to make the logs aware that these other folks exist :) | |
| 14:56:56 | diablo_rojo | I saw someone from windriver starting to work on Cinder things and I wanted to make sure they are aware of what the NDSU students are doing | |
| 14:57:05 | gtema | good, do we have something else? | |
| 14:58:03 | diablo_rojo | But I don't know thiago's irc nick so I don't know if they are here | |
| 14:59:04 | diablo_rojo | So I guess I don't have much else atm. | |
| 14:59:11 | gtema | then we are good to end the meeting | |
| 14:59:21 | diablo_rojo | gtema, yep I believe so | |
| 14:59:25 | gtema | thanks everybody for participation | |
| 14:59:26 | diablo_rojo | thanks for running it! | |
| 14:59:36 | gtema | was more than I was actually expecting ;-) | |
| 14:59:48 | gtema | #endmeeting | |
| 14:59:50 | openstack | Meeting ended Thu Feb 18 14:59:48 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 14:59:51 | openstack | Minutes: http://eavesdrop.openstack.org/meetings/sdk_osc/2021/sdk_osc.2021-02-18-14.05.html | |
| 14:59:52 | openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/sdk_osc/2021/sdk_osc.2021-02-18-14.05.txt | |
| 14:59:53 | openstack | Log: http://eavesdrop.openstack.org/meetings/sdk_osc/2021/sdk_osc.2021-02-18-14.05.log.html | |
| 14:59:59 | markypharaoh[m] | cheers | |
| 15:00:20 | gouthamr | thanks everyone! | |
| 15:00:25 | amotoki | thanks! | |
| 15:00:26 | diablo_rojo | thanks for attending markypharaoh[m] :) NicoleBU24 and ashrod98[m] as well :) | |
| 15:00:52 | diablo_rojo | gouthamr, zimmerry is the OSU student :) | |
| 15:01:10 | amotoki | we moved into too specific discussions and I am a bit afraid we can have good input for them..... | |
| 15:01:11 | ashrod98[m] | of course! looking forward to the next one :D | |
| 15:01:13 | markypharaoh[m] | ofcourse... excited to work with y'all | |
| 15:01:45 | gtema | see you | |
| 15:01:54 | gtema | (and some already in an hour) | |
| 15:02:10 | ashrod98[m] | enjoy the rest of your day! | |
| 15:02:23 | diablo_rojo | gtema, you and stephenfin will be sick of me before long lol | |
| 15:02:32 | gouthamr | zimmerry: o/ | |
| 15:02:36 | gtema | :) | |
| 15:02:38 | gouthamr | hello, and welcome! | |
| 15:03:22 | zimmerry | hello! | |
| 15:03:49 | diablo_rojo | gouthamr, zimmerry is working on Nova stuff for us :) | |
| 15:04:14 | diablo_rojo | Doesn't look like any of the NDSU students are currently on IRC> | |
| 15:04:33 | openstackgerrit | Takashi Kajinami proposed openstack/openstackclient master: Remove retired Karbor usage https://review.opendev.org/c/openstack/openstackclient/+/776477 | |
| 15:04:37 | gtema | they are still sleeping ;-) | |
| 15:06:34 | diablo_rojo | gtema, definitely possible. I would be sleeping if I were them lol | |
| 15:07:18 | diablo_rojo | unfortunately, I am now in a TC meeting and then we meet and then I meet with the BU students and then I have a storyboard meeting and then I have one other meeting before I can nap lol | |
| 16:38:41 | openstackgerrit | Goutham Pacha Ravi proposed openstack/keystoneauth master: Specify manila microversion header https://review.opendev.org/c/openstack/keystoneauth/+/774042 | |
| 16:48:39 | openstackgerrit | Goutham Pacha Ravi proposed openstack/openstacksdk master: Add shared file systems support https://review.opendev.org/c/openstack/openstacksdk/+/773556 | |
| 17:01:09 | openstackgerrit | Goutham Pacha Ravi proposed openstack/openstacksdk master: Add a manila functional check job https://review.opendev.org/c/openstack/openstacksdk/+/776312 | |