| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-03-23 | |||
| 08:28:22 | gibi | Uggla: o/ | |
| 08:29:40 | EugenMayer | sean-k-mooney[m] i see only warnings about deprecations in horizon.log - i found some logs on the compute from yesterday failing to contact rabbitmq which are most probably unrelated: https://gist.github.com/EugenMayer/cf06fadc52c66018f09a470fe0ed260f - that's about what i could find | |
| 08:32:36 | gibi | Uggla: thanks again fort the docs :) | |
| 08:34:25 | Uggla | gibi, you are welcome. | |
| 09:00:44 | bauzas | gibi: \o | |
| 09:01:29 | gibi | bauzas: o/ | |
| 09:33:12 | opendevreview | ribaudr proposed openstack/nova master: Fix unit tests when they are run with OS_DEBUG=True https://review.opendev.org/c/openstack/nova/+/833115 | |
| 09:58:49 | opendevreview | Stephen Finucane proposed openstack/nova master: doc: Remove useless contributor/api-2 doc https://review.opendev.org/c/openstack/nova/+/828599 | |
| 10:59:39 | opendevreview | Rajat Dhasmana proposed openstack/nova master: Add conductor RPC interface for rebuild https://review.opendev.org/c/openstack/nova/+/831219 | |
| 10:59:39 | opendevreview | Rajat Dhasmana proposed openstack/nova master: Add support for volume backed server rebuild https://review.opendev.org/c/openstack/nova/+/820368 | |
| 10:59:40 | opendevreview | Rajat Dhasmana proposed openstack/nova master: Add API support for rebuilding BFV instances https://review.opendev.org/c/openstack/nova/+/830883 | |
| 11:05:49 | EugenMayer | when an instance keeps in the 'spawning' phase - where can i check for the cause? | |
| 11:09:52 | kashyap | gibi: Hey, on git/main with a patch, I see this: https://cpaste.org/?37b24bb8794dcdca#9jFENZMo38Bxjskev5QkLoGncaKucSgpx1tBHhvVKAYd | |
| 11:10:04 | kashyap | gibi: Am I missing anything trivial? os_traits is installed in my virtualenv. | |
| 11:36:49 | gibi | you probably have old os_traits lib in the venv | |
| 11:37:09 | opendevreview | Elod Illes proposed openstack/nova stable/xena: [stable-only] Set nova-tox-functional-centos8-py36 non-voting https://review.opendev.org/c/openstack/nova/+/834854 | |
| 11:38:06 | gibi | kashyap: you need 2.7.0 from os-traits https://github.com/openstack/os-traits/commits/2.7.0 | |
| 12:42:40 | opendevreview | Balazs Gibizer proposed openstack/nova master: Record SRIOV PF MAC in the binding profile https://review.opendev.org/c/openstack/nova/+/829248 | |
| 12:45:19 | kashyap | gibi: Darn lemme check | |
| 12:46:19 | kashyap | $> pip show os_traits | grep Version | |
| 12:46:19 | kashyap | gibi: Hm, I do have it: | |
| 12:46:20 | kashyap | Version: 2.7.0 | |
| 12:46:48 | gibi | that is strange | |
| 12:49:29 | gibi | https://paste.opendev.org/show/buddLlqeEzZc7ClpZWF3/ I can import that constant | |
| 12:51:23 | gibi | kashyap: could you try just import that constan in an interpreter started in that env? | |
| 12:54:12 | Uggla | I though a "reverify" comment would restart the CI... but it appears to not be the case. How to restart CI on a patches ? | |
| 12:55:10 | kashyap | gibi: Lemme try | |
| 12:56:02 | kashyap | gibi: Hmm, import of that constant does work just fine here. | |
| 12:57:59 | kashyap | gibi: I tell a lie; it's not git/main, but on top of the CPU APIs patch (that I rebased). I'll try on git/main and report back. (Bad me) | |
| 12:58:34 | kashyap | Hmm, same on git/master too. | |
| 13:22:40 | kashyap | Okay, that fixed it. (Sorry for the noise. | |
| 13:22:41 | kashyap | ) | |
| 13:32:29 | artom | Uggla, it's 'recheck' | |
| 13:32:38 | artom | 'reverify' is only in our downstream CI | |
| 13:32:49 | artom | Trigger words can be configured in Zuul, IIUC | |
| 13:34:44 | opendevreview | Kashyap Chamarthy proposed openstack/nova master: libvirt: Use improved guest CPU config APIs https://review.opendev.org/c/openstack/nova/+/762330 | |
| 13:36:16 | Uggla | artom, I tried "rebuild", "reverify".... ;) | |
| 13:36:26 | artom | ;) | |
| 13:38:57 | Uggla | artom, thanks, it seems to be in the zull queue now. | |
| 13:39:13 | gibi | kashyap: no worries, then something was mixed in that env | |
| 13:39:46 | Uggla | zuul* | |
| 14:13:41 | sean-k-mooney | Uggla: reverify in the long ago before times used to be use to tirgger just the gate pipeline rather then both gate and check. we removed it whith the green check policy dission about 8 years ago but downstream we trigger off both recheck and reverify for legacy reasons but ya as artom said recheck is what is used upstream when reqruied | |
| 14:41:56 | Uggla | sean-k-mooney, thx for the info. | |
| 14:44:52 | sean-k-mooney | gmann: bauzas since ye have already +2'd the followup care to review https://review.opendev.org/c/openstack/nova/+/828598/1 so that the docs pathes can proceed | |
| 14:45:32 | bauzas | sean-k-mooney: shit I missed the fact it was depending on this | |
| 14:46:02 | gmann | I had in list then missed somehow. thanks | |
| 14:46:03 | sean-k-mooney | no worreis this has been on my review list for a while | |
| 14:56:42 | sean-k-mooney | gibi: just looking at https://review.opendev.org/q/topic:any-traits-support the remaining patches are just using any traits right e.g. client supprot nova testing ectra. the feature is "complete" from the placmenet side in yoga? | |
| 14:57:49 | sean-k-mooney | just going through my review backlog but i had mentally check that off as complete | |
| 14:58:50 | gibi | sean-k-mooney: yes the feature is complete from placement perspective | |
| 14:59:17 | gibi | I have the placement client patch open, but that is not super important | |
| 14:59:29 | gibi | and I have the todo to check the sql performance and improve if needed | |
| 15:00:58 | opendevreview | Merged openstack/nova master: doc: Split up notifications document https://review.opendev.org/c/openstack/nova/+/828598 | |
| 15:01:19 | gibi | hm I missed that review ^^ but quickly checked and looks good to me too | |
| 15:13:34 | opendevreview | Merged openstack/nova master: doc: Remove useless contributor/api-2 doc https://review.opendev.org/c/openstack/nova/+/828599 | |
| 15:15:43 | opendevreview | Stephen Finucane proposed openstack/nova master: objects: Remove 'NovaObjectDictCompat' from 'Migration' https://review.opendev.org/c/openstack/nova/+/723572 | |
| 15:16:03 | opendevreview | Stephen Finucane proposed openstack/nova master: objects: Remove 'NovaObjectDictCompat' from 'InstancePCIRequest' https://review.opendev.org/c/openstack/nova/+/723573 | |
| 15:45:47 | outbrito | Morning folks! I'm trying to run tox @ master locally on my work laptop (ubuntu 20.04) and I'm getting a weird error: oslo_config.cfg.RequiredOptError: value required for option lock_path in group [DEFAULT] | |
| 15:45:54 | outbrito | Is that a known issue? | |
| 15:47:26 | sean-k-mooney | am i have seen that locally too | |
| 15:47:32 | sean-k-mooney | but not sure if we have a tacker for it | |
| 15:47:34 | dansmith | rebuild your venv? | |
| 15:47:38 | dansmith | tox -r | |
| 15:47:57 | outbrito | It was a brand new clone, but I'll try anyway | |
| 15:49:13 | dansmith | ack, usually random oslo things for me are due to a stale venv when reqs had changed, so that's the first thing I try | |
| 15:49:57 | sean-k-mooney | outbrito: is this actully causing test failures for you or just an warning/error message | |
| 15:50:07 | outbrito | 5 test failing | |
| 15:50:13 | outbrito | tests* | |
| 15:50:49 | sean-k-mooney | ok that actully the same as i saw on my mac book air tests yesterday also on a clean clone | |
| 15:51:11 | sean-k-mooney | let me see if i can repoduce on my normal system | |
| 15:53:41 | sean-k-mooney | i updated my ubuntu 20.04 server to 21.10 at the weekend but its running them now | |
| 15:54:17 | sean-k-mooney | so it wont be idential hum so i did get 1 failure with master | |
| 15:54:22 | sean-k-mooney | ill try in a clean clone | |
| 15:54:58 | sean-k-mooney | outbrito: what version of python did you run them under | |
| 15:55:09 | sean-k-mooney | 3.8 i assume? | |
| 15:55:14 | sean-k-mooney | just in case that matters | |
| 15:57:13 | opendevreview | Merged openstack/placement master: tests: Silence noisy tests https://review.opendev.org/c/openstack/placement/+/801101 | |
| 15:58:34 | outbrito | 3.9 | |
| 16:00:26 | outbrito | The only differences I see on requirements is Yappi vs. yappi and PyNaCl vs. pynacl | |
| 16:01:53 | gibi | hm with 20.04 + py38 + fresh master I could not reproduce the lock_path issue | |
| 16:03:54 | outbrito | The tests failing are these btw: | |
| 16:04:02 | outbrito | nova.tests.unit.virt.test_block_device.TestDriverBlockDeviceNewFlow.test_volume_attach_with_shared_targets | |
| 16:04:02 | outbrito | nova.tests.unit.compute.test_compute_mgr.ComputeManagerUnitTestCase.test_detach_volume_evacuate | |
| 16:04:02 | outbrito | nova.tests.unit.compute.test_compute_mgr.ComputeManagerUnitTestCase.test_detach_volume_evacuate_legacy | |
| 16:04:02 | outbrito | nova.tests.unit.compute.test_compute_mgr.ComputeManagerUnitTestCase.test_detach_volume_evacuate_mismatch | |
| 16:04:04 | outbrito | nova.tests.unit.virt.test_block_device.TestDriverBlockDevice.test_volume_attach_with_shared_targets | |
| 16:04:57 | outbrito | All related to volumes for that matter | |
| 16:04:58 | opendevreview | Merged openstack/placement master: tox: Remove psycopg2 warning filter https://review.opendev.org/c/openstack/placement/+/800911 | |
| 16:08:48 | sean-k-mooney | on a clean clone on 21.10 i did not have any failure under python 39 | |
| 16:10:37 | outbrito | py38 also broken for me | |
| 16:11:24 | outbrito | just to be sure, last commit is d2b2516 right? | |
| 16:11:39 | gibi | yepp | |
| 16:11:45 | gibi | I see that as head too | |
| 16:11:51 | gibi | but no luck reproducing the issue | |
| 16:12:07 | gibi | does it happen the same way for you each time you rerun these tests? | |
| 16:12:15 | gibi | or is it unstable? | |
| 16:13:47 | outbrito | consistent | |
| 16:15:30 | gibi | outbrito: do you also get a stack trace? If yes could you past it via https://paste.opendev.org/ (or similar) | |
| 16:17:04 | opendevreview | Merged openstack/placement master: setup: Replace dashes with underscores https://review.opendev.org/c/openstack/placement/+/801102 | |
| 16:17:34 | outbrito | https://paste.opendev.org/show/bZm6XstZ4MTPM7SPVuCR/ | |