Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2023-11-14
16:19:15 bauzas who else nays ?
16:19:59 gibi yay from me
16:20:10 gibi at least I can add implementation to review :)
16:20:18 bauzas cool
16:20:21 dansmith heh
16:20:27 gibi https://review.opendev.org/q/topic:bp%252Flibvirt-migrate-with-hostname-instead-of-ip
16:20:33 dansmith I admit I'm biased since I just pushed an implementation _to_ review :P
16:20:40 bauzas #agreed Tomorrow (Nov 15) will be implementation review day
16:20:42 gibi same here :)
16:21:21 bauzas #action bauzas to create an etherpad for tracking features and bugs like we agreed at PTG, so it would be used tomorrow
16:21:25 sean-k-mooney well any review that are rety can be reviewd including they ones you just pushed so
16:21:39 bauzas I too much punted that action item
16:22:07 bauzas #action bauzas to notify by email about both the implementation review day and the etherpad to use
16:22:18 bauzas the email will be late today for sure
16:23:22 bauzas #topic Review priorities
16:23:32 bauzas #action bauzas need to create the etherpad he just said above
16:23:41 bauzas #topic Stable Branches
16:23:46 bauzas elodilles: I'm all ears
16:23:53 elodilles o7
16:24:02 elodilles #info stable gates don't seem blocked
16:24:20 elodilles i've seen some failing jobs but nothing constantly failing...
16:24:34 elodilles (or I'm not aware of it)
16:24:41 elodilles #info stable release patches still open for review: https://review.opendev.org/q/project:openstack/releases+is:open+intopic:nova
16:24:53 elodilles +1: yoga is going to be unmaintained, so final stable/yoga release should happen ASAP - https://etherpad.opendev.org/p/nova-stable-yoga-eom
16:25:34 elodilles if we want to do a final release at all
16:26:01 bauzas that's a need
16:26:08 elodilles otherwise we can just live with the last release from summer(?)
16:26:20 bauzas elodilles: nah, I'd prefer we do some release
16:26:31 bauzas elodilles: bug me this week or next week
16:26:37 elodilles now we have 1 merged patch as can be seen in the etherpad
16:26:50 bauzas I guess you know that I'm busy, so I'll forget if nobody harasses me
16:27:11 elodilles bauzas: i'll ping you then some time
16:27:18 bauzas thanks
16:27:31 elodilles the open patches are also listed on the etherpad
16:27:40 bauzas I'm unfortunately not very good at doing multiple things at the same time
16:27:42 elodilles feel free to use the pad, edit it if you want
16:27:50 bauzas like, my mind is not good for concurrency
16:28:46 elodilles i'll try to add details to the pad so that will be easier to reviewers to look at 'things-to-be-reviewed'
16:28:54 bauzas ack
16:29:01 elodilles anyway, last but not least:
16:29:04 elodilles #info stable branch status / gate failures tracking etherpad: https://etherpad.opendev.org/p/nova-stable-branch-ci
16:29:25 elodilles add stable gate issues here if you find one ^^^
16:29:36 elodilles and that was all from me
16:30:08 bauzas cool
16:30:13 bauzas last topic
16:30:19 bauzas #topic open discussion
16:30:22 bauzas anything anyone ?
16:30:26 dvo-plv o/ In the open discussion topic I would like to remind about ability to reviwe this cool spec file:) https://review.opendev.org/c/openstack/nova-specs/+/895924/8
16:30:35 dvo-plv Sorry, I forgot to fill agenda with this
16:30:43 sean-k-mooney i have two related things we we can talk about after ^
16:31:40 sean-k-mooney dvo-plv: if dansmith and bauzas feel stronly that the tait is not required and want to use a min compute service check instead i can relent on that
16:32:00 sean-k-mooney but you will need to modify the spec to say that the feature will only work when its fully upgraded
16:32:20 sean-k-mooney bauzas: dansmith ^ is that still your prefernce
16:32:39 dansmith I have never been super opinionated about it.. I'm pretty meh all around
16:32:50 dvo-plv i beleive that we alreadt agreed with trait usage
16:32:51 dansmith I still feel like a trait is just too much work for such a thing
16:33:13 dvo-plv I added upgrade impact https://review.opendev.org/c/openstack/nova-specs/+/895924/8/specs/2024.1/approved/virtio_packedring_configuration_support.rst#126
16:33:33 sean-k-mooney dvo-plv: is this what you wanted to discuss by the way
16:33:43 sean-k-mooney i want to make sure we have a path to unblocking you
16:34:27 dvo-plv yes, I also want a path to unblock this feature and i thought that nova core already agreed with that
16:34:41 dvo-plv so I've updated spec and nova code
16:34:46 sean-k-mooney im fine with the trait i assuemd that was the only open
16:34:54 sean-k-mooney i can re review the new version of the spec
16:35:02 bauzas I think I said loudly a couple of times that I eventually give up
16:35:18 bauzas so, I let other cores accept using traits
16:36:04 bauzas my only concern is that I just want to make sure nobody would use that trait in a flavor so we could eventually remove it if we want, but given I won't review the spec, I leave others to say what they want
16:36:47 dvo-plv I suggest to add thi item somewhere regarding trait usage
16:37:09 dvo-plv but it does not required as far as it will assigned automatically
16:37:45 dvo-plv https://review.opendev.org/c/openstack/nova/+/876075/27/nova/scheduler/request_filter.py#275 here for instance
16:38:08 dvo-plv https://review.opendev.org/c/openstack/nova/+/876075/27/nova/virt/libvirt/driver.py#12429 here for node
16:40:33 sean-k-mooney so can i suggest we take a look at this (the spec and the implematnion) tomorrow n the implemation review day
16:40:37 sean-k-mooney for those that are interested
16:40:58 bauzas I am not but that's a good idea
16:41:24 bauzas anyway, can we end the meeting now ?
16:41:34 sean-k-mooney i would like to add 2 thigns quickly
16:41:42 sean-k-mooney https://blueprints.launchpad.net/nova/+spec/remove-hyperv-driver and https://blueprints.launchpad.net/nova/+spec/remove-vmware-support
16:42:00 sean-k-mooney are we ok to appove those or would you like use to expeand the description with more details
16:42:35 bauzas I think gmann wrote specs
16:42:45 sean-k-mooney yes which they abandoned
16:42:51 bauzas okay
16:42:55 sean-k-mooney because multiple of us said it was not requried
16:42:58 bauzas so we need to review both blueprints in a meeting
16:43:04 bauzas yeah, including me
16:43:11 dvo-plv Thank you
16:43:17 sean-k-mooney so i just want to decied on a path forward for the paper work
16:43:21 bauzas but I don't wanna force someone to not write a spec if he/she really wants to :)
16:43:22 sean-k-mooney if we want to review them next week we can
16:43:37 sean-k-mooney but i would like to merge the first patches for the removal sooner rather then later
16:43:38 bauzas sure
16:43:44 bauzas and sure
16:43:58 sean-k-mooney bauzas: i was orgianlly hoping ot merge the vmware removal patch tomorrow in the implemation day
16:44:02 sean-k-mooney but it can wait
16:44:12 bauzas then, we can approve those today
16:44:15 bauzas folks,
16:44:32 bauzas anyone having concerns with https://blueprints.launchpad.net/nova/+spec/remove-hyperv-driver as a specless BP for one ?
16:46:26 sean-k-mooney im fine with that. for the vmware one i do have more patches ill need to write to complete the removal but the inital patch https://review.opendev.org/c/openstack/nova/+/897017 is the entirty of the driver removal. i need to remove the console and objects in follow up patches
16:46:38 bauzas okay, so
16:46:52 bauzas #agreed https://blueprints.launchpad.net/nova/+spec/remove-hyperv-driver to be accepted as a specless bp
16:46:57 bauzas next one
16:47:22 bauzas anyone having concerns with https://blueprints.launchpad.net/nova/+spec/remove-vmware-support as a specless BP ?

Earlier   Later