Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2021-03-25
19:53:59 gibi dansmith: ack, then my guess way right about the upgrade
20:40:08 openstackgerrit sean mooney proposed openstack/nova master: Test numa and vcpu topologies bug: #1910466 https://review.opendev.org/c/openstack/nova/+/769601
20:40:09 openstack bug 1910466 in OpenStack Compute (nova) "NUMA instance spawn fails on get_best_cpu_topology when there is no 'threads' preference" [Medium,In progress] https://launchpad.net/bugs/1910466 - Assigned to sean mooney (sean-k-mooney)
20:40:10 openstackgerrit sean mooney proposed openstack/nova master: Fix max cpu topologies with numa affinity https://review.opendev.org/c/openstack/nova/+/769614
21:31:30 gibi o/
21:31:30 gibi dansmith: the 6.0 fix is failed on the gate on unrealted error. But zuul haven't voted yet. I left a comment in the RC1 release patch and delegated power to you to update / +1 it if the fix lands while I'm asleep.
21:32:30 dansmith gibi: okay, will keep an eye on it
22:39:51 openstackgerrit Dan Smith proposed openstack/nova master: Fix check_instance_shared_storage() call https://review.opendev.org/c/openstack/nova/+/783026
#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

Earlier   Later