| *** ministry is now known as __ministry | 00:06 | |
| opendevreview | Merged openstack/nova master: Add service version check workaround for FFU https://review.opendev.org/c/openstack/nova/+/826097 | 00:37 |
|---|---|---|
| opendevreview | Merged openstack/nova master: block_device: Ignore VolumeAttachmentNotFound during detach https://review.opendev.org/c/openstack/nova/+/812127 | 00:48 |
| opendevreview | Merged openstack/nova master: Add check job for FIPS https://review.opendev.org/c/openstack/nova/+/790519 | 01:34 |
| *** hemna9 is now known as hemna | 07:38 | |
| plibeau | hello, sean-k-mooney thx for the review, lyarwood if you have time to review: https://review.opendev.org/c/openstack/nova/+/820531 | 08:39 |
| *** Uggla|afk is now known as Uggla | 08:47 | |
| bauzas | sean-k-mooney: gibi: when you're around, I'm about working on a specific implementation detail for https://specs.openstack.org/openstack/nova-specs/specs/yoga/approved/boot-vm-with-unaddressed-port.html | 10:12 |
| bauzas | sean-k-mooney: gibi: in https://review.opendev.org/c/openstack/nova/+/669411/2/nova/network/neutron.py stephenfin said we should verify the l2 connectivity for the port binding | 10:12 |
| bauzas | sean-k-mooney: gibi: I can do it in https://github.com/openstack/nova/blob/master/nova/network/neutron.py#L3546 | 10:13 |
| bauzas | sean-k-mooney: gibi: but what kind of exception should I provide if it's not cool ? | 10:14 |
| bauzas | exception.PortUpdateFailed I guess? | 10:14 |
| gibi | hm, | 10:14 |
| gibi | the faulty case is when a port has ip_allocation=none but when it is bound neutron sets connectivity to other than l2 | 10:15 |
| gibi | but this should be a neutron error, isn't it? | 10:15 |
| gibi | I mean such port has no sense | 10:15 |
| bauzas | gibi: this is correct iiuc | 10:16 |
| gibi | could neutron detect this and fail the binding? | 10:16 |
| gibi | that would be a cleaner approach | 10:16 |
| bauzas | gibi: well, I don't know | 10:17 |
| gibi | sean-k-mooney: ^^ ? | 10:17 |
| bauzas | stephenfin: if you're around, I'd like to understand why you wanted to verify the l2 connectivity with https://review.opendev.org/c/openstack/nova/+/669411/2/nova/network/neutron.py | 10:17 |
| * stephenfin looks | 10:18 | |
| bauzas | thanks | 10:18 |
| bauzas | context : unaddressed ports | 10:18 |
| bauzas | we haven't discussed it in the spec | 10:18 |
| stephenfin | Ah, so ralonsoh is really the person to ask. He told me to do that :) If I were to guess, it's because deferred IP allocation obviously only makes sense for L2 ports (you must have an IP to operate at layer 3) but that's a guess | 10:22 |
| bauzas | stephenfin: yeah, I understand why | 10:22 |
| bauzas | I mean the "why we should verify" | 10:23 |
| bauzas | but gibi had concerns with "where we should do it" | 10:23 |
| bauzas | and I'm quite able with him | 10:23 |
| ralonsoh | bauzas, there are some backends that don't allow to have IP-less ports | 10:23 |
| stephenfin | ah, yes, I didn't read all the conversation | 10:23 |
| ralonsoh | this is why we introduced this parameter | 10:23 |
| bauzas | ralonsoh: sure, I understand but why it should be nova which should verify when binding ? | 10:24 |
| bauzas | and why not neutron when creating the port ? | 10:24 |
| ralonsoh | because the port creation is just a DB representation | 10:24 |
| ralonsoh | this is not bound to any backend | 10:24 |
| bauzas | until binding, I guess then ? | 10:24 |
| ralonsoh | yes, that's the point | 10:25 |
| bauzas | ralonsoh: ok, then why it should be nova which would verify it when binding and why not neutron ? | 10:25 |
| ralonsoh | because this is how it was designed | 10:26 |
| ralonsoh | we can change it back again | 10:26 |
| bauzas | gibi: ^ | 10:27 |
| gibi | ack | 10:28 |
| bauzas | ralonsoh: sorry, havn't seen it in https://specs.openstack.org/openstack/nova-specs/specs/yoga/approved/boot-vm-with-unaddressed-port.html | 10:28 |
| gibi | I do belive that if the port binding does not make sense then such bindig should be rejected by neutron | 10:28 |
| gibi | but I don't have deep understanding of the neutron side of this | 10:28 |
| bauzas | oh my bad, this was somehow explained in the spec : | 10:29 |
| bauzas | " The changes introduced as part of the “Port binding event extended information for Nova” 4 spec means neutron will now provide the type of back-end to which the port is bound, with the parameter connectivity, included now in binding:vif_details. Nova can determine whether a given driver back-end has “l2” connectivity and, if so, know that a port without an IP address can be assigned to a virtual machine." | 10:29 |
| bauzas | I guess I'd appreciate sean-k-mooney's thoughts on this one | 10:30 |
| * bauzas also needs to jump on an internal bug priority :( | 10:30 | |
| gibi | if this was agreed before then I rest my case | 10:32 |
| bauzas | gibi: well, yes and no | 10:35 |
| bauzas | this was agreed in a neutron spec | 10:35 |
| bauzas | this wasn't really discussed in the nova spec | 10:35 |
| bauzas | except saying "look, we could have neutron backends that'd have problems, we should verify the connectivity" | 10:35 |
| bauzas | but we accepted it as it's phrased | 10:36 |
| ralonsoh | bauzas, this can be changed and we can make Neutron to decide this | 10:36 |
| ralonsoh | if we have the port connectivity value and the backend one too, that's easy | 10:36 |
| bauzas | ralonsoh: I think I'm personnally OK with moving on and doing this check in nova first but I'd somehow appreciate second thoughts for a neutron change too | 10:37 |
| ralonsoh | bauzas, I'll propose this in a drivers meeting | 10:38 |
| ralonsoh | if a new spec is needed, I'll push it | 10:38 |
| bauzas | well, that'd mean a Z change | 10:38 |
| ralonsoh | yes, in Z probably | 10:38 |
| ralonsoh | well, I'm not sure | 10:38 |
| bauzas | that's why I'm saying I'm OK with checking this in nova first if gibi is OK | 10:38 |
| bauzas | (provided I'm able to write such thing :D ) | 10:39 |
| ralonsoh | bauzas, I'll raise this question this friday, at 1400UTC | 10:39 |
| ralonsoh | drivers meeting. The change should be small in Neutron | 10:40 |
| ralonsoh | bauzas, good news | 10:45 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/678027 | 10:45 |
| ralonsoh | (sorry, I didn't remember that part) | 10:45 |
| ralonsoh | Neutron will reject this port binding | 10:45 |
| ralonsoh | and that was merged in Train | 10:45 |
| bauzas | ralonsoh: ok, so no need for a check in nova | 10:46 |
| bauzas | all good | 10:46 |
| bauzas | I can remove stephenfin's WIP | 10:46 |
| bauzas | and repropose it against Yoga nova | 10:46 |
| gibi | bauzas, ralonsoh: https://review.opendev.org/c/openstack/neutron/+/678027 is the best outcome of this discussion :) | 11:09 |
| chateaulav | sean-k-mooney: appreciate the os-traits review! | 12:02 |
| bkranendonk | hi folks, is there a way to utilize multiple image backends (e.g. rbd for instance X and lvm for instance Y) on the same hypervisor? | 12:55 |
| bkranendonk | cant find info on this | 12:55 |
| lyarwood | bkranendonk: not with the libvirt driver, it's a single static choice in the compute config used by all instances hosted on it | 13:08 |
| lyarwood | bkranendonk: if you want choice use volume types has always been our suggestion | 13:09 |
| bkranendonk | lyrarwood: Boot from volume with a volume_type? | 13:09 |
| sean-k-mooney | bkranendonk: yep | 13:10 |
| bkranendonk | ok, thanks! | 13:10 |
| lyarwood | bkranendonk: I think we proxy the type yeah, otherwise just create the volume directly in cinder with a given type and then use it in nova | 13:10 |
| sean-k-mooney | the images type can be used to set a host wide default stoage backing and then you can suplement that with cinder volumes and volume types to select based on differnt perfromacne requiements | 13:10 |
| lyarwood | yeah block_device_mapping_v2.volume_type is available from 2.67 | 13:11 |
| elodilles | bauzas: i'll add some update to meetings wiki if you are not doing that right now | 13:30 |
| bauzas | elodilles: sure, please do | 13:46 |
| elodilles | bauzas: thx, done | 13:55 |
| sean-k-mooney | bauzas: you were asking previously about adressless port and why nova needs to check the connectivty? | 14:51 |
| sean-k-mooney | did you get an answer | 14:51 |
| sean-k-mooney | its a security issue | 14:51 |
| sean-k-mooney | for backends like calico that provie l3 only connectivity its invlade to use l2 networking and it will not be fucntional | 14:52 |
| sean-k-mooney | in general port with l2 connectivy and no ip will not work with security groups proerly | 14:53 |
| sean-k-mooney | techincally you can magch on thernet type but almost all security group rules assme an ip | 14:53 |
| sean-k-mooney | the check tha tnova needs to is basicaly a simple reasonablness check | 14:54 |
| sean-k-mooney | e.g. is it reasonable given the info provide by neutron that the port would be usable without an addres. | 14:54 |
| sean-k-mooney | if the the connectivy type is l2 then yes | 14:54 |
| sean-k-mooney | if l3 then no | 14:54 |
| sean-k-mooney | simple as that | 14:54 |
| sean-k-mooney | bauzas: ralonsoh the connectivy par to fhte spec shoudl already be implemented in neutron | 14:55 |
| sean-k-mooney | the binding driver info is not implemtned but that is not needed for adress less ports | 14:55 |
| sean-k-mooney | here is where ml2/ovs reports l2 connectivy for example https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/drivers/openvswitch/mech_driver/mech_openvswitch.py#L59-L61 | 14:57 |
| sean-k-mooney | the main issue we have is this is obviouls a late fail on the compute node when we attpemt to do the port binding | 14:58 |
| sean-k-mooney | but that is better then booting with out any networking | 14:58 |
| sean-k-mooney | all of the in tree drivers supprot this feature as far as i am aware | 15:00 |
| sean-k-mooney | and they all provide l2 connectivity | 15:00 |
| sean-k-mooney | calico is really the only one i know of that is l3 only | 15:00 |
| bauzas | sean-k-mooney: ralonsoh told me that eventually Neutron verified the backend | 15:07 |
| bauzas | (from Train) | 15:07 |
| bauzas | sean-k-mooney: https://review.opendev.org/c/openstack/neutron/+/678027 | 15:08 |
| sean-k-mooney | it may but i think nova should still enforce the connectivy | 15:08 |
| bauzas | folks, nova meeting in 52 mins here at #openstack-nova | 15:08 |
| bauzas | sean-k-mooney: honestly, gibi and me were asking for that | 15:08 |
| bauzas | see the discussion we had this morning | 15:09 |
| sean-k-mooney | asking for nova ot also enforce it or just relay on neutron | 15:09 |
| gibi | if neutron enforces it then I think nova does not need to enforce it | 15:10 |
| ralonsoh | the port connectivity parameter was for Nova | 15:10 |
| sean-k-mooney | i guess im just surpesed because neutron did not want to enfoce it orginally which is why we added the connectivy | 15:10 |
| ralonsoh | in Neutron we check the backend connectivity (doesn't need to be in the port) | 15:10 |
| ralonsoh | and if the port has or not IP addresses | 15:10 |
| ralonsoh | sean-k-mooney, https://review.opendev.org/c/openstack/neutron/+/678027/3/neutron/plugins/ml2/managers.py | 15:11 |
| sean-k-mooney | right but we added the connectivty parmater becasue neuton did not want to do that backend check when we first dicussed this featre | 15:11 |
| ralonsoh | is this what you need? | 15:11 |
| sean-k-mooney | yes but that means we dont need the connectivty parmater in the respocne any more | 15:12 |
| ralonsoh | so we can remove it from the port dict | 15:12 |
| ralonsoh | in any case, the ML2 plugin should keep it | 15:12 |
| opendevreview | Merged openstack/nova master: libvirt: Create qcow2 disks with the correct size without extending https://review.opendev.org/c/openstack/nova/+/779275 | 15:12 |
| sean-k-mooney | am we could although it proably does not hurt to leave it there | 15:12 |
| ralonsoh | perfect, doesn't add any overhead | 15:12 |
| sean-k-mooney | the user might want to check what connectivy is aviaable to them i guess | 15:13 |
| bauzas | sean-k-mooney: ok, so you're OK if we don't verify it in nova ? | 15:13 |
| bauzas | as neutron already does it ? | 15:13 |
| sean-k-mooney | bauzas: ya given that bug fix im ok with it | 15:13 |
| bauzas | ++ | 15:13 |
| bauzas | simplier for me then | 15:13 |
| bauzas | I just need to rebase the change from stephenfin | 15:14 |
| sean-k-mooney | bauzas: as i said im just surprised that was done since it was orginaly rejected. | 15:14 |
| sean-k-mooney | yep | 15:14 |
| bauzas | and I'll just provide a Tempest test | 15:14 |
| sean-k-mooney | ralonsoh: qq while your here | 15:14 |
| sean-k-mooney | ralonsoh: did you recently add the binding driver to the port details as part of the recent ovn work you were doing | 15:15 |
| ralonsoh | yes | 15:15 |
| ralonsoh | let me find the patch | 15:15 |
| sean-k-mooney | it was part of https://specs.openstack.org/openstack/neutron-specs/specs/xena/ovn-auxiliary-port-bridge-live-migration.html right | 15:15 |
| sean-k-mooney | is "backend" now populated for all ml2 drivers | 15:15 |
| ralonsoh | https://review.opendev.org/c/openstack/neutron/+/814143 | 15:16 |
| ralonsoh | "connectivity" you mean | 15:16 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: WIP neutron: Allow to spawn VMs with port without IP address https://review.opendev.org/c/openstack/nova/+/669411 | 15:16 |
| ralonsoh | yes for all in-tree drivers | 15:16 |
| bauzas | there it goes ^ | 15:16 |
| ralonsoh | the default value is "legacy" | 15:16 |
| sean-k-mooney | ralonsoh: no i mean backend | 15:16 |
| sean-k-mooney | ralonsoh: https://specs.openstack.org/openstack/neutron-specs/specs/xena/ovn-auxiliary-port-bridge-live-migration.html#nova-neutron-events | 15:16 |
| sean-k-mooney | this is unrelated to the connectivy question | 15:17 |
| bauzas | holy shit, we actually don't need to have it WIP | 15:17 |
| ralonsoh | sean-k-mooney, TBH, I don't remember now, let me check | 15:17 |
| sean-k-mooney | bauzas: stephens patch? im pretty sure it worked in the past | 15:17 |
| sean-k-mooney | ralonsoh: i dont see the constant defiend in neutron-lib | 15:18 |
| sean-k-mooney | so i think this is missing again | 15:19 |
| ralonsoh | sean-k-mooney, this is part of VIF_DETAILS | 15:19 |
| ralonsoh | we don't need to add it to the API | 15:19 |
| opendevreview | Sylvain Bauza proposed openstack/nova master: neutron: Allow to spawn VMs with port without IP address https://review.opendev.org/c/openstack/nova/+/669411 | 15:19 |
| sean-k-mooney | ralonsoh: yes you do | 15:19 |
| bauzas | sean-k-mooney: stephenfin's rebased patch without WIP | 15:19 |
| sean-k-mooney | the keys in VIF_DETAILS are part of the prot binding extenion | 15:19 |
| ralonsoh | sean-k-mooney, yes, but not the content | 15:20 |
| sean-k-mooney | we normaly defien the keys of vif_details here https://github.com/openstack/neutron-lib/blob/master/neutron_lib/api/definitions/portbindings.py#L76 | 15:21 |
| sean-k-mooney | https://github.com/openstack/neutron-lib/blob/master/neutron_lib/api/definitions/portbindings.py#L49-L76 | 15:21 |
| sean-k-mooney | so if you have not defiend the key there its a bug IMO | 15:21 |
| ralonsoh | sean-k-mooney, I didn't add anything to the vif details yet | 15:21 |
| ralonsoh | I missed that part | 15:21 |
| sean-k-mooney | ack so the backend is not specifed ok | 15:22 |
| sean-k-mooney | so nova still cannot make decsion based on if its ovn | 15:22 |
| sean-k-mooney | vs ml2/ovs | 15:22 |
| ralonsoh | yeah sorry, I really missed that part | 15:23 |
| ralonsoh | it will be this release for sure | 15:23 |
| sean-k-mooney | ack no worries. i dont have time to start usign it this cycle anyway | 15:24 |
| sean-k-mooney | but when its there we might be able to adress some of the livmigratio/move op issue we have with ovn | 15:24 |
| sean-k-mooney | ralonsoh: on a related note we have merged the self-annouch change so that should help with ovn migration downtime | 15:25 |
| sean-k-mooney | ralonsoh:i have started the backports too | 15:25 |
| ralonsoh | sean-k-mooney, perfect | 15:25 |
| bauzas | last reminder : nova meeting in 8 mins here | 15:53 |
| bauzas | s/8/7 | 15:53 |
| bauzas | #startmeeting nova | 16:00 |
| opendevmeet | Meeting started Tue Jan 25 16:00:10 2022 UTC and is due to finish in 60 minutes. The chair is bauzas. Information about MeetBot at http://wiki.debian.org/MeetBot. | 16:00 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 16:00 |
| opendevmeet | The meeting name has been set to 'nova' | 16:00 |
| bauzas | gosh I was late | 16:00 |
| bauzas | howdy | 16:00 |
| gibi | of a whole 10 seconds :) | 16:00 |
| sean-k-mooney | o/ | 16:00 |
| bauzas | #link https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | 16:01 |
| bauzas | should be a quick one | 16:01 |
| elodilles | o/ | 16:02 |
| bauzas | okay, let's start | 16:02 |
| bauzas | #topic Bugs (stuck/critical) | 16:02 |
| bauzas | #info No Critical bug | 16:02 |
| bauzas | #link https://bugs.launchpad.net/nova/+bugs?search=Search&field.status=New 36 new untriaged bugs (+4 since the last meeting) | 16:02 |
| bauzas | #help Nova bug triage help is appreciated https://wiki.openstack.org/wiki/Nova/BugTriage | 16:02 |
| bauzas | #link https://storyboard.openstack.org/#!/project/openstack/placement 23 open stories (-2 since the last meeting) in Storyboard for Placement | 16:02 |
| bauzas | #undo | 16:02 |
| opendevmeet | Removing item from minutes: #link https://storyboard.openstack.org/#!/project/openstack/placement | 16:02 |
| bauzas | #link https://storyboard.openstack.org/#!/project/openstack/placement 25 open stories (+2 since the last meeting) in Storyboard for Placement | 16:03 |
| bauzas | there it is | 16:03 |
| bauzas | (sorry, wrong paste) | 16:03 |
| bauzas | any bug to discuss ? | 16:03 |
| gibi | yepp that +2 is the two any-traits ticket that are open again as I pushed implementation | 16:03 |
| bauzas | okidoki thanks | 16:04 |
| bauzas | ok, let's continue | 16:05 |
| bauzas | #topic Gate status | 16:05 |
| bauzas | #link https://bugs.launchpad.net/nova/+bugs?field.tag=gate-failure Nova gate bugs | 16:05 |
| bauzas | #link https://zuul.openstack.org/builds?project=openstack%2Fplacement&pipeline=periodic-weekly Placement periodic job status | 16:05 |
| bauzas | #info Please look at the gate failures, file a bug, and add an elastic-recheck signature in the opendev/elastic-recheck repo (example: https://review.opendev.org/#/c/759967) | 16:05 |
| bauzas | that's been a while since I've seen a gate failure | 16:05 |
| sean-k-mooney | as a house keeping thing | 16:06 |
| bauzas | so, /me crosses fingers for the last 5 weeks | 16:06 |
| sean-k-mooney | do we want to continue to suggest filing elastic-recheck patches | 16:06 |
| gibi | it was probably ignored so far so I'm not against dropping it | 16:07 |
| sean-k-mooney | well it was more related to the fact infra will be discontinuing it eventually | 16:07 |
| sean-k-mooney | and im not sure how many reviw the repo | 16:07 |
| bauzas | sean-k-mooney: well, good question | 16:08 |
| sean-k-mooney | its not really imporant but as a new contibutor i migth get confused | 16:08 |
| bauzas | at least I'll leave to ask to create bug report | 16:08 |
| sean-k-mooney | yes | 16:08 |
| bauzas | ok, done. | 16:08 |
| gibi | agree about asking for bugreports tagged with gate-failure | 16:09 |
| bauzas | gibi: done => Ctrl-R for https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting :) | 16:09 |
| gibi | thanks :) | 16:09 |
| bauzas | moving | 16:10 |
| bauzas | #topic Release Planning | 16:10 |
| bauzas | #info FeatureApprovalFreeze was Jan 13st | 16:10 |
| bauzas | #link https://releases.openstack.org/yoga/schedule.html#y-ff FeatureFreeze in 4 weeks | 16:11 |
| bauzas | (actually, 4.5 but meh) | 16:11 |
| bauzas | #link https://etherpad.opendev.org/p/nova-yoga-blueprint-status Etherpad for blueprints tracking | 16:12 |
| bauzas | I started to update it | 16:12 |
| bauzas | if people can look at it, it would be nice | 16:12 |
| bauzas | and if owners can provide their changes, it would be super too | 16:12 |
| bauzas | for the moment, I'm not asking to review them | 16:12 |
| gibi | bauzas: can I add the any-traits placement feature to the pad? | 16:13 |
| bauzas | gibi: sure | 16:13 |
| bauzas | actually, this would be better | 16:13 |
| bauzas | as I was only looking at the nova features | 16:13 |
| bauzas | so yeah, we should look at this etherpad rather than launchpad | 16:14 |
| bauzas | moving on ? | 16:15 |
| sean-k-mooney | sure | 16:15 |
| sean-k-mooney | i have bookmarked that now | 16:15 |
| bauzas | #topic Review priorities | 16:17 |
| bauzas | #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 | 16:17 |
| bauzas | nothing to tell | 16:18 |
| sean-k-mooney | i have one topic for opendiscuss by the way when we get there just an fyi | 16:18 |
| bauzas | maybe next week, we should wonder whether we should provide new priorities as we would be closer to ff | 16:18 |
| bauzas | sean-k-mooney: sure, should be in 2 mins | 16:19 |
| gibi | bauzas: yeah we should be agressive on priorities as we close to ff | 16:19 |
| bauzas | #link https://review.opendev.org/c/openstack/nova/+/816861 is now merged | 16:19 |
| bauzas | sean-k-mooney: if you want to provide a change for ^ | 16:19 |
| Uggla | bauzas, maybe it's me but the link above seems not ok. | 16:20 |
| sean-k-mooney | ack ill try and work on it sometime this week | 16:20 |
| sean-k-mooney | ill ping you when i push a patch | 16:20 |
| bauzas | Uggla: which one ? | 16:20 |
| bauzas | sean-k-mooney: sure thanks | 16:20 |
| Uggla | bauzas, the really long one with priorities. | 16:20 |
| bauzas | Uggla: I guess this is due to your IRC client | 16:21 |
| elodilles | for me it works | 16:21 |
| sean-k-mooney | it might be you client | 16:21 |
| bauzas | this is a oneliner | 16:21 |
| sean-k-mooney | i have to go to raw mode to click it | 16:21 |
| sean-k-mooney | otherwise the wrappign cuts it off | 16:21 |
| bauzas | ok, lemme see if I can provide a openstack bitly | 16:21 |
| Uggla | bauzas, you are right, sorry. | 16:21 |
| * bauzas can't remember the openstack bitly btw. | 16:22 | |
| bauzas | I mean the shortening URL | 16:22 |
| bauzas | anyway, lemme modify the url | 16:23 |
| bauzas | next topic ? | 16:23 |
| sean-k-mooney | sure | 16:23 |
| bauzas | #topic Stable Branches | 16:25 |
| bauzas | elodilles: floor is yours | 16:25 |
| elodilles | stable gates were impacted with centos-8-stream job failure again (it is non-voting again) | 16:25 |
| elodilles | #info stable gates are not blocked, but patches usually needs recheck due to intermittent failures | 16:25 |
| elodilles | #info heads-up: stable/victoria going to transition to Extended Maintenance in 3 months | 16:25 |
| elodilles | #info patches waiting for a 2nd +2 (please review some if you have time): https://review.opendev.org/q/project:openstack/nova+branch:%255Estable/.*+is:open+label:Code-Review%253D%252B2 | 16:25 |
| elodilles | and that's all | 16:26 |
| bauzas | cool and thanks | 16:27 |
| elodilles | no problem :] | 16:27 |
| bauzas | I'll try to do reviews but honestly the priorities will be for some spec implementations atm :) | 16:28 |
| elodilles | that's understandable | 16:28 |
| elodilles | yoga-3 is coming :) | 16:29 |
| bauzas | winter is already there too | 16:29 |
| elodilles | :S | 16:29 |
| bauzas | anyway | 16:30 |
| bauzas | #topic Sub/related team Highlights | 16:30 |
| bauzas | #info No subteam left | 16:30 |
| bauzas | nothing to say about it, let's move on | 16:30 |
| bauzas | #topic Open discussion | 16:30 |
| bauzas | nothing on the agenda, but sean-k-mooney had a thought | 16:30 |
| sean-k-mooney | cool | 16:31 |
| bauzas | shoot | 16:31 |
| sean-k-mooney | so related to the off path network backends | 16:31 |
| sean-k-mooney | https://review.opendev.org/c/openstack/os-traits/+/818514 | 16:31 |
| sean-k-mooney | gibi if you are oke with ^ still i will +w | 16:31 |
| sean-k-mooney | but i want to see if we can do an os trait release soon | 16:31 |
| gibi | sean-k-mooney: I'm OK | 16:31 |
| sean-k-mooney | either this week or next week | 16:31 |
| sean-k-mooney | any objectsion to doing a release soon? | 16:32 |
| sean-k-mooney | if not ill propose a patch to the releases repo | 16:32 |
| sean-k-mooney | there are one or two other patches up for review | 16:32 |
| sean-k-mooney | liek the one for emulation support | 16:32 |
| sean-k-mooney | https://review.opendev.org/c/openstack/os-traits/+/824050 | 16:32 |
| sean-k-mooney | so it might be good to to a quick scan of the os-traits reviews and merge thos that matter | 16:33 |
| sean-k-mooney | or rather correspond to thos that have approved specs | 16:33 |
| sean-k-mooney | any objections if i tag them as review priorities | 16:33 |
| bauzas | no | 16:34 |
| bauzas | no objections of course | 16:34 |
| bauzas | and yeah we could release | 16:34 |
| bauzas | if needed | 16:34 |
| bauzas | it's better to do it now rather than in 2 weeks | 16:34 |
| bauzas | we could do another release if needed for sure | 16:34 |
| sean-k-mooney | the nova unit/func test dont work with depend on so we will need a release before the code that uses them can merge | 16:34 |
| bauzas | really? | 16:34 |
| sean-k-mooney | ya tox pulls form pypi | 16:35 |
| bauzas | I thought we were able to Depends-on | 16:35 |
| sean-k-mooney | tempest works | 16:35 |
| bauzas | making sense... somehow | 16:35 |
| sean-k-mooney | since devstack using the repos prepared by zuul but ya lib deps dont work for tox | 16:35 |
| sean-k-mooney | ok thats all i had | 16:36 |
| bauzas | sean-k-mooneyok, ping me when you wanna release | 16:36 |
| bauzas | sean-k-mooney: ^ | 16:36 |
| bauzas | and I'll +1 | 16:36 |
| sean-k-mooney | ack will do | 16:36 |
| * gibi just approved the traits for the emulator feature | 16:36 | |
| bauzas | on the release patch | 16:36 |
| bauzas | ok, that's it for all we had | 16:36 |
| chateaulav | gibi: thanks! | 16:36 |
| bauzas | I guess we can wrap it up and return to what we did before | 16:36 |
| gibi | bauzas: +1 | 16:37 |
| bauzas | that being said, | 16:37 |
| bauzas | #endmeeting | 16:37 |
| opendevmeet | Meeting ended Tue Jan 25 16:37:50 2022 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:37 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/nova/2022/nova.2022-01-25-16.00.html | 16:37 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/nova/2022/nova.2022-01-25-16.00.txt | 16:37 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/nova/2022/nova.2022-01-25-16.00.log.html | 16:37 |
| sean-k-mooney | chateaulav: fyi both the trait patch i know of that should be in the next release are now making them selves through the gate | 16:38 |
| sean-k-mooney | chateaulav: so im going to check the rest but assuming nothing else jumps out at me ill propose a patch to the reslese repo later today | 16:38 |
| sean-k-mooney | chateaulav: there is curretly a blocker issue with the release repo | 16:39 |
| chateaulav | sean-k-mooney: awesome! appreciate it. | 16:39 |
| sean-k-mooney | so it likely will be a day or two before the release happens as we cant currently merge the patch | 16:39 |
| sean-k-mooney | gibi: thanks :) | 16:41 |
| opendevreview | Merged openstack/os-traits master: Adds Pick guest CPU architecture based on host arch in libvirt driver support https://review.opendev.org/c/openstack/os-traits/+/824050 | 16:46 |
| opendevreview | Merged openstack/os-traits master: Add a trait for remote_managed port-capable nodes https://review.opendev.org/c/openstack/os-traits/+/818514 | 16:48 |
| opendevreview | sean mooney proposed openstack/os-traits master: Add Train Ussuri Victoria release info https://review.opendev.org/c/openstack/os-traits/+/768889 | 16:52 |
| sean-k-mooney | does anyone rememeber what that sate of owner traits is | 16:55 |
| sean-k-mooney | https://review.opendev.org/c/openstack/os-traits/+/770569 | 16:55 |
| sean-k-mooney | did we decided we were going to add them or not | 16:56 |
| sean-k-mooney | im looking at what patches are old and can be abandonded | 16:56 |
| sean-k-mooney | looking at the ptg gibi you were goign to work on a spec fo addign a OWNER_NOVA trait https://etherpad.opendev.org/p/nova-yoga-ptg#L77 | 16:57 |
| sean-k-mooney | that was https://review.opendev.org/c/openstack/nova-specs/+/819510 | 16:58 |
| gibi | sean-k-mooney: there was somebody from cyborg promised to look into it | 16:58 |
| sean-k-mooney | but we are past the deadline so i guess we will pick it up next cycle again | 16:58 |
| sean-k-mooney | gibi: ya the proposed a spec | 16:59 |
| gibi | yepp the spec is stalled out on upgrade impact | 16:59 |
| sean-k-mooney | ah ok | 16:59 |
| sean-k-mooney | ill abandon the os-traits patch for now and link to the spec | 16:59 |
| sean-k-mooney | we can reopen the review when we figure out the upgrade bit | 16:59 |
| gibi | OK, thanks | 17:01 |
| sean-k-mooney | elodilles: do you know where the release team hang out on irc | 17:02 |
| sean-k-mooney | i guess i coudl hop on infra too | 17:02 |
| sean-k-mooney | elodilles: just wonderign what the status of [release][infra] SSH host key issue - RedFlag stop approving is | 17:03 |
| sean-k-mooney | ill swap to infra | 17:03 |
| elodilles | sean-k-mooney: #openstack-release | 17:04 |
| elodilles | sean-k-mooney: infra team is working on it, but hopefully will be fixed soon | 17:05 |
| sean-k-mooney | elodilles: ack ill create the os-traits releaes patch so in parallel but obvioulys that will fail with the same issue. | 17:08 |
| elodilles | sean-k-mooney: check jobs will pass, and release team won't approve it until the tag job isn't fully functional :) | 17:18 |
| elodilles | sean-k-mooney: so feel free to propose the patches, it won't be a problem | 17:19 |
| sean-k-mooney | ack was gettign coffee :) | 17:20 |
| ralonsoh | sean-k-mooney, https://review.opendev.org/c/openstack/neutron-lib/+/645288/6/releasenotes/notes/new-vif-details-parameters-71e70ab5e7c26c45.yaml | 17:22 |
| ralonsoh | I totally forgot that: "bound_drivers", that is already in n-lib since Train | 17:22 |
| ralonsoh | this is what you need (and Neutron needs to populate right now) | 17:23 |
| sean-k-mooney | ralonsoh: oh i know i rememebre it and pointed it out in the spec review | 17:23 |
| sean-k-mooney | i was not sure why you were not using it and addign backend | 17:23 |
| ralonsoh | yeah, I think this is the right parameter to populate | 17:23 |
| ralonsoh | why? we never implemented that hehehe | 17:24 |
| ralonsoh | I'll start now | 17:24 |
| sean-k-mooney | proably a donwstream esclation caused a context switch that has just been unsuspended in your brain :) | 17:24 |
| ralonsoh | or too much beer during weekend | 17:25 |
| ralonsoh | most probably | 17:25 |
| sean-k-mooney | ralonsoh: its not critical but the sunner its avaiabel the soone we can use it | 17:25 |
| ralonsoh | for sure | 17:25 |
| *** Uggla is now known as Uggla|afk | 18:21 | |
| sean-k-mooney | bauzas: https://review.opendev.org/c/openstack/releases/+/826353 os-traits 2.7.0 release | 18:39 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!