| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2021-03-26 | |||
| 03:06:39 | openstackgerrit | Merged openstack/nova stable/stein: Use subqueryload() instead of joinedload() for (system_)metadata https://review.opendev.org/c/openstack/nova/+/761812 | |
| 03:12:34 | openstackgerrit | sean mooney proposed openstack/nova master: libvirt: Delegate OVS plug to os-vif https://review.opendev.org/c/openstack/nova/+/602432 | |
| 03:16:19 | eandersson | melwitt do you know what is going on at the rocky ci? | |
| 03:16:33 | eandersson | > Cherry pick hash 9e9c022bde3a3ffdf0dd87e21bf9afde0dbc1e74 not on any master or stable branches | |
| 03:17:26 | sean-k-mooney | thats form the cherry pick script we have to ensure we dont skip branches wehn we backport | |
| 03:17:37 | sean-k-mooney | its run as part of the pep8 job | |
| 03:18:25 | sean-k-mooney | eandersson: this has not merge in trian yet so all the previous version will fail | |
| 03:18:53 | sean-k-mooney | that has been cherry picked incorrectly | |
| 03:19:06 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/356020 is merged | |
| 03:19:23 | sean-k-mooney | https://review.opendev.org/c/openstack/nova/+/782982 is pending | |
| 03:19:48 | sean-k-mooney | but we shoudl have the cherry picked form lines for each branch | |
| 03:20:15 | sean-k-mooney | well | |
| 03:20:24 | sean-k-mooney | its bit that its incorrect persay | |
| 03:20:37 | sean-k-mooney | but untill https://review.opendev.org/c/openstack/nova/+/782982 merges it cant pass pep8 | |
| 03:21:01 | sean-k-mooney | eandersson: https://github.com/openstack/nova/blob/master/tools/check-cherry-picks.sh | |
| 03:21:08 | sean-k-mooney | that is what is causing it to fail | |
| 03:22:11 | sean-k-mooney | eandersson: im not sure why the first cherry pick start a train | |
| 03:23:01 | sean-k-mooney | i guess this merged a year ago so maybe train was correct | |
| 03:23:15 | sean-k-mooney | assumign it merged in ussuri | |
| 03:25:08 | sean-k-mooney | eandersson: after the train and stien version merge a recheck will allow it to pass | |
| 03:25:19 | sean-k-mooney | eandersson: so dont need to worry about it for now | |
| 03:37:56 | eandersson | Sounds good | |
| 03:38:14 | eandersson | It's just weird that it was under pep8 and the message for sure wasn't clear :D | |
| 03:38:36 | eandersson | Would never have figured that out. | |
| 03:38:39 | sean-k-mooney | pep8 tradtionally was used for all linting jobs | |
| 03:38:44 | eandersson | At least not without digging into the code. | |
| 03:38:57 | sean-k-mooney | some non python porject starte using a tox linters env | |
| 03:39:24 | sean-k-mooney | but nova and the rest of the old "core" project all had lintere in pep8 already so we just extend it | |
| 03:39:39 | eandersson | Is this the one you looked at btw? https://review.opendev.org/c/openstack/nova/+/761813 | |
| 03:40:00 | eandersson | I re-checked it after the Stein backport was patched. | |
| 03:40:07 | eandersson | but will try again just in case | |
| 03:40:25 | sean-k-mooney | no | |
| 03:40:33 | sean-k-mooney | i serched for 9e9c022bde3a3ffdf0dd87e21bf9afde0dbc1e74 | |
| 03:40:58 | eandersson | I knew it was a mistake to include the example id :D | |
| 03:40:58 | sean-k-mooney | whcih gave me https://review.opendev.org/q/9e9c022bde3a3ffdf0dd87e21bf9afde0dbc1e74 | |
| 03:41:08 | eandersson | I was considering replacing 9e9c with X or something | |
| 03:41:37 | eandersson | Anyway lets wait for the re-check and see I guess | |
| 03:42:39 | sean-k-mooney | same think there though 66a4ea9f25cdc75691ed55adc842b795c10b4191 was not merged when it ran | |
| 03:42:56 | eandersson | Yea it's very likely I got trigger happy there. | |
| 03:43:35 | sean-k-mooney | its merge now though https://review.opendev.org/c/openstack/nova/+/761812 so the recheck should pass | |
| 03:45:18 | sean-k-mooney | hum hopfully the grenade job will pass with the recheck too | |
| 03:50:32 | sean-k-mooney | looks like a filure in the post test hook https://zuul.opendev.org/t/openstack/build/23129f86fb5146a084bb04f3eb9fcfec/log/logs/devstack-gate-post_test_hook.txt#95 | |
| 03:50:40 | sean-k-mooney | when testing ceph | |
| 05:03:56 | openstackgerrit | Merged openstack/nova master: Fix check_instance_shared_storage() call https://review.opendev.org/c/openstack/nova/+/783026 | |
| 06:04:42 | openstackgerrit | Josephine Seifert proposed openstack/nova stable/stein: Add config parameter 'live_migration_scheme' to live migration with tls guide https://review.opendev.org/c/openstack/nova/+/783199 | |
| 06:11:44 | gibi | dansmith: thanks for the night whatch and the rechecks. The fix is merged and the RC1 patch is updated now. | |
| 06:11:55 | gibi | bauzas: ^^ | |
| 07:16:52 | openstackgerrit | OpenStack Release Bot proposed openstack/nova stable/wallaby: Update .gitreview for stable/wallaby https://review.opendev.org/c/openstack/nova/+/783195 | |
| 07:17:22 | openstackgerrit | OpenStack Release Bot proposed openstack/nova stable/wallaby: Update TOX_CONSTRAINTS_FILE for stable/wallaby https://review.opendev.org/c/openstack/nova/+/783216 | |
| 07:17:41 | openstackgerrit | OpenStack Release Bot proposed openstack/nova master: Update master for stable/wallaby https://review.opendev.org/c/openstack/nova/+/783217 | |
| 07:18:38 | openstackgerrit | OpenStack Release Bot proposed openstack/nova master: Add Python3 xena unit tests https://review.opendev.org/c/openstack/nova/+/783218 | |
| 08:24:55 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/wallaby: [stable-only]Update .gitreview for stable/wallaby https://review.opendev.org/c/openstack/nova/+/783195 | |
| 08:26:23 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/wallaby: Update TOX_CONSTRAINTS_FILE for stable/wallaby https://review.opendev.org/c/openstack/nova/+/783216 | |
| 08:27:38 | openstackgerrit | Balazs Gibizer proposed openstack/nova stable/wallaby: [stable-only]Update TOX_CONSTRAINTS_FILE for stable/wallaby https://review.opendev.org/c/openstack/nova/+/783216 | |
| 08:32:18 | gibi | sean-k-mooney: could you please hit this two automatic os-vif patches https://review.opendev.org/c/openstack/os-vif/+/781891 ? | |
| 08:38:34 | bauzas | morning | |
| 08:38:50 | bauzas | gibi: sorry, looks like yesterday Frenode had some split | |
| 08:38:57 | bauzas | and maybe you pinged me | |
| 08:39:24 | bauzas | thanks for the RC1 patch | |
| 08:54:00 | gibi | bauzas: no worries everything went well during the night :) | |
| 08:54:56 | bauzas | gibi: I'll need to be off this afternoon btw. | |
| 08:55:12 | gibi | ack | |
| 08:55:17 | gibi | nothing pressing is in the queue | |
| 08:58:42 | openstackgerrit | Merged openstack/osc-placement master: Update master for stable/victoria https://review.opendev.org/c/openstack/osc-placement/+/751320 | |
| 09:00:49 | swp20 | gibi: morning, i find disk_filter is removed from U version, could u please give me some ref links about the reason? | |
| 09:01:10 | stephenfin | swp20: It'll be in the release notes | |
| 09:01:35 | stephenfin | swp20: https://docs.openstack.org/releasenotes/nova/train.html | |
| 09:01:44 | stephenfin | The CoreFilter, DiskFilter and RamFilter, which were deprecated in Stein (19.0.0), are now removed. VCPU, DISK_GB and MEMORY_MB filtering is performed natively using the Placement service. | |
| 09:02:16 | swp20 | stephenfin: thanks a lot. | |
| 09:07:16 | openstackgerrit | Merged openstack/osc-placement master: Add Python3 wallaby unit tests https://review.opendev.org/c/openstack/osc-placement/+/751321 | |
| 09:12:39 | swp20 | stephenfin: do we plan to move the remain filters to placement? | |
| 09:19:01 | bauzas | swp20: no, that depends | |
| 09:19:36 | bauzas | swp20: some filters can be deprecated if we can create what we name 'prefilters' using the Placement API | |
| 09:19:46 | bauzas | swp20: but some others, no | |
| 09:22:17 | swp20 | bauzas: got it, thanks. | |
| 09:25:00 | swp20 | bauzas: placemet has support PCI filter such as GPU/FPGA devices, isn't it? | |
| 09:27:09 | gibi | vgpu mdevs are modelled in placement but PCI devices not yet | |
| 09:27:49 | gibi | due to qos min bandwidth scheduling support some sriov PFs are modelled in placement but that only used for the bandwdith resource at the moment | |
| 09:28:00 | gibi | the PCI schduling still happens in the PCIFilter in nova | |
| 09:30:59 | bauzas | swp20: we only model vGPUs | |
| 09:31:11 | bauzas | swp20: but we don't really model PCI devices | |
| 09:31:36 | bauzas | and given we also have NUMA PCI affinity, we would also need to support this | |
| 09:31:45 | bauzas | a long story here. | |
| 09:33:49 | swp20 | gibi, buazas: got, thanks a lot. | |
| 09:35:41 | openstackgerrit | OpenStack Release Bot proposed openstack/placement stable/wallaby: Update .gitreview for stable/wallaby https://review.opendev.org/c/openstack/placement/+/783232 | |
| 09:35:45 | openstackgerrit | OpenStack Release Bot proposed openstack/placement stable/wallaby: Update TOX_CONSTRAINTS_FILE for stable/wallaby https://review.opendev.org/c/openstack/placement/+/783233 | |
| 09:35:50 | openstackgerrit | OpenStack Release Bot proposed openstack/placement master: Update master for stable/wallaby https://review.opendev.org/c/openstack/placement/+/783234 | |
| 09:36:13 | openstackgerrit | OpenStack Release Bot proposed openstack/placement master: Add Python3 xena unit tests https://review.opendev.org/c/openstack/placement/+/783235 | |
| 11:25:26 | kashyap | gibi: or anyone else: Can I borrow your eyes: In this method here _add_rng_device() — do you see a way where 'hw_rng:allowed = False' on image property might *not* take effect? -- https://github.com/openstack/nova/blob/master/nova/virt/libvirt/driver.py#L5665,#L5682 | |
| 11:31:16 | openstackgerrit | Slawek Kaplonski proposed openstack/nova master: [neutron] Get only ID and name of the SGs from Neutron https://review.opendev.org/c/openstack/nova/+/783275 | |
| 12:24:17 | gibi | kashyap: the image_meta is unused in that function | |
| 12:24:41 | gibi | only the flavor extra_spec is used | |
| 12:24:53 | gibi | so that function does not care about any image property | |
| 12:25:23 | kashyap | gibi: Oh, so the "hw_rng:allowed" image meta is useless in this case then | |
| 12:28:50 | gibi | seem so | |
| 12:28:55 | gibi | it might be a bug | |
| 12:29:16 | kashyap | Hmm, OSP QE reported it to me... | |
| 12:29:47 | gibi | as the function gets an image_meta parameter but the it does not use it | |
| 12:31:11 | kashyap | gibi: Right; I guess we should: | |
| 12:31:19 | kashyap | flavor.extra_specs.get() or image_imeta.properties.get() ... | |