| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-06-25 | |||
| 19:30:32 | opendevreview | Merged openstack/nova stable/train: Add a workaround config toggle to refuse ceph image upload https://review.opendev.org/c/openstack/nova/+/757177 | |
| 23:11:15 | opendevreview | Merged openstack/nova stable/train: Use absolute path during qemu img rebase https://review.opendev.org/c/openstack/nova/+/757084 | |
| #openstack-nova - 2021-06-26 | |||
| 01:47:42 | opendevreview | Jorhson Deng proposed openstack/nova master: process instance in soft-deleting task state during _init_instance https://review.opendev.org/c/openstack/nova/+/761264 | |
| 09:58:17 | opendevreview | Eric Xie proposed openstack/nova master: Add logs when cannot fit numa https://review.opendev.org/c/openstack/nova/+/798187 | |
| 10:08:38 | opendevreview | Jorhson Deng proposed openstack/nova master: recheck the attachment_id after the reschedule successful https://review.opendev.org/c/openstack/nova/+/796209 | |
| #openstack-nova - 2021-06-27 | |||
| 03:04:07 | opendevreview | sean mooney proposed openstack/os-vif master: [WIP] add configurable per port bridges https://review.opendev.org/c/openstack/os-vif/+/798055 | |
| 07:27:07 | opendevreview | Merged openstack/nova stable/wallaby: libvirt: Set driver_iommu when attaching virtio devices to SEV instance https://review.opendev.org/c/openstack/nova/+/796607 | |
| 11:07:40 | opendevreview | Eric Xie proposed openstack/nova master: Add logs when cannot fit numa https://review.opendev.org/c/openstack/nova/+/798187 | |
| 13:18:03 | gibi | stephenfin: left comments in https://review.opendev.org/c/openstack/nova/+/798145 . I agree with teh generic idea to validate the input | |
| #openstack-nova - 2021-06-28 | |||
| 05:44:40 | opendevreview | Slawek Kaplonski proposed openstack/nova stable/ussuri: [neutron] Get only ID and name of the SGs from Neutron https://review.opendev.org/c/openstack/nova/+/787253 | |
| 06:06:38 | opendevreview | Yongli He proposed openstack/nova master: Smartnic support - cyborg drive https://review.opendev.org/c/openstack/nova/+/771362 | |
| 06:06:39 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - new vnic type https://review.opendev.org/c/openstack/nova/+/771363 | |
| 06:06:42 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - create arqs https://review.opendev.org/c/openstack/nova/+/758944 | |
| 06:06:42 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - cleanup arqs https://review.opendev.org/c/openstack/nova/+/798054 | |
| 06:06:43 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - reject server move and suspend https://review.opendev.org/c/openstack/nova/+/779913 | |
| 06:06:45 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - functional tests https://review.opendev.org/c/openstack/nova/+/780147 | |
| 06:06:45 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - build instance with smartnic arqs https://review.opendev.org/c/openstack/nova/+/798249 | |
| 07:39:54 | gibi | sean-k-mooney[m]: do you still hold your -1 on https://review.opendev.org/c/openstack/nova/+/797142 ? the follow up is green | |
| 07:49:34 | gibi | lyarwood: I have a comment in https://review.opendev.org/c/openstack/nova/+/779275 about the assumption that size is always provided to create_image | |
| 08:26:57 | MrClayPole | Morning all, We currently have an OpenStack ansible rocky deployment running on Ubuntu 18.04. We've been having failures during live migrations. We are seeing the following error in the journal logs but are not having much luck trying to trace it "error : qemuDomainObjBeginJobInternal:4945 : Timed out during operation: cannot acquire state change lock (held by remoteDispatchDomainFSFreeze)" & "error : | |
| 08:26:57 | MrClayPole | qemuDomainObjBeginJobInternal:4945 : Timed out during operation: cannot acquire state change lock (held by remoteDispatchDomainMigratePrepareTunnel3Params)" | |
| 09:01:49 | lyarwood | gibi: ack, I think this is because ramdisk and kernel files are always RAW but let me grep around again and confirm | |
| 09:02:03 | lyarwood | was anyone working on the VIR_CONNECT_LIST_NODE_DEVICES_CAP_VDPA libvirt regression btw? | |
| 09:02:13 | lyarwood | https://zuul.opendev.org/t/openstack/build/68e59744ef7444a5ae108118983c9353/log/controller/logs/screen-n-cpu.txt#1525 - the new centos job is hitting it | |
| 09:03:19 | lyarwood | https://bugs.launchpad.net/nova/+bug/1933096 ah ha | |
| 09:06:05 | stephenfin | In the docs on AZs, we have this sentence "A host can be part of multiple aggregates but it can only be in one availability zone". Anyone know off the top of their heads what enforces this? | |
| 09:06:09 | stephenfin | from https://docs.openstack.org/nova/latest/admin/availability-zones.html | |
| 09:06:20 | stephenfin | I wrote that, but I think I copy-pasted it from elsewhere | |
| 09:20:29 | stephenfin | ah, found it. 'is_safe_to_update_az' in nova/compute/api.py | |
| 09:24:05 | gibi | lyarwood: Sean looked in https://bugs.launchpad.net/nova/+bug/1933096 before | |
| 09:24:33 | lyarwood | yup and closed it invalid as it was third party CI, this time it's our own upstream CI | |
| 09:24:58 | lyarwood | was going to ask them how we can proceed here, no idea how we cache packages on CI nodes tbh | |
| 09:25:01 | gibi | lyarwood: I guess we also has a bad cache somewher then | |
| 09:25:18 | lyarwood | yeah likely | |
| 09:31:16 | opendevreview | Jorhson Deng proposed openstack/nova master: recheck the attachment_id after the reschedule successful https://review.opendev.org/c/openstack/nova/+/796209 | |
| 11:37:04 | sean-k-mooney[m] | lyarwood: so im still not conviced that that is a valid bug | |
| 11:37:37 | sean-k-mooney[m] | or rather we could adress it but only by nolonger relying on any libvirt version checks in our code | |
| 11:38:11 | sean-k-mooney[m] | libvirt-python is not really intended to be installed as a wheel | |
| 11:38:57 | sean-k-mooney[m] | its intended to generate bindings when its installed for your current libvirt version which it wont do if you have prebuilt it as a wheel | |
| 11:39:22 | sean-k-mooney[m] | stephenfin: yes we enforce that a host can only be in one az | |
| 11:41:46 | sean-k-mooney[m] | lyarwood: i can add an extra guard conditon for this specific case but it would just be a wack a mole problem for any other case where we use code that is generated on install | |
| 12:48:36 | opendevreview | sean mooney proposed openstack/nova master: fix sr-iov support on Cavium ThunderX hosts. https://review.opendev.org/c/openstack/nova/+/777679 | |
| 13:00:08 | bauzas | stephenfin: when you're around, we can discuss on https://review.opendev.org/c/openstack/nova/+/798145 if you wish | |
| 13:00:45 | bauzas | tl;dr: problem is that we don't verify the AZs if you don't use the AZfilter | |
| 13:01:07 | bauzas | so we can't just look at them by the API service unless we know that the AZFilter is used | |
| 13:01:53 | sean-k-mooney | our down stream customer could avoid the issue they had if they just enabled the placemnt preilter | |
| 13:02:12 | sean-k-mooney | that would enforece the AZ existance check | |
| 13:02:20 | sean-k-mooney | but they could still select the host using the hack | |
| 13:03:03 | sean-k-mooney | bauzas: i do agree though that we should remove that in a new microversion now that we have teh new way to do it | |
| 13:03:08 | bauzas | sean-k-mooney: my thought is that we should just not using the az hack after a new microversion | |
| 13:03:18 | sean-k-mooney | yep | |
| 13:03:31 | sean-k-mooney | i was expecting that to have been done in the one that added --host | |
| 13:03:39 | bauzas | for sure, it wouldn't fix the issue of a requested AZ not good but... | |
| 13:04:14 | sean-k-mooney | i also agree with our assement tha the az in the request spec and instance are not always intended to match | |
| 13:04:34 | sean-k-mooney | classic example being request spec is none but instance has a value set | |
| 13:04:59 | sean-k-mooney | in princiapl i think that is the only ligitimat case where they should disagree | |
| 13:05:12 | sean-k-mooney | if the request spec is non None then they should agree | |
| 13:05:21 | sean-k-mooney | if they dont you forced a live migration | |
| 13:05:22 | stephenfin | bauzas: we don't currently, but I'm adding that | |
| 13:05:40 | stephenfin | and the AZFilter is no use to us if we're bypassing the scheduler by forcing a host | |
| 13:05:41 | sean-k-mooney | stephenfin: right but im not conviced you should | |
| 13:05:53 | sean-k-mooney | stephenfin: that is not how that works | |
| 13:06:03 | sean-k-mooney | we check that the az exists | |
| 13:06:04 | bauzas | stephenfin: what sean-k-mooney said | |
| 13:06:18 | stephenfin | requesting zone:host makes no sense if $host is not in $zone | |
| 13:06:18 | sean-k-mooney | and only proceed if it does when you use the az hack | |
| 13:06:35 | bauzas | stephenfin: it's an hack, we should just remove it | |
| 13:06:46 | stephenfin | we can't remove it for the older APIs | |
| 13:06:50 | bauzas | surely | |
| 13:06:51 | sean-k-mooney | stephenfin: that is something we could check potentally but im not sure the api is the right place | |
| 13:06:55 | stephenfin | so people will keep hitting this | |
| 13:07:04 | bauzas | stephenfin: it's an hack, right? | |
| 13:07:12 | sean-k-mooney | well its was a supported feature | |
| 13:07:13 | bauzas | and you need to be an operator | |
| 13:07:21 | bauzas | soooo | |
| 13:07:22 | sean-k-mooney | but yes | |
| 13:07:33 | bauzas | the az hack can't be used by an end user | |
| 13:07:42 | sean-k-mooney | yes it can | |
| 13:07:51 | bauzas | not by default | |
| 13:07:55 | sean-k-mooney | they just need to use an older microverion | |
| 13:08:12 | bauzas | the default policy is admin | |
| 13:08:20 | sean-k-mooney | bauzas: is it? | |
| 13:08:26 | bauzas | for the az hack ? yes | |
| 13:08:41 | sean-k-mooney | i tought we did not have a sepreate policy for it | |
| 13:08:42 | stephenfin | I must admit I don't understand the issue | |
| 13:08:43 | bauzas | (and fortunately) | |
| 13:08:45 | sean-k-mooney | just the az one | |
| 13:08:56 | stephenfin | why wouldn't a simple "does this host belong to this AZ" check make sense? | |
| 13:09:03 | stephenfin | it's not too expensive fwict | |
| 13:09:09 | gibi | bauzas: do you suggest to keep allowing calling --availability_zone my-az:host-not-in-my-az and succeed in old microversions? | |
| 13:09:13 | bauzas | sean-k-mooney: I'm 100% sure about the different policy | |
| 13:09:14 | stephenfin | a simple lookup in the API DB | |
| 13:09:53 | bauzas | gibi: we *could* fix this for old versions only, but then I have another concern | |
| 13:10:12 | sean-k-mooney | stephenfin: my main issue is that you are doing it in a different location to the other az check | |
| 13:10:15 | bauzas | gibi: my other concern is that I know some environments that don't use the AZfilter | |
| 13:10:19 | sean-k-mooney | stephenfin: whihc is doen in the schduler i belive | |
| 13:11:03 | bauzas | gibi: and previously, you were able to use the az hack without the AZFilter | |
| 13:11:22 | gibi | bauzas: yes, but that hack resulted in an inconsistent system | |
| 13:11:28 | gibi | as described in the bug | |
| 13:11:35 | bauzas | gibi: not if you don't use the filter | |