| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-03-25 | |||
| 17:05:32 | sean-k-mooney | i was just looking at a different xml an realise i read your one wrong the first time | |
| 17:06:03 | sean-k-mooney | noonedeadpunk: if there isnt an issue great but we found we are missing coverage in whitebox in either case | |
| 17:16:13 | sean-k-mooney | noonedeadpunk: if the bug is not required feel free to close it but if its still does not work after you fix the typo let me know | |
| 17:17:45 | sean-k-mooney | oh s/resouces/resources/ | |
| 17:17:50 | sean-k-mooney | ya i would never spot that | |
| 21:24:06 | opendevreview | Merged openstack/nova master: Adds regression test for bug LP#1944619 https://review.opendev.org/c/openstack/nova/+/833166 | |
| #openstack-nova - 2022-03-26 | |||
| 03:03:27 | opendevreview | Ghanshyam proposed openstack/nova stable/yoga: DNM: test grenade skip job running https://review.opendev.org/c/openstack/nova/+/835350 | |
| 03:04:28 | opendevreview | Ghanshyam proposed openstack/nova master: DNM: test grenade skip job running https://review.opendev.org/c/openstack/nova/+/835351 | |
| 14:33:43 | opendevreview | Takashi Natsume proposed openstack/python-novaclient master: Add Python3 zed unit tests https://review.opendev.org/c/openstack/python-novaclient/+/835368 | |
| 14:43:23 | opendevreview | Takashi Natsume proposed openstack/osc-placement master: Add Python3 zed unit tests https://review.opendev.org/c/openstack/osc-placement/+/835369 | |
| 15:06:19 | opendevreview | Takashi Natsume proposed openstack/python-novaclient master: Add openssl in bindep.txt https://review.opendev.org/c/openstack/python-novaclient/+/835370 | |
| 15:10:12 | opendevreview | Takashi Natsume proposed openstack/python-novaclient master: Add openssl in bindep.txt https://review.opendev.org/c/openstack/python-novaclient/+/835370 | |
| #openstack-nova - 2022-03-28 | |||
| 03:01:50 | opendevreview | Takashi Natsume proposed openstack/python-novaclient master: Remove USE_PYTHON3 in .zuul.yaml https://review.opendev.org/c/openstack/python-novaclient/+/835405 | |
| 12:12:39 | viks__ | i'm facing the issue "Affinity policy violated with parallel requests".. i saw the resolution give in https://docs.openstack.org/nova/latest/admin/troubleshooting/affinity-policy-violated.html. I have tried setting `disable_group_policy_check_upcall`, but it does not seem to hve any effect.. Can someone please guide, why the solution provide there does not work? | |
| 12:14:10 | sean-k-mooney | so you have set disable_group_policy_check_upcall = False | |
| 12:14:19 | sean-k-mooney | on the compute nodes | |
| 12:14:33 | sean-k-mooney | and its still allowing the policy to be violated | |
| 12:17:51 | viks__ | yes... basically the instances are going in to same host | |
| 12:19:24 | viks__ | i'm using train version | |
| 12:20:05 | sean-k-mooney | and you have ensured the cell conductor has ascess to the api db | |
| 12:20:34 | sean-k-mooney | the way this up call works is one of the vm should boot but then the rest should fail and reschdule to other hosts | |
| 12:24:43 | viks__ | sean-k-mooney: `[api_database] connection=`?. yes i have set this | |
| 12:25:20 | sean-k-mooney | in the conductors? if so then the second vm shoudl fail to boot. | |
| 12:25:31 | sean-k-mooney | have you eneabled debug loging | |
| 12:25:44 | sean-k-mooney | do you see any references to upcalls in teh compute-agent log | |
| 12:34:52 | viks__ | sean-k-mooney: yes... in conductors..will check that | |
| 14:46:15 | dansmith | can someone +W this test fixture fix? https://review.opendev.org/c/openstack/nova/+/831084 | |
| 14:46:27 | dansmith | apparently it will reduce some CI failures | |
| 14:46:44 | sean-k-mooney | give me 2 mins and ill take a look | |
| 15:02:39 | bauzas | dansmith: done | |
| 15:04:36 | bauzas | cores : https://review.opendev.org/c/openstack/nova-specs/+/835272 (moving specs to implemented) | |
| 15:28:38 | opendevreview | Merged openstack/nova master: Retry in CellDatabases fixture when global DB state changes https://review.opendev.org/c/openstack/nova/+/831084 | |
| 15:51:20 | opendevreview | Erlon R. Cruz proposed openstack/nova master: Fix pre_live_migration rollback https://review.opendev.org/c/openstack/nova/+/815324 | |
| 15:57:32 | bauzas | folks, I know it's late but I'm asked by the Foundation folks to present the Nova Yoga higlights on Thurs | |
| 15:57:52 | bauzas | so I'm writing two slides, one for the yoga meat and one for the Zed plans we have | |
| 15:58:00 | bauzas | obviously the Zed slide is the beefy one | |
| 16:11:40 | bauzas | so I'll ask folks tomorrow at the nova meeting what are their plans | |
| 17:26:44 | opendevreview | Merged openstack/python-novaclient master: Remove USE_PYTHON3 in .zuul.yaml https://review.opendev.org/c/openstack/python-novaclient/+/835405 | |
| #openstack-nova - 2022-03-29 | |||
| 02:26:16 | opendevreview | Takashi Natsume proposed openstack/python-novaclient master: Add Python3 zed unit tests https://review.opendev.org/c/openstack/python-novaclient/+/835368 | |
| 02:26:37 | opendevreview | Takashi Natsume proposed openstack/python-novaclient master: Add openssl in bindep.txt https://review.opendev.org/c/openstack/python-novaclient/+/835370 | |
| 02:36:45 | opendevreview | Takashi Natsume proposed openstack/osc-placement master: Change minversion of tox to 3.18.0 https://review.opendev.org/c/openstack/osc-placement/+/791972 | |
| 06:23:20 | opendevreview | Merged openstack/nova master: Move FIPS jobs to experimental and periodic queue https://review.opendev.org/c/openstack/nova/+/833431 | |
| 08:55:38 | opendevreview | Stephen Finucane proposed openstack/nova master: objects: Remove 'NovaObjectDictCompat' from 'Service' https://review.opendev.org/c/openstack/nova/+/835595 | |
| 08:59:23 | opendevreview | Stephen Finucane proposed openstack/nova master: objects: Remove unnecessary type aliases, exceptions https://review.opendev.org/c/openstack/nova/+/738240 | |
| 08:59:23 | opendevreview | Stephen Finucane proposed openstack/nova master: objects: Don't use generic 'Field' container https://review.opendev.org/c/openstack/nova/+/738239 | |
| 08:59:24 | opendevreview | Stephen Finucane proposed openstack/nova master: objects: Remove wrappers around ovo mixins https://review.opendev.org/c/openstack/nova/+/738019 | |
| 08:59:24 | opendevreview | Stephen Finucane proposed openstack/nova master: objects: Use imports instead of type aliases https://review.opendev.org/c/openstack/nova/+/738018 | |
| 08:59:25 | opendevreview | Stephen Finucane proposed openstack/nova master: WIP: add ovo-mypy-plugin to type hinting o.vos https://review.opendev.org/c/openstack/nova/+/758851 | |
| 08:59:25 | opendevreview | Stephen Finucane proposed openstack/nova master: objects: Remove 'NovaObjectDictCompat' from 'Service' https://review.opendev.org/c/openstack/nova/+/835595 | |
| 13:37:08 | bauzas | folks, I have a problem with the nova meeting today | |
| 13:37:33 | bauzas | since Europe is now on DST, our meeting will be at 4pm UTC but 6pm for me | |
| 13:37:54 | bauzas | but I need to go to someway by 7pm | |
| 13:38:41 | sean-k-mooney | ack we coudl just cancel it since the ptg is next week | |
| 13:38:54 | bauzas | so either we have a small meeting until 4.15pm UTC or someone else could continue to discuss it | |
| 13:38:55 | dansmith | ++ | |
| 13:38:57 | sean-k-mooney | unless there is imporant topics that cant wait | |
| 13:38:59 | dansmith | TC also canceled this week | |
| 13:39:13 | bauzas | I have a small agenda at the time | |
| 13:39:31 | bauzas | at least for asking folks to provide their PTG topics | |
| 13:39:38 | bauzas | https://wiki.openstack.org/wiki/Meetings/Nova#Agenda_for_next_meeting | |
| 13:39:53 | sean-k-mooney | ok do you want to just skip the normal standing itmes then and have a short meetign to cover that topic and wrap early | |
| 13:40:04 | bauzas | sean-k-mooney: we could do this | |
| 13:40:35 | bauzas | sean-k-mooney: just opening the meeting for the PTG | |
| 13:40:51 | bauzas | and then I'll provide a email thread | |
| 13:45:36 | noonedeadpunk | hey there! I was wondering - any reason we do this https://opendev.org/openstack/nova/src/branch/master/nova/virt/libvirt/driver.py#L986 ? | |
| 13:47:14 | noonedeadpunk | Why I'm asking is bassically after talk in #virt regarding MPX flag, that makes IceLake CPUs being identified as Broadwell..... | |
| 13:47:47 | noonedeadpunk | I mean - there's masking of flags being already implemented | |
| 13:48:08 | noonedeadpunk | so technically comparing provided model to virsh capabilities is not needed | |
| 13:50:06 | sean-k-mooney | if you have configured a cpu model that is not compatibale with the host the agent shoudl refuse to start | |
| 13:50:19 | sean-k-mooney | so we have to check cpu compatiabity on agent start up | |
| 13:50:57 | sean-k-mooney | noonedeadpunk: we have some patches in flight to use the newer libvirt api to do this which takes into account which flags are masked or emulated | |
| 13:51:03 | noonedeadpunk | So I have intel xeon gold 6338 and trying to provide Icelake, but only Broadwell is available for it | |
| 13:51:13 | noonedeadpunk | oh | |
| 13:51:54 | sean-k-mooney | presumable icelake reference cpu feature flags that are not present on the host correct | |
| 13:52:03 | sean-k-mooney | perhaps tsk | |
| 13:52:06 | sean-k-mooney | *tsz | |
| 13:52:10 | noonedeadpunk | well, the only "streight" way is to add dozen of extra flags to cover difference... | |
| 13:52:10 | sean-k-mooney | ... tsx | |
| 13:53:04 | noonedeadpunk | sean-k-mooney: so the falg that is missing is MPX which is sunset since 2019... But #virt said they can't jsut drop it for $reasons | |
| 13:53:36 | noonedeadpunk | and they just suggested to mask it as they don't have other option | |
| 13:55:01 | sean-k-mooney | ya https://paste.opendev.org/show/bmfNxhnzVYp54W9LQK0J/ so mpx is listed in the cpu model | |
| 13:55:56 | noonedeadpunk | and I mean - https://lore.kernel.org/lkml/tip-eb012ef3b4e331ae479dd7cd9378041d9b7f851c@git.kernel.org/ and Intel also said that since 2019 no CPU will have it | |
| 13:56:53 | noonedeadpunk | so I'm kind of... either use cpu model from 2014 or have dozens of flags specified in extras... | |
| 13:57:05 | noonedeadpunk | or manually edit cpu_map which is too bad | |
| 13:57:22 | sean-k-mooney | well that is a abi break on intels part which libvirt could accoutn for by adding a no_MPX verison of the cpu model | |
| 13:57:32 | sean-k-mooney | but looking at https://opendev.org/openstack/nova/src/branch/master/nova/virt/libvirt/driver.py#L982-L1008 | |
| 13:57:56 | sean-k-mooney | we coudl rework this os that we apply the extra cpu flags to the model before we do the compat test | |
| 13:58:21 | sean-k-mooney | that woudl allow you to add cpu_model_extra_flags=-mpx | |
| 13:59:01 | sean-k-mooney | and provide that combination was valid it would proceed | |
| 13:59:28 | noonedeadpunk | well, self._get_cpu_info() is jsut `virsh capabilities`? | |
| 13:59:29 | sean-k-mooney | right now it checks that the each model is valid on its own and then the default mode + the extra flags is valid | |
| 13:59:47 | noonedeadpunk | and `virsh capabilities` falls back to Broadwell | |
| 14:00:02 | noonedeadpunk | because all else have that damn MPX | |
| 14:00:24 | sean-k-mooney | yes again its an abi break by intel effectivly | |
| 14:00:59 | sean-k-mooney | this might be address by the new cpu compare api that we are moving too | |
| 14:01:07 | sean-k-mooney | but if not we can rework that validation | |
| 14:01:17 | sean-k-mooney | the curertn two pahse apprch is not really correct | |
| 14:01:25 | sean-k-mooney | it predates the ablity to remove flags | |
| 14:01:42 | sean-k-mooney | when we added the ability to remove flags this shoudl have been factored into one loop | |
| 14:02:09 | sean-k-mooney | for now yes your only option is to ude broadwall and add the missing flags | |