Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2021-08-31
16:26:00 gibi review is there: #link https://review.opendev.org/c/openstack/releases/+/800755
16:26:04 gibi I will do a bp status cleanup tomorrow and along that I will update the highlights patch. But you can help me with the highlight by bringing up features in the comments.
16:26:13 opendevreview Stephen Finucane proposed openstack/nova master: policy: Deprecate field from 'os-extended-server-attributes' policy https://review.opendev.org/c/openstack/nova/+/806131
16:26:13 opendevreview Stephen Finucane proposed openstack/nova master: api: Add support for 'hostname' parameter https://review.opendev.org/c/openstack/nova/+/778550
16:26:14 opendevreview Stephen Finucane proposed openstack/nova master: tests: Address nits for configurable-instance-hostnames series https://review.opendev.org/c/openstack/nova/+/806735
16:26:14 opendevreview Stephen Finucane proposed openstack/nova master: tests: Speed up 'servers' API tests https://review.opendev.org/c/openstack/nova/+/778732
16:26:44 gibi the other todo is
16:26:45 gibi We need a volunteer to write the release notes prelude (deadline for the prelude is RC1)
16:27:04 dansmith gosh I wonder who will do that
16:27:44 bauzas hmmm
16:28:01 gibi :D
16:28:08 bauzas but ok, I can volunteer for it
16:28:12 gibi thanks
16:28:55 gibi anything else about the release before we jump to open bps?
16:29:41 gibi #topic Review priorities
16:30:06 gibi this is a new topic as the gerrit patch enabling the new Review-Priority label has been landed
16:30:47 gibi you can query the label with something like https://review.opendev.org/q/project:openstack/nova+label:Review-Priority%252B1
16:31:15 gibi the doc about the label is here https://review.opendev.org/c/openstack/nova/+/792357
16:31:53 gibi anyhow as we are close the FF sean-k-mooney collected couple of bps from our list that we quickl agree on to mark as priority
16:32:03 gibi instance host name https://review.opendev.org/q/topic:%22bp%252Fconfigurable-instance-hostnames%22+(status:open%20OR%20status:merged)
16:32:14 gibi this is basically approved
16:32:17 gibi as far as I see
16:32:32 bauzas let me ask a stupid question
16:32:41 bauzas do we have a formal process on how to use this flag ?
16:32:47 gmann or we can run this to filter already +W one #link https://review.opendev.org/q/project:openstack/nova+Review-Priority:1+label:Verified%253D1++NOT+label:Workflow%253C%253D-1
16:32:53 bauzas I remember some change I reviewed a while ago
16:32:56 gibi bauzas: this is where we discussing that https://review.opendev.org/c/openstack/nova/+/792357
16:33:01 bauzas but did we get into some direction ?
16:33:08 sean-k-mooney bauzas: yes
16:33:12 gibi I see you have negative comments there :)
16:33:31 sean-k-mooney well it was +w'd so i assuems we had agreed
16:33:38 bauzas I can see some consensus
16:33:46 bauzas so I can rereview this
16:34:09 gibi sure I have to back to that too
16:34:43 gibi lets then just quickly check some of the open bps that close to lend and ignore the new label until we agree on the doc
16:34:53 gibi s/lend/land
16:35:15 gibi so instance hostname is a basically done we can move on
16:35:20 gibi vhost multi queue flavor extra spec https://review.opendev.org/q/topic:%22bp%252Fmultiqueue-flavor-extra-spec%22+(status:open%20OR%20status:merged)
16:35:26 sean-k-mooney well i would prefer to use the new lable and adjust our useage if we need too in a follow up t be hosest but sure lets proced
16:36:01 gibi sean-k-mooney: I don't want to hold up the meeting agreeing on the usage as we have FF this week
16:36:19 gibi so the multique series is a simple one but still lack reviews
16:36:19 sean-k-mooney gibi sure we can move on
16:36:58 sean-k-mooney ya tl;dr that is supporting a mutlique via the flaovr in addtion to the existing image property
16:37:18 bauzas gibi: I was looking into the multiqueue series before the meeting
16:37:22 sean-k-mooney its pretty straigt forward but need reviews
16:37:32 bauzas so I could mark it with Review-Prio flag :p
16:37:38 gibi bauzas: you can :P
16:38:16 gibi if nobody else shows up I can read that too
16:38:21 gibi moving on
16:38:26 gibi generic mdevs https://review.opendev.org/q/topic:%22bp%252Fgeneric-mdevs%22+(status:open%20OR%20status:merged)
16:38:31 gibi as far as I see the code is approved
16:38:36 gibi the doc patch is open
16:38:45 bauzas thanks all
16:38:58 gibi we can land doc only patches after FF so this is in good shape
16:38:59 bauzas I can work on the doc change btw.
16:39:12 bauzas but my priority for this weeks is reviews.
16:39:21 gibi that sounds good to me
16:39:25 gibi moving on
16:39:30 gibi pps qos https://review.opendev.org/q/topic:%2522bp/qos-minimum-guaranteed-packet-rate%2522+(status:open+OR+status:merged)+project:openstack/nova
16:39:37 gibi this is easy as we don't have to rush
16:39:57 gibi the neturon API extension enabling the new logic probably wont land in Xena
16:40:06 gibi (but already works on in my env)
16:40:39 sean-k-mooney ah ok so this is moving to yoga then?
16:40:46 gibi the neutron impact seems so
16:41:05 gibi the nova impact is good to land now but it wont be triggered if neutron does not have the new api extension
16:41:10 gibi so it is dead code
16:41:47 sean-k-mooney i mean if its dead code its also going to have little or no impact on people deploying xena but sure
16:41:54 gibi yes
16:42:08 gibi moving on
16:42:22 gibi melwitt updated the unified limits https://review.opendev.org/q/topic:%22bp%252Funified-limits-nova%22+(status:open%20OR%20status:merged) sries
16:42:50 dansmith I saw her note about not reaching into limit internals but haven't gone back to look
16:42:50 gibi dansmith: you had comments before. Will you have time this week to look back to this?
16:43:14 dansmith I will, but are we really expecting to land this? if we are, then I'll try to prioritize, but if not, I'd focus on other things that are time-critical
16:43:30 dansmith seems like there are a lot of patches yet to land for thursday,
16:43:45 dansmith and I know some have had some review, but I don't think all (but could be wrong)
16:43:53 bauzas agreed with dansmith
16:44:06 bauzas I'd love this series to merge, but this is very late in time
16:45:33 gibi dansmith: it was open for a long time so I guess it is not that time critical
16:46:27 dansmith lots of odd gate fail on the bottom few, which I'm sure is not related to those patches, but just in terms of likelihood for merging all of that by thursday seems unlikely
16:46:33 gibi I don't see that much thing to close to land in nova but I guess you have commitments outside of nova too
16:46:53 dansmith yeah I meant other stuff
16:47:04 dansmith we should recheck the bottom three that are +W at least I guess
16:47:21 gibi yepp
16:47:40 dansmith the API ones at the top of the set are some I want to look closer at, and I haven't looked at the tempest tests that are available yet
16:48:08 gibi ack. if you not get to it the it will be moved to yoga, no hard feelings :)
16:48:21 gibi moving on
16:48:29 gibi here are the rest of the open bps https://launchpad.net/nova/+milestone/xena-3
16:48:45 gibi (some of the should be closed which is will do tomorrow)_
16:49:02 gibi is there any of those somebody want to get attention?
16:49:31 gmann remove tenant-id is still not in shape for Xena, we are still need all code up before we start merging first patch
16:49:46 gmann may be for Yoga
16:49:56 gibi gmann: thanks for the info. that is also a long time open series.
16:50:05 gmann yeah.
16:50:30 gmann we are reviewing it on time but just all the code should be up before we start merging it
16:50:54 gibi yeah, that is a big change in a single microversion.
16:51:01 gibi any other bp?
16:51:32 gibi the moving on
16:51:35 gibi #topic PTG Planning
16:51:39 gibi every info is in the PTG etherpad #link https://etherpad.opendev.org/p/nova-yoga-ptg
16:51:45 gibi please add topics ^^

Earlier   Later