Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2023-05-04
14:06:53 bauzas I think I can trace the nova-api call with May 04 12:02:50.750575 np0033942429 devstack@n-api.service[165740]: DEBUG nova.api.openstack.wsgi [None req-49c6f58c-b29d-4c83-8ad1-5a588383533f tempest-SecurityGroupRulesTestJSON-878638434 tempest-SecurityGroupRulesTestJSON-878638434-project-member] Action: 'create', calling method: <function Controller.__getattribute__.<locals>.version_select at 0x7f7d37c793f0>, body: {"security_group_
14:06:53 bauzas rule": {"parent_group_id": "b90c2750-9bbd-44fc-9ad2-2eba1fcc4c76", "ip_protocol": "tcp", "from_port": 22, "to_port": 22}} {{(pid=165740) _process_stack /opt/stack/new/nova/nova/api/openstack/wsgi.py:511}}
14:08:01 bauzas so now I'm trying to find logs from req-49c6f58c-b29d-4c83-8ad1-5a588383533f
14:10:17 bauzas look, I can see the req-id in neutron May 04 12:02:50.886890 np0033942429 neutron-server[162430]: INFO neutron.wsgi [req-49c6f58c-b29d-4c83-8ad1-5a588383533f req-bc14944a-4c80-4107-9ce5-de32f17088f8 tempest-SecurityGroupRulesTestJSON-878638434 tempest-SecurityGroupRulesTestJSON-878638434-project-member] 173.231.255.102 "GET /v2.0/security-groups/b90c2750-9bbd-44fc-9ad2-2eba1fcc4c76 HTTP/1.1" status: 200 len: 1820 time: 0.1247852
14:12:00 ralonsoh but this is a get call, not the SG rule creation
14:12:48 ralonsoh well, actually this is the req-id of the SG rule creation
14:12:48 lajoskatona bauzas, ralonsoh: Hi, this is related to https://bugs.launchpad.net/neutron/+bug/2015065 as I see (sorry perhaps you already linked the lp link)
14:13:31 bauzas lajoskatona: excellent, will mention it then
14:13:56 bauzas lajoskatona: fwiw, this is not only on neutron-ovs-grenade-dvr-multinode but also on our jobs too :)
14:15:26 lajoskatona bauzas: yeah, with rodolfo we added later that as comment with opensearch links and examples also from tempest
14:15:36 bauzas cool ++
14:16:56 ralonsoh bauzas, this is using neutronclient, right?
14:17:09 bauzas good question
14:17:20 bauzas afaik yes, but recently we wanted to use the sdk
14:17:38 bauzas I don't think we merged any change yet, but lemme doublecheck
14:17:41 opendevreview Artom Lifshitz proposed openstack/nova stable/2023.1: Reproduce bug 1995153 https://review.opendev.org/c/openstack/nova/+/882313
14:17:42 opendevreview Artom Lifshitz proposed openstack/nova stable/2023.1: Save cell socket correctly when updating host NUMA topology https://review.opendev.org/c/openstack/nova/+/882314
14:18:01 ralonsoh I don't see any change in neutronclient nor nova
14:18:29 opendevreview Artom Lifshitz proposed openstack/nova stable/zed: Reproduce bug 1995153 https://review.opendev.org/c/openstack/nova/+/882315
14:18:30 opendevreview Artom Lifshitz proposed openstack/nova stable/zed: Save cell socket correctly when updating host NUMA topology https://review.opendev.org/c/openstack/nova/+/882316
14:18:58 opendevreview Artom Lifshitz proposed openstack/nova stable/yoga: Reproduce bug 1995153 https://review.opendev.org/c/openstack/nova/+/882317
14:18:59 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
14:19:24 bauzas ralonsoh: you're correct, still neutronclient https://github.com/openstack/nova/blob/master/nova/network/security_group_api.py#L379
14:19:37 ralonsoh yes
14:19:54 bauzas anyway, I need to go get my daughter from school, bbiab (~15 mins)
14:20:36 opendevreview Artom Lifshitz proposed openstack/nova stable/xena: Reproduce bug 1995153 https://review.opendev.org/c/openstack/nova/+/882319
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

Earlier   Later