| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-04-25 | |||
| 16:30:58 | bauzas | we had precedents | |
| 16:31:11 | bauzas | + a relnote obviously | |
| 16:31:24 | bauzas | ykarel: do you agree with the direction ? | |
| 16:31:30 | ykarel | bauzas, yes | |
| 16:31:31 | sean-k-mooney | im ok with specless too assuming all of the above are done | |
| 16:31:41 | bauzas | anyone disagreeing ? | |
| 16:32:07 | bauzas | looks not | |
| 16:32:20 | ykarel | Thanks folks \o/ | |
| 16:32:41 | sean-k-mooney | the only other thin i would suggest is once this is done a devstack patch shoudl be added to set this by default to say 32mb if qemu is used instead of kvm | |
| 16:33:37 | sean-k-mooney | thats out of scope fo this meeting and coud be doen on a per job basis too | |
| 16:33:39 | bauzas | #agreed enabling tb cache seems a specless blueprint, provided it only adds a config knob defaulting to unset, init_host failing on an older libvirt and just libvirt config tweak | |
| 16:34:01 | bauzas | #action ykarel to ping bauzas once the blueprint is created so that he can approve it | |
| 16:34:34 | bauzas | ykarel: and yeah, the scope of this feature can include devstack change and testing, for sure | |
| 16:34:53 | bauzas | like we could enable it in nova-next | |
| 16:35:41 | sean-k-mooney | bauzas: it will reduce the memory pressue in all our jobs so ocne we know it does not have a negitive impact we will proably want to have it enable din all of them | |
| 16:35:42 | bauzas | anything else to add on this item ? | |
| 16:35:55 | sean-k-mooney | but we can take it slow like the mariadb reduced memroy | |
| 16:36:00 | bauzas | sean-k-mooney: oh yeah, but I'd be in favor of testing it first | |
| 16:36:08 | bauzas | yah | |
| 16:36:33 | bauzas | ok, fwiw, I have another item | |
| 16:36:53 | bauzas | (bauzas) Can https://blueprints.launchpad.net/nova/+spec/cold-migrate-to-host-policy be specless ? | |
| 16:37:05 | bauzas | tl;dr: we discussed this at the PTG | |
| 16:37:29 | sean-k-mooney | assuming there is no change in the default policy then yes i think so | |
| 16:37:32 | bauzas | operators want a better granularity and maybe change the cold-migrate action to be admin_or_owner | |
| 16:37:46 | bauzas | but, here, we're just adding a new policy which is admin-only | |
| 16:37:55 | bauzas | so no API change, and no policy changes | |
| 16:38:11 | bauzas | it will just go check a separate policy if host is set | |
| 16:38:32 | bauzas | (literally a one-liner patch besides the policy file) | |
| 16:38:55 | sean-k-mooney | so basicaly there will be two poicies now for cold migration one for migratoin with a host adn one without but admin only by default | |
| 16:38:55 | bauzas | any objections to have it specless ? | |
| 16:39:01 | sean-k-mooney | and then operators can choose | |
| 16:39:18 | sean-k-mooney | +1 | |
| 16:39:37 | bauzas | correct, like we have for os_compute_api:servers:create:forced_host | |
| 16:40:10 | bauzas | except I won't change the defaut rule for os_compute_api:os-migrate-server:migrate | |
| 16:40:34 | bauzas | both being admin-only | |
| 16:40:34 | bauzas | there will be os_compute_api:os-migrate-server:migrate and os_compute_api:os-migrate-server:migrate:host | |
| 16:40:48 | bauzas | (and operators can decide to open os_compute_api:os-migrate-server:migrate to endusers) | |
| 16:41:10 | bauzas | so I reiterate, any objection to have it specless ? | |
| 16:41:40 | bauzas | looks not | |
| 16:41:49 | bauzas | if so, | |
| 16:42:08 | sean-k-mooney | as long ast there is at least a blueprint im happy. i dislike changing policy without any tracker to works for me | |
| 16:42:17 | bauzas | #agreed https://blueprints.launchpad.net/nova/+spec/cold-migrate-to-host-policy accepted as a specless feature for Bobcat | |
| 16:42:31 | bauzas | sean-k-mooney: there is a blueprint, and there will be a relnote | |
| 16:42:45 | bauzas | and there will be functional tests covering this | |
| 16:42:53 | sean-k-mooney | yep all good. | |
| 16:43:15 | bauzas | I don't think we need a tempest change, do you think it's a nice to have ? | |
| 16:43:29 | sean-k-mooney | i dont think tempst shoudl test non default policy | |
| 16:43:41 | bauzas | yeah, that was my question | |
| 16:43:48 | bauzas | I'm not a QA expert | |
| 16:43:55 | bauzas | tempest is branchless | |
| 16:44:05 | bauzas | so that would be a bit hard to test it with tempest | |
| 16:44:22 | sean-k-mooney | i suspect you could reuse some fo the exsiting test with the right config if you needed too | |
| 16:44:24 | bauzas | anyway, I think we're done on this | |
| 16:44:44 | bauzas | about tempest, we could discuss this on the review time | |
| 16:44:50 | bauzas | thanks folks | |
| 16:44:51 | opendevreview | Artom Lifshitz proposed openstack/nova master: Save cell socket correctly when updating host NUMA topology https://review.opendev.org/c/openstack/nova/+/862964 | |
| 16:44:51 | opendevreview | Artom Lifshitz proposed openstack/nova master: Reproduce bug 1995153 https://review.opendev.org/c/openstack/nova/+/862967 | |
| 16:45:01 | bauzas | any other item to add before we end the meeting ? | |
| 16:45:22 | auniyal | small thing o/ | |
| 16:45:27 | auniyal | https://review.opendev.org/c/openstack/nova/+/839922 | |
| 16:45:27 | auniyal | CI on yoga: this one keep failing for different reasons, mostly | |
| 16:45:28 | bauzas | shot | |
| 16:45:42 | auniyal | mostly volume tests | |
| 16:46:23 | sean-k-mooney | ya that kind fo a pain im not sure there is anythin we can do beyond recheck | |
| 16:46:29 | sean-k-mooney | is it the volume detach tests | |
| 16:46:45 | sean-k-mooney | auniyal: gibi found some tests that are not waiting properly | |
| 16:46:47 | bauzas | I think this is also tracked on the stable CI failures etherpad | |
| 16:46:50 | auniyal | yes, attach and detach , but they are always different | |
| 16:47:04 | auniyal | sometime tomeout | |
| 16:47:15 | sean-k-mooney | yoga is not EM right so its still using tempest master? | |
| 16:47:20 | auniyal | no | |
| 16:47:46 | auniyal | tbc no, its not EM | |
| 16:47:50 | sean-k-mooney | ok | |
| 16:47:58 | sean-k-mooney | so it still can get tempest fixes if we fix those tests | |
| 16:48:09 | bauzas | yup | |
| 16:48:40 | bauzas | are we done ? | |
| 16:48:59 | auniyal | sorry I didn't get, any action on above | |
| 16:49:20 | auniyal | we need to fix tempest tests ? | |
| 16:49:47 | sean-k-mooney | i think just continue to reheck it. gibi found at least on test that is not waiting for sshable | |
| 16:49:53 | bauzas | no, we have some tempest patches up | |
| 16:49:57 | sean-k-mooney | and notice other dont appear to eb waiting but i dont have the context | |
| 16:50:01 | bauzas | and yoga would benefit from those | |
| 16:50:10 | bauzas | since tempest is branchless | |
| 16:50:14 | sean-k-mooney | oh do you have a link? | |
| 16:50:16 | auniyal | ack thanks | |
| 16:51:54 | bauzas | I was referring to gibi's recent discoveries of testing gap for ssh wait | |
| 16:52:13 | bauzas | (sorry was looking at the -tc meeting) | |
| 16:52:23 | bauzas | -tc chan* | |
| 16:52:35 | bauzas | can we close this meeting now ? | |
| 16:52:38 | sean-k-mooney | its fine we can wrap this here and chat after | |
| 16:53:06 | bauzas | cool | |
| 16:53:08 | bauzas | thanks all | |
| 16:53:12 | opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2023/nova.2023-04-25-16.00.log.html | |
| 16:53:12 | opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2023/nova.2023-04-25-16.00.txt | |
| 16:53:12 | opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2023/nova.2023-04-25-16.00.html | |
| 16:53:12 | opendevmeet | Meeting ended Tue Apr 25 16:53:12 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | |
| 16:53:12 | bauzas | #endmeeting | |
| 16:53:14 | sean-k-mooney | https://review.opendev.org/c/openstack/tempest/+/880891 | |
| 16:53:17 | sean-k-mooney | seams to be related | |
| 16:53:43 | elodilles | thanks o/ | |