| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-05-04 | |||
| 14:20:37 | opendevreview | Artom Lifshitz proposed openstack/nova stable/xena: Save cell socket correctly when updating host NUMA topology https://review.opendev.org/c/openstack/nova/+/882320 | |
| 14:21:07 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: Reproduce bug 1995153 https://review.opendev.org/c/openstack/nova/+/882321 | |
| 14:21:08 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: Save cell socket correctly when updating host NUMA topology https://review.opendev.org/c/openstack/nova/+/882322 | |
| 14:42:14 | dvo-plv_ | gibi: we have finished with os-traits patch. Could you please propose the release | |
| 14:56:15 | bauzas | dvo-plv_: I can do it | |
| 14:58:50 | dvo-plv_ | great, thank you | |
| 15:01:08 | gibi | bauzas: if you have cycles right now then thanks for proposing it | |
| 15:01:25 | bauzas | just doing it now | |
| 15:02:39 | gibi | bauzas++ | |
| 15:09:30 | bauzas | Uggla: your approval is nice for https://review.opendev.org/c/openstack/releases/+/882325 | |
| 15:09:36 | bauzas | gibi: dvo-plv_: ^ | |
| 15:10:02 | Uggla | bauzas, I will have a look | |
| 15:11:44 | sean-k-mooney | dvo-plv_: when that is released plese ensure you bump the min requirement for os-traits in the nova patch that uses it | |
| 15:17:07 | bauzas | sean-k-mooney: he'll need to wait for the upper-constraints bot patch to be generated first if I'm not wrong | |
| 15:17:43 | sean-k-mooney | for it to pass ci | |
| 15:17:49 | sean-k-mooney | but they can do the bump | |
| 15:22:16 | bauzas | correct, but dvo-plv_'s main concern was that zuul wasn't happy with its patch, hence his request to super-fast-approve traits | |
| 15:22:36 | bauzas | so I'm just explaining that the release is only half of the definition of done | |
| 15:25:27 | sean-k-mooney | dvo-plv_: for what its worth this should all be resolved early next week | |
| 15:25:58 | sean-k-mooney | i would expect the release to hapeen todya or tomorrow and the uper constraits bump should land shortly after | |
| 15:27:29 | bauzas | well, there are humans behind the releases approval and the upper-constraints patch approval too, so I'd give them a few more days | |
| 15:27:45 | bauzas | but yeah, early next week seems a reasonable ETA | |
| 15:32:56 | gibi | I can cosy up to elodilles to get attention on https://review.opendev.org/c/openstack/releases/+/882325 :) | |
| 15:52:57 | opendevreview | Artom Lifshitz proposed openstack/nova stable/yoga: Reproduce bug 1995153 https://review.opendev.org/c/openstack/nova/+/882317 | |
| 15:52:58 | opendevreview | Artom Lifshitz proposed openstack/nova stable/yoga: Save cell socket correctly when updating host NUMA topology https://review.opendev.org/c/openstack/nova/+/882318 | |
| 16:01:19 | Uggla | bauzas, in case you haven't seen https://review.opendev.org/c/openstack/manila-specs/+/881934 and https://review.opendev.org/c/openstack/manila-specs/+/881894 | |
| 16:03:53 | bauzas | Uggla: ack, nice to see | |
| 16:04:17 | Uggla | bauzas, yep pretty cool. | |
| 16:04:36 | bauzas | I'll to review both specs one day | |
| 16:04:39 | Uggla | bauzas, despite I have not fully read them yet. | |
| 16:04:55 | bauzas | (saying one day because I didn't had time this week yet to do what I promised) | |
| 16:12:26 | elodilles | bauzas gibi : there is a problem with the os-traits release version bump, see my comment on the patch | |
| 16:13:49 | bauzas | elodilles: hmmm, I'm a bit torn but I trust you | |
| 16:13:57 | bauzas | elodilles: we haven't removed any support afaik | |
| 16:15:18 | elodilles | bauzas: py36 & py37 classifiers were removed from setup.cfg, which means pypi would not offer the released package to py36 and py37 systems | |
| 16:15:38 | bauzas | yup yup I understand the reasoning | |
| 16:15:41 | elodilles | note that we always bump MAJOR version in these cases | |
| 16:17:28 | elodilles | so while the code might still support py36 and/or py37, still we are stating it isn't :/ i'm not fond of dropping these classifiers either, but that went through most of the deliverables already | |
| 16:18:08 | bauzas | ok, thanks for the catch anyway | |
| 16:18:10 | bauzas | elodilles++ | |
| 16:18:34 | bauzas | elodilles: https://review.opendev.org/c/openstack/releases/+/882325 is updated | |
| 16:19:57 | elodilles | bauzas: thanks, +2'd | |
| 16:20:14 | bauzas | gracias | |
| 16:21:56 | elodilles | de nada | |
| 16:24:44 | gibi | elodilles: good catch | |
| 16:24:51 | gibi | thanks for looking at it | |
| 16:42:25 | sean-k-mooney | elodilles: python 3.7 wupport was remvoed form placment and nova quite a while ago | |
| 16:42:54 | sean-k-mooney | elodilles: i guess we missed those in os-traits | |
| 16:43:13 | sean-k-mooney | elodilles: i dont think python 36 has been supproted since wallaby | |
| 16:43:29 | sean-k-mooney | that was the last release as we kept it for centos 8 support | |
| 16:43:48 | sean-k-mooney | we can do the major bump just point out that this is long overdue | |
| 16:57:24 | elodilles | sean-k-mooney: yes the last time py3.7 listed amongst supported runtimes was victoria: https://governance.openstack.org/tc/reference/runtimes/victoria.html | |
| 16:59:05 | elodilles | sean-k-mooney: what i meant is that there is difference between we don't list as supported and not allowing it to being installed on non-supported python | |
| 17:01:05 | elodilles | if we know that there is some clear incompatibility, then it is OK to remove the classifier for that python version. (though i understand we might not realise when we introduce such changes and that might be a worse situation) | |
| 17:59:55 | dvo-plv_ | sean-k-mooney: yes, sure, guys already mentioned that I have to udpate requirments for nova. I will do it, when os-trait packag will be ready | |
| 18:32:23 | opendevreview | Artom Lifshitz proposed openstack/nova stable/xena: Reproduce bug 1995153 https://review.opendev.org/c/openstack/nova/+/882319 | |
| 18:32:24 | opendevreview | Artom Lifshitz proposed openstack/nova stable/xena: Save cell socket correctly when updating host NUMA topology https://review.opendev.org/c/openstack/nova/+/882320 | |
| 20:48:17 | opendevreview | Merged openstack/nova stable/yoga: Fix LM rollback w/o multi port bindings extension https://review.opendev.org/c/openstack/nova/+/839922 | |
| #openstack-nova - 2023-05-05 | |||
| 03:13:19 | auniyal | finally https://review.opendev.org/c/openstack/nova/+/839922 merged, feels like a good morning :D | |
| 06:12:39 | opendevreview | Tobias Urdin proposed openstack/nova master: Fix wrong nova-manage command in upgrade check https://review.opendev.org/c/openstack/nova/+/880819 | |
| 06:49:58 | opendevreview | Amit Uniyal proposed openstack/nova master: WIP: Reproducer for dangling volumes https://review.opendev.org/c/openstack/nova/+/881457 | |
| 06:49:59 | opendevreview | Amit Uniyal proposed openstack/nova master: WIP: Delete dangling volumes https://review.opendev.org/c/openstack/nova/+/882284 | |
| 07:23:57 | opendevreview | yatin proposed openstack/nova master: Add config option to configure TB cache size https://review.opendev.org/c/openstack/nova/+/868419 | |
| 07:30:11 | sahid | o/ | |
| 08:27:30 | opendevreview | Danylo Vodopianov proposed openstack/nova-specs master: Add support for Napatech LinkVirt SmartNICs https://review.opendev.org/c/openstack/nova-specs/+/859290 | |
| 08:34:09 | gibi | bauzas: thanks for the +2 on https://review.opendev.org/c/openstack/nova/+/862687 , could you please check the test patch below that as well? | |
| 08:35:25 | bauzas | gibi: ah I forgot to click 'submit' | |
| 08:35:34 | bauzas | shitty today | |
| 08:36:27 | bauzas | I could be off for 15 mins, the Tesla garage (changing my windshield due to a rock) will pass me a TMX for 10 mins :) | |
| 08:57:35 | gibi | enjoy :) | |
| 10:07:53 | opendevreview | Merged openstack/nova master: Reproduce asym NUMA mixed CPU policy bug https://review.opendev.org/c/openstack/nova/+/862686 | |
| 11:43:01 | sean-k-mooney | im off today but while i remember this i found an issue with nova unit tests yesterday when trying to setup my new laptop | |
| 11:43:31 | sean-k-mooney | some of our unit tests are calling systemctl becasue of a lack of mocks | |
| 11:43:50 | sean-k-mooney | i noticed this by runing the tests in a ubuntu:22.04 container that did not have it | |
| 11:44:27 | sean-k-mooney | ill try an repoduce on monday and file a bug or fix it depeneind on how much time i have to look at it | |
| 11:46:58 | sean-k-mooney | gibi: https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_0ae/862687/3/gate/nova-tox-functional-py38/0aea18b/testr_results.html there is an odd db issue in teh func test result on your patch | |
| 11:47:16 | sean-k-mooney | that often meens there is a sharing of global state somewhere | |
| 11:47:54 | sean-k-mooney | it passed in other runs so we can proably just recheck it and see if its persitent but that test might be flaky so we should kep an eye on it | |
| 11:48:33 | sean-k-mooney | the failure is not related to your change | |
| 11:54:29 | gibi | sean-k-mooney: it is tracked in https://bugs.launchpad.net/nova/+bug/2002782 | |
| 11:54:45 | sean-k-mooney | oh cool | |
| 11:55:42 | sean-k-mooney | i have seen it once or twice but not often | |
| 11:55:58 | sean-k-mooney | o/ | |
| 11:56:18 | gibi | o/ | |
| 12:14:55 | opendevreview | Franciszek Przewoźny proposed openstack/placement master: Changed /tmp/migrate-db.rc to /root/migrate-db.rc https://review.opendev.org/c/openstack/placement/+/882436 | |
| 12:14:55 | opendevreview | Franciszek Przewoźny proposed openstack/placement master: Changed /tmp/migrate-db.rc to /root/migrate-db.rc https://review.opendev.org/c/openstack/placement/+/882436 | |
| 12:26:58 | opendevreview | Franciszek Przewoźny proposed openstack/placement master: Changed /tmp/migrate-db.rc to /root/migrate-db.rc https://review.opendev.org/c/openstack/placement/+/882436 | |
| 14:11:45 | opendevreview | Dan Smith proposed openstack/nova master: Populate ComputeNode.service_id https://review.opendev.org/c/openstack/nova/+/879904 | |
| 14:11:45 | opendevreview | Dan Smith proposed openstack/nova master: Add compute_id columns to instances, migrations https://review.opendev.org/c/openstack/nova/+/879499 | |
| 14:11:46 | opendevreview | Dan Smith proposed openstack/nova master: Add dest_compute_id to Migration object https://review.opendev.org/c/openstack/nova/+/879682 | |
| 14:11:46 | opendevreview | Dan Smith proposed openstack/nova master: Add compute_id to Instance object https://review.opendev.org/c/openstack/nova/+/879500 | |
| 14:11:47 | opendevreview | Dan Smith proposed openstack/nova master: Online migrate missing Instance.compute_id fields https://review.opendev.org/c/openstack/nova/+/879905 | |
| 14:12:40 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: Reproduce bug 1995153 https://review.opendev.org/c/openstack/nova/+/882321 | |
| 14:12:41 | opendevreview | Artom Lifshitz proposed openstack/nova stable/wallaby: Save cell socket correctly when updating host NUMA topology https://review.opendev.org/c/openstack/nova/+/882322 | |
| 15:38:13 | dansmith | kashyap: another kernel crash this morning: https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_3b1/881764/9/check/cinder-tempest-plugin-basic-zed/3b198f9/testr_results.html | |
| 15:39:06 | dansmith | again, in a test that is attaching volumes, but a very different scenario than the original one.. the former was ceph backed and a full tempest run, this one is non-ceph, small set of tests for just the cinder plugin | |
| 15:39:47 | dansmith | I wonder if it would make sense to just open a kernel bug instead of hitting up the virt team? | |
| 15:47:17 | kashyap | dansmith: Hmm, so it is "fairly intermittently reproducible". I'm assuming you're meaning an upstream kernel bug? (I've filed those in the past - depending on the subsystem, they get looked at; or it just rots) | |
| 15:47:29 | kashyap | s/upstream/distro/ | |
| 15:47:55 | dansmith | kashyap: well, starting with ubuntu kernel might be reasonable, yeah | |