| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-02-11 | |||
| 12:20:34 | gibi | yes, we have to allow the compute to start but then this interface will not be ever selected for a port that has qos request | |
| 12:20:43 | sean-k-mooney | yep | |
| 12:20:53 | gibi | thanks for the triage on this bug | |
| 12:22:05 | sean-k-mooney | so there are two ways we can adress this it looks like the old logic just logged. | |
| 12:22:12 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/739131/12/nova/pci/utils.py | |
| 12:22:28 | sean-k-mooney | so we can do that | |
| 12:22:47 | sean-k-mooney | or we can also maybe stop using sysfs and use libvirt | |
| 12:22:55 | sean-k-mooney | or both in seperate patchs | |
| 12:23:09 | lyarwood | gibi: looking | |
| 12:23:18 | gibi | lyarwood: thanks | |
| 12:23:26 | openstackgerrit | Stephen Finucane proposed openstack/nova master: Uncap PrettyTable https://review.opendev.org/c/openstack/nova/+/775142 | |
| 12:23:39 | gibi | sean-k-mooney: as a fix I would just log now | |
| 12:24:14 | sean-k-mooney | oh actully | |
| 12:24:27 | sean-k-mooney | this code is common its not allowed to talk to libvirt | |
| 12:24:28 | gibi | sean-k-mooney: as far as I understand your patch moved away from libvirt for certain reasons | |
| 12:24:34 | sean-k-mooney | that is why we are using sysfs | |
| 12:25:13 | sean-k-mooney | ya libvirt can get out os sync | |
| 12:25:28 | sean-k-mooney | for nodedev names | |
| 12:25:51 | sean-k-mooney | so ya using libvirt here would not be right | |
| 12:26:22 | openstackgerrit | Stephen Finucane proposed openstack/python-novaclient master: Uncap PrettyTable https://review.opendev.org/c/openstack/python-novaclient/+/775143 | |
| 12:26:22 | openstackgerrit | Stephen Finucane proposed openstack/python-novaclient master: requirements: Remove simplejson https://review.opendev.org/c/openstack/python-novaclient/+/775144 | |
| 12:26:41 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/739131/12/nova/virt/libvirt/driver.py#b6960 | |
| 12:26:52 | sean-k-mooney | that might be the issue | |
| 12:28:08 | sean-k-mooney | ya so my note | |
| 12:28:10 | sean-k-mooney | # NOTE(sean-k-mooney): if the VF is a parent of a netdev | |
| 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 | |