| opendevreview | liuyulong proposed openstack/neutron master: Refactor marionette external-network update to router gateway https://review.opendev.org/c/openstack/neutron/+/908393 | 03:25 |
|---|---|---|
| opendevreview | Takashi Kajinami proposed openstack/neutron-tempest-plugin master: Replace usage of deprecated path for test decorator https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/908549 | 04:08 |
| opendevreview | Takashi Kajinami proposed openstack/neutron-tempest-plugin master: Replace usage of deprecated path for test decorator https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/908549 | 04:17 |
| opendevreview | Takashi Kajinami proposed openstack/neutron-tempest-plugin master: Replace usage of deprecated path for test decorators https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/908549 | 04:17 |
| opendevreview | Takashi Kajinami proposed openstack/neutron-tempest-plugin master: Remove yoga jobs https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/908556 | 05:27 |
| opendevreview | Takashi Kajinami proposed openstack/neutron-tempest-plugin master: Remove train and ussuri jobs https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/908557 | 05:31 |
| opendevreview | Takashi Kajinami proposed openstack/neutron-tempest-plugin master: Replace usage of deprecated path for test decorators https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/908549 | 05:32 |
| opendevreview | Takashi Kajinami proposed openstack/neutron-tempest-plugin master: Replace usage of deprecated path for test decorators https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/908549 | 06:19 |
| opendevreview | Takashi Kajinami proposed openstack/neutron-tempest-plugin master: Remove train and ussuri jobs https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/908557 | 06:21 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: Stop running yoga jobs in check/gate https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/908419 | 06:52 |
| opendevreview | Merged openstack/neutron master: Use different process title for RpcReportsWorker and RpcWorker https://review.opendev.org/c/openstack/neutron/+/907712 | 07:16 |
| ralonsoh | slaweq, finished the review of these 3 patches | 08:29 |
| ralonsoh | please check https://review.opendev.org/c/openstack/neutron/+/907402/4/neutron/services/portforwarding/drivers/ovn/driver.py#156 | 08:29 |
| slaweq | ralonsoh thx, I will update it in about 1h | 08:30 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Log warning about port forwardings that won't work properly https://review.opendev.org/c/openstack/neutron/+/907402 | 08:40 |
| opendevreview | Lajos Katona proposed openstack/networking-bagpipe master: Bump hacking https://review.opendev.org/c/openstack/networking-bagpipe/+/907189 | 08:51 |
| sahid | hum OK so I start to make some progresses regarding dhcp_agents_per_network, it seems that when creating a new network/subnet, the scheduler well honnors the option, but when I do restart a dhcp agent which has not being assigned to handle the network, for some reasons it starts to handle it | 08:56 |
| sahid | It think the issue is somewhere when we retrieve active netowkr for a local agent | 08:57 |
| sahid | there are some conditions it I imagine our evn does not fit the last one which return all networks | 08:58 |
| sahid | https://github.com/openstack/neutron/blob/master/neutron/api/rpc/handlers/dhcp_rpc.py#L85 | 09:00 |
| opendevreview | Lajos Katona proposed openstack/networking-bgpvpn master: Bump hacking https://review.opendev.org/c/openstack/networking-bgpvpn/+/907177 | 09:24 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Add extension "subnet-external-network" https://review.opendev.org/c/openstack/neutron-lib/+/907949 | 09:24 |
| slaweq | ralonsoh ykarel lajoskatona hey, I am checking issue in the CI jobs (not neutron jobs) where random tests are failing due to ssh timeout, like https://3c8c3cc132d3ca41c1a0-8df332a8f6cbb54ee498032ff97f9d17.ssl.cf1.rackcdn.com/882350/2/check/cinder-plugin-ceph-tempest-mn-aa/df2995a/job-output.txt or https://ac3deee033df2f80309a-9b1010a8ed0ed23e4a7e66dfa043a295.ssl.cf5.rackcdn.com/907418/2/check/tempest-slow-py3/6dff044/job-output.txt | 09:27 |
| ralonsoh | did you find something? | 09:28 |
| slaweq | now, I checked it and so far I see one common pattern. Issue happens due to problems with metadata: https://paste.opendev.org/show/bplyv0UbDx1zKkyadsOO/ | 09:28 |
| slaweq | in both cases it is similar | 09:28 |
| slaweq | when I dig deeper into it, in both cases vm was spawned on compute1 node | 09:28 |
| slaweq | and in the ovn-metadata-agent's log I see haproxy for that network is started | 09:29 |
| slaweq | but in journal log I see absolutely nothing logged by this haproxy | 09:29 |
| slaweq | it seems for me like those metadata requests are not going to that haproxy server at all | 09:30 |
| slaweq | did You maybe saw something like that already? | 09:30 |
| ralonsoh | so you think haproxy never started | 09:30 |
| opendevreview | Lajos Katona proposed openstack/tap-as-a-service master: Bump hacking https://review.opendev.org/c/openstack/tap-as-a-service/+/907249 | 09:31 |
| HighLow | Hello Guys, I have one query about the neutron external interfaces.. I have 3 compute nodes and 2 neutron nodes. I'm using kolla ansible to deploy my infra. One of the external network needs communication only though neutron and other external interface needs to directly communicate from compute from VLAN.. So can I have only one external on compute and two external on compute and have provider network enabled? | 09:34 |
| HighLow | Or should all the compute and neutron nodes should have all the external interfaces present?? | 09:36 |
| opendevreview | Lajos Katona proposed openstack/networking-bagpipe master: Bump hacking https://review.opendev.org/c/openstack/networking-bagpipe/+/907189 | 09:45 |
| opendevreview | Lajos Katona proposed openstack/networking-sfc master: Bump hacking https://review.opendev.org/c/openstack/networking-sfc/+/907531 | 09:48 |
| opendevreview | Lajos Katona proposed openstack/neutron-fwaas master: Bump hacking https://review.opendev.org/c/openstack/neutron-fwaas/+/908113 | 09:55 |
| opendevreview | Lajos Katona proposed openstack/neutron-fwaas master: Bump hacking https://review.opendev.org/c/openstack/neutron-fwaas/+/908113 | 09:58 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-lib master: Add "socket" NUMA affinity policy https://review.opendev.org/c/openstack/neutron-lib/+/903531 | 10:34 |
| ralonsoh | slaweq, if you have 2 mins: | 10:49 |
| ralonsoh | * https://review.opendev.org/c/openstack/neutron/+/906277 | 10:49 |
| ralonsoh | * https://review.opendev.org/c/openstack/neutron/+/906868 | 10:49 |
| ralonsoh | thanks! | 10:49 |
| ralonsoh | (documentation) | 10:49 |
| slaweq | ralonsoh: sure | 10:51 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] Document the OVN L3 scheduler: AZs distribution https://review.opendev.org/c/openstack/neutron/+/906868 | 10:59 |
| slaweq | ralonsoh done for both | 11:01 |
| ralonsoh | slaweq, ^^ how did you do that? a rebase "on behalf of..." | 11:04 |
| ralonsoh | that's the first time I see it | 11:04 |
| ralonsoh | what kind of sorcery is that!! | 11:04 |
| slaweq | I have no idea, I just clicked "rebase on parent change" in gerrit | 11:06 |
| slaweq | because it was not based on the latest one | 11:07 |
| slaweq | I didn't do anything else :) | 11:07 |
| opendevreview | Merged openstack/neutron master: [OVN] Document the OVN L3 scheduler https://review.opendev.org/c/openstack/neutron/+/906277 | 11:16 |
| opendevreview | Merged openstack/neutron master: [OVN] Document the OVN L3 scheduler: AZs distribution https://review.opendev.org/c/openstack/neutron/+/906868 | 11:26 |
| opendevreview | Lajos Katona proposed openstack/neutron master: Add tap_mirror to extension to OVN supported extensions https://review.opendev.org/c/openstack/neutron/+/905840 | 12:15 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/2023.2: [stable-only] Fix to cover multiple listener on IPv6 LB https://review.opendev.org/c/openstack/ovn-octavia-provider/+/908311 | 12:21 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider stable/2023.1: [stable-only] Fix to cover multiple listener on IPv6 LB https://review.opendev.org/c/openstack/ovn-octavia-provider/+/908569 | 12:32 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [OVN] A LRP in an external tunnelled network has no chassis https://review.opendev.org/c/openstack/neutron/+/908325 | 12:37 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron-tempest-plugin master: Add router check, subnet attached gateway IP update or deletion https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/904710 | 12:41 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Change SG rules backref load method to "joined" https://review.opendev.org/c/openstack/neutron/+/908571 | 13:07 |
| ralonsoh | slaweq, lajoskatona obondarev ^^ please check this. I've manually tested with different loading method (in the commit message) | 13:08 |
| ralonsoh | this is temporary: we need to implement a method to only load the shared field without loading any other synthetic field (that will be useful for any other RBAC resource) | 13:08 |
| ralonsoh | but for now we need to push it (I'll also backport this patch if accepted) | 13:09 |
| ralonsoh | thanks! | 13:09 |
| ralonsoh | (I'm going for lunch) | 13:11 |
| lajoskatona | ralonsoh: checking | 13:14 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: DNM Test periodic jobs with sqlalchemy from master https://review.opendev.org/c/openstack/neutron/+/908573 | 13:16 |
| opendevreview | Christian Rohmann proposed openstack/neutron master: Allow HA routers to have automatic l3agent failover https://review.opendev.org/c/openstack/neutron/+/908434 | 13:35 |
| opendevreview | Jakub Libosvar proposed openstack/ovn-bgp-agent master: WIP: Test devstack job with tempest https://review.opendev.org/c/openstack/ovn-bgp-agent/+/905789 | 13:41 |
| opendevreview | Jakub Libosvar proposed openstack/ovn-bgp-agent master: Add bgp tempest job https://review.opendev.org/c/openstack/ovn-bgp-agent/+/905789 | 13:42 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.2: Change SG rules backref load method to "joined" https://review.opendev.org/c/openstack/neutron/+/908586 | 14:44 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/2023.1: Change SG rules backref load method to "joined" https://review.opendev.org/c/openstack/neutron/+/908587 | 14:45 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/zed: Change SG rules backref load method to "joined" https://review.opendev.org/c/openstack/neutron/+/908588 | 14:45 |
| ralonsoh | haleyb, hello! if you have 5 mins: https://review.opendev.org/c/openstack/neutron/+/908571 | 14:51 |
| ralonsoh | thanks! | 14:51 |
| ralonsoh | (I'm planning to backport it too) | 14:51 |
| ykarel | slaweq, for that metadata issue i recall https://bugs.launchpad.net/neutron/+bug/2045549 but that was filed for ml2/ovs | 15:11 |
| ykarel | may be similar issue with ovn too | 15:11 |
| opendevreview | Merged openstack/ovn-octavia-provider stable/2023.1: [stable-only] Fix to cover multiple listener on IPv6 LB https://review.opendev.org/c/openstack/ovn-octavia-provider/+/908569 | 15:30 |
| haleyb | ralonsoh: ack. seems it failed py38 due to list ordering? | 15:46 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Log warning about port forwardings that won't work properly https://review.opendev.org/c/openstack/neutron/+/907402 | 15:48 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: [Docs] Add info about incompatible PF configuration in ML2/OVN gaps https://review.opendev.org/c/openstack/neutron/+/907403 | 15:48 |
| opendevreview | Elod Illes proposed openstack/tap-as-a-service stable/2023.2: [stable-only] reno: fix indentation https://review.opendev.org/c/openstack/tap-as-a-service/+/908598 | 15:48 |
| haleyb | wonder whether it needs to use self.assertDictEqual(sg_dict, sg_dict_got) | 15:48 |
| ralonsoh | haleyb, checking now, thanks! | 15:49 |
| opendevreview | Elod Illes proposed openstack/tap-as-a-service stable/2023.1: [stable-only] reno: fix indentation https://review.opendev.org/c/openstack/tap-as-a-service/+/908599 | 15:50 |
| opendevreview | Elod Illes proposed openstack/tap-as-a-service stable/zed: [stable-only] reno: fix indentation https://review.opendev.org/c/openstack/tap-as-a-service/+/908600 | 15:51 |
| ralonsoh | haleyb, nope, I manually tried that and self.assertDictEqual will do the same | 16:10 |
| ralonsoh | actually we replaced self.assertDictEqual with self.assertEqual | 16:11 |
| ralonsoh | because the result is the same | 16:11 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Change SG rules backref load method to "joined" https://review.opendev.org/c/openstack/neutron/+/908571 | 16:15 |
| elodilles | lajoskatona ralonsoh : these patches should fix the reno job on tap-as-a-service master, could you have a quick review on them? https://review.opendev.org/q/topic:reno_fix+project:openstack/tap-as-a-service | 16:16 |
| ralonsoh | checking now | 16:16 |
| haleyb | ralonsoh: ordering them works too :) | 16:17 |
| elodilles | ralonsoh: now only the reno on yoga is causing the issue, but we would iterate this over and over if we don't fix this on all other branches | 16:17 |
| elodilles | ralonsoh: and thanks in advance o:) | 16:17 |
| ralonsoh | yw | 16:17 |
| ralonsoh | that's curious... I only have +1 powers on unmantained | 16:18 |
| ralonsoh | I didn't know that we were going to lose these permissions on um branches | 16:19 |
| elodilles | ralonsoh: well, maybe we should advertise it again, but the unmaintained process goal is to not force team's to feel the responsibility of keeping the non-stable branches maintained. it is welcome if someone helps to maintain them though | 16:24 |
| ralonsoh | perfect then | 16:25 |
| elodilles | ralonsoh: hence only openstack-unmaintained-core group has +2+W rights on unmaintained/* branches | 16:25 |
| elodilles | ralonsoh: if someone, or a team, decides to help there time to time, then a neutron-unmaintained-core group can be created, and populated with people who wants to help when they have time | 16:26 |
| ralonsoh | I'll make this reminder next day in the Neutron team meeting | 16:26 |
| elodilles | ralonsoh: ACK, thanks | 16:27 |
| elodilles | ralonsoh: an example of a <project>-unmaintained-core group creation: https://review.opendev.org/c/openstack/project-config/+/902796 | 16:27 |
| ralonsoh | added to the neutron meeting log, to have this as an example | 16:28 |
| elodilles | thanks o/ | 16:28 |
| lajoskatona | ralonsoh, haleyb, elodilles: +1 for meeting discussion | 16:32 |
| lajoskatona | ralonsoh, haleyb, elodilles: another thing is which is surprise for me that um/yoga patches are not advertised on the irc channel, perhaps that's what we want but would be good than to make that clear and find a way where I can have notifications for such reviews :P | 16:33 |
| frickler | that's configured here ... feel free to change if you feel like it https://opendev.org/openstack/project-config/src/branch/master/gerritbot/channels.yaml#L927-L929 | 16:35 |
| ralonsoh | lajoskatona, ^ I think we can also track um branches here in this channel | 16:36 |
| lajoskatona | frickler: thanks, I was sure you know where to set it :-) | 16:36 |
| ralonsoh | frickler, thanks! | 16:36 |
| ralonsoh | I'll propose a patch (but tomorrow, I need to leave early today) | 16:36 |
| ralonsoh | have a nice weekend! | 16:36 |
| lajoskatona | ralonsoh: +1 | 16:37 |
| lajoskatona | ralonsoh: I can propose, but thught we can discuss new team meeting, to avoid over spamming sombody | 16:37 |
| lajoskatona | have a nice weekend | 16:37 |
| ralonsoh | for sure | 16:37 |
| opendevreview | Merged openstack/tap-as-a-service stable/2023.2: [stable-only] reno: fix indentation https://review.opendev.org/c/openstack/tap-as-a-service/+/908598 | 16:51 |
| opendevreview | Merged openstack/tap-as-a-service stable/2023.1: [stable-only] reno: fix indentation https://review.opendev.org/c/openstack/tap-as-a-service/+/908599 | 16:51 |
| opendevreview | Merged openstack/tap-as-a-service stable/zed: [stable-only] reno: fix indentation https://review.opendev.org/c/openstack/tap-as-a-service/+/908600 | 16:51 |
| elodilles | this mail's second part explains also what i wrote above about the <project>-unmaintained-core groups: https://lists.openstack.org/archives/list/openstack-discuss@lists.openstack.org/thread/V7H6M5ANYOMFWSZ5BJSWAXVNQCUXI6D7/ | 16:51 |
| elodilles | maybe it is better phrased :) so, feel free to use it on the team meeting | 16:51 |
| elodilles | thanks in advance :) | 16:52 |
| opendevreview | Miguel Lavalle proposed openstack/neutron master: Enable HA for OVN router flavors https://review.opendev.org/c/openstack/neutron/+/901513 | 17:50 |
| opendevreview | Merged openstack/neutron master: [OVN] Warn about invalid OVN and FIP PF config during start of Neutron https://review.opendev.org/c/openstack/neutron/+/892542 | 17:55 |
| opendevreview | Merged openstack/neutron master: Not process security group for none active ports https://review.opendev.org/c/openstack/neutron/+/905289 | 19:46 |
| opendevreview | Brian Haley proposed openstack/neutron master: Change OVN client to account for nested networks https://review.opendev.org/c/openstack/neutron/+/907504 | 21:00 |
| opendevreview | alisafari proposed openstack/ovn-octavia-provider master: Fix lb association to routers process https://review.opendev.org/c/openstack/ovn-octavia-provider/+/908679 | 21:57 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!