| opendevreview | Ghanshyam proposed openstack/os-vif master: Update gate jobs as per the 2023.1 cycle testing runtime https://review.opendev.org/c/openstack/os-vif/+/861468 | 01:50 |
|---|---|---|
| opendevreview | yatin proposed openstack/neutron-tempest-plugin master: Drop unused branch_override https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/864995 | 07:45 |
| lajoskatona | ralonsoh: Hi, I added the taas old branch EOLing topic to the rel-mgmt meeting agenda | 08:06 |
| lajoskatona | ralonsoh: the rel-mgmt meeting is also Friday 1400 UTC, so I will be split between drivers meeting and rel-mgmt meeting :-) | 08:07 |
| ralonsoh | lajoskatona, thanks. I think there is no agenda for today] | 08:10 |
| ralonsoh | so I'll attend to the mgmt meeting too | 08:10 |
| lajoskatona | ralonsoh: ahh, cool, makes it easier :-) | 08:10 |
| ralonsoh | I'm going to send a mail now | 08:11 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Always create a "router_extra_attributes" register per router https://review.opendev.org/c/openstack/neutron/+/864051 | 08:51 |
| *** ges_ is now known as ges | 08:58 | |
| opendevreview | weilingfei proposed openstack/neutron master: Update url and package name https://review.opendev.org/c/openstack/neutron/+/863013 | 09:36 |
| opendevreview | huanghailun proposed openstack/neutron master: Update tap ip in metadata agent when metadata port ip updated https://review.opendev.org/c/openstack/neutron/+/864715 | 09:50 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Add support for HM on a fullypopulated load balancers https://review.opendev.org/c/openstack/ovn-octavia-provider/+/865003 | 09:50 |
| slaweq | ralonsoh: thx for making Stein, Rocky and Queens EOL - it's finally mergeg \o/ | 10:02 |
| ralonsoh | slaweq, hehehe those branches need to rest | 10:02 |
| slaweq | yeah | 10:03 |
| slaweq | now it's time for Train ;) | 10:03 |
| slaweq | and we will EOL py27 then | 10:03 |
| ralonsoh | exactly | 10:03 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron stable/zed: OVN: Add support for DHCP option "domain-search" for IPv4 https://review.opendev.org/c/openstack/neutron/+/865006 | 10:20 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron stable/yoga: OVN: Add support for DHCP option "domain-search" for IPv4 https://review.opendev.org/c/openstack/neutron/+/865007 | 10:20 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron stable/xena: OVN: Add support for DHCP option "domain-search" for IPv4 https://review.opendev.org/c/openstack/neutron/+/865008 | 10:20 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron stable/wallaby: OVN: Add support for DHCP option "domain-search" for IPv4 https://review.opendev.org/c/openstack/neutron/+/865009 | 10:20 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron stable/victoria: OVN: Add support for DHCP option "domain-search" for IPv4 https://review.opendev.org/c/openstack/neutron/+/865010 | 10:21 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron stable/ussuri: OVN: Add support for DHCP option "domain-search" for IPv4 https://review.opendev.org/c/openstack/neutron/+/865011 | 10:21 |
| opendevreview | Lucas Alvares Gomes proposed openstack/networking-ovn stable/train: OVN: Add support for DHCP option "domain-search" for IPv4 https://review.opendev.org/c/openstack/networking-ovn/+/865013 | 10:26 |
| opendevreview | Merged openstack/neutron stable/zed: Since OVN 20.06, config is stored in "Chassis.other_config" https://review.opendev.org/c/openstack/neutron/+/863280 | 10:31 |
| opendevreview | Felix Huettner proposed openstack/neutron stable/yoga: Since OVN 20.06, config is stored in "Chassis.other_config" https://review.opendev.org/c/openstack/neutron/+/863281 | 11:05 |
| opendevreview | Felix Huettner proposed openstack/neutron stable/yoga: Since OVN 20.06, config is stored in "Chassis.other_config" https://review.opendev.org/c/openstack/neutron/+/863281 | 11:06 |
| ralonsoh | slaweq, https://bugs.launchpad.net/neutron/+bug/1996836 | 11:43 |
| ralonsoh | should I mark it as fix merged? | 11:43 |
| ralonsoh | the bug is tracking neutron but the patch is on n-lib | 11:43 |
| slaweq | ralonsoh: no, it's not fixed with patch from gmann :/ | 11:48 |
| slaweq | I'm actually working on it now | 11:48 |
| slaweq | and trying to understand what's wrong there | 11:48 |
| ralonsoh | slaweq, ahhh ok, thanks for the clarification | 11:49 |
| slaweq | np | 11:49 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron stable/xena: OVN: Add support for DHCP option "domain-search" for IPv4 https://review.opendev.org/c/openstack/neutron/+/865008 | 11:54 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron stable/wallaby: OVN: Add support for DHCP option "domain-search" for IPv4 https://review.opendev.org/c/openstack/neutron/+/865009 | 11:54 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron stable/victoria: OVN: Add support for DHCP option "domain-search" for IPv4 https://review.opendev.org/c/openstack/neutron/+/865010 | 11:55 |
| opendevreview | Lucas Alvares Gomes proposed openstack/neutron stable/ussuri: OVN: Add support for DHCP option "domain-search" for IPv4 https://review.opendev.org/c/openstack/neutron/+/865011 | 11:55 |
| opendevreview | Lucas Alvares Gomes proposed openstack/networking-ovn stable/train: OVN: Add support for DHCP option "domain-search" for IPv4 https://review.opendev.org/c/openstack/networking-ovn/+/865013 | 11:56 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add an env variable "PROCESS_TAG" in ``ProcessManager`` https://review.opendev.org/c/openstack/neutron/+/865018 | 12:34 |
| ralonsoh | slaweq, https://bugs.launchpad.net/neutron/+bug/1997025 | 12:41 |
| ralonsoh | pffff we have merge something that is breaking 90% of the times this test | 12:41 |
| ralonsoh | https://zuul.opendev.org/t/openstack/builds?job_name=neutron-ovs-tempest-multinode-full&skip=0 | 12:41 |
| ralonsoh | actually nothing in master for the last two days (apart from the doc commit) | 12:42 |
| ykarel | ralonsoh, may be the libraries releases causing it | 12:53 |
| ykarel | i see os-vif released yesterday | 12:53 |
| ykarel | https://github.com/openstack/os-vif/compare/2.8.0...3.1.0 | 12:54 |
| ykarel | https://review.opendev.org/c/openstack/neutron/+/837780 looks related | 13:05 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Add support for HM on a fullypopulated load balancers https://review.opendev.org/c/openstack/ovn-octavia-provider/+/865003 | 13:46 |
| opendevreview | Elvira García Ruiz proposed openstack/neutron master: Fix behaviour of enable/disable in OVN network log https://review.opendev.org/c/openstack/neutron/+/864152 | 13:54 |
| ralonsoh | ykarel, hmmm you are right, I was looking at this repo now | 14:01 |
| *** dmellado_ is now known as dmellado | 14:03 | |
| ralonsoh | ykarel, I think you are right, https://review.opendev.org/c/openstack/requirements/+/864687 | 14:07 |
| ralonsoh | 3.1.0 was releases 1 d and 11 hrs ago | 14:07 |
| ralonsoh | just when the errors started | 14:07 |
| ykarel | ralonsoh, also the issue looks not consistent but happening most of the time | 14:08 |
| ralonsoh | yes, this is not happening 100% of the times, but too often | 14:08 |
| *** dasm|off is now known as dasm | 14:09 | |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Block os-vif<3.0.0 because of an issue with trunk and live-migration https://review.opendev.org/c/openstack/neutron/+/865026 | 14:21 |
| ykarel | ralonsoh, but doesn't ^ require update in openstack/requirements upper-constraints and global requirements? | 14:23 |
| ralonsoh | ykarel, if Neutron project is limiting the version, that should work in neutron-ovs-tempest-multinode-full | 14:25 |
| ralonsoh | this is a temporary patch until we solve this issue | 14:25 |
| ykarel | ralonsoh, ohkk if that works then fine, but iirc we using upper-constraints in those job and pip will try to use that version only and due to <3.0.0 it will raise conflict | 14:26 |
| ralonsoh | yes, once we have all fixed, we'll block those versions in requirements | 14:26 |
| ralonsoh | ah no | 14:27 |
| ralonsoh | but it will also use neutron requirements file when installing | 14:27 |
| lajoskatona | Happy Friday? | 14:27 |
| ykarel | ralonsoh, ok it failed https://ec90f02bffde618895a2-a4b7caf30d0c3c6b54d6e7d8d6b7808b.ssl.cf1.rackcdn.com/865026/1/check/openstack-tox-cover/e11af02/job-output.txt | 14:27 |
| ralonsoh | yeah, I saw it | 14:27 |
| ralonsoh | ykarel, how is using os-vif==3.1.0 ? | 14:29 |
| ykarel | ralonsoh, due to upper-constraints | 14:29 |
| ralonsoh | ykarel, no no, this is the maximum version to be used | 14:29 |
| ykarel | https://github.com/openstack/requirements/blob/master/upper-constraints.txt#L500 | 14:29 |
| ralonsoh | but not the version to be installed | 14:29 |
| ykarel | that file is maintained so all projects use common set of versions of libraries | 14:30 |
| ralonsoh | I'll propose a requirements patch now | 14:32 |
| ykarel | k | 14:34 |
| ralonsoh | #link https://review.opendev.org/c/openstack/requirements/+/864974 | 14:36 |
| ralonsoh | lajoskatona, thanks for proposing this topic | 14:42 |
| ralonsoh | now we know how to do it | 14:42 |
| ykarel | ralonsoh, i think also need to update update global-requirements.txt to block those bad versions | 14:47 |
| ralonsoh | ykarel, right | 14:47 |
| ralonsoh | ykarel, but this patch is reverting the max version | 14:48 |
| ralonsoh | nobody can exceed the upper-constraints version | 14:48 |
| ralonsoh | and I would prefer for now just reverting the patch | 14:48 |
| ralonsoh | mlavalle, we have a problem with trunk ports and live migraiton | 14:49 |
| ralonsoh | most probably is due to https://review.opendev.org/c/openstack/os-vif/+/841499 | 14:49 |
| ralonsoh | I've pushed a patch to revert the os-vif bump to 3.1.0 | 14:49 |
| ralonsoh | mlavalle, my question is: do we need https://review.opendev.org/c/openstack/neutron/+/837780? | 14:49 |
| ykarel | ralonsoh, os-vif!=1.8.0,!=1.12.0,!=3.0.0 | 14:49 |
| ykarel | in global requirements, and now we need to include 3.1.0 too | 14:50 |
| ralonsoh | ykarel, but not for now | 14:50 |
| ykarel | with this i know we hit it with 3.0.0 too | 14:50 |
| ykarel | https://github.com/openstack/requirements/commit/ce3bd6830a51382c8d53989ab174572b177a903d | 14:50 |
| ralonsoh | this revert will block any version above 2.8.0 | 14:50 |
| ralonsoh | once we solve this issue and if we release a new os-vif version | 14:50 |
| ralonsoh | will block 3.1.0 too | 14:50 |
| ralonsoh | but with the revert this is not needed for now | 14:51 |
| ykarel | if requirements team is ok without it i am fine too | 14:51 |
| mlavalle | ralonsoh: as you might remember, I've been trying to remove some hard coded waits with trunk bridges. But for now, we can revert that patch and I'll come back to it later on | 14:52 |
| ralonsoh | mlavalle, but instead of reverting it, should we continue with https://review.opendev.org/c/openstack/neutron/+/837780 | 14:52 |
| ralonsoh | ? | 14:52 |
| mlavalle | ralonsoh: yes, that is the ideal solution, although I cannot do it right now. It's going to take me some time | 14:53 |
| ralonsoh | mlavalle, ok, because this is blocking the CI severely, I'll continue with the patch | 14:57 |
| ralonsoh | to avoid reverting a library bump | 14:57 |
| mlavalle | ralonsoh: ok | 14:58 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Avoid race condition when deleting trunk bridges https://review.opendev.org/c/openstack/neutron/+/837780 | 15:00 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Add support for HM on a fullypopulated load balancers https://review.opendev.org/c/openstack/ovn-octavia-provider/+/865003 | 15:03 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: Remove policy rule for get_network:router:external https://review.opendev.org/c/openstack/neutron/+/865032 | 15:09 |
| slaweq | gmann: ralonsoh ^^ this should fix problem with S-RBAC and router:external attribute of network | 15:10 |
| slaweq | but for security group it is different issue | 15:10 |
| slaweq | I will open another LP for that failed test | 15:10 |
| slaweq | gmann: ralonsoh: https://bugs.launchpad.net/neutron/+bug/1997089 | 15:13 |
| ralonsoh | slaweq, looking now | 15:13 |
| slaweq | ralonsoh: no rush, lets wait for whole CI results :) | 15:14 |
| ralonsoh | for sure | 15:14 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [doc] Metadata service supports IPv6 networks https://review.opendev.org/c/openstack/neutron/+/865036 | 15:27 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [DNM] Testing patch https://review.opendev.org/c/openstack/neutron/+/865039 | 15:32 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron master: ovn: first tear down old metadata namespaces, then deploy new https://review.opendev.org/c/openstack/neutron/+/864777 | 15:34 |
| opendevreview | Slawek Kaplonski proposed openstack/neutron master: WIP Don't check policy rules for admin users https://review.opendev.org/c/openstack/neutron/+/865040 | 16:05 |
| slaweq | gmann: ralonsoh: ^^ this should be solution for the SG issue with S-RBAC | 16:06 |
| slaweq | but I'm not really sure if that is good approach - please check description of the patch and tell me what You think about it | 16:06 |
| slaweq | I'm done for today | 16:06 |
| ralonsoh | slaweq, see you! | 16:07 |
| opendevreview | Fernando Royo proposed openstack/ovn-octavia-provider master: Add support for HM on a fullypopulated load balancers https://review.opendev.org/c/openstack/ovn-octavia-provider/+/865003 | 16:07 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Avoid race condition when deleting trunk bridges https://review.opendev.org/c/openstack/neutron/+/837780 | 16:26 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: [DNM] Testing patch https://review.opendev.org/c/openstack/neutron/+/865039 | 16:29 |
| gmann | slaweq: thanks a lot, I will check those. | 18:19 |
| opendevreview | Brian Haley proposed openstack/neutron master: Fix some pylint indentation warnings https://review.opendev.org/c/openstack/neutron/+/865061 | 21:53 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron stable/zed: ovn: first tear down old metadata namespaces, then deploy new https://review.opendev.org/c/openstack/neutron/+/864977 | 22:08 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron stable/yoga: ovn: first tear down old metadata namespaces, then deploy new https://review.opendev.org/c/openstack/neutron/+/864978 | 22:08 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron stable/xena: ovn: first tear down old metadata namespaces, then deploy new https://review.opendev.org/c/openstack/neutron/+/864979 | 22:09 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron stable/wallaby: ovn: first tear down old metadata namespaces, then deploy new https://review.opendev.org/c/openstack/neutron/+/864980 | 22:09 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron stable/victoria: ovn: first tear down old metadata namespaces, then deploy new https://review.opendev.org/c/openstack/neutron/+/865064 | 22:13 |
| opendevreview | Ihar Hrachyshka proposed openstack/neutron stable/ussuri: ovn: first tear down old metadata namespaces, then deploy new https://review.opendev.org/c/openstack/neutron/+/864981 | 22:14 |
| *** dasm is now known as dasm|off | 22:49 | |
| opendevreview | Merged openstack/neutron master: Remove policy rule for get_network:router:external https://review.opendev.org/c/openstack/neutron/+/865032 | 22:53 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!