| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2022-11-15 | |||
| 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" | |
| 11:01:00 | zigo | What does this mean, and how to avoid it? | |
| 11:01:55 | sean-k-mooney | that sound like the qemu instance crashed or at least teh qemu monitor connection | |
| 11:15:04 | zigo | ok... | |
| 12:17:38 | opendevreview | Kirill proposed openstack/nova-specs master: new spec: support of vnc console for ironic https://review.opendev.org/c/openstack/nova-specs/+/863773 | |
| 12:31:39 | opendevreview | Sahid Orentino Ferdjaoui proposed openstack/nova-specs master: spec: allowing target state for evacuate https://review.opendev.org/c/openstack/nova-specs/+/857838 | |
| 12:40:01 | opendevreview | Kirill proposed openstack/nova-specs master: new spec: support of vnc console for ironic https://review.opendev.org/c/openstack/nova-specs/+/863773 | |
| 12:42:42 | Kirill_ | hope i covered all wishes) | |
| 12:57:42 | sean-k-mooney | Kirill_: one benifit of looking up the password wehn you connec tby the way is you can actully rotate teh vnc passworkd on the ironic side if that is required by your security policy | |
| 12:58:00 | sean-k-mooney | iw we saved it in the nova db you could never change the vnc password on the ironic host | |