| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-06-25 | |||
| 15:04:31 | sean-k-mooney | but did not see really what it was doiing | |
| 15:04:52 | sean-k-mooney | when you said neutron cleint code do you mean realy code or fixtures | |
| 15:05:10 | sean-k-mooney | i assume /nova/network/neutron.py | |
| 15:05:14 | gibi | that one ^^ | |
| 15:05:43 | sean-k-mooney | ok if it was just the fixture i woudl have jsut said make two | |
| 15:06:14 | gibi | I have multiple fixtures to support old and new resource request format | |
| 15:06:21 | gibi | that is a relatively small duplication | |
| 15:06:32 | gibi | https://review.opendev.org/c/openstack/nova/+/794306/2 | |
| 15:06:59 | gibi | and here https://review.opendev.org/c/openstack/nova/+/792394/9/nova/tests/fixtures/neutron.py | |
| 15:07:51 | gibi | the test logic, especially the the asserts needs to be changed as a) there will be more allocations b) the bindig profile allocation key changes format | |
| 15:09:01 | gibi | I could pull out all the asserts to functions and override the functions in the child test class according to the new needs | |
| 15:10:09 | gibi | it just probably needs a lot of refactoring, and a way to foresee how all assert will look like in the with the new resource request format in place | |
| 15:10:29 | gibi | to know what direction to refactor to | |
| 15:11:25 | sean-k-mooney | it might be simpler to just duplicate things | |
| 15:11:37 | sean-k-mooney | and if you have time later see if the commonality can be abstrated | |
| 15:12:03 | gibi | yeah that is one way to go | |
| 15:12:12 | sean-k-mooney | rather then trying to do both up front | |
| 15:12:45 | gibi | if i copy paste, then I need some reminder not to forget to abstract things out later | |
| 15:12:51 | gibi | as I will be lazy | |
| 15:13:14 | gibi | so not a reminder like a # TODO, but a reminder like a pain in the ass | |
| 15:13:58 | sean-k-mooney | have a test that emails you every time it runs :P | |
| 15:14:28 | gibi | :D | |
| 15:14:50 | sean-k-mooney | duplication when its not extream i think is ok | |
| 15:16:10 | gibi | yeah, in test I allow duplication if in return it helps seeing the test scenario cleanly | |
| 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 - cleanup arqs https://review.opendev.org/c/openstack/nova/+/798054 | |
| 06:06:42 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - create arqs https://review.opendev.org/c/openstack/nova/+/758944 | |
| 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 - build instance with smartnic arqs https://review.opendev.org/c/openstack/nova/+/798249 | |
| 06:06:45 | opendevreview | Yongli He proposed openstack/nova master: smartnic support - functional tests https://review.opendev.org/c/openstack/nova/+/780147 | |
| 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 | qemuDomainObjBeginJobInternal:4945 : Timed out during operation: cannot acquire state change lock (held by remoteDispatchDomainMigratePrepareTunnel3Params)" | |
| 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 : | |
| 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 | |