| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-11-15 | |||
| 16:24:35 | sean-k-mooney | oh never mind we dont need too | |
| 16:24:41 | sean-k-mooney | we only change them on stable to aovid that | |
| 16:24:45 | elodilles | then even better :) | |
| 16:25:13 | sean-k-mooney | i saw the email come in for the release this mornign but didnt open the patch yet | |
| 16:26:06 | elodilles | the generated patches on master branch seem to be merged | |
| 16:26:14 | sean-k-mooney | looking at the oneline summery i think this makes sense and i agree a feature bump is correct | |
| 16:26:29 | elodilles | ++ | |
| 16:26:51 | sean-k-mooney | so there was os-vif and novaclient if i rememebr correctly | |
| 16:27:06 | bauzas | ok, then I'll +1 it | |
| 16:27:09 | sean-k-mooney | https://review.opendev.org/c/openstack/releases/+/864536 | |
| 16:28:00 | bauzas | sean-k-mooney: that's a duplicate of https://review.opendev.org/c/openstack/releases/+/861360 | |
| 16:28:01 | sean-k-mooney | both have the swap to the new testing runtime merged and not much else so i think we can release that too | |
| 16:28:40 | sean-k-mooney | yes and no | |
| 16:28:49 | sean-k-mooney | the release was being doen for different reasons | |
| 16:29:15 | sean-k-mooney | but sure we can go with the takashi's patch | |
| 16:29:16 | bauzas | surely, but the result is the same even if the gerrit topics are different | |
| 16:34:22 | opendevreview | Ghanshyam proposed openstack/placement master: Policy defaults improvement spec https://review.opendev.org/c/openstack/placement/+/864385 | |
| 16:34:56 | gmann | bauzas: sean-k-mooney dansmith ^^ updated placement spec also for RBAC to add admin-or-service default what we discussed on IRC | |
| 16:35:07 | dansmith | ack | |
| 16:39:43 | bauzas | ack | |
| 16:48:38 | opendevreview | Merged openstack/nova-specs master: Policy service role spec https://review.opendev.org/c/openstack/nova-specs/+/864379 | |
| 17:27:47 | bauzas | sean-k-mooney: gibi: thanks for having commented out the CPU spec | |
| 17:27:51 | bauzas | will update it tomorrow | |
| 17:28:17 | sean-k-mooney | cool ill take a look then | |
| 17:28:19 | gibi | bauzas: no worries. feel free to ping me later and I cen re-review | |
| 17:51:13 | opendevreview | Ghanshyam proposed openstack/placement master: Policy defaults improvement spec https://review.opendev.org/c/openstack/placement/+/864385 | |
| 17:52:30 | gmann | sean-k-mooney: ^^ updated the upgrade impact and scope_type change. | |
| 18:37:05 | opendevreview | Merged openstack/nova-specs master: Re-propose spec for ephemeral storage encryption https://review.opendev.org/c/openstack/nova-specs/+/864138 | |
| 18:38:29 | opendevreview | Merged openstack/nova-specs master: Re-propose spec for ephemeral encryption for libvirt https://review.opendev.org/c/openstack/nova-specs/+/864147 | |
| 18:42:40 | opendevreview | Dan Smith proposed openstack/nova-specs master: Add stable-compute-uuid spec https://review.opendev.org/c/openstack/nova-specs/+/863152 | |
| 23:42:24 | opendevreview | Ghanshyam proposed openstack/nova master: Add service role in nova policy https://review.opendev.org/c/openstack/nova/+/864594 | |
| #openstack-nova - 2022-11-16 | |||
| 02:34:05 | opendevreview | Nobuhiro MIKI proposed openstack/nova master: libvirt: fix typo in test_config https://review.opendev.org/c/openstack/nova/+/864641 | |
| 03:44:26 | opendevreview | Ghanshyam proposed openstack/nova master: Add service role in nova policy https://review.opendev.org/c/openstack/nova/+/864594 | |
| 04:49:37 | opendevreview | Ghanshyam proposed openstack/nova master: api-ref update: Move the service APIs in a separate section https://review.opendev.org/c/openstack/nova/+/864655 | |
| 05:38:39 | opendevreview | Ghanshyam proposed openstack/nova master: api-ref update: Move the service APIs in a separate section https://review.opendev.org/c/openstack/nova/+/864655 | |
| 05:44:34 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: Refactor volume connection cleanup out of _post_live_migration https://review.opendev.org/c/openstack/nova/+/864670 | |
| 05:44:35 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: Move pre-3.44 Cinder post live migration test to test_compute_mgr https://review.opendev.org/c/openstack/nova/+/864671 | |
| 05:44:36 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: functional: Change order of two classes https://review.opendev.org/c/openstack/nova/+/864672 | |
| 05:51:41 | opendevreview | Ghanshyam proposed openstack/nova master: DNM: test new defaults https://review.opendev.org/c/openstack/nova/+/864673 | |
| 06:10:48 | opendevreview | Younghwan Yoo proposed openstack/nova master: Update a return value of mdev_name2uuid function to be a valid UUID value https://review.opendev.org/c/openstack/nova/+/864674 | |
| 06:13:48 | han-guangyu | Hello, I want to package ovn in openEuler(a OS had been supported by openstack ci)) | |
| 06:14:13 | han-guangyu | Could I can get a recommended version of ovn to Train and Wallaby | |
| 06:14:37 | han-guangyu | I see: | |
| 06:14:49 | han-guangyu | > OpenStack doesn’t set explicit version requirements for OVN installation, but it’s recommended to follow at least the version that is used in upstream CI, in https://docs.openstack.org/neutron/latest/ovn/faq/index.html | |
| 06:16:06 | han-guangyu | But I am not sure, if I use the latest version of ovn, whether the neutron of train and wallaby both can be connected normally | |
| 06:16:22 | han-guangyu | Do they have a capped version that supports | |
| 06:17:27 | opendevreview | Younghwan Yoo proposed openstack/nova master: Update mdev_name2uuid function return value to be a valid UUID https://review.opendev.org/c/openstack/nova/+/864674 | |
| 06:21:13 | opendevreview | Younghwan Yoo proposed openstack/nova master: Update return value to be a valid UUID https://review.opendev.org/c/openstack/nova/+/864674 | |
| 06:21:22 | han-guangyu | oh | |
| 06:21:34 | han-guangyu | I send to a incorret channel | |
| 06:21:42 | han-guangyu | I want to neutron , sorry | |
| 06:37:31 | opendevreview | Younghwan Yoo proposed openstack/nova master: Update return value to be a valid UUID https://review.opendev.org/c/openstack/nova/+/864674 | |
| 06:42:59 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: functional: Change order of two classes https://review.opendev.org/c/openstack/nova/+/864672 | |
| 07:56:32 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: Add boot from volume functional test with a huge request https://review.opendev.org/c/openstack/nova/+/864679 | |
| 09:02:29 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova-specs master: spec: allowing target state for evacuate https://review.opendev.org/c/openstack/nova-specs/+/857838 | |
| 09:03:15 | sahid | gibi, sean-k-mooney o/ if you can have a second look at ^ there is one point that I don't get | |
| 09:05:09 | gibi | sahid: looking... | |
| 09:08:47 | sean-k-mooney[m] | sahid gibi is just asking that you bump the compute service version and add a min compute service version check in the api | |
| 09:09:13 | sean-k-mooney[m] | so if you upgrade the api and conductors but not the computes | |
| 09:09:26 | sean-k-mooney[m] | we would still reject the new microversion | |
| 09:09:36 | sean-k-mooney[m] | until you completed updating all the computes | |
| 09:10:48 | gibi | yepp | |
| 09:10:52 | gibi | I left a reply in the review too | |
| 09:11:01 | gibi | but it is what sean-k-mooney[m] described above | |
| 09:11:41 | sean-k-mooney[m] | we do this with a decorator on the api method | |
| 09:11:43 | sean-k-mooney[m] | https://github.com/openstack/nova/blob/bcdf5988f6ae902dba9b41144a7b4a60688b627c/nova/compute/api.py#L283 | |
| 09:11:49 | sean-k-mooney[m] | that is an example for vdpa | |
| 09:12:58 | sean-k-mooney[m] | you will need to reject the new microverion untill the requied min service version is reached | |
| 09:15:31 | sean-k-mooney[m] | you might want to do this eailer then that actully | |
| 09:15:41 | sean-k-mooney[m] | i dont think we have the microverion at that point | |
| 09:16:39 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: functional: Unify '_wait_until_deleted' implementations https://review.opendev.org/c/openstack/nova/+/864712 | |
| 09:16:40 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: functional: Unify '_build_minimal_create_server_request' implementations https://review.opendev.org/c/openstack/nova/+/864713 | |
| 09:16:49 | sean-k-mooney[m] | so it would be better to do it here https://github.com/openstack/nova/blob/bcdf5988f6ae902dba9b41144a7b4a60688b627c/nova/api/openstack/compute/evacuate.py instead | |
| 09:17:29 | sahid | gibi, sean-k-mooney[m] Ok I think I have your point | |
| 09:20:52 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova-specs master: spec: allowing target state for evacuate https://review.opendev.org/c/openstack/nova-specs/+/857838 | |
| 09:21:02 | sahid | fixed ^ | |
| 09:42:54 | zigo | Hi there! I have a compute node with 3 VMs that look like having an affinity server group, so I can't live-evacuate the node (for kernel upgrade). How do I force the affinity to become a soft-affinity in the nova scheduler ? | |
| 09:43:16 | zigo | I think that's possible, right? | |
| 09:43:31 | kashyap | Good question, I don't even know top off my head. Perhaps bauzas or gibi | |
| 09:43:44 | gibi | zigo: you cannot change the policy of the group | |
| 09:44:06 | zigo | gibi: What's the way to evacuate my node then? | |
| 09:44:35 | gibi | zigo: if you want to live migrate the VM then you can use old microversion to specify the target host with force=true | |
| 09:44:52 | gibi | then the scheduler will not do any checks | |
| 09:44:58 | zigo | Oh, thanks! :) | |
| 09:45:09 | gibi | force available until 2.67 | |
| 09:46:14 | zigo | I'm on Victoria on that cluster, so that's 2.87... :) | |
| 09:48:55 | zigo | openstack server migrate: error: unrecognized arguments: --force | |
| 09:50:18 | zigo | "The Force" isn't with me ... :/ | |
| 09:51:58 | gibi | hm , I don't see --force in openstack client | |
| 09:53:34 | zigo | Right, so how do I do force=true ? | |
| 09:55:28 | gibi | if you use microversion < 2.30 the --host will also mean force as well. but please note that nova will not check the target host if it is valid or not | |
| 09:56:04 | gibi | also if you have an instance with nested allocation like bandwith, vgpu then forcing a host will not work properly, you will probably use the nested allocations | |
| 09:56:25 | gibi | s/use/lose/ | |
| 09:57:33 | gibi | if you want to have safe live migration then you need to disable the server group filter in the scheduler temporarily and do the migration without --host and then restore the filter | |
| 09:58:00 | zigo | Ah, ok. | |
| 09:58:30 | gibi | we lack the capability to change a policy on a server group so affinity is very sticky | |
| 10:01:33 | gibi | (with a small DB hack you can change the affinity to soft-affinity on the group :D) | |
| 10:17:03 | opendevreview | Amit Uniyal proposed openstack/nova stable/train: functional: Rework '_delete_server' https://review.opendev.org/c/openstack/nova/+/864721 | |
| 11:00:43 | zigo | gibi: That's what I ended-up doing, yes ... | |
| 11:00:54 | zigo | I just had a few "Error monitoring migration: internal error: client socket is closed: libvirt.libvirtError: internal error: client socket is closed" | |