*** jkulik has quit IRC | 00:02 | |
rm_work | ahh figured it out | 00:10 |
---|---|---|
rm_work | the nova_api DB stores cell connection strings that need to be updated if the connection string changes at all | 00:11 |
rm_work | this seems a little awkward? but I assume it is necessary for some reason to cache this data here | 00:11 |
artom | rm_work, if you're familiar with why and how cells v2 came about, it makes sense | 00:13 |
rm_work | yeah so if we cycle the DB password, or the certs move, we need to make sure to update the cells first via `nova-manage cells_v2 update_cell` | 00:15 |
artom | rm_work, yep | 00:19 |
*** artom has quit IRC | 00:21 | |
*** artom has joined #openstack-nova | 00:22 | |
*** eharney has quit IRC | 00:29 | |
*** vishalmanchanda has quit IRC | 00:41 | |
*** hamalq has quit IRC | 00:41 | |
*** eharney has joined #openstack-nova | 00:43 | |
*** artom has quit IRC | 01:04 | |
*** k_mouza has joined #openstack-nova | 01:08 | |
*** jawad_axd has quit IRC | 01:19 | |
*** k_mouza has quit IRC | 01:20 | |
*** k_mouza has joined #openstack-nova | 01:20 | |
*** k_mouza has quit IRC | 01:25 | |
*** mlavalle has quit IRC | 01:35 | |
*** rcernin has quit IRC | 01:43 | |
*** xinranwang has joined #openstack-nova | 01:58 | |
*** rcernin has joined #openstack-nova | 02:20 | |
*** rcernin has quit IRC | 02:24 | |
*** rcernin has joined #openstack-nova | 02:28 | |
*** rcernin has quit IRC | 02:31 | |
*** rcernin has joined #openstack-nova | 02:32 | |
*** lemko has quit IRC | 02:53 | |
*** lemko has joined #openstack-nova | 02:53 | |
*** brinzhang has joined #openstack-nova | 03:03 | |
openstackgerrit | Brin Zhang proposed openstack/nova master: Cyborg shelve/unshelve support https://review.opendev.org/729563 | 03:12 |
*** mkrai has joined #openstack-nova | 03:21 | |
*** songwenping__ has quit IRC | 04:15 | |
*** k_mouza has joined #openstack-nova | 04:21 | |
*** zzzeek has quit IRC | 04:24 | |
*** zzzeek has joined #openstack-nova | 04:25 | |
*** k_mouza has quit IRC | 04:26 | |
openstackgerrit | Brin Zhang proposed openstack/nova-specs master: Remove tenant_id https://review.opendev.org/737241 | 04:33 |
*** zzzeek has quit IRC | 04:46 | |
*** zzzeek has joined #openstack-nova | 04:49 | |
*** ratailor has joined #openstack-nova | 04:54 | |
*** evrardjp has quit IRC | 05:33 | |
*** evrardjp has joined #openstack-nova | 05:33 | |
*** ociuhandu has joined #openstack-nova | 05:44 | |
*** zzzeek has quit IRC | 05:47 | |
*** zzzeek has joined #openstack-nova | 05:48 | |
*** ociuhandu has quit IRC | 05:49 | |
*** jkulik has joined #openstack-nova | 05:57 | |
*** vishalmanchanda has joined #openstack-nova | 06:00 | |
*** gyee has quit IRC | 06:08 | |
*** spatel has joined #openstack-nova | 06:49 | |
*** rpittau|afk is now known as rpittau | 06:49 | |
*** spatel has quit IRC | 06:53 | |
openstackgerrit | Xinran WANG proposed openstack/nova-specs master: SRIOV Enabled Nic Support Specification https://review.opendev.org/742785 | 07:20 |
*** sapd1 has joined #openstack-nova | 07:26 | |
*** ralonsoh has joined #openstack-nova | 07:31 | |
*** dklyle has quit IRC | 07:35 | |
*** links has joined #openstack-nova | 07:41 | |
*** slaweq has joined #openstack-nova | 07:43 | |
*** dtantsur|afk is now known as dtantsur | 07:51 | |
*** tesseract has joined #openstack-nova | 08:03 | |
bauzas | good morning Nova | 08:04 |
tacco | Morning * | 08:13 |
*** andrewbonney has joined #openstack-nova | 08:16 | |
*** tesseract has quit IRC | 08:16 | |
*** tesseract has joined #openstack-nova | 08:18 | |
*** lbragstad_ has joined #openstack-nova | 08:18 | |
gibi | o/ | 08:19 |
*** k_mouza has joined #openstack-nova | 08:22 | |
*** lbragstad has quit IRC | 08:22 | |
*** k_mouza has quit IRC | 08:26 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova stable/victoria: Add upgrade check about old computes https://review.opendev.org/761924 | 08:32 |
openstackgerrit | Balazs Gibizer proposed openstack/nova stable/victoria: Improve error handling during service level check https://review.opendev.org/762471 | 08:32 |
eandersson | Is there a tool to clean up placement entries that no longer exists in nova? :D | 08:33 |
eandersson | https://github.com/openstack/nova/blob/stable/rocky/nova/compute/resource_tracker.py#L1326 | 08:34 |
eandersson | It's not something super widespread, but I have 22 computes that have at least one of these ghost allocations. | 08:35 |
gibi | eandersson: $ nova-manage placement audit # https://docs.openstack.org/nova/latest/cli/nova-manage.html | 08:36 |
gibi | eandersson: https://bugs.launchpad.net/nova/+bug/1793569 | 08:37 |
openstack | Launchpad bug 1793569 in OpenStack Compute (nova) "Add placement audit commands" [Wishlist,Fix released] - Assigned to Sylvain Bauza (sylvain-bauza) | 08:37 |
eandersson | Nice. Guessing that there isn't a back-port on that one. ;D | 08:37 |
gibi | eandersson: there was backport proposals https://review.opendev.org/#/q/status:abandoned+topic:placement-audit-backport | 08:38 |
eandersson | Thanks yea just found it | 08:38 |
eandersson | I'll try it out in the lab | 08:40 |
*** iurygregory has quit IRC | 08:44 | |
*** tosky has joined #openstack-nova | 08:45 | |
*** ociuhandu has joined #openstack-nova | 08:47 | |
*** iurygregory has joined #openstack-nova | 08:50 | |
*** rcernin has quit IRC | 08:52 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova stable/victoria: Restore retrying the RPC connection to conductor https://review.opendev.org/763138 | 08:56 |
*** songwenping__ has joined #openstack-nova | 09:06 | |
*** mgoddard has joined #openstack-nova | 09:07 | |
eandersson | Worked great in the lab | 09:09 |
*** derekh has joined #openstack-nova | 09:10 | |
gibi | eandersson: good to hear that | 09:11 |
*** rcernin has joined #openstack-nova | 09:15 | |
*** slaweq has quit IRC | 09:17 | |
*** slaweq has joined #openstack-nova | 09:19 | |
*** yankcrime has quit IRC | 09:24 | |
*** martinkennelly has joined #openstack-nova | 09:25 | |
*** ociuhandu has quit IRC | 09:32 | |
lyarwood | Morning | 09:42 |
*** rcernin has quit IRC | 09:45 | |
*** ociuhandu has joined #openstack-nova | 09:49 | |
*** ociuhandu has quit IRC | 09:49 | |
*** ociuhandu has joined #openstack-nova | 09:49 | |
*** yoctozepto has quit IRC | 09:50 | |
*** brinzhang_ has joined #openstack-nova | 09:51 | |
*** yoctozepto has joined #openstack-nova | 09:51 | |
*** brinzhang has quit IRC | 09:54 | |
*** rcernin has joined #openstack-nova | 09:55 | |
*** zzzeek has quit IRC | 09:56 | |
*** zzzeek has joined #openstack-nova | 09:59 | |
*** k_mouza has joined #openstack-nova | 10:14 | |
*** yankcrime has joined #openstack-nova | 10:16 | |
*** sapd1 has quit IRC | 10:18 | |
*** kevinz has quit IRC | 10:26 | |
openstackgerrit | Merged openstack/nova master: Reproduce bug 1896463 in func env https://review.opendev.org/754100 | 10:29 |
openstack | bug 1896463 in OpenStack Compute (nova) rocky "evacuation failed: Port update failed : Unable to correlate PCI slot " [Low,In progress] https://launchpad.net/bugs/1896463 | 10:29 |
openstackgerrit | Merged openstack/nova master: Set instance host and drop migration under lock https://review.opendev.org/754815 | 10:32 |
openstackgerrit | Merged openstack/nova master: [Trivial]Add missing white space in the log message https://review.opendev.org/762804 | 10:33 |
openstackgerrit | Merged openstack/nova master: [docs] Fix a placement client's command https://review.opendev.org/762663 | 10:33 |
openstackgerrit | Balazs Gibizer proposed openstack/nova stable/victoria: Reproduce bug 1896463 in func env https://review.opendev.org/763152 | 10:35 |
openstack | bug 1896463 in OpenStack Compute (nova) rocky "evacuation failed: Port update failed : Unable to correlate PCI slot " [Low,In progress] https://launchpad.net/bugs/1896463 | 10:35 |
openstackgerrit | Balazs Gibizer proposed openstack/nova stable/victoria: Set instance host and drop migration under lock https://review.opendev.org/763154 | 10:38 |
*** sapd1 has joined #openstack-nova | 10:38 | |
*** belmoreira has joined #openstack-nova | 10:46 | |
*** sapd1 has quit IRC | 10:47 | |
*** sapd1 has joined #openstack-nova | 10:53 | |
*** sapd1 has quit IRC | 10:57 | |
*** hemanth_n has quit IRC | 11:04 | |
*** lpetrut has joined #openstack-nova | 11:10 | |
*** rcernin has quit IRC | 11:18 | |
*** rcernin has joined #openstack-nova | 11:19 | |
*** mkrai has quit IRC | 11:24 | |
*** rcernin has quit IRC | 11:27 | |
*** ratailor has quit IRC | 11:30 | |
*** spatel has joined #openstack-nova | 11:31 | |
*** rcernin has joined #openstack-nova | 11:33 | |
*** spatel has quit IRC | 11:36 | |
openstackgerrit | Balazs Gibizer proposed openstack/nova master: Revert "Temporarily disable parts of heal port allocation test" https://review.opendev.org/755995 | 11:36 |
*** slaweq has quit IRC | 11:39 | |
stephenfin | Does unshelving a server always result in that server going to ACTIVE or ERROR state? | 11:41 |
stephenfin | I'm adding a '--wait' option to 'openstack server unshelve' and was checking for 'SHUTOFF' status, but a quick check suggests we don't restore to the original state after unshelving https://review.opendev.org/#/c/763166/1/openstackclient/compute/v2/server.py@3713 | 11:42 |
stephenfin | So I guess I just need to check for 'ACTIVE'? | 11:43 |
*** slaweq has joined #openstack-nova | 11:45 | |
*** rcernin has quit IRC | 11:46 | |
sean-k-mooney | i shoudl go to active but if there is an error i dont knwo if it will go back to shelved or error | 11:49 |
sean-k-mooney | we wont restore the orgininal state as far as i know | 11:49 |
*** rcernin has joined #openstack-nova | 11:52 | |
*** ociuhandu has quit IRC | 11:52 | |
*** rcernin has quit IRC | 11:55 | |
stephenfin | okay, I'll just check for ACTIVE so | 11:56 |
*** ociuhandu has joined #openstack-nova | 11:58 | |
*** ociuhandu has quit IRC | 11:58 | |
*** tbachman has quit IRC | 11:59 | |
*** songwenping_ has joined #openstack-nova | 12:05 | |
*** songwenping__ has quit IRC | 12:08 | |
*** zzzeek has quit IRC | 12:13 | |
*** xek has joined #openstack-nova | 12:14 | |
*** zzzeek has joined #openstack-nova | 12:16 | |
openstackgerrit | Shi Yan proposed openstack/nova master: Fix the instance_uuid for test https://review.opendev.org/763171 | 12:24 |
*** zzzeek has quit IRC | 12:27 | |
*** zzzeek has joined #openstack-nova | 12:28 | |
*** ociuhandu has joined #openstack-nova | 12:30 | |
*** gibi is now known as gibi_away | 12:52 | |
*** xek has quit IRC | 12:54 | |
*** xek has joined #openstack-nova | 12:56 | |
*** dtantsur is now known as dtantsur|brb | 13:06 | |
*** artom has joined #openstack-nova | 13:17 | |
*** nweinber has joined #openstack-nova | 13:26 | |
*** ociuhandu has quit IRC | 13:32 | |
*** sapd1 has joined #openstack-nova | 13:51 | |
*** xek_ has joined #openstack-nova | 13:53 | |
*** xek has quit IRC | 13:56 | |
openstackgerrit | Lee Yarwood proposed openstack/nova-specs master: WIP libvirt: Allow the default machine type to be changed https://review.opendev.org/762199 | 13:58 |
aarents | Hi nova, | 14:02 |
aarents | lyarwood: can you confirm, when you can, that those both change are ok ? https://review.opendev.org/#/c/743708/ https://review.opendev.org/#/c/752463/2 thanks ! | 14:03 |
*** macz_ has joined #openstack-nova | 14:08 | |
*** macz_ has quit IRC | 14:12 | |
*** xek_ has quit IRC | 14:13 | |
*** xek has joined #openstack-nova | 14:14 | |
*** dtantsur|brb is now known as dtantsur | 14:22 | |
*** ociuhandu has joined #openstack-nova | 14:23 | |
*** ociuhandu has quit IRC | 14:28 | |
*** ociuhandu has joined #openstack-nova | 14:28 | |
*** links has quit IRC | 14:49 | |
*** lbragstad_ is now known as lbragstad | 14:59 | |
gmann | stephenfin: sean-k-mooney I agree on full list of hypervisors to project admin if no limit - https://review.opendev.org/#/c/763043/1/specs/wallaby/approved/modernize-os-hypervisors-api.rst@162 | 15:10 |
*** ociuhandu has quit IRC | 15:11 | |
gmann | stephenfin: if you can update the other comments I am good on this. | 15:11 |
owalsh | artom, rm_work: FYI there is a better option for DB creds etc... for cell uris: https://docs.openstack.org/nova/latest/user/cells.html#template-urls-in-cell-mappings | 15:18 |
*** artom has quit IRC | 15:19 | |
*** mkrai has joined #openstack-nova | 15:19 | |
*** lpetrut has quit IRC | 15:21 | |
stephenfin | gmann: Cool, I will respin that shortly | 15:21 |
stephenfin | dansmith: Is 'nova refresh-network' worth porting to OSC? | 15:21 |
dansmith | stephenfin: isn't that a xen-specific thing? | 15:22 |
stephenfin | It uses the os-server-external-events extension | 15:22 |
stephenfin | "extension" | 15:22 |
stephenfin | is that XenAPI specific? | 15:22 |
*** artom has joined #openstack-nova | 15:22 | |
stephenfin | surely not; that's what neutron uses iirc | 15:22 |
stephenfin | asking you specifically because you authored the novaclient change waaaay back https://review.opendev.org/#/c/74763/ | 15:23 |
dansmith | oh, then I'm thinking of something else | 15:23 |
owalsh | dansmith: re cell nova-conductor still requiring api db for affinity checks, should I assume that will eventually change? | 15:24 |
dansmith | oh, right, this probably really shouldn't even be a CLI thing, we probably added it there so we could poke it for testing | 15:24 |
rm_work | owalsh: oooo will take a look, thanks | 15:24 |
stephenfin | okay, that was my gut feel too; I'll skip it so | 15:24 |
stephenfin | dansmith++ thanks | 15:25 |
*** mlavalle has joined #openstack-nova | 15:25 | |
dansmith | stephenfin: I was thinking of this: https://docs.openstack.org/api-ref/compute/#reset-networking-on-a-server-resetnetwork-action | 15:26 |
dansmith | note the "no longer supported by any virt driver" because xen | 15:26 |
stephenfin | aha, yes, I've done work on that recently | 15:26 |
dansmith | owalsh: probably not any time soon.. it was hinging on affinity in placement, and I gather that people aren't interested in furthering those goals these days, | 15:28 |
dansmith | owalsh: so probably no change on the horizon | 15:28 |
*** dklyle has joined #openstack-nova | 15:37 | |
owalsh | dansmith: ack, thanks, so it wouldn't be crazy to use these creds to create the cell mapping when deploying a new cell conductor? | 15:39 |
*** ociuhandu has joined #openstack-nova | 15:40 | |
dansmith | owalsh: nope, that's reasonable at the moment I'd say | 15:40 |
owalsh | dansmith: ack, great. Also does metadata api need the api db config even when local_metadata_per_cell=True? | 15:46 |
dansmith | owalsh: shouldn't, no | 15:47 |
dansmith | owalsh: yeah, if that's set, we never run the mapping lookup, which is the api hit | 15:48 |
dansmith | owalsh: so unless there's a bug or something buried deep, that's the intent.. no api db hit if =True | 15:48 |
owalsh | dansmith: ack, yea, didn't initially make sense that local_metadata_per_cell existed when everything else just checks if the api db is configured, but need to support cell conductor & metadata api sharing nova.conf | 15:49 |
dansmith | owalsh: well, I think metadata was the first thing to need that determination, so we added a conf knob for it specifically.. it's also potentially security-related because the workloads can talk to it, so explicit-ness is good | 15:53 |
dansmith | owalsh: but we could deprecate that conf knob and actually just honor the presence of the api creds if you think it'd be better for consistency | 15:53 |
owalsh | dansmith: but that wouldn't work in cases where the cell conductor shares nova conf with cell metadata-api | 15:58 |
dansmith | cell conductor, yeah | 15:58 |
*** ociuhandu_ has joined #openstack-nova | 15:59 | |
owalsh | dansmith: which goes back to the recommendation on the ML to follow the devstack approach to conf - it doesn't seem to work when using cell local metadata api... | 16:01 |
owalsh | since metadata-api always uses nova.conf, never nova-cellN.conf | 16:02 |
*** ociuhandu has quit IRC | 16:02 | |
dansmith | owalsh: but devstack runs a global metadata api right? | 16:02 |
owalsh | dansmith: yup, doesn't seem to support cell local metadata api | 16:03 |
dansmith | devstack's goal, of course, is not to support every possible deployment arrangement... | 16:03 |
owalsh | dansmith: indeed, and for CI/dev running multiple cells makes sense but in production it does not... | 16:03 |
owalsh | multiple cells on the same host that is | 16:04 |
dansmith | I've lost track of what we're arguing about | 16:04 |
owalsh | dansmith: http://lists.openstack.org/pipermail/openstack-discuss/2020-November/018732.html | 16:06 |
owalsh | I'm saying we can't use a similar strategy to separate files | 16:06 |
dansmith | owalsh: okay I'm sorry, I've really lost track of who "we" and what the problem is. You're saying "we=tripleo" deploy a metadata per cell and can't, for some reason, point it at the cellN conductor's config file? | 16:10 |
owalsh | dansmith: rpms/debs in particular are the issue, but yes, can't point metadata-api at the conductor's config file | 16:11 |
*** macz_ has joined #openstack-nova | 16:13 | |
openstackgerrit | Lee Yarwood proposed openstack/nova-specs master: WIP libvirt: Allow the default machine type to be changed https://review.opendev.org/762199 | 16:14 |
dansmith | owalsh: okay, I'm not sure why not currently, because we have that other knob | 16:15 |
owalsh | dansmith: nova.conf is hardcoded in wsgi_api.py | 16:15 |
*** mkrai has quit IRC | 16:19 | |
*** mkrai has joined #openstack-nova | 16:20 | |
owalsh | dansmith: TBH it would make a lot more sense to just say: use nova.conf for everything except nova-compute.... | 16:21 |
*** ociuhandu_ has quit IRC | 16:22 | |
dansmith | owalsh: I'm really lost, I'm sorry | 16:22 |
dansmith | you can, of course, just use nova.conf for everything, | 16:22 |
*** ociuhandu has joined #openstack-nova | 16:22 | |
dansmith | but if they're all on the same host and you have cell and super conductors, they can't all use the same config file | 16:23 |
*** belmoreira has quit IRC | 16:27 | |
bauzas | grmblbl, I get exceptions about RPC API 6.0 that I don't understand... | 16:29 |
bauzas | "ERROR nova.api.openstack.wsgi oslo_messaging.rpc.client.RPCVersionCapError: Requested message version, 5.0 is incompatible. It needs to be equal in major version and less than or equal in minor version as the specified version cap 6.0" | 16:31 |
bauzas | WTF | 16:31 |
bauzas | dansmith: do you have any idea why the RPC API only supports 6.0 as the minor RPC version while I wrote a 5.12 proxy in https://review.opendev.org/#/c/761452/2 ? | 16:36 |
bauzas | I got tons of tempest and func tests failing because of this | 16:37 |
*** ociuhandu has quit IRC | 16:37 | |
bauzas | in theory, I should expose 5.x as a valid RPC target thanks to the added service RPC endpoint | 16:37 |
*** mkrai has quit IRC | 16:38 | |
*** ociuhandu has joined #openstack-nova | 16:45 | |
gmann | stephenfin: bauzas dansmith can you review this, sample config file fix - https://review.opendev.org/#/c/762721/ | 16:50 |
*** xek_ has joined #openstack-nova | 16:57 | |
*** xek has quit IRC | 17:00 | |
noonedeadpunk | hey! | 17:01 |
noonedeadpunk | having a question regarding http://lists.openstack.org/pipermail/openstack-discuss/2020-November/018663.html | 17:02 |
*** ociuhandu_ has joined #openstack-nova | 17:03 | |
noonedeadpunk | to be specific - was trying to provide --config-file for uwsgi via pyargv but nova seems to ignore it... | 17:03 |
noonedeadpunk | and it indeed works for neutron... so not sure if it's just me having issues or what, as things are set via oslo anyway.... | 17:04 |
*** ociuhandu has quit IRC | 17:06 | |
*** ociuhandu_ has quit IRC | 17:07 | |
*** mgoddard has quit IRC | 17:33 | |
*** tesseract has quit IRC | 17:37 | |
dansmith | noonedeadpunk: funny you should ask | 17:39 |
dansmith | noonedeadpunk: I think it should, I dunno why it doesn't, and I'm about to caffeinate myself and write a tome to the ML thread about it and other things | 17:40 |
*** jangutter_ has joined #openstack-nova | 17:58 | |
*** jangutter has quit IRC | 18:02 | |
*** k_mouza has quit IRC | 18:05 | |
*** k_mouza has joined #openstack-nova | 18:06 | |
*** hamalq has joined #openstack-nova | 18:07 | |
*** k_mouza has quit IRC | 18:10 | |
*** derekh has quit IRC | 18:14 | |
*** dtantsur is now known as dtantsur|afk | 18:15 | |
*** LinPeiWen67 has quit IRC | 18:26 | |
*** ralonsoh has quit IRC | 18:31 | |
openstackgerrit | sean mooney proposed openstack/nova master: libvirt: delegate ovs plug to os-vif https://review.opendev.org/602432 | 18:32 |
sean-k-mooney | melwitt: i hope that adresses your comments ^ | 18:32 |
sean-k-mooney | stephenfin: if you have time to have a look again at ^ it basicaly the same + some typo fixes and a rebase | 18:33 |
melwitt | thanks | 18:33 |
sean-k-mooney | based on your question i set the min verion to 1.15.2 instaed of 1.15.1 since it has imporant bugfixes for linuxbridge and added that to the commit | 18:34 |
sean-k-mooney | 1.15.0 was broken for ovs 1.15.1 fixed that but still was broken in some cases for linux bridge 1.15.2 is what we released with and support on stable stine in upper constratints | 18:35 |
*** xinranwang has quit IRC | 18:36 | |
-openstackstatus- NOTICE: The Gerrit service at review.opendev.org is being restarted quickly as a pre-upgrade sanity check, estimated downtime is less than 5 minutes. | 18:36 | |
melwitt | thanks for explaining that, makes sense now. I had been confused by the gap in the release notes and couldn't find the significance of 1.15.1 | 18:37 |
melwitt | sean-k-mooney: oh, one more question that I had on the review that I think was missed, is this new delegation to os-vif for the plug, does it run during tempest tests in the gate? or is it for a config that is not running in the gate | 18:39 |
*** rpittau is now known as rpittau|afk | 18:39 | |
sean-k-mooney | oh right i will run in any job using iptables | 18:41 |
sean-k-mooney | i think we have on that does that but ill check now | 18:41 |
sean-k-mooney | that used to be the default | 18:41 |
*** songwenping_ has quit IRC | 18:46 | |
sean-k-mooney | still checking look like we change the default in devstack but we have an iptable job in experimental so i can kick that off or add a DNM testing patch on top | 18:47 |
sean-k-mooney | melwitt: which would you prefer | 18:47 |
melwitt | sean-k-mooney: whatever is easier | 18:48 |
sean-k-mooney | experimental is eaiser but it was a bunch of resouce so ill do a dnm patch with both the os-vif and neutron iptables tempest job | 18:49 |
sean-k-mooney | they are the same but the os-vif on uses master and the neutorn one uses the released version | 18:49 |
melwitt | ah ok, sounds cool | 18:49 |
sean-k-mooney | im kind of surprised we dont have any iptables testing in nova actully but i guess it has not been the default for a few releases at this point | 18:50 |
sean-k-mooney | ya even the greade job uses the ovs firewall at this point https://zuul.opendev.org/t/openstack/build/ec9d4bf702c84bcf9ae355ae0a735cd1/log/controller/logs/etc/neutron/plugins/ml2/ml2_conf.ini#299 | 18:51 |
sean-k-mooney | although to be faire we also dont have any ovn testing so its not surprising | 18:52 |
melwitt | interesting. so the original bug only affects deployments using iptables. I hadn't realized that | 18:53 |
sean-k-mooney | oh wait sorry | 18:55 |
sean-k-mooney | i had that backwards | 18:55 |
sean-k-mooney | it only affect deployment not using iptables | 18:55 |
sean-k-mooney | so ya all the jobs test this | 18:55 |
sean-k-mooney | for iptables ovs already plugs the interface | 18:55 |
sean-k-mooney | when not using iptables libvirt does | 18:55 |
sean-k-mooney | now os-vif will always do it regardesll fo if you use iptables or not | 18:56 |
sean-k-mooney | melwitt: we fixed it for iptables alreeady via os-vif but we could not fix it via os-vif in this case since os-vif did not do the plugging libvirt did | 18:56 |
melwitt | I see, thanks | 18:57 |
sean-k-mooney | for context the vlan specification defines vlan 4095 as reserved for implemantion use. in ovs its used for the "dead vlan" per the specification swithc whould not forward packets with vlan 4095 so ovs adopted vlan 4095 as a reserved vlan on which all traffic will be droped | 19:00 |
*** k_mouza has joined #openstack-nova | 19:00 | |
sean-k-mooney | so the mitigation is to place allports on vlan 4095 until they are moved to the tenant vlan by neutron | 19:00 |
sean-k-mooney | you enable that behavior by setting isolate_vif but we cant do it via libvirt since it will conflict with neutron if you set vlan 4095 in the xml | 19:01 |
sean-k-mooney | * isolate_vif is set in the nova.conf in the os vif plugin section | 19:02 |
*** openstackgerrit has quit IRC | 19:02 | |
sean-k-mooney | so [os_vif_ovs]/isolate_vif=true which i should proably have destack set by default at somepoint | 19:04 |
*** k_mouza has quit IRC | 19:04 | |
melwitt | ah cool, thanks for those details, nice context for the fix approach | 19:04 |
*** andrewbonney has quit IRC | 19:04 | |
sean-k-mooney | anyway to your orginial question all the tempest jobs are exefcuting the code change so i wont submit a DNM patch | 19:08 |
melwitt | sean-k-mooney: is there any change we would see in the logs as a result of the os-vif plug vs the old libvirt plug? | 19:12 |
*** efried has quit IRC | 19:18 | |
*** bbowen has quit IRC | 19:25 | |
melwitt | looks like in the guest xml the device changes to <interface type="ethernet"> from <interface type="bridge"> | 19:26 |
sean-k-mooney | yes the xml would be different | 19:29 |
sean-k-mooney | and that is in the nova compute log | 19:29 |
melwitt | cool, yeah I am looking at it | 19:29 |
sean-k-mooney | sorry was away just putting on dinner | 19:30 |
melwitt | np, I found it :) | 19:30 |
sean-k-mooney | huh i wonder if infra changed something in zuul | 19:34 |
sean-k-mooney | its not rendering the xmls correctly anymore | 19:34 |
sean-k-mooney | https://zuul.opendev.org/t/openstack/build/00ff397ca5374a0b9441036d43ee3416/log/controller/logs/screen-n-cpu.txt#7371 should be the full xml but you can only see it in the raw view | 19:34 |
melwitt | I'm looking at the raw files so I didn't see the zuul thing. but I did notice the UI changed recently in other ways | 19:34 |
sean-k-mooney | ya i might need to repot that as a bug since you cant link to lines in the raw view | 19:35 |
sean-k-mooney | i guess they have an issue with multi line stringes or soemthing | 19:35 |
melwitt | hm yeah, need to ask in -infra | 19:36 |
*** vishalmanchanda has quit IRC | 19:49 | |
*** gyee has joined #openstack-nova | 20:10 | |
*** efried has joined #openstack-nova | 20:31 | |
melwitt | sean-k-mooney: I'm trying to confirm that the nova-grenade-multinode fails on PS25 is unrelated to the change. I can't tell why the live migrations failed. do you expect there would be any issues with a live migration going from N to N+1 or vice versa during an upgrade wrt to the plugging mechanism? | 20:41 |
*** jdillaman has joined #openstack-nova | 20:46 | |
*** mgoddard has joined #openstack-nova | 20:50 | |
*** nweinber has quit IRC | 20:51 | |
*** slaweq has quit IRC | 20:58 | |
*** bbowen has joined #openstack-nova | 21:00 | |
*** sapd1 has quit IRC | 21:04 | |
*** rcernin has joined #openstack-nova | 21:13 | |
sean-k-mooney | the port would be created on ovs twice but that is ok | 21:13 |
sean-k-mooney | os-vif would create it via pre live migate on the new host then libvirt would create it again by deleteing it and readding it | 21:14 |
sean-k-mooney | that should not break anything | 21:14 |
sean-k-mooney | libvirt basiclaly does ovs-vsctl del-port --if-exist ... ovs-vsctl add-port | 21:14 |
*** sapd1 has joined #openstack-nova | 21:17 | |
*** slaweq has joined #openstack-nova | 21:31 | |
*** mgoddard has quit IRC | 21:44 | |
*** slaweq has quit IRC | 22:16 | |
*** jmlowe has quit IRC | 22:26 | |
*** CeeMac has quit IRC | 22:26 | |
*** xek_ has quit IRC | 22:26 | |
*** johnsom has quit IRC | 22:27 | |
*** tinwood_ has joined #openstack-nova | 22:28 | |
*** tinwood has quit IRC | 22:28 | |
*** johnsom has joined #openstack-nova | 22:29 | |
*** CeeMac has joined #openstack-nova | 22:30 | |
*** mandre has quit IRC | 23:12 | |
*** raildo_ has quit IRC | 23:42 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!