| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-02-11 | |||
| 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 update command https://review.opendev.org/c/openstack/nova/+/774896 | |
| 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: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: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: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: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: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 | lyarwood | ah cool I see sorry | |
| 17:43:31 | dansmith | oh, sorry sorry, you're just saying I called out the wrong test, I see | |
| 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) | |
| 20:55:26 | openstackgerrit | Merged openstack/nova master: db: Compact Stein database migrations https://review.opendev.org/c/openstack/nova/+/759090 | |
| 20:58:28 | 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 | |
| 20:58:37 | elod | sean-k-mooney: sure, added to my TODOs, will look into it tomorrow | |
| 23:01:40 | openstackgerrit | Dan Smith proposed openstack/nova master: Make a couple test jobs run async devstack https://review.opendev.org/c/openstack/nova/+/775293 | |
| 23:22:30 | 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 | |
| #openstack-nova - 2021-02-12 | |||
| 01:03:48 | openstackgerrit | Ghanshyam proposed openstack/placement stable/victoria: DNM: testing https://review.opendev.org/c/openstack/placement/+/775303 | |