Thursday, 2015-09-03

*** SumitNaiksatam has quit IRC00:00
*** yamamoto has joined #openstack-neutron00:01
*** absubram has joined #openstack-neutron00:01
*** srilatha has quit IRC00:01
*** yongshenggong_ has quit IRC00:01
openstackgerritAssaf Muller proposed openstack/neutron: CONF.reset does not happen for all tests  https://review.openstack.org/21987800:02
*** emagana has quit IRC00:03
*** vikram__ has quit IRC00:04
*** mchestr has quit IRC00:06
*** thurloat_isgone is now known as thurloat00:11
*** sthillma_ has joined #openstack-neutron00:11
*** rushil_ has quit IRC00:13
*** sthillma has quit IRC00:14
*** sthillma_ is now known as sthillma00:14
*** Swami has quit IRC00:14
*** rushil has joined #openstack-neutron00:14
*** Swami_ has quit IRC00:14
*** pavel_bondar has quit IRC00:14
*** Swami__ has joined #openstack-neutron00:14
*** Swami has joined #openstack-neutron00:14
*** pavel_bondar has joined #openstack-neutron00:15
*** tobasco_ has quit IRC00:15
*** tobasco has joined #openstack-neutron00:16
*** vikram__ has joined #openstack-neutron00:16
openstackgerritKevin Benton proposed openstack/neutron: Deprecate external_network_bridge option in L3 agent  https://review.openstack.org/21992800:17
*** banix has quit IRC00:17
*** padkrish has joined #openstack-neutron00:19
*** goodygum has quit IRC00:19
*** goodygum has joined #openstack-neutron00:21
*** thorst has joined #openstack-neutron00:21
*** padkrish has quit IRC00:23
*** shadower has quit IRC00:23
*** padkrish has joined #openstack-neutron00:23
*** shadower has joined #openstack-neutron00:23
*** thorst has quit IRC00:23
*** outworlder has quit IRC00:24
openstackgerritArmando Migliaccio proposed openstack/neutron: Retain logs from functional test runs  https://review.openstack.org/21993500:24
*** banix has joined #openstack-neutron00:25
*** Swami__ has quit IRC00:25
*** Swami has quit IRC00:25
*** Aish has quit IRC00:26
*** yalie has quit IRC00:26
openstackgerritArmando Migliaccio proposed openstack/neutron: Retain logs from functional test runs  https://review.openstack.org/21993500:27
*** vikram__ has quit IRC00:27
*** achanda has joined #openstack-neutron00:28
*** abhiraut has quit IRC00:28
*** jbell8 has joined #openstack-neutron00:29
*** Swami has joined #openstack-neutron00:30
*** moshele has quit IRC00:31
*** miyagishi_t has joined #openstack-neutron00:31
*** nlahouti has quit IRC00:32
*** abhiraut has joined #openstack-neutron00:33
*** sthillma_ has joined #openstack-neutron00:34
*** sthillma has quit IRC00:37
*** sthillma_ is now known as sthillma00:37
*** nlahouti has joined #openstack-neutron00:37
*** Sukhdev_ has joined #openstack-neutron00:37
*** vikram__ has joined #openstack-neutron00:39
*** tiswanso has joined #openstack-neutron00:39
*** ameybhide has quit IRC00:40
*** nlahouti has quit IRC00:40
*** shivrao has quit IRC00:41
*** nlahouti has joined #openstack-neutron00:42
*** roeyc has joined #openstack-neutron00:42
*** nlahouti has quit IRC00:42
*** sthillma has quit IRC00:44
*** armax has quit IRC00:46
*** pritesh has quit IRC00:47
*** armax has joined #openstack-neutron00:47
*** achanda has quit IRC00:51
*** achanda has joined #openstack-neutron00:52
*** achanda has quit IRC00:52
*** banix has quit IRC00:54
openstackgerritstephen-ma proposed openstack/neutron: Descheduling DVR routers when ports are unbound from VM  https://review.openstack.org/21906700:54
*** shoutm_ has joined #openstack-neutron00:56
*** shoutm has quit IRC00:56
openstackgerritArmando Migliaccio proposed openstack/neutron: Retain logs from functional test runs  https://review.openstack.org/21993500:58
*** hichihara has joined #openstack-neutron00:59
openstackgerritEmett Speer proposed openstack/neutron-specs: [Labs] Update links to Cloud Admin Guide  https://review.openstack.org/21994200:59
*** rushil has quit IRC01:02
*** fzdarsky_ has joined #openstack-neutron01:02
*** zzzeek has joined #openstack-neutron01:03
*** vikram__ has quit IRC01:04
*** fzdarsky has quit IRC01:05
*** julim has joined #openstack-neutron01:05
*** rossella_s has quit IRC01:06
*** rossella_s has joined #openstack-neutron01:06
*** ducnc has joined #openstack-neutron01:07
*** SumitNaiksatam has joined #openstack-neutron01:08
*** thurloat is now known as thurloat_isgone01:09
*** namptit has joined #openstack-neutron01:12
*** shivrao has joined #openstack-neutron01:13
*** tiswanso has quit IRC01:13
*** tiswanso has joined #openstack-neutron01:13
*** namptit has quit IRC01:14
*** namptit has joined #openstack-neutron01:15
*** namptit has quit IRC01:15
*** julim has quit IRC01:15
*** banix has joined #openstack-neutron01:16
*** shivrao_ has joined #openstack-neutron01:16
*** vikram__ has joined #openstack-neutron01:16
*** vivek-ebay has quit IRC01:17
*** shivrao has quit IRC01:18
*** shivrao_ is now known as shivrao01:18
*** erkules has joined #openstack-neutron01:19
*** jwarendt has quit IRC01:20
*** ijw__ has quit IRC01:20
*** julim has joined #openstack-neutron01:20
*** erkules_ has quit IRC01:22
*** yushiro has quit IRC01:23
*** abhiraut has quit IRC01:23
*** ociuhandu has joined #openstack-neutron01:25
*** padkrish has quit IRC01:26
*** padkrish has joined #openstack-neutron01:26
*** tflynn has joined #openstack-neutron01:27
*** nplanel has joined #openstack-neutron01:27
*** tflynn has quit IRC01:31
*** padkrish has quit IRC01:31
*** sbalukoff has quit IRC01:32
*** vikram__ has quit IRC01:34
*** sdake has quit IRC01:36
*** sridhar_ram has quit IRC01:36
*** sballe has quit IRC01:36
openstackgerritArmando Migliaccio proposed openstack/neutron: Retain logs from functional test runs  https://review.openstack.org/21993501:39
*** Swami has quit IRC01:44
*** achanda has joined #openstack-neutron01:44
*** Aish has joined #openstack-neutron01:45
*** vikram__ has joined #openstack-neutron01:45
*** julim has quit IRC01:47
*** minwang2 has quit IRC01:48
*** Aish has left #openstack-neutron01:49
*** boris-42 has quit IRC01:50
*** banix has quit IRC01:51
*** jbell8 has quit IRC01:55
openstackgerritArmando Migliaccio proposed openstack/neutron: Retain logs from functional test runs  https://review.openstack.org/21993501:56
*** ybabenko has joined #openstack-neutron01:57
*** fzdarsky__ has joined #openstack-neutron01:59
*** ybabenko has quit IRC02:01
*** fzdarsky_ has quit IRC02:02
*** vikram__ has quit IRC02:04
*** SumitNaiksatam has quit IRC02:05
*** baoli has quit IRC02:06
*** baoli has joined #openstack-neutron02:06
*** jwy has quit IRC02:06
*** SumitNaiksatam has joined #openstack-neutron02:07
*** baoli has quit IRC02:07
*** nihilifer has quit IRC02:07
*** baoli has joined #openstack-neutron02:08
*** nihilifer has joined #openstack-neutron02:08
*** zzzeek has quit IRC02:09
*** mickeys has quit IRC02:10
*** padkrish has joined #openstack-neutron02:10
openstackgerritArmando Migliaccio proposed openstack/neutron: Retain logs from functional test runs  https://review.openstack.org/21993502:11
*** lyanchih has joined #openstack-neutron02:12
*** armax has quit IRC02:12
*** iwamoto has joined #openstack-neutron02:12
*** Leonr_ has quit IRC02:14
*** Leonr has joined #openstack-neutron02:14
*** banix has joined #openstack-neutron02:16
*** vikram__ has joined #openstack-neutron02:16
*** spandhe has quit IRC02:17
*** banix has quit IRC02:17
*** bharathm has quit IRC02:19
iwamotokevinbenton: ping02:20
*** padkrish has quit IRC02:21
*** madhu_ak has quit IRC02:23
kevinbentoniwamoto: pong02:25
iwamotokevinbenton: hi02:25
kevinbentoniwamoto: what's up?02:26
iwamotoI'd ask your +2 on https://review.openstack.org/15394602:26
*** shwetaap has joined #openstack-neutron02:26
*** julim has joined #openstack-neutron02:26
*** banix has joined #openstack-neutron02:27
*** Leonr has quit IRC02:27
*** vikram__ has quit IRC02:28
*** sdake has joined #openstack-neutron02:28
*** Kennan has quit IRC02:29
*** Kennan has joined #openstack-neutron02:30
*** sdake_ has joined #openstack-neutron02:31
*** roeyc has quit IRC02:31
*** Sukhdev_ has quit IRC02:32
*** jbell8 has joined #openstack-neutron02:32
kevinbentoniwamoto: done02:33
iwamotokevinbenton: thanks a lot02:33
iwamotokevinbenton: I need another +2. To whom should I ask? Any idea?02:34
*** sdake has quit IRC02:34
*** vivek-ebay has joined #openstack-neutron02:35
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: DO NOT MERGE: OVS-agent: Switch the default to "native" of_interface  https://review.openstack.org/20632902:38
*** vikram__ has joined #openstack-neutron02:40
*** bharathm has joined #openstack-neutron02:41
*** kiran-r has joined #openstack-neutron02:43
*** saksham has quit IRC02:44
*** shivrao has quit IRC02:44
*** shivrao has joined #openstack-neutron02:45
*** armax has joined #openstack-neutron02:49
openstackgerritHirofumi Ichihara proposed openstack/neutron: Add availability_zone support for Scheduler  https://review.openstack.org/20443602:52
openstackgerritArmando Migliaccio proposed openstack/neutron: Retain logs from functional test runs  https://review.openstack.org/21993502:53
*** hakimo_ has quit IRC02:54
*** vikram__ has quit IRC02:55
*** minwang2 has joined #openstack-neutron02:55
*** hakimo has joined #openstack-neutron02:57
*** kiran-r has quit IRC02:58
*** rushil has joined #openstack-neutron03:07
*** vikram__ has joined #openstack-neutron03:07
openstackgerritArmando Migliaccio proposed openstack/neutron: Retain logs from functional test runs  https://review.openstack.org/21993503:11
*** tflynn has joined #openstack-neutron03:12
*** vikram__ has quit IRC03:13
*** fedexo has joined #openstack-neutron03:14
*** rushil has quit IRC03:17
kevinbentoniwamoto: Miguel Ajo03:19
kevinbentoniwamoto: or see if armax is willing to bump to +203:19
*** banix has quit IRC03:19
iwamotokevinbenton: thanks. I'll ask them :)03:20
*** shoutm_ has quit IRC03:20
iwamotoajo, armax: ping03:20
*** tiswanso has quit IRC03:23
*** shashank_hegde has joined #openstack-neutron03:23
*** vikram__ has joined #openstack-neutron03:25
*** Sukhdev has quit IRC03:26
*** Sukhdev has joined #openstack-neutron03:27
*** mickeys has joined #openstack-neutron03:27
*** Samos123 has quit IRC03:27
*** vishwanathj has quit IRC03:28
*** achanda has quit IRC03:28
*** bobmel has joined #openstack-neutron03:29
*** erhudy1 has quit IRC03:29
*** dguerri` has quit IRC03:30
*** jroll has quit IRC03:30
*** lyanchih_ has joined #openstack-neutron03:30
*** lyanchih has quit IRC03:31
*** lyanchih_ is now known as lyanchih03:31
*** clev-away has quit IRC03:31
*** bobmel_ has quit IRC03:32
*** fedexo has quit IRC03:32
*** dguerri` has joined #openstack-neutron03:32
*** clev has joined #openstack-neutron03:32
*** dguerri` is now known as dguerri03:32
*** dguerri has joined #openstack-neutron03:32
*** vikram__ has quit IRC03:32
*** fedexo has joined #openstack-neutron03:32
*** jroll has joined #openstack-neutron03:33
*** Samos123 has joined #openstack-neutron03:33
*** adam_g has quit IRC03:33
*** adam_g has joined #openstack-neutron03:35
*** adam_g has quit IRC03:35
*** adam_g has joined #openstack-neutron03:35
*** dims has joined #openstack-neutron03:35
*** shashank_hegde_ has joined #openstack-neutron03:40
*** shashank_hegde has quit IRC03:40
*** shashank_hegde_ is now known as shashank_hegde03:40
*** fedexo has quit IRC03:41
*** shoutm has joined #openstack-neutron03:41
*** sdake_ is now known as sdake03:42
*** deepthi has joined #openstack-neutron03:42
*** shaohe_feng has joined #openstack-neutron03:44
*** vikram__ has joined #openstack-neutron03:45
*** tflynn_ has joined #openstack-neutron03:45
*** tflynn has quit IRC03:48
*** vikram__ has quit IRC03:50
*** SridharG has joined #openstack-neutron03:52
*** shivrao has quit IRC03:53
*** julim has quit IRC03:54
openstackgerritHirofumi Ichihara proposed openstack/neutron: Add functional test for availability_zone support  https://review.openstack.org/21097703:55
*** boris-42 has joined #openstack-neutron03:59
*** dims has quit IRC04:00
*** links has joined #openstack-neutron04:02
*** vikram__ has joined #openstack-neutron04:02
openstackgerritSridhar Gaddam proposed openstack/neutron: Add IPv6 Address Resolution protection  https://review.openstack.org/20165004:03
openstackgerritMerged openstack/neutron: DHCP agent: allow using gateway IPs instead of uniquely allocated  https://review.openstack.org/20607804:05
*** Sukhdev_ has joined #openstack-neutron04:06
*** vikram__ has quit IRC04:08
*** amotoki has joined #openstack-neutron04:11
*** baojg has joined #openstack-neutron04:11
*** baoli has quit IRC04:14
*** shaohe_feng has quit IRC04:14
openstackgerritOpenStack Proposal Bot proposed openstack/neutron: Updated from global requirements  https://review.openstack.org/21949604:15
*** mickeys has quit IRC04:16
*** slaweq has joined #openstack-neutron04:16
*** vikram__ has joined #openstack-neutron04:20
openstackgerritSergey Vilgelm proposed openstack/neutron: Fix a wrong condition for the _purge_metering_info function  https://review.openstack.org/21888504:21
yongshenggong__ping amotoki04:22
*** vishwanathj has joined #openstack-neutron04:22
yongshenggong__https://review.openstack.org/#/c/139928/ the availability zone spec rest is not approved yet.04:23
*** gsagie_ has joined #openstack-neutron04:23
*** minwang2 has quit IRC04:24
openstackgerritArmando Migliaccio proposed openstack/neutron: Retain logs for functional test cases  https://review.openstack.org/21993504:25
*** slaweq has quit IRC04:25
*** vikram__ has quit IRC04:26
*** carl_baldwin has joined #openstack-neutron04:26
*** SridharG has quit IRC04:26
*** yfried has quit IRC04:31
*** baojg has quit IRC04:31
*** lyanchih has quit IRC04:32
*** yalie has joined #openstack-neutron04:32
*** garyk has quit IRC04:35
hichiharayongshenggong__: Please look at https://review.openstack.org/#/c/169612/04:35
*** SridharG has joined #openstack-neutron04:35
yongshenggong__hichihara: thanks, I will review that patch.04:36
hichiharayongshenggong__: Thank you for your help.04:36
yongshenggong__hichihara: I like the ideas to improve the agents and related scheduler, which is my initial idea. thanks for the improvement.04:37
*** shoutm has quit IRC04:38
*** vikram__ has joined #openstack-neutron04:38
yongshenggong__hi guys, who know how to upgrade the pip version in tox create virtual env?04:39
hichiharayongshenggong__: I'm happy :)04:40
yongshenggong__me too :)04:42
*** moshele has joined #openstack-neutron04:43
*** vikram__ has quit IRC04:43
*** carl_baldwin has quit IRC04:43
*** kobayashi has joined #openstack-neutron04:44
*** shivrao has joined #openstack-neutron04:44
*** anilvenkata has joined #openstack-neutron04:44
gusyongshenggong__:  .tox/$env/bin/pip install --upgrade pip04:46
*** shoutm has joined #openstack-neutron04:47
yalieping armax04:48
yaliearmax: ping04:48
*** shwetaap has quit IRC04:48
*** BhavyaM has joined #openstack-neutron04:48
*** gsagie_ has quit IRC04:49
*** vikram has joined #openstack-neutron04:49
*** moshele has quit IRC04:50
*** minwang2 has joined #openstack-neutron04:51
HenryGarmax: ping04:52
*** anilvenkata has quit IRC04:52
HenryGCONF.reset ...04:52
*** dannywilson has joined #openstack-neutron04:52
*** BhavyaM is now known as BhavyaM_afk04:53
yongshenggong__gus, do u know why tox -e py27 will reset the virtual env's pip back into old version?04:53
gusyongshenggong__:  hrm, I don't think it should - unless the tox env is being recreated (tox -r, or a change to tox.ini, etc)04:54
gusDoes it say it is recreating the env?04:54
yongshenggong__no. I don't recreate it04:54
*** ducnc is now known as ducnguyen04:55
*** vikram__ has joined #openstack-neutron04:55
gusyongshenggong__: tox can decide to recreate it itself automatically sometimes (modified tox.ini, new python, etc) - but the output should say it is recreating it when it does.04:56
*** BhavyaM_afk is now known as BhavyaM04:56
yongshenggong__gus: I will create paste bin04:58
yongshenggong__gus: could you please have a look at http://paste.openstack.org/show/443588/04:58
yongshenggong__gus: I am struggling with the pip version04:59
*** dims has joined #openstack-neutron04:59
armaxHenryG: oing04:59
armaxHenryG: pong04:59
HenryGarmax: I just hit send on a reply in the comments04:59
* armax looks05:00
*** srikanth_p has joined #openstack-neutron05:00
armaxHenryG: that I know05:00
HenryGarmax: so what am I missing?05:00
armaxit’s more the ‘static’ decomposition of responsability05:00
srikanth_pca someone help me how to write CLI commands to neutronclient?05:00
armaxrather than the fact that eventually the cleanup function is called05:00
armaxbut say for instance that we fail in the base setup05:01
armaxluckily the reset method is idempotent I think05:01
yongshenggong__srikanth_p: you can look at https://review.openstack.org/#/c/219455/05:01
yongshenggong__to learn how to write neutron client commands05:02
yongshenggong__It is not difficult05:02
*** lyanchih has joined #openstack-neutron05:02
HenryGwe already do things like mock.patch.stopall for everyone05:02
HenryGthe magic is rampant05:03
*** vikram__ has quit IRC05:03
armaxthat’s slightly different05:03
armaxbut maybe I am arguing for nothing05:03
*** afazekas has joined #openstack-neutron05:04
HenryGyou are usually right though05:04
armaxHenryG: at the moment the movement feels awkward05:05
*** rossella_s has quit IRC05:06
*** dims has quit IRC05:06
armaxmy point is that I would like to see self.setup_config() and addcleanup closer together05:06
*** rossella_s has joined #openstack-neutron05:06
gusyongshenggong__: oh, try .tox/pu27/bin/pip install --upgrade pbr05:06
armaxif possible05:06
iwamotoarmax: hi. could you consider changing your +1 to +2 on the ryu patch?05:07
armaxno05:07
armaxwhy?05:07
armaxthere would be no point05:08
armaxiwamoto: we’re cutting L3 any moment now05:08
armaxthe ryu patch will have to be deferred to RC105:08
iwamotoshould I apply for FFE?05:08
armaxhaving said that, please reach out to anorher L2 Lt05:08
*** shaohe_feng has joined #openstack-neutron05:08
armaxiwamoto: hold on05:08
iwamotoit seems kevinbenton is the only L2 lieutenant05:10
armaxwhat about rossella_s?05:13
armaxshe is the other core reviewer for L205:13
*** vivek-ebay has quit IRC05:13
*** sdake_ has joined #openstack-neutron05:13
*** kiran-r has joined #openstack-neutron05:14
*** sbalukoff has joined #openstack-neutron05:14
HenryGarmax: I think addCleanup(CONF.reset) must be done in Diet because test classes can derive from Diet and then change CONF in other ways than setup_config(). It seems that tests deriving from Diet do not want to do setup_config(). ?05:15
*** vikram__ has joined #openstack-neutron05:15
armaxthat’s fine05:15
HenryGarmax: but at this hour my brain is fried. You can educate me more tomorrow05:15
armaxwe can leave the magic be05:16
*** shausy has joined #openstack-neutron05:16
*** sdake has quit IRC05:17
*** nlahouti has joined #openstack-neutron05:17
armaxHenryG: feel free to nudge it in, it’ll be unlikely if it merges any time soon05:17
armaxHenryG: but the patch is speculative anyway05:17
*** sthillma has joined #openstack-neutron05:17
HenryGarmax: no way, I am not approving that :)05:18
armaxHenryG: why?05:18
HenryGYou sow doubt in my mind05:18
*** n00btube has joined #openstack-neutron05:18
armaxtake it out05:18
armaxit’s not worth it05:19
*** nlahouti1 has joined #openstack-neutron05:19
*** slaweq has joined #openstack-neutron05:19
*** vikram__ has quit IRC05:21
*** nlahouti has quit IRC05:21
*** n00btube has quit IRC05:23
*** lyanchih has quit IRC05:23
*** Kennan2 has joined #openstack-neutron05:24
*** Kennan has quit IRC05:25
*** shausy has quit IRC05:27
*** SridharG has quit IRC05:29
*** neeti has joined #openstack-neutron05:30
*** shaohe_feng has quit IRC05:30
*** yfried has joined #openstack-neutron05:33
*** vikram__ has joined #openstack-neutron05:33
*** achanda has joined #openstack-neutron05:34
*** shoutm has quit IRC05:36
*** nlahouti1 has quit IRC05:37
*** shoutm has joined #openstack-neutron05:37
*** yfried_ has joined #openstack-neutron05:37
*** yfried has quit IRC05:37
*** vikram__ has quit IRC05:38
*** mkolesni has joined #openstack-neutron05:38
*** itzikb has joined #openstack-neutron05:39
*** achanda has quit IRC05:40
*** tflynn has joined #openstack-neutron05:41
*** dannywilson has quit IRC05:41
*** tflynn_ has quit IRC05:42
*** kobayashi has quit IRC05:42
yaliearmax: ping05:42
*** sthillma has quit IRC05:43
*** kobayashi has joined #openstack-neutron05:43
*** ducnguyen has quit IRC05:44
*** kiran-r has quit IRC05:44
armaxyalie: pong05:46
*** anilvenkata has joined #openstack-neutron05:46
yalieha armax, I want to confirm that the unaddressed-port issue,05:47
yalieis that because this port could spoofing any ip in the same network?05:47
armaxyalie: what do you mean?05:47
yaliefor example, a network with IP A, B05:48
armaxyalie: do you have a good understanding of the vulnerability?05:48
*** sdake has joined #openstack-neutron05:48
armaxbeacuse if you don’t this discussion is pointless05:48
yalieI am not sure, and want to confirm with you05:48
armaxk05:48
yalieif the unaddrss port not spoof A, B, would it be ok?05:48
*** moshele has joined #openstack-neutron05:49
*** VikasC has joined #openstack-neutron05:49
armaxlet’s back out for a sec05:49
yalieok05:49
armaxyour proposed solution prevented l3 security rules to be applied to the port05:50
armaxif it didn’t have an address05:50
armaxcorrect05:50
armax?05:50
yalieyes05:50
armaxnow imagine the scenario where you creatae a network05:50
armaxcreate a port05:50
armaxand only then create subnet05:50
yalieyes05:50
armaxnow the port will have no l3 rules correct?05:51
yaliethe port in the same broadcast domian with the new created subnet05:52
*** sdake_ has quit IRC05:52
yalieit could pretend to be any ip of the network?05:52
yalieright05:52
yaliethe port still not have L3 rules05:53
*** tfukushima has joined #openstack-neutron05:53
*** sdake_ has joined #openstack-neutron05:54
yalieit will accept all the traffic of other ip?05:54
armaxare you asking me?05:54
yaliesorry, I don't understand05:55
yaliedo you think all the l3 rules should be need if the port is in a network includeing subnet?05:56
openstackgerritOleg Bondarev proposed openstack/neutron: Avoid DB errors when deleting network's ports and subnets  https://review.openstack.org/21211005:57
*** sdake has quit IRC05:57
*** shoutm has quit IRC05:58
yalieI dont' know if it 's adequate that add a mech that make unaddressport knows other ip in the network, and not spoof these ones.05:58
*** shoutm has joined #openstack-neutron05:58
*** yfried_ is now known as yfried|afk06:00
*** achanda has joined #openstack-neutron06:00
*** lyanchih has joined #openstack-neutron06:01
*** ekuris has joined #openstack-neutron06:01
*** itzikb has quit IRC06:01
*** yfried|afk is now known as yfried_06:02
*** itzikb has joined #openstack-neutron06:03
*** vthapar has joined #openstack-neutron06:03
*** shashank_hegde has quit IRC06:04
*** shashank_hegde has joined #openstack-neutron06:05
armaxyalie: I think in lieu of a good understanding of the problem it’s better to err on the side of caution and ensure that we don’t open up a huge hole into a shared network06:05
*** Sukhdev_ has quit IRC06:06
*** garyk has joined #openstack-neutron06:07
*** tflynn_ has joined #openstack-neutron06:07
yalieso, rules should be kept if the network include the subnet(mayshared)?06:07
yalieor, we should say share network should not involved in unadressed port06:09
armaxboth can perhaps work06:09
armaxcheck with kevinbenton, he may have other ideas too06:09
*** numans has joined #openstack-neutron06:10
yalieyes, I checked with kevin, he adviced that maybe need a new knob06:10
*** roeyc has joined #openstack-neutron06:10
yaliehow do you think about it ?06:10
*** tflynn has quit IRC06:10
*** qwebirc29265 has joined #openstack-neutron06:11
*** abregman has joined #openstack-neutron06:11
*** vivek-ebay has joined #openstack-neutron06:14
*** baoli has joined #openstack-neutron06:14
yalieok, thanks armax, I will check more about it.06:14
*** shivrao has quit IRC06:15
*** vivek-ebay has quit IRC06:19
*** baoli has quit IRC06:19
*** achanda has quit IRC06:19
openstackgerritOpenStack Proposal Bot proposed openstack/neutron: Imported Translations from Transifex  https://review.openstack.org/21874206:21
*** slaweq has quit IRC06:22
*** slaweq has joined #openstack-neutron06:23
*** slaweq has quit IRC06:25
*** shoutm_ has joined #openstack-neutron06:25
*** dedery has joined #openstack-neutron06:25
*** shoutm has quit IRC06:26
*** slaweq has joined #openstack-neutron06:26
*** slaweq has quit IRC06:28
openstackgerritLi Ma proposed openstack/neutron: Implement external physical bridge mapping in linuxbridge  https://review.openstack.org/20900606:29
*** gampel has joined #openstack-neutron06:32
*** SridharG has joined #openstack-neutron06:36
*** minwang2 has quit IRC06:40
openstackgerritArmando Migliaccio proposed openstack/neutron: Add support for PluginWorker and Process creation notification  https://review.openstack.org/18939106:40
*** gampel1 has joined #openstack-neutron06:41
*** gampel has quit IRC06:44
*** kiran-r has joined #openstack-neutron06:44
*** maishsk has joined #openstack-neutron06:45
maishskGood morning Neutron06:45
maishskIs it possible to set a domain suffix per tenant?06:45
maishskOr is is a general neutron setting done in dhcp_agent.ini (dhcp_domain)?06:46
*** achanda has joined #openstack-neutron06:48
maishskAnd therefore only one per neutron?06:49
*** ybabenko has joined #openstack-neutron06:51
*** alex_klimov has joined #openstack-neutron06:55
*** ybabenko has quit IRC06:56
*** ybabenko has joined #openstack-neutron06:56
*** ushkalim has joined #openstack-neutron06:56
*** jlibosva has joined #openstack-neutron06:56
*** safchain has joined #openstack-neutron06:56
*** armax has quit IRC06:58
*** qwebirc29265 has quit IRC06:58
*** shz has quit IRC06:59
*** kiran-r has quit IRC06:59
*** kawa2014 has joined #openstack-neutron07:02
*** ildikov has joined #openstack-neutron07:05
*** tmorin has joined #openstack-neutron07:06
*** Murali has joined #openstack-neutron07:07
*** karthiks has joined #openstack-neutron07:07
*** srikanth_p has quit IRC07:09
*** roeyc has quit IRC07:11
*** tflynn_ has quit IRC07:13
*** yfried_ is now known as yfried|afk07:14
*** yfried|afk is now known as yfried_07:14
*** Marga_ has joined #openstack-neutron07:15
*** Marga_ has quit IRC07:15
*** Marga_ has joined #openstack-neutron07:16
kevinbentonmaishsk: I believe a new feature just went in this cycle to allow dns for each tenant07:16
maishskkevinbenton: in Kilo? link?07:17
*** y_sawai has joined #openstack-neutron07:18
kevinbentonmaishsk: no, in liberty07:19
kevinbentonhttps://review.openstack.org/#/c/88624/07:19
SridharGkevinbenton: I raised a new bug and updated the commit message. Can you please take a look, thanks - https://review.openstack.org/#/c/201650/07:20
*** gparaskevas has joined #openstack-neutron07:23
SridharGkevinbenton: thanks for reviewing the patch.07:23
kevinbentonSridharG: np07:24
*** jpena has joined #openstack-neutron07:24
*** shashank_hegde has quit IRC07:25
*** ccard has quit IRC07:26
*** ronis has joined #openstack-neutron07:26
*** dedery has quit IRC07:26
*** bharathm has quit IRC07:26
*** dedery has joined #openstack-neutron07:26
maishskkevinbenton: I am not sure how that bp addresses the fact that each tenant will need a different DNS?07:27
maishskor I should rather say - not need - but a domain suffix could be configured per tenant - or even more so - per subnet07:27
maishskaka network07:27
*** matrohon has joined #openstack-neutron07:28
kevinbentonmaishsk: isn't a different dns domain per network what you want?07:29
*** evgenyf has joined #openstack-neutron07:30
maishskkevinbenton: Yes. Today it is limited to one per neutron - if I understand correctly ?07:30
*** obondarev has joined #openstack-neutron07:30
*** anshul has joined #openstack-neutron07:31
*** tfukushima has quit IRC07:31
kevinbentonmaishsk: yes07:31
*** tfukushima has joined #openstack-neutron07:32
kevinbentonmaishsk: I haven't tried overriding with the extra dhcp options API though07:32
kevinbentonmaishsk: have you tried that?07:32
*** anshul has quit IRC07:33
*** anshul has joined #openstack-neutron07:33
*** afazekas__ has joined #openstack-neutron07:33
maishskkevinbenton: I don’t that is available - honestly would not know really how to look for is07:34
maishskdon’t think*07:34
kevinbentonmaishsk: I gtg. Try shooting an email to the list07:34
maishskfrom a neutron subnet-show I don’t see any dns information - except the nameservers.07:34
maishskkevinbenton: will do.07:35
maishskThanks07:35
*** jlanoux has joined #openstack-neutron07:36
*** mickeys has joined #openstack-neutron07:39
hichiharamaishsk: I guess that we can configure dns name servers but cannot do domain-suffix07:40
*** vikram has quit IRC07:41
*** rotbeard has joined #openstack-neutron07:41
*** ociuhandu has quit IRC07:41
*** lajos-katona has joined #openstack-neutron07:41
*** jschwarz has joined #openstack-neutron07:41
maishskhichihara: that is my understanding as well.07:43
maishskI am not sure that the bp that kevinbenton mentioned above covers this use case07:44
*** y_sawai has quit IRC07:45
hichiharaI don't think so. That is external dns case. I guess that dnsmasq run with "--domain-suffix" which you want.07:46
maishskthis looks more suitable https://blueprints.launchpad.net/neutron/+spec/dns-subsystem07:48
*** lyanchih has quit IRC07:48
*** jbell8 has quit IRC07:49
openstackgerritAnn Kamyshnikova proposed openstack/neutron-vpnaas: Drop for 'tenant_id' column for 'cisco_csr_identifier_map' table  https://review.openstack.org/21317407:49
openstackgerritAnn Kamyshnikova proposed openstack/neutron-vpnaas: Implement ModelMigrationSyncTest  https://review.openstack.org/20994307:49
hichiharaYeah but it's big bp... and I cannot see the spec and code07:51
*** mickeys has quit IRC07:51
*** Marga_ has quit IRC07:52
*** ociuhandu has joined #openstack-neutron07:52
hichiharaWell, you might want to send your question to ML as kevinbenton said.07:52
*** vikram has joined #openstack-neutron07:54
*** Marga_ has joined #openstack-neutron07:56
*** lyanchih has joined #openstack-neutron07:57
*** dedery_ has joined #openstack-neutron07:59
*** iwamoto has quit IRC07:59
openstackgerritMerged openstack/neutron: Add tenant_id to flavor service profiles attributes  https://review.openstack.org/21744008:00
*** dedery has quit IRC08:02
*** links has quit IRC08:02
*** zhipeng has joined #openstack-neutron08:03
*** y_sawai has joined #openstack-neutron08:03
*** kobayashi has quit IRC08:04
*** y_sawai has quit IRC08:08
*** frobware_ has joined #openstack-neutron08:08
*** y_sawai has joined #openstack-neutron08:10
*** y_sawai has quit IRC08:10
*** y_sawai has joined #openstack-neutron08:11
openstackgerritMerged openstack/neutron: Enable to update external network subnet's gateway-ip  https://review.openstack.org/19493908:11
*** txomon|fon has quit IRC08:11
openstackgerritMerged openstack/neutron: Make Neutron service flavor save service_type  https://review.openstack.org/21853408:12
openstackgerritMerged openstack/neutron: Deprecate --service option for neutron-db-manage  https://review.openstack.org/21945608:12
rossella_sgongysh ping08:13
*** zhipeng has quit IRC08:14
openstackgerritMerged openstack/neutron-fwaas: Removing unused dependency: discover  https://review.openstack.org/21842008:14
*** y_sawai has quit IRC08:15
*** zhipeng has joined #openstack-neutron08:15
*** zhipeng_ has joined #openstack-neutron08:23
*** txomon|fon has joined #openstack-neutron08:24
*** zhipeng has quit IRC08:25
*** vikram has quit IRC08:25
*** zhipeng_ has quit IRC08:27
*** y_sawai has joined #openstack-neutron08:27
*** zhipeng has joined #openstack-neutron08:29
*** gampel has joined #openstack-neutron08:30
*** yfried_ is now known as yfried|afk08:31
*** gampel1 has quit IRC08:31
*** ygbo has joined #openstack-neutron08:32
*** l6unchpad has joined #openstack-neutron08:34
*** feleouet has joined #openstack-neutron08:36
*** achanda has quit IRC08:36
*** jistr has joined #openstack-neutron08:37
*** hichihara has quit IRC08:37
*** zhipeng has quit IRC08:39
*** zhipeng has joined #openstack-neutron08:41
*** bharathm has joined #openstack-neutron08:41
*** doude_ has quit IRC08:44
*** y_sawai has quit IRC08:46
*** tfukushima has quit IRC08:46
*** y_sawai has joined #openstack-neutron08:49
*** tfukushima has joined #openstack-neutron08:51
*** yfried|afk is now known as yfried_08:51
*** bobmel has quit IRC08:51
*** diga has joined #openstack-neutron08:52
*** y_sawai has quit IRC08:54
*** dedery_ has quit IRC08:55
*** dedery has joined #openstack-neutron08:55
*** neeti has quit IRC08:55
openstackgerritNeil Jerram proposed openstack/neutron: Use US spelling for 'behavior', instead of 'behaviour'  https://review.openstack.org/22003908:59
*** anilvenkata has quit IRC09:00
*** l6unchpad has quit IRC09:01
yongshenggong__pong rossella_s>09:02
*** zhipeng has quit IRC09:02
*** neeti has joined #openstack-neutron09:02
rossella_syongshenggong__ so you are gongysh ?09:04
yongshenggong__yes09:05
yongshenggong__gongysh is taken by my machine at office.09:05
rossella_sgood to know...I wanted to bother you again regarding https://review.openstack.org/#/c/211191/09:05
*** vikram has joined #openstack-neutron09:08
*** ccard has joined #openstack-neutron09:08
yongshenggong__rossella_s: it is good for the dependent patch. I think in this case it makes sense. but why not just merge these two patches?09:09
yongshenggong__but anyway, it is good to remove the local states keeping.09:09
rossella_syongshenggong__, I tried to make small reviewable patches09:09
yongshenggong__yes, but small patch sometimes does not make obvious point for change. thanks09:11
*** john-davidge has joined #openstack-neutron09:12
*** yfried_ is now known as yfried|afk09:12
yongshenggong__rossella_s:  I scored +109:12
*** yamamoto has quit IRC09:13
*** yongshenggong__ has quit IRC09:14
*** john-davidge_ has joined #openstack-neutron09:15
*** alex_klimov has quit IRC09:15
*** alex_klimov has joined #openstack-neutron09:15
*** SridharG has quit IRC09:16
*** john-davidge has quit IRC09:16
*** john-davidge_ is now known as john-davidge09:16
ZZelleSteap, hi09:17
*** Marga_ has quit IRC09:17
*** Marga_ has joined #openstack-neutron09:17
*** vthapar has quit IRC09:19
SteapZZelle: hi09:20
*** jamespd has quit IRC09:22
*** gongysh__ has joined #openstack-neutron09:22
*** ihrachys has joined #openstack-neutron09:23
*** l6unchpad has joined #openstack-neutron09:23
*** Marga_ has quit IRC09:24
*** Marga_ has joined #openstack-neutron09:24
*** evgenyf has quit IRC09:26
*** vthapar has joined #openstack-neutron09:26
ZZelleSteap, could you (or Victor?) look at https://review.openstack.org/219389? One test is still not passing ... and i have a poor understanding of unicode/utf8 en/decoding09:26
SteapI was looking at this09:27
Steapwill ask Victor09:27
*** gongysh__ has quit IRC09:28
*** jamespd has joined #openstack-neutron09:28
Steapworst case scenario, we just skip this test as well :)09:28
*** ivar-lazzaro has joined #openstack-neutron09:29
ZZelleSteap, i fear it could a bigger trouble with string encoding/decoding09:29
ZZelleSteap, i fear it could HIDE a bigger trouble with string encoding/decoding09:29
*** Marga_ has quit IRC09:29
*** gongysh__ has joined #openstack-neutron09:30
Steapthis is usually a huge PITA indeed09:30
*** yfried|afk is now known as yfried_09:32
*** vthapar has quit IRC09:32
*** vthapar_ has joined #openstack-neutron09:32
*** SridharG has joined #openstack-neutron09:33
*** gongysh__ has quit IRC09:34
*** vthapar_ has quit IRC09:36
*** gongysh__ has joined #openstack-neutron09:37
*** jbell8 has joined #openstack-neutron09:37
ihrachysSteap: if it takes a long time, then I think it's better to merge the fix with the skip, and then follow-up, to avoid lack of test coverage while you investigate.09:38
ihrachysZZelle: btw thanks for looking into the patch, really appreciated, and great progress!09:38
Steapwould be nice to merge this today09:38
Steapbefore thte FF09:38
*** gongysh__ has quit IRC09:42
*** jbell8 has quit IRC09:42
*** Marga_ has joined #openstack-neutron09:42
*** gongysh__ has joined #openstack-neutron09:42
*** apuimedo has joined #openstack-neutron09:42
*** vthapar has joined #openstack-neutron09:44
ZZelleSteap, ihrachys, +1 to test skip09:45
ajoping moshele09:47
ajomoshele: were you able to check the random failures of https://review.openstack.org/#/c/214218/?09:48
*** gongysh__ has quit IRC09:48
ajoI'm writing a patch to fix https://bugs.launchpad.net/neutron/+bug/1486039 that modifies the logic a bit09:48
openstackLaunchpad bug 1486039 in neutron "Setting a policy to a network will limit the router/dhcp/net-device ports, that's not expected" [High,New] - Assigned to Miguel Angel Ajo (mangelajo)09:48
ajoI guess they will conflict a bit, but I can help to rebase yours09:49
ajoor may be09:49
ajoI can just build it on top of yours09:49
ajothat's probably going to be easier09:49
ajono need to replicate the sriov logic09:49
*** gongysh__ has joined #openstack-neutron09:49
*** y_sawai has joined #openstack-neutron09:50
*** y_sawai_ has joined #openstack-neutron09:51
*** miyagishi_t has quit IRC09:52
*** gampel has quit IRC09:54
*** gongysh__ has quit IRC09:54
mosheleajo: I am not sure I will be able today I want to finish the SR-IOV part in https://review.openstack.org/#/c/214055/, also yesterday my devstack env was ruined I just now able to recreate it :(09:54
*** y_sawai has quit IRC09:54
*** gongysh__ has joined #openstack-neutron09:54
*** bobmel has joined #openstack-neutron09:55
ajomoshele, ack, I will try to find the bug in your review once I finish my bugfix09:56
*** y_sawai_ has quit IRC09:56
mosheleajo: thanks :) I may continue also tomorrow because I have to leave early today09:57
ajomoshele :) np, I just wanted to let you know,09:57
*** jlanoux has quit IRC09:58
*** gampel has joined #openstack-neutron09:58
*** gongysh__ has quit IRC10:00
*** jlanoux has joined #openstack-neutron10:02
*** Marga_ has quit IRC10:02
*** Marga_ has joined #openstack-neutron10:02
*** neeti has quit IRC10:07
*** bharathm has quit IRC10:07
*** neeti has joined #openstack-neutron10:07
*** evgenyf has joined #openstack-neutron10:09
apuimedogsagie: I sent some comments to your patch ;-)10:11
*** frobware_ has quit IRC10:12
*** kbyrne has quit IRC10:12
*** sc68cal has quit IRC10:13
*** yamamoto has joined #openstack-neutron10:13
*** tfukushima has quit IRC10:13
openstackgerritIhar Hrachyshka proposed openstack/neutron: Revert "Decentralize the managemement of service providers"  https://review.openstack.org/22007010:15
*** kbyrne has joined #openstack-neutron10:15
*** vivek-ebay has joined #openstack-neutron10:16
*** yamamoto has quit IRC10:18
*** nyechiel has joined #openstack-neutron10:19
*** vivek-ebay has quit IRC10:20
*** l6unchpad has quit IRC10:20
*** BhavyaM has quit IRC10:21
*** BhavyaM has joined #openstack-neutron10:22
*** ybabenko has quit IRC10:23
rossella_sihrachys is it too late to approve patches? I am not sure how it work with the so close release cut10:25
ihrachysrossella_s: in which patch have I violated something?10:26
*** ekuris has quit IRC10:26
rossella_sihrachys I am not saying you violated anything...I am just asking you since you are an "old" core. I just want to know how I should behave...if I should approve patches or not10:27
*** ildikov has quit IRC10:28
ihrachysrossella_s: ok I see armax email now.10:28
rossella_sihrachys, I missed that actually, reading it now10:28
*** oomichi is now known as oomichi_away10:29
ihrachysrossella_s: well, I just pushed a test only patch for gate. I believe that one is ok. but generally, I guess now the armax email should be considered, I guess we should mind the gate (if queue is long, refrain to push without approval from armax), and invasiveness10:29
ihrachysrossella_s: well, that's cool you notified me to check, I didn't see the email too ;)10:29
ihrachysrossella_s: am I an old core?.. I believe I'm like 4 months in the position :)10:30
rossella_sihrachys and actually I didn't notice your patch. I picked you because as far as I know you are the "older" core online now10:30
ihrachysok, even if you picked me for the patch, it's absolutely fine to call out :)10:31
rossella_syes it's fine but that was not my intention... :)10:31
*** links has joined #openstack-neutron10:31
*** pc_m has joined #openstack-neutron10:35
*** l6unchpad has joined #openstack-neutron10:35
*** yfried_ has quit IRC10:37
*** nplanel_ has joined #openstack-neutron10:37
*** mkolesni has quit IRC10:38
*** jschwarz has quit IRC10:39
*** anilvenkata has joined #openstack-neutron10:40
*** SridharG1 has joined #openstack-neutron10:40
openstackgerritMerged openstack/neutron: Add info to debug test_keepalived_respawns gate failure  https://review.openstack.org/21857810:42
*** claudiub has joined #openstack-neutron10:44
*** Kennan2 is now known as Kennan_on_vacati10:44
*** SridharG has quit IRC10:44
*** Kennan_on_vacati is now known as Kennan_Vacation10:45
*** aix has quit IRC10:46
*** ZZelle has quit IRC10:48
*** ZZelle has joined #openstack-neutron10:48
*** tfukushima has joined #openstack-neutron10:48
*** kawa2014 has quit IRC10:49
*** kawa2014 has joined #openstack-neutron10:50
*** evgenyf has quit IRC10:50
*** ihrachys has quit IRC10:52
*** neeti has quit IRC10:52
*** itzikb has quit IRC10:53
*** banix has joined #openstack-neutron10:54
*** neeti has joined #openstack-neutron10:57
*** mattt has left #openstack-neutron10:57
*** vikschw has joined #openstack-neutron10:58
openstackgerritMerged openstack/neutron-fwaas: Updated from global requirements  https://review.openstack.org/21890910:58
*** vikschw has quit IRC11:00
*** hdaniel has joined #openstack-neutron11:00
*** lazy_prince has joined #openstack-neutron11:02
*** y_sawai has joined #openstack-neutron11:03
apuimedogsagie: banix: diga: rossella_s: irenab: tfukushima: daneyon_: Irena and I have updated the virtual sprint etherpad11:03
apuimedohttps://etherpad.openstack.org/p/kuryr-virtual-sprint11:03
apuimedoit's starting to be ready ;-)11:03
digaapuimedo: Great! will look at it11:04
tfukushimaapuimedo: thanks11:04
*** y_sawai has quit IRC11:05
digaapuimedo: I need your inputs on this - https://review.openstack.org/#/c/214002/13/usr/libexec/kuryr/binding/ovs.py11:05
digaapuimedo: blocked at this, How to proceed on this11:05
*** y_sawai has joined #openstack-neutron11:05
apuimedodiga: sorry, I thought I commented here on #neutron that I agreed with Irena11:06
apuimedoI also tend to agree with ajo that this executable would be much simpler if it were just the bash call11:07
*** ivar-lazzaro has quit IRC11:07
apuimedoand it being a bash script11:07
apuimedoit gets the input as args11:07
*** bharathm has joined #openstack-neutron11:08
digaokay11:08
apuimedoand then the communication back via stdout, stderr11:08
*** gparaskevas has quit IRC11:08
apuimedoand return code11:08
apuimedofor exceptions11:08
apuimedo0 -> fine11:08
digaokay11:08
apuimedoanything else it is an exception11:08
apuimedodiga: thanks ;-)11:08
digacan we import nova_vif library here ?11:09
apuimedono11:10
digaok11:10
*** y_sawai has quit IRC11:10
digaI will write bash script for this11:10
apuimedothe executables should be as simple as possible11:10
digaokay11:10
apuimedoI added it as a review comment now11:11
* apuimedo going for lunch11:11
*** apuimedo is now known as apuimedo|lunch11:11
*** Marga_ has quit IRC11:12
digaapuimedo|lunch: thanks :) will take a look at it11:12
*** bharathm has quit IRC11:12
banixapuimedo|lunch: ok thx11:13
openstackgerritSridhar Gaddam proposed openstack/neutron: Add IPv6 Address Resolution protection  https://review.openstack.org/20165011:13
*** aix has joined #openstack-neutron11:15
*** banix has quit IRC11:15
*** shoutm has joined #openstack-neutron11:17
*** shoutm_ has quit IRC11:18
*** y_sawai has joined #openstack-neutron11:18
*** lyanchih has quit IRC11:19
*** y_sawai has quit IRC11:23
*** sayali has quit IRC11:26
*** alex_klimov has quit IRC11:26
*** yamamoto has joined #openstack-neutron11:29
*** nyechiel has quit IRC11:31
*** BhavyaM has quit IRC11:32
*** ybabenko has joined #openstack-neutron11:32
*** ybabenko has quit IRC11:32
*** nplanel_ has quit IRC11:33
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: create_router: Handle the case with a surrounding transaction  https://review.openstack.org/21921511:33
openstackgerritAnn Kamyshnikova proposed openstack/neutron: [neutron-db-manage] Add check for branches  https://review.openstack.org/22009111:34
*** yamamoto has quit IRC11:34
*** Marga_ has joined #openstack-neutron11:35
*** ociuhandu has quit IRC11:36
*** julim has joined #openstack-neutron11:37
*** thorst has joined #openstack-neutron11:37
*** ildikov has joined #openstack-neutron11:39
*** y_sawai has joined #openstack-neutron11:39
*** ushkalim has quit IRC11:40
*** ushkalim has joined #openstack-neutron11:40
*** y_sawai has quit IRC11:43
openstackgerritMoshe Levi proposed openstack/neutron: Qos SR-IOV: Refactor extension delete to get mac and pci slot  https://review.openstack.org/21405511:43
*** y_sawai has joined #openstack-neutron11:43
*** ihrachys has joined #openstack-neutron11:44
*** ihrachys has quit IRC11:44
*** ihrachys has joined #openstack-neutron11:46
*** nplanel_ has joined #openstack-neutron11:46
*** y_sawai_ has joined #openstack-neutron11:47
*** amotoki has quit IRC11:47
*** neeti has quit IRC11:47
*** y_sawai has quit IRC11:48
*** nplanel_ has quit IRC11:50
*** y_sawai_ has quit IRC11:51
*** breitz has quit IRC11:51
*** breitz has joined #openstack-neutron11:51
*** shaohe_feng has joined #openstack-neutron11:52
*** ociuhandu has joined #openstack-neutron11:55
*** alexpilotti_ has joined #openstack-neutron11:58
*** amuller has joined #openstack-neutron11:58
*** akamyshnikova has quit IRC11:58
mosheleajo, ihrachys can you review again https://review.openstack.org/214055 ?11:59
*** mohankumar has joined #openstack-neutron12:01
*** Marga_ has quit IRC12:01
*** vthapar has quit IRC12:02
ihrachysmoshele: hm, why isn't the blueprint scheduled for L?12:02
ihrachysmoshele: 'Is pci_lib used by anything outside the sr-iov driver?' ?12:04
mosheleihrachys: no it is part of the sriov mech driver only12:05
mosheleihrachys: I don't think so12:05
*** alex_klimov has joined #openstack-neutron12:05
mosheleihrachys: I update the bp for L12:06
*** thurloat_isgone is now known as thurloat12:10
*** mohankumar has quit IRC12:11
*** shaohe_feng has quit IRC12:11
*** shaohe_feng has joined #openstack-neutron12:11
ihrachysmoshele: yes, it should be included12:12
ihrachysmoshele: replied to the patch12:12
*** mohankumar has joined #openstack-neutron12:14
*** jpena is now known as jpena|lunch12:15
mosheleihrachys: what should be included?12:15
ihrachysmoshele: the bp - in L release12:15
mosheleihrachys: sure I updated to be in L12:16
irenabgaryk: hi12:17
mosheleihrachys: I failed to understand you concern12:18
moshele regrading the port_update12:18
*** deepthi has quit IRC12:19
irenabgaryk: can you please take a look at https://review.openstack.org/#/c/199112/ , updates to the QoS spec12:21
ihrachysmoshele: so... before the change, all ports got into updated_devices, right? but now you seem to put there only those that have pci_slot set. I wonder whether you are sure it won't have some other unexpected impact on the code that uses the updated_devices set12:21
*** nyechiel has joined #openstack-neutron12:22
garykirenab: ok, i'll try12:23
irenabgaryk: thanks12:23
mosheleihrachys: I can double check, but we will get only direct/mavtap vnic_type which has to have pci_slot, unless there is bug in nova12:24
*** absubram has quit IRC12:24
*** SridharG1 has left #openstack-neutron12:24
garykirenab: is liberty no in freeze now?12:24
*** SridharG has joined #openstack-neutron12:24
mosheleihrachys: I added the check for safety  is should not happen12:24
irenabgaryk: it is pec update to reflect changes to the code that is already merged for Liberty. I am not sure if freeze applies to spec repo. Is it?12:25
garykirenab: ah, ok12:26
garykirenab: we need one of the guys who is on the spec team to approve.12:26
irenabgaryk: it has already +2 from ajo and ihrachys12:27
irenabihrachys: can you approve the spec update?12:27
*** VikasC has quit IRC12:28
*** shaohe_feng has quit IRC12:28
garykirenab: it also has from me, but none of us can approve it. that can only be done by the drivers team. that is kind of like the a-team. i wonder who mt-t is. i think that armando could be12:28
irenabgaryk: ok, thanks!12:29
openstackgerritPaul Michali proposed openstack/neutron-vpnaas: VPNaaS: Add debugging to tox_install.sh  https://review.openstack.org/20320112:30
*** ivar-lazzaro has joined #openstack-neutron12:33
*** baoli has joined #openstack-neutron12:33
openstackgerritOleg Bondarev proposed openstack/neutron: OVS agent: flush firewall rules for all deleted ports at once  https://review.openstack.org/22011512:34
*** baoli_ has joined #openstack-neutron12:37
*** dedery has quit IRC12:38
*** baoli has quit IRC12:38
*** dims has joined #openstack-neutron12:41
*** njohnston is now known as nate_gone12:42
*** sayali has joined #openstack-neutron12:42
*** y_sawai has joined #openstack-neutron12:43
*** pradk has quit IRC12:45
*** mohankumar has quit IRC12:46
*** shoutm_ has joined #openstack-neutron12:47
*** gildub has quit IRC12:47
*** kairo has joined #openstack-neutron12:48
*** shoutm has quit IRC12:48
*** y_sawai has quit IRC12:48
*** diga has quit IRC12:48
*** anshul has quit IRC12:49
*** dguitarbite has quit IRC12:51
*** thurloat is now known as thurloat_isgone12:52
*** sayali has quit IRC12:52
*** dguitarbite has joined #openstack-neutron12:52
*** akamyshnikova has joined #openstack-neutron12:52
*** rpothier has joined #openstack-neutron12:52
*** jagter has quit IRC12:54
*** tbachman has quit IRC12:54
*** ivar-lazzaro has quit IRC12:55
*** chutwig has joined #openstack-neutron12:55
*** sayali has joined #openstack-neutron12:56
*** lazy_prince has quit IRC12:57
*** nplanel has quit IRC12:58
*** jagter has joined #openstack-neutron13:01
*** rossella_s has quit IRC13:06
*** rossella_s has joined #openstack-neutron13:06
*** nyechiel has quit IRC13:07
openstackgerritAssaf Muller proposed openstack/neutron: Retain logs for functional test cases  https://review.openstack.org/21993513:11
*** nplanel has joined #openstack-neutron13:12
openstackgerritvenkata anil proposed openstack/neutron: select router with subnet's gateway_ip for floatingip  https://review.openstack.org/22013513:12
*** tbh has joined #openstack-neutron13:12
*** lazy_prince has joined #openstack-neutron13:13
*** edmondsw has joined #openstack-neutron13:13
*** yamamoto has joined #openstack-neutron13:14
*** jckasper has joined #openstack-neutron13:16
*** itlinux has joined #openstack-neutron13:16
*** sayali has quit IRC13:16
*** arajagopal has joined #openstack-neutron13:16
*** evgenyf has joined #openstack-neutron13:17
*** cbouch has joined #openstack-neutron13:19
*** yamamoto has quit IRC13:19
ihrachysirenab: I am not a driver13:19
*** tbachman has joined #openstack-neutron13:20
ihrachysmoshele: hm, but that port_update, who generates the notification?13:20
irenabihrachys: got it. Added armax to review. Hope this will be merged soon. I do not like the spec to be out of  sync13:20
*** jpena|lunch is now known as jpena13:21
cbouchihrachys:  could you +2 instead of +1 on cherry pick for https://review.openstack.org/#/c/219836/?  Only limited people can +2 there.13:21
*** sayali has joined #openstack-neutron13:22
cbouchiharachys:  Just saw your note.   I'll chat with gessau.13:22
ihrachyscbouch: replied with the rationale of +1 and not +2. sorry. you can ask garyk.13:22
ihrachysor someone else. I don't believe it's correct to vote with +2 to something that you are not sure about13:22
*** tiswanso has joined #openstack-neutron13:24
HenryGihrachys: no problem, thanks for taking the time to look and comment13:24
*** sayali has quit IRC13:24
cbouchihrachys:  Yes thank you for the review13:24
amullerihrachys: fyi https://review.openstack.org/#/c/219935/13:25
*** ivar-lazzaro has joined #openstack-neutron13:26
*** sayali has joined #openstack-neutron13:27
garykihrachys: am i the stable rubber stamp?13:28
garykcbouch: i feel that due to the fact that this has the henry's thumbs up then i am fine with this.13:29
*** amotoki has joined #openstack-neutron13:29
garyki feel that stable core privileges should be given to the decomposed teams. that is something we should talk about at a meeting when mestery is back13:30
*** lori|away is now known as lori13:30
ihrachysgaryk: yes, teams should have at least one +2 stable core for the project13:31
ihrachysanother one is ok to have from horizontal team, to validate bureaucracy and general sanity13:31
ihrachysI don't know much about any vendor repo, and frankly, I don't want to get involved too deep13:32
*** tbh has quit IRC13:33
*** numans has quit IRC13:33
cbouchgaryk:  Thank you!13:34
openstackgerritOleg Bondarev proposed openstack/neutron: OVS agent: handle deleted ports on each rpc_loop iteration  https://review.openstack.org/22014013:34
garykcbouch: np. thanks for doing the backports.13:34
garykmy two cents that vendors doing the backports and making sure that the decomposed versions are stable worthy is a healthy move13:35
garyka luta continua13:35
*** links has quit IRC13:40
*** l6unchpad has quit IRC13:41
*** lyanchih has joined #openstack-neutron13:41
*** hdaniel has quit IRC13:43
openstackgerritBrent Eagles proposed openstack/neutron-vpnaas: Manage cleanup of .ctl/.pid files for LibreSwan  https://review.openstack.org/21267313:43
*** shaohe_feng has joined #openstack-neutron13:43
*** y_sawai has joined #openstack-neutron13:44
*** moshele has quit IRC13:44
akamyshnikovaihrachys: Hi, I answered on some your comments on https://review.openstack.org/#/c/220091/13:46
*** abregman is now known as abregman|afk13:46
*** ajmiller has joined #openstack-neutron13:47
*** kbringard has joined #openstack-neutron13:47
ihrachysakamyshnikova: yeah, I saw, I believe I agree with all that is said :)13:48
*** y_sawai has quit IRC13:49
*** lyanchih has quit IRC13:50
*** anshul has joined #openstack-neutron13:51
*** zzzeek has joined #openstack-neutron13:51
akamyshnikovaihrachys: OK, Thanks!13:51
*** ajmiller has quit IRC13:51
*** l6unchpad has joined #openstack-neutron13:52
*** alexpilotti_ is now known as alexpilotti13:53
*** DanyC has joined #openstack-neutron13:53
*** DanyC has left #openstack-neutron13:53
*** pradk has joined #openstack-neutron13:54
*** jroll has quit IRC13:54
*** jroll has joined #openstack-neutron13:54
*** sayali has quit IRC13:57
*** nate_gone is now known as njohnston13:57
*** rtheis has joined #openstack-neutron13:58
*** nyechiel has joined #openstack-neutron13:58
*** shaohe_feng1 has joined #openstack-neutron13:59
*** shaohe_feng has quit IRC13:59
*** korzen has joined #openstack-neutron13:59
*** sayali has joined #openstack-neutron13:59
*** clev is now known as clev-away14:00
*** clev-away is now known as clev14:00
*** clev is now known as clev-away14:00
*** nplanel has quit IRC14:00
*** clev-away is now known as clev14:00
*** abregman|afk is now known as abregman14:01
*** SridharG has quit IRC14:01
*** rushil has joined #openstack-neutron14:04
openstackgerritMoshe Levi proposed openstack/neutron: Qos SR-IOV: Refactor extension delete to get mac and pci slot  https://review.openstack.org/21405514:05
*** txomon|fon has quit IRC14:06
*** apuimedo|lunch has quit IRC14:08
*** rushil has quit IRC14:09
*** bobmel has quit IRC14:09
*** korzen has quit IRC14:10
*** apuimedo|lunch has joined #openstack-neutron14:12
*** n00btube has joined #openstack-neutron14:12
*** regXboi has joined #openstack-neutron14:13
*** n00btube has quit IRC14:13
openstackgerritPaul Michali proposed openstack/neutron-vpnaas: VPNaaS: Add debugging to tox_install.sh  https://review.openstack.org/20320114:14
*** vivek-ebay has joined #openstack-neutron14:17
*** dane_leblanc has joined #openstack-neutron14:17
*** bobmel has joined #openstack-neutron14:18
*** shoutm_ has quit IRC14:18
*** dave-mccowan has quit IRC14:18
*** txomon|fon has joined #openstack-neutron14:19
*** mlavalle has joined #openstack-neutron14:20
*** Murali has quit IRC14:21
*** njohnston is now known as nate_gone14:22
*** vivek-ebay has quit IRC14:22
*** yamahata has joined #openstack-neutron14:22
*** evgenyf has quit IRC14:23
SteapZZelle: so, should we update that "last" py3 patch ?14:23
*** ajmiller has joined #openstack-neutron14:24
*** kbringard has quit IRC14:26
openstackgerritMiguel Angel Ajo proposed openstack/neutron: QoS agent extension and driver refactoring  https://review.openstack.org/21421814:26
openstackgerritMiguel Angel Ajo proposed openstack/neutron: No network devices on network attached qos policies  https://review.openstack.org/22016414:26
*** karthiks has quit IRC14:27
*** kbringard has joined #openstack-neutron14:28
*** rushil_ has joined #openstack-neutron14:28
*** anilvenkata has left #openstack-neutron14:29
*** Piet has quit IRC14:30
*** dave-mccowan has joined #openstack-neutron14:31
amullerihrachys: about tempest_set_log_file14:33
*** devlaps has quit IRC14:33
amullerihrachys: any idea of an alternative way of accomplishing what we need without using a super duper private do-not-use mega method of doom?14:33
ihrachysamuller: maybe passing --log-file in BaseTestCase.config_parse?14:34
amullerihrachys: let me try14:35
amullerit14:35
mlavallepc_m: ping14:36
*** chutwig has quit IRC14:36
pc_mdougwig: ping14:36
pc_mmlavalle: pong14:37
mlavallepc_m: I am digging into https://bugs.launchpad.net/neutron/+bug/1461172, that has been makin some noise lately14:37
openstackLaunchpad bug 1461172 in neutron "neutron.tests.functional.agent.test_l3_agent.MetadataL3AgentTestCase.test_access_to_metadata_proxy times out intermittently" [Critical,Confirmed] - Assigned to Miguel Lavalle (minsel)14:37
amullerihrachys: reading oslo logging code that should be doable14:37
*** markstur has quit IRC14:38
*** prad- has joined #openstack-neutron14:38
*** cbouch has quit IRC14:38
*** sbelous has quit IRC14:39
mlavallepc_m: it seems you have a patchset, https://review.openstack.org/#/c/196149, that creates a failure on purpose. This failure is what I found in the latest ocurrence of the bug in logstach14:39
*** sbelous_ has joined #openstack-neutron14:39
*** sayali has quit IRC14:40
mlavallepc_m: if you go to the latest comment in the bug, you can read more about my finding14:40
*** prad- has quit IRC14:41
*** dmartls1 has joined #openstack-neutron14:41
pc_mmlavalle: 196149 is just a dummy commit that should fail some of the VPN functional tests.14:41
pc_mmlavalle: I'm trying to test out a new experimental job that would run on Neutron patch sets, and invoke the VPN functional tests, to check cross repo breakages.14:42
mlavallepc_m: does thos VPN functional test involve L3 agent functional tests?14:42
mlavallethat^^^14:42
pc_mmlavalle: Currently, when 196149 runs it is NOT using the actual Neutron patchset for the VPN test.14:42
*** Marga_ has joined #openstack-neutron14:43
pc_mmlavalle: It is a VPN functional job (pair actually) that sets up devstack, and then runs a VPN scenario test to make sure that VPN agent is working correctly.14:43
*** itlinux has quit IRC14:43
*** sbelous has joined #openstack-neutron14:44
mlavallepc_m: ok, I might be querying logstash wrongly.... will keep investigating14:44
pc_mNow, if 196149 is run against neutron tests, it'll probably fail on tests that do rootwrap stuff (as it changes the return code to indicate an error).14:44
pc_mmlavalle: please ignore any test results for 196149.14:45
pc_m(hipefully it isn't throwing off the logstash queries)14:45
*** y_sawai has joined #openstack-neutron14:45
*** sbelous has quit IRC14:45
*** sbelous has joined #openstack-neutron14:45
mlavallepc_m: yeah, but if I am querying logstash right, then the patchset is throwing it off signifcantly.... I'll just keep an eye on it and will confirm that I am queruing correctly14:46
mlavallepc_m: for the time being, you ignore me... thanks :-)14:46
pc_mmlavalle: OK. If it is causing problems with you researching the bug, let me know and I can defer to next week (I'm on PTO tomorrow).14:48
*** phil_h has joined #openstack-neutron14:48
mlavallepc_m: thanks. I'll mention this in the L3 meeting in a few minutes14:48
*** tbachman has quit IRC14:48
*** dmartls1 has quit IRC14:49
pc_msee you there14:49
*** gampel has quit IRC14:49
openstackgerritAssaf Muller proposed openstack/neutron: Retain logs for functional test cases  https://review.openstack.org/21993514:49
amullerihrachys: done14:50
amullerihrachys: now question is are we actually getting all of the logs14:50
*** y_sawai has quit IRC14:50
amullerihrachys: the job is running 198 tests, I don't think we're getting 198 logs14:50
*** mickeys has joined #openstack-neutron14:51
*** vikram__ has joined #openstack-neutron14:51
*** dmartls1 has joined #openstack-neutron14:52
*** carl_baldwin has joined #openstack-neutron14:52
*** wznoinsk has quit IRC14:52
amullermlavalle: I'll be joining the L3 meeting a bit late, but I'd love to get a mention on that bug :)14:52
*** amuller is now known as amuller_afk14:53
*** anshul has quit IRC14:53
*** txomon|fon has quit IRC14:54
*** y_sawai has joined #openstack-neutron14:54
*** maishsk has quit IRC14:57
mlavalleamuller_afk: if you mis it during the l3 meeting, ping me later and we can chat about it14:58
*** tbachman has joined #openstack-neutron14:58
*** abregman is now known as abregman|afk14:58
*** johnbelamaric has joined #openstack-neutron14:58
*** y_sawai has quit IRC14:59
*** emagana has joined #openstack-neutron15:01
*** jistr is now known as jistr|call15:02
*** ronis has quit IRC15:02
*** Leonr has joined #openstack-neutron15:02
*** armax has joined #openstack-neutron15:03
*** rushil_ has quit IRC15:04
*** kbringard has quit IRC15:04
*** rotbeard has quit IRC15:04
*** yamamoto has joined #openstack-neutron15:05
*** Marga_ has quit IRC15:05
*** sayali has joined #openstack-neutron15:05
*** TrevorV has joined #openstack-neutron15:07
openstackgerritgongysh proposed openstack/neutron: Add list routes  https://review.openstack.org/21847915:07
*** markvoelker has joined #openstack-neutron15:08
*** txomon|fon has joined #openstack-neutron15:08
*** txomon|fon has quit IRC15:08
*** jbell8 has joined #openstack-neutron15:08
*** arajagopal has quit IRC15:08
*** sayali has quit IRC15:10
*** moha_hunt has joined #openstack-neutron15:10
*** tbachman has quit IRC15:12
*** chutwig has joined #openstack-neutron15:14
*** nate_gone is now known as njohnston15:14
*** amuller_afk is now known as amuller15:14
*** ivar-lazzaro has quit IRC15:15
*** akamyshnikova has quit IRC15:15
pc_marmax: hi15:15
*** alexpilotti_ has joined #openstack-neutron15:16
*** wwriverrat has quit IRC15:17
*** alexpilotti has quit IRC15:17
*** alexpilotti_ is now known as alexpilotti15:17
*** shwetaap has joined #openstack-neutron15:18
*** jprovazn has joined #openstack-neutron15:19
*** rkukura has quit IRC15:20
*** shaohe_feng1 has quit IRC15:21
*** annegentle has joined #openstack-neutron15:21
*** tbachman has joined #openstack-neutron15:22
*** sayali has joined #openstack-neutron15:22
*** SumitNaiksatam has quit IRC15:23
*** SumitNaiksatam has joined #openstack-neutron15:23
ihrachysarmax: hey. I have stuff to discuss around provider_configuration.15:23
armaxihrachys: I was just going through the bug report15:23
armaxihrachys: let me catch up15:23
ihrachysarmax: cool. we had discussion with dhellmann in #*-oslo too, you can read latest logs15:24
*** srilatha has joined #openstack-neutron15:24
*** kbringard has joined #openstack-neutron15:25
*** dims has quit IRC15:26
armaxihrachys: what is puzzling to me is how we regressed15:26
ihrachysarmax: easy: we don't gate neutron against puppet or delorean15:26
*** maishsk has joined #openstack-neutron15:26
armaxihrachys: since I haven’t touched the logic of loading the config15:26
armaxihrachys: not that, that I get15:27
ihrachysarmax: haven't you? config_dir usage is in new code only15:27
armaxno15:27
armaxihrachys: it was here: https://review.openstack.org/#/c/202207/23/neutron/common/repos.py15:27
armaxalready15:27
ihrachysah nope, sorry, indeed it's in repos.py15:27
armaxihrachys: something else must have happened15:28
*** neophy has joined #openstack-neutron15:28
armaxihrachys: so config_dir might be a red herring15:28
*** shivrao has joined #openstack-neutron15:28
*** yamahata has quit IRC15:28
ihrachysEmilienM: re that puppet neutron regression: does it work in kilo though?15:28
*** matrohon has quit IRC15:29
ihrachysarmax: the workaround in LP comments suggests it's not red herring, but maybe it's not that patch?15:29
armaxihrachys: um that I don’t know yet, but it looks too much of a coincidence15:29
*** gampel has joined #openstack-neutron15:29
*** arajagopal has joined #openstack-neutron15:30
*** sthillma has joined #openstack-neutron15:30
*** emagana has quit IRC15:30
*** shivrao_ has joined #openstack-neutron15:30
*** sthillma_ has joined #openstack-neutron15:31
*** dannywilson has joined #openstack-neutron15:31
EmilienMihrachys: yes15:31
*** emagana has joined #openstack-neutron15:32
*** sayali has quit IRC15:32
ihrachysEmilienM: and it worked for L several days ago15:33
*** shivrao has quit IRC15:33
*** shivrao_ is now known as shivrao15:33
armaxihrachys: going through the logs15:33
EmilienMright15:33
*** tbachman has quit IRC15:33
*** sthillma has quit IRC15:34
*** sthillma_ is now known as sthillma15:34
*** sballe has joined #openstack-neutron15:35
armaxihrachys: this part was removed15:35
armaxihrachys: https://review.openstack.org/#/c/220070/1/neutron/services/provider_configuration.py  line 7515:35
armaxthat’s what happens15:35
armaxI think15:36
armaxRDO must catch up perhaps?15:36
*** dims has joined #openstack-neutron15:36
ihrachysarmax: delorean is always latest master15:36
armaxihrachys: um...15:36
*** sayali has joined #openstack-neutron15:36
ihrachys(unless there are build issues)15:36
*** tbachman has joined #openstack-neutron15:36
* armax keeps digging15:37
*** y_sawai has joined #openstack-neutron15:37
*** lajos-katona has quit IRC15:38
*** y_sawai has quit IRC15:38
ihrachysarmax: why don't we just drop that manual read of config files and rely on oslo.config? what's the limitation that made us avoid the lib?15:38
armaxihrachys: the problem as I understand it is that we have the same config option15:39
armaxnamed the same way15:39
armaxin multiple files15:39
ihrachysarmax: does oslo.config override it instead of merging? I thought MultiStrOpt should merge.15:39
*** gampel1 has joined #openstack-neutron15:40
ihrachysbut that's more common sense than actual knowledge15:40
armaxihrachys: it doesn’t afaik15:40
*** amotoki has quit IRC15:40
ihrachysok, then I get the rationale15:40
armaxihrachys: still going through the logs15:41
*** gampel has quit IRC15:41
*** markvoelker_ has joined #openstack-neutron15:42
ihrachysEmilienM: I think there could be recent changes in puppet for neutron. where can I check? I think I saw some patch recently that tried to adopt liberty. was it merged?15:44
*** jistr|call is now known as jistr15:44
*** markvoelker has quit IRC15:45
*** blogan_ has joined #openstack-neutron15:45
*** wwriverrat has joined #openstack-neutron15:45
*** garyk has quit IRC15:47
*** gampel1 has quit IRC15:47
*** gampel has joined #openstack-neutron15:47
armaxihrachys: see here:15:48
armaxhttp://logs.openstack.org/07/202207/23/check/gate-tempest-dsvm-neutron-full/b425da1/logs/screen-q-svc.txt.gz#_2015-09-01_04_03_27_84115:48
armaxhttp://logs.openstack.org/50/216950/5/check/gate-puppet-neutron-puppet-beaker-rspec-dsvm-centos7/9e7e510/logs/neutron/server.txt.gz#_2015-09-01_12_40_22_73415:48
armaxin the second log, the service providers are empty15:49
*** sthillma has quit IRC15:49
*** shivrao has quit IRC15:49
*** breitz has quit IRC15:49
*** yamahata has joined #openstack-neutron15:50
openstackgerritDongcan Ye proposed openstack/neutron-vpnaas: Fix AH-ESP transform protocol in ipsec policy  https://review.openstack.org/21878815:50
*** absubram has joined #openstack-neutron15:51
*** bharathm has joined #openstack-neutron15:51
*** blogan__ has joined #openstack-neutron15:52
ihrachysarmax: but in neutron_lbaas.conf, I still see "service_provider=LOADBALANCER:Haproxy:neutron_lbaas.services.loadbalancer.drivers.haproxy.plugin_driver.HaproxyOnHostPluginDriver:default"15:52
*** blogan_ has quit IRC15:52
absubramhi.. could someone please help me get a pointer to the code where the —config-dir, —config-file options are picked up from for the neutron server?15:53
armaxihrachys: https://review.openstack.org/#/c/220070/1/neutron/services/provider_configuration.py something must have changed in parse_service_provider_opt15:53
armaxihrachys: but not quite sure yet why and what15:54
HenryGarmax: sorry to interrupt15:55
HenryGarmax: I suspect it might be critical to get https://bugs.launchpad.net/bugs/1490767 into L-315:56
openstackLaunchpad bug 1490767 in neutron "DB migration of geneve type driver should be 'expand'" [High,In progress] - Assigned to Henry Gessau (gessau)15:56
*** rushil has joined #openstack-neutron15:56
pc_mabsubram: I think it is in neutron/server/__init__.py15:56
*** SourabhP has joined #openstack-neutron15:56
*** vikram has quit IRC15:56
HenryGarmax: otherwise too many users will pick up a bad migration15:57
*** gampel1 has joined #openstack-neutron15:57
absubrampc_m: thanks!15:57
armaxHenryG: it’s too far down the queue15:57
armax:(15:58
EmilienMihrachys: the only change was using rdo trunk15:58
*** nlahouti has joined #openstack-neutron15:58
pc_mabsubram: I think it goes to common/config.py to parse config.15:58
amullerarmax: ihrachys: I just approved https://review.openstack.org/#/c/219935/15:58
dhellmannihrachys, armax : afaict, oslo.config does merge multistr values from different sources: https://review.openstack.org/22021015:58
*** SourabhP_ has joined #openstack-neutron15:58
armaxdhellmann: you rock, thanks for confirming this publicly :)15:58
armaxdhellmann: sometimes else must be at play then15:59
dhellmannarmax: sure. next time ask!15:59
*** afazekas__ has quit IRC15:59
*** gampel has quit IRC16:00
*** jlibosva has quit IRC16:00
*** sridhar_ram has joined #openstack-neutron16:00
ihrachysarmax: well we don't use oslo.config to merge the values16:00
*** maishsk has quit IRC16:00
*** SourabhP has quit IRC16:01
*** SourabhP_ is now known as SourabhP16:01
HenryGarmax: if it does not make L-3 we should probably abandon it instead. /cc ihrachys16:01
*** jlibosva has joined #openstack-neutron16:01
armaxHenryG: why?16:01
absubrampc_m: got it.. any idea is the —config-dir option has been removed?.. it’s what I’m looking for.. for some reason that config is not being picked up16:01
*** ihrachys_ has joined #openstack-neutron16:02
HenryGarmax: because patching the migration timeline should only be done if we are fairly sure it will not affect anyone16:02
armaxHenryG: but L3 is not going to be the final release16:02
armaxHenryG: there is going to be an RC1, etc...16:02
ihrachys_armax: some people may want to rebase their distros to L316:03
armaxihrachys_: they would be nuts, but ok16:03
armaxihrachys_: it’s a possibility16:03
ihrachys_armax: I believe zigo once tracked milestones16:03
*** ihrachys has quit IRC16:05
*** jlanoux has quit IRC16:05
openstackgerritOleg Bondarev proposed openstack/neutron: OVS agent: handle deleted ports on each rpc_loop iteration  https://review.openstack.org/22014016:05
openstackgerritOleg Bondarev proposed openstack/neutron: OVS agent: flush firewall rules for all deleted ports at once  https://review.openstack.org/22011516:05
pc_mabsubram: I'm not really sure.16:06
*** jistr has quit IRC16:06
*** jaypipes has joined #openstack-neutron16:07
*** ijw_ has joined #openstack-neutron16:08
*** blogan__ has quit IRC16:09
*** abregman|afk is now known as abregman16:09
*** blogan_ has joined #openstack-neutron16:09
*** s3wong has joined #openstack-neutron16:10
*** alex_klimov has quit IRC16:11
*** slaweq has joined #openstack-neutron16:11
openstackgerritCedric Brandily proposed openstack/neutron: Enable most unit tests for py34 job  https://review.openstack.org/21938916:11
*** aix has quit IRC16:12
*** Piet has joined #openstack-neutron16:12
*** VikasC has joined #openstack-neutron16:12
*** ygbo has quit IRC16:13
*** crose has joined #openstack-neutron16:14
*** ihrachys_ has quit IRC16:14
*** ihrachys has joined #openstack-neutron16:14
*** ivar-lazzaro has joined #openstack-neutron16:15
*** vikram__ has quit IRC16:17
*** sbelous has quit IRC16:19
*** sbelous has joined #openstack-neutron16:19
HenryGabsubram: why are you asking about --config-dir? It seems armax and ihrachys are discussing some regression with it.16:20
*** ivar-lazzaro has quit IRC16:20
*** neiljerr` has joined #openstack-neutron16:21
*** l6unchpad has quit IRC16:21
armaxihrachys: this is from a build prior to the merge of my patch16:21
*** banix has joined #openstack-neutron16:21
armaxihrachys: http://logs.openstack.org/50/216950/4/check/gate-puppet-neutron-puppet-beaker-rspec-dsvm-centos7/cb6a577/logs/neutron/server.txt.gz#_2015-08-31_19_22_40_42416:21
*** maishsk has joined #openstack-neutron16:21
armaxclearly removing https://review.openstack.org/#/c/220070/1/neutron/services/provider_configuration.py16:21
armaxihrachys line 95-9116:21
armaxihrachys: is upsetting things16:21
*** neiljerram has quit IRC16:21
armaxihrachys: now I wonder why it upsets RDO and not devstack16:22
ihrachysarmax: devstack does not use multiple --config-dirs16:23
*** tmorin has quit IRC16:23
armaxihrachys: gotcha16:23
*** ivar-lazzaro has joined #openstack-neutron16:23
ihrachysarmax: remember I proposed to use it? you said it's yagni :)16:23
absubramHenryG: thanks for pointing this out.. it is excalty the same issue..16:24
absubramthe —config-dir option is not being picked up16:24
rossella_smlavalle if you need help with https://bugs.launchpad.net/neutron/+bug/1461172 feel free to ping me16:25
openstackLaunchpad bug 1461172 in neutron "neutron.tests.functional.agent.test_l3_agent.MetadataL3AgentTestCase.test_access_to_metadata_proxy times out intermittently" [Critical,Confirmed] - Assigned to Miguel Lavalle (minsel)16:25
ihrachysabsubram: can you tell what's your issue?16:25
*** sbelous has quit IRC16:25
*** chutwig has quit IRC16:25
absubramwe are trying out baremetal deployments along with using Heat templates and puppet.. we see the overcloud nodes being deployed16:26
absubramand the ini files all being written correctly16:26
*** sayali has quit IRC16:26
absubrambut neutron-server doesn’t seem to read the ini file we have in the —config-dir option16:26
armaxihrachys: what’s weird though is that the option is not in here:16:27
armaxhttp://logs.openstack.org/50/216950/4/check/gate-puppet-neutron-puppet-beaker-rspec-dsvm-centos7/cb6a577/logs/etc/neutron/neutron.conf.txt.gz16:27
armaxabsubram: we’re actively looking for a fix16:27
ihrachysarmax: it's passed in args16:27
absubramihrachys: ExecStart=/usr/bin/neutron-server --config-file /usr/share/neutron/neutron-dist.conf --config-dir /usr/share/neutron/server --config-file /etc/neutron/neutron.conf --config-file /etc/neutron/plugin.ini --config-dir /etc/neutron/conf.d/common --config-dir /etc/neutron/conf.d/neutron-server --log-file /var/log/neutron/server.log16:27
absubramthat’s how we start neutron16:27
ihrachysabsubram: we ?16:27
armaxihrachys: what is?16:28
absubraminternally here in Cisco :)16:28
ihrachysabsubram: get it. I believe you base on RDO/Delorean16:28
absubramyes16:28
*** chutwig has joined #openstack-neutron16:29
ihrachysso yes, that's the same issue we discuss. puppet-neutron CI was broken because they use delorean among other things16:29
ihrachysarmax: the config-dir option is not defined in neutron.conf, but in args to neutron-server16:29
ihrachysas in snippet above16:30
absubramah.. ok.. thanks armax, ihrachys16:30
armaxihrachys: I meant the service_providers option in neutron.conf16:30
*** vikram__ has joined #openstack-neutron16:30
ihrachysarmax: I believe it should not be put there anymore, right?16:30
armaxihrachys: correct16:30
armaxihrachys: and indeed it isn't16:30
ihrachysarmax: I believe all the mess could be fixed by getting rid of provider_configuration. why not leaving it to oslo.config to read the value? are we still afraid to break someone?16:31
*** bdemers has joined #openstack-neutron16:32
armaxihrachys: at this point no16:32
*** vivek-ebay has joined #openstack-neutron16:32
armaxihrachys: let post a patch to see if I can clear this mess16:32
ihrachysarmax: ok. I honestly feel myself uncapable to come up with better ideas here, I still lack full understanding of the code.16:33
*** neiljerr` has quit IRC16:33
*** ronis has joined #openstack-neutron16:34
*** sayali has joined #openstack-neutron16:35
*** sbelous has joined #openstack-neutron16:36
armaxihrachys: bear with me while I whip something up16:36
*** annegentle has quit IRC16:36
*** SourabhP has quit IRC16:36
*** annegentle has joined #openstack-neutron16:37
*** minwang2 has joined #openstack-neutron16:39
*** afazekas__ has joined #openstack-neutron16:41
*** srilatha has quit IRC16:41
*** vikram__ has quit IRC16:47
*** SourabhP has joined #openstack-neutron16:47
*** nyechiel has quit IRC16:48
*** garyk has joined #openstack-neutron16:49
armaxihrachys: btw, you said that devstack doesn’t use config-dir16:50
ihrachysarmax: not two --config-dir args16:50
ihrachysmaybe one16:50
armaxihrachys: not that I can se16:50
armaxe16:50
armaxhttp://logs.openstack.org/07/202207/23/check/gate-neutron-lbaasv2-dsvm-api/99778b6/logs/screen-q-svc.txt.gz16:50
armaxhere for instance16:50
armaxwe don’t pass the *aas conf file16:50
*** bharathm has quit IRC16:50
ihrachysarmax: because it's loaded by provider_configuration, no?16:51
armaxihrachys: right16:51
*** clev is now known as clev-away16:52
*** Swami has joined #openstack-neutron16:52
armaxso we can use config_dir, but for the time being (to avoid breaking devstack) we need to keep both logics16:52
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Fix fip agent gateway port delete when migrating a fip  https://review.openstack.org/18040816:53
*** vikram has joined #openstack-neutron16:53
*** maishsk has quit IRC16:54
*** s3wong has quit IRC16:54
*** s3wong has joined #openstack-neutron16:55
*** hswayne77 has joined #openstack-neutron16:55
*** klindgren_ has joined #openstack-neutron16:55
klindgren_dougwig, ping16:55
ihrachysarmax: I feel like my brain fails to follow the logic. I will leave now and maybe return later today to check the status. if you don't complete the fix today, please update the bug with findings.16:56
armaxihrachys: I am filing a fix shortly16:56
*** jpena has quit IRC16:56
ihrachysok, I'll check it in 1-2h16:56
*** safchain has quit IRC16:56
armaxihrachys: for the time being I am thinking of allowing both scenarios where one can load neutron by passing config-dir or just a single config-file and let the modules load their providers automagically16:57
*** bharathm has joined #openstack-neutron16:57
*** ihrachys has quit IRC16:57
armaxihrachys: if we intend to keep devstack work the way it does, that’s our best bet16:57
pc_marmax: Casually following the discussion... doesn't VPN use multiple --config-file args?16:58
*** sbelous has quit IRC16:58
armaxpc_m: it should, but can you find out a run from the gate?16:59
pc_marmax: Will check16:59
*** maishsk has joined #openstack-neutron17:02
ajmillergaryk markmcclain could I please get reviews of this stable/kilo backport?  https://review.openstack.org/#/c/216810/17:03
pc_marmax: tempest uses neutron.conf and ml2_conf.ini for neutron. For VPN, the functional tests don't stack, they just configure devstack.17:03
*** lazy_prince has quit IRC17:04
pc_mso guess it is not an issue for VPN tests.17:04
*** rossella_s has quit IRC17:05
*** jlibosva has quit IRC17:06
*** rossella_s has joined #openstack-neutron17:06
*** jlibosva has joined #openstack-neutron17:06
*** nyechiel has joined #openstack-neutron17:07
*** blogan_ has quit IRC17:07
*** afazekas__ has quit IRC17:08
pc_marmax: I see for grenade multiple config-file entries are used when starting the agent.17:12
armaxpc_m: looking17:12
pc_mhttp://logs.openstack.org/49/196149/8/check/gate-grenade-dsvm-neutron/0bed589/logs/grenade.sh.txt.gz#_2015-09-01_12_12_49_45717:12
*** afazekas__ has joined #openstack-neutron17:12
*** maishsk has quit IRC17:12
armaxpc_m: not here:17:13
armaxhttp://logs.openstack.org/49/196149/8/check/gate-grenade-dsvm-neutron/0bed589/logs/grenade.sh.txt.gz#_2015-09-01_12_01_48_64717:13
armaxpc_m: that’s what we care about17:13
*** azbiswas has joined #openstack-neutron17:13
armaxthe server-side rather than the agent side17:13
pc_marmax: gotcha17:13
pc_mwill that still have ml2 and neutron confs?17:14
armaxyes17:14
*** carl_baldwin has quit IRC17:14
*** achanda has joined #openstack-neutron17:14
*** jbell8 has quit IRC17:15
*** kawa2014 has quit IRC17:15
*** madhu_ak has joined #openstack-neutron17:16
*** markvoelker_ has quit IRC17:16
openstackgerritPaul Michali proposed openstack/neutron-vpnaas: VPNaaS: Add debugging to tox_install.sh  https://review.openstack.org/20320117:19
*** tflynn has joined #openstack-neutron17:20
*** achanda has quit IRC17:21
*** sthillma has joined #openstack-neutron17:21
*** julim has quit IRC17:21
*** abhiraut has joined #openstack-neutron17:21
*** afazekas__ has quit IRC17:21
*** kairo has quit IRC17:22
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Delete FIP agent gateway port with external gw port  https://review.openstack.org/19444117:22
dougwigklindgren_: ack17:22
*** achanda has joined #openstack-neutron17:22
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Add RPC command and delete if last FIP on Agent  https://review.openstack.org/19444617:22
pc_marmax: dougwig: Can you guys review https://review.openstack.org/#/c/209887/ that adds experimental job for neutron-client. In Neutron meeting we agreed to have a job for core changes and one for adv. svcs. Infra folks would like to see one of you guys review 1st. This is to clean up devstack plugin and re-enable VPN tests.17:23
*** sthillma has quit IRC17:26
openstackgerritRoman Bogorodskiy proposed openstack/neutron: sriov: update port state even if ip link fails  https://review.openstack.org/19506017:27
*** banix_ has joined #openstack-neutron17:29
*** bharathm has quit IRC17:29
*** banix has quit IRC17:30
*** banix_ is now known as banix17:30
garykajmiller: lgtm17:31
ajmillergaryk thanks!17:31
*** saksham has joined #openstack-neutron17:31
*** abhiraut has quit IRC17:32
*** annegentle has quit IRC17:33
*** SourabhP has quit IRC17:34
amullerarmax: I think we have a new failure: message:"Details: {u'message': u'Network not found', u'code': 404}" AND tags:"console" AND build_name:"gate-tempest-dsvm-neutron-full"17:34
armaxthat’s new17:35
amullerarmax: 9 unique hits from today17:35
armaxamuller: the one about ports is gone17:35
amullerit started today17:35
amullerarmax++17:35
armaxamuller: but we have traded one for another?17:35
armaxoh joy17:35
armaxamuller: bug #?17:35
amullerarmax: not sure anyone reported a bug yet17:36
amullerit's brand new :)17:36
*** abhiraut has joined #openstack-neutron17:36
*** divyac has joined #openstack-neutron17:36
*** vikram has quit IRC17:36
divyacAnand Shanmugam17:36
*** gampel1 has quit IRC17:38
*** gampel has joined #openstack-neutron17:39
*** luqas has joined #openstack-neutron17:39
*** srilatha has joined #openstack-neutron17:39
*** hswayne77 has quit IRC17:40
*** annegentle has joined #openstack-neutron17:42
*** xgerman is now known as the_real_blogan17:42
*** the_real_blogan is now known as xgerman17:42
openstackgerritArmando Migliaccio proposed openstack/neutron: Make sure service providers can be loaded correctly  https://review.openstack.org/22025417:42
*** absubram has quit IRC17:43
*** vishwanathj has quit IRC17:43
*** jlibosva has quit IRC17:44
*** gsagie_ has joined #openstack-neutron17:45
*** jbell8 has joined #openstack-neutron17:46
*** sballe is now known as sballe_17:47
*** abhiraut has quit IRC17:47
*** ijw_ has quit IRC17:48
*** sballe_ is now known as sballe17:49
*** sayali has quit IRC17:50
armaxamuller: you mean this one?17:50
armaxhttp://logs.openstack.org/46/153946/136/gate/gate-tempest-dsvm-neutron-full/05cfd06/logs/testr_results.html.gz17:50
armaxbummer17:50
amullerarmax: ya17:50
amullerarmax: reporting bug as we speak17:50
armaxamuller: cool17:50
armaxassign it to me, I’ll triage it17:50
amullerarmax: https://bugs.launchpad.net/neutron/+bug/149196517:51
openstackLaunchpad bug 1491965 in neutron "test_list_show_tenant_networks triggering error "itemNotFound": {"message": "Network not found", "code": 404}" [Critical,New] - Assigned to Armando Migliaccio (armando-migliaccio)17:51
armaxthanks17:51
*** azbiswas has quit IRC17:52
*** kbringard has quit IRC17:52
*** aranjan has joined #openstack-neutron17:52
*** clev-away is now known as clev17:53
*** kbringard has joined #openstack-neutron17:53
*** jckasper has quit IRC17:53
amullerarmax: looks like another test is deleting that network17:53
*** aranjan has quit IRC17:53
*** sthillma has joined #openstack-neutron17:53
armaxamuller: need to step away for a bit17:53
armaxbut I’ll have time to look at it today17:53
amullerarmax: ok, I'll add info to the bug17:53
*** SourabhP has joined #openstack-neutron17:53
*** sayali has joined #openstack-neutron17:53
*** bradjones has joined #openstack-neutron17:53
*** bradjones has quit IRC17:53
*** bradjones has joined #openstack-neutron17:53
*** aranjan has joined #openstack-neutron17:53
*** emagana has quit IRC17:54
*** bharathm has joined #openstack-neutron17:54
*** carl_baldwin has joined #openstack-neutron17:54
*** carl_baldwin has quit IRC17:54
*** debedb has joined #openstack-neutron17:55
*** carl_baldwin has joined #openstack-neutron17:55
*** evgenyf has joined #openstack-neutron17:55
*** gampel has quit IRC17:55
*** VikasC has quit IRC17:55
*** azbiswas has joined #openstack-neutron17:55
*** nlahouti has quit IRC17:56
*** aranjan has quit IRC17:56
*** nlahouti has joined #openstack-neutron17:56
*** aranjan has joined #openstack-neutron17:56
*** VikasC has joined #openstack-neutron17:57
*** nlahouti has quit IRC17:57
kevinbentonarmax, amuller: this had come up before17:57
kevinbentonthat test was bad for neutron anyway17:57
kevinbentonit assumes networks never change17:58
amullerI'm looking at the test right now17:58
*** shivrao has joined #openstack-neutron17:58
amulleryeah it looks like a really stupid test? I don't get it17:58
kevinbentonit's for nova net17:58
amullerBTW observing the logs I see which (other) test is deleting the network17:58
amullerok17:58
amullerkevinbenton: so, let's just skip it for Neutron17:58
kevinbentonamuller: but it's this test that is wrong even though it doesn't immediately appear that way17:58
*** julim has joined #openstack-neutron17:58
kevinbentonamuller: https://review.openstack.org/#/c/195345/17:58
*** nlahouti has joined #openstack-neutron18:01
divyacHi i have a simple change for review (backport from master to kilo), Could any one please review this change? https://review.openstack.org/#/c/217843/18:02
HenryGkevinbenton: amuller: can we request tempest to merge the skip now? Salvatore can follow up later with a redesign of the test.18:03
*** tfukushima has quit IRC18:03
amullerthat makes sense to me18:03
kevinbentonHenryG: +118:04
amullerI guess I'll propose another patch18:05
amullersince salv is offline18:05
amulleractually I just rebased the patch =D18:05
amullerthat removed the WIP heh18:05
HenryGamuller: change commit msg to "Related-bug"18:05
*** sayali has quit IRC18:06
*** wwriverrat has quit IRC18:07
*** bobmel has quit IRC18:08
*** carl_baldwin has quit IRC18:08
*** rkukura has joined #openstack-neutron18:08
markmcclainajmiller: approved18:13
*** abregman has quit IRC18:15
*** abhiraut has joined #openstack-neutron18:17
dougwigpc_m: is that test failure good news?18:17
pc_mIn 169149? Yes.18:18
ajmillermarkmcclain thanks!18:18
pc_mdougwig: I think I have a mechanism in 203201, that works for the VPN functional tests. Need you to look it over to see if it looks right, though.18:19
*** debedb has quit IRC18:20
dougwigpc_m: interesting. i think if you add a -e to that pip install, it'd be good.18:21
*** roeyc has joined #openstack-neutron18:21
*** debedb has joined #openstack-neutron18:22
*** ameybhide has joined #openstack-neutron18:22
pc_mdougwig: instead of the -U?18:22
pc_mdougwig: or in addition?18:23
dougwigpc_m: both18:23
pc_mgotcha.18:23
pc_mWill try to revise.18:24
*** sbelous has joined #openstack-neutron18:24
pc_mDo you think that will work for all cases?18:24
pc_m(the script)18:24
pc_mdougwig: Looks like the script is causing a problem with py34 test run.18:27
openstackgerritArmando Migliaccio proposed openstack/neutron: CONF.reset does not happen for all tests  https://review.openstack.org/21987818:29
*** rushil has quit IRC18:29
armaxamuller: I have taken https://review.openstack.org/#/c/219878/ out of the gate unfortuntately18:29
armaxit was failing the py27 unit tests18:30
armaxfailure here: http://logs.openstack.org/78/219878/3/gate/gate-neutron-python27/68420e4/18:30
amullerwat18:30
amullerarmax: ok18:30
*** sbelous has quit IRC18:30
amullerarmax: dealing with the tempest failure now in openstack-qa18:30
armaxok18:31
armaxjust a heads-up18:31
*** harlowja has quit IRC18:31
*** annegentle has quit IRC18:31
*** localloop127 has joined #openstack-neutron18:31
*** harlowja has joined #openstack-neutron18:32
*** rkukura has quit IRC18:32
amullerarmax: it was too simple :)18:32
*** roeyc has quit IRC18:33
amullerperhaps salv will have insight, he created the diet base class, and it looks like it was not resetting the conf purposely (According to the comments)18:33
amullerI just have no idea why18:33
*** aditirajagopal has joined #openstack-neutron18:33
*** mickeys has quit IRC18:34
armaxlet’s dig around to see whether that trace happened before18:35
debedbhi all, question on the pluggable ipam… if an ipam driver plugin needs some configuration, what is the right way of getting it ?18:35
armaxit may not be rlated to your move at all18:35
openstackgerritJames Arendt proposed openstack/neutron: WIP: Fix Neutron flavor framework  https://review.openstack.org/21948618:35
amullerarmax: maybe18:35
amullerkevinbenton: HenryG: Did you happen to catch the convo in openstack-qa? Looks like the situation is a bit more complicated than we thought18:36
HenryGamuller: yes, trying to grok18:37
*** ajmiller has quit IRC18:38
*** arajagopal has left #openstack-neutron18:39
*** aditirajagopal is now known as arajagopal_18:39
*** arajagopal_ has quit IRC18:40
*** aditirajagopal has joined #openstack-neutron18:40
*** aditirajagopal is now known as arajagopal_18:40
*** ajmiller has joined #openstack-neutron18:40
pc_mdougwig: Any idea what the py34 failure is with tox_install.sh? Error code 1, but I'm not sure what the failure is.18:41
*** emagana has joined #openstack-neutron18:46
*** debedb has quit IRC18:48
*** slaweq has quit IRC18:48
*** minwang2 has quit IRC18:49
*** ijw_ has joined #openstack-neutron18:50
*** yamahata has quit IRC18:50
*** tongli has joined #openstack-neutron18:51
*** yamahata has joined #openstack-neutron18:51
*** nlahouti has quit IRC18:51
*** tongli has quit IRC18:52
*** s3wong has quit IRC18:53
*** nlahouti has joined #openstack-neutron18:54
*** ijw_ has quit IRC18:55
daneyon_gsagie gsagie_ It would be great to have a joint magnum/kuryr meeting at the design summit to work through integration details, roadmap, etc.. Does the Kuryr team have any plans established for the design summit?18:57
*** armax has quit IRC18:57
*** debedb has joined #openstack-neutron18:58
*** nlahouti has quit IRC18:58
*** cbouch has joined #openstack-neutron18:58
gsagie_daneyon_ : hi, yes we will probably have a time in the design summit for Neutron18:58
gsagie_Do you have one for Magnum networking as well? we can discuss it there as well18:59
*** nlahouti has joined #openstack-neutron18:59
*** VikasC has quit IRC19:00
*** jprovazn has quit IRC19:00
*** annegentle has joined #openstack-neutron19:01
*** amuller is now known as amuller_afk19:02
*** dedery has joined #openstack-neutron19:04
*** tbachman has quit IRC19:04
*** srilatha has quit IRC19:06
*** srilatha has joined #openstack-neutron19:06
*** ivar-lazzaro has quit IRC19:09
*** yfried_ has joined #openstack-neutron19:09
*** srilatha has quit IRC19:10
*** azbiswas has quit IRC19:11
*** azbiswas has joined #openstack-neutron19:11
*** pc_m is now known as pc_m_AWAY19:12
*** aranjan has quit IRC19:12
*** azbiswas has quit IRC19:16
*** SourabhP has quit IRC19:16
*** SourabhP has joined #openstack-neutron19:16
openstackgerritMichael Bayer proposed openstack/neutron: Add non-model index names to autogen exclude filters  https://review.openstack.org/21330319:17
openstackgerritMichael Bayer proposed openstack/neutron: Implement expand/contract autogenerate extension  https://review.openstack.org/20384219:17
*** nyechiel has quit IRC19:17
*** achanda has quit IRC19:18
*** jobewan has joined #openstack-neutron19:19
*** madhu_ak has quit IRC19:20
*** abhiraut has quit IRC19:21
*** nyechiel has joined #openstack-neutron19:21
*** john-davidge has quit IRC19:21
*** alex_klimov has joined #openstack-neutron19:23
*** rtheis has quit IRC19:25
*** rtheis has joined #openstack-neutron19:26
*** alexpilotti has quit IRC19:26
*** korzen has joined #openstack-neutron19:26
*** arajagopal_ has quit IRC19:27
*** tbachman has joined #openstack-neutron19:27
*** Marga_ has joined #openstack-neutron19:29
*** _cjones_ has joined #openstack-neutron19:29
*** armax has joined #openstack-neutron19:29
*** dguerri is now known as dguerri`19:29
*** changbl has quit IRC19:30
*** yfried_ has quit IRC19:31
*** _cjones_ has quit IRC19:31
*** _cjones_ has joined #openstack-neutron19:32
*** gsagie_ has quit IRC19:32
*** thumpba has joined #openstack-neutron19:32
*** matrohon has joined #openstack-neutron19:33
*** ijw_ has joined #openstack-neutron19:34
*** kbringard has quit IRC19:34
*** yfried_ has joined #openstack-neutron19:34
*** _cjones_ has quit IRC19:36
*** saksham has quit IRC19:39
*** banix has quit IRC19:41
*** ihrachys has joined #openstack-neutron19:41
*** ihrachys has quit IRC19:41
*** ihrachys has joined #openstack-neutron19:41
*** yamamoto has quit IRC19:42
daneyon_gsagie_ Yes, we will have a magnum networking design summit session. Can you keep me in the loop as you work out the details? I would like to make sure the 2 teams get together at the design summit.19:44
*** minwang2 has joined #openstack-neutron19:44
*** amuller_afk is now known as amuller19:44
*** madhu_ak has joined #openstack-neutron19:46
zzzeekihrachys: ping19:47
ihrachyszzzeek: pong19:47
zzzeek“overall, we won't have new per-release branches” - means….. “expand” “contract”, but not “liberty/expand” “liberty/contract” ?19:47
zzzeekim confused19:47
zzzeeki see liberty there still19:47
armaxamuller: so is anyone looking into https://bugs.launchpad.net/tempest/+bug/146852319:48
openstackLaunchpad bug 1468523 in tempest "tempest.api.compute.test_tenant_networks should be skipped when neutron is enabled" [Undecided,In progress] - Assigned to Assaf Muller (amuller)19:48
armax?19:48
amullerarmax: I left a comment on the patch, and there's new comments on the bug by kevinbenton and mtreinish19:48
armaxok19:48
*** vivek-ebay has quit IRC19:48
amullerit's not clear to me why it's happening19:48
amullerit looks like a tenant is for some reason seeing an external network created by another tenant19:49
* armax looks19:49
armaxkevinbenton: are you by any chance looking into it?19:49
*** kbringard has joined #openstack-neutron19:49
ihrachyszzzeek: it's in file names but not in script chain.19:49
zzzeekihrachys: if im to “expand” to liberty and “expand” to M, i just do “Expand” and the idea is, the source tree I’ve checked out is only one major release past the one I have anyway ?19:49
*** ivar-lazzaro has joined #openstack-neutron19:50
zzzeekihrachys: are we going to update the blueprint?  this is a big change?19:50
ihrachyszzzeek: I think we updated devref with that. not sure it's worth updating a spec?19:50
zzzeekihrachys: jsut the spec is not what we’re doing anymore here.  seems confusing.   if the spec had been updated i woudl have noticed this aspect of the model is changing :)19:51
ihrachyszzzeek: the rationale was noted at: https://review.openstack.org/#/c/215076/19:51
*** azbiswas has joined #openstack-neutron19:51
ihrachyszzzeek: it's mostly the same, it's a tiny bit different. the core idea is left intact.19:52
zzzeekihrachys: feel free to link me on major changes to what I spec’ed :)   reading now19:52
*** slaweq has joined #openstack-neutron19:53
ihrachyszzzeek: I actually thought you were looped in that one. But now I see we forgot to add you to the list there. sorry for that.19:53
*** ivar-lazzaro has quit IRC19:53
zzzeekihrachys: what im not sure of is, is it safe to “Expand” from kilo->liberty->mitaka at once, then “contract” across all three at once19:53
ihrachysI added you for the dep patch, and thought I did for that one too19:53
zzzeekihrachys: the reason for the leapfrogging was to prevent that19:53
*** ivar-lazzaro has joined #openstack-neutron19:53
ihrachyszzzeek: I believe it should work assuming we don't have depends_on from expand to contract in the row (that's one thing to tackle too - to guard against it somehow).19:54
*** evgenyf has quit IRC19:56
*** nlahouti has quit IRC19:57
*** arajagop_ has joined #openstack-neutron19:57
zzzeekihrachys: im not sure either that the way we had it allows “liberty_expand@<some_revision_from_mitaka>”19:58
zzzeekbecause mitaka and liberty are different bases19:58
zzzeekthey are only linked via depends_on19:58
zzzeeknot the same branches19:58
zzzeekif that’s the only problem with how it was, we should just disallow complex versions like that from neutron-db-manage anyway19:58
zzzeekneutron-db-manage should only allow simple, major targets19:59
*** emagana has quit IRC20:00
ihrachyszzzeek: if we would link with depends_on, we would have as much heads as we have releases * 2. doesn't sound ideal. so the idea was that we actually link them with down_revision instead.20:01
ihrachyszzzeek: some people chase master, they want to upgrade in steps20:01
*** Marga_ has quit IRC20:01
ihrachysarmax: I left silly comments in https://review.openstack.org/#/c/220254/, please have a look20:01
zzzeekihrachys: I understand that use case, I’d propose they use the alembic CLI directly20:01
armaxihrachys: looking20:01
armaxihrachys: all in all what do you think?20:02
zzzeekihrachys: its true, this doesnt matter that much, but I now see that ppl are no longer able to do an expand contract for a single version at a time, there is no more label or anything20:02
*** aranjan has joined #openstack-neutron20:02
*** nlahouti has joined #openstack-neutron20:03
amullerarmax: it's trivial to fix that test in tempest20:03
amullerarmax: in a million different ways20:03
armaxihrachys: replied20:03
ihrachyszzzeek: that's true. the assumption is that you want the version that you checked out, and that upgrade for multiple versions should work.20:03
amullerarmax: but wouldn't you like to figure out why one tenant is seeing an external network created by another tenant?20:04
*** chutwig is now known as erhud120:04
*** erhud1 is now known as erhudy120:04
armaxamuller: according to kevinbenton, this is supposed to be the design20:05
amullerarmax: I don't think he's right20:05
armaxamuller: I can’t recall on the top of my head if external networks are shared by default20:05
amullerarmax: they aren't20:05
* armax still going through the weeds20:05
HenryGzzzeek: We mark each release with a milestone name migration. So 'upgrade expand@liberty"20:05
amullerarmax: an external network that is not shared is not (in my devstack) and should not be visible to another tenant in a net-list command20:05
zzzeekihrachys: OK, I guess maybe it will, I dont know offhand if they are going to work across many major versions. example.  Liberty replaces column A with column B.  Mitaka replaces column B with column C.  There is no version of the code that translates column A to C at once20:05
*** abhiraut has joined #openstack-neutron20:06
armaxso both salv and kevinbenton are wrong?20:06
armaxsomething is up20:06
amullerarmax: http://paste.openstack.org/show/444565/20:06
zzzeekHenryG: OK, so you can expand/contract specific to a single major relasee?  then its fine20:06
*** emagana has joined #openstack-neutron20:06
*** dave-mccowan has quit IRC20:06
zzzeekHenryG / ihrachys : although, what if they just “upgrade expand”.   now with the column A->B->C problem, they are sunk20:07
zzzeekits a mistake that can now be made20:07
*** localloop127 has quit IRC20:07
HenryGzzzeek: yeah, that's not great20:08
amullerarmax: my instinct is that there's some sort of mix up in Tempest20:08
zzzeekHenryG: it can be solved by just placing limitations on neutron-migrate-db20:08
ihrachyszzzeek: that replace scenario, is it really expand?20:08
HenryGzzzeek: moar checks20:08
armaxamuller: that’s what we gotta find out20:08
zzzeekihrachys: by “replace” I mean, liberty “adds” column B, and assumes the app will translate data from A to B.  mitaka “Adds” columnn C, and assumes the app will translate from B to C20:09
zzzeekin their “contract”, liberty drops A, mitaka drops B20:09
ihrachyszzzeek: I believe you consider data migration, and we are still in schema migration phase.20:09
zzzeekihrachys: that is correct however the whole point is that data migration has to integrate :)20:09
zzzeekihrachys: without that, all this effort is a waste of time :)20:09
ihrachyszzzeek: I believe we may need checkpoints for data migrations that are not allowed to pass with unmigrated data20:10
armaxamuller: the problem may be that other tests may be creating shared networks20:10
ihrachysI think nova was looking into checkpoints too20:10
armaxand those would accidentally be visible when getting all networks from  the offending test20:10
mtreinisharmax: the logs clearly show shared: false in the network create response20:10
zzzeekihrachys: nova is doing this witout any help from me :)20:10
ihrachyswe have sanity checks in neutron-db-manage, I believe those would belong there20:11
zzzeekihrachys: they’re in the “hey lets try this!” stage, i think there’s a high liklihood the whole thign isn’t going to work for them20:11
*** localloop127 has joined #openstack-neutron20:11
zzzeekihrachys: then again i think the data migration aspect of this whole thing is going to be unmanagable anyway20:11
mtreinisharmax: I mean there is definitely a potential bug there with shared networks showing up in list responses which is why we try to guard against that in tests20:11
armaxamuller, mtreinish: I’m still going through all the data20:11
ihrachyszzzeek: define 'this whole thing'20:11
amullerarmax: the test it interacts with is tempest.api.network.admin.test_external_network_extension.ExternalNetworksTestJSON.test_delete_external_networks_with_floating_ip20:11
ihrachysnova? neutron?20:11
mtreinisharmax: that nova list test is poorly written for a neutron env, but I don't think that's the issue here20:12
zzzeekihrachys: the, “I want to install mitaka and do a full upgrade without shutting off my openstack service”20:12
mtreinishthat'll have to be fixed either way20:12
armaxamuller, mtreinish: I haven’t formulated an opinion yet, neither I am concuring with kevinbenton nor salv20:12
mtreinisharmax: I don't have a horse in the race either, since I don't know all the details of the api20:12
armaxmtreinish: if we can guard the test making it isolated as a result, fine…but yes, I would like to better understand what’s going on20:12
mtreinishI'm just opposed to a skip20:12
*** baoli_ has quit IRC20:13
armaxmtreinish: agreed20:13
amulleraye20:13
ihrachyszzzeek: I don't believe you can upgrade code without shutting off. it's not the problem at stake. reducing downtime is.20:13
*** mickeys has joined #openstack-neutron20:13
armaxmtreinish: let me mull over this a bit20:13
amullermtreinish: running test_show_external_networks_attribute on its own also fails20:13
amullermtreinish: seems like a common occurence20:14
mtreinishamuller: that seems unexpected, do you have a pastebin of the failure20:14
zzzeekihrachys: restate more accurately. “ i want to install mitaka and do rolling schema upgrades without downtime while the database moves from one version to another, transparently maintaining the data model in both logical versions of the schema simultanously"20:14
openstackgerritCedric Brandily proposed openstack/neutron: Enable most unit tests for py34 job  https://review.openstack.org/21938920:15
ihrachyszzzeek: that's absolutely doable! :)20:15
*** dedery has quit IRC20:16
zzzeekihrachys: if i had started writing openstack for NASA however many years ago, it would have really really been not like this at all.  but then again, it woudl not have created 5000 new programming jobs :)20:16
amullermtreinish: http://paste.openstack.org/show/444663/20:16
ihrachyszzzeek: job security woooot20:16
zzzeekihrachys: i woudl have thought, “web application that spawns VMs and tracks status in a database”20:17
ihrachysso in any case, I hope we are ok with that silly change I made without asking you. we seem to have all needed, but we'll need more checks and probably check-points etc. work to do in M+.20:18
*** julim has quit IRC20:18
*** dguerri` is now known as dguerri20:18
mtreinishamuller: that's weird it does a create and then does a get on it and it gets a 404 resp20:18
*** achanda has joined #openstack-neutron20:18
amullermtreinish: it's created by the admin client and the show is with the normal client20:18
mtreinishamuller: neutron returns 201 so I'd expect that to be a syncronous create20:18
amullerit is synchronous yes20:18
mtreinishamuller: ah20:18
mtreinishyeah that owuld do it20:19
ihrachyszzzeek: I don't believe that simplistic approach at the start would leave you anywhere, but whatever20:19
ihrachysarmax: another silly comment20:19
*** bobmel has joined #openstack-neutron20:19
mtreinishamuller: how is that ever passing then?20:20
armaxihrachys: you’re being whimsical today aren’t you?20:20
amullermtreinish: I don't know20:20
*** dguerri is now known as dguerri`20:20
amullermtreinish: maybe my tempest conf is off?20:20
zzzeekihrachys: the change is fine, im a little less certain about it but if you are good with it then im sure it will work out20:20
amullermtreinish: I really don't know20:20
mtreinishamuller: that test seems to be there to verify that external nets can be seen by any user20:20
amullermtreinish: yeah, I'm lost at this point20:20
mtreinishthe comment seems to say the same thing20:20
ihrachysarmax: I learn from salv20:21
*** julim has joined #openstack-neutron20:21
armaxihrachys: replied20:23
ihrachysarmax: lol for the latter20:23
armax:)20:23
*** achanda has quit IRC20:24
ihrachysarmax: what is the format of service_provider that you match self.module_name against?20:24
armaxthe module_name is something like neutron_lbaas20:24
armaxor networking_l2gw20:24
armaxor neutron_vpaas20:24
armaxit’s the module’s name20:24
*** ronis has quit IRC20:24
armaxwe could pass the Service Label20:24
armaxto have a better match20:25
*** cbouch has quit IRC20:25
armaxihrachys: but that would have to go through a tango of changes that involves all the projects that use the service provider stuff20:25
*** sdake has joined #openstack-neutron20:25
ihrachysso f.e. it's 'VPN:vyatta:neutron_vpnaas.services.vpn.service_drivers.vyatta_ipsec.VyattaIPsecDriver:default'20:25
openstackgerritMichael Bayer proposed openstack/neutron: Add non-model index names to autogen exclude filters  https://review.openstack.org/21330320:25
openstackgerritMichael Bayer proposed openstack/neutron: Implement expand/contract autogenerate extension  https://review.openstack.org/20384220:25
armaxihrachys: yes20:25
ihrachysyou try to match against 'neutron_vpnaas.services.vpn.service_drivers.vyatta_ipsec.VyattaIPsecDriver:default'20:25
ihrachysright?20:26
armaxno20:26
ihrachysmeh20:26
armaxjust neutron_vpnaas20:26
armaxI mean...20:26
ihrachysyeah, ok, sure. but I mean, that part, not e.g. VPN20:26
*** ZZelle_ has joined #openstack-neutron20:26
armaxihrachys: I filter all the drivers that contain the module_name being given to input20:26
armaxto NeutronModule20:26
armaxsurely the provider can be a random thing that doesn’t belong to the neutron_vpnaas_project20:27
*** Marga_ has joined #openstack-neutron20:27
ihrachysarmax: so that part is device_driver, right? and it's read thru stevedore, which allows any symbolic labels for drivers, that could have no module_name in the aliases at all20:27
armaxhence matching by service label, ie. VPN would be better…but that would require more tahn a single patch to fix this20:27
ihrachysyeah, in the last comment of yours, I see that we start to converge20:28
*** zhhuabj_ has joined #openstack-neutron20:28
*** jwy has joined #openstack-neutron20:28
ihrachysso yeah, my comment was mostly about 'what about drivers that are defined by aliases, as we actually should push *aas users to specify them in long run'20:29
*** korzen has quit IRC20:29
*** sdake_ has quit IRC20:29
ihrachysusing import paths is so.... wrong20:29
*** achanda has joined #openstack-neutron20:29
armaxihrachys: yes, at one point when I moved the NeutronModule class I thought about passing the service label20:29
armaxihrachys: but didn’t see a need for it, I do now20:30
ihrachysthat said, I don't see any drivers in *aas trees that would define aliases.20:30
armaxihrachys: and I can clean this up, once we untangle #149099020:30
ihrachys(not that there can't be external drivers that do)20:30
armaxihrachys: right20:30
*** zhhuabj has quit IRC20:31
*** azbiswas has quit IRC20:31
ihrachysyeah, I agree we should fix it first and foremost even if the fix is not ideal from silly commenter's view20:31
armaxthey are not silly at all20:31
armaxihrachys: I had thought about that, as the module name is not ideal for the reasons why just discussed20:32
*** dave-mccowan has joined #openstack-neutron20:32
*** julim has quit IRC20:33
armaxI’ll clean this up as soon as I put out a few other fires.20:33
ihrachysarmax: put or set?20:33
ihrachys:)20:33
ihrachysthe latter is more fun20:34
armaxihrachys: that’s mean, but I guess I earned that one20:34
armax:)20:34
*** azbiswas has joined #openstack-neutron20:36
ihrachyshuh, not really :)20:37
*** aranjan has quit IRC20:37
ihrachysEmilienM: still here?20:38
EmilienMyes Sir20:38
EmilienMI should stop saying that20:38
EmilienMihrachys: what's up?20:38
*** aranjan has joined #openstack-neutron20:38
ihrachysEmilienM: so armax came up with a fix. I wonder whether we have an option to validate the fix before we merge it (f.e. thru Depends-On)20:38
*** achanda has quit IRC20:38
EmilienMoh20:38
ihrachysEmilienM: of course I mean validate for puppet20:38
armaxamuller: how did you create that external network? which account?20:38
EmilienMihrachys: not in my (puppet) case20:38
ihrachysEmilienM: that's sad. doesn't Depends-On work for you?20:39
EmilienMihrachys: neutron/oslo patches can't be tested without merged (rdo trunk build them)20:39
EmilienMit does but we use packaging20:39
amullerarmax: I was talking to Kevin in #openstack-qa, turns out I'm just a dumbass and forgot I played around with Neutron's policy.json on the machine I was experimenting on20:39
EmilienMwe are not devstack20:39
ihrachysEmilienM: ah right, indeed.20:39
amullerarmax: tenants can infact see external networks created by another tenant20:39
armaxamuller: ok20:39
armaxso we’re good?20:39
armaxbesides I can see here ExternalNetworksTestJSON20:40
ihrachysEmilienM: you should build your packages with delorean ;)20:40
armaxthat the extenral network is admin created20:40
EmilienMihrachys: but if you provide me a fix, I can test it20:40
armaxand that’s definitely visible20:40
EmilienMihrachys: yeah :-)20:40
ihrachysEmilienM: the fix: https://review.openstack.org/#/c/220254/20:40
armaxthink about the ‘public’ network created by devstack itself.20:40
armaxamuller: capisce?20:40
amullerarmax: the proper fix is to change test_list_show_tenant_networks to create its own network20:40
armaxamuller: aye20:40
armaxthere’s a caveat though20:40
EmilienMihrachys: no depends-on?20:40
amullerarmax: BTW that test is utterly broken, if you run it on its own it fails20:40
armaxthis test is actually a test for https://blueprints.launchpad.net/neutron/+spec/get-me-a-network ;)20:41
amullerarmax: regardless of any test interaction20:41
ihrachysEmilienM: ? I presumed you will test it locally somehow?20:41
EmilienMihrachys: I mean, no dependency in oslo?20:41
ihrachysEmilienM: no, not really. which is good.20:41
EmilienMcool20:41
EmilienMI'll test it20:41
ihrachysEmilienM: thanks!20:42
EmilienMthx for the fixes, armax++20:42
*** srilatha has joined #openstack-neutron20:42
*** yamamoto has joined #openstack-neutron20:42
armaxEmilienM: well, apologies for the glitch20:42
*** julim has joined #openstack-neutron20:42
ihrachysEmilienM: armax: I will drop off now for a nap, please sync on testing results. cheers!20:42
armaxamuller: so we’re good then?20:42
*** kairo has joined #openstack-neutron20:43
EmilienMno need to apology, I'm happy we found it before the release20:43
armaxihrachys: nap?20:43
amullerarmax: ask mtreinish20:43
armaxihrachys: who’s got time for a nap?20:43
armaxihrachys: slacker20:43
EmilienMihrachys: sleep well20:43
ihrachysactually not a nap, but a full blown sleep!20:43
ihrachysit's a bit late here already, I deserve it!20:43
* armax thought ihrachys didn’t sleep20:43
EmilienMit's feature freeze anyway, go sleep :-P20:43
amullerihrachys: 11pm is an early night for you :)20:43
armaxamuller: now that we have a settled on the matter20:43
armaxthen we can go ahead and ‘fix’ the test rather than skip it20:44
armaxmtreinish: ^^^20:44
*** localloop127 has quit IRC20:44
amulleryeah20:44
* armax goes and types on the keyboard like a monkey20:44
amullerarmax: so, nova network doesn't have a create network API does it20:44
*** localloop127 has joined #openstack-neutron20:45
armaxthat’s the whole point of get-me-network monkey businees20:45
amullerbecause that test is currently running against both nova network and neutron20:45
mtreinishamuller: I laid out the basic steps in the bug on how to fix the test20:46
amullermtreinish: ack20:46
*** achanda has joined #openstack-neutron20:47
armaxmtreinish: thanks for spelling it out for us20:47
*** yamamoto has quit IRC20:47
mtreinisharmax: sure, np :)20:47
armaxmtreinish: I’ll get a patch to your attention in a few minutes20:47
armaxmtreinish: after a dozen patchsets we’ll probably be able to get it merged20:48
armaxwell20:48
armaxapproved20:48
armaxa few weeks from now we’ll get it merged20:48
openstackgerritKevin Benton proposed openstack/neutron: Deprecate external_network_bridge option in L3 agent  https://review.openstack.org/21992820:49
mtreinisharmax: heh, I think we can get it done a bit quicker than that20:49
armaxmtreinish: you overestimate me20:49
armax:)20:49
*** localloop127 has quit IRC20:49
amullerthat whole set_network_resources() thing is so weird20:50
amullerit's the most roundabout way to create a network =D20:50
armaxamuller, mtreinish: posting a patch shortly20:50
*** saksham has joined #openstack-neutron20:50
*** l6unchpad has joined #openstack-neutron20:51
mtreinishamuller: well it's the method to create/access networking resources that're implicitly expected to be there20:51
mtreinishit was needed because of the change in model from n-net where a network was always there for you20:52
mtreinishsalv and I took a lot of time to figure out an approach that worked there20:52
amullerI see, it doesn't feel like code you'd right like that from the get go. I prefer test setup as close to the test as possible, not hidden away in some base class 5 levels separated20:52
amulleryou'd write*20:52
openstackgerritKevin Benton proposed openstack/neutron: ovs agent resync may miss port remove event  https://review.openstack.org/16502320:52
mtreinishamuller: fwiw, I originally wanted to just always have a network, subnet, etc always created20:53
mtreinishbut back in the icehouse days that actually crashed neutron20:53
mtreinishamuller: well except that would basically mean every test class would copy and paste the same setup and teardown code20:54
*** julim has quit IRC20:54
mtreinishwe already had the abstract mechanism to provide isolation20:54
mtreinishso we just added networks to that20:55
mtreinishfwiw, I think in the future we'll probably drop the explicit method and make it a class attr20:55
mtreinishlike we did for creds20:55
openstackgerritStephen Balukoff proposed openstack/python-neutronclient: LBaaS updates to reflect independent pools feature  https://review.openstack.org/21856320:56
*** fzdarsky__ has quit IRC21:02
*** julim has joined #openstack-neutron21:02
*** abhiraut has quit IRC21:04
*** thorst has quit IRC21:04
*** rkukura has joined #openstack-neutron21:05
*** abhiraut has joined #openstack-neutron21:05
*** rkukura has quit IRC21:05
*** tiswanso has quit IRC21:06
*** vivek-ebay has joined #openstack-neutron21:07
*** pc_m_AWAY is now known as pc_m21:09
*** aranjan has quit IRC21:09
*** aranjan has joined #openstack-neutron21:10
*** Piet has quit IRC21:11
*** ivar-lazzaro has quit IRC21:14
*** slaweq has quit IRC21:15
*** mlavalle has quit IRC21:15
*** rtheis has quit IRC21:16
*** slaweq has joined #openstack-neutron21:17
*** crose has quit IRC21:18
*** slaweq has quit IRC21:18
*** ihrachys has quit IRC21:20
*** jbell8 has quit IRC21:21
*** flaviof is now known as flaviof_afk21:22
*** aranjan has quit IRC21:22
*** clev is now known as clev-away21:26
*** pc_m is now known as pc_m_AWAY21:27
*** arajagop_ has quit IRC21:28
*** arajagop_ has joined #openstack-neutron21:28
*** nyechiel has quit IRC21:31
*** Leonr has quit IRC21:32
*** arajagop_ has quit IRC21:32
amullerarmax: going offline for now21:35
armaxok21:35
armaxI got derailed with that tempest patch, pushing it imminently21:35
armaxleave it with me21:35
*** rpothier has quit IRC21:35
armaxI won’t disappoint you21:35
amullerarmax: I trust you won't :)21:35
*** amuller has quit IRC21:38
*** annegentle has quit IRC21:38
*** l6unchpad has quit IRC21:38
*** njohnston is now known as nate_gone21:43
*** johnbelamaric has quit IRC21:43
*** aranjan has joined #openstack-neutron21:46
*** vivek-ebay has quit IRC21:46
*** matrohon has quit IRC21:49
*** aranjan has quit IRC21:50
*** localloop127 has joined #openstack-neutron21:51
*** flaviof_afk is now known as flaviof21:52
*** sridhar_ram has quit IRC21:53
*** sridhar_ram has joined #openstack-neutron21:54
*** annegentle has joined #openstack-neutron21:57
*** tbachman has quit IRC21:58
*** localloop127 has quit IRC21:59
*** nyechiel has joined #openstack-neutron22:00
*** tflynn_ has joined #openstack-neutron22:00
*** annegentle has quit IRC22:03
*** tflynn has quit IRC22:04
openstackgerritPaul Michali proposed openstack/neutron-vpnaas: Support for VPN functional tests on Neutron commits  https://review.openstack.org/20320122:05
*** tflynn_ has quit IRC22:05
*** dane_leblanc has quit IRC22:05
*** aranjan has joined #openstack-neutron22:06
*** arajagop_ has joined #openstack-neutron22:07
*** clev-away is now known as clev22:07
*** moha_hunt has quit IRC22:08
*** abhiraut has quit IRC22:09
*** emagana has quit IRC22:10
*** pc_m_AWAY has quit IRC22:10
*** kbringard has quit IRC22:10
*** wwriverrat has joined #openstack-neutron22:14
*** jbell8 has joined #openstack-neutron22:15
*** thorst has joined #openstack-neutron22:16
*** abhiraut has joined #openstack-neutron22:16
openstackgerritKevin Benton proposed openstack/neutron: Process user iptables rules before INVALID  https://review.openstack.org/21851722:17
*** Piet has joined #openstack-neutron22:17
*** mlavalle has joined #openstack-neutron22:19
*** moha_hunt has joined #openstack-neutron22:20
*** azbiswas has quit IRC22:23
*** azbiswas has joined #openstack-neutron22:23
*** dmartls1 has quit IRC22:25
*** alex_klimov has quit IRC22:28
*** Leonr has joined #openstack-neutron22:30
*** Leon has joined #openstack-neutron22:31
*** Leon is now known as Guest5823322:31
*** Leonr has quit IRC22:34
*** nyechiel has quit IRC22:35
*** Marga_ has quit IRC22:35
*** regXboi has quit IRC22:35
*** Marga_ has joined #openstack-neutron22:37
*** tbachman has joined #openstack-neutron22:37
*** thumpba has quit IRC22:40
*** Marga_ has quit IRC22:43
*** Marga_ has joined #openstack-neutron22:44
*** markstur has joined #openstack-neutron22:46
*** phil_h has quit IRC22:48
*** shwetaap has quit IRC22:48
*** edmondsw has quit IRC22:53
*** vivek-ebay has joined #openstack-neutron22:55
*** sridhar_ram has quit IRC22:55
*** sridhar_ram has joined #openstack-neutron22:56
*** vivek-ebay has quit IRC22:56
*** abhiraut has quit IRC22:56
*** srilatha has quit IRC22:59
*** srilatha has joined #openstack-neutron22:59
*** srilatha has quit IRC23:00
*** srilatha has joined #openstack-neutron23:00
*** n00btube has joined #openstack-neutron23:02
*** sdake has quit IRC23:06
*** zzzeek has quit IRC23:10
*** arajagop_ has quit IRC23:12
*** Marga_ has quit IRC23:12
*** n00btube has quit IRC23:12
*** markvoelker has joined #openstack-neutron23:13
*** arajagopal has joined #openstack-neutron23:13
*** moha_hunt has quit IRC23:14
*** n00btube has joined #openstack-neutron23:15
*** pradk has quit IRC23:16
*** n00btube has quit IRC23:16
*** mickeys has quit IRC23:17
*** arajagopal has quit IRC23:17
*** markvoelker has quit IRC23:17
*** n00btube has joined #openstack-neutron23:21
*** tidwellr1 has joined #openstack-neutron23:22
*** gildub has joined #openstack-neutron23:22
*** Sukhdev_ has joined #openstack-neutron23:23
*** dannywilson has quit IRC23:23
Sukhdev_armax: Ping23:24
armaxpong23:24
Sukhdev_armax: I noticed one patch got merged recently which is causing our CI to break - please see the paste here - http://paste.openstack.org/show/444792/23:25
*** markvoelker has joined #openstack-neutron23:25
Sukhdev_armax: I noticed you commented on the patch and approved it - so, I thought you will have context to the issue -23:25
armaxthe paste alone is not enought for me to understand what;’s going on23:25
armaxwhat patch are you talking about?23:25
Sukhdev_this has to do something with the API workers -23:26
armaxwhich one?23:26
armaxthis one?23:26
armaxhttps://review.openstack.org/#/c/189391/23:26
armaxthis hasn’t merged23:26
*** n00btube has quit IRC23:26
*** moha_hunt has joined #openstack-neutron23:27
Sukhdev_armax: thanks - yes, I think this one23:27
armaxyour ci fails but neutron starts fine23:27
armaxit passed on PS1723:27
*** abhiraut has joined #openstack-neutron23:27
*** azbiswas has quit IRC23:27
armaxand PS18 is a rebase23:28
*** azbiswas has joined #openstack-neutron23:28
armaxSukhdev_: so I am not sure you’re looking at the right thing23:28
Sukhdev_armax: hang on - let me paste more details23:28
*** azbiswas has quit IRC23:29
*** annegentle has joined #openstack-neutron23:29
Sukhdev_armax: actually you can look at this log - http://172.22.255.11:8080/job/Arista%20ML2%20Driver%20Testing/30425/console23:29
armaxit’s not opening up23:30
*** azbiswas has joined #openstack-neutron23:30
Sukhdev_armax: the paste I pasted earlier was from the log gf q-svc (the link to the logs is also on that line)23:30
armaxthe server must be behind firewall23:30
Sukhdev_armax: oh may be firewall issue :)23:30
Sukhdev_let me paste it and send it23:31
*** bradjones has quit IRC23:31
*** achanda has quit IRC23:32
Sukhdev_armax: here is the past of devstack failure - http://paste.openstack.org/show/444794/23:32
*** y_sawai has joined #openstack-neutron23:32
Sukhdev_armax: and here it the link to the logs - you should be able to open this - http://arista-ci.arista.com:8000/September-2015/Arista_Tempest_Test/56/219456/4/23:33
* armax looks23:33
*** n00btube has joined #openstack-neutron23:33
*** jobewan has quit IRC23:33
*** azbiswas has quit IRC23:34
armaxthe patch is actually this one23:35
armaxhttps://review.openstack.org/#/c/219456/23:35
armaxthis is very odd23:35
armaxbecause that patch is a noop23:35
*** clev is now known as clev-away23:35
armaxHenryG: might be able to shed more light23:35
armaxuse git bisect to figure out where things go south23:36
Sukhdev_armax: you may be right -23:36
*** n00btube has quit IRC23:36
Sukhdev_armax: what is git bisect?23:36
*** neophy has quit IRC23:36
armaxa tool command that lets you identify what git commit caused the regression23:37
armaxSukhdev_: http://webchick.net/node/9923:37
armaxhttp://git-scm.com/docs/git-bisect23:37
*** jhyangtw has joined #openstack-neutron23:38
Sukhdev_armax: cool - thanks - let me try it23:38
*** oomichi_away is now known as oomichi23:39
armaxthe fact that the error says it can’t bind makes me wonder whether teh port is already open23:39
*** y_sawai has quit IRC23:41
*** ZZelle_ has quit IRC23:42
*** clev-away is now known as clev23:42
*** jhyangtw has quit IRC23:42
Sukhdev_armax: interesting information - this patch that you mentioned merged around 1:12 am, my CI is broken as of yesterday 6:40pm (apps) -23:42
armaxso it must be something else ;)23:43
Sukhdev_armax: every run is failing with exact same error23:43
*** dims has quit IRC23:44
*** ijw_ has quit IRC23:45
*** mickeys has joined #openstack-neutron23:45
*** watanabe_isao has joined #openstack-neutron23:46
*** changbl has joined #openstack-neutron23:48
*** shoutm has joined #openstack-neutron23:50
*** watanabe_isao has quit IRC23:51
*** miyagishi_t has joined #openstack-neutron23:52
*** dims has joined #openstack-neutron23:53
*** Swami has quit IRC23:56
*** tidwellr1 has quit IRC23:58
*** aix has joined #openstack-neutron23:58

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