| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-06-20 | |||
| 07:49:58 | Uggla | Good morning | |
| 09:27:27 | stephenfin | Morning. Can someone look at this real simple unit test-only patch for me? https://review.opendev.org/c/openstack/nova/+/802893 | |
| 09:29:52 | sean-k-mooney | nova manage | |
| 09:29:57 | sean-k-mooney | sure ill take a look | |
| 09:31:19 | sean-k-mooney | oh its a followup to https://review.opendev.org/c/openstack/nova/+/800855 | |
| 09:31:51 | sean-k-mooney | ah yes i remember that | |
| 09:32:07 | stephenfin | thanks :) | |
| 09:39:01 | sean-k-mooney | +2 a minor unrelated formating chnage is included but im fine with that | |
| 10:03:14 | opendevreview | Rajesh Tailor proposed openstack/nova master: Remove unnecessary if condition https://review.opendev.org/c/openstack/nova/+/844418 | |
| 15:20:05 | opendevreview | Merged openstack/nova master: tests: Add test for bug #1936278 https://review.opendev.org/c/openstack/nova/+/802893 | |
| 16:22:44 | opendevreview | Sylvain Bauza proposed openstack/nova master: WIP: Fix verifying all the alloc requests from a multi-create https://review.opendev.org/c/openstack/nova/+/846786 | |
| #openstack-nova - 2022-06-21 | |||
| 03:49:51 | opendevreview | OpenStack Proposal Bot proposed openstack/nova master: Imported Translations from Zanata https://review.opendev.org/c/openstack/nova/+/846876 | |
| 03:56:49 | opendevreview | OpenStack Proposal Bot proposed openstack/python-novaclient master: Imported Translations from Zanata https://review.opendev.org/c/openstack/python-novaclient/+/846880 | |
| 07:40:44 | bauzas | gibi: you maybe missed https://review.opendev.org/c/openstack/nova/+/846786 | |
| 07:42:52 | gibi | bauzas: I think what you did is what we discussed. I would add some testing around the for:else: and the breaks but otherwise I agree with the direction | |
| 07:43:17 | bauzas | gibi: yup, of course, this is a WIP b/c I would add a UT | |
| 07:44:07 | gibi | so then that WIP looks OK to me | |
| 09:16:05 | elodilles | bauzas sean-k-mooney : can you please review the release patches (whether you see something that requires MINOR version bump, etc). I don't think it's worth to wait more time, as every week we'll have new patches + gate is sometimes slow and days are passing o:) | |
| 09:16:57 | elodilles | bauzas sean-k-mooney : of course if you see any patch that is a MUST to have in any of the release, then we can wait. but otherwise i think it's better to release now o:) | |
| 09:17:54 | elodilles | bauzas sean-k-mooney : and release managers won't review the release patches until PTL or release liaisons haven't reviewed the patches ;) | |
| 09:18:55 | sean-k-mooney[m] | actully i ment to update the patches | |
| 09:19:01 | sean-k-mooney[m] | to use the new sha last week | |
| 09:19:33 | sean-k-mooney[m] | most of the patches i wanted to wait for have alredy landed last week | |
| 09:19:50 | sean-k-mooney[m] | we just need to repin the patches to point to the currnent tip of the stable branches | |
| 09:21:02 | sean-k-mooney[m] | elodilles give me 10 mins to grab a coffee and ill go do that when i get back and +1 them | |
| 09:21:31 | elodilles | sean-k-mooney[m]: i've updated the patches with the latest hashes :) | |
| 09:21:50 | elodilles | sean-k-mooney[m]: no need to hurry as i also will go to have lunch ;) | |
| 09:22:38 | elodilles | so enjoy your coffee, and review the patches some time later today o:) | |
| 09:22:48 | sean-k-mooney[m] | oh ok thanks i stilll ment to do this last week so before i start anything else ill review them | |
| 09:23:28 | bauzas | elodilles: ack, will look | |
| 09:23:44 | bauzas | gibi: reopened https://bugs.launchpad.net/nova/+bug/1890244 fwiw | |
| 09:23:45 | elodilles | sean-k-mooney[m] bauzas: ++, thanks in advance! \o/ | |
| 09:26:29 | bauzas | gibi: I'm curious how you didn't went able to reproduce the bug in https://bugs.launchpad.net/nova/+bug/1890244/comments/1 | |
| 09:26:31 | gibi | bauzas: so probably CONF.workarounds.disable_group_policy_check_upcall was the trick why I was not able to reproduce it | |
| 09:26:42 | bauzas | gibi: hah | |
| 09:27:06 | bauzas | gibi: but IMHO we default to False | |
| 09:27:22 | gibi | then I dont know | |
| 09:28:11 | gibi | bauzas: did you reproduced it? | |
| 09:28:14 | bauzas | I need to respin my devstack env | |
| 09:28:42 | bauzas | gibi: no, I only triaged it and we have a downstream BZ that was related | |
| 09:28:49 | bauzas | https://bugzilla.redhat.com/show_bug.cgi?id=2099279 | |
| 09:29:01 | bauzas | the code is pretty simple to look at | |
| 09:30:35 | bauzas | oh wait, you could be right, shit. | |
| 09:31:00 | bauzas | when we populate the ReqSpec, we rehydrate the fields and we can set the group value to None if the group was deleteed | |
| 09:32:04 | bauzas | https://github.com/openstack/nova/commit/94fd36f0582c5dbcf2b9886da7c7bf986d3ad5d1 is super old | |
| 09:35:35 | gibi | https://paste.opendev.org/show/bXPvHPVJ7FfPgjfhTsSv/ | |
| 09:35:35 | gibi | I still cannot reproduce | |
| 09:37:46 | bauzas | gibi: then I don't understand | |
| 09:37:54 | bauzas | gibi: I verified master | |
| 09:38:07 | gibi | the above repro trial I run on master too | |
| 09:38:16 | bauzas | gibi: when we evacuate, we regenerate the old RequestSpec record | |
| 09:38:20 | gibi | I don't have a multinode train env | |
| 09:38:30 | bauzas | and then, we set the hints | |
| 09:39:57 | bauzas | gibi: can you look at your reqspec record and see whether you have the hints dict ? | |
| 09:40:06 | gibi | sure | |
| 09:41:07 | gibi | https://paste.opendev.org/show/baqepgHm3PhuQ9VhvfWi/ | |
| 09:41:29 | gibi | there is the group hint in the dfb | |
| 09:41:30 | gibi | db | |
| 09:41:52 | bauzas | gibi: ok, and you can confirm that df92c028-728a-4321-9df6-6424ca56969e no longer exists in the instance_groups table ? | |
| 09:42:19 | bauzas | ideally we should introspect into https://github.com/openstack/nova/blob/ebe08834f311e8e22bfd9685d7e6e91dab967382/nova/compute/manager.py#L3657 | |
| 09:42:44 | gibi | yes https://paste.opendev.org/show/b9NTbUJdRy4zI9qVlFsL/ | |
| 09:43:22 | bauzas | gibi: could you see what we get as hints from https://github.com/openstack/nova/blob/ebe08834f311e8e22bfd9685d7e6e91dab967382/nova/compute/manager.py#L3656 ? | |
| 09:43:48 | bauzas | b/c I don't see where we could be smart | |
| 09:43:56 | bauzas | we're just getting the hints there | |
| 09:44:01 | gibi | I can add a LOG and repro give me 5 | |
| 09:44:05 | bauzas | and see whether this is a group | |
| 09:44:26 | bauzas | if this a group, we directly lookup the instance_groups table with this UUID | |
| 09:44:37 | bauzas | and that's the upcall | |
| 09:45:57 | sean-k-mooney | so honestly im surpiesed we allow you to delete an instance group if there are instances in it | |
| 09:46:12 | sean-k-mooney | to me that is the really issue here | |
| 09:46:41 | sean-k-mooney | is there a reason we allow that | |
| 09:46:57 | sean-k-mooney | unlike the flavor for example we are not embdeding a copy fo the instance group | |
| 09:47:16 | sean-k-mooney | so we really should not allow ti to be remvoed if its in use | |
| 09:47:23 | sean-k-mooney | do we know why we do today? | |
| 09:48:44 | bauzas | sean-k-mooney: because Instance Groups are a terrible concept ?N | |
| 09:49:03 | sean-k-mooney | not nessiarly they may be implemented badly | |
| 09:49:23 | sean-k-mooney | the concept is not a bad one but the way we implemeted them is not good | |
| 09:52:56 | gibi | bauzas: could this be the reason we bail out https://github.com/openstack/nova/blob/ebe08834f311e8e22bfd9685d7e6e91dab967382/nova/compute/manager.py#L1742 | |
| 09:53:11 | gibi | (still instrumenting the code with logs then I will rerun the repro attempt) | |
| 09:55:00 | gibi | btw I agree with sean that group is a better concept than same_host / different_host hints, as the hints were not reflexive | |
| 09:55:24 | gibi | strike reflexive | |
| 09:55:28 | gibi | symmetric | |
| 09:55:34 | gibi | they wasnt symmetirc | |
| 09:56:01 | bauzas | let's discuss this tomorrow at the bug call | |
| 09:56:04 | bauzas | shit, d/s | |
| 10:11:03 | gibi | I have /etc/nova/nova-cpu.conf:disable_group_policy_check_upcall = True in my env, restaring repro .... | |
| 10:12:25 | sean-k-mooney | isnt that our default | |
| 10:14:45 | gibi | nope | |
| 10:14:49 | gibi | the default is false | |
| 10:14:57 | gibi | I had an explicit True set for some reason | |
| 10:17:21 | sean-k-mooney | devstack set it to true by defualt i think | |
| 10:17:37 | gibi | yepp devastack by default does not even support the upcall | |
| 10:17:54 | gibi | so If I enable the upcall nova blows as no DB access | |
| 10:19:00 | sean-k-mooney | wait what | |
| 10:19:08 | sean-k-mooney | it shoudl be doing that via rpc to conductor | |
| 10:19:20 | sean-k-mooney | is it trying to use the old local conductor path | |
| 10:19:23 | sean-k-mooney | and directly access the db | |
| 10:20:37 | gibi | https://paste.opendev.org/show/bxWFPkh1OoBfWlM15mS0/ | |
| 10:20:59 | gibi | it calls up to the cell conductor | |
| 10:21:06 | gibi | and cell conductor fails to read the api db | |