Thursday, 2020-07-30

« Wednesday, 2020-07-29 Index Friday, 2020-07-31 »
*** mlavalle has quit IRC00:29
openstackgerritMerged openstack/neutron master: Remove "six" library  https://review.opendev.org/74051000:36
*** xiaolin has joined #openstack-neutron00:52
openstackgerritNing Yao proposed openstack/neutron-tempest-plugin master: fix firewall group deletion occasionally failed in fwaas tests  https://review.opendev.org/74368201:19
openstackgerritFlavio Fernandes proposed openstack/neutron master: [ovn]: port forwarding -- Functional test for IDL changes  https://review.opendev.org/74386201:44
openstackgerritMerged openstack/neutron master: Use "replace" to remove a section of a string  https://review.opendev.org/74357702:37
openstackgerritTerry Wilson proposed openstack/neutron master: Cosmetic: Change ._ovn to ._nb_ovn to match ._sb_ovn  https://review.opendev.org/71876703:12
openstackgerritTerry Wilson proposed openstack/ovsdbapp stable/ussuri: Add indexed column support to ovsdbapp  https://review.opendev.org/72935903:15
*** sapd1 has joined #openstack-neutron03:21
*** ramishra has joined #openstack-neutron03:52
*** gthiemonge has quit IRC04:02
*** gthiemonge has joined #openstack-neutron04:03
*** ociuhandu has joined #openstack-neutron04:22
*** ociuhandu has quit IRC04:26
openstackgerritMerged openstack/neutron master: Change some tempest tests to explicitly use ML2/OVS  https://review.opendev.org/74096804:29
*** takamatsu has quit IRC04:38
*** takamatsu has joined #openstack-neutron04:38
*** ramishra has quit IRC04:59
*** ramishra has joined #openstack-neutron05:04
*** ratailor has joined #openstack-neutron05:11
*** ramishra has quit IRC05:14
*** ramishra has joined #openstack-neutron05:15
*** armax has quit IRC05:18
*** gcheresh has joined #openstack-neutron05:29
*** sapd1 has quit IRC05:32
*** lajoskatona has joined #openstack-neutron05:39
*** ralonsoh has joined #openstack-neutron05:43
*** lpetrut has joined #openstack-neutron06:22
*** Luzi has joined #openstack-neutron06:27
*** ociuhandu has joined #openstack-neutron06:30
*** ociuhandu has quit IRC06:35
*** lajoskatona has quit IRC06:51
*** maciejjozefczyk has joined #openstack-neutron06:52
*** sapd1 has joined #openstack-neutron07:04
*** lajoskatona has joined #openstack-neutron07:04
*** bengates has joined #openstack-neutron07:07
*** slaweq has joined #openstack-neutron07:09
*** tesseract has joined #openstack-neutron07:14
*** rcernin has quit IRC07:32
*** ociuhandu has joined #openstack-neutron07:39
openstackgerritThomas Goirand proposed openstack/neutron master: [WIP] [PoC] Floating IP's for routed networks  https://review.opendev.org/66939507:51
*** jpena|off is now known as jpena07:56
*** lucasagomes has joined #openstack-neutron08:08
*** jlibosva has joined #openstack-neutron08:09
openstackgerritMerged openstack/neutron stable/ussuri: ovn migration: Support stack name  https://review.opendev.org/74369308:19
*** markvoelker has joined #openstack-neutron08:20
*** KeithMnemonic has quit IRC08:23
openstackgerritJakub Libosvar proposed openstack/neutron master: Add George framework and tests  https://review.opendev.org/69692608:23
*** markvoelker has quit IRC08:25
*** rcernin has joined #openstack-neutron08:28
openstackgerritLucas Alvares Gomes proposed openstack/neutron stable/ussuri: Use "replace" to remove a section of a string  https://review.opendev.org/74392508:33
*** rcernin has quit IRC08:34
*** ccamposr has joined #openstack-neutron08:40
*** spatel has joined #openstack-neutron08:58
*** dtantsur|afk is now known as dtantsur09:03
*** spatel has quit IRC09:04
*** jangutter_ has joined #openstack-neutron09:11
*** jangutter has quit IRC09:13
openstackgerritLucas Alvares Gomes proposed openstack/networking-ovn stable/train: Use "replace" to remove a section of a string  https://review.opendev.org/74393709:30
*** stingrayza has quit IRC09:35
*** jangutter_ is now known as jangutter09:39
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron master: Do not skip ports with ofport unset or invalid  https://review.opendev.org/64025809:39
bcafarellajoskatona: o/ can you check last comment in https://review.opendev.org/#/c/725533 ? is that mock dep needed for functional tests in ussuri?09:42
lajoskatonabcafarel: Hi, in ussuri it is needed09:43
lajoskatonabcafarel: mock was removed in this cycle as I remember09:43
bcafarellajoskatona: ok that was my understanding too but I preferred to have confirmation :)09:44
openstackgerritMaciej Józefczyk proposed openstack/neutron master: [DNM] Test OVN multinode with release branch  https://review.opendev.org/74049109:45
*** jangutter has quit IRC09:45
*** jangutter has joined #openstack-neutron09:46
lajoskatonabcafarel: you are right, it's better to have it written09:46
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron master: Do not skip ports with ofport unset or invalid  https://review.opendev.org/64025809:52
slaweqralonsoh: hi10:03
ralonsohhi10:03
slaweqralonsoh: do we still allow contract db changes in alembic migration?10:03
*** ataraday_ has joined #openstack-neutron10:03
ralonsohnope10:03
slaweqlike e.g. deletion of the columns in db10:03
slaweqralonsoh: that's what I thought, because I'm reviewing spec https://review.opendev.org/#/c/739549/2/specs/victoria/port-forwarding-port-ranges.rst now and it has such db migration proposed10:04
ralonsohslaweq, since Ocata we don't allow contractions10:05
ralonsohbut I don't have a good reason to support this statement10:05
slaweqralonsoh: that's what I thought (but I didn't remember since when it was)10:05
ataraday_Hello everyone! I've got functional/dvsm-functional tests failing on my devstack master/stable/ussuri with http://paste.openstack.org/show/796455/ Have someone seen something like this?10:07
ataraday_ralonsoh, slaweq contract migrations support was dropped to allow rolling upgrades with no downtime as far as I remember10:12
lajoskatonaataraday_: Hi, have you executed tools/configure_for_func_testing.sh? As I remember that do these preparations for fullstack10:13
ralonsohataraday_, yes but we can have both, leaving the non-safe changes in the contract branches10:13
ralonsohataraday_, let me check this error10:14
ralonsohlajoskatona, that happens even after executing the script10:14
ataraday_lajoskatona, but should not tox -e functional/dsvm-functional run all required preparation steps?10:15
openstackgerritFlavio Fernandes proposed openstack/neutron master: [ovn]: port forwarding -- Functional test for IDL changes  https://review.opendev.org/74386210:16
*** ociuhandu_ has joined #openstack-neutron10:17
*** ociuhandu has quit IRC10:19
*** ociuhandu_ has quit IRC10:21
*** ociuhandu has joined #openstack-neutron10:22
*** dtantsur is now known as dtantsur|brb10:23
*** ociuhandu has quit IRC10:27
*** psahoo has joined #openstack-neutron10:28
ralonsohataraday_, download OVN and OVS10:35
ralonsohhttps://github.com/ovn-org/ovn10:35
ralonsohhttps://github.com/openvswitch/ovs10:36
ralonsohand export OVS_SRCDIR and OVN_SRCDIR10:36
*** rcernin has joined #openstack-neutron10:37
lajoskatonaataraday_: no, check the guide for it: https://docs.openstack.org/neutron/latest/contributor/testing/testing.html#id310:43
lajoskatonaralonsoh: configure_for_func_testing doesn't do some ovs/ovn setup as well? That was my understanding previously10:44
ralonsohlajoskatona, it should, I'll check it10:44
ralonsohbut I've tested executing FTs in a system using OVS mech driver, not OVN, and I had the same error10:45
ralonsohovsvenv = venv.OvsOvnVenvFixture(tempfile.mkdtemp(),10:45
ralonsoh                                     ovsdir=os.getenv('OVS_SRCDIR'),10:45
ralonsoh                                     ovndir=os.getenv('OVN_SRCDIR'),10:45
ralonsoh                                     remove=not bool(os.getenv('KEEP_VENV')))10:45
ralonsohin ovsdbapp10:45
ralonsohthat needs the OVN and OVS src dirs10:45
*** rcernin has quit IRC10:54
ataraday_lajoskatona, but there is mentioned that tox -e functional should work without it10:56
ataraday_and yes, even running it before - does not help10:57
ataraday_ralonsoh, Thanks! I will check this, although I would expect that it should be done automatically10:58
lajoskatonaataraday_: than I don't know what zuul does for every runk as extra, and what I have done ~1 year ago that my functional/fullstack env works now :-)10:58
*** ociuhandu has joined #openstack-neutron11:00
openstackgerritvinay harsha mitta proposed openstack/neutron master: Supports Pagination and Sorting for agent List  https://review.opendev.org/74395311:05
openstackgerritvinay harsha mitta proposed openstack/neutron master: Supports Pagination and Sorting for agent List  https://review.opendev.org/74395311:12
*** markvoelker has joined #openstack-neutron11:19
*** raildo has joined #openstack-neutron11:25
*** markvoelker has quit IRC11:27
*** jpena is now known as jpena|lunch11:29
*** elod is now known as elod_afk11:42
*** ociuhandu has quit IRC11:44
*** sapd1 has quit IRC11:46
*** spatel has joined #openstack-neutron11:52
zigoralonsoh: Around?11:52
zigoI replied to your comments. I wouldn't be against a few advices on what direction to take.11:53
*** spatel has quit IRC11:56
*** ociuhandu has joined #openstack-neutron11:57
*** ataraday_ has quit IRC11:57
ralonsohzigo, do you have a link?12:09
zigoralonsoh: Still that same patch: https://review.opendev.org/#/c/66939512:12
zigoI wonder why action isn't == 'create' when _validate_segment is called, and if this isn't a mistake somewhere.12:13
*** ociuhandu_ has joined #openstack-neutron12:15
*** psahoo has quit IRC12:18
*** ociuhandu has quit IRC12:18
*** jpena|lunch is now known as jpena12:32
*** psahoo has joined #openstack-neutron12:32
*** lpetrut_ has joined #openstack-neutron12:33
*** lpetrut has quit IRC12:36
*** nweinber has joined #openstack-neutron12:37
ralonsohzigo, because you are not passing action='create'12:37
ralonsohhttps://review.opendev.org/#/c/669395/12/neutron/db/ipam_backend_mixin.py@60112:37
zigoOh, my bad then ! :/12:38
ralonsohhttps://review.opendev.org/#/c/669395/12/neutron/db/ipam_backend_mixin.py@361 is incorrectly formatted12:38
ralonsohyou should add a parenthesis12:38
zigoralonsoh: In def _save_subnet, is the action *always* 'create' ?12:39
zigoralonsoh: Yeah, I know, that part is bad, I fixed it locally.12:39
ralonsohlet me check12:39
ralonsoh_save_subnet is only called from allocate_subnet12:40
ralonsohand this is called when a subnet is created, but that needs to be confirmed12:40
*** tkajinam has quit IRC12:47
zigoRight, thanks a lot, that's helpful.12:48
zigoCurrently, my new unit test fails at https://github.com/openstack/neutron/blob/master/neutron/api/v2/base.py#L74812:49
*** dtantsur|brb is now known as dtantsur12:53
fnordahlAnyone have a spare moment for a second review of https://review.opendev.org/#/c/741410/ ?12:58
openstackgerritThomas Goirand proposed openstack/neutron master: [WIP] [PoC] Floating IP's for routed networks  https://review.opendev.org/66939512:59
zigoOk, it's neutron-lib which is complaining:13:01
zigoUnrecognized attribute(s) 'service_types'13:01
zigo:/13:01
*** ratailor has quit IRC13:07
openstackgerritMerged openstack/networking-odl master: Stop to use the __future__ module.  https://review.opendev.org/73287213:08
openstackgerritThomas Goirand proposed openstack/neutron-lib master: Add DEVICE_OWNER_ROUTED and service_types in subnet.py  https://review.opendev.org/74160713:13
openstackgerritThomas Goirand proposed openstack/neutron-lib master: Add DEVICE_OWNER_ROUTED constant and service_types  https://review.opendev.org/74160713:15
zigoralonsoh: Does this neutron_lib patch makes sense?13:15
ralonsohzigo, no, you can't modify an API13:17
ralonsohyou can extend it creating another extension13:17
ralonsohbut you already have it: ALIAS = 'subnet-service-types'13:17
ralonsohthere is an extension to add this parameter in subnets13:18
zigoAh ...13:18
zigoralonsoh: How does this translates so that attr_ops.verify_attributes(res_dict) doesn't fail?13:19
* zigo is going to revert13:19
openstackgerritRodolfo Alonso Hernandez proposed openstack/neutron master: SimpleInterfaceMonitor filter events by bridge name  https://review.opendev.org/73844613:30
*** sapd1 has joined #openstack-neutron13:42
openstackgerritThomas Goirand proposed openstack/neutron-lib master: Add DEVICE_OWNER_ROUTED constant and service_types  https://review.opendev.org/74160713:51
*** yoctozepto has quit IRC13:54
*** yoctozepto has joined #openstack-neutron13:55
*** psahoo has quit IRC13:55
openstackgerritMaciej Józefczyk proposed openstack/neutron master: [DNM] Test OVN multinode with release branch  https://review.opendev.org/74049114:06
*** Luzi_ has joined #openstack-neutron14:10
*** Luzi has quit IRC14:11
openstackgerritTerry Wilson proposed openstack/ovsdbapp stable/ussuri: Fix pep8 job  https://review.opendev.org/74398314:18
openstackgerritTerry Wilson proposed openstack/ovsdbapp stable/train: Fix pep8 job  https://review.opendev.org/74398414:18
openstackgerritTerry Wilson proposed openstack/ovsdbapp stable/stein: Fix pep8 job  https://review.opendev.org/74398514:18
*** lpetrut_ has quit IRC14:19
*** stingrayza has joined #openstack-neutron14:21
*** elod_afk is now known as elod14:27
*** stingrayza has quit IRC14:29
*** stingrayza has joined #openstack-neutron14:30
*** mlavalle has joined #openstack-neutron14:43
lajoskatonagmann, amotoki: Hi, I think we can revisit networking-l2gw retirement and movement to x/ namespace15:02
gmannlajoskatona: thanks, I will check those15:03
lajoskatonagmann, amotoki: the pike patch to make zuul not execute netowrking-l2gw-tempest tests is merged and on ocata as I see there is no zuul job definition, so there we are safe15:04
gmannlajoskatona: +1, just recheck on https://review.opendev.org/#/c/738458/215:04
lajoskatonagmann: thanks, hope we can finish this change with extra complications :-)15:04
lajoskatonagmann: ----^without15:05
openstackgerritMerged openstack/networking-odl stable/ussuri: This change is a squash of 2 different patches to pass the gate  https://review.opendev.org/72553315:08
gmannlajoskatona: yeah, so once we disable the zuul jobs then we can start the importing to x/namespace(remove the code from openstack/ and add in x/)15:08
openstackgerritLajos Katona proposed openstack/networking-odl stable/ussuri: Update .gitreview for stable/ussuri  https://review.opendev.org/72226315:09
openstackgerritLajos Katona proposed openstack/networking-odl stable/ussuri: Update TOX_CONSTRAINTS_FILE for stable/ussuri  https://review.opendev.org/72226415:09
lajoskatonagmann: thanks15:10
openstackgerritBence Romsics proposed openstack/neutron master: metadata-ipv6: Accept link local address in X-Forwarded-For  https://review.opendev.org/71872915:12
openstackgerritMerged openstack/networking-odl master: Remove neutron-fwaas from the required projects  https://review.opendev.org/73583715:15
mlavalleralonsoh: would you take another look at https://review.opendev.org/#/c/741784/7. I made some small changes15:15
gmannlajoskatona: we need to keep noop job in networking-l2gw-tempest-plugin zuul.yaml to get the patches merged but first let's check if job removal is good15:16
ralonsohmlavalle, sure, right now15:18
mlavalleralonsoh: thanks!15:19
gmannlajoskatona: ah no, its good. we do have pep8 job running on that https://opendev.org/openstack/project-config/src/branch/master/zuul.d/projects.yaml#L335015:22
*** armax has joined #openstack-neutron15:23
*** gcheresh has quit IRC15:35
*** Luzi_ has quit IRC15:36
gmannlajoskatona: this passed, https://review.opendev.org/#/c/738458/215:46
gmannlajoskatona: once this is merged, we can move to next step 1. create new repo of networking-l2gw and networking-l2gw-tempest-plugin using import repo of openstack/15:47
gmannlajoskatona: and once import is done when you will have all history and code under x/ then we can remove the code from openstack/15:48
gmannlajoskatona: let me know if you need any help on new project creating with import15:49
*** livelace has joined #openstack-neutron15:52
gmannextra thing you need to set while new repo creation is to set “upstream” field to existing openstack/ repo location15:53
*** obondarev has joined #openstack-neutron15:56
*** obondarev has quit IRC15:57
*** hamalq has joined #openstack-neutron16:03
*** hamalq has quit IRC16:03
openstackgerritRafael Weingärtner proposed openstack/neutron-lib master: Add local-ip-prefix to Neutron metering label rules  https://review.opendev.org/74382816:03
*** hamalq has joined #openstack-neutron16:04
*** bengates has quit IRC16:04
*** gcheresh has joined #openstack-neutron16:08
*** lucasagomes has quit IRC16:09
*** tesseract has quit IRC16:11
*** jlibosva has quit IRC16:16
*** dtantsur is now known as dtantsur|afk16:25
*** gcheresh has quit IRC16:26
*** slaweq has quit IRC16:37
lajoskatonagmann: tomorrow I check if I am educated enough for that :-)16:41
*** ociuhandu_ has quit IRC17:02
*** jpena is now known as jpena|off17:04
*** ociuhandu has joined #openstack-neutron17:09
*** ociuhandu has quit IRC17:14
*** sapd1 has quit IRC17:29
ralonsohbcafarel, hi, quick question if you know that17:31
ralonsohhttps://review.opendev.org/#/c/743807/1/zuul.d/base.yaml17:31
ralonsohwhy NEUTRON_CORE_PLUGIN_CONF is not found?17:31
*** lajoskatona has quit IRC17:42
openstackgerritMerged openstack/ovsdbapp stable/ussuri: Fix pep8 job  https://review.opendev.org/74398317:56
*** ralonsoh has quit IRC18:00
*** gcheresh has joined #openstack-neutron18:02
openstackgerritMerged openstack/neutron master: [ovn]: port forwarding -- documentation  https://review.opendev.org/74095518:09
*** livelace has quit IRC18:11
openstackgerritMerged openstack/ovsdbapp stable/train: Fix pep8 job  https://review.opendev.org/74398418:28
*** gcheresh has quit IRC18:29
*** livelace has joined #openstack-neutron18:36
*** Reepicheep has quit IRC18:41
*** maciejjozefczyk has quit IRC19:13
*** maciejjozefczyk has joined #openstack-neutron19:13
openstackgerritDan Radez proposed openstack/neutron master: Updating the pecan version requirement  https://review.opendev.org/74403519:23
mlavallenjohnston: if you have a chance, would you push this over the edge: https://review.opendev.org/#/c/741784/19:29
mlavalle?19:29
njohnstonmlavalle: Absolutely19:48
mlavallenjohnston: thanks :-)19:48
*** maciejjozefczyk has quit IRC19:58
openstackgerritFlavio Fernandes proposed openstack/neutron master: [ovn]: port forwarding -- core changes  https://review.opendev.org/74130320:06
openstackgerritNate Johnston proposed openstack/neutron master: Add Loki testing for Neutron  https://review.opendev.org/73913920:43
*** ociuhandu has joined #openstack-neutron20:46
openstackgerritFlavio Fernandes proposed openstack/neutron master: [ovn]: port forwarding -- core changes  https://review.opendev.org/74130320:59
openstackgerritFlavio Fernandes proposed openstack/neutron master: [ovn]: port forwarding -- functional tests for maintenance task  https://review.opendev.org/74173121:03
openstackgerritTerry Wilson proposed openstack/ovsdbapp stable/ussuri: Add indexed column support to ovsdbapp  https://review.opendev.org/72935921:04
*** ociuhandu_ has joined #openstack-neutron21:04
*** ociuhandu has quit IRC21:06
openstackgerritFlavio Fernandes proposed openstack/neutron master: [ovn]: port forwarding -- feature support under ovn_db_sync  https://review.opendev.org/74377221:09
*** ociuhandu_ has quit IRC21:10
*** ociuhandu has joined #openstack-neutron21:11
*** livelace has quit IRC21:14
*** nweinber has quit IRC21:20
*** raildo has quit IRC21:29
openstackgerritMerged openstack/neutron-lib master: API definition for SG group rules address groups.  https://review.opendev.org/74178421:31
*** tkajinam has joined #openstack-neutron22:05
openstackgerritFlavio Fernandes proposed openstack/neutron master: [ovn]: port forwarding -- feature support under ovn_db_sync  https://review.opendev.org/74377222:08
openstackgerritFlavio Fernandes proposed openstack/neutron master: [ovn]: port forwarding -- Functional test for IDL changes  https://review.opendev.org/74386222:10
*** Reepicheep has joined #openstack-neutron22:22
*** yangyapeng has quit IRC22:30
*** yangyapeng has joined #openstack-neutron22:34
openstackgerritGhanshyam Mann proposed openstack/os-vif master: [goal] migrate testing to ubuntu focal  https://review.opendev.org/73813022:49
*** rcernin has joined #openstack-neutron23:13
*** rcernin has quit IRC23:14
*** rcernin has joined #openstack-neutron23:14
*** mlavalle has quit IRC23:31
« Wednesday, 2020-07-29 Index Friday, 2020-07-31 »

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!