| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2024-05-07 | |||
| 06:25:58 | gibi | bauzas: do you have anything special for today's nova meeting for me to not forget? | |
| 06:28:34 | bauzas | gibi: nothing special, but I'll create a change for the schedule deadlines | |
| 06:28:58 | bauzas | for the review priorities, I'll also modify the etherpad | |
| 06:38:04 | gibi | ack, thanks | |
| 06:49:38 | bauzas | thanks again for running the meeting | |
| 06:50:06 | bauzas | fwiw, I'm eventually taking my afternoon off ( I was about to stop at 4pm, so I prefer just to take the whole afternoon) | |
| 06:50:21 | bauzas | and I'll be back on Monday | |
| 06:50:35 | bauzas | (Wed + Thur are bank holidays here) | |
| 07:01:06 | gibi | bauzas: have a nice time off | |
| 07:21:14 | bauzas | thanks, visiting long distance friends :) | |
| 09:11:37 | opendevreview | Sylvain Bauza proposed openstack/nova master: mtty: add devstack compilation for fedora builds https://review.opendev.org/c/openstack/nova/+/918420 | |
| 09:42:00 | bauzas | gibi: I just uploaded https://review.opendev.org/c/openstack/releases/+/918422 which is the Dalmatian schedule for nova | |
| 09:42:12 | bauzas | gibi: actually we will have a spec review day on next Tuesday | |
| 09:42:32 | bauzas | I'll just write an email | |
| 09:42:39 | bauzas | but please tell it during the nova meeting | |
| 11:10:00 | gibi | sure I will | |
| 11:46:42 | sean-k-mooney | JayF: feel free to rework my base patch as needed | |
| 12:45:48 | opendevreview | Fabian Wiesel proposed openstack/nova master: Vmware: Remove uuid parameter from get_vmdk_info call https://review.opendev.org/c/openstack/nova/+/910627 | |
| 14:34:40 | opendevreview | Elod Illes proposed openstack/nova stable/2023.2: DNM: grenade-skip-level fix https://review.opendev.org/c/openstack/nova/+/918454 | |
| 15:18:58 | ganso | gibi: hi! I would like to ask you about https://review.opendev.org/c/openstack/nova/+/822050 and https://review.opendev.org/c/openstack/nova/+/810915 | |
| 15:21:00 | opendevreview | Dan Smith proposed openstack/nova master: Enable virtio-scsi in nova-next https://review.opendev.org/c/openstack/nova/+/918458 | |
| 15:21:01 | ganso | gibi: I copied the entire nova/tests/functional/libvirt/test_numa_servers.py file resulted from applying all 5 patches in ussuri and placed it in the 21.2.4 tag (where the fixes do not exist) and ran the functional text. The 2 new tests are still passing | |
| 15:21:09 | dansmith | sean-k-mooney: ^ | |
| 15:21:46 | ganso | gibi: is there anything that I am doing wrong? or is it that due to being a race condition sometimes the test will not fail? | |
| 15:37:39 | gibi | ganso: could you push what you have locally to gerrit so I can look at it? | |
| 15:38:25 | gibi | ganso: the test are stable in newer branches but maybe something changed in nova inbetween that making the tests invalid | |
| 15:47:26 | gibi | folks, nova meeting starts in 13 minutes here on the channel. | |
| 15:52:24 | ganso | gibi: ok I just realized what I did wrong, I was running the tests in a container, using "tox -efunctional-py38 -vv --skip-missing-interpreters=false | |
| 15:52:47 | ganso | gibi: just like the CI is doing, as I expected that I wouldn't have to deploy a real env for that | |
| 15:52:55 | ganso | gibi: but not functional tests are really running, just unit tests | |
| 15:53:28 | ganso | gibi: I tried forcing it to run only the numa tests and it says it doesn't match anything | |
| 15:53:43 | ganso | gibi: "tox -efunctional-py38 -vv --skip-missing-interpreters=false -- nova.tests.functional.libvirt.test_numa_servers" | |
| 15:54:13 | ganso | gibi: how is the CI job running it without a full deployment? do I need to set a parameter or flag? | |
| 15:54:15 | ganso | gibi: https://0b2ab51d56a67edcd93c-a3ef59e9a62996701b6308cf25770b88.ssl.cf2.rackcdn.com/918195/6/check/nova-tox-functional-py38/cb62697/job-output.txt | |
| 15:54:38 | ganso | gibi: I am deploying a focal-ussuri env in parallel just in case it really needs a real env to run | |
| 15:55:37 | clarkb | ganso: are you running the test-setup.sh step? I dont' really know anything about nova's functional tests but that is a step the CI jobs take and the log you link confirms it is running | |
| 15:58:11 | ganso | clarkb: hmm no. I just tried to run it and it tried to run a mysql command. I just installed mysql and ran it again, and it says the syntax is wrong... | |
| 16:00:20 | gibi | ganso: check tox.ini to see what envs are defined in ussrui | |
| 16:00:41 | gibi | but now, lets have a quick nova meeting | |
| 16:00:44 | gibi | #startmeeting nova | |
| 16:00:44 | opendevmeet | Meeting started Tue May 7 16:00:44 2024 UTC and is due to finish in 60 minutes. The chair is gibi. Information about MeetBot at http://wiki.debian.org/MeetBot. | |
| 16:00:44 | opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | |
| 16:00:44 | opendevmeet | The meeting name has been set to 'nova' | |
| 16:01:01 | erlon | o/ | |
| 16:01:12 | tkajinam | o/ | |
| 16:01:16 | gibi | bauzas is off today, so I will be your chair | |
| 16:01:16 | auniyal | o/ | |
| 16:01:25 | elodilles | o/ | |
| 16:01:45 | sean-k-mooney | dansmith: using local.sh works too | |
| 16:01:51 | sean-k-mooney | o/ | |
| 16:01:54 | fwiesel | o/ | |
| 16:01:54 | gibi | #link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | |
| 16:01:55 | dansmith | o/ | |
| 16:02:35 | gibi | #topic Bugs (stuck/critical) | |
| 16:02:51 | gibi | #info No Critical bug | |
| 16:03:12 | gibi | is there any bug that needs discussion today? | |
| 16:03:26 | erlon | Must be critical I suppose? | |
| 16:04:33 | auniyal | gibi, not critical but seems imp to me https://bugs.launchpad.net/nova/+bug/2063342 | |
| 16:04:38 | tkajinam | this is not very critical now but there is one remaining follow-up about excludes cleanup. would be nice if it can get 2nd +2 https://review.opendev.org/c/openstack/os-traits/+/917713 | |
| 16:04:40 | gibi | erlon: if you feel it worth discussing with the foks then sure | |
| 16:05:50 | erlon | So, I need some reviews on the RBAC support patch. Its the last one fron the series: https://review.opendev.org/c/openstack/nova/+/811521?usp=search | |
| 16:06:40 | gibi | erlon: OK, noted. | |
| 16:06:50 | erlon | thanks | |
| 16:07:16 | gibi | auniyal: my immediate reaction that bug is to ask if they tried to archive deleted rows | |
| 16:07:20 | erlon | Should I add it to the etherepad or you do? | |
| 16:07:58 | gibi | erlon: I will | |
| 16:08:11 | erlon | Thanks | |
| 16:08:22 | gibi | auniyal: but reading it more it is about instance action | |
| 16:08:35 | gibi | growing without bound for non deleted instances | |
| 16:09:26 | sean-k-mooney | erlon's patch is for a blueprint that is not approved | |
| 16:09:27 | auniyal | yes, can we add some preodic updated nova can do to DB | |
| 16:09:28 | gibi | and I see they proposed a nova-manage command to trim the action list. We at least need a blueprint for it | |
| 16:09:49 | sean-k-mooney | so it need to be appoved here first before we can proceed with it | |
| 16:10:02 | sean-k-mooney | https://blueprints.launchpad.net/nova/+spec/shared-security-groups | |
| 16:10:14 | erlon | what does that require? to aprove? | |
| 16:11:06 | sean-k-mooney | agreement that this does not requrie a spec and that we are ok to proceed with adding supprot for neutron SRBC for security groups to nova | |
| 16:11:52 | sean-k-mooney | basically this is a case of the new feature was added to neutron a few release ago and the nova side was not done | |
| 16:11:56 | erlon | ok, so I suppose that the blueprint was mostly a formality | |
| 16:12:26 | sean-k-mooney | well no the blueprint is alwasy requried for features like this but the quetion is does it also need a spec | |
| 16:13:14 | erlon | a spec seems a overkill | |
| 16:13:15 | gibi | erlon: does it need any nova API, db, RPC, or object model changes? Does it have upgrade impct? | |
| 16:13:33 | sean-k-mooney | in this particalr case there is not direct api impact. do we want to move this to the opendicussion stection by the way | |
| 16:13:36 | sean-k-mooney | or resolve this now | |
| 16:14:16 | gibi | lets resolve it | |
| 16:14:17 | sean-k-mooney | the reason i say there is no direct api impact is the api bechiaov changes but the requesst does not and i dont think we need a microversion | |
| 16:14:28 | gibi | OK. | |
| 16:14:42 | sean-k-mooney | so without this feature you can only reference secuirty groups that are created in your project | |
| 16:14:42 | gibi | I see it depends on a new neutron extension | |
| 16:14:55 | erlon | @gibi, no, just adds some calls to neutron client | |
| 16:15:04 | sean-k-mooney | with this feature you can use a secrutiy group defiend in another project but shread with your project | |
| 16:15:27 | sean-k-mooney | so before the change you would have got a error and after the vm would boot | |
| 16:15:46 | gibi | sean-k-mooney: that is a reasonable behavior change | |
| 16:16:02 | sean-k-mooney | yep its tradign a vm in error due to SecurityGroupNotFound: Security group 0c649378-1cf8-48e0-9eb4-b72772c35a62 not found. in swpan | |
| 16:16:05 | sean-k-mooney | for a booting vm | |
| 16:16:20 | sean-k-mooney | so i dont think that a breaking change and therfor no api impact | |
| 16:17:24 | sean-k-mooney | gibi: erlon has a patch for review https://review.opendev.org/c/openstack/nova/+/811521 and the code changes are confied to the neutorn module in nova | |
| 16:17:25 | gibi | There is a preformance impact discussed in the impl, does that sever enough to pull it out to a spec? | |
| 16:17:54 | sean-k-mooney | the performace impact is 1 addtion call to list security group that are shared with you | |
| 16:18:26 | gibi | as far as I see if you don't use shared SG then you don't have the extra overhead | |
| 16:18:36 | sean-k-mooney | based on the trace back in the bug it got to the comptue manger beforfe that happend | |