Tuesday, 2019-10-01

openstackgerritYAMAMOTO Takashi proposed openstack/networking-midonet stable/stein: Fix versions of neutron and friends  https://review.opendev.org/68579000:12
openstackgerritYAMAMOTO Takashi proposed openstack/networking-midonet stable/stein: Update sphinx requirements  https://review.opendev.org/68586000:12
*** donghao has joined #openstack-neutron00:13
*** goldyfruit_ has joined #openstack-neutron00:14
*** donghao has quit IRC00:17
*** yamamoto has quit IRC00:24
*** markvoelker has quit IRC00:26
*** yamamoto has joined #openstack-neutron00:28
*** yamamoto has quit IRC00:43
*** yamamoto has joined #openstack-neutron00:44
*** yamamoto has quit IRC00:49
*** yamamoto has joined #openstack-neutron00:55
openstackgerritYAMAMOTO Takashi proposed openstack/networking-midonet stable/stein: Add bindep.txt to fix unit tests  https://review.opendev.org/66882401:00
*** dave-mccowan has joined #openstack-neutron01:22
*** igordc has quit IRC01:26
*** lseki has quit IRC01:47
*** hongbin has joined #openstack-neutron01:55
*** yamamoto has quit IRC02:14
*** donghao has joined #openstack-neutron02:15
*** yamamoto has joined #openstack-neutron02:17
*** macz has joined #openstack-neutron02:19
*** donghao has quit IRC02:19
*** macz has quit IRC02:23
*** hongbin has quit IRC02:33
*** hongbin has joined #openstack-neutron02:34
*** BjoernT has joined #openstack-neutron02:47
openstackgerritYAMAMOTO Takashi proposed openstack/networking-midonet stable/stein: Update test_update_provider_net_unsupported test case  https://review.opendev.org/65175002:49
*** yamamoto has quit IRC02:59
*** dave-mccowan has quit IRC03:04
*** BjoernT has quit IRC03:14
*** BjoernT has joined #openstack-neutron03:18
*** nweinber has joined #openstack-neutron03:19
*** BjoernT has quit IRC03:29
*** spsurya has joined #openstack-neutron03:29
*** dsneddon has quit IRC03:31
*** gmann_afk is now known as gmann03:33
*** markvoelker has joined #openstack-neutron03:41
*** markvoelker has quit IRC03:42
*** markvoelker has joined #openstack-neutron03:43
*** yamamoto has joined #openstack-neutron03:44
*** dsneddon has joined #openstack-neutron03:46
*** yamamoto has quit IRC03:49
*** dsneddon has quit IRC03:53
*** gkadam has joined #openstack-neutron03:55
*** hongbin has quit IRC03:56
*** igordc has joined #openstack-neutron03:58
openstackgerritMerged openstack/networking-midonet master: PDF documentation build  https://review.opendev.org/67888003:59
*** gkadam has quit IRC04:08
*** yamamoto has joined #openstack-neutron04:15
*** nweinber has quit IRC04:16
*** donghao has joined #openstack-neutron04:16
*** macz has joined #openstack-neutron04:20
*** yamamoto has quit IRC04:21
*** donghao has quit IRC04:22
*** macz has quit IRC04:25
*** yamamoto has joined #openstack-neutron04:28
*** macz has joined #openstack-neutron04:28
*** dsneddon has joined #openstack-neutron04:30
*** macz has quit IRC04:30
*** macz has joined #openstack-neutron04:30
*** macz has quit IRC04:35
*** ajay33 has joined #openstack-neutron04:35
openstackgerritYAMAMOTO Takashi proposed openstack/networking-midonet stable/stein: Update test_update_provider_net_unsupported test case  https://review.opendev.org/65175004:36
*** spsurya has quit IRC04:37
*** spsurya has joined #openstack-neutron04:38
*** lajoskatona has joined #openstack-neutron04:43
openstackgerritAndreas Jaeger proposed openstack/networking-midonet stable/stein: Remove ocata jobs from branch  https://review.opendev.org/66905904:45
*** SmearedBeard has quit IRC04:45
*** yamamoto_ has joined #openstack-neutron04:49
*** yamamoto has quit IRC04:51
*** SmearedBeard has joined #openstack-neutron04:54
*** pcaruana has joined #openstack-neutron05:01
*** Luzi has joined #openstack-neutron05:02
*** ratailor has joined #openstack-neutron05:07
*** yamamoto_ has quit IRC05:16
*** yamamoto has joined #openstack-neutron05:16
*** gcheresh_ has joined #openstack-neutron05:17
*** markvoelker has quit IRC05:47
*** igordc has quit IRC05:52
*** yamamoto has quit IRC05:57
openstackgerritdharmendra kushwaha proposed openstack/networking-sfc master: PDF documentation build  https://review.opendev.org/67996905:58
*** SmearedBeard has quit IRC06:12
*** donghao has joined #openstack-neutron06:19
*** gcheresh_ has quit IRC06:20
*** donghao has quit IRC06:23
*** sapd1_x has joined #openstack-neutron06:23
openstackgerritdharmendra kushwaha proposed openstack/networking-sfc master: PDF documentation build  https://review.opendev.org/67996906:27
*** lpetrut has joined #openstack-neutron06:31
*** macz has joined #openstack-neutron06:32
*** yamamoto has joined #openstack-neutron06:35
*** macz has quit IRC06:36
*** cshen has joined #openstack-neutron06:41
*** SmearedBeard has joined #openstack-neutron06:42
*** jlibosva has joined #openstack-neutron06:44
*** yamamoto has quit IRC06:47
*** SmearedBeard has quit IRC06:47
*** Smeared_Beard has joined #openstack-neutron06:47
*** dsneddon has quit IRC06:55
*** hongbin has joined #openstack-neutron06:56
*** slaweq has joined #openstack-neutron06:56
*** dsneddon has joined #openstack-neutron06:57
*** hongbin has quit IRC07:00
*** dsneddon has quit IRC07:03
*** rpittau|afk is now known as rpittau07:04
bcafarelslaweq: hi, catching up on meeting backlog don't worry I have my tabs open for bug deputy week :)07:05
bcafarel(and some nice fellow already triaged the first one yesterday)07:05
slaweqbcafarel: hi07:05
slaweqbcafarel: good to know, I just wanted to remind it to You :)07:05
bcafarel:) also indeed all branches for train should be good to go, midonet got a fix in as mentioned and networking-ovn is set with tripleo-ci check temp disabled07:07
bcafarelfor once I was off to bed before the meeting start07:07
*** zbr is now known as zbr|ruck07:08
*** maciejjozefczyk has joined #openstack-neutron07:08
*** awalende has joined #openstack-neutron07:10
*** gdwornicki has joined #openstack-neutron07:11
*** gdwornicki has quit IRC07:12
slaweqbcafarel: thx for confirmation07:13
slaweqabout networking-ovn, this patch is only for CI so it will not require rc2 IMO07:13
slaweqfor midonet patch was for actual code, so we should probably release rc2 with it07:13
*** tesseract has joined #openstack-neutron07:16
*** dtantsur|afk is now known as dtantsur07:23
*** dsneddon has joined #openstack-neutron07:27
bcafarelslaweq: sounds good yep07:27
slaweqamotoki: hi07:33
amotokislaweq: hi07:33
slaweqamotoki: I don't know if You checked logs from last neutron's meeting but it seems that we will need rc2 for networking-midonet, with https://review.opendev.org/#/c/685649/07:33
amotokislaweq: I saw today's meeting log. The PTG plan is fine to me.07:33
slaweqok with PTG, thx for info :)07:34
amotokislaweq: I am aware of the need for RC2 for netwokring-midonet too.07:34
slaweqabout midonet, will this be ok for You to do rc2 next week?07:34
slaweqamotoki: ok, thx a lot07:35
amotokislaweq: is yamamoto fine with the current status?07:35
slaweqAFAIU him on meeting yesterday, he is07:35
amotokislaweq: sounds good. the gate of the midonet looks fine, so we can cut RC2.07:36
*** dsneddon has quit IRC07:36
slaweqamotoki: ok, will You push patch for that?07:37
amotokislaweq: yes, I will push it.07:38
slaweqamotoki: thx a lot07:38
*** luksky has joined #openstack-neutron07:38
*** jpena|off is now known as jpena07:43
*** ralonsoh has joined #openstack-neutron07:45
*** markvoelker has joined #openstack-neutron07:47
*** markvoelker has quit IRC07:51
amotokislaweq: another follow-up of the today's meeting: the onboarding session will be a part of PTG session in Shanghai. The team can assign any slot to it and announce it. http://lists.openstack.org/pipermail/openstack-discuss/2019-June/007079.html07:56
*** jangutter has quit IRC07:56
slaweqamotoki: thx for confirmation :)07:56
slaweqI knew that I saw it somewhere07:56
amotoki:)07:57
*** sapd1_x has quit IRC07:57
gregoiremaheHi, is Rodolfo Alonso around ? :) I'd like to discuss about his review on my change proposal07:57
slaweqgregoiremahe: he should be, ping ralonsoh07:57
slaweq^^07:58
gregoiremahethanks :)07:58
openstackgerritOpenStack Proposal Bot proposed openstack/neutron master: Imported Translations from Zanata  https://review.opendev.org/68521507:59
*** Jeffrey4l has quit IRC07:59
*** Jeffrey4l has joined #openstack-neutron08:00
*** ociuhandu has joined #openstack-neutron08:02
*** rcernin has quit IRC08:02
openstackgerritAkihiro Motoki proposed openstack/networking-midonet master: Correct the PDF filename  https://review.opendev.org/68588908:02
*** rcernin has joined #openstack-neutron08:03
*** tkajinam has quit IRC08:04
*** dsneddon has joined #openstack-neutron08:04
openstackgerritAkihiro Motoki proposed openstack/networking-midonet master: Correct the PDF filename  https://review.opendev.org/68588908:07
*** dsneddon has quit IRC08:09
*** jangutter has joined #openstack-neutron08:14
openstackgerritzhanghao proposed openstack/neutron master: Make the MTU attribute not nullable  https://review.opendev.org/67939908:16
openstackgerritAkihiro Motoki proposed openstack/os-ken master: doc: Improve the document structure  https://review.opendev.org/68255308:19
*** ociuhandu has quit IRC08:21
*** ociuhandu has joined #openstack-neutron08:22
*** yamamoto has joined #openstack-neutron08:26
slaweqamotoki: hi again08:26
amotokislaweq: hi08:26
slaweqamotoki: qq about rc2 and translations08:26
slaweqshould we also release rc2 for neutron when we will merge https://review.opendev.org/#/c/685215/3 and backport it to train?08:27
slaweqor translations don't require rc2?08:27
amotokislaweq: looking at the patch, it is related to exception messages (potentially visible to the API) but they are not actually used now. I don't think we need to backport it.08:28
slaweqamotoki: ok, thx for checking that08:29
amotokislaweq: the translation support in the API is implemented but it is not tested in all back-end services.08:29
openstackgerritMaciej Józefczyk proposed openstack/networking-ovn master: Add OVN Octavia CI jobs  https://review.opendev.org/58971208:30
amotokislaweq: in addition, stable/train branch has been created in the translation site, so we don't need to backport translations.08:30
slaweqamotoki: ahh, ok08:35
slaweqgood to know that :)08:35
slaweq(sorry, I'm still learning many of those things :)008:35
amotokislaweq: it is not easy to know all. I am just familiar with it as I also work on translations.08:36
amotokislaweq: no worries.08:36
slaweqamotoki: :)08:36
*** dsneddon has joined #openstack-neutron08:37
*** macz has joined #openstack-neutron08:51
*** gdwornicki has joined #openstack-neutron08:53
*** macz has quit IRC08:55
openstackgerritMerged openstack/networking-midonet master: Correct the PDF filename  https://review.opendev.org/68588908:58
AJaegerslaweq: no need to backport translations09:00
AJaegerthey get proposed by the bot separately09:00
AJaeger(ah, amotoki said this as well)09:00
*** ociuhandu has quit IRC09:02
bcafarelralonsoh: https://review.opendev.org/#/c/685118/ is not needed for train GA right? it is mostly for gate failure09:06
slaweqAJaeger: thx, amotoki already explained it to me09:09
slaweqbcafarel: it's test only change, so it's not needed before Train release, we can backport it later09:10
ralonsohbcafarel, well, this patch reduces the FTs problems in the CI09:10
ralonsohbcafarel, but not needed for GA09:10
bcafarelthanks for the double confirmation :)09:10
AJaegerslaweq: most teams do single core +2A on translation imports, are you handling that different?09:11
slaweqAJaeger: tbh, I don't know09:13
slaweqNeutron patches which I remember got 2x +209:13
AJaegeramotoki: do you know? ^09:13
AJaegerhttps://review.opendev.org/#/c/651750/ fixes midonet stein branch - thanks, yamamoto !09:14
amotokiAJaeger: slaweq: IIRC we haven't discussed this explicitly. Sometimes two +2 and sometimes single +2.09:14
slaweqamotoki: AJaeger: I can propose update to https://docs.openstack.org/neutron/latest/contributor/policies/code-reviews.html#neutron-code-review-practices to make it clearer09:18
AJaegerAh - in that case, want to +2A 685215 before it gets proposed again? ;)09:18
AJaegerslaweq: you won't get many translations for neutron, dashboards are a prio. Your call ;)09:19
amotokiAJaeger: I pushed it into the gate.09:19
slaweqAJaeger: ok :)09:19
slaweqthx09:19
amotokislaweq: good idea to add a note on this. we can approve with a single +2 on mechanical things like translation imports. let's note "if you have any question or something not confident, feel free to wait another +2."09:21
openstackgerritzhanghao proposed openstack/python-neutronclient master: Remove 'public' and 'private' parameters in fwaas_v2  https://review.opendev.org/68563609:21
slaweqamotoki: ok, I will propose such patch today09:21
amotokislaweq: thanks09:21
*** donghao has joined #openstack-neutron09:21
*** ociuhandu has joined #openstack-neutron09:23
*** ratailor_ has joined #openstack-neutron09:24
*** salmankhan has joined #openstack-neutron09:25
*** donghao has quit IRC09:26
*** ratailor has quit IRC09:26
*** ociuhandu has quit IRC09:28
openstackgerritSlawek Kaplonski proposed openstack/neutron master: Code review policy - add note about approving mechanical changes  https://review.opendev.org/68591709:40
*** tssurya has joined #openstack-neutron09:41
*** ratailor__ has joined #openstack-neutron09:48
*** markvoelker has joined #openstack-neutron09:48
*** ratailor_ has quit IRC09:50
*** markvoelker has quit IRC09:53
openstackgerritAkihiro Motoki proposed openstack/networking-bgpvpn master: PDF documentation build  https://review.opendev.org/68057509:54
*** yamamoto has quit IRC10:02
openstackgerritMaciej Józefczyk proposed openstack/networking-ovn master: Add OVN Octavia CI jobs  https://review.opendev.org/58971210:05
bcafarelslaweq: FYI https://review.opendev.org/#/c/685315/ is a valid bug, I was talking with mjozefcz and it should get in train GA (the revert)10:10
bcafarelrace condition on db update while playing with subports, looks worth merging in stable/train (when zuul+1) and getting a rc2 for it10:11
bcafarel(zuul is almost +1 on the review)10:11
slaweqbcafarel: do we have bug report for it?10:11
openstackgerritMaciej Józefczyk proposed openstack/networking-ovn master: Switch master relase jobs to use OVS 2.12  https://review.opendev.org/68496710:20
openstackgerritMaciej Józefczyk proposed openstack/networking-ovn master: Change passed domain_name parameter value  https://review.opendev.org/68502110:20
openstackgerritMaciej Józefczyk proposed openstack/networking-ovn master: Enable InternalDNS related tests  https://review.opendev.org/68454810:20
*** ociuhandu has joined #openstack-neutron10:20
*** stephenfin has joined #openstack-neutron10:35
stephenfinMorning, all. Say, is there something explicit in neutron that says we have to have at least one security group? A default, if you will10:36
slaweqstephenfin: hi, are You asking about docs? or code?10:36
stephenfincode10:36
slaweqIIRC, You can't remove default security group but You don't need to use it10:37
slaweqYou can have port without security groups at all10:37
stephenfinAh, you can?10:37
slaweqstephenfin: yes, but You have to set port-security to false and than You have no security groups for such port10:38
slaweqif port-security is set to true, You always have to have at least 1 SG for such port10:38
stephenfinslaweq: Hmm, okay, so fair to say it's possible but you'd have to do something different from the default? I ask because of https://review.opendev.org/68592710:39
stephenfinThat nova change is modifying the NeutronFixture we use to mock out Neutron to "properly" mock the security group calls10:40
stephenfinhere https://review.opendev.org/#/c/685927/1/nova/tests/fixtures.py10:40
slaweqstephenfin: yes, IMO it's fair to say that10:41
slaweqby default ports have got port_security enabled and default SG is assigned to such port if there was no other SG specified in request10:42
stephenfinand all the unmocked function is doing is getting all ports for each instance, extracting the security groups from those ports, and using that to build an instance-security groups mapping10:42
slaweqso it will always have some SG10:42
stephenfinsweet10:42
stephenfinthanks very much, slaweq++ :)10:42
slaweqstephenfin: yw10:42
*** macz has joined #openstack-neutron10:52
*** macz has quit IRC10:57
openstackgerritMerged openstack/neutron master: Imported Translations from Zanata  https://review.opendev.org/68521511:00
*** yamamoto has joined #openstack-neutron11:00
openstackgerritAkihiro Motoki proposed openstack/networking-sfc master: PDF documentation build  https://review.opendev.org/67996911:01
*** Smeared_Beard has quit IRC11:02
openstackgerritAkihiro Motoki proposed openstack/networking-sfc master: PDF documentation build  https://review.opendev.org/67996911:05
bcafarelslaweq: sorry had to run just after sending message, I don't think we have a bug reported (yet)11:10
slaweqbcafarel: np, I already talked with maciejjozefczyk about that11:10
bcafarel:)11:10
bcafarelso no need to ping him again to create a bug I suppose11:11
*** dave-mccowan has joined #openstack-neutron11:12
*** donghao has joined #openstack-neutron11:24
*** donghao has quit IRC11:28
*** tssurya has quit IRC11:31
*** tssurya_ has joined #openstack-neutron11:31
*** cshen has quit IRC11:37
*** jpena is now known as jpena|lunch11:38
maciejjozefczykslaweq, bcafarel https://bugs.launchpad.net/networking-ovn/+bug/184547911:38
openstackLaunchpad bug 1845479 in networking-ovn "random failures of neutron_tempest_plugin.scenario.test_trunk.TrunkTest.test_trunk_subport_lifecycle" [Undecided,In progress] - Assigned to Maciej Jozefczyk (maciej.jozefczyk)11:38
maciejjozefczykits there, those reverts are linked to it11:38
bcafarelok, next time I'll check and click on linked bugs before speaking11:40
bcafarelmaciejjozefczyk: can you set its importance? "undecided" currently does not sound too bad11:41
maciejjozefczykbcafarel, I cant, looks like I don't have permissions11:45
*** bobmel has joined #openstack-neutron11:46
*** cshen has joined #openstack-neutron11:46
*** bobmel has quit IRC11:47
*** bobmel has joined #openstack-neutron11:47
openstackgerritMerged openstack/networking-ovn stable/stein: Fix container configuration file path  https://review.opendev.org/68432011:49
bcafarelhmm not sure where permissions are listed in launchpad11:49
*** markvoelker has joined #openstack-neutron11:49
bcafareldalvarez: jlibosva: ^^ can you edit launchpad networking-ovn bus?11:50
*** bobmel has quit IRC11:50
bcafarel*bugs11:50
*** markvoelker has quit IRC11:54
maciejjozefczyk'changable only by a project maintainer or bug supervisor'11:55
njohnstonbcafarel maciejjozefczyk: The list of people with powers is here - https://launchpad.net/~networking-ovn-bugs/+members11:56
maciejjozefczykdalvarez, can you add please us, as you have the power there?11:57
bcafarelnjohnston: ah thanks so this is the "Bug supervisor" group mentioned in bugs pages11:58
njohnstonyep11:58
*** yamamoto has quit IRC11:58
*** lseki has joined #openstack-neutron12:06
openstackgerritGregoire Mahe proposed openstack/neutron master: avoid 500 err when deleting port if designate down  https://review.opendev.org/68564412:10
*** gdwornicki has quit IRC12:12
*** gdwornicki has joined #openstack-neutron12:13
AJaegerneutron stable cores, for midonet stable branches, we need https://review.opendev.org/#/c/651750/4 and https://review.opendev.org/#/c/668858/1 to fix - and then I'd like to see https://review.opendev.org/#/c/669059/ and https://review.opendev.org/#/c/669065/2 merged (same change, need it for some Zuul cleanup). Could I get another +2 for these, please?12:13
AJaegerthanks, slaweq ^12:16
slaweqAJaeger: done :)12:16
AJaegergreat!12:17
*** dsneddon has quit IRC12:17
AJaegeris anbody looking at bagpipe queens branch? That is broken - see https://review.opendev.org/#/q/status:open+project:openstack/networking-bagpipe+branch:stable/queens12:17
AJaegerand midonet queens and rocky branches are also broken, I think yamato is aware of that12:18
slaweqAJaeger: I will check that12:19
AJaegerthanks12:19
*** dsneddon has joined #openstack-neutron12:20
jlibosvabcafarel:yep, I can set it12:21
jlibosvaslaweq: re. port_security - I think even when you set port_security_enabled to False, you still get default SG. I think you have to set --no-security-group on port to have no security group12:22
slaweqjlibosva: maybe, I'm not sure about that, but I'm sure that if You want to do --no-security-group You have to also set port_security_enabled to False, otherwise it's not possible12:23
jlibosvaslaweq: ok, I did not know that :) I thought you can have port security enabled and have no security group12:24
*** lpetrut has quit IRC12:24
*** dsneddon has quit IRC12:25
*** yamamoto has joined #openstack-neutron12:25
dalvarezbcafarel: maciejjozefczyk what do i need?12:26
dalvarezi mean what do you guys want me to do?12:26
bcafareljlibosva: thanks!12:26
dalvarezbcafarel: do you want me to add you to that list?12:27
maciejjozefczykdalvarez, could you add me? :)12:27
bcafareldalvarez: nothing for me jlibosva was faster, though you may want to add maciejjozefczyk to the networking-ovn bug gurus12:27
dalvarezack12:27
dalvarezmaciejjozefczyk: you're there12:27
maciejjozefczykdalvarez, thanks :D12:28
maciejjozefczykdalvarez, I'll mark everything as urgent, since networking_ovn is the most important project right? :D12:28
dalvarezlol12:30
dalvarezyeah12:30
openstackgerritMerged openstack/networking-midonet stable/stein: Update test_update_provider_net_unsupported test case  https://review.opendev.org/65175012:33
*** trident has quit IRC12:34
*** trident has joined #openstack-neutron12:35
*** yamamoto has quit IRC12:37
*** jpena|lunch is now known as jpena12:38
AJaegerslaweq: did you create a feature/graphql branch?12:41
openstackgerritBrian Haley proposed openstack/neutron master: Code review policy - add note about approving mechanical changes  https://review.opendev.org/68591712:42
AJaegerDidn't we delete it?12:42
AJaegerOh, let me ask release team...12:42
bcafarelhmm I thought we had deleted it indeed (after mlavalle closed all relevant reviews)12:43
bcafarelAJaeger: for midonet queens/rocky yamamoto pushed a squash of fixes for stein, that will probably be the base for fixing them12:44
slaweqAJaeger: no, I didn't create such branch12:44
bcafareland bagpipe queens well https://review.opendev.org/#/c/657828/ needs some additional help I think12:45
AJaegerslaweq: release automation did by accident, I'll discuss in #openstack-release now and will ask for removal again12:45
AJaegerbcafarel great!12:46
slaweqAJaeger: thx12:46
lajoskatonaslaweq, AJaeger: In this mail mlavalle talked about these feature branches: http://lists.openstack.org/pipermail/openstack-discuss/2019-September/009191.html12:47
AJaegeryes, I know - I talked with him about this.12:47
AJaegerhttps://review.opendev.org/685955 is the "fix", slaweq if you want to +112:48
AJaegerlajoskatona thanks for reference12:48
lajoskatonaAJaeger:Ok12:48
slaweqAJaeger: done, thx12:48
AJaegerthanks!12:49
*** macz has joined #openstack-neutron12:53
*** Luzi has quit IRC12:55
*** macz has quit IRC12:57
*** rcernin has quit IRC12:58
*** dsneddon has joined #openstack-neutron13:00
*** do3meli has joined #openstack-neutron13:01
openstackgerritMaciej Józefczyk proposed openstack/networking-ovn stable/stein: Add specified vip address when create vip  https://review.opendev.org/68595813:03
do3melihi there - anyone else experiencing problems migrating vm's with trunk ports in stein release? we see issues migrating such vm's since upgrade from r -> s. unfortunately trunk subports wont get re-mapped to the new host and we see RPC timeouts.13:03
openstackgerritzhanghao proposed openstack/neutron master: Make the MTU attribute not nullable  https://review.opendev.org/67939913:05
*** nweinber has joined #openstack-neutron13:06
*** yamamoto has joined #openstack-neutron13:10
*** dave-mccowan has quit IRC13:13
*** jawad_axd has joined #openstack-neutron13:14
*** mriedem has joined #openstack-neutron13:15
*** yamamoto has quit IRC13:17
*** ratailor__ has quit IRC13:19
*** trident has quit IRC13:22
*** trident has joined #openstack-neutron13:23
slaweqAJaeger: I don't know why this one UT is failing in networking-bagpipe, I opened bug for it https://bugs.launchpad.net/networking-bagpipe/+bug/184620913:24
openstackLaunchpad bug 1846209 in BaGPipe "[Queens] UT for BGPVPNPortAssociation OVO is failing" [Undecided,New]13:24
slaweqand will ask njohnston for help to take a look into that13:24
* njohnston enqueues that bug13:25
*** donghao has joined #openstack-neutron13:26
*** BjoernT has joined #openstack-neutron13:26
*** donghao has quit IRC13:31
*** SmearedBeard has joined #openstack-neutron13:37
*** awalende has quit IRC13:38
AJaegerthanks13:38
*** awalende has joined #openstack-neutron13:39
*** ociuhandu has quit IRC13:42
*** ivve has joined #openstack-neutron13:42
*** ociuhandu has joined #openstack-neutron13:43
*** awalende has quit IRC13:43
*** gilfoyle has joined #openstack-neutron13:45
*** ociuhandu has quit IRC13:48
openstackgerritMerged openstack/networking-midonet stable/pike: Add bindep.txt to fix unit tests  https://review.opendev.org/66885813:50
openstackgerritMerged openstack/networking-midonet stable/stein: Remove ocata jobs from branch  https://review.opendev.org/66905913:50
openstackgerritMerged openstack/networking-midonet stable/pike: Remove ocata jobs from branch  https://review.opendev.org/66906513:50
*** markvoelker has joined #openstack-neutron13:50
openstackgerritMerged openstack/neutron-tempest-plugin master: Move neutron-dynamic-routing BGP tests from stadium  https://review.opendev.org/65209913:54
*** jawad_axd has quit IRC13:54
*** markvoelker has quit IRC13:55
*** markvoelker has joined #openstack-neutron13:57
*** markvoelker has quit IRC14:01
AJaegerso, branch feature/graphql is deleted again - and hopefully does not come back ;)14:02
amotokiAJaeger: what was the reason that feature/graphql branch came again?14:04
openstackgerritDeepak proposed openstack/neutron master: Add SR-IOV ML2 driver support for VLAN trunking.  https://review.opendev.org/66546714:04
amotokiAJaeger: ah, I found your change in openstack/releases which dropped the graphql branch https://opendev.org/openstack/releases/commit/e09d9b868c53414503f9010672789623202e821f14:06
slaweqralonsoh: hi, did You saw such error like in https://34d700b43805bba2aa0d-f2bf793752c46f34c030f7713979ca54.ssl.cf1.rackcdn.com/679399/13/check/neutron-tempest-plugin-scenario-linuxbridge/ace4bc6/controller/logs/screen-n-cpu.txt.gz before?14:09
slaweqralonsoh: please look for "ERROR os_vif" there14:09
AJaegeramotoki: release team made a change to the file which recreated the branch14:09
slaweqit caused failure to spawn an instance14:10
slaweqor maybe sean-k-mooney ^^14:10
ralonsohslaweq, let me check14:10
*** ociuhandu has joined #openstack-neutron14:10
slaweqralonsoh: thx14:11
amotokiAJaeger: yeah, I noticed it. it would happen sooner or later without your change. e.g. a new rocky stable release could cause it.14:11
sean-k-mooneyslaweq: reading back14:13
slaweqsean-k-mooney: thx14:13
openstackgerritMaciej Józefczyk proposed openstack/networking-ovn master: Disable ip_version checking while updating member without new ip  https://review.opendev.org/68568614:14
openstackgerritMaciej Józefczyk proposed openstack/networking-ovn master: WIP Add missing unittests to OVN provider driver  https://review.opendev.org/67426114:14
sean-k-mooney slaweq no i have not seen that. while it should never happen we are ment to write the os vif plugins to handel the case where the interface already exisits14:15
slaweqsean-k-mooney: I saw it just this one time so far14:15
sean-k-mooneyits technically a logic error to plug a prot that is already plugged but we tried to make it work anyway14:15
sean-k-mooneyits possible we only have the extra saftey checks for ovs14:16
bcafareljlibosva: talking about https://review.opendev.org/#/c/658414 in #openstack-meeting if you are around14:16
*** awalende has joined #openstack-neutron14:16
jlibosvabcafarel: I'm eavesdroping in another meeting but can have a look14:16
jlibosvaaaah14:17
jlibosvathat one14:17
bcafarel:)14:17
sean-k-mooneyslaweq: no the protection is there https://github.com/openstack/os-vif/blob/master/vif_plug_linux_bridge/linux_net.py#L65-L8714:17
sean-k-mooneywe call that from inside a lock so that should only fail if the ovs linux bridge agent was also trying to add the port to the birdge or something like that14:19
*** awalende has quit IRC14:21
*** markvoelker has joined #openstack-neutron14:22
*** markvoelker has quit IRC14:26
*** markvoelker has joined #openstack-neutron14:27
openstackgerritMerged openstack/os-ken master: doc: Improve the document structure  https://review.opendev.org/68255314:28
ralonsohsean-k-mooney, slaweq this is not _ensure_vlan_privileged but _ensure_bridge_privileged14:28
ralonsohsean-k-mooney, slaweq the method failing14:28
ralonsohand there could be a race condition if between L191 and L193 the bridge is created14:29
ralonsohthis is the error in the logs14:29
ralonsohthis could be solved by adding in L192 check_exit_code=[17]14:29
ralonsohthis will prevent this method to exit in case of (17, 'File exists')14:30
slaweqralonsoh: sean-k-mooney do You want me to open bug for os-vif for that?14:30
ralonsohslaweq, I can do yhis14:30
slaweqralonsoh: thx14:30
*** munimeha1 has joined #openstack-neutron14:31
sean-k-mooneyits not a race14:32
sean-k-mooneyensure_bridge_privaladge is called form ensure_bridge which is also in a lock14:32
sean-k-mooneyhttps://github.com/openstack/os-vif/blob/master/vif_plug_linux_bridge/linux_net.py#L90-L9614:32
sean-k-mooneyi looked at both but i guess i linked the wrong one14:33
*** lajoskatona has quit IRC14:34
sean-k-mooneyso no dont file a bug since that is not the issue :)14:34
*** tbachman has joined #openstack-neutron14:34
sean-k-mooneyralonsoh: ^14:35
slaweqsean-k-mooney: but don't You think that in this log from nova compute there is some bug?14:35
sean-k-mooneyit is not clear what the issue is yes14:36
sean-k-mooney*yet14:36
sean-k-mooneythere could be a bug but so far the code looks to be correct14:36
sean-k-mooneywe have gards to only create the ports or bridge if they do not exist and we have both protected by a lock14:37
ralonsohsean-k-mooney, a lock?14:37
sean-k-mooneyyes14:37
ralonsohoh yes14:37
sean-k-mooneyralonsoh: https://github.com/openstack/os-vif/blob/master/vif_plug_linux_bridge/linux_net.py#L90-L9614:37
ralonsohI see it now14:37
ralonsohyes14:37
sean-k-mooneythat a file system lock too14:37
sean-k-mooneyits not just in memory14:37
slaweqsean-k-mooney: ralonsoh: look here https://34d700b43805bba2aa0d-f2bf793752c46f34c030f7713979ca54.ssl.cf1.rackcdn.com/679399/13/check/neutron-tempest-plugin-scenario-linuxbridge/ace4bc6/controller/logs/screen-q-agt.txt.gz - at 08:52:59.795059 linuxbridge agent created this bridge14:38
slaweqand than at 08:52:59.821081 nova failed with this error14:38
sean-k-mooneyright this had to be an interaction with a seperate process14:39
slaweqso IMO it could be that it was like ralonsoh said, bridge was created by linuxbridge agent between L191 and 19314:39
sean-k-mooneyso the bug is in the linux bridge agent for creating a bridge that should be created by os-vif only :)14:39
sean-k-mooneythe actul fix is that both os-vif and linux bridge agent should be using the same filesystem lock14:40
*** lpetrut has joined #openstack-neutron14:40
sean-k-mooneyso that if both have the ablity to create bridge that they cant race14:40
ralonsohsean-k-mooney, let me check this14:41
*** mlavalle has joined #openstack-neutron14:42
sean-k-mooneyralonsoh: im pretty sure its a thing. e.g. the linux bridge agent can create the bridge too.14:42
*** gcheresh_ has joined #openstack-neutron14:42
ralonsohsean-k-mooney, you are right: there is a race condition between LB agent and os-vif14:43
sean-k-mooneysince ye have os-vif in neutron now it would be nice if it called os-vif to do it if it was going to continue to create bridges14:43
ralonsohslaweq, ^14:43
slaweqralonsoh: will You than open bug for neutron for that?14:44
slaweqjust to have it recorded somewhere :)14:44
ralonsohslaweq, I'll do it14:44
slaweqralonsoh: thx a lot14:44
ralonsohsean-k-mooney, thanks a lot, as usual!!!14:44
slaweqsean-k-mooney: yeah, thx :)14:46
sean-k-mooneyno worries any reason to not replace https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/drivers/linuxbridge/agent/linuxbridge_neutron_agent.py#L423-L474 with a call to os-vif ensure_bridge which will correctly take the lock14:46
*** lpetrut has quit IRC14:47
sean-k-mooneyexample 2 on https://github.com/openstack/neutron/blob/master/neutron/plugins/ml2/drivers/linuxbridge/agent/linuxbridge_neutron_agent.py#L43214:47
sean-k-mooneyshows that the person that wrote it was at least aware it  could race14:47
slaweqsean-k-mooney: only reason I can see is lack of time :)14:48
slaweqand other priorities ;)14:48
sean-k-mooneyok we techncially are not ment to call those function directly actully so i guess that is one factor14:49
sean-k-mooneye.g. they are considerd internal to hte linux bridge function14:49
sean-k-mooney*plugin14:49
sean-k-mooneywhch is a reaso to not use them but im sure we can come up with a sane way to converge them14:50
sean-k-mooneyi would really like it if all the code for adding prots to things could live in os-vif and neutron and nova just called it includeing things liek vxlan subports but time...14:51
slaweqsean-k-mooney: yeah, time... :)14:52
sean-k-mooneyralonsoh: if it proves hard to fix the issue on the neutron side i would not be agains catching the file exists exception and logging it at debug in os-vif but it would be just masking the problem14:54
*** tbachman has quit IRC15:02
*** cshen has quit IRC15:03
*** tbachman has joined #openstack-neutron15:04
openstackgerritMerged openstack/networking-ovn stable/stein: Fix LogicalSwitchPortUpdateEvent in Octavia provider driver  https://review.opendev.org/68310715:05
*** SmearedBeard has quit IRC15:08
*** BjoernT has quit IRC15:12
*** lpetrut has joined #openstack-neutron15:13
*** ociuhandu has quit IRC15:17
*** BjoernT has joined #openstack-neutron15:18
*** ociuhandu has joined #openstack-neutron15:18
openstackgerritGregoire Mahe proposed openstack/neutron master: avoid 500 err when deleting port if designate down  https://review.opendev.org/68564415:19
openstackgerritBrian Haley proposed openstack/neutron-specs master: Toward Convergence of ML2+OVS+DVR and OVN  https://review.opendev.org/65841415:22
openstackgerritBrian Haley proposed openstack/neutron-specs master: Toward Convergence of ML2+OVS+DVR and OVN  https://review.opendev.org/65841415:23
*** ociuhandu has quit IRC15:24
*** ociuhandu has joined #openstack-neutron15:25
*** gcheresh_ has quit IRC15:27
*** donghao has joined #openstack-neutron15:28
*** ociuhandu has quit IRC15:30
*** ociuhandu has joined #openstack-neutron15:31
*** mattw4 has joined #openstack-neutron15:32
*** bobmel has joined #openstack-neutron15:33
*** tesseract has quit IRC15:33
*** donghao has quit IRC15:33
*** itlinux has joined #openstack-neutron15:35
*** lpetrut has quit IRC15:35
*** ociuhandu has quit IRC15:36
*** BjoernT has quit IRC15:39
*** jpena is now known as jpena|brb15:42
*** ociuhandu has joined #openstack-neutron15:42
*** rpittau is now known as rpittau|afk15:42
*** BjoernT has joined #openstack-neutron15:44
*** ociuhandu has quit IRC15:46
bcafarelhaleyb: nitpicking hat on, should the convergence spec stay in train/ ? I  know for db migration scripts, but less sure for specs15:47
*** BjoernT_ has joined #openstack-neutron15:48
*** BjoernT has quit IRC15:49
haleybbcafarel: i guess we can move it, forgot we're onto U15:49
*** ociuhandu has joined #openstack-neutron15:49
tidwellrbcafarel: I'm not surprised it's dragged on to U, I proposed it against Train only because U wasn't open when I pushed the first patch set :)15:50
*** gdwornicki has quit IRC15:52
bcafareltidwellr: :) yeah "small" stuff like that tends to take time15:52
*** ivve has quit IRC15:55
*** lajoskatona has joined #openstack-neutron15:56
ildikovmlavalle: morning15:56
mlavalleildikov: hi!15:57
ildikovmlavalle: quick question, would you mark any of the features that got implemented in Train as relevant to edge/5G?15:57
*** bobmel has quit IRC15:58
*** macz has joined #openstack-neutron16:04
mlavalleildikov: all the is in the highlights apply to edge/5G, although not specifically targeted to it. If you are asking something specifically targeted at edge / 5G, then no16:07
*** SmearedBeard has joined #openstack-neutron16:07
*** ociuhandu has quit IRC16:11
ildikovmlavalle: I was asking more in the sense of targeted or something to highlight from that aspect16:15
openstackgerritLajos Katona proposed openstack/neutron stable/queens: Create _mech_context before delete to avoid race [refactored for Queens]  https://review.opendev.org/68455216:16
ildikovI had the thought that in some sense/extent all is applicable16:16
njohnstonlajoskatona: Could you join #meeting to talk about networking-odl python 3 status?16:17
lajoskatonanjohnston: Hi, #openstack-meeting?16:17
njohnstonyes thanks16:18
njohnstonsorry I thought I had said that :-)16:18
lajoskatonanjohnston: you said, I just read it carefully16:18
*** jpena|brb is now known as jpena16:19
*** lpetrut has joined #openstack-neutron16:22
*** ociuhandu has joined #openstack-neutron16:25
*** ociuhandu has quit IRC16:29
openstackgerritJakub Libosvar proposed openstack/networking-ovn master: functional: Remove TODOs related to ovsdbapp 0.17.0  https://review.opendev.org/68340716:35
openstackgerritJakub Libosvar proposed openstack/networking-ovn master: functional: Fix test_router tests LSP binding  https://review.opendev.org/67966316:35
openstackgerritJakub Libosvar proposed openstack/networking-ovn master: functional: Wait for port binding update  https://review.opendev.org/68600416:35
*** jlibosva has quit IRC16:36
*** salmankhan has quit IRC16:39
*** dtantsur is now known as dtantsur|afk16:40
*** BjoernT has joined #openstack-neutron16:40
*** BjoernT_ has quit IRC16:42
*** do3meli has quit IRC16:43
*** SmearedBeard has quit IRC16:46
*** imacdonn has quit IRC16:48
*** gcheresh_ has joined #openstack-neutron16:48
*** lajoskatona has left #openstack-neutron16:51
openstackgerritMerged openstack/python-neutronclient master: Remove 'public' and 'private' parameters in fwaas_v2  https://review.opendev.org/68563616:55
*** cshen has joined #openstack-neutron16:56
*** ccamposr has quit IRC17:00
*** armax has quit IRC17:06
*** cshen has quit IRC17:06
*** cshen has joined #openstack-neutron17:07
*** cshen has quit IRC17:12
*** yamamoto has joined #openstack-neutron17:15
*** jpena is now known as jpena|off17:16
*** yamamoto has quit IRC17:19
*** tbachman has quit IRC17:20
*** tssurya_ has quit IRC17:22
*** cshen has joined #openstack-neutron17:23
*** ralonsoh has quit IRC17:27
*** cshen has quit IRC17:28
*** ccamposr has joined #openstack-neutron17:28
*** donghao has joined #openstack-neutron17:29
*** gcheresh_ has quit IRC17:30
*** lpetrut has quit IRC17:31
*** tbachman has joined #openstack-neutron17:32
*** ccamposr has quit IRC17:32
*** donghao has quit IRC17:34
*** rubasov has joined #openstack-neutron17:36
*** igordc has joined #openstack-neutron17:37
*** gilfoyle has quit IRC17:38
*** ivve has joined #openstack-neutron17:41
*** armax has joined #openstack-neutron17:47
*** ccamposr has joined #openstack-neutron17:49
*** ccamposr has quit IRC17:56
*** ccamposr has joined #openstack-neutron17:57
*** gcheresh_ has joined #openstack-neutron18:00
*** ccamposr has quit IRC18:02
*** ccamposr has joined #openstack-neutron18:03
*** ccamposr has quit IRC18:07
*** ccamposr__ has joined #openstack-neutron18:07
*** ccamposr__ has quit IRC18:09
*** gcheresh_ has quit IRC18:18
*** ccamposr has joined #openstack-neutron18:19
*** gilfoyle has joined #openstack-neutron18:34
*** ociuhandu has joined #openstack-neutron18:34
*** ociuhandu has quit IRC18:37
*** ociuhandu has joined #openstack-neutron18:39
*** bobmel has joined #openstack-neutron18:42
*** ccamposr__ has joined #openstack-neutron18:42
*** ccamposr has quit IRC18:42
*** ccamposr has joined #openstack-neutron18:45
*** ociuhandu has quit IRC18:46
*** ccamposr__ has quit IRC18:46
*** ociuhandu has joined #openstack-neutron18:46
*** gilfoyle has quit IRC18:49
*** ccamposr__ has joined #openstack-neutron18:53
*** tbachman has quit IRC18:54
*** ccamposr has quit IRC18:54
*** ccamposr has joined #openstack-neutron18:57
*** tbachman has joined #openstack-neutron18:58
*** ccamposr has quit IRC18:58
*** ccamposr has joined #openstack-neutron18:59
*** ccamposr__ has quit IRC19:00
*** maciejjozefczyk has quit IRC19:01
*** ccamposr has quit IRC19:04
*** ccamposr has joined #openstack-neutron19:05
openstackgerritSwaminathan Vasudevan proposed openstack/neutron-fwaas stable/rocky: FWaaS-DVR: FWaaS rules not updated in DVR routers on compute host  https://review.opendev.org/68602919:06
*** njohnston has quit IRC19:07
*** ociuhandu has quit IRC19:07
*** ccamposr__ has joined #openstack-neutron19:09
*** spsurya has quit IRC19:09
*** ccamposr has quit IRC19:10
*** markvoelker has quit IRC19:21
*** ccamposr__ has quit IRC19:23
*** ccamposr has joined #openstack-neutron19:25
openstackgerritDeepak proposed openstack/neutron master: Add SR-IOV ML2 driver support for VLAN trunking.  https://review.opendev.org/66546719:30
*** SmearedBeard has joined #openstack-neutron19:31
*** donghao has joined #openstack-neutron19:32
*** njohnston has joined #openstack-neutron19:36
*** donghao has quit IRC19:36
*** slaweq has quit IRC19:59
*** nweinber has quit IRC20:00
*** markvoelker has joined #openstack-neutron20:00
*** SmearedBeard has quit IRC20:11
*** tbachman has quit IRC20:11
*** Smeared_Beard has joined #openstack-neutron20:12
*** tbachman has joined #openstack-neutron20:20
*** pcaruana has quit IRC20:33
*** ccamposr has quit IRC20:34
*** ccamposr has joined #openstack-neutron20:34
*** yamamoto has joined #openstack-neutron20:37
openstackgerritGhanshyam Mann proposed openstack/neutron-tempest-plugin master: [train][goal] Define new neutron-tempest-ipv6-only job  https://review.opendev.org/68604320:50
openstackgerritGhanshyam Mann proposed openstack/neutron-tempest-plugin master: [train][goal] Define new neutron-tempest-ipv6-only job  https://review.opendev.org/68604320:52
openstackgerritGhanshyam Mann proposed openstack/neutron-tempest-plugin master: [train][goal] Define new neutron-tempest-ipv6-only job  https://review.opendev.org/68604320:54
*** tbachman has quit IRC21:01
*** ociuhandu has joined #openstack-neutron21:08
*** yamamoto has quit IRC21:08
*** yamamoto has joined #openstack-neutron21:11
*** ociuhandu has quit IRC21:12
*** ajay33 has quit IRC21:19
*** openstackstatus has quit IRC21:32
*** donghao has joined #openstack-neutron21:34
*** openstackstatus has joined #openstack-neutron21:35
*** ChanServ sets mode: +v openstackstatus21:35
*** donghao has quit IRC21:38
*** tbachman has joined #openstack-neutron21:49
*** munimeha1 has quit IRC21:51
openstackgerritYAMAMOTO Takashi proposed openstack/networking-midonet stable/rocky: Fix versions of neutron and friends  https://review.opendev.org/68606021:56
*** mriedem has quit IRC21:56
*** luksky has quit IRC22:10
*** do3meli has joined #openstack-neutron22:14
openstackgerritSwaminathan Vasudevan proposed openstack/neutron-fwaas stable/rocky: FWaaS-DVR: FWaaS rules not updated in DVR routers on compute host  https://review.opendev.org/68602922:21
*** do3meli has left #openstack-neutron22:27
*** tkajinam has joined #openstack-neutron22:50
*** yamamoto has quit IRC22:57
*** yamamoto has joined #openstack-neutron23:01
*** BjoernT has quit IRC23:04
*** macz has quit IRC23:05
*** macz has joined #openstack-neutron23:07
*** macz has quit IRC23:10
*** mlavalle has quit IRC23:31
*** BjoernT has joined #openstack-neutron23:33
*** rcernin has joined #openstack-neutron23:33
*** donghao has joined #openstack-neutron23:36
*** markvoelker has quit IRC23:38
*** yamamoto has quit IRC23:39
*** yamamoto has joined #openstack-neutron23:39
*** donghao has quit IRC23:41
*** yamamoto has quit IRC23:43
*** markvoelker has joined #openstack-neutron23:44
openstackgerritSwaminathan Vasudevan proposed openstack/neutron-fwaas stable/rocky: FWaaS-DVR: FWaaS rules not updated in DVR routers on compute host  https://review.opendev.org/68602923:44
*** ivve has quit IRC23:47
*** yamamoto has joined #openstack-neutron23:48
*** yamamoto has quit IRC23:52
*** yamamoto has joined #openstack-neutron23:55

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!