ykarel | ralonsoh, hi | 07:29 |
---|---|---|
ykarel | pyroute2 0.7.1 broke some jobs, /me creating bug | 07:29 |
ykarel | ex: https://37a49967c371d50badcf-d0788a990c172de672e24591209c14b2.ssl.cf5.rackcdn.com/849122/8/check/neutron-functional-with-uwsgi/1971bae/testr_results.html | 07:30 |
ykarel | https://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_ce1/849122/8/check/neutron-ovs-tempest-dvr-ha-multinode-full/ce10281/testr_results.html | 07:32 |
ykarel | reported https://bugs.launchpad.net/neutron/+bug/1981963 | 07:44 |
ykarel | can you check, or meanwhile we can revert until the issue get's fixed | 07:48 |
ykarel | https://review.opendev.org/c/openstack/requirements/+/850034 | 07:53 |
lajoskatona | ykarel: thanks | 07:56 |
lajoskatona | ykarel: last week ralonsoh warned me for the pyroute2 issue: https://meetings.opendev.org/irclogs/%23openstack-neutron/%23openstack-neutron.2022-07-14.log.html#t2022-07-14T08:02:37 | 07:57 |
ykarel | lajoskatona, ohkk but that's issue is just for switching to single package | 07:59 |
lajoskatona | ykarel: ohh, I see the bug for something else | 08:00 |
ralonsoh | ykarel, let me check first what could happen before reverting | 09:22 |
opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: ovs: make vlanmanager to handle more vlan mapping per network https://review.opendev.org/c/openstack/neutron/+/839479 | 09:45 |
opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: ovs: use a local vlan per network/segmentation https://review.opendev.org/c/openstack/neutron/+/840415 | 09:45 |
opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp/rpc: retrieve network details with segments https://review.opendev.org/c/openstack/neutron/+/840416 | 09:45 |
opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: utils: add option to generate dhcp device id per segmentation https://review.opendev.org/c/openstack/neutron/+/840417 | 09:45 |
opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: ml2/mech: bind port on segment that subnet belongs https://review.opendev.org/c/openstack/neutron/+/840418 | 09:45 |
opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp: make device manager to clean only not used devices https://review.opendev.org/c/openstack/neutron/+/840419 | 09:45 |
opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp: add/use cleanup stale devices API https://review.opendev.org/c/openstack/neutron/+/840420 | 09:45 |
opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp: support multiple segmentations per network https://review.opendev.org/c/openstack/neutron/+/840421 | 09:45 |
opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: db: add segment_index to the unique constraint https://review.opendev.org/c/openstack/neutron/+/841459 | 09:45 |
opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: segment: enable multisegments support for host https://review.opendev.org/c/openstack/neutron/+/841838 | 09:45 |
opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: ovs: improve log message when ofport is not configured https://review.opendev.org/c/openstack/neutron/+/850147 | 09:45 |
ykarel | ralonsoh, ok please check, i have proposed the revert but yes we can hold taht | 09:49 |
opendevreview | Merged openstack/neutron master: Doc: make the contributor guide more visible https://review.opendev.org/c/openstack/neutron/+/849530 | 11:06 |
*** sfinucan is now known as stephenfin | 11:30 | |
opendevreview | Lajos Katona proposed openstack/networking-bagpipe master: Remove "distutils" library https://review.opendev.org/c/openstack/networking-bagpipe/+/842369 | 11:39 |
opendevreview | Lajos Katona proposed openstack/networking-bagpipe master: py310: Add required-projects list to py310 job https://review.opendev.org/c/openstack/networking-bagpipe/+/841917 | 11:40 |
opendevreview | Lajos Katona proposed openstack/networking-bagpipe master: Remove "distutils" library https://review.opendev.org/c/openstack/networking-bagpipe/+/842369 | 11:40 |
opendevreview | Dmitriy Rabotyagov proposed openstack/neutron-vpnaas master: [doc] Fix configuration files naming https://review.opendev.org/c/openstack/neutron-vpnaas/+/850154 | 12:07 |
*** dasm|off is now known as dasm|ruck | 13:43 | |
lajoskatona | ralonsoh: Hi, regarding https://bugs.launchpad.net/neutron/+bug/1981963 | 13:51 |
lajoskatona | ralonsoh: as I see this pr change how ip rule works: https://github.com/svinota/pyroute2/commit/511ddadbc92e0328daed3606bd01ea77fe3cf192 | 13:51 |
ralonsoh | So far I've seen two errors | 13:51 |
ralonsoh | yes | 13:51 |
ralonsoh | this is one | 13:51 |
ralonsoh | we need the table and we can change that in Neutron | 13:52 |
ralonsoh | the second one is in ip neigh | 13:52 |
lajoskatona | ralonsoh: in my testenv if I changed back how RuleIPRouteFilter logic works to how IPRuleRequest worked I fixed few fdailures | 13:52 |
ralonsoh | the ipv6 dump command does not work and this is a bug in pyroute2 | 13:52 |
lajoskatona | ralonsoh: cool | 13:52 |
ralonsoh | I'll report that to pyroute2 | 13:52 |
ralonsoh | and I'll push a patch for Neutron | 13:52 |
ralonsoh | but we need to (1) revert the pyroute bump | 13:52 |
ralonsoh | (2) blacklist this version in a future | 13:53 |
ralonsoh | (and (3) open the pyroute2 bug) | 13:53 |
ralonsoh | I'll do it now | 13:53 |
lajoskatona | ralonsoh: ack, to tell the truth I don't know if the original logic was good or bad :-) so now we can adopt Neutron code to it, but better to have an issue in pyroute2 also | 13:53 |
opendevreview | Merged openstack/neutron stable/xena: ovn: Wait for northd in functional tests https://review.opendev.org/c/openstack/neutron/+/849673 | 13:55 |
opendevreview | Merged openstack/neutron stable/yoga: Add release note for OVN "requested-chassis" feature https://review.opendev.org/c/openstack/neutron/+/849819 | 13:55 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add a default table in "ip rule" command https://review.opendev.org/c/openstack/neutron/+/850223 | 13:56 |
opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: ovs: improve log message when ofport is not configured https://review.opendev.org/c/openstack/neutron/+/850147 | 14:14 |
opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: ovs: make vlanmanager to handle more vlan mapping per network https://review.opendev.org/c/openstack/neutron/+/839479 | 14:14 |
opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: ovs: use a local vlan per network/segmentation https://review.opendev.org/c/openstack/neutron/+/840415 | 14:14 |
opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp/rpc: retrieve network details with segments https://review.opendev.org/c/openstack/neutron/+/840416 | 14:14 |
opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: utils: add option to generate dhcp device id per segmentation https://review.opendev.org/c/openstack/neutron/+/840417 | 14:14 |
opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: ml2/mech: bind port on segment that subnet belongs https://review.opendev.org/c/openstack/neutron/+/840418 | 14:14 |
opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp: make device manager to clean only not used devices https://review.opendev.org/c/openstack/neutron/+/840419 | 14:14 |
opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp: add/use cleanup stale devices API https://review.opendev.org/c/openstack/neutron/+/840420 | 14:14 |
opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: dhcp: support multiple segmentations per network https://review.opendev.org/c/openstack/neutron/+/840421 | 14:14 |
opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: db: add segment_index to the unique constraint https://review.opendev.org/c/openstack/neutron/+/841459 | 14:14 |
opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: segment: enable multisegments support for host https://review.opendev.org/c/openstack/neutron/+/841838 | 14:14 |
opendevreview | Sahid Orentino Ferdjaoui proposed openstack/neutron master: ovs: add fdb_entries details to the logs https://review.opendev.org/c/openstack/neutron/+/850226 | 14:15 |
amoralej | lajoskatona, i see networking-l2gw doesn't have stable/xena or yoga branches. Should master branch work in previous releases? | 14:41 |
lajoskatona | amoralej: hi, yes it should, but there was no content, and I haven't cut out new branch or added tag | 14:42 |
amoralej | apparently https://review.opendev.org/c/x/networking-l2gw/+/845141 broke compatibility | 14:42 |
amoralej | from yoga testing https://logserver.rdoproject.org/openstack-periodic-integration-main/review.rdoproject.org/rdoinfo/master/weirdo-yoga-promote-puppet-scenario004-centos-stream-9/27d5018/logs/weirdo-project/logs/neutron/server.txt.gz | 14:42 |
amoralej | AttributeError: module 'neutron.common.config' has no attribute 'register_common_config_options' | 14:43 |
amoralej | i guess it's related to that review | 14:43 |
lajoskatona | amoralej: yeah, it was a recent Neutron change that pulled that, so for zed there should be a branch and release | 14:43 |
opendevreview | Rodolfo Alonso proposed openstack/neutron master: Add a default table in "ip rule" command https://review.opendev.org/c/openstack/neutron/+/850223 | 14:43 |
amoralej | ack, lajoskatona you plan to create the branch soon? otherwise i can pin the package in RDO to the last known-good commit | 14:44 |
lajoskatona | amoralej: if you have this Neutron change: https://review.opendev.org/c/openstack/neutron/+/837392 , it will work | 14:44 |
amoralej | lajoskatona, that's master, i'm talking about yoga | 14:44 |
amoralej | it fails with yoga, that's what i meant | 14:45 |
lajoskatona | amoralej: I am not sure what would be the best for these low activity projects, to release and cut branch anyway even if there's no content, or just if there's something.... | 14:45 |
lajoskatona | amoralej: for yoga neutron you can use the last l2gw branch wallaby than, sorry for the issue | 14:46 |
amoralej | from RDO (that's my context) we prefer to release, cut and branch, but we can also pin a package to a specific commit. however it will need manual action in case a branch is created later to push patches for stable releases | 14:46 |
amoralej | uh, pinning a release to a branch for another, previous release, that's new for us :), but can be done | 14:48 |
lajoskatona | amoralej: perhaps that's what I will do in next releases to have in sync with core openstack/neutron | 14:48 |
amoralej | that'd be good! | 14:49 |
opendevreview | Merged openstack/python-neutronclient master: Add support to floating ip port forwarding https://review.opendev.org/c/openstack/python-neutronclient/+/837725 | 14:57 |
*** dkehn_ is now known as dkehn | 15:01 | |
opendevreview | Merged openstack/neutron stable/yoga: ovn: use requested-chassis list format for live migration https://review.opendev.org/c/openstack/neutron/+/848845 | 16:08 |
*** amoralej is now known as amoralej|off | 16:16 | |
opendevreview | Merged openstack/ovn-octavia-provider master: Fix IPv6 member delete on batch operation https://review.opendev.org/c/openstack/ovn-octavia-provider/+/849713 | 17:22 |
*** dasm|ruck is now known as dasm|off | 21:27 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!