| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-07-25 | |||
| 16:11:49 | opendevreview | Balazs Gibizer proposed openstack/nova master: Remove dead code from PhysicalPciAddress https://review.opendev.org/c/openstack/nova/+/844628 | |
| 16:11:49 | opendevreview | Balazs Gibizer proposed openstack/nova master: Fix PciAddressSpec descendants to call super.__init__ https://review.opendev.org/c/openstack/nova/+/844565 | |
| 16:11:50 | opendevreview | Balazs Gibizer proposed openstack/nova master: Remove unused PF checking from get_function_by_ifname https://review.opendev.org/c/openstack/nova/+/845775 | |
| 16:11:50 | dansmith | right but that would be a neutron issue | |
| 16:11:50 | opendevreview | Balazs Gibizer proposed openstack/nova master: Clean up mapping input to address spec types https://review.opendev.org/c/openstack/nova/+/845765 | |
| 16:11:51 | opendevreview | Balazs Gibizer proposed openstack/nova master: Move __str__ to the PciAddressSpec base class https://review.opendev.org/c/openstack/nova/+/845781 | |
| 16:11:51 | opendevreview | Balazs Gibizer proposed openstack/nova master: Fix type annotation of pci.Whitelist class https://review.opendev.org/c/openstack/nova/+/845780 | |
| 16:11:58 | sean-k-mooney | dansmith: yes it would | |
| 16:12:23 | dansmith | ack, just confirming ;) | |
| 16:12:31 | sean-k-mooney | so your suggesting that marking it confirmed for nova on the bug is wrong and we should likely change it | |
| 16:13:15 | dansmith | I dunno about that I just want to be clear that nova shouldn't be trying to interpret vif-plugged differently for trunks | |
| 16:13:23 | sean-k-mooney | bauzas: gibi by the way with the escalation and everything that happened in the last few days i have not been doing upstream bug triage this week sorry | |
| 16:13:34 | gibi | sean-k-mooney: no worries | |
| 16:13:38 | sean-k-mooney | dansmith: agreeed | |
| 16:13:48 | bauzas | sean-k-mooney: no worries at all, again, bug triage is just down any prio | |
| 16:13:57 | sean-k-mooney | dansmith: nova should jsut care about thte one port that is attached to the vm (the trunk) | |
| 16:14:03 | sean-k-mooney | the rest is up to neutron to care about | |
| 16:14:07 | dansmith | yes | |
| 16:14:23 | sean-k-mooney | dansmith: if tempest should check this at all is proably TBD | |
| 16:14:27 | gibi | maybe the tempest test verify the system from neutron perspective hence the assert on the subport too | |
| 16:14:57 | dansmith | sean-k-mooney: also true | |
| 16:15:06 | sean-k-mooney | gibi: yes but in that casae it is indicating that neutron is not correctly seting up the trunk | |
| 16:15:11 | gibi | yes | |
| 16:15:14 | gibi | I agree | |
| 16:15:44 | sean-k-mooney | i would suggest seting the nova part to incomplete for now | |
| 16:15:55 | sean-k-mooney | as its not clear that nova shoudl be doing anything it is not already doing | |
| 16:20:19 | gibi | works for me | |
| 16:20:32 | gibi | later we can set it to invalid if turn out only neutron needs a fix | |
| 16:24:12 | opendevreview | Billy Olsen proposed openstack/nova master: Handle mdev devices in libvirt 7.7+ https://review.opendev.org/c/openstack/nova/+/838976 | |
| 19:30:40 | opendevreview | Amit Uniyal proposed openstack/nova master: For evacuation, ignore if task_state is not None https://review.opendev.org/c/openstack/nova/+/848886 | |
| #openstack-nova - 2022-07-26 | |||
| 05:31:15 | auniyal | logs from nova/db/main/api should come in devstack@n-cond-cell1, super-cond or cpu ? | |
| 05:41:22 | auniyal | it should come under devstack@n-super-cond, but exceptions are not coming | |
| 07:48:43 | bauzas | good morning Nova | |
| 07:49:05 | gibi | o/ | |
| 07:55:30 | bauzas | I just sent an email | |
| 07:55:44 | bauzas | tl;dr: HOLD YOUR RECHECKS | |
| 08:07:39 | gibi | I see a nice buzz around the issue in the bug report | |
| 08:08:48 | bauzas | like I said in -neutron, I'm looking at gerrit now based on mlavalle's comment | |
| 08:09:52 | gibi | I did that already and commented on the bug, but feel free to double check | |
| 08:17:36 | opendevreview | Sylvain Bauza proposed openstack/nova master: DNM: Avoid os-vif 3.0.0 https://review.opendev.org/c/openstack/nova/+/850998 | |
| 08:17:39 | bauzas | gibi: ^ | |
| 08:18:23 | gibi | bauzas: thanks | |
| 08:53:04 | sean-k-mooney[m] | why? | |
| 08:53:41 | sean-k-mooney[m] | what change in os-vif do you think is related | |
| 08:55:13 | sean-k-mooney[m] | the trunk bridge patch? | |
| 08:56:40 | sean-k-mooney[m] | https://github.com/openstack/os-vif/commit/75b290fb2a8f706583e0c12c5c5a4c0fc80e6481 ? | |
| 08:58:07 | sean-k-mooney[m] | if its related to that then the but is in neutron | |
| 09:03:30 | bauzas | sean-k-mooney: we're discussing it alot in the -neutron room | |
| 09:04:06 | bauzas | basically, neutron wanted to defer the trunk creation logic to os-vif but since we have rolling upgrades, looks like we now fail | |
| 09:04:36 | bauzas | it looks to me the upgrade approach was invalid and we should have waited nova to be fully upgraded | |
| 09:07:43 | opendevreview | Sylvain Bauza proposed openstack/nova master: Block os-vif 3.0.0 https://review.opendev.org/c/openstack/nova/+/850998 | |
| 09:07:45 | sean-k-mooney[m] | ill hop over | |
| 09:07:57 | sean-k-mooney[m] | but the nova and os-vif code is right i think | |
| 09:08:22 | sean-k-mooney[m] | but they may have dropped the bridge cleanup code to soon in neutron | |
| 09:08:28 | bauzas | sean-k-mooney: it seems so yeah | |
| 09:09:05 | bauzas | some logic was possibly modified in Neutron | |
| 09:09:24 | bauzas | this would explain why nova-next and grenade were failing | |
| 09:09:45 | bauzas | for the ovs-hybrid-plug job, I don't k | |
| 09:41:24 | opendevreview | Sylvain Bauza proposed openstack/nova master: DNM for testing drop of os-vif to 2.8.0 https://review.opendev.org/c/openstack/nova/+/851006 | |
| 09:48:04 | opendevreview | Takashi Natsume proposed openstack/nova-specs master: Create specs directory for Antelope https://review.opendev.org/c/openstack/nova-specs/+/851007 | |
| 10:26:51 | opendevreview | sean mooney proposed openstack/os-vif master: [WIP] make os-vif jobs multinode and enable trunk testing https://review.opendev.org/c/openstack/os-vif/+/851011 | |
| 10:30:59 | whoami-rajat | dansmith, sean-k-mooney[m] hey, just a reminder to request to review the series of rebuilding volume backed instance, would like to get it in Zed release https://review.opendev.org/c/openstack/nova/+/820368/ | |
| 12:16:55 | bauzas | so, https://review.opendev.org/c/openstack/nova/+/851006 has a -1 but not because of the os-vif issue | |
| 12:17:49 | ratailor | bauzas, could you please provide your feedback on https://review.opendev.org/c/openstack/nova/+/844418 ? | |
| 12:18:19 | bauzas | ratailor: for the moment, I'm working on another CI issue | |
| 12:18:22 | bauzas | but ok | |
| 12:18:51 | ratailor | bauzas, no problem. whenever you have time. Thanks! | |
| 13:02:46 | opendevreview | ribaudr proposed openstack/nova master: [WIP] Attach Manila shares via virtiofs (objects) https://review.opendev.org/c/openstack/nova/+/839401 | |
| 13:02:47 | opendevreview | ribaudr proposed openstack/nova master: [WIP] Attach Manila shares via virtiofs (drivers) https://review.opendev.org/c/openstack/nova/+/833090 | |
| 13:02:47 | opendevreview | ribaudr proposed openstack/nova master: [WIP] Attach Manila shares via virtiofs (manila abstraction) https://review.opendev.org/c/openstack/nova/+/831194 | |
| 13:02:48 | opendevreview | ribaudr proposed openstack/nova master: [WIP] Bump compute version and check shares support https://review.opendev.org/c/openstack/nova/+/850499 | |
| 13:02:48 | opendevreview | ribaudr proposed openstack/nova master: [WIP] Attach Manila shares via virtiofs (api) https://review.opendev.org/c/openstack/nova/+/836830 | |
| 13:02:49 | opendevreview | ribaudr proposed openstack/nova master: [WIP] Add instance.share_attach notification https://review.opendev.org/c/openstack/nova/+/850501 | |
| 13:02:49 | opendevreview | ribaudr proposed openstack/nova master: [WIP] Add metadata for shares https://review.opendev.org/c/openstack/nova/+/850500 | |
| 13:02:50 | opendevreview | ribaudr proposed openstack/nova master: [WIP] Add shares to InstancePayload https://review.opendev.org/c/openstack/nova/+/851029 | |
| 13:02:50 | opendevreview | ribaudr proposed openstack/nova master: [WIP] Add instance.share_detach notification https://review.opendev.org/c/openstack/nova/+/851028 | |
| 13:14:53 | bauzas | fwiw https://lists.openstack.org/pipermail/openstack-discuss/2022-July/029725.html | |
| 14:07:59 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: Add a workaround to skip hypervisor version check https://review.opendev.org/c/openstack/nova/+/851034 | |
| 14:14:49 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: Add a workaround to skip hypervisor version check on LM https://review.opendev.org/c/openstack/nova/+/851034 | |
| 14:17:01 | kashyap | gibi: sean-k-mooney: --^ When you get a minu | |
| 14:19:06 | gibi | kashyap: we need a launchpad bug for tracking and a releasenotes | |
| 14:19:11 | gibi | otherwise looks good to me | |
| 15:18:19 | kashyap | gibi: Ah, yes; rel-no | |
| 15:18:22 | kashyap | Gonna do it | |
| 15:23:55 | kashyap | gibi: I hope you're cool with the what/why format here :) - https://bugs.launchpad.net/nova/+bug/1982853 | |
| 15:24:45 | sean-k-mooney | we dont actully enforce a format upstream so sure | |
| 15:24:55 | gibi | kashyap: it a bit strange as a bug report as it does not focus on the faulty behavior, but as sean-k-mooney said it is OK | |
| 15:24:57 | sean-k-mooney | although that makes it sound more like a feature then a bug | |
| 15:25:25 | kashyap | sean-k-mooney gibi Yeah, I can describe a buggy behaviour from OSP usage | |
| 15:25:48 | sean-k-mooney | gibi: ya i had the same tought, framing it as what/why focuses on solution not the problem | |
| 15:26:00 | sean-k-mooney | well ter isnt a but in nova really | |
| 15:26:20 | gibi | but I understand if we don't want to embarrass ourselves with the actual fault we made downstream ;) | |
| 15:26:24 | kashyap | The "what" can also describe a problem space | |
| 15:26:29 | sean-k-mooney | but there are day 2 issues that people might hit that his will help wiht | |
| 15:26:30 | kashyap | I was just too lazy to add it | |
| 15:26:52 | sean-k-mooney | so im ok to tack it as a UX bug or operator nice to have | |
| 15:28:01 | sean-k-mooney | gibi: well im expecting we will close our downstream bz as not a bug honestly | |
| 15:28:04 | kashyap | Yeah, that's why the "why" describes an example use-case, instead of an elaborate mistake of a specific OpenStack distro :) | |
| 15:28:25 | sean-k-mooney | but we will se we might have to kep it open for backport reaons | |
| 15:28:33 | sean-k-mooney | and doc text ectra | |