| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-06-25 | |||
| 15:16:38 | gibi | I'm affraid there will be extreme copy past now to have all the server create scenarios and then all the server move scenarios covered | |
| 15:16:48 | gibi | ~1000 LOC | |
| 15:17:20 | gibi | anyhow you are right If I cannot predic which direction I should refactor then probably I need to duplicate first | |
| 15:17:29 | opendevreview | Merged openstack/nova master: tests: Allow bindep and test-setup.sh to run on EL distros https://review.opendev.org/c/openstack/nova/+/796428 | |
| 15:40:36 | opendevreview | Merged openstack/nova master: zuul: Add nova-tox-functional-centos8-py36 job https://review.opendev.org/c/openstack/nova/+/796684 | |
| 16:19:05 | opendevreview | Elod Illes proposed openstack/nova stable/stein: compute: Validate a BDMs disk_bus when provided https://review.opendev.org/c/openstack/nova/+/744554 | |
| 16:24:05 | gibi | stephenfin: can we release https://review.opendev.org/c/openstack/oslo.db/+/797426 so that nova can benefit from the silence deprecation warning? | |
| 16:24:16 | stephenfin | gibi: sure | |
| 16:25:20 | stephenfin | I'll ping hberaud | |
| 16:25:27 | stephenfin | in a call atm | |
| 16:25:31 | gibi | cool, thanks | |
| 16:37:13 | opendevreview | Elod Illes proposed openstack/nova stable/train: Use absolute path during qemu img rebase https://review.opendev.org/c/openstack/nova/+/757084 | |
| 16:41:09 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] create pps resource on OVS agent RP https://review.opendev.org/c/openstack/nova/+/787205 | |
| 16:42:26 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] move port creation to the NeutronFixture https://review.opendev.org/c/openstack/nova/+/787206 | |
| 16:43:42 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] refactor assertPortMatchesAllocation https://review.opendev.org/c/openstack/nova/+/792458 | |
| 16:45:06 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] neutron fixture for extended resource request https://review.opendev.org/c/openstack/nova/+/794306 | |
| 16:46:24 | opendevreview | Balazs Gibizer proposed openstack/nova master: Detect extended_resource_request neutron API extension https://review.opendev.org/c/openstack/nova/+/793618 | |
| 16:47:45 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject server create with extended resource req https://review.opendev.org/c/openstack/nova/+/793619 | |
| 16:49:15 | opendevreview | Balazs Gibizer proposed openstack/nova master: Reject server operations with extended resource req https://review.opendev.org/c/openstack/nova/+/793620 | |
| 16:50:37 | opendevreview | Balazs Gibizer proposed openstack/nova master: Add same_subtree field to RequestLevelParams https://review.opendev.org/c/openstack/nova/+/791503 | |
| 16:51:56 | opendevreview | Balazs Gibizer proposed openstack/nova master: Bump min placement microversion to 1.36 https://review.opendev.org/c/openstack/nova/+/791504 | |
| 16:53:21 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support same_subtree in allocation_canadidate query https://review.opendev.org/c/openstack/nova/+/791505 | |
| 16:54:43 | opendevreview | Balazs Gibizer proposed openstack/nova master: Support the new port resource_request format https://review.opendev.org/c/openstack/nova/+/787208 | |
| 16:56:22 | opendevreview | Balazs Gibizer proposed openstack/nova master: Transfer RequestLevelParams from ports to scheduling https://review.opendev.org/c/openstack/nova/+/791506 | |
| 16:57:49 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] ports with both bw and pps resources https://review.opendev.org/c/openstack/nova/+/792394 | |
| 16:59:22 | opendevreview | Balazs Gibizer proposed openstack/nova master: [func test] move unshelve test to the proper place https://review.opendev.org/c/openstack/nova/+/793621 | |
| 17:10:19 | opendevreview | Rodrigo Barbieri proposed openstack/nova stable/ussuri: Error anti-affinity violation on migrations https://review.opendev.org/c/openstack/nova/+/796719 | |
| 17:51:10 | opendevreview | Elod Illes proposed openstack/nova stable/victoria: Stop leaking ceph df cmd in RBD utils https://review.opendev.org/c/openstack/nova/+/791938 | |
| 18:02:46 | opendevreview | Stephen Finucane proposed openstack/nova master: WIP: api: Validate host belongs to availability zone https://review.opendev.org/c/openstack/nova/+/798145 | |
| 18:03:25 | stephenfin | gibi: bauzas: Would appreciate input on https://review.opendev.org/c/openstack/nova/+/798145 on Monday, just so I know I'm not wasting my time on that solution. I'll write up a bug if there isn't one already available | |
| 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 | |