Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2022-08-23
16:34:10 dansmith gibi: thanks
16:34:51 sean-k-mooney i think thats the main set that we needed to talk about
16:35:18 gibi ack then moving on
16:35:28 gibi #topic Review priorities
16:35:33 gibi #link https://review.opendev.org/q/status:open+(project:openstack/nova+OR+project:openstack/placement+OR+project:openstack/os-traits+OR+project:openstack/os-resource-classes+OR+project:openstack/os-vif+OR+project:openstack/python-novaclient+OR+project:openstack/osc-placement)+(label:Review-Priority%252B1+OR+label:Review-Priority%252B2)
16:35:56 gibi looks fairly relevant
16:36:06 gibi #topic Stable Branches
16:36:14 gibi elodilles: any news from stable/
16:36:15 gibi ?
16:36:20 elodilles yepp
16:36:24 elodilles #info stable/train was unblocked last week (with the backport of the patch that migrated jobs to zuul v3)
16:36:40 elodilles #info now only stable/stein (and older) are blocked: grenade and other devstack based jobs fail with the same timeout issue as stable/train was previously
16:36:50 elodilles #info stable branch status / gate failures tracking etherpad: https://etherpad.opendev.org/p/nova-stable-branch-ci
16:37:19 sean-k-mooney elodilles i think there is an issue with cliff on yoga and older by the way
16:37:30 sean-k-mooney more for cilometer then us
16:37:32 elodilles sean-k-mooney: is it? :-o
16:37:49 sean-k-mooney but cliff 4. something droped python 3.6 support
16:37:59 sean-k-mooney so centos 8 stream jobs are failing
16:38:13 sean-k-mooney i did see it on stable/victoria on nova however if i am rememebring correctly
16:38:17 elodilles isn't that non-voting?
16:38:35 elodilles i mean in that case it's not blocking the gate
16:38:47 sean-k-mooney not for nova
16:39:09 sean-k-mooney i work with peopel that work on ceilometer that are trigng to fix it for there jobs
16:39:23 sean-k-mooney just mentioning it as an fyi incase you see stabel review related to it
16:39:41 elodilles sean-k-mooney: ack, thanks, i'll look into that
16:40:09 sean-k-mooney https://review.opendev.org/c/openstack/requirements/+/854130
16:40:34 elodilles sean-k-mooney: ++
16:40:59 elodilles i've noted that patch
16:41:29 gibi anything else about stable?
16:41:42 elodilles nothing else from me
16:41:45 gibi thanks elodilles
16:41:49 elodilles np
16:42:05 gibi #topic Open discussion
16:42:10 gibi nothing on the agenda.
16:42:25 gibi does anybody has a topic still for today?
16:43:27 gibi I take the silence as no
16:43:33 gibi so thanks for joining
16:43:35 opendevmeet Log: https://meetings.opendev.org/meetings/nova/2022/nova.2022-08-23-16.00.log.html
16:43:35 opendevmeet Minutes (text): https://meetings.opendev.org/meetings/nova/2022/nova.2022-08-23-16.00.txt
16:43:35 opendevmeet Minutes: https://meetings.opendev.org/meetings/nova/2022/nova.2022-08-23-16.00.html
16:43:35 opendevmeet Meeting ended Tue Aug 23 16:43:35 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4)
16:43:35 gibi #endmeeting
16:43:58 elodilles thanks o/
16:47:48 sean-k-mooney elodilles: i put the details in the etherpad
16:47:55 sean-k-mooney https://etherpad.opendev.org/p/nova-stable-branch-ci#L22
16:50:16 JayF btw; thanks to you all for the help with the stable reviews, trying to get the ironic driver as good as it can be in the stable branch. https://review.opendev.org/c/openstack/nova/+/821350 still needs another core vote in victoria
16:50:36 JayF I know you rarely need us, but I'm happy to help if you all run into Ironic shenanigans in the future :D
16:51:41 gibi sean-k-mooney: fyi I created a test case and now I can confirm that different PFs are tracked as separate PCI pools even if they have same kind of VFs. Fortunately both the apply and the consume logic in stats able to consume from two pools for a single pci request. So everything works \o/ even with splitted pools
16:51:46 sean-k-mooney well there is one open spec that we really need your import on
16:52:18 sean-k-mooney gibi: nice
16:52:37 sean-k-mooney JayF: if you have not read https://review.opendev.org/c/openstack/nova-specs/+/842015
16:52:51 sean-k-mooney JayF: we dont currently have a good way forward on that
16:53:33 JayF I've absolutely had backchannel chats with Julia about that (and felt the pain it is trying to solve). I will scratch my brain for it again but I'm not sure I know how to break out of the logjam
16:54:32 sean-k-mooney we were hopign to have a face to face session on this at the ptg before it went virtual
16:54:52 sean-k-mooney it really feels like it needs a white board and some time to talk thing through
16:54:59 JayF I have $800 of Alaska Airlines credit proving my intention to have showed up to that :(
16:55:19 JayF hopefully we can find time in virtual summit, anyway, although you're right it's often easier to solve things irl
16:56:36 sean-k-mooney +2w on that victoria backport
16:57:03 JayF wonderful, thanks. I'll have probably three fresh in the morning as I pull everything down another level.
16:57:30 JayF sean-k-mooney: https://etherpad.opendev.org/p/JayFTODO I put all the branches on this so I'd stop doing that
16:57:48 JayF sean-k-mooney: it's worse for Ironic, we have mid-stream bugfix releases that are numbered, and they might as well be named after colors for all the meaning the names have for me lolsob
16:58:28 sean-k-mooney i see :)
17:01:07 sean-k-mooney ok im gong to more or less call it a day.
17:01:34 sean-k-mooney ill likely do some code review on my ipad while i cook/order dinner but im going to drop off irc ffor the day
17:01:45 gibi sean-k-mooney: have a nice evening o/
17:15:02 opendevreview Elod Illes proposed openstack/nova stable/train: [ironic] Minimize window for a resource provider to be lost https://review.opendev.org/c/openstack/nova/+/853546
17:16:05 JayF elodilles: thanks, I had that on my todo list but I appreciate it \o/
17:22:33 elodilles JayF: np, just re-applied the cherry-pick to have the right format of the commit message, otherwise it seems right, so +2'd it already
17:22:46 JayF So, what you want in the commit is
17:23:05 elodilles it's not the commit message in general o:)
17:23:06 JayF cherry-picked-from (master SHA)\ncherry-picked-from (master-1 sha)\n etc etc
17:23:17 JayF which is achieved by a clean cherry pick from stable N -> stable N-1
17:23:23 elodilles but to do the cherry-picking branch-by-branch,
17:24:22 elodilles so that one can see that the cherry picking was in the right order, from the right (latest) patch sets, etc.
17:25:09 JayF so for my most recent master change, I'd cherry-pick it to stable/yoga, then cherry pick *the yoga change* onto xena, and so on
17:25:40 JayF and if at any point one patch in the cycle is changed or rebased (whoops), the ones after it have to be re-cherry-picked
17:26:26 elodilles JayF: exactly :)
17:26:44 JayF is this a nova-ism? Or something generally that most OpenStack projects do that Ironic has never really enforced?
17:27:54 elodilles the backports should be done branch-by-branch according to stable policy
17:28:08 JayF ack, so I'll update how I do it in Ironic world too
17:28:37 elodilles but yes, probably only nova has this check as (we) stable cores were picky about this one o:)
17:29:00 JayF Yeah, I've been focused on stable reviews in Ironic for a long time, and this is *never* something I've checked for
17:29:03 elodilles to avoid wrong/not latest/missing backports
17:29:11 JayF frankly, I doubt I'd -1 a change for this now -- but I do always manually check the earlier branches
17:29:18 JayF and I'll start doing it right for me ;D
17:29:22 JayF **my changes
17:30:12 elodilles :)
17:32:35 elodilles also note, that the rebase is unnecessary as Zuul is always doing it. the only time when it is needed, when there's a merge conflict
17:43:54 JayF tbh rebase button is my shortcut for "clear verify and re-run tests on this" except now I realize there are negative side effects in this context
17:59:47 opendevreview Balazs Gibizer proposed openstack/nova master: Request filter for PCI in placement https://review.opendev.org/c/openstack/nova/+/852771
17:59:48 opendevreview Balazs Gibizer proposed openstack/nova master: Map PCI pools to RP UUIDs https://review.opendev.org/c/openstack/nova/+/854118
17:59:48 opendevreview Balazs Gibizer proposed openstack/nova master: Support resource_class and traits in PCI alias https://review.opendev.org/c/openstack/nova/+/853316
17:59:49 opendevreview Balazs Gibizer proposed openstack/nova master: Filter PCI pools based on Placement allocation https://review.opendev.org/c/openstack/nova/+/854120
17:59:49 opendevreview Balazs Gibizer proposed openstack/nova master: Make allocation candidates available for scheduler filters https://review.opendev.org/c/openstack/nova/+/854119
17:59:50 opendevreview Balazs Gibizer proposed openstack/nova master: Func test for PCI in placement scheduling https://review.opendev.org/c/openstack/nova/+/854122
17:59:50 opendevreview Balazs Gibizer proposed openstack/nova master: Store allocated RP in InstancePCIRequest https://review.opendev.org/c/openstack/nova/+/854121
17:59:51 opendevreview Balazs Gibizer proposed openstack/nova master: Test PCI scheduling during move operations https://review.opendev.org/c/openstack/nova/+/854247
18:08:11 opendevreview Merged openstack/nova master: Revert "Test attached volume extend actions in the nova-next job" https://review.opendev.org/c/openstack/nova/+/854132
18:23:41 opendevreview Elod Illes proposed openstack/nova stable/queens: WIP: [stable-only] Remove grenade jobs https://review.opendev.org/c/openstack/nova/+/854252
18:47:52 opendevreview Merged openstack/nova master: nova-live-migration tests not needed for Ironic https://review.opendev.org/c/openstack/nova/+/853529

Earlier   Later