Earlier  
Posted Nick Remark
#openstack-nova - 2022-05-10
16:45:56 bauzas s/should be/are for the last item :)
16:46:06 bauzas thanks elodilles
16:46:16 bauzas last topic
16:46:18 bauzas #topic Open discussion
16:46:22 bauzas (sean-k-mooney) VDPA
16:46:42 bauzas sean-k-mooney: we're listening to you
16:46:50 sean-k-mooney :)
16:47:21 sean-k-mooney so short version is as agreed at the ptg i filed a bug for the move operation that were blocked but actully working
16:47:33 sean-k-mooney and i have a patch to adress it
16:47:45 bauzas cool
16:47:52 sean-k-mooney i then have 3 patches that follow up adding the remaining life cycle operations
16:48:06 sean-k-mooney i would like to proceed with those as a specless blueprint
16:48:12 sean-k-mooney those are inteface detach
16:48:17 sean-k-mooney suspend which need detach
16:48:36 sean-k-mooney and hotplug live migration which just need detach and adding vdpa to a list
16:48:49 sean-k-mooney and a compute version bump+ conductor check
16:49:06 sean-k-mooney idential to sriov hotplug migraiton
16:49:13 sean-k-mooney how do people feel about that
16:49:36 sean-k-mooney i have the code writen for everything excpot the compute service bump
16:49:48 bauzas hmmmm
16:49:49 sean-k-mooney test code is still need so patches are WIP
16:50:11 bauzas sean-k-mooney: no RPC changes on the compute service, just a version bump for API check ?
16:50:50 sean-k-mooney yep like this https://github.com/openstack/nova/blob/master/nova/conductor/tasks/live_migrate.py#L34-L44=
16:51:10 sean-k-mooney the sriov live migration code works for vdpa too
16:51:21 bauzas I'd say a specless BP seems OK to me
16:51:23 sean-k-mooney just need to tell it to hot unplug the interface as we do for direct passthough
16:51:41 bauzas for exposing the new lifecycle ops
16:51:53 sean-k-mooney the compute service bump is just need for rooling upgrades
16:52:02 bauzas I know
16:52:09 bauzas but this means there is an upgrade impact
16:52:18 bauzas seamless for ops tho
16:52:42 bauzas but I don't see any controversial requiring a spec
16:52:46 sean-k-mooney yep so the api block will move form the api to conductor baseed on compute service version
16:52:59 bauzas so, looks like a feature, but less paper-ish
16:53:25 bauzas sean-k-mooney: this will be explained in a feature relnote, so definitely a blueprint
16:53:50 bauzas like 'now, nova supports this, only if you have your whole cloud uptodate'
16:53:53 sean-k-mooney ok we can continue to discuss in gerrit https://blueprints.launchpad.net/nova/+spec/vdpa-suspend-detach-and-live-migrate is the blueprint
16:54:23 sean-k-mooney but yes i can detial this in the release note
16:54:24 bauzas we have 5 mins for officially blessing it
16:54:37 bauzas do you want to propose https://blueprints.launchpad.net/nova/+spec/vdpa-suspend-detach-and-live-migrate as a specless BP ?
16:55:01 sean-k-mooney yes
16:55:20 bauzas okay, then anyone DISAGREEING with this ?
16:56:14 bauzas looks not,
16:56:47 bauzas #agreed https://blueprints.launchpad.net/nova/+spec/vdpa-suspend-detach-and-live-migrate approved for Zed release as a specless BP, no objections so far be seen
16:56:53 bauzas voila
16:57:20 bauzas sean-k-mooney: a short summary and a better title may help
16:57:31 bauzas but I'll approve your BP
16:58:18 bauzas that's it for today, any other lastminute item to discuss ?
16:58:54 bauzas apparently not
16:58:56 bauzas thanks all
16:59:03 bauzas productive meeting again
16:59:06 bauzas #endmeeting
16:59:06 opendevmeet Meeting ended Tue May 10 16:59:06 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)
16:59:06 opendevmeet Minutes: https://meetings.opendev.org/meetings/nova/2022/nova.2022-05-10-16.00.html
16:59:06 opendevmeet Minutes (text): https://meetings.opendev.org/meetings/nova/2022/nova.2022-05-10-16.00.txt
16:59:06 opendevmeet Log: https://meetings.opendev.org/meetings/nova/2022/nova.2022-05-10-16.00.log.html
16:59:09 elodilles thanks bauzas o/
16:59:33 bauzas as a side note, I just remembered I'll be on PTO tomorrow /o\
16:59:43 bauzas sean-k-mooney: I made false promises
16:59:52 sean-k-mooney :)
17:00:01 sean-k-mooney if you forgot about PTO you likely need it
17:00:10 elodilles bauzas: i know it's spec review day, but... o:) maybe these worth some quick reviews (perhaps tomorrow :)): https://review.opendev.org/q/Ieba7daf39fa3323e8c9a7396747449f24189fcd5
17:00:38 bauzas joys of May and my number of PTO days which is too high for the end of the paperwork year
17:00:40 elodilles bauzas: or if you'll be on PTO then the day after your PTO :X
17:01:08 bauzas I may have to take more PTO days before end of May, but I'll restrict myself to one per week
17:01:26 sean-k-mooney i can hit the master patch
17:01:38 elodilles sean-k-mooney: awesome, thanks! \o/
17:02:03 bauzas done
17:02:16 elodilles thanks \o/
17:03:21 sean-k-mooney elodilles: are there any other "nova" repos these are still pending for
17:03:28 sean-k-mooney i think we already did some of them
17:03:55 elodilles sean-k-mooney: to tell you the truth i don't know. maybe os-vif also have l-c?
17:04:13 sean-k-mooney it did but i think i approve the removal already
17:04:34 sean-k-mooney yep if anyone want to be the second +2 https://review.opendev.org/c/openstack/os-vif/+/840020
17:04:57 sean-k-mooney or i can just merge this with a singel +2 if peopel dont revew by the end of the week
17:05:42 sean-k-mooney ah nova itself
17:05:54 sean-k-mooney https://review.opendev.org/c/openstack/nova/+/840021
17:06:49 elodilles yepp, in nova i started with stable-only patches as stable branches were blocked
17:07:13 elodilles so, nice catch sean-k-mooney :)
17:07:26 sean-k-mooney stephenfin: ^ care to take a look at the nova and os-vif lc patches
17:07:33 elodilles those are still waiting for some reviews :X
17:08:15 stephenfin sure
17:17:01 sean-k-mooney ok im going to call it a day
17:19:50 gibi artom: thanks for the bug triage!
17:24:28 artom gibi, happy to help :)
17:28:16 melwitt bauzas: I added a note about optionalness of etherpads/docs https://etherpad.opendev.org/p/nova-bug-triage-roster feel free to change it or remove it or whatever
18:15:42 opendevreview Merged openstack/placement master: Drop lower-constraints.txt and its testing https://review.opendev.org/c/openstack/placement/+/840018
18:44:28 donny Hi everyone I recently joined the project and would like to contribute....can someone please help me on how to go about understanding the codebase
19:55:12 opendevreview Merged openstack/placement stable/yoga: Drop lower-constraints.txt and its testing https://review.opendev.org/c/openstack/placement/+/840728
21:48:16 opendevreview Merged openstack/placement stable/xena: Drop lower-constraints.txt and its testing https://review.opendev.org/c/openstack/placement/+/840756
22:14:07 opendevreview Miguel Lavalle proposed openstack/os-vif master: Revert "Fix race with DPDK and vhostuserclient mode" https://review.opendev.org/c/openstack/os-vif/+/841276
22:18:02 opendevreview Miguel Lavalle proposed openstack/os-vif master: Revert "Fix race with DPDK and vhostuserclient mode" https://review.opendev.org/c/openstack/os-vif/+/841276
#openstack-nova - 2022-05-11
03:15:47 donny Hi everyone I recently joined the project and would like to contribute....can someone please help me on how to go about understanding the codebase
09:06:51 kashyap gibi: Have you seen this one before: https://zuul.opendev.org/t/openstack/build/ab5025372ff34fdf99180efe049de5cd
09:07:02 kashyap rsync: [sender] change_dir "/var/lib/zuul/builds/ab5025372ff34fdf99180efe049de5cd/work/CA" failed: No such file or directory (2)
09:10:52 kashyap (Aside: The 'py310' 'mysqld' error was solved by your bindep.txt change: https://review.opendev.org/c/openstack/nova/+/840822)
09:13:41 gibi kashyap: I think there is another error before that
09:13:55 gibi https://zuul.opendev.org/t/openstack/build/ab5025372ff34fdf99180efe049de5cd/log/job-output.txt#24628
09:14:12 gibi rsync: connection unexpectedly closed (0 bytes received so far) [Receiver]\nrsync error: unexplained error (code 255) at io.c(228) [Receiver=3.2.3
09:15:01 kashyap Oh, yeah. Amazing, how the error itself says "unexplained error"

Earlier   Later