| opendevreview | weilingfei proposed openstack/neutron master: fix: Fix url https://review.opendev.org/c/openstack/neutron/+/859748 | 05:49 |
|---|---|---|
| opendevreview | weilingfei proposed openstack/neutron master: fix: Fix url of Floodlight https://review.opendev.org/c/openstack/neutron/+/859748 | 05:52 |
| opendevreview | weilingfei proposed openstack/neutron stable/zed: fix: Fix url of Floodlight https://review.opendev.org/c/openstack/neutron/+/859758 | 05:54 |
| opendevreview | weilingfei proposed openstack/neutron stable/yoga: fix: Fix url of Floodlight https://review.opendev.org/c/openstack/neutron/+/859759 | 05:55 |
| opendevreview | weilingfei proposed openstack/neutron stable/xena: fix: Fix url of Floodlight https://review.opendev.org/c/openstack/neutron/+/859760 | 05:55 |
| opendevreview | weilingfei proposed openstack/neutron stable/wallaby: fix: Fix url of Floodlight https://review.opendev.org/c/openstack/neutron/+/859761 | 05:55 |
| opendevreview | weilingfei proposed openstack/neutron stable/victoria: fix: Fix url of Floodlight https://review.opendev.org/c/openstack/neutron/+/859762 | 05:56 |
| opendevreview | weilingfei proposed openstack/neutron stable/ussuri: fix: Fix url of Floodlight https://review.opendev.org/c/openstack/neutron/+/859763 | 05:56 |
| opendevreview | weilingfei proposed openstack/neutron stable/train: fix: Fix url of Floodlight https://review.opendev.org/c/openstack/neutron/+/859764 | 05:56 |
| opendevreview | weilingfei proposed openstack/neutron stable/stein: fix: Fix url of Floodlight https://review.opendev.org/c/openstack/neutron/+/859765 | 05:57 |
| opendevreview | weilingfei proposed openstack/neutron stable/rocky: fix: Fix url of Floodlight https://review.opendev.org/c/openstack/neutron/+/859766 | 05:57 |
| opendevreview | weilingfei proposed openstack/neutron stable/queens: fix: Fix url of Floodlight https://review.opendev.org/c/openstack/neutron/+/859767 | 05:57 |
| frickler | tobias-urdin: is this just general shyness or is there a specific reason you didn't approve https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/858876 ? makes me always wonder what's wrong when there is a second +2 but no W+1 | 06:27 |
| opendevreview | Dr. Jens Harbott proposed openstack/neutron-tempest-plugin master: Bump cirros version to 0.6.0 https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/859774 | 06:45 |
| *** elvira1 is now known as elvira | 06:45 | |
| tobias-urdin | frickler: my bad, I was either shy because it was stable branches I'm usually very cautious there or just plain missed it | 07:14 |
| frickler | tobias-urdin: np, I just thought I'd better ask instead of simply workflowing myself, thx | 07:28 |
| opendevreview | Merged openstack/neutron-dynamic-routing stable/yoga: Add releasenotes for recently added fixes https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/858876 | 07:40 |
| opendevreview | Dr. Jens Harbott proposed openstack/neutron-dynamic-routing stable/xena: Add releasenotes for recently added fixes https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/859791 | 08:23 |
| opendevreview | Dr. Jens Harbott proposed openstack/neutron-dynamic-routing stable/wallaby: Add releasenotes for recently added fixes https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/859792 | 08:23 |
| frickler | tobias-urdin: ralonsoh: ^^ if you have another minute. once these are in, I'll propose releases for those three branches, too | 08:24 |
| tobias-urdin | frickler: ack | 08:34 |
| *** elvira1 is now known as elvira | 08:37 | |
| ralonsoh | sure, i'll check it | 08:41 |
| opendevreview | kiran pawar proposed openstack/neutron-lib master: Use oslo_context.from_dict() for context generation https://review.opendev.org/c/openstack/neutron-lib/+/859813 | 08:46 |
| opendevreview | Merged openstack/neutron-dynamic-routing stable/xena: Add releasenotes for recently added fixes https://review.opendev.org/c/openstack/neutron-dynamic-routing/+/859791 | 09:15 |
| opendevreview | Merged openstack/neutron master: fix: Fix url of Floodlight https://review.opendev.org/c/openstack/neutron/+/859748 | 09:47 |
| opendevreview | Merged openstack/neutron-lib stable/zed: Update TOX_CONSTRAINTS_FILE for stable/zed https://review.opendev.org/c/openstack/neutron-lib/+/856741 | 10:51 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Port provisioning should retry only for VM ports https://review.opendev.org/c/openstack/neutron/+/859645 | 11:13 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Since OVN 22.09, config is stored in "Chassis.other_config" https://review.opendev.org/c/openstack/neutron/+/859642 | 11:29 |
| ralonsoh | lucasagomes, ^^ no rush, but whenever you have sometime, please this patch | 11:30 |
| lajoskatona | tkajinam: Hi, if you have any news regarding networking-l2gw, please share it with me:-) I submitted the patch that adopted it to SQLAlchemy2.0, but I am not sure if my local env and testing together with tempest covered everything (https://review.opendev.org/c/x/networking-l2gw/+/853669 ) | 11:41 |
| opendevreview | Merged openstack/ovn-octavia-provider master: Switch to 2023.1 Python3 unit tests and generic template name https://review.opendev.org/c/openstack/ovn-octavia-provider/+/858147 | 12:12 |
| lucasagomes | ralonsoh, thanks will take a look! | 12:17 |
| lucasagomes | ralonsoh, reviewed. I think we need to update the commit message and some TODOs in the code cause the other_config exists for awhile already | 12:33 |
| lucasagomes | ralonsoh, what happened is that before we were storing the config in both places to be backward compatible | 12:33 |
| lucasagomes | and now core OVN got rid of that backward compatible layer | 12:34 |
| lucasagomes | but other_config exists for some time (since 2020) | 12:34 |
| lucasagomes | ralonsoh, that said, code-wise it looks good. But, maybe u want to reconsider having to use both and just use other_config for now on... | 12:34 |
| *** dasm|off is now known as dasm | 12:48 | |
| ralonsoh | lucasagomes, actually I deployed an env with ovn-nbctl 20.03.0 and I don't have chassis.other_config | 12:56 |
| ralonsoh | this is why I prefer to keep it there for a while | 12:56 |
| ralonsoh | just in case | 12:56 |
| lucasagomes | ralonsoh, right on | 12:56 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Since OVN 20.06, config is stored in "Chassis.other_config" https://review.opendev.org/c/openstack/neutron/+/859642 | 13:10 |
| *** dkehn_ is now known as dkehn | 14:04 | |
| opendevreview | Rodolfo Alonso proposed openstack/neutron stable/xena: [stable-only] Add writer DB context to "add_provisioning_component" https://review.opendev.org/c/openstack/neutron/+/859865 | 14:36 |
| opendevreview | Fernando Royo proposed openstack/neutron master: Check subnet overlapping after add router interface https://review.opendev.org/c/openstack/neutron/+/859143 | 15:11 |
| opendevreview | Rodolfo Alonso proposed openstack/neutron master: Port provisioning should retry only for VM ports https://review.opendev.org/c/openstack/neutron/+/859645 | 15:40 |
| opendevreview | Arkady Shtempler proposed openstack/neutron-tempest-plugin master: Adds use of "Designate API" into "_verify_dns_records" method https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/859871 | 15:44 |
| opendevreview | Merged openstack/neutron master: Add .pylintrc to irrelevant-files for non pep8 jobs https://review.opendev.org/c/openstack/neutron/+/859374 | 16:05 |
| opendevreview | Merged openstack/neutron stable/zed: fix: Fix url of Floodlight https://review.opendev.org/c/openstack/neutron/+/859758 | 16:05 |
| opendevreview | Merged openstack/neutron stable/train: Sync the dns-assignment with the actual designate dns-domain https://review.opendev.org/c/openstack/neutron/+/858466 | 16:42 |
| opendevreview | Merged openstack/neutron stable/stein: Sync the dns-assignment with the actual designate dns-domain https://review.opendev.org/c/openstack/neutron/+/858467 | 16:43 |
| opendevreview | Merged openstack/neutron-tempest-plugin master: Add .pylintrc to irrelevant-files https://review.opendev.org/c/openstack/neutron-tempest-plugin/+/859375 | 17:23 |
| gmann | slaweq: ralonsoh: reminder for operator-hour booking in PTG. please check and book it for neutron if not yet done - https://lists.openstack.org/pipermail/openstack-discuss/2022-September/030301.html | 19:28 |
| opendevreview | Ghanshyam proposed openstack/os-vif stable/stein: Fix zuul config error for os-vif https://review.opendev.org/c/openstack/os-vif/+/859892 | 19:33 |
| opendevreview | Ghanshyam proposed openstack/os-vif stable/rocky: Fix zuul config error for os-vif https://review.opendev.org/c/openstack/os-vif/+/859893 | 19:38 |
| *** lifeless_ is now known as lifeless | 20:14 | |
| ralonsoh | gmann, I'll do it tomorrow | 20:34 |
| ralonsoh | thanks for the heads up | 20:34 |
| gmann | thanks | 20:41 |
| *** dasm is now known as dasm|off | 21:24 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!