| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2023-05-04 | |||
| 13:46:49 | bauzas | and https://zuul.opendev.org/t/openstack/build/de4b1d075b0745aab2bdec9bc8319877 | |
| 13:47:20 | bauzas | then I forgot about it | |
| 13:48:24 | ralonsoh | bauzas, if recall correctly (maybe I'm wrong), the issue with create_security_group_rule was that we where calling the nova client | |
| 13:52:49 | ralonsoh | sean-k-mooney, I'm checking https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_6e0/882052/1/gate/grenade-skip-level-always/6e0e087/testr_results.html | |
| 13:53:00 | ralonsoh | and I don't see where Neutron is failing | |
| 13:55:17 | bauzas | ralonsoh: I can be wrong but in https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_6e0/882052/1/gate/grenade-skip-level-always/6e0e087/testr_results.html we have test_security_group_rules_create[id-850795d7-d4d3-4e55-b527-a774c0123d3a,smoke] failing on File "/opt/stack/new/tempest/tempest/lib/services/compute/security_group_rules_client.py", line 35, in create_security_group_rule | |
| 13:55:47 | bauzas | this is an API call to the os-security-groups API which belongs to Nova | |
| 13:56:01 | bauzas | but this actually is a proxy call to Neutron | |
| 13:56:36 | ralonsoh | yes and I'm reviewing the logs and I can't find any error | |
| 13:56:48 | ralonsoh | I'm checking the same call in other tempest runs | |
| 14:00:19 | bauzas | that's unfortunate that given the timeout, we don't have the request-id | |
| 14:03:25 | ralonsoh | bauzas, but what call is timing out? because is not the SG creation | |
| 14:06:20 | bauzas | ralonsoh: this is failing on https://github.com/openstack/tempest/blob/master/tempest/api/compute/security_groups/test_security_group_rules.py#L67 | |
| 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: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: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 | 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:12:48 | ralonsoh | well, actually this is the req-id of the SG rule creation | |
| 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 | |