Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2022-02-03
15:26:03 dmitriis gibi: could also do a debug or info level message at least
15:26:20 gibi dmitriis: go with a debug then
15:27:43 dmitriis gibi: ack, I mentioned this in the "limitations" note in the doc change and also in the conf module doc
15:28:16 gibi dmitriis: cool
15:28:19 gibi then it is settle
15:28:20 gibi d
15:29:58 dmitriis gibi: ack, will also document this in the Neutron guide since other features cover the compute part as well
15:40:38 sean-k-mooney ya proably debug makes sense but i have not got that far in the change set sorry
15:41:02 sean-k-mooney i have been looking at stephens docs patchs and some downstream stuff so this is sitll on my todo list
15:49:34 dmitriis sean-k-mooney: ack, np
15:57:23 sean-k-mooney stephenfin: by the way i think we should exclude that failign test in nova-next for now until we find a workaround
15:57:43 stephenfin I agree
15:57:51 stephenfin what does gibi think?
15:57:51 sean-k-mooney artom: started a mail thread on it but its a q35 issue
15:58:21 artom sean-k-mooney, yeah, my latest DNM patch that removed the q35 machine type from nova-next passed
15:58:24 gibi sean-k-mooney: do we have the result back from the testing that proves that it is q35?
15:58:30 gibi artom: ohh
15:58:40 gibi artom: but you also added the waiters in the same patch isn't it?
15:58:54 sean-k-mooney i think those are seperate
15:58:55 artom gibi, waiters were there before the q35 removal, and they failed
15:59:03 sean-k-mooney ah
15:59:09 gibi artom: ok then q35 it is
15:59:24 sean-k-mooney so i think this runs in other test jobs if it does i think we can skip it in nova-next
15:59:30 sean-k-mooney which is the only q35 one for now
15:59:49 gibi I'm OK to remove the test temporarily from nova-next while we figure out how to fix it
16:00:09 sean-k-mooney artom: are you going to try any of my suggestions form the mail
16:00:15 artom sean-k-mooney, I haven't read it yet
16:00:20 sean-k-mooney no worries
16:00:51 gibi btw, my dnm patch also passed nova-next and it only added logs to tempest :/
16:01:24 sean-k-mooney you can see on one of stephens patchs that it passed in check and failed in gate
16:01:37 sean-k-mooney so it not a 100% failure just common
16:01:46 gibi yeah
16:01:47 yuval is there a nova-meeting today?
16:01:53 sean-k-mooney no
16:02:02 sean-k-mooney its on tuseday
16:02:10 sean-k-mooney but you can still bring stuff up any time
16:02:11 yuval ohh ok sorr
16:02:17 gibi which also means passing on non q35 might be just luck :/
16:02:36 sean-k-mooney well maybe but the other jobs seam to be ok
16:03:01 sean-k-mooney i guess we could check logstash and confirm
16:03:15 artom gibi, huh... I mean, it was never 100%, but this does seem like a weird coincidence
16:03:32 artom I wonder if we can check the qemu and/or libvirt versions in ubuntu
16:03:39 artom What changed and when
16:03:44 gibi OK, the tagged attach runs in tempest-integrated-compute and that is nicely green
16:03:54 sean-k-mooney they are in the devstack logs
16:04:05 sean-k-mooney if you want to check but i doubt that is the problem
16:28:35 artom_ So one interesting thing is that in the console logs for a failing test_tagged_attachment, I'm seeing "[ 5.322454] pcieport 0000:00:04.5: pciehp: Failed to check link status"
16:33:44 sean-k-mooney oh
16:33:53 sean-k-mooney so that could be related to the qemu patch
16:33:58 sean-k-mooney realted to state tracking
16:37:29 artom_ I'm going to try and see what's in the console for the passing run on gibi's patch
16:42:54 rosmaita sean-k-mooney bauzas: do you need a pre-yoga-release release of os-brick so you can test lightbits code for nova?
16:43:18 sean-k-mooney rosmaita: yes
16:44:03 sean-k-mooney we do not allow code to changes to merge if they depend of unrelease libs
16:44:07 rosmaita sean-k-mooney: ok, i will propose one today ... hopefully will get released right away, since it's not friday yet
16:44:29 sean-k-mooney well anytime before the non-client lib freeze is technically fine
16:44:31 artom Oh wait, we don't log the console by default, do we
16:44:39 sean-k-mooney but the nova patches wont pass ci until the release is done
16:45:05 sean-k-mooney rosmaita: so as long as there is a 5.2.0 before m3 it shoudl be ok
16:45:18 sean-k-mooney the sonner before m3 the less risk to the nova change
16:46:10 rosmaita sean-k-mooney: ok, we are planning to release os-brick one week early this cycle (so next week)
16:46:25 rosmaita if that would be ok, i won't do a pre-release to avoid confusion
16:46:48 sean-k-mooney ack yuval ^ are you ok with that
16:47:11 sean-k-mooney rosmaita: its release with intermediay so you can do addtional release at any point by the way
16:47:19 sean-k-mooney but next week likely will be fine
16:48:26 yuval yes, got it - we finish the followup til 10 feb - so it will leave window to merge the nova code till 21
16:48:58 rosmaita that sounds good, it would be better with the followup patches merged, i think
16:54:57 spatel sean-k-mooney does multiple pci_alias address allow like this in nova.conf file ? - https://paste.opendev.org/show/812504/
16:56:05 sean-k-mooney you can have multiple alsiases i need to check if its a multiopt like that or a json list or both
16:56:17 sean-k-mooney it should be in our docs but i dont recall off the top of my head
16:56:58 sean-k-mooney spatel: so yes https://docs.openstack.org/nova/latest/configuration/config.html#pci.alias
16:57:17 sean-k-mooney that sould work
16:57:25 spatel nice thanks
17:16:18 gibi bauzas: left some comment for you in the ipless port patch
17:16:25 gibi bauzas: nothing major
18:27:57 opendevreview Merged openstack/placement master: Extra tests around required traits https://review.opendev.org/c/openstack/placement/+/825846
18:28:04 opendevreview Merged openstack/nova master: docs: Add new architecture guide https://review.opendev.org/c/openstack/nova/+/814563
18:51:10 opendevreview Merged openstack/nova master: Add 'hw:vif_multiqueue_enabled' flavor extra spec https://review.opendev.org/c/openstack/nova/+/792356
19:37:02 opendevreview melanie witt proposed openstack/nova master: Raise InstanceNotFound on fkey constraint fail saving info cache https://review.opendev.org/c/openstack/nova/+/826942
20:32:24 admin1 hi .. what does qemu unexpectedly closed the monitor mean during migration ? log snippet here: https://gist.githubusercontent.com/a1git/3029cfd14883531c8ea7b12d0491d8bb/raw/74f57d9ff1f278f79dc0dca685f71508b779d40d/gistfile1.txt
20:33:12 admin1 qemu-system-x86_64: Length mismatch: 0000:00:03.0/virtio-net-pci.rom: 0x40000 in != 0x80000: Invalid argument
20:45:08 opendevreview melanie witt proposed openstack/nova master: Raise InstanceNotFound on fkey constraint fail saving info cache https://review.opendev.org/c/openstack/nova/+/826942
20:48:58 opendevreview Dmitrii Shcherbakov proposed openstack/nova master: Introduce remote_managed tag for PCI devs https://review.opendev.org/c/openstack/nova/+/824834
20:48:59 opendevreview Dmitrii Shcherbakov proposed openstack/nova master: Bump os-traits to 2.7.0 https://review.opendev.org/c/openstack/nova/+/826675
20:48:59 opendevreview Dmitrii Shcherbakov proposed openstack/nova master: [yoga] Add support for VNIC_TYPE_SMARTNIC https://review.opendev.org/c/openstack/nova/+/824835
20:49:00 opendevreview Dmitrii Shcherbakov proposed openstack/nova master: Filter computes without remote-managed ports early https://review.opendev.org/c/openstack/nova/+/812111
20:57:50 frickler admin1: https://bugs.launchpad.net/ubuntu/+source/qemu/+bug/1713490
22:20:06 admin1 frickler .. thanks .. i tried the migration again to another hypervisor with the exact same cpu and it worked without issues .. the original error was between E5-2660 v2 @ 2.20GHz -> E5-2670 v2 @ 2.50GHz .. since it was an upgraded version of the cpu, it should have worked
#openstack-nova - 2022-02-04
11:10:43 gibi sean-k-mooney: hi! could you upgrade my vote from +1 to +2 on https://review.opendev.org/q/topic:bug%252F1952941 ? thanks!
11:11:25 opendevreview Dmitrii Shcherbakov proposed openstack/nova master: [yoga] Add support for VNIC_TYPE_SMARTNIC https://review.opendev.org/c/openstack/nova/+/824835
11:11:26 opendevreview Dmitrii Shcherbakov proposed openstack/nova master: Filter computes without remote-managed ports early https://review.opendev.org/c/openstack/nova/+/812111
11:11:26 opendevreview Dmitrii Shcherbakov proposed openstack/nova master: Add supports_remote_managed_ports capability https://review.opendev.org/c/openstack/nova/+/827839
11:12:26 sean-k-mooney[m] gibi: ah the requestspec fix sure
11:12:45 dmitriis gibi: apologies for an extra respin of https://review.opendev.org/c/openstack/nova/+/812111. I moved the addition of the compute capability to a separate change https://review.opendev.org/c/openstack/nova/+/827839 and made an extra check in the whitelist. This way we can check if Libvirt has the right version and reject remote_managed ports if it
11:12:45 dmitriis doesn't early at startup.
11:12:47 sean-k-mooney[m] ill review the series shortly
11:16:04 gibi dmitriis: no worry I will check after lunch
11:16:12 dmitriis gibi: ack, tyvm
11:18:39 opendevreview sean mooney proposed openstack/nova master: [WIP] add initial healthcheck support https://review.opendev.org/c/openstack/nova/+/825015
11:18:39 opendevreview sean mooney proposed openstack/nova master: [WIP] add healthcheck manager to manager base https://review.opendev.org/c/openstack/nova/+/827844

Earlier   Later