| opendevreview | Merged openstack/neutron-lib stable/xena: Use neutron-tempest-plugin-api-xena job in stable/xena branch https://review.opendev.org/c/openstack/neutron-lib/+/879423 | 01:33 |
|---|---|---|
| opendevreview | Merged openstack/neutron-lib stable/yoga: Use neutron-tempest-plugin-api-yoga job in the stable/yoga branch https://review.opendev.org/c/openstack/neutron-lib/+/879422 | 01:33 |
| opendevreview | ZhouHeng proposed openstack/neutron master: Set result when lswitch port exist https://review.opendev.org/c/openstack/neutron/+/881771 | 02:01 |
| *** gthiemon1e is now known as gthiemonge | 07:12 | |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Don't set and remove immediately DEAD VLAN tag in tests https://review.opendev.org/c/openstack/neutron/+/881798 | 07:31 |
| ralonsoh | vvr-plv, hi | 07:38 |
| ralonsoh | vvr-plv, hello? | 07:39 |
| ralonsoh | vvr-plv, please, use this channel | 07:41 |
| ralonsoh | so your question in infra channel was about using a filter for the neutron tempest plugin tests | 07:41 |
| ralonsoh | right? | 07:41 |
| ralonsoh | as I commented you last day, you can propose a patch with the tests that will be executed in your CI | 07:42 |
| ralonsoh | using this tag | 07:42 |
| ralonsoh | this is the first step. About the tag used, please propose a name and it will be reviewed in the patch | 07:42 |
| ralonsoh | if you think "ntnic-verify" is ok, go for it | 07:43 |
| vvr-plv | ralonsoh: hello, no, it was about filtering Gerrit review changes by "magic" word. It isn't about tests. | 07:43 |
| ralonsoh | so the point is that you want to execute in your CI only some tests | 07:43 |
| ralonsoh | right? | 07:43 |
| ralonsoh | and from some patches only | 07:44 |
| vvr-plv | yeah, but it more about triggering of run tests, not about tests | 07:44 |
| ralonsoh | I have no idea if gerrit has this capability, to be honest | 07:44 |
| ralonsoh | if you can't run the CI job for all Neutron patches | 07:45 |
| ralonsoh | what you can do is to run periodic runs, every day | 07:45 |
| ralonsoh | adding a CI job in out periodic queue | 07:45 |
| vvr-plv | for example somebody add comment to review with ntnic-verify (for example) and our regression starts/share results | 07:45 |
| ralonsoh | we can discuss that in the Neutron meeting | 07:47 |
| ralonsoh | I don't like to tint the commit message with a tag only for testting | 07:47 |
| ralonsoh | in 5 years we'll review the commited messages and won't know why that was that for | 07:47 |
| ralonsoh | but you can present that in the Neutron community and the drivers meeting | 07:48 |
| ralonsoh | but please, think about the periodic queue | 07:48 |
| ralonsoh | and you can also execute, by demand, the experimental queue | 07:48 |
| ralonsoh | for example: you can have your own CI job | 07:48 |
| ralonsoh | "neutron-tempest-plugin-openvswitch" for example | 07:49 |
| ralonsoh | you can have this job in the "periodic" and "experimental" queues | 07:49 |
| ralonsoh | when you push a patch, you can comment in the patch "check experimental" | 07:49 |
| ralonsoh | that will trigger the CI queue "experimental" | 07:49 |
| ralonsoh | look, this patch https://review.opendev.org/c/openstack/neutron/+/869510 | 07:50 |
| ralonsoh | this is the zuul status now | 07:50 |
| ralonsoh | https://zuul.opendev.org/t/openstack/status#869510 | 07:50 |
| ralonsoh | for this patch | 07:50 |
| ralonsoh | you can see nothing is running | 07:50 |
| ralonsoh | now I'm going to comment in the patch "check experimental" | 07:50 |
| ralonsoh | check the zuul status now | 07:50 |
| ralonsoh | did you see the experimental queue is now running the patch? | 07:51 |
| vvr-plv | yes, I see. I'm talking about comment in the patch "check experimental". "check experimental" is like a "magic" trigger. | 07:53 |
| vvr-plv | as I understood, it's okay if our regression will start on such a trigger | 07:54 |
| ralonsoh | you can use it, yes | 07:54 |
| ralonsoh | you can create your own CI job and add it to experimental queue | 07:55 |
| vvr-plv | great, that was exactly my question, thank you very much | 07:55 |
| ralonsoh | yw | 07:55 |
| sahid_ | o/ i have added some details regarding the fix https://review.opendev.org/c/openstack/neutron/+/880131 | 08:03 |
| ralonsoh | sahid_, I was asking for a reproducer. I know this is happenign with OVS and routed networks | 08:05 |
| ralonsoh | but how to reproduce this issue: create a network, create segments, etc | 08:05 |
| ralonsoh | the steps to reproduce it | 08:05 |
| ralonsoh | and if there is a patch that introduced this error, comment that too | 08:07 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Drop "__autocommit=False" from context manager https://review.opendev.org/c/openstack/neutron-lib/+/881803 | 08:08 |
| opendevreview | Ke Niu proposed openstack/neutron-specs master: setup.cfg: Replace dashes with underscores https://review.opendev.org/c/openstack/neutron-specs/+/881772 | 08:12 |
| sahid | the reproducer in indicated in the bug report and i have mentioned the commit which introduced the bug | 08:13 |
| ralonsoh | in the patch commit's message | 08:14 |
| sahid | let me double check and i will add that to the commit message | 08:14 |
| sahid | ah yes rights | 08:14 |
| sahid | will do that thanks | 08:14 |
| opendevreview | Ke Niu proposed openstack/neutron-vpnaas-dashboard master: setup.cfg: Replace dashes with underscores https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/881773 | 08:19 |
| opendevreview | Ke Niu proposed openstack/neutron-specs master: setup.cfg: Replace dashes with underscores https://review.opendev.org/c/openstack/neutron-specs/+/881772 | 08:20 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Make "project_id" in "L3HARouterNetwork" unique index https://review.opendev.org/c/openstack/neutron/+/881735 | 08:27 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Introduce "HasProjectPrimaryUniqueKey" class https://review.opendev.org/c/openstack/neutron-lib/+/881804 | 08:31 |
| opendevreview | liuxie proposed openstack/neutron-fwaas master: Support standard_attrs for fwaas resources https://review.opendev.org/c/openstack/neutron-fwaas/+/866988 | 08:34 |
| opendevreview | liuxie proposed openstack/neutron-fwaas master: Support standard_attrs for fwaas resources https://review.opendev.org/c/openstack/neutron-fwaas/+/866988 | 08:41 |
| opendevreview | Merged openstack/neutron-specs master: setup.cfg: Replace dashes with underscores https://review.opendev.org/c/openstack/neutron-specs/+/881772 | 08:54 |
| opendevreview | Merged openstack/neutron-vpnaas-dashboard master: setup.cfg: Replace dashes with underscores https://review.opendev.org/c/openstack/neutron-vpnaas-dashboard/+/881773 | 08:56 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: ``_get_ovn_version`` returns a 3 element tuple https://review.opendev.org/c/openstack/neutron/+/881708 | 08:57 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Make "project_id" in "L3HARouterNetwork" unique index https://review.opendev.org/c/openstack/neutron/+/881735 | 08:58 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: New ``network-ha`` API definition https://review.opendev.org/c/openstack/neutron-lib/+/881734 | 09:04 |
| ralonsoh | ykarel, hi, should we move out from nested virt until vexxhost nodes are fixed? | 09:16 |
| ralonsoh | the neutron CI is completely blocked | 09:16 |
| ralonsoh | guilhermesp_____, hi, do you have any update on the mirror errors? | 09:23 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Don't set and remove immediately DEAD VLAN tag in tests https://review.opendev.org/c/openstack/neutron/+/881798 | 09:58 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: Doc: add documentation for usage and flow examples for OVS https://review.opendev.org/c/openstack/tap-as-a-service/+/828382 | 10:18 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: Doc: add documentation for usage and driver details for SRIOV driver https://review.opendev.org/c/openstack/tap-as-a-service/+/881807 | 10:18 |
| ralonsoh | slaweq, hey, we have a problem with the CI | 10:21 |
| ralonsoh | I'm trying to reach someone in infra | 10:21 |
| slaweq | ralonsoh with mirrors, right? | 10:21 |
| ralonsoh | but I'm going to send a mail, saying that we are completely stopped | 10:21 |
| ralonsoh | right | 10:21 |
| slaweq | yes, I saw it | 10:22 |
| slaweq | I pinged fungi about it on #openstack-tc channel | 10:22 |
| ralonsoh | I know ykarel was working on this with some folks | 10:22 |
| lajoskatona | thanks for bringing it here, I just wanted to ask if it worth to recheck mirror issue failures | 10:22 |
| ralonsoh | no | 10:22 |
| ralonsoh | most of the times, we use a host that cannot connect to the mirrros | 10:22 |
| lajoskatona | yesterday ykarel has some chat with some vexhost engineer: https://meetings.opendev.org/irclogs/%23openstack-neutron/%23openstack-neutron.2023-04-27.log.html | 10:23 |
| ralonsoh | yes | 10:23 |
| ralonsoh | but we still don't have any feedback | 10:23 |
| lajoskatona | but I don't know if there was any progress | 10:23 |
| ralonsoh | of course, I'm not saying the problem is easy to solve | 10:23 |
| lajoskatona | ahh ,ok | 10:23 |
| ralonsoh | but at least some feedback | 10:23 |
| ralonsoh | ok, I'll keep an eye on -tc and -infra channels | 10:24 |
| opendevreview | Lajos Katona proposed openstack/neutron stable/2023.1: Delete sg rule which remote is the deleted sg https://review.opendev.org/c/openstack/neutron/+/881661 | 10:24 |
| opendevreview | Lajos Katona proposed openstack/neutron stable/zed: Delete sg rule which remote is the deleted sg https://review.opendev.org/c/openstack/neutron/+/881662 | 10:24 |
| opendevreview | Lajos Katona proposed openstack/neutron stable/yoga: Delete sg rule which remote is the deleted sg https://review.opendev.org/c/openstack/neutron/+/881663 | 10:25 |
| lajoskatona | ralonsoh: thanks, don't know if a mail is necessary to the openstack-dev if we have more details and references.... | 10:32 |
| ralonsoh | at least to make public that we have a problem and we don't have any information, regardless of asking for that | 10:33 |
| ykarel | yes right haven't heard back from guilhermesp_____ about the mirror issue, may be we can push patch to temporary disable vexxhost provider until the issue get's clear? | 10:33 |
| ralonsoh | right, we should not use vexxhost | 10:34 |
| ralonsoh | ykarel, but how can we block it from zuul config? | 10:34 |
| ralonsoh | use other nodeset? | 10:34 |
| ykarel | not sure what's the fix for it, but i recall mnaser fixed some issues in past like this | 10:34 |
| ralonsoh | https://review.opendev.org/c/airship/kubernetes-entrypoint/+/815045 | 10:35 |
| ykarel | ralonsoh, let me find, no not nodeset but disable provider | 10:35 |
| ralonsoh | but this is just for the opposite taks | 10:35 |
| ykarel | the best would be those compute nodes are disable from infra side as other compute nodes are working fine | 10:37 |
| ralonsoh | ykarel, we can request that in infra | 10:38 |
| ykarel | https://github.com/openstack/project-config/blob/master/nodepool/nl03.opendev.org.yaml#L140 | 10:43 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/yoga: Use ovsdbapp commands to add/del backends to ip_port_mappings https://review.opendev.org/c/openstack/ovn-octavia-provider/+/881809 | 10:43 |
| ykarel | pushed https://review.opendev.org/c/openstack/project-config/+/881810 | 10:49 |
| lajoskatona | ralonsoh, ykarel: ack | 10:49 |
| ralonsoh | perfect | 10:50 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Use the new network HA parameter https://review.opendev.org/c/openstack/neutron/+/881742 | 10:53 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Make "project_id" in "L3HARouterNetwork" unique index https://review.opendev.org/c/openstack/neutron/+/881735 | 11:06 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Use the new network HA parameter https://review.opendev.org/c/openstack/neutron/+/881742 | 11:06 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/yoga: Use ovsdbapp commands to add/del backends to ip_port_mappings https://review.opendev.org/c/openstack/ovn-octavia-provider/+/880953 | 11:10 |
| ralonsoh | ok, I think we are getting some traction in #opendev | 11:21 |
| opendevreview | yatin proposed openstack/neutron-tempest-plugin master: [DNM] Check status jammy nested-virt nodes https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/867609 | 11:38 |
| lajoskatona | ralonsoh: cool | 11:56 |
| opendevreview | yatin proposed openstack/neutron-tempest-plugin master: [DNM] Check status jammy nested-virt nodes https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/867609 | 12:00 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/xena: Use ovsdbapp commands to add/del backends to ip_port_mappings https://review.opendev.org/c/openstack/ovn-octavia-provider/+/880954 | 12:13 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/wallaby: Use ovsdbapp commands to add/del backends to ip_port_mappings https://review.opendev.org/c/openstack/ovn-octavia-provider/+/880955 | 12:18 |
| guilhermesp_____ | ykarel: ralonsoh hi there. I’m a bit confused here. Nested virt issues are different from mirror issues. Nested virt issues should be fixed by now in ca-ymq-1 hypervisors | 12:20 |
| guilhermesp_____ | I’ve asked the following yesterday | 12:20 |
| guilhermesp_____ | 11:11 <guilhermesp_____> ykarel: is there any tickets on our side in regards to that. I particularly dont know if we maintain mirrors tbh so would be good a ticket with context so our team can go ahead and see if thats something on our side | 12:20 |
| ykarel | guilhermesp_____, ok can create launchpad bug if that works or need to create ticket somewhere else? | 12:25 |
| ykarel | with mirror we meant https://mirror.ca-ymq-1.vexxhost.opendev.org/ubuntu | 12:25 |
| ykarel | but seems it's not mirror specific but general ipv6 connectivity | 12:26 |
| guilhermesp_____ | Please open a ticket on our side ( send email to support@vexxhost.com ) — it’s better for us to organize on our queue here. | 12:30 |
| ykarel | ok sure doing | 12:31 |
| guilhermesp_____ | Is this a vm created by you? | 12:31 |
| ykarel | by CI | 12:31 |
| ykarel | you checked host_ids in https://paste.opendev.org/show/bCbxIrXR1P01q4JYUh3i/ ? if they match the nodes that you recently upgraded | 12:32 |
| ralonsoh | guilhermesp_____, please join #opendev too | 12:32 |
| ralonsoh | this is being discussed now there too | 12:32 |
| ralonsoh | ykarel, are you sending the mail? If so, please add me in CC. If not, I'll do it. | 12:36 |
| ykarel | ralonsoh, sending now, will add you, also reported https://bugs.launchpad.net/neutron/+bug/2017992 to track publicly | 12:40 |
| ralonsoh | perfect | 12:40 |
| ykarel | ralonsoh, guilhermesp_____ sent mail | 12:49 |
| ralonsoh | ykarel++ | 12:49 |
| ykarel | just now noticed it cannot connect even with IPv4 as per one of the logs, so looks general issue | 12:50 |
| ralonsoh | right, the problem is also present with ipv4 addresses | 12:54 |
| guilhermesp_____ | ok we received the ticket on our end. thanks | 12:59 |
| ralonsoh | cool | 12:59 |
| ralonsoh | even with https://review.opendev.org/c/openstack/project-config/+/881810 merged, we are still using these nodes | 13:15 |
| ralonsoh | https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_52c/881708/2/check/neutron-tempest-plugin-openvswitch/52cb019/job-output.txt | 13:15 |
| ykarel | ralonsoh, it takes some time to reflect project-config changes | 14:00 |
| ralonsoh | okidoki | 14:00 |
| ykarel | there runs a post job after the change which applies settings | 14:01 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [WIP] Fix race condition when creating two routers without HA network https://review.opendev.org/c/openstack/neutron/+/881826 | 14:01 |
| ykarel | change is merged* | 14:01 |
| lajoskatona | so for Monday we will ave CI again? 🤞 | 14:01 |
| ralonsoh | I'll try tomorrow and I | 14:01 |
| ralonsoh | and I'll send an update | 14:01 |
| lajoskatona | tomorrow even the grass will stop growing, and Monday is the special day to celebrate work so we do not work (at least some part of the world) :-) | 14:02 |
| ykarel | infra-prod-service-nodepool https://zuul.opendev.org/t/openstack/build/171491fadf4d4ca48ff5b982b5a6b942 : SUCCESS in 5m 54s | 14:03 |
| ykarel | i see ^ so should be fine for jobs triggered after that | 14:03 |
| ralonsoh | cool | 14:03 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Remove "neutron-ovn-tempest-ovs-release-ubuntu-old" job https://review.opendev.org/c/openstack/neutron/+/881342 | 14:15 |
| opendevreview | Fernando Royo proposed openstack/neutron master: WIP: Return 403 Forbidden to tenant user deleting port attached to FIP https://review.opendev.org/c/openstack/neutron/+/881827 | 14:19 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Avoid retrieving ports if network list is empty https://review.opendev.org/c/openstack/neutron/+/881830 | 14:40 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/yoga: Replace context decorators with context managers https://review.opendev.org/c/openstack/neutron/+/881572 | 14:57 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: Replace context decorators with context managers https://review.opendev.org/c/openstack/neutron/+/881573 | 14:57 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Make "project_id" in "L3HARouterNetwork" unique constraint https://review.opendev.org/c/openstack/neutron/+/881735 | 15:16 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: WIP - Add ``OVNGatewayHAChassisGroup`` scheduler class https://review.opendev.org/c/openstack/neutron/+/872033 | 15:25 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Mark "ipv6_pd_enabled" as deprecated and experimental. https://review.opendev.org/c/openstack/neutron/+/879030 | 15:36 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Make "project_id" in "L3HARouterNetwork" unique constraint https://review.opendev.org/c/openstack/neutron/+/881735 | 16:23 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Use the new network HA parameter https://review.opendev.org/c/openstack/neutron/+/881742 | 16:23 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [WIP] Fix race condition when creating two routers without HA network https://review.opendev.org/c/openstack/neutron/+/881826 | 16:23 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Mark "test_port_creation_and_deletion" as unstable https://review.opendev.org/c/openstack/neutron/+/881876 | 17:14 |
| opendevreview | Merged openstack/neutron master: Add support for localnet_learn_fdb OVN option https://review.opendev.org/c/openstack/neutron/+/877675 | 17:36 |
| opendevreview | Merged openstack/neutron-lib master: Add a shim extension standard-attr-fwaas-v2 https://review.opendev.org/c/openstack/neutron-lib/+/881694 | 17:52 |
| opendevreview | Merged openstack/neutron stable/wallaby: Honor debug mode in keepalived-state-change script logs https://review.opendev.org/c/openstack/neutron/+/881357 | 18:11 |
| opendevreview | Merged openstack/neutron master: Doc: Add FWaaS v2 install details https://review.opendev.org/c/openstack/neutron/+/881239 | 18:36 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: ``_get_ovn_version`` returns a 3 element tuple https://review.opendev.org/c/openstack/neutron/+/881708 | 20:37 |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: [WIP][DNM][OVN] Change the default firewall policy https://review.opendev.org/c/openstack/neutron/+/839066 | 23:25 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!