Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2021-02-11
12:28:13 sean-k-mooney # the PF should also have a netdev.
12:28:20 sean-k-mooney i guess is not true for this hardware
12:28:36 sean-k-mooney so we jus tneed to readd that try except
12:29:50 sean-k-mooney ill triage it now
12:31:10 gibi thanks
12:36:21 lyarwood gibi / dansmith ; https://bugs.launchpad.net/nova/+bug/1914664 - the test failure looks like a volume quota issue in that job, we might want to track that in another bug while we look at the unrelated detach device trace
12:36:22 openstack Launchpad bug 1914664 in OpenStack Compute (nova) "QEMU monitor read failure in ServerStableDeviceRescueTest" [Undecided,New]
12:45:27 openstackgerrit Stephen Finucane proposed openstack/python-novaclient master: Add support for microversion v2.88 https://review.opendev.org/c/openstack/python-novaclient/+/770573
12:47:56 stephenfin dansmith: Now that most of the migration compaction series has merged for the main DB, could I ask that you look at the API DB equivalent? I employed the same testing strategy so it should all be very familiar https://review.opendev.org/q/topic:%2522bp/compact-db-migrations-wallaby%2522+status:open+NOT+label:workflow%253E%253D1
13:10:05 gibi lyarwood: thanks
14:01:46 dansmith lyarwood: I already had a fix up for that, I think we lost bug linkage recently
14:02:15 gibi dansmith: we don't have bug linkage since we upgraded the gerrit server
14:02:22 dansmith right
14:02:47 dansmith https://review.opendev.org/c/openstack/devstack/+/774172
14:03:26 dansmith I opened a different bug I guess I thought it was a separate issue
14:18:58 lyarwood dansmith: ah wonderful
14:32:15 openstackgerrit Merged openstack/os-vif master: Resolve dependency issues https://review.opendev.org/c/openstack/os-vif/+/774694
15:46:07 openstackgerrit Balazs Gibizer proposed openstack/nova master: Support interface attach with QoS ports https://review.opendev.org/c/openstack/nova/+/756530
15:47:37 gibi bauzas: fixed up based on your comments ^^
15:48:03 bauzas gibi: coolio, but working atm on stephenfin's nits with routed networks :p
15:48:13 gibi ack, no worries
15:51:09 openstackgerrit Balazs Gibizer proposed openstack/nova master: Remove unused Instance method https://review.opendev.org/c/openstack/nova/+/757615
15:51:48 openstackgerrit Balazs Gibizer proposed openstack/nova master: Enable mypy on scheduler/report.py https://review.opendev.org/c/openstack/nova/+/765152
15:52:18 gibi nova meeting starts in 8 minutes on #openstack-meeting-3
15:54:43 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Record the machine_type of instances in system_metadata https://review.opendev.org/c/openstack/nova/+/767533
15:54:44 openstackgerrit Lee Yarwood proposed openstack/nova master: nova-manage: Add machine_type get command https://review.opendev.org/c/openstack/nova/+/769548
15:54:44 openstackgerrit Lee Yarwood proposed openstack/nova master: nova-manage: Add machine_type update command https://review.opendev.org/c/openstack/nova/+/774896
15:54:45 openstackgerrit Lee Yarwood proposed openstack/nova master: WIP nova-manage: Add machine_type list_unset command https://review.opendev.org/c/openstack/nova/+/774897
15:54:45 openstackgerrit Lee Yarwood proposed openstack/nova master: nova-status: Add hw_machine_type check for libvirt instances https://review.opendev.org/c/openstack/nova/+/770643
15:54:46 openstackgerrit Lee Yarwood proposed openstack/nova master: libvirt: Add a config update workflow test for [libvirt]hw_machine_type https://review.opendev.org/c/openstack/nova/+/774898
15:54:46 openstackgerrit Lee Yarwood proposed openstack/nova master: docs: Add admin docs for configuring and updating machine types https://review.opendev.org/c/openstack/nova/+/774899
15:54:47 openstackgerrit Lee Yarwood proposed openstack/nova master: WIP: nova-next: Start testing the 'q35' machine type https://review.opendev.org/c/openstack/nova/+/708701
16:25:05 supamatt nah don't need it
16:25:09 supamatt err sorry wrong chan
16:34:24 gmann lyarwood: gibi elod on placement stable/victoria failure- basically we can backport this to stable/victoria along with .gitreview change - https://review.opendev.org/c/openstack/placement/+/766994
16:34:54 gmann if that looks fine as per backport policy then I can propose
16:35:35 gibi gmann: I let the stable cores decide on backportability
16:35:44 gibi for me that change make sense
16:36:06 lyarwood gmann: how clean is it on stable/victoria ?
16:36:49 lyarwood gmann: I personally don't mind if it gives us a passing branch in-line with the later releases tbh, my only concern would be if we need to fixup more code across the tree to land it
16:36:50 gmann lyarwood: basically fixing the hacking 4.0.0 checks. so some code style change
16:37:12 lyarwood gmann: yeah if they are limited to the same or just a few more then I'm fine
16:37:29 dansmith also seems okay to me.. we don't want to be bumping the requirements on stable, but this is just for our own tooling/testing I think,
16:37:43 dansmith and distros really have to do their own version mappings anyway
16:37:58 dansmith gmann: how about a reno to highlight the change and reasoning?
16:38:05 dansmith in the backport
16:38:35 gmann ok, that make sense. usually we do not need to bump hacking in stable but these version conflict made us to do
16:54:37 openstackgerrit Ghanshyam proposed openstack/placement stable/victoria: Fix l-c job and move to latest hacking 4.0.0 https://review.opendev.org/c/openstack/placement/+/775214
16:57:25 openstackgerrit Ghanshyam proposed openstack/placement stable/victoria: Fix l-c job and move to latest hacking 4.0.0 https://review.opendev.org/c/openstack/placement/+/775214
17:02:20 elod gmann: ++ \o/
17:04:55 gibi lyarwood: I have feedback in https://review.opendev.org/c/openstack/os-traits/+/759878
17:05:16 lyarwood stephenfin: re https://review.opendev.org/c/openstack/nova/+/772271 - stupid question, that didn't replace the instance.name, just the instance.hostname right?
17:05:37 stephenfin lyarwood: correct. Display name isn't affected
17:05:53 lyarwood kk well dansmith has voted now anyway so we can close this out
17:06:22 lyarwood gibi: looking
17:07:08 dansmith lyarwood: what about what I said is related to display vs hostname?
17:08:36 lyarwood dansmith: nothing but the arguments you put forward suggested that you thought the impact of this landed outside of just instance.hostname that AFAIK is something we only expose through the metadata API
17:12:41 lyarwood ah nvm it's in our show server response, ignore me if you weren't already
17:15:35 openstackgerrit Lee Yarwood proposed openstack/os-traits master: Add COMPUTE_EPHEMERAL_ENCRYPTION tratis https://review.opendev.org/c/openstack/os-traits/+/759878
17:16:53 openstackgerrit Merged openstack/os-traits master: Add COMPUTE_SOCKET_PCI_NUMA_AFFINITY trait https://review.opendev.org/c/openstack/os-traits/+/771705
17:24:32 openstackgerrit Merged openstack/os-traits master: Add a trait for UEFI Secure Boot support https://review.opendev.org/c/openstack/os-traits/+/770570
17:33:36 dansmith lyarwood: heh okay
17:33:59 dansmith lyarwood: it's still visible to the user of the instance in a variety of ways, not just the API, but yeah
17:34:35 dansmith lyarwood: back to that volumes quota thing, were you asserting that the qemu monitor reset was related to the inability to create a volume on the glance side?
17:36:41 lyarwood dansmith: no just that the QEMU monitor issue was unrelated to the actual test failure that failed the overall job
17:37:41 lyarwood dansmith: iirc it's a ipv6 test spawned that instance and it looks like the monitor issue was during cleanup and ignored
17:37:52 lyarwood ipv6 test that spawned*
17:38:49 dansmith ah okay I filed separately because they were separate, so you're just saying that's a known problem? I've seen it before obviously, but haven't in a while and since it was stable rescue, I thought maybe it was related to disk attachments
17:41:20 lyarwood dansmith: I've not seen an EOF from the monitor while detaching a nic recently
17:41:47 lyarwood dansmith: and again to be clear, that trace and the failed test are separate
17:42:08 dansmith yeah I get that
17:42:32 lyarwood kk well we can use this bug for the monitor part as you already have one for the quota bit
17:42:50 dansmith right, gibi commented on the monitor bug saying it was being tracked in the quota bug
17:43:02 lyarwood oh really?
17:43:08 dansmith so just wanted to makes ure
17:43:31 dansmith oh, sorry sorry, you're just saying I called out the wrong test, I see
17:43:31 lyarwood ah cool I see sorry
17:44:20 lyarwood yup indeed, AttachInterfacesTest is what we want to list, I've updated the subject
17:44:25 dansmith in my mind I had moved past that with the quota bug and fix, but i see what you mean about the title on the other.. the trace is the important thing
17:44:26 dansmith yep, gotcha
17:54:48 lyarwood dansmith: ah weird, so it looks like something didn't wait for the nic to detach before deleting the instance
17:55:15 dansmith ah, I guess that would explain the lack of fail
17:55:29 lyarwood dansmith: the EOF monitor error comes out of a request to handle a network-vif-deleted:064543b1-709d-445f-b852-98b59f977aed event from neutron
17:55:34 dansmith kinda sucks to barf something that serious into the logs if we're just nuking the instance underneath
17:55:41 lyarwood dansmith: and right after that n-api gets a DELETE request for the server
17:55:53 dansmith maybe we could ignore if the instance is deleted when we get that error?
17:56:08 dansmith or log.warn instead of EXPLODE
17:58:15 lyarwood dansmith: https://github.com/openstack/nova/blob/fec44e5d38baa0232bf41367303b82dc332eb512/nova/compute/manager.py#L7778-L7790 looks like we try to log at DEBUG in that case but didn't in this instance
17:58:43 lyarwood oh because it's looking at the exception and not checking if the instance is around still
18:00:46 dansmith yeah, so it probably does that right if we triggered the NotFound as a result of pulling up info on the instance,
18:01:04 dansmith but if we failed because its been nuked, we should refresh our world view before we decide who to wake up
18:01:50 dansmith although it's logging a trace, but I don't see it passing the exc_info there
18:03:48 lyarwood need to run and help put a baby to bed, I'll try and finish writing this up before I call it for the day
18:11:55 openstackgerrit Merged openstack/python-novaclient master: Uncap PrettyTable https://review.opendev.org/c/openstack/python-novaclient/+/775143
18:12:05 openstackgerrit Merged openstack/python-novaclient master: requirements: Remove simplejson https://review.opendev.org/c/openstack/python-novaclient/+/775144
20:34:30 sean-k-mooney lyarwood: can you take a look at https://review.opendev.org/c/openstack/nova/+/759522 and https://review.opendev.org/c/openstack/nova/+/759151
20:35:09 sean-k-mooney elod: if you could take a look too that would be great that has to go back to train
20:35:48 sean-k-mooney victoria is merged so ussuri is up next
20:36:19 sean-k-mooney there are a few people askinf for this on the bug https://bugs.launchpad.net/nova/+bug/1888395
20:36:22 openstack Launchpad bug 1888395 in OpenStack Compute (nova) ussuri "live migration of a vm using the single port binding work flow is broken in train as a result of the introduction of sriov live migration" [High,In progress] - Assigned to Billy Olsen (billy-olsen)

Earlier   Later