| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-05-24 | |||
| 17:26:57 | sean-k-mooney | ya ok | |
| 17:27:53 | melwitt | I had been looking at it from the context of it also providing a way to disable monkey patching, but due to my lack of understanding of eventlet and mixing with native threads, it did not click for me to think it could have fixed things to monkey patch earlier | |
| 17:28:27 | melwitt | it makes sense when you say it now though.. | |
| 17:28:31 | dansmith | a combination of references to the un-patched library and the patched one could very much be relevant | |
| 17:28:43 | dansmith | and that's what that change was aabout | |
| 17:28:54 | dansmith | and it's also the argument against monkeypatching altogether of course :P | |
| 17:29:42 | sean-k-mooney | its ok stephenfin will reventyly get around to deleteing all the eventlet code like all the ohter stuff he has deleted :) | |
| 17:29:53 | melwitt | yeah. that change was mostly non understandable by my brain | |
| 17:29:56 | sean-k-mooney | but ya we did this for urllib3 eventully | |
| 17:30:00 | sean-k-mooney | *orginally | |
| 17:30:13 | sean-k-mooney | and som eohter service i guess but it makes sesne | |
| 17:30:40 | dansmith | urllib3 being socket-oriented, along with pymysql ... :) | |
| 17:30:58 | sean-k-mooney | we did actully backport this https://review.opendev.org/c/openstack/nova/+/647310 | |
| 17:31:02 | sean-k-mooney | but only to stien | |
| 17:31:19 | melwitt | ah, ok, my bad | |
| 17:31:27 | melwitt | I had thought it landed in stein | |
| 17:31:30 | dansmith | okay I was sure we did backport it some, but .. fair enough | |
| 17:31:32 | melwitt | (originally) | |
| 17:31:59 | sean-k-mooney | i guess that stien is where the orginal bug was reported | |
| 17:32:08 | sean-k-mooney | and we just did not bring it back before that | |
| 17:33:19 | sean-k-mooney | hum https://bugs.launchpad.net/nova/+bug/1808951 | |
| 17:33:21 | openstack | Launchpad bug 1808951 in tripleo "python3 + Fedora + SSL + wsgi nova deployment, nova api returns RecursionError: maximum recursion depth exceeded while calling a Python object" [High,Incomplete] | |
| 17:33:34 | sean-k-mooney | oh i miss read SSL as SQL | |
| 17:33:44 | sean-k-mooney | i was going to say it refrence SQL too | |
| 17:34:16 | melwitt | ok, I think it would be interesting if I build them a test package with that change and see if they can try it out | |
| 17:34:30 | melwitt | that would be good proof that it is/was the fix | |
| 17:34:36 | dansmith | melwitt: yeah if they're willing I think that'd be a good test | |
| 17:35:20 | melwitt | I'll get that done and give them the option | |
| 17:55:04 | sean-k-mooney | melwitt: lyarwood so it look like the new resovled is breaking lowerconstraits on stable os-vif branches | |
| 17:55:16 | sean-k-mooney | how is that adressed for stabel brances | |
| 17:55:29 | sean-k-mooney | do we update to the oldest lib that works? | |
| 17:55:51 | sean-k-mooney | hacking seams to be what is breakign things | |
| 17:56:01 | sean-k-mooney | although there coudl be other issues | |
| 17:57:16 | sean-k-mooney | ok on master stephenfin removed any non direct deps | |
| 17:57:18 | sean-k-mooney | https://github.com/openstack/os-vif/commit/44d8937148aac1a61f40e59d3271c45f9fe6aa03 | |
| 17:57:48 | sean-k-mooney | ill see if i can do somethign similar | |
| 17:58:45 | lyarwood | sean-k-mooney: ack yeah or backport a modified version of that? | |
| 17:59:13 | sean-k-mooney | well that is waht i was going to do but with lower min version to match the branch its going too | |
| 17:59:44 | sean-k-mooney | im not sure if starting from that patch will be faster or not | |
| 18:30:24 | openstackgerrit | sean mooney proposed openstack/os-vif stable/victoria: Resolve dependency issues https://review.opendev.org/c/openstack/os-vif/+/792840 | |
| 18:56:29 | sean-k-mooney | elod: lyarwood: am i allowed to squash two change in a backport upstream | |
| 18:57:23 | sean-k-mooney | basicaly i have 2 chocie squash https://review.opendev.org/c/openstack/os-vif/+/716223 and the ussuri version of https://review.opendev.org/c/openstack/os-vif/+/792840 | |
| 18:57:51 | sean-k-mooney | or i can incoperate ignoring W504 | |
| 18:58:52 | sean-k-mooney | oh wait no that is not adding W504 | |
| 18:59:02 | sean-k-mooney | that not the patch i need to add | |
| 18:59:39 | sean-k-mooney | its https://github.com/openstack/os-vif/commit/d57a5f39edcb8ef3de09e80925c8fe628e5e0f3a | |
| 19:01:15 | sean-k-mooney | but since that raise the min version hackign i cant really backport | |
| 19:02:16 | sean-k-mooney | ok ill take a look at this again tomorrow | |
| 19:06:26 | lyarwood | <sean-k-mooney "elod: lyarwood: am i allowed to "> Yes FWIW, if it fixes an otherwise unsolvable problem you can merge multiple. | |
| 19:07:18 | sean-k-mooney | i tought that i need to backport the cleanup patch and merge it with the lower constratint one but that is not what intoduced the w504 skip | |
| 19:07:35 | sean-k-mooney | lyarwood: it was the patch that bumps hacking form 1.x to 3.0 for python 3 support | |
| 19:08:26 | sean-k-mooney | ill see if i can figure out how to cap flak8 and py code style to avoid the hacking bump | |
| 19:09:06 | sean-k-mooney | but if i cant we will have to deciside if we are oke with the bump to hacking in the pep8 tox env | |
| 19:09:27 | sean-k-mooney | given it wont impackt any other test and wont be used at runtime | |
| 19:10:54 | sean-k-mooney | if we are ok with going to hacking 3.0 ill look at squashing thsoe two patches | |
| 19:11:25 | sean-k-mooney | well tomorrow im going to go get dinner now o/ | |
| 22:01:16 | openstackgerrit | Merged openstack/nova master: image_meta: Provide image_ref as the id when fetching from instance https://review.opendev.org/c/openstack/nova/+/790659 | |
| #openstack-nova - 2021-05-25 | |||
| 06:11:00 | icey | hey - any chance of getting a review on https://review.opendev.org/c/openstack/nova/+/761824/ ? It got a +1 from Zuul a while back but has been sitting in limbo since February | |
| 07:43:35 | lyarwood | icey: Apologies, I'll take a look now | |
| 07:46:55 | bauzas | morning folks | |
| 07:47:29 | bauzas | * bauzas starts his specs review day | |
| 07:47:38 | bauzas | if folks want me to look at some spec, ping me | |
| 07:58:22 | openstackgerrit | Merged openstack/nova-specs master: Allow migrating PMEM's data https://review.opendev.org/c/openstack/nova-specs/+/785563 | |
| 08:01:39 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/wallaby: Add regression test for bug #1928063 https://review.opendev.org/c/openstack/nova/+/792902 | |
| 08:03:30 | openstackgerrit | Lee Yarwood proposed openstack/nova stable/victoria: Add regression test for bug #1928063 https://review.opendev.org/c/openstack/nova/+/792903 | |
| 08:10:53 | icey | lyarwood: thanks, it's appreciated! I'm hopimng to get it into Ubuntu, and the cloud archive, for Queens (and, incidentally, Rocky, to ensure that upgrades don't break) | |
| 08:11:21 | lyarwood | ack np elod ^ if you could also review icey's change above that would be great | |
| 08:11:52 | icey | lyarwood: sadly, not quite my change, but I do get to hear the desire from the patch's author to get it in :) | |
| 08:13:39 | elod | lyarwood icey : yes, it is on my todo for a long time, I'll try to review it now :S | |
| 08:13:49 | lyarwood | many thanks :) | |
| 08:14:22 | icey | elod: thanks! there's a cherry-pick to Queens as well, it currently has a -1 because it wasn't picked from Rocky; I'd appreciate advice ont he best path forward on that one as well :) | |
| 08:51:04 | stephenfin | lyarwood: have you ever used the 'cinder attachment-*' commands? | |
| 08:51:14 | stephenfin | particularly 'cinder attachment-create' | |
| 08:51:16 | lyarwood | yes | |
| 08:51:19 | lyarwood | oh | |
| 08:51:24 | lyarwood | well only to reserve the attachment | |
| 08:51:31 | lyarwood | never with an actual connector | |
| 08:51:53 | stephenfin | yeah, I'm not sure how that's supposed to work | |
| 08:52:09 | lyarwood | yeah providing an actual connector is awkward as hell AFAICT | |
| 08:52:58 | stephenfin | So it would seem. The docs suggest that you don't need to provide much of it and nova and cinder will fill in the blanks between them | |
| 08:53:07 | stephenfin | but I can't get it to move from attaching to attached. Not sure if it's me or my DevStack deploy or what | |
| 08:53:24 | stephenfin | the cinder docs are significantly worse than even the nova docs :( | |
| 08:54:19 | lyarwood | oh there's a final API call to cinder to complete the attachment | |
| 08:54:25 | lyarwood | that moves it from attaching to attached | |
| 08:54:44 | lyarwood | I'm not sure if there's a command for that in cinderclient tbh | |
| 08:55:10 | lyarwood | https://docs.openstack.org/api-ref/block-storage/v3/index.html#complete-attachment | |
| 08:55:11 | stephenfin | * stephenfin wonders what good is attachment-create so :D | |
| 08:55:22 | stephenfin | oh, that exists | |
| 08:55:31 | stephenfin | cinder attachment-complete | |
| 08:55:43 | lyarwood | cool, use that and it should move | |
| 08:56:14 | lyarwood | well create on it's own is useful when we just need to reserve (create without a connector) I guess | |
| 08:56:23 | lyarwood | but if you want to do the entire workflow it's a PITA at the moment | |
| 08:56:31 | lyarwood | a higher level command that does all three would be cool | |
| 08:56:35 | lyarwood | create, update and complete | |
| 08:56:56 | lyarwood | but then that doesn't actually attach it on the nova side | |
| 08:57:11 | lyarwood | so it's only going to be useful in terms of mapping a volume to a host | |
| 08:59:22 | stephenfin | aha, complete did the trick alright | |
| 08:59:39 | stephenfin | I assume there's no way to trigger things on the nova side without invoking the whole flow though? | |
| 08:59:49 | stephenfin | * stephenfin apologises for his lack of knowledge in this area | |
| 09:00:27 | lyarwood | not at the moment, this is why I've wanted to write some nova-manage commands so we can update our BDMs with the attachment_id and connection_info you've just created here | |