Wednesday, 2015-02-04

*** yamamoto has joined #openstack-neutron00:00
openstackgerritCarl Baldwin proposed openstack/neutron: Introduce External IPAM Interface  https://review.openstack.org/13433900:00
*** julien_ZTE has quit IRC00:01
*** yamamoto has quit IRC00:01
*** julien_ZTE has joined #openstack-neutron00:01
*** yamamoto has joined #openstack-neutron00:02
*** shashankhegde has quit IRC00:03
*** banix has joined #openstack-neutron00:03
*** dkingshott has quit IRC00:04
openstackgerritCarl Baldwin proposed openstack/neutron: Introduce External IPAM Interface  https://review.openstack.org/13433900:04
*** a_le_ has quit IRC00:05
*** julien_ZTE has quit IRC00:06
*** markvoelker has quit IRC00:07
Sukhdevmestery:Ping00:09
*** alexpilotti has quit IRC00:10
Sukhdevmestery: when you get a chance, can you tag this for K2, please - https://bugs.launchpad.net/neutron/+bug/141257700:10
*** thomasem has quit IRC00:10
*** marun has quit IRC00:11
*** claudiub has quit IRC00:14
*** igordcard has joined #openstack-neutron00:16
*** jasondotstar has joined #openstack-neutron00:16
*** Sukhdev has quit IRC00:16
*** igordcard has quit IRC00:17
*** yamamoto has quit IRC00:17
*** banix has quit IRC00:18
*** s3wong has quit IRC00:18
*** s3wong has joined #openstack-neutron00:18
*** s3wong has quit IRC00:20
*** dims has quit IRC00:22
*** heyongli has quit IRC00:23
*** dane_leblanc has quit IRC00:23
*** dims has joined #openstack-neutron00:23
*** alex_xu_ has quit IRC00:23
openstackgerritMerged openstack/neutron: Configures RADVD to send managed flag in RA for DHCP_STATEFUL  https://review.openstack.org/14463000:23
*** moshele has quit IRC00:25
*** oomichi has joined #openstack-neutron00:26
*** flwang2 has left #openstack-neutron00:30
openstackgerritCarl Baldwin proposed openstack/neutron: Fix extra-dhcp-opt on stateless dhcpv6 subnet  https://review.openstack.org/14745500:30
*** carl_baldwin has quit IRC00:33
*** aranjan has quit IRC00:36
*** aranjan has joined #openstack-neutron00:37
*** vivek-ebay has joined #openstack-neutron00:39
*** xuhanp has quit IRC00:40
*** chenleji has quit IRC00:40
*** jasondotstar has quit IRC00:41
*** claudiub has joined #openstack-neutron00:43
*** signed8bit is now known as signed8bit_ZZZzz00:46
*** a_le has joined #openstack-neutron00:47
*** banix has joined #openstack-neutron00:49
*** apuimedo has quit IRC00:51
*** markvoelker has joined #openstack-neutron00:52
*** yamamoto has joined #openstack-neutron00:53
*** manishg has quit IRC00:55
*** yamamoto_ has joined #openstack-neutron00:55
*** manishg has joined #openstack-neutron00:55
salv-orlandodougwig: btw, I actually remembered we already have a mechanism for not loading extensions twice, why did you add to add another one?00:56
*** david-lyle is now known as david-lyle_afk00:57
*** yamamoto has quit IRC00:57
dougwigBecause the other imports twice, which was causing problems.  But I only needed it while moving the exts, so I'll yank it in the delete exts review.00:58
dougwigsalv-orlando: ^^00:58
salv-orlandodougwig: sorry, who's "the other"?00:58
dougwigThe other mechanism00:59
*** markvoelker has quit IRC01:00
*** zzzeek has quit IRC01:00
*** manishg has quit IRC01:00
salv-orlandodougwig: do you mean that it does imp.load_source twice and then discards if it's duplicated?01:02
*** shashankhegde has joined #openstack-neutron01:06
*** gildub has quit IRC01:07
*** marun has joined #openstack-neutron01:09
*** manishg has joined #openstack-neutron01:10
*** mohitsharma has joined #openstack-neutron01:12
*** signed8bit_ZZZzz is now known as signed8bit01:13
*** marun has quit IRC01:13
*** signed8bit is now known as signed8bit_ZZZzz01:13
*** signed8bit_ZZZzz is now known as signed8bit01:13
*** tiswanso has joined #openstack-neutron01:13
*** tiswanso has quit IRC01:17
*** tiswanso has joined #openstack-neutron01:17
*** markvoelker has joined #openstack-neutron01:19
*** achanda_ has joined #openstack-neutron01:19
*** manishg has quit IRC01:19
*** manishg has joined #openstack-neutron01:20
*** achanda has quit IRC01:22
*** manishg has quit IRC01:22
*** achanda_ has quit IRC01:24
openstackgerritSalvatore Orlando proposed openstack/neutron: Restore and fix vmware unit tests  https://review.openstack.org/15192801:24
*** jasondotstar has joined #openstack-neutron01:24
*** _cjones_ has quit IRC01:29
*** _cjones_ has joined #openstack-neutron01:29
openstackgerritShweta P proposed openstack/neutron: Passes the context variable in the ML2 Extension Driver API  https://review.openstack.org/15275901:29
*** wanjing has quit IRC01:30
*** wanjing has joined #openstack-neutron01:30
*** jamielennox is now known as jamielennox|away01:33
*** shwetaap has quit IRC01:33
*** dims has quit IRC01:33
*** _cjones_ has quit IRC01:34
*** wanjing has quit IRC01:34
*** stanzgy has joined #openstack-neutron01:37
*** dims has joined #openstack-neutron01:38
*** dims has quit IRC01:42
*** dims has joined #openstack-neutron01:42
*** gildub has joined #openstack-neutron01:42
*** banix has quit IRC01:46
*** markmcclain has quit IRC01:46
*** popw has joined #openstack-neutron01:46
*** xuhanp has joined #openstack-neutron01:47
*** markmcclain has joined #openstack-neutron01:48
openstackgerritXu Han Peng proposed openstack/neutron: Not assign dynamic IPv6 address on dhcp interface  https://review.openstack.org/14950301:48
*** jasondotstar has quit IRC01:50
*** a_le has quit IRC01:51
*** a_le has joined #openstack-neutron01:51
*** Marga_ has quit IRC01:51
*** Marga_ has joined #openstack-neutron01:52
*** a_le has quit IRC01:56
*** aranjan has quit IRC01:56
dougwigsalv-orlando: right, it was the double import that was causing bombs for me.01:58
dougwigbecause it imports before it checks for the dup.01:58
*** yamamoto_ has quit IRC02:01
*** aranjan has joined #openstack-neutron02:02
openstackgerritJian LI proposed openstack/neutron: Fix extra-dhcp-opt on stateless dhcpv6 subnet  https://review.openstack.org/14745502:05
*** yamamoto_ has joined #openstack-neutron02:06
*** aleksandr_null has quit IRC02:06
*** Marga_ has quit IRC02:07
*** rwsu is now known as rwsu-afk02:11
*** aleksandr_null has joined #openstack-neutron02:11
*** erkules_ has joined #openstack-neutron02:13
*** aleksandr_null has quit IRC02:13
*** erkules has quit IRC02:16
*** aleksandr_null has joined #openstack-neutron02:18
*** shashankhegde has quit IRC02:22
*** leenheer has quit IRC02:22
openstackgerritTerry Wilson proposed openstack/neutron: WIP! Add native OVSDB implmentation of OVSDB API  https://review.openstack.org/15276102:24
*** jckasper has joined #openstack-neutron02:26
*** banix has joined #openstack-neutron02:29
*** salv-orlando has quit IRC02:29
*** diegows has quit IRC02:33
*** sthillma has joined #openstack-neutron02:33
*** nlahouti has joined #openstack-neutron02:34
openstackgerritRobert Kukura proposed openstack/neutron: ML2: DB changes for hierarchical port binding  https://review.openstack.org/11612202:34
openstackgerritRobert Kukura proposed openstack/neutron: ML2: Hierarchical port binding  https://review.openstack.org/11620902:34
*** vivek-ebay has quit IRC02:35
*** dims has quit IRC02:35
*** dims has joined #openstack-neutron02:36
*** sthillma has quit IRC02:38
*** aepifanov has joined #openstack-neutron02:39
*** nlahouti has quit IRC02:39
*** nlahouti has joined #openstack-neutron02:39
*** ajmiller has joined #openstack-neutron02:40
*** aranjan has quit IRC02:43
*** aepifanov has quit IRC02:44
*** Murali has joined #openstack-neutron02:49
*** spandhe has quit IRC02:50
*** dims has quit IRC02:51
*** tidwellr has left #openstack-neutron02:53
*** yalie has joined #openstack-neutron02:54
*** Xurong has quit IRC02:56
*** Murali_ has joined #openstack-neutron02:57
*** Murali has quit IRC02:58
*** Murali_ is now known as Murali02:58
*** claudiub has quit IRC03:00
*** Xurong has joined #openstack-neutron03:00
*** yeungp has quit IRC03:03
*** Murali has quit IRC03:03
*** tellesnobrega_ has joined #openstack-neutron03:03
*** ijw has joined #openstack-neutron03:09
*** sgran has quit IRC03:11
openstackgerritTerry Wilson proposed openstack/neutron: WIP! Add native OVSDB implmentation of OVSDB API  https://review.openstack.org/15276103:14
*** vivek-ebay has joined #openstack-neutron03:19
*** SridharRamaswamy has quit IRC03:20
openstackgerritTerry Wilson proposed openstack/neutron: Fix flake exclude matching of .*  https://review.openstack.org/15276403:26
*** signed8bit has quit IRC03:28
*** sthillma has joined #openstack-neutron03:28
*** harlowja is now known as harlowja_away03:28
*** sthillma has quit IRC03:30
*** manishg has joined #openstack-neutron03:36
*** sgran has joined #openstack-neutron03:37
openstackgerritRomil Gupta proposed openstack/neutron: External Networks in ML2 have invalid providernet types  https://review.openstack.org/14954603:40
*** manishg has quit IRC03:40
*** ajmiller has quit IRC03:42
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: Fix a usage error of joinedload + filter in dhcp scheduler  https://review.openstack.org/15028403:46
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: Fix a usage error of joinedload + filter in dhcp scheduler  https://review.openstack.org/15028403:49
*** tiswanso has quit IRC03:50
*** manishg has joined #openstack-neutron03:50
humboltIn OVS VLAN mode, why are br-ex and br-data being interlinked by OVS?! This is creating a routing loop for me.03:50
humboltHow is that supposed to be configured?03:51
*** SridharG has joined #openstack-neutron03:53
jaypipeskevinbenton: fascinating ML thread on the DHCP renewal times.03:55
*** banix has quit IRC03:56
*** mitz_ has quit IRC03:56
*** yuanying has quit IRC03:56
*** ijw has quit IRC03:56
*** yuanying_ has joined #openstack-neutron03:56
*** mitz has joined #openstack-neutron03:56
*** a_le has joined #openstack-neutron03:56
*** sgran has quit IRC03:58
*** ijw has joined #openstack-neutron03:58
*** sgran has joined #openstack-neutron03:58
openstackgerritDoug Wiegley proposed openstack/neutron: Nuke remaining service config and extensions from main repo  https://review.openstack.org/15154104:01
*** manishg has quit IRC04:02
*** manishg has joined #openstack-neutron04:03
*** vivek-eb_ has joined #openstack-neutron04:03
openstackgerritZhiyuan Cai proposed openstack/neutron-specs: Add port timestamp  https://review.openstack.org/15276704:03
*** humbolt has quit IRC04:04
*** vivek-ebay has quit IRC04:04
*** manishg has quit IRC04:07
*** ajmiller has joined #openstack-neutron04:07
openstackgerritZhiyuan Cai proposed openstack/neutron-specs: Add route whose nexthop not in router's attaching subnets  https://review.openstack.org/15276804:09
*** mmdurrant has quit IRC04:09
*** anilvenkata has joined #openstack-neutron04:12
openstackgerritZhiyuan Cai proposed openstack/neutron-specs: Support creating GRE tunnel between routers  https://review.openstack.org/15276904:12
*** SridharG has quit IRC04:15
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: ml2: Simplify _process_provider_create  https://review.openstack.org/15113004:18
*** wanjing has joined #openstack-neutron04:19
*** achanda has joined #openstack-neutron04:20
*** anilvenkata has quit IRC04:21
*** Sukhdev has joined #openstack-neutron04:22
*** achanda has quit IRC04:22
*** achanda has joined #openstack-neutron04:22
*** julim has quit IRC04:27
openstackgerritJuergen Brendel proposed openstack/neutron: ARP spoofing patch: Low level ebtables integration  https://review.openstack.org/14113004:28
*** _cjones_ has joined #openstack-neutron04:30
*** SridharG has joined #openstack-neutron04:31
*** _cjones_ has quit IRC04:32
*** _cjones_ has joined #openstack-neutron04:33
*** _cjones_ has quit IRC04:37
*** aranjan has joined #openstack-neutron04:55
*** changbl has joined #openstack-neutron04:57
*** yatin has joined #openstack-neutron04:59
*** oomichi has quit IRC05:04
*** tflynn_ has joined #openstack-neutron05:05
*** SridharG has left #openstack-neutron05:07
*** tflynn has quit IRC05:08
*** tflynn_ has quit IRC05:09
*** david-lyle_afk has quit IRC05:11
*** achanda has quit IRC05:11
*** david-lyle_afk has joined #openstack-neutron05:11
*** ajo has joined #openstack-neutron05:12
*** yshafrir has joined #openstack-neutron05:16
*** a_le has quit IRC05:18
*** a_le has joined #openstack-neutron05:19
*** harlowja_away has quit IRC05:19
*** ivar-laz_ has joined #openstack-neutron05:24
*** alex_xu has joined #openstack-neutron05:24
*** anish_ has joined #openstack-neutron05:25
*** bradjones has quit IRC05:27
*** achanda has joined #openstack-neutron05:27
*** ivar-lazzaro has quit IRC05:27
*** yamamot__ has joined #openstack-neutron05:27
*** ivar-laz_ has quit IRC05:28
*** bradjones has joined #openstack-neutron05:31
*** yamamoto_ has quit IRC05:31
*** pradipta_away is now known as pradipta05:35
*** vthapar has joined #openstack-neutron05:35
*** aranjan_ has joined #openstack-neutron05:35
*** aranjan has quit IRC05:39
*** vivek-eb_ has quit IRC05:39
*** mtreinish has quit IRC05:40
*** Murali has joined #openstack-neutron05:41
*** shwetaap has joined #openstack-neutron05:44
*** tomoe_ has joined #openstack-neutron05:45
*** tflynn has joined #openstack-neutron05:48
openstackgerritZhang Hua proposed openstack/neutron-vpnaas: IPsec strongSwan driver implemention  https://review.openstack.org/14439105:48
*** ivar-lazzaro has joined #openstack-neutron05:49
*** killer_prince is now known as lazy_prince05:49
*** tflynn_ has joined #openstack-neutron05:49
*** shwetaap1 has joined #openstack-neutron05:49
*** gongysh has joined #openstack-neutron05:52
*** dims has joined #openstack-neutron05:52
*** yfried_ has quit IRC05:52
*** irenab has joined #openstack-neutron05:53
*** shwetaap has quit IRC05:53
*** tflynn has quit IRC05:53
*** Marga_ has joined #openstack-neutron05:53
*** Marga_ has quit IRC05:53
*** Marga_ has joined #openstack-neutron05:54
*** mtreinish has joined #openstack-neutron05:56
*** tflynn_ has quit IRC05:56
*** tflynn has joined #openstack-neutron05:56
*** anilvenkata has joined #openstack-neutron05:57
*** dims has quit IRC05:57
*** coolsvap_ is now known as coolsvap05:57
*** achanda has quit IRC06:00
*** tflynn has quit IRC06:01
*** tflynn has joined #openstack-neutron06:01
*** achanda has joined #openstack-neutron06:05
*** anish__ has joined #openstack-neutron06:05
*** abehl has joined #openstack-neutron06:06
*** anish_ has quit IRC06:06
*** markvoelker has quit IRC06:08
*** aranjan_ has quit IRC06:10
*** achanda has quit IRC06:13
openstackgerritOpenStack Proposal Bot proposed openstack/neutron: Imported Translations from Transifex  https://review.openstack.org/15280106:15
*** jobewan has quit IRC06:15
*** SridharG has joined #openstack-neutron06:21
*** ivar-laz_ has joined #openstack-neutron06:21
*** Sukhdev has quit IRC06:22
*** numan has joined #openstack-neutron06:22
*** ivar-lazzaro has quit IRC06:24
*** gongysh has quit IRC06:24
ajohanzhang: https://review.openstack.org/#/c/147713/06:26
ajoI posted comments06:26
ajoit seems to work, but I found considerations we must take care of (probably documenting)06:26
openstackgerritArmando Migliaccio proposed openstack/neutron: Provide more details about testing strategies  https://review.openstack.org/15270706:26
ajohaleyb ^ if you have time, I believe it's an interesting thing to read: https://review.openstack.org/#/c/147713/06:27
*** itzikb has joined #openstack-neutron06:31
*** gildub has quit IRC06:31
*** tellesnobrega_ has quit IRC06:32
*** tellesnobrega__ has joined #openstack-neutron06:32
*** trinaths has joined #openstack-neutron06:33
*** yfauser has joined #openstack-neutron06:34
*** ijw has quit IRC06:34
*** achanda has joined #openstack-neutron06:35
*** yfauser has left #openstack-neutron06:37
*** markvoelker has joined #openstack-neutron06:39
hanzhangajo, I will read the comments as soon as possible06:40
*** markvoelker has quit IRC06:43
*** aepifanov has joined #openstack-neutron06:48
*** trinaths has quit IRC06:51
*** zz_avozza is now known as avozza06:52
*** avozza is now known as zz_avozza06:56
*** shashankhegde has joined #openstack-neutron06:58
*** erkules_ is now known as erkules06:58
*** josecastroleon has joined #openstack-neutron06:59
*** ivar-laz_ has quit IRC07:03
*** ivar-lazzaro has joined #openstack-neutron07:04
*** yfried_ has joined #openstack-neutron07:05
*** chlong has joined #openstack-neutron07:06
*** aranjan has joined #openstack-neutron07:08
*** yfried__ has joined #openstack-neutron07:10
*** yfried_ has quit IRC07:10
*** trinaths has joined #openstack-neutron07:10
*** obondarev has joined #openstack-neutron07:11
*** yfried__ is now known as yfried|afk07:11
*** yfried|afk is now known as yfried__07:12
*** tflynn has quit IRC07:14
*** shaohe_robot has joined #openstack-neutron07:14
*** tflynn has joined #openstack-neutron07:14
*** neeti has joined #openstack-neutron07:15
*** ivar-laz_ has joined #openstack-neutron07:16
*** Xurong has quit IRC07:17
*** a_le has quit IRC07:17
shaohe_robot jogo:   krtaylor:   hi.07:18
*** ivar-lazzaro has quit IRC07:19
*** ivar-laz_ has quit IRC07:19
*** a_le has joined #openstack-neutron07:21
*** xgerman_ has joined #openstack-neutron07:21
shaohe_robot jogo:   krtaylor:  I   come from intel.  We are  working on a SR-IOV CI.  It is closing to ready these days.   Hope this CI will be useful to neutron.07:21
*** ajo has quit IRC07:22
*** Xurong has joined #openstack-neutron07:22
*** shwetaap1 has quit IRC07:28
*** chlong has quit IRC07:28
*** shashankhegde has quit IRC07:30
*** shaohe_feng has joined #openstack-neutron07:30
*** ajmiller has quit IRC07:33
*** ajmiller has joined #openstack-neutron07:33
*** Miouge has joined #openstack-neutron07:35
*** belmoreira has joined #openstack-neutron07:36
*** ajo has joined #openstack-neutron07:39
*** markvoelker has joined #openstack-neutron07:39
*** moshele has joined #openstack-neutron07:40
*** akamyshnikova has joined #openstack-neutron07:44
*** markvoelker has quit IRC07:44
*** achanda has quit IRC07:46
*** luqas has joined #openstack-neutron07:47
*** nlahouti has quit IRC07:47
hanzhangDoes this SR-IOV CI also work for nova?07:52
*** yamamot__ has quit IRC07:52
*** zz_avozza is now known as avozza07:55
*** trinaths has quit IRC07:56
*** tflynn has quit IRC07:57
*** tflynn has joined #openstack-neutron07:57
*** xgerman_ has quit IRC07:57
*** scheuran has joined #openstack-neutron07:57
*** achanda has joined #openstack-neutron08:01
*** ronis has joined #openstack-neutron08:02
kevinbentonjaypipes: it's a microcosm of many warts in the Neutron API08:03
*** armax has quit IRC08:03
lukasakevinbenton: Do you have a sec to answer an ML2 question?08:04
kevinbentonlukasa: possibly08:06
kevinbentonlukasa: what is it?08:06
*** tflynn has quit IRC08:07
lukasaI'm looking at enhancing the Calico ML2 driver to support multiple instances of neutron-server running at the same time (e.g. behind a loadbalancer)08:07
lukasaAs a first question, am I right in thinking that, if I make two nearly-simultaneous port updates, one to each neutron-server, there is no defined order in which the *_postcommit methods will execute?08:09
lukasa(Because they're outside the DB transaction and so not serialized)08:09
*** Miouge_ has joined #openstack-neutron08:09
*** ajo has quit IRC08:10
kevinbentonlukasa: right08:11
*** Miouge has quit IRC08:11
*** Miouge_ is now known as Miouge08:11
lukasaSo, am I right in assuming that most of the old in-tree ML2 drivers simply don't worry about this case? Because I've seen no code anywhere handling the possibility of two port updates getting their order reversed.08:12
kevinbentonlukasa: in fact, if you are using an SQL galera cluster, the DB lock doesn't work so this can happen even on one server08:12
lukasaOh good point, I hadn't even thought of that08:12
lukasaOk, so I must be missing something08:12
kevinbentonlukasa: no, you have a really good point08:12
kevinbentonlukasa: i don't most drivers don't handle this well08:13
kevinbentonlukasa: i know*08:14
lukasaIt's definitely a small window08:14
lukasaBut that's really good to know08:14
*** trinaths has joined #openstack-neutron08:14
lukasaI just worried I was missing something really obvious08:14
*** SridharG has quit IRC08:14
*** SridharG1 has joined #openstack-neutron08:14
lukasaOne final question (this is more about the sqlalchemy setup in Neutron)08:14
lukasaI assume it's not possible for me to get a strictly ordered transaction ID in the *_postcommit hook08:15
lukasaIs that correct?08:16
*** pask81 has joined #openstack-neutron08:16
*** doude has joined #openstack-neutron08:18
kevinbentonlukasa: no, we don't even have a db transaction ID08:19
lukasaFair enough, that's pretty much the answer I expected. =)08:19
*** yamamoto_ has joined #openstack-neutron08:19
lukasaExcellent, that's been extremely helpful, thank you!08:19
lukasaOh, and congratulations on your recent 'promotion' to core!08:20
kevinbentonlukasa: thanks08:20
kevinbentonlukasa: about the update_port thing08:20
kevinbentonlukasa: i think that's a side effect of ML2 not offering great protection in this case08:21
*** kawa2014 has joined #openstack-neutron08:21
lukasaHow do you mean?08:21
kevinbentonlukasa: it should be updated to use an UPDATE where statement08:22
lukasaOh, sure. =)08:22
*** devvesa has joined #openstack-neutron08:22
lukasaFWIW, when I next see a mailing list thread about this, I might gently hint that some kind of ordered transaction ID would be really useful as well.08:23
kevinbentonlukasa: if we do that, we can guaruntee that the update_port_postcommit is handed the updated object and the object in its state directly before08:23
kevinbentonlukasa: that would work too, but would require a schema change08:23
*** yamamoto_ has quit IRC08:24
lukasaOh for sure, it's not something that's trivial to do.08:24
lukasaBut given that the DB is used to provide the ordering of events for neutron-server, it'd be good for plugins to be able to see what that ordering is.08:25
lukasaRegardless, I'm not suggesting it for Kilo (that ship has sailed long ago)08:25
*** achanda has quit IRC08:26
*** avozza is now known as zz_avozza08:26
kevinbentonlukasa: the reference plugin at least becomes eventually consistent in these cases08:26
*** yamamoto_ has joined #openstack-neutron08:26
kevinbentonlukasa: because the agents poll for state occasionally08:26
lukasaYeah, that's what we do too08:26
lukasaBut we'd like to move that 'eventually' in our case to be a lot shorter without increasing the load on neutron-server08:27
*** nmagnezi has joined #openstack-neutron08:27
*** jlibosva has joined #openstack-neutron08:27
kevinbentonlukasa: can you do the same model as the reference where you get push events for faster reactions08:28
kevinbentonlukasa: and slow-poll to correct for ordering errors/lost notifications08:28
lukasaSo that's basically exactly what we do08:28
lukasaWe get pushed every event08:28
lukasaCurrently we poll on a configurable interval to resynchronize08:29
*** jistr has joined #openstack-neutron08:29
lukasaWe're looking to add a way of spotting when we might need to08:29
lukasaUsing a lamport clock is one option (DVC won't go well, too many nodes involved)08:29
lukasaBut we might also just use an OSS distributed database to solve this problem for us rather than effectively writing one ourselves08:30
shaohe_robothanzhang: actually, we create a sriov port assign a vm created by nova.08:30
shaohe_robothanzhang:  and then ping the nova08:31
shaohe_robothanzhang: ping the vm created by nova. sorry.08:31
shaohe_robothanzhang:  the network, subnet, and port are created by neutron.08:31
*** apuimedo has joined #openstack-neutron08:32
hanzhangok, it is very good, now we are using intel 82599 for sriov nic08:32
shaohe_robothanzhang: I also use 82599 on my desktop08:34
kevinbentonlukasa: how would you use a lamport clock without changes to neutron to handle the two-server case?08:34
*** redbeard has joined #openstack-neutron08:35
lukasakevinbenton: With difficulty. ;)08:36
lukasaScoping it out, you increment the lamport clock in the *_precommit hook, and then deal with distributing messages in *_postcommit08:36
lukasaI'd also have to add a communication mechanism betweem multiple ML2 driver instances08:37
lukasa(UDP via IP multicast seemed like the way to go there)08:37
kevinbentonlukasa: right, that last bit seems painful08:37
lukasaIt's certainly not ideal.08:37
lukasaYou're right that the real solution for this is to change neutron, but that's not necessarily *less* painful. ;)08:38
*** jlanoux has joined #openstack-neutron08:38
lukasaBut more importantly, I don't know enough neutron to know what that change entails, at least at the moment.08:38
kevinbentonlukasa: where do you store your clock value in each driver?08:39
lukasaAs an instance variable on the driver class. That means having multiple api_workers effectively acts like having multiple neutron servers.08:40
*** markvoelker has joined #openstack-neutron08:40
*** yamamoto_ has quit IRC08:41
*** jpena has joined #openstack-neutron08:44
kevinbentonlukasa: in your postcommit method, just re-read from the database08:45
*** anish__ has quit IRC08:45
*** markvoelker has quit IRC08:45
kevinbentonlukasa: that's much easier :)08:45
lukasaYeah, that's true. =)08:46
*** nyechiel has joined #openstack-neutron08:47
lukasaStill need the Lamport clock value to deal with misbehaving networks, but agreed08:47
*** yuanying has joined #openstack-neutron08:47
lukasaThanks for that reminder08:47
lukasa=)08:47
kevinbentonlukasa: do you mean a partitioned database backend?08:48
kevinbentonlukasa: or what do you mean by a misbehaving network?08:49
*** yuanying_ has quit IRC08:50
*** gongysh has joined #openstack-neutron08:52
*** aranjan has quit IRC08:52
*** anish_ has joined #openstack-neutron08:52
*** stanzgy has quit IRC08:52
lukasaSo, for example08:54
*** matrohon has joined #openstack-neutron08:54
*** SridharG has joined #openstack-neutron08:54
*** gongysh has quit IRC08:54
*** SridharG1 has quit IRC08:54
*** stanzgy has joined #openstack-neutron08:54
*** gongysh has joined #openstack-neutron08:55
lukasaNetwork congestion or high latency on a link from one neutron-server instance to our agent, but no equivalent latency on the link from a different instance to our agent08:55
*** gongysh has quit IRC08:55
*** gongysh has joined #openstack-neutron08:55
lukasaCan lead to the agent processing the two events backwards08:55
*** jpich has joined #openstack-neutron08:55
kevinbentonlukasa: oh, i see what you mean08:55
*** gongysh_ has joined #openstack-neutron08:56
lukasaStandard distributed systems problem, sadly.08:56
lukasa=(08:56
kevinbentonlukasa: just need a distributed locking mechanism to serialize the requests :)08:56
*** ygbo has joined #openstack-neutron08:57
lukasaHaha, yeah, but at that stage it might be better just to use an eventually-consistent distributed database and call it a day. =D08:57
kevinbentonlukasa: acquire lock, read from db, send to agent, relase lock08:57
kevinbentonlukasa: yeah :)08:57
lukasaDefinitely a fun problem to solve though. =)08:58
kevinbentonlukasa: https://github.com/openstack/neutron/commit/cdaa502f899d4c90c5e40ccd745f1d92bfd1127b09:01
kevinbentonlukasa: i dealt with the serialization problem before09:01
kevinbentonlukasa: because of another problem we had with multiple servers09:01
kevinbentonlukasa: but i think we have the same update-port problem you just described09:02
lukasaNice! That's one fantastic commit message, by the way.09:03
*** karimb has joined #openstack-neutron09:04
*** humbolt has joined #openstack-neutron09:04
*** humbolt has quit IRC09:04
*** ajo has joined #openstack-neutron09:05
kevinbentonlukasa: thanks09:05
kevinbentonlukasa: was a bit much09:05
kevinbentonlukasa: but was trying to make it easy for reviews to see what was going on and why09:05
kevinbentonreviewers*09:06
lukasaYeah, that's really the value. Any commit like that that does some black magic should be pretty explanatory09:06
*** tomoe_ has quit IRC09:09
*** gongysh_ has quit IRC09:09
*** gongysh has quit IRC09:09
*** gongysh has joined #openstack-neutron09:09
gongysh#HenryG, hi09:11
*** safchain has joined #openstack-neutron09:19
*** belmoreira has quit IRC09:21
*** belmoreira has joined #openstack-neutron09:22
*** Miouge has quit IRC09:25
*** afazekas has joined #openstack-neutron09:25
*** igordcard has joined #openstack-neutron09:27
*** jckasper has quit IRC09:30
*** Miouge has joined #openstack-neutron09:32
*** nyechiel_ has joined #openstack-neutron09:35
*** salv-orlando has joined #openstack-neutron09:37
ZZellegongysh, hi09:38
*** nyechiel has quit IRC09:38
*** yfried__ is now known as yfried|afk09:38
*** boris-42 has joined #openstack-neutron09:39
*** dsneddon has quit IRC09:40
*** markvoelker has joined #openstack-neutron09:41
openstackgerritAlexey I. Froloff proposed openstack/neutron: Pass '--dhcp-authoritative' option to dnsmasq  https://review.openstack.org/15208009:45
*** igordcard has quit IRC09:45
*** anish_ has quit IRC09:46
*** markvoelker has quit IRC09:47
*** anish_ has joined #openstack-neutron09:47
*** igordcard has joined #openstack-neutron09:49
*** anish_ has quit IRC09:49
*** yfried|afk is now known as yfried__09:49
*** anish_ has joined #openstack-neutron09:50
*** igordcard has quit IRC09:54
trinathsHi all, Freescale CI is active now.10:01
*** gongysh has quit IRC10:03
*** SridharG has quit IRC10:04
*** scheuran has quit IRC10:04
openstackgerritZhiyuan Cai proposed openstack/neutron: Return false for empty subnet_ids  https://review.openstack.org/15284510:09
*** tomoe_ has joined #openstack-neutron10:10
*** SridharG has joined #openstack-neutron10:10
*** ihrachyshka has joined #openstack-neutron10:12
*** blues-man has joined #openstack-neutron10:12
*** tomoe_ has quit IRC10:14
*** gongysh has joined #openstack-neutron10:15
*** moshele has quit IRC10:16
*** dsneddon has joined #openstack-neutron10:16
*** scheuran has joined #openstack-neutron10:18
*** ajmiller has quit IRC10:21
*** ajmiller has joined #openstack-neutron10:22
*** aranjan has joined #openstack-neutron10:23
*** xuhanp has quit IRC10:23
*** yfauser has joined #openstack-neutron10:24
*** yfauser has left #openstack-neutron10:24
kevinbentonlukasa: read from the db, do your backend call, read from the db to see if anything changed, if so, do your backend call again :)10:26
*** gongysh has quit IRC10:26
*** oreillyd has joined #openstack-neutron10:26
*** aranjan has quit IRC10:27
kevinbentonlukasa: protects against slow call out of order issues and no distributed locking10:29
*** yfried__ is now known as yfried|afk10:31
*** Murali_ has joined #openstack-neutron10:32
*** Murali has quit IRC10:34
*** Murali_ is now known as Murali10:34
lukasakevinbenton: Mmm, another solid alternative10:36
*** yfried|afk is now known as yfried__10:37
*** yfauser has joined #openstack-neutron10:37
*** igordcard has joined #openstack-neutron10:39
*** yfauser has left #openstack-neutron10:39
*** devvesa has quit IRC10:41
*** markvoelker has joined #openstack-neutron10:43
*** jasondotstar has joined #openstack-neutron10:46
*** gongysh has joined #openstack-neutron10:46
*** jp_at_hp has joined #openstack-neutron10:46
*** alexpilotti has joined #openstack-neutron10:47
*** stanzgy has quit IRC10:47
openstackgerritSalvatore Orlando proposed openstack/neutron: Restore and fix vmware unit tests  https://review.openstack.org/15192810:47
*** markvoelker has quit IRC10:48
*** yfried__ is now known as yfried|afk10:48
*** belmoreira has quit IRC10:48
*** yfried|afk is now known as yfried__10:49
*** devvesa has joined #openstack-neutron10:51
*** nyechiel_ has quit IRC10:51
*** igordcard has quit IRC10:54
*** Longgeek has joined #openstack-neutron10:55
openstackgerritMathieu Rohon proposed openstack/neutron: Check if the port is ACTIVE before sending update_fdb message  https://review.openstack.org/15004510:57
*** dims has joined #openstack-neutron10:57
openstackgerritMathieu Rohon proposed openstack/neutron: Check if the port is ACTIVE before sending update_fdb message  https://review.openstack.org/15004510:58
*** moshele has joined #openstack-neutron10:58
*** dims has quit IRC11:01
*** dims has joined #openstack-neutron11:02
openstackgerritTrinath Somanchi proposed openstack/neutron-fwaas: Code cleanup for Freescale FWaaS Plugin  https://review.openstack.org/15285611:03
*** lori|away is now known as lori11:03
*** ihrachyshka has quit IRC11:04
*** irenab has quit IRC11:06
*** irenab has joined #openstack-neutron11:17
openstackgerritPavel Bondar proposed openstack/neutron: Implement IPAM Driver loader  https://review.openstack.org/14747911:21
openstackgerritTrinath Somanchi proposed openstack/neutron-fwaas: Code cleanup for Freescale FWaaS Plugin  https://review.openstack.org/15285611:21
*** claudiub has joined #openstack-neutron11:24
*** tnurlygayanov has joined #openstack-neutron11:27
*** redbeard has quit IRC11:29
*** Longgeek has quit IRC11:33
openstackgerritYoni Shafrir proposed openstack/neutron: Remove use of keepalived 'vrrp_sync_group' as it is unused  https://review.openstack.org/15286111:33
*** scheuran has quit IRC11:34
*** tomoe_ has joined #openstack-neutron11:34
*** aranjan has joined #openstack-neutron11:38
*** tomoe_ has quit IRC11:39
*** tellesnobrega__ has quit IRC11:43
HenryGgongysh: hi11:44
*** markvoelker has joined #openstack-neutron11:44
*** pc_m has joined #openstack-neutron11:45
*** nyechiel_ has joined #openstack-neutron11:49
*** scheuran has joined #openstack-neutron11:49
ajokevinbenton, option 58 in DHCP seems like an optimal solution to me11:49
*** markvoelker has quit IRC11:49
*** nyechiel_ has quit IRC11:49
*** moshele has quit IRC11:49
*** nyechiel_ has joined #openstack-neutron11:49
*** nyechiel_ is now known as nyechiel11:49
ajokevinbenton, we just keep the best of both needs, only incrementing the dhcp traffic a bit, but renewals should be unicast... so..11:49
*** yfauser has joined #openstack-neutron11:49
ajokevinbenton, may be setting option 58 can be made optional for those who don't need the extra traffic...11:50
*** igordcard has joined #openstack-neutron11:50
ajoneed=want11:50
*** ihrachyshka has joined #openstack-neutron11:52
*** yfauser has left #openstack-neutron11:52
*** byte69 has joined #openstack-neutron11:53
byte69Hi, i have a little problem in understanding network node tunneling, what is it meant to be and is this network configuration correct? http://paste.openstack.org/show/166750/ i cannot ping from network node to compute node but i can ping to google.com and controller node11:54
*** diegows has joined #openstack-neutron11:57
*** moshele has joined #openstack-neutron11:57
*** sbfox has joined #openstack-neutron11:59
*** david-lyle_afk is now known as david-lyle12:01
*** jyuso1 has joined #openstack-neutron12:03
openstackgerritSalvatore Orlando proposed openstack/neutron: VMware: consolidate NSX models  https://review.openstack.org/15287012:03
*** byte69 has quit IRC12:03
*** nyechiel_ has joined #openstack-neutron12:04
*** nyechiel has quit IRC12:04
openstackgerritCedric Brandily proposed openstack/neutron: Do not run neutron-ns-metadata-proxy as root on dhcp agent  https://review.openstack.org/14743712:04
shaohe_robotgongysh:  are you online ?12:04
shaohe_robotgongysh:  I'm Feng Shao He.12:05
shaohe_robothanzhang:  are you online?12:05
shaohe_robothi all:  I  counter a issue we I start a VM with a sriov port.12:06
shaohe_robotthe log as follow:12:06
shaohe_robotCommand: ['sudo', '/usr/local/bin/neutron-rootwrap', '/etc/neutron/rootwrap.conf', 'ip', 'link', 'set', 'em3', 'vf', '4', 'state', 'enable']12:06
shaohe_robotExit code: 212:06
shaohe_robotStdout:12:06
shaohe_robotStderr: RTNETLINK answers: Operation not supported12:06
*** rpothier has joined #openstack-neutron12:07
shaohe_robotsorry,   s/counter/encounter12:07
shaohe_robotanyone can help me?12:08
*** Dark_Matt3r has joined #openstack-neutron12:09
*** enikanorov has joined #openstack-neutron12:10
*** bradjones has quit IRC12:11
*** sbfox1 has joined #openstack-neutron12:11
*** aranjan has quit IRC12:13
*** jyuso has joined #openstack-neutron12:13
*** bradjones has joined #openstack-neutron12:13
*** alexpilotti has quit IRC12:13
*** trown|call has joined #openstack-neutron12:16
*** redbeard has joined #openstack-neutron12:16
*** Murali_ has joined #openstack-neutron12:16
*** yatin_ has joined #openstack-neutron12:16
*** vthapar has quit IRC12:16
*** vthapar has joined #openstack-neutron12:17
*** ilyashakhat_ has joined #openstack-neutron12:17
*** Murali has quit IRC12:17
*** Murali_ is now known as Murali12:17
trinathsajo: thanks for the review :) .12:17
*** kevinbenton_ has joined #openstack-neutron12:17
ajotrinaths, np, it looked simple for my untrained VPNaaS eye ;)12:18
ajoFWaaS sorry12:18
ajo:D12:18
*** rodrigod` has joined #openstack-neutron12:18
trinathsajo: :D.. these were suggested by dougwig during the final review of another patch. So pushed a new commit for that.12:19
ajo:-)12:19
*** trinaths has quit IRC12:19
*** sbfox has quit IRC12:20
*** jasondotstar has quit IRC12:20
*** afazekas has quit IRC12:20
*** jyuso1 has quit IRC12:20
*** yatin has quit IRC12:20
*** trown has quit IRC12:20
*** mwagner_lap has quit IRC12:20
*** kevinbenton has quit IRC12:20
*** rodrigods has quit IRC12:20
*** enikanorov__ has quit IRC12:20
*** ilyashakhat has quit IRC12:20
*** sbadia has quit IRC12:20
*** Chinorro has quit IRC12:20
*** Chinorro_ has joined #openstack-neutron12:20
*** sbfox1 has quit IRC12:21
*** rodrigod` is now known as rodrigods12:22
*** rodrigods has quit IRC12:22
*** rodrigods has joined #openstack-neutron12:22
*** lukasa has quit IRC12:22
*** lukasa has joined #openstack-neutron12:23
*** mohitsha_ has joined #openstack-neutron12:23
*** mohitsh__ has joined #openstack-neutron12:24
*** mohitsharma has quit IRC12:25
*** claudiub has quit IRC12:25
*** kawa2014 has quit IRC12:25
*** afazekas has joined #openstack-neutron12:27
*** mwagner_lap has joined #openstack-neutron12:27
*** EmilienM|afk is now known as EmilienM12:27
*** sbadia has joined #openstack-neutron12:27
*** mohitsha_ has quit IRC12:27
HenryGsalv-orlando: hi12:29
*** sbfox has joined #openstack-neutron12:30
openstackgerritRomil Gupta proposed openstack/neutron: External Networks in ML2 have invalid providernet types  https://review.openstack.org/14954612:35
*** nyechiel__ has joined #openstack-neutron12:35
*** david-lyle is now known as david-lyle_afk12:36
*** ihrachyshka_ has joined #openstack-neutron12:36
*** ihrachyshka has quit IRC12:36
openstackgerritYoni Shafrir proposed openstack/neutron: Remove use of keepalived 'vrrp_sync_group' as it is unused  https://review.openstack.org/15286112:36
*** baoli_ has joined #openstack-neutron12:36
*** david-lyle_afk is now known as david-lyle12:36
*** nyechiel_ has quit IRC12:37
*** baoli_ has quit IRC12:40
*** baoli has joined #openstack-neutron12:40
*** vthapar has quit IRC12:40
salv-orlandohi HenryG12:41
salv-orlandohow can I help you?12:41
HenryGsalv-orlando: https://review.openstack.org/#/c/151928/7/tox.ini12:41
*** vthapar has joined #openstack-neutron12:41
HenryGsalv-orlando: If you do a quick respin I will +2 again12:41
salv-orlandoHenryG: thanks will do12:42
openstackgerritSalvatore Orlando proposed openstack/neutron: Restore and fix vmware unit tests  https://review.openstack.org/15192812:43
salv-orlandoHenryG: ^^12:43
*** juice has quit IRC12:43
*** zz_avozza is now known as avozza12:43
HenryGsalv-orlando: Thanks! done. Let's hope jenkins plays nice.12:44
*** david-lyle is now known as david-lyle_afk12:44
*** david-lyle_afk is now known as david-lyle12:45
*** markvoelker has joined #openstack-neutron12:45
*** blues-man has quit IRC12:46
*** juice has joined #openstack-neutron12:46
*** haramaki has joined #openstack-neutron12:49
*** markvoelker has quit IRC12:49
salv-orlandoHenryG: never trust a Jenkins12:50
salv-orlandothey're fiddly and shady people12:50
*** yatin_ has left #openstack-neutron12:53
*** Murali_ has joined #openstack-neutron12:53
HenryGI tend to think jenkins is like Jeeves the PG Wodehouse character. A lot smarter than us.12:55
*** Murali has quit IRC12:55
*** Murali_ is now known as Murali12:55
HenryGAnd he looks like Stephen Fry.12:56
*** anilvenkata has left #openstack-neutron12:56
*** tomoe_ has joined #openstack-neutron12:57
*** claudiub has joined #openstack-neutron12:58
*** juice has quit IRC12:59
*** xuhanp has joined #openstack-neutron13:00
*** rook has joined #openstack-neutron13:00
*** sbfox has quit IRC13:00
*** aix has joined #openstack-neutron13:01
*** Murali has quit IRC13:01
*** gongysh has quit IRC13:02
*** rmar04 has joined #openstack-neutron13:03
*** juice has joined #openstack-neutron13:03
*** Murali has joined #openstack-neutron13:03
openstackgerritYoni Shafrir proposed openstack/neutron: Remove use of keepalived 'vrrp_sync_group' as it is unused  https://review.openstack.org/15286113:04
openstackgerritRossella Sblendido proposed openstack/neutron: Add devices to update in RPC call security_groups_provider_updated  https://review.openstack.org/14823513:04
*** mohitsh__ has quit IRC13:06
*** mohitsharma has joined #openstack-neutron13:07
*** neeti has quit IRC13:07
*** Networkn3rd has quit IRC13:08
*** haramaki has quit IRC13:08
*** mohitsharma has quit IRC13:08
*** aranjan has joined #openstack-neutron13:09
*** Miouge has quit IRC13:09
openstackgerritPavel Bondar proposed openstack/neutron: Implement IPAM Driver loader  https://review.openstack.org/14747913:09
*** dims has quit IRC13:10
*** pradipta has quit IRC13:13
*** shaohe_feng has quit IRC13:14
*** gongysh has joined #openstack-neutron13:15
*** dims has joined #openstack-neutron13:19
*** prad has joined #openstack-neutron13:19
*** markvoelker has joined #openstack-neutron13:20
openstackgerritenikanorov proposed openstack/neutron: Unify logic that determines liveliness of DHCP agent  https://review.openstack.org/15289113:24
*** dims has quit IRC13:28
*** karimb has quit IRC13:29
*** amuller has joined #openstack-neutron13:30
*** prad has quit IRC13:31
*** doude has quit IRC13:33
openstackgerritZhang Hua proposed openstack/neutron-vpnaas: IPsec strongSwan driver implemention  https://review.openstack.org/14439113:39
*** britthouser has quit IRC13:39
openstackgerritenikanorov proposed openstack/neutron: Set default of api_workers to number of CPUs  https://review.openstack.org/14049313:40
*** raildo_away is now known as raildo13:41
openstackgerritRomil Gupta proposed openstack/neutron: External Networks in ML2 have invalid providernet types  https://review.openstack.org/14954613:43
*** trown|call is now known as trown13:43
*** aranjan has quit IRC13:43
*** karimb has joined #openstack-neutron13:43
*** doude has joined #openstack-neutron13:44
*** pc_m has quit IRC13:48
*** shwetaap has joined #openstack-neutron13:51
*** thomasem has joined #openstack-neutron13:54
*** anish_ has quit IRC13:58
*** redbeard has quit IRC13:58
*** yshafrir has quit IRC13:58
*** vthapar has quit IRC14:00
*** jckasper has joined #openstack-neutron14:01
*** pc_m has joined #openstack-neutron14:01
*** mohitsharma has joined #openstack-neutron14:01
*** yamamoto has joined #openstack-neutron14:02
*** haramaki has joined #openstack-neutron14:02
*** ajmiller has quit IRC14:03
*** thomasem has quit IRC14:03
*** thomasem has joined #openstack-neutron14:04
*** ggnel has quit IRC14:04
*** thomasem has quit IRC14:05
openstackgerritAssaf Muller proposed openstack/neutron: Move extra routes processing to router classes  https://review.openstack.org/15015414:05
*** thomasem has joined #openstack-neutron14:05
openstackgerritAssaf Muller proposed openstack/neutron: Move extra routes processing to router classes  https://review.openstack.org/15015414:06
*** kbringard has joined #openstack-neutron14:08
*** dane_leblanc has joined #openstack-neutron14:10
*** jlibosva has quit IRC14:10
mesteryajo: Good afternoon! Any word on a new version of this patch: https://review.openstack.org/#/c/152512/14:11
*** belmoreira has joined #openstack-neutron14:11
ajomestery; I was going for it, I had a bit of a shaky day (had to go and bring my wife back from work... flu...)14:11
ajoand I was looking into a rootwrap issue we have for functional testing, but that's done for now14:11
mesteryajo: Yikes, that's awful! I was dealing with the same for my oldest son yesterday, please take care of your wife!14:12
ajoso I was jumping into it... will be interrupted by other events, but... we shall have a new version in 4h max14:12
*** shwetaap has quit IRC14:12
*** jlibosva has joined #openstack-neutron14:12
ajomestery, ahh, poor him (and you), at least this flu is striking quite hard around here...14:12
mesteryajo: Yeah, it's no fun for sure.14:13
*** johnbelamaric has joined #openstack-neutron14:13
*** shashankhegde has joined #openstack-neutron14:14
*** reed has joined #openstack-neutron14:14
*** johnbelamaric has quit IRC14:14
*** johnbelamaric has joined #openstack-neutron14:14
*** redbeard has joined #openstack-neutron14:15
*** Marga_ has quit IRC14:17
*** jckasper has quit IRC14:18
*** prad has joined #openstack-neutron14:19
*** jckasper has joined #openstack-neutron14:19
*** julim has joined #openstack-neutron14:19
*** julim has quit IRC14:21
openstackgerritLinChengyong proposed openstack/neutron-specs: blueprint virtual-network-performance-monitor  https://review.openstack.org/14238614:21
*** julim has joined #openstack-neutron14:24
openstackgerritAndrey Kurilin proposed openstack/neutron: Remove versioning import of novaclient  https://review.openstack.org/15290714:28
*** chenleji has joined #openstack-neutron14:28
*** dims has joined #openstack-neutron14:29
*** dims_ has joined #openstack-neutron14:33
*** Longgeek has joined #openstack-neutron14:33
*** ZZelle has quit IRC14:34
*** yfauser has joined #openstack-neutron14:35
*** ZZelle has joined #openstack-neutron14:35
*** yfauser has left #openstack-neutron14:35
*** Longgeek has quit IRC14:35
*** tiswanso has joined #openstack-neutron14:36
*** dims has quit IRC14:37
*** aranjan has joined #openstack-neutron14:39
*** shwetaap has joined #openstack-neutron14:40
*** baoli has quit IRC14:41
*** baoli has joined #openstack-neutron14:43
*** humbolt1 has joined #openstack-neutron14:43
*** apuimedo has quit IRC14:44
humbolt1I would like to know, what exactly  external_network_bridge  is for?!14:45
humbolt1I understand the intention, but do I really need this interface? Or can I run my external networks over the same bridge as my provider networks?14:46
*** packet has joined #openstack-neutron14:47
*** Murali_ has joined #openstack-neutron14:51
*** gongysh has quit IRC14:53
*** rushil has joined #openstack-neutron14:54
*** signed8bit has joined #openstack-neutron14:54
*** ollivier has quit IRC14:54
*** Murali has quit IRC14:54
*** Murali_ is now known as Murali14:54
rkukurahumbolt1: Unless things have changed, you absolutely can use a normal provider network of any type instead of external_network_bridge. Just set include “external_network_bridge=“ in your config to override the default of “br-ex”, and create the external network with the proper provider attributes.14:54
humbolt1provider:network_type     | vlan, provider:physical_network | physdata, provider:segmentation_id  | 120, router:external           | True14:56
humbolt1rkukura: I was able to set everything up, but the routers don't show in "ip netns list"14:57
humbolt1and I am totally confused about whether one needs an extra br-data or not. br-data is currently my VLAN trunk port bridge and associated with physdata.14:59
*** absubram has left #openstack-neutron14:59
humbolt1or would I just put everything on br-int?14:59
*** tiswanso has quit IRC14:59
*** tiswanso has joined #openstack-neutron15:01
rkukurahumbolt1: I’m not sure where br-data comes from. I’d typically setup br-eth1 or something like that and configure the phyiscal NIC on it. You need to then pass —provider:physical_network=br-eth1 when creating your external network.15:01
*** kawa2014 has joined #openstack-neutron15:02
*** johnbelamaric has quit IRC15:02
*** itzikb has quit IRC15:02
*** johnbelamaric has joined #openstack-neutron15:02
rkukurahumbolt1: My hands-on experience with these deployment details dates to icehouse, so things may have changed, particularly with DVR.15:02
*** phil_h has joined #openstack-neutron15:03
mesteryrkukura: I think that humbolt1 is just using a br-data in place of br-eth1 in your example.15:03
*** signed8bit is now known as signed8bit_ZZZzz15:03
mesteryAt least from reading the backscrool, that's what I'd assume15:03
humbolt1mestery: right you are15:04
rkukurahumbolt1: Looks like you were using “physdata” as your physical network name. You’re ovs or linuxbridge agents need to have mappings configured from the birdge name to the actual bridge name.15:04
rkukuraI meant from the physical_network name to the bridge name.15:04
*** rushil has quit IRC15:04
humbolt1rkukura: this mapping is in place15:04
rkukuramestery: OK15:05
rkukurahumbolt1: So physical_network name is “physdata” and you’ve got a bridge_mapping from “physdata” to “br-data”. That all sounds like it should work.15:06
humbolt1I can ping my nova VM with this: ip netns exec qdhcp-6daf4cfe-02fc-4770-b80a-924a8f5a8762 ping -c3 10.30.0.3415:07
humbolt1rkukura: but there are no qrouter entries in "ip netns list"15:07
humbolt110.30.0.254 network:router_interfaceDOWN15:08
humbolt1says the dashboard15:08
*** ijw has joined #openstack-neutron15:09
*** Murali has quit IRC15:09
humbolt1Internal Interface and External Gateway have Status DOWN but admin state UP.15:09
*** zz_jgrimm is now known as jgrimm15:09
humbolt1rkukura: I don't get it.15:09
*** britthouser has joined #openstack-neutron15:09
humbolt1mestery: and I had this running before with an extra br-ex15:09
*** numan has quit IRC15:11
*** baoli has quit IRC15:11
*** irenab has quit IRC15:12
*** baoli has joined #openstack-neutron15:12
*** ihrachyshka_ is now known as ihrachyshka15:13
ChuckC_humbolt1: how do you currently have external_network_bridge set?15:13
humbolt1ChuckC_: no, I have commented it out15:13
*** aranjan has quit IRC15:13
ChuckC_try 'external_network_bridge=' in the config file15:14
*** EmilienM has quit IRC15:14
*** rm_work|away is now known as rm_work15:14
*** julim has quit IRC15:15
ChuckC_i believe that the default value is br-ex, so it might be expecting that bridge15:15
humbolt1ChuckC_: good hint! I'll set it to "external_network_bridge=" then15:15
*** EmilienM has joined #openstack-neutron15:16
*** signed8bit_ZZZzz is now known as signed8bit15:17
humbolt1ChuckC_: do I have to restart anything else but neutron-l3-agent after start?15:17
*** julim has joined #openstack-neutron15:18
humbolt1after changing the value, I mean.15:18
ChuckC_humbolt1: I don't think so15:18
humbolt1and setting external_network_bridge=br-data, does that make any sense?15:19
openstackgerritIhar Hrachyshka proposed openstack/neutron: Check conntrack rule cleanup on floating IP disassociate  https://review.openstack.org/13393415:19
*** jgjl has joined #openstack-neutron15:19
*** redbeard has quit IRC15:19
*** amotoki has joined #openstack-neutron15:20
ChuckC_humbolt1: I haven't played with that, so not sure15:20
humbolt1damn me, it works now!!! thanks ChuckC_15:21
ChuckC_humbolt1: np15:21
humbolt1ChuckC_: I have read different things about whether or now one must "ip link set eth1 promisc on" on the interface of br-data in my case and/or br-ex in other scenarios. Any comments to that?15:23
*** rpothier has quit IRC15:23
*** avozza is now known as zz_avozza15:23
openstackgerritDoug Wiegley proposed openstack/neutron: Nuke remaining service config and extensions from main repo  https://review.openstack.org/15154115:24
*** Networkn3rd has joined #openstack-neutron15:25
*** johnbelamaric has quit IRC15:25
ChuckC_humbolt1: no, sorry, I've seen similar things said, but I haven't had to do it myself15:26
ChuckC_humbolt1: folks in #openstack may have broader experience15:26
*** SridharG has left #openstack-neutron15:26
*** moshele has quit IRC15:28
openstackgerritvishwanath jayaraman proposed openstack/neutron-fwaas: Implementation of Brocade Vyatta Firewall driver  https://review.openstack.org/14760015:28
*** shivharis has joined #openstack-neutron15:28
*** jlibosva has quit IRC15:28
*** yamahata has joined #openstack-neutron15:28
*** johnbelamaric has joined #openstack-neutron15:29
rkukurahumbolt1: I had advized “Just set include “external_network_bridge=“ in your config to override the default of “br-ex”, and create the external network with the proper provider attributes.” above, so thought you had already done this, but I didn’t mention restarting the L3 agent. I haven’t need to set promisc, but that could be platform or even NIC driver dependent.15:30
*** Sukhdev has joined #openstack-neutron15:30
*** sbelous has joined #openstack-neutron15:31
openstackgerritAssaf Muller proposed openstack/neutron: Move extra routes processing to router classes  https://review.openstack.org/15015415:31
*** tomoe_ has quit IRC15:32
*** redbeard has joined #openstack-neutron15:33
humbolt1rkukura: I thought “external_network_bridge=“ would do the same as #“external_network_bridge=“.15:33
humbolt1I just realized it does not.15:33
Sukhdevmestery: Ping15:34
rkukurahumbolt1: That’s why I said “to override the default of ‘br-ex’”15:34
ChuckC_humbolt1: i had made the same mistake in the past15:34
*** vivek-ebay has joined #openstack-neutron15:34
humbolt1rkukura: thanks. I did overlook this after troubleshooting this for 2 days now with barely any sleep.15:35
rkukuraI’d really prefer that the default was empty. The external_bridge was supposed to be a temporary solution since provider networks and the router were being developed concurrently.15:35
*** EmilienM is now known as EmilienM|afk15:35
ChuckC_rkukura: would be less confusing to some of us ;)15:35
*** sbelous has quit IRC15:36
openstackgerritIhar Hrachyshka proposed openstack/neutron: execute: enforce C locale unless explicitly requested  https://review.openstack.org/14995115:36
Sukhdevmestery: Can you please tag this for K2, please - it is already merged. - https://bugs.launchpad.net/neutron/+bug/141257715:36
*** sbelous has joined #openstack-neutron15:37
*** ChuckC has joined #openstack-neutron15:40
*** ChuckC_ has quit IRC15:41
*** kkxue has joined #openstack-neutron15:43
*** ajmiller has joined #openstack-neutron15:43
*** aranjan has joined #openstack-neutron15:43
*** diegows has quit IRC15:44
openstackgerritShweta P proposed openstack/neutron: Passes the context variable in the ML2 Extension Driver API  https://review.openstack.org/15275915:44
*** trinaths has joined #openstack-neutron15:45
*** popw has quit IRC15:45
*** vivek-ebay has quit IRC15:46
*** shashankhegde has quit IRC15:46
*** trinaths1 has joined #openstack-neutron15:47
*** aranjan has quit IRC15:47
*** tomoe_ has joined #openstack-neutron15:49
*** trinaths has quit IRC15:49
*** tomoe_ has quit IRC15:49
*** tomoe_ has joined #openstack-neutron15:50
*** matrohon has quit IRC15:50
dougwigif anyone is interested in how we handle two lbaas command sets in the neutron CLI, please comment here: https://review.openstack.org/#/c/111475/15:50
*** leenheer has joined #openstack-neutron15:53
*** johnbelamaric1 has joined #openstack-neutron15:54
*** johnbelamaric has quit IRC15:54
*** trinaths1 has left #openstack-neutron15:54
*** trinaths1 has joined #openstack-neutron15:55
*** xgerman_ has joined #openstack-neutron15:56
mesterySukhdev: Done!15:57
Sukhdevmestery: thanks a ton15:57
mesterySukhdev: Thanks for your awesome work on thinning the Arista ML2 driver!15:58
Sukhdevmestery: the stackforge side is done as well - getting the CI done15:58
Sukhdevmestery: Neutron side CI is fully functional - Stackforge side CI is almost there15:59
*** mmdurrant has joined #openstack-neutron16:00
*** johnbelamaric1 has quit IRC16:00
*** johnbelamaric has joined #openstack-neutron16:00
*** johnbelamaric has quit IRC16:01
*** johnbelamaric has joined #openstack-neutron16:01
*** mohitsha_ has joined #openstack-neutron16:02
mesterySukhdev: So awesome!16:02
*** neeti has joined #openstack-neutron16:03
*** jgjl has quit IRC16:03
*** Marga_ has joined #openstack-neutron16:03
*** ronis has quit IRC16:04
*** mohitsharma has quit IRC16:05
*** diegows has joined #openstack-neutron16:05
*** abehl has quit IRC16:05
*** vivek-ebay has joined #openstack-neutron16:07
*** moshele has joined #openstack-neutron16:07
*** xuhanp has quit IRC16:09
*** manishg has joined #openstack-neutron16:09
*** lori is now known as lori|away16:10
*** anish_ has joined #openstack-neutron16:13
otherwiseguymestery: I posted the native OVSDB patch: https://review.openstack.org/#/c/152761/  I need to modify functional tests to also bang on it, but the biggest issue is that OVS's python module is not in PyPI. I seem to remember that it actually uses a value from the build/packaging process to determine a directory location that gets hardcoded into the library, but other than that it shouldn't be too hard to setuptools-ify16:13
otherwiseguyfor pypi.16:13
*** igordcard has quit IRC16:13
mesteryotherwiseguy: That's excellent! Thanks for posting that work!16:14
mesteryotherwiseguy: I'll add this to my review and testing queue, I want to try this out this week yet.16:14
otherwiseguyJust not exactly sure what the best way to go on packaging thing.16:14
mesteryotherwiseguy: I'm working some final Kilo-2 things today, so it's likely tomorrow before I'll get to it.16:14
*** ChuckC_ has joined #openstack-neutron16:14
mesteryotherwiseguy: Do the python bindings get installed as a package as part of the distro itself?16:15
mesteryFor OVS I mean16:15
*** nmagnezi has quit IRC16:15
otherwiseguymestery: yes, at least on RH systems it is just a sub package of the openvswitch package, python-openvswitch. So it has to be installed separately.16:15
*** carl_baldwin has joined #openstack-neutron16:16
mesteryotherwiseguy: As long as we can install something similar on Ubuntu and SuSe we're good then.16:16
*** abehl has joined #openstack-neutron16:16
*** armax has joined #openstack-neutron16:16
otherwiseguytox will die on tests unless people manually install it, though.16:17
*** ChuckC has quit IRC16:17
mesteryotherwiseguy: We'll have to figure that out then somehow. Maybe we push the upstream OVS guys to have the python bindings pushed to pypi?16:17
* mestery realizes he just came full circle with otherwiseguy here ... :)16:17
otherwiseguymestery: Yeah, and I'm thinking the response might be something like "patches welcome" :)16:17
ajomestery, jumping into mending my patch... time to count function complexity to make sure I'm not screwing it up... ;)16:18
otherwiseguyWhich would be totally fair. :)16:18
ajohi otherwiseguy  :)16:18
mesteryotherwiseguy: Heh, you're right.16:18
mesteryajo: Nice, thanks!16:18
*** EmilienM|afk is now known as EmilienM16:18
*** tomoe_ has quit IRC16:18
mesteryotherwiseguy: That's a pretty big thing I would guess.16:18
mesteryotherwiseguy: Lets have a plan to get there but go with something easier in the short term. Sound good?16:18
*** Networkn3rd has quit IRC16:19
*** Networkn3rd has joined #openstack-neutron16:19
openstackgerritTrinath Somanchi proposed openstack/neutron-fwaas: Code cleanup for Freescale FWaaS Plugin  https://review.openstack.org/15285616:19
*** tomoe_ has joined #openstack-neutron16:20
otherwiseguymestery: There is also some, uh, serious magic in my patch to basically patch the OVS library to add a cheap lookup by indexed columns to avoid every lookup of a bridge/port/interface by name being a linear scan of the whole table. Lots of subclassing dicts and making the OVS library use those "special dicts" without knowing it. Things much better done as patches to the OVS library itself and that I plan on16:21
otherwiseguy submitting.16:21
*** tiswanso has quit IRC16:22
otherwiseguySo I'll most likely be doing some work with them on that anyway. But deadlines, etc. I went ahead and did what I could to make it work.16:22
*** vivek-ebay has quit IRC16:24
otherwiseguymestery: and yeah, easier in the short term I can work with. :)16:26
otherwiseguyMy brain. It is on it's last legs. ;)16:26
otherwiseguyWhich I guess would be my legs.16:26
amullermestery: Please see https://bugs.launchpad.net/neutron/+bug/141809716:26
otherwiseguyand by "it's" I mean "its" which just proves my point.16:27
*** tiswanso has joined #openstack-neutron16:27
*** neeti has quit IRC16:27
*** coolsvap is now known as coolsvap_16:28
mesteryotherwiseguy: lol :)16:29
mesterythanks otherwiseguy16:29
mesteryamuller: Looking16:29
*** marun has joined #openstack-neutron16:30
mesteryamuller: Trying it now.16:30
amullerthanks16:30
*** SridharRamaswamy has joined #openstack-neutron16:30
ihrachyshkaamuller, re https://review.openstack.org/#/c/149818/16:31
ihrachyshkaamuller, "L3_HA is completely unusable in it's current state without this patch." do you have details?16:31
*** mohitsha_ has quit IRC16:32
mesteryamuller: I'm trying it now, let me get a VM up so I run it16:32
amullerihrachyshka: yeah, the bug report has more information16:33
*** SridharRamaswam1 has joined #openstack-neutron16:33
amullerbasically new floating IP assignments cause agent-side failures after agent restarts16:33
*** yatin has joined #openstack-neutron16:33
ihrachyshkaamuller, agent as in 'l3 agent'16:33
ihrachyshka?16:33
trinaths1mestery: Hi..16:34
amullerihrachyshka: yes16:34
trinaths1mestery: Good Morning Sir16:34
ihrachyshkaamuller, you mean, if failover occurred, then some failures was seen. what was those failures?16:34
*** SridharRamaswamy has quit IRC16:34
*** kkxue has quit IRC16:35
ihrachyshka"network flaps for all addresses" in LP seems vagye16:35
ihrachyshka*vague16:35
mesterytrinaths1: Hi, looking at something for amuller right now, FYI16:35
trinaths1mestery: okay. Want to put forward on Freescale CI - Voting rights16:36
*** ijw has quit IRC16:36
*** abehl has quit IRC16:36
*** redbeard has quit IRC16:37
amullerihrachyshka: Configure an HA router, configure some floating IPs. Now go restart the L3 agent running the master. Subsequent floating IP configurations will configue keepalived incorrectly16:37
*** rmar04 has quit IRC16:38
ihrachyshkaamuller, what's the implication? consequent fips are not working at all?16:38
*** rnm has quit IRC16:38
amullercorrect16:38
ihrachyshkasorry, I have vague understanding of ha :)16:38
amullerit can also mess up existing FIPs16:38
ihrachyshkahm, sounds serious. we need an exception for that.16:38
*** Marga_ has quit IRC16:39
*** nlahouti has joined #openstack-neutron16:39
*** rwsu-afk is now known as rwsu16:40
*** diegows has quit IRC16:41
*** nyechiel__ has quit IRC16:41
dougwigtrinaths1: isn't that CI fairly new?16:42
mesteryamuller: The functional tests seem to assume neutron-rootwrap is in /usr/local/bin, but on Fedora, that's not the case. Looks like OS_ROOTWRAP_CMD should override that, but it's not for me.16:42
mesteryamuller: Thus, having trouble running that functional test on my Fedora 20 system16:42
mesteryamuller: Any pointers?16:42
*** jorgem has joined #openstack-neutron16:42
amullermestery: I copied the rootwrap binary to /usr/local/bin16:43
*** banix_ has joined #openstack-neutron16:43
amullerwe really should do something about that =D16:43
dougwigmestery: pointer #1 - don't run fedora.16:43
mesteryamuller: Really? #sadpanda :(16:43
* dougwig hides.16:43
mesterydougwig: I'm Fedora to the core, so to speak.16:43
otherwiseguylols16:43
* mestery sends amuller ihrachyshka otherwiseguy after dougwig.16:43
amullerCommon Fedora is OK =D16:44
* otherwiseguy has no strong opinions16:44
trinaths1dougwig: nope. It history is here http://stackalytics.com/?module=neutron-group&company=freescale&metric=ci  in JUNO i have ML2 and FW jobs running for each neutron commit. Now I have splitted then to neutorn and neutron-fwaas16:44
* otherwiseguy used Slackware when it shipped on lots of floppies16:45
ihrachyshkaotherwiseguy, man, you're dying16:46
* otherwiseguy quickly checks his pulse16:46
ihrachyshkamestery, hm, don't we install neutron-rootwrap with setup.py for func tests? or do you mean setuptools put the file in different place just for fedora?16:47
*** thedodd has joined #openstack-neutron16:47
mesteryihrachyshka: The latter16:47
ihrachyshkaTHAT IS WEIRD. I guess it depends on some system level envvars. So are we hardcoding the path somewhere? Maybe that's the problem.16:48
dougwigwhat's our review guidance on code in __init__.py these days?  specifically, some oslo.config setup stuff?16:48
ihrachyshkamestery, can't we just remove /usr/local/bin from tox.ini and allow sudo to find it?16:49
mesteryihrachyshka: ++ to that. The functional tests seem to assume it's in /usr/local/bin, though there is an environment variable which claims to override it in the gate.16:49
mesteryihrachyshka: The override didn't work for me when running the functional tets locally though16:49
*** shashankhegde has joined #openstack-neutron16:50
*** doude has quit IRC16:50
mesteryThe functional test which amuller references in his bug is in fact causing me all sorts of issues not even related to the issue in the bug :(16:50
dougwigtrinaths1: heads up, this looks like a CI setup failure: https://review.openstack.org/#/c/151541/1416:51
*** jprovazn has joined #openstack-neutron16:51
*** wanjing has quit IRC16:51
mesteryamuller: I can't even get that functional test to run, see my comment in the bug16:51
openstackgerritIhar Hrachyshka proposed openstack/neutron: Allow sudo to determine neutron-rootwrap location  https://review.openstack.org/15297816:52
ihrachyshkamestery, let's see whether it just works (tm) ^^16:52
*** belmoreira has quit IRC16:52
rm_workmestery: psst, does this seem sane to you for plugin loading? https://review.openstack.org/#/c/146210/7/neutron_lbaas/common/cert_manager/__init__.py16:52
*** jorgem has quit IRC16:54
*** jorgem has joined #openstack-neutron16:54
*** jobewan has joined #openstack-neutron16:55
*** redbeard has joined #openstack-neutron16:55
trinaths1dougwig: looking16:55
ihrachyshkacarl_baldwin, heyo16:55
ihrachyshkacarl_baldwin, I wonder when's the good time for updating https://review.openstack.org/#/q/status:open+project:openstack/neutron+branch:master+topic:split_ra_unit_tests,n,z16:55
carl_baldwinihrachyshka: hi16:55
carl_baldwinihrachyshka: I’ve got a few more refactoring patches to generate which will take another week at least to produce.  They will churn that code quite a bit.16:56
*** diegows has joined #openstack-neutron16:57
ihrachyshkacarl_baldwin, ack, thanks, keep me in touch :)16:58
mesteryihrachyshka: Nice, thanks!16:58
*** annegent_ has joined #openstack-neutron16:58
carl_baldwinihrachyshka: Thanks for checking in.16:58
trinaths1dougwig: unable to open the review link16:59
* ihrachyshka notes that he will need mestery's and marun's attention tomorrow for amazing 'change a thousand of oslo imports in one go throughout four repos' quest16:59
mesteryihrachyshka: I'm onboard with that if marun is. :)16:59
*** lazy_prince is now known as killer_prince17:00
*** banix_ has quit IRC17:01
*** aranjan has joined #openstack-neutron17:01
*** moshele has quit IRC17:02
*** shivharis has quit IRC17:02
otherwiseguydougwig: who do I pester for a 2nd +2 on https://review.openstack.org/#/c/152579/ ? :) (and also https://review.openstack.org/#/c/152578/)17:02
otherwiseguyPlan on updating the remove-root_helper patch today, and the tests will fail unless I get those 2 patches in.17:02
*** jlanoux has quit IRC17:03
dougwigi can look at the second.  ping blogan for the first.17:03
otherwiseguydougwig: thanks!17:03
*** s3wong has joined #openstack-neutron17:04
*** jorgem has quit IRC17:05
*** ihrachyshka has quit IRC17:05
otherwiseguyblogan_: Any chance you could spare a minute to check out https://review.openstack.org/#/c/152579/ sometime? Simple patch. Just converting passing root_helper as kwarg so root_helper removal doesn't break the world. :)17:05
mesteryI got the first otherwiseguy.17:05
otherwiseguyah, cool. thanks.17:05
amullermestery: I think that's the issue I'm seeing. Let me guess: If you run ps aux | grep ovs-vswitchd you'll see that it's dead17:05
amullermestery: but service openvswitch status shows that it's running17:05
*** chenleji has quit IRC17:05
*** yfried__ has quit IRC17:05
mesteryamuller: Let me check, I rebooted the node but I'll know soon enough.17:05
blogan_ah thanks mestery17:06
amullermestery: Also if you could, rebase back to 1 patch before the patch I linked in the bug17:06
amullerIt probably won't reproduce17:06
amullerthen you rebase 1 patch further17:06
amullerand suddenly it happens17:06
otherwiseguydougwig: mestery: blogan_ : thanks. That was fast. Now I guess I have to get to work. :p17:06
dougwigmush!17:06
mesteryamuller: Ack, will give it a try.17:06
blogan_lol17:06
mesteryotherwiseguy: lol17:06
marunmestery: I'm on board17:07
*** absubram has joined #openstack-neutron17:07
mesterymarun: Cool!17:07
amullermarun: carl_baldwin: https://review.openstack.org/#/c/150154/ is ready to merge17:07
*** lori|away is now known as lori17:07
amullerpassing all CI, rebased17:07
*** scheuran has quit IRC17:07
marunamuller: sez you! ;)17:07
amullerExactly! :)17:07
carl_baldwinamuller: I already have it up.  Reviewing it now.17:08
carl_baldwinamuller: Thanks for the ping.17:08
amullercarl_baldwin: Thank you for the review17:08
otherwiseguyoh good, rebasing the root_helper patch only has conflicts in 13 files.17:09
openstackgerritCarl Baldwin proposed openstack/neutron: Introduce External IPAM Interface  https://review.openstack.org/13433917:09
dougwigoh my, please don't merge that until friday.  :)17:09
dougwig+a i mean, gerrit is fine.17:10
carl_baldwinotherwiseguy: This is the first rebase since the work was shelved a while back?  I think you’re lucky if that is the case!17:10
amullerHuh appearently the Patriots parade is going on half a mile from here17:10
*** akamyshnikova has quit IRC17:10
*** manishg has quit IRC17:10
amullermight need to go check that out =D17:10
*** ygbo has quit IRC17:11
*** jobewan has quit IRC17:11
*** manishg has joined #openstack-neutron17:11
otherwiseguycarl_baldwin: it probably also means I should have broken it up into more patches. Like some to fix the unit tests to not reference root_helper everywhere, etc.17:11
otherwiseguyhelper functions FTW.17:11
*** leenheer has quit IRC17:11
otherwiseguyoh well, live and learn.17:11
*** Murali has joined #openstack-neutron17:13
mesteryamuller: Nice17:13
*** manishg_ has joined #openstack-neutron17:13
*** vishwanathj has joined #openstack-neutron17:15
*** manishg has quit IRC17:15
*** jorgem has joined #openstack-neutron17:15
*** mlavalle has joined #openstack-neutron17:17
*** pask81 has quit IRC17:19
*** safchain has quit IRC17:19
*** spandhe has joined #openstack-neutron17:20
*** signed8bit is now known as signed8bit_ZZZzz17:20
*** matrohon has joined #openstack-neutron17:20
*** Xurong has quit IRC17:21
*** Marga_ has joined #openstack-neutron17:21
*** Xurong has joined #openstack-neutron17:21
*** ijw has joined #openstack-neutron17:23
*** yatin has quit IRC17:23
*** jobewan has joined #openstack-neutron17:24
*** jistr has quit IRC17:25
*** ronis has joined #openstack-neutron17:26
*** Sukhdev has quit IRC17:30
*** jpich has quit IRC17:31
carl_baldwinamuller: ping17:32
*** devvesa has quit IRC17:34
*** shashankhegde has quit IRC17:34
*** wanjing has joined #openstack-neutron17:34
*** prad has quit IRC17:36
*** prad has joined #openstack-neutron17:36
*** bjornar_ has joined #openstack-neutron17:37
amullercarl_baldwin: aye?17:37
carl_baldwinamuller: Just posted a review.17:37
*** spandhe has quit IRC17:38
amullercarl_baldwin: Cool I'll look at it later, thanks17:38
*** SridharRamaswam1 has quit IRC17:38
*** SridharRamaswamy has joined #openstack-neutron17:38
*** trinaths1 has left #openstack-neutron17:38
carl_baldwinamuller: Okay.  Let me know if you want to discuss17:38
carl_baldwinamuller: I should be around all day.17:38
amullerok I'll be on later, gotta go to the new place now to install Comcast17:39
*** banix has joined #openstack-neutron17:39
*** yamamoto has quit IRC17:39
*** johnbelamaric has quit IRC17:39
*** johnbelamaric has joined #openstack-neutron17:39
*** s3wong has quit IRC17:40
*** Murali has quit IRC17:42
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Refactor _remove_unused_security_group_info  https://review.openstack.org/15251217:44
ajo^ mestery, carl_baldwin , haleyb , rkukura17:44
carl_baldwinajo: ack17:45
*** sthillma has joined #openstack-neutron17:45
*** sthillma has quit IRC17:45
ajocarl_baldwin, analyzing the complexity while looking on the extra loops, I believe that the sets are actually going to make it faster17:46
ajogod bless the sets ;D17:46
carl_baldwinajo: I wasn’t questioning the complexity.  I hope that was clear.17:47
*** anish__ has joined #openstack-neutron17:47
ajobecause we have a few O(N)*O(M)  parts, not really important actually17:47
ajowe have17:47
ajowe had17:47
ajonow we don't17:47
ajocarl_baldwin, yes, I saw, but I wanted to re-check, for just in case :)17:47
ajonow, carl_baldwin , I have a doubt,17:47
ajois this code well covered by tests?,17:48
*** tomoe_ has quit IRC17:48
ajoI'm sure that we check from a higher level, that ipset destroys are done when we remove filters, or security groups, etc...17:48
ajothat is safe17:48
ajobut I'm not sure if we're checking the internal structures to make sure they are actually cleaned up..17:49
ajomay be I should prepend extended unit tests for this function, and then, this patch...17:49
ajosorry, I'm a bit verbose17:49
*** doude has joined #openstack-neutron17:49
*** shashankhegde has joined #openstack-neutron17:50
ajoI need to AFK until later (now+4h or so..)17:50
*** anish_ has quit IRC17:50
*** johnbelamaric has left #openstack-neutron17:50
*** johnbelamaric has joined #openstack-neutron17:51
*** zzzeek has joined #openstack-neutron17:51
*** _cjones_ has joined #openstack-neutron17:51
*** annegent_ has quit IRC17:53
*** amuller_ has joined #openstack-neutron17:55
*** killer_prince is now known as lazy_prince17:55
*** tflynn has joined #openstack-neutron17:56
*** spandhe has joined #openstack-neutron17:57
*** coolsvap_ is now known as coolsvap17:57
*** tflynn has quit IRC17:57
*** tflynn has joined #openstack-neutron17:58
*** amuller has quit IRC17:58
*** moshele has joined #openstack-neutron18:00
*** amuller_ has quit IRC18:00
*** SumitNaiksatam has joined #openstack-neutron18:01
*** doude has quit IRC18:01
*** doude has joined #openstack-neutron18:01
*** lazy_prince is now known as killer_prince18:02
openstackgerritMerged openstack/python-neutronclient: Skip None id when getting security_group_ids  https://review.openstack.org/15072818:03
*** manishg has joined #openstack-neutron18:04
*** manishg_ has quit IRC18:04
*** aix has quit IRC18:05
*** shashankhegde has quit IRC18:06
*** karimb has quit IRC18:06
*** doude has quit IRC18:07
*** doude has joined #openstack-neutron18:08
*** rushil has joined #openstack-neutron18:09
openstackgerritMerged openstack/python-neutronclient: Fix TypeError for six.text_type  https://review.openstack.org/14568218:09
openstackgerritMerged openstack/python-neutronclient: Fix columns setup base on csv formatter  https://review.openstack.org/14055618:09
carl_baldwinajo: I can’t vouch for the unit test coverage of that code.18:09
*** leenheer has joined #openstack-neutron18:10
*** harlowja has joined #openstack-neutron18:11
*** jasondotstar has joined #openstack-neutron18:11
*** luqas has quit IRC18:11
*** moshele has quit IRC18:13
*** yamahata has quit IRC18:14
*** jasondotstar has quit IRC18:14
*** thedodd has quit IRC18:14
*** signed8bit_ZZZzz is now known as signed8bit18:17
openstackgerritMerged openstack/python-neutronclient: Parameter support both id and name  https://review.openstack.org/7857718:18
*** SridharRamaswam1 has joined #openstack-neutron18:19
*** SridharRamaswamy has quit IRC18:19
openstackgerritJuergen Brendel proposed openstack/neutron: ARP spoofing patch: Low level ebtables integration  https://review.openstack.org/14113018:20
*** coolsvap is now known as coolsvap_18:20
*** shashankhegde has joined #openstack-neutron18:29
openstackgerritIhar Hrachyshka proposed openstack/neutron: execute: enforce C locale unless explicitly requested  https://review.openstack.org/14995118:30
*** kawa2014 has quit IRC18:30
*** Marga_ has quit IRC18:30
*** Marga_ has joined #openstack-neutron18:31
*** doude has quit IRC18:31
*** achanda has joined #openstack-neutron18:31
*** oreillyd has quit IRC18:32
*** humbolt11 has joined #openstack-neutron18:32
*** humbolt11 has quit IRC18:32
*** amotoki has quit IRC18:32
*** signed8bit is now known as signed8bit_ZZZzz18:33
*** doude has joined #openstack-neutron18:33
*** humbolt11 has joined #openstack-neutron18:34
*** humbolt11 has quit IRC18:34
*** vivek-ebay has joined #openstack-neutron18:35
*** signed8bit_ZZZzz is now known as signed8bit18:35
*** humbolt1 has quit IRC18:35
*** yamahata has joined #openstack-neutron18:37
*** yfried__ has joined #openstack-neutron18:42
*** rpothier has joined #openstack-neutron18:45
*** ihrachyshka has joined #openstack-neutron18:46
*** leenheer has quit IRC18:47
*** padkrish has joined #openstack-neutron18:50
*** afazekas has quit IRC18:51
*** humbolt1 has joined #openstack-neutron18:51
*** humbolt1 has quit IRC18:51
*** annegent_ has joined #openstack-neutron18:54
*** jckasper has quit IRC18:54
*** Marga_ has quit IRC18:58
*** Marga_ has joined #openstack-neutron18:58
*** annegent_ has quit IRC18:59
*** annegent_ has joined #openstack-neutron19:00
openstackgerritDoug Wiegley proposed openstack/neutron: Nuke a useless lambda wrapper (review feedback)  https://review.openstack.org/15301519:01
ihrachyshkarussellb, hey. could you please check https://review.openstack.org/#/c/136106/16/neutron/agent/rpc.py ? I'm not completely sure my reasoning is correct, so please advise.19:02
openstackgerritDoug Wiegley proposed openstack/neutron: Nuke a useless lambda wrapper and call to iterkeys (review feedback)  https://review.openstack.org/15301519:04
*** rushil has quit IRC19:04
*** thedodd has joined #openstack-neutron19:05
*** signed8bit is now known as signed8bit_ZZZzz19:08
*** ivar-lazzaro has joined #openstack-neutron19:09
*** humbolt1 has joined #openstack-neutron19:10
*** humbolt1 has quit IRC19:10
*** jp_at_hp has quit IRC19:11
*** ivar-lazzaro has quit IRC19:11
*** signed8bit_ZZZzz is now known as signed8bit19:11
*** anish__ has quit IRC19:11
*** ivar-lazzaro has joined #openstack-neutron19:12
*** humbolt1 has joined #openstack-neutron19:12
*** humbolt1 has quit IRC19:12
*** gaud has joined #openstack-neutron19:13
*** humbolt1 has joined #openstack-neutron19:13
*** humbolt1 has quit IRC19:13
*** dtroyer has joined #openstack-neutron19:14
*** padkrish has quit IRC19:16
*** humbolt1 has joined #openstack-neutron19:17
*** padkrish has joined #openstack-neutron19:17
*** humbolt1 has quit IRC19:17
*** padkrish_ has joined #openstack-neutron19:17
dtroyerSo I really want to be able to leave Neutron as the default in DevStack, but can't find the magic incantation to make it work with linuxbridge.  I'm using ml2 as I understand that is the future (can we remove linuxbridge-agent from devstack if so?)19:18
dtroyerThe constraints are to be as close to nova-net's flat dhcp config with one interface as possible.19:18
*** humbolt1 has joined #openstack-neutron19:18
*** humbolt1 has quit IRC19:18
dougwigwhy lb instead of ova?19:19
dougwigovs?19:20
*** devlaps has joined #openstack-neutron19:20
dtroyersimple, and it all comes 'in the box'19:20
*** humbolt1 has joined #openstack-neutron19:20
*** humbolt1 has quit IRC19:20
dtroyerfor devs that need a working stack and don't want the overhead of the rest of stuff.19:21
*** padkrish has quit IRC19:21
mesteryShow me a Linux distrubuition which doesn't package OVS today dtroyer. It's there for all of them. :)19:21
mesteryBut I digress :)19:21
mesteryOVS has been in kernel for 2 years now19:22
mesteryAnd pacakged in distros for longer than that19:22
dtroyermestery: yes, that is true today…19:22
dtroyerI don't want the overhead19:22
mesteryWhat overhead? An extra agent and DB process on the host?19:22
dtroyersimple singe interface setup that matches nova-net flat dhcp19:22
mesteryGot it19:22
*** humbolt1 has joined #openstack-neutron19:22
*** humbolt1 has quit IRC19:22
*** jobewan has quit IRC19:23
dougwigthe requirement of a single interface setup that matches flat dhcp would seem to be orthogonal to lb vs ova, and i'm not sure how many people use lb anymore. i never have, and have never noticed the overhead.19:24
dougwigovs.  dya!19:24
*** devlaps has quit IRC19:24
*** humbolt1 has joined #openstack-neutron19:25
*** humbolt1 has quit IRC19:25
dtroyerthat may be, I don't know neutron well enough (obviously), but from the start this is what we have asked for.19:25
dtroyerif nobody uses linuxbridge why is it still in the codebase?  why is it still in devstack?19:26
dougwigi'd also note that the official install guides use ovs.  i think the default devstack should match as close as possible to the "recommended" install as possible, and i'm not sure that lb fits that.19:26
dtroyerthat's a point, is the official guide also suitable for a typical development install from source? (I haven't read, I really don't know)19:29
dtroyersingle node, single interface?19:29
*** achanda has quit IRC19:35
dougwigit has a multi-node multi-nic sample architecture, but if you install everything on the same host and use the same network for management/tunnels/storage, it does work fine.19:35
*** leenheer has joined #openstack-neutron19:37
*** vishwanathj has quit IRC19:37
*** jwy has joined #openstack-neutron19:38
*** Marga_ has quit IRC19:38
*** Marga_ has joined #openstack-neutron19:39
*** jobewan has joined #openstack-neutron19:41
*** diegows has quit IRC19:41
*** doude has quit IRC19:41
*** jckasper has joined #openstack-neutron19:42
*** humbolt1 has joined #openstack-neutron19:43
*** humbolt1 has quit IRC19:43
*** humbolt1 has joined #openstack-neutron19:44
*** humbolt1 has quit IRC19:44
*** achanda has joined #openstack-neutron19:45
*** humbolt1 has joined #openstack-neutron19:45
*** humbolt1 has quit IRC19:45
*** carl_baldwin has quit IRC19:49
*** harlowja is now known as harlowja_away19:50
*** padkrish_ has quit IRC19:50
*** padkrish has joined #openstack-neutron19:50
*** carl_baldwin has joined #openstack-neutron19:52
*** jprovazn has quit IRC19:52
*** padkrish_ has joined #openstack-neutron19:53
*** EmilienM is now known as EmilienM|afk19:53
*** redbeard has quit IRC19:53
*** signed8bit is now known as signed8bit_ZZZzz19:54
*** ZZelle_ has joined #openstack-neutron19:54
*** padkrish has quit IRC19:55
*** signed8bit_ZZZzz is now known as signed8bit19:56
*** nlahouti has quit IRC19:57
*** padkrish_ has quit IRC19:57
*** padkrish has joined #openstack-neutron19:58
*** annegent_ has quit IRC19:58
*** diegows has joined #openstack-neutron19:59
dtroyerI don't have a week to sort this out, but what I hear is "don't do that" when my question is "how do I do that?"  If it is not possible than I'll start submitting deprecation reviews.20:00
*** tsufiev_ is now known as tsufiev20:02
*** padkrish_ has joined #openstack-neutron20:02
*** nlahouti has joined #openstack-neutron20:02
*** padkrish has quit IRC20:02
openstackgerritQin Zhao proposed openstack/neutron: Add InvalidIpForNetwork and InvalidIpForSubnet exception  https://review.openstack.org/12176720:02
*** annegent_ has joined #openstack-neutron20:04
openstackgerritCarl Baldwin proposed openstack/neutron: Move Floating IP processing to Router classes  https://review.openstack.org/14286320:04
openstackgerritCarl Baldwin proposed openstack/neutron: Use weak ref to avoid deleting fip namespace through agent  https://review.openstack.org/15234220:04
openstackgerritCarl Baldwin proposed openstack/neutron: Move DVR floating ip methods to dvr_router  https://review.openstack.org/15062820:04
*** prad has quit IRC20:06
*** thedodd has quit IRC20:07
openstackgerritCarl Baldwin proposed openstack/neutron: Move Floating IP processing to Router classes  https://review.openstack.org/14286320:08
openstackgerritCarl Baldwin proposed openstack/neutron: Use weak ref to avoid deleting fip namespace through agent  https://review.openstack.org/15234220:08
openstackgerritCarl Baldwin proposed openstack/neutron: Move DVR floating ip methods to dvr_router  https://review.openstack.org/15062820:08
*** yfried__ has quit IRC20:10
*** yfried__ has joined #openstack-neutron20:10
*** prad has joined #openstack-neutron20:10
dougwigdtroyer: my answer was, i've never done that, i don't think it's standard, and i'm not sure who the right person to point you at it is. i'm far from omniscient.20:12
*** wanjing has quit IRC20:13
dtroyerdougwig: sure, thanks.   I didn't think this would be a config that was that far off the mainstream.  It is what we've talked about for years though, even mentioned again last week.20:14
*** wanjing has joined #openstack-neutron20:14
*** aranjan has quit IRC20:17
*** wanjing has quit IRC20:19
*** achanda has quit IRC20:20
*** achanda has joined #openstack-neutron20:21
*** shashankhegde has quit IRC20:21
*** aranjan has joined #openstack-neutron20:21
*** thedodd has joined #openstack-neutron20:21
openstackgerritMerged openstack/neutron-vpnaas: Updated from global requirements  https://review.openstack.org/15271920:22
openstackgerritMerged openstack/neutron-fwaas: Updated from global requirements  https://review.openstack.org/15271820:22
*** shashankhegde has joined #openstack-neutron20:24
*** vishwanathj has joined #openstack-neutron20:24
*** carl_baldwin has quit IRC20:25
*** aranjan has quit IRC20:26
*** rushil has joined #openstack-neutron20:28
*** achanda has quit IRC20:32
*** ihrachyshka has quit IRC20:32
*** harlowja_away is now known as harlowja20:33
*** 6A4ABBR4J has quit IRC20:34
openstackgerritMerged openstack/python-neutronclient: Utility method for boolean argument  https://review.openstack.org/15052620:37
*** raildo has quit IRC20:39
openstackgerritMichael Johnson proposed openstack/python-neutronclient: Implement LBaaS object model v2  https://review.openstack.org/11147520:39
*** achanda has joined #openstack-neutron20:40
openstackgerritJuergen Brendel proposed openstack/neutron: ARP spoofing patch: Low level ebtables integration  https://review.openstack.org/14113020:40
*** raildo has joined #openstack-neutron20:41
*** baoli has quit IRC20:42
*** matrohon has quit IRC20:48
*** tomoe_ has joined #openstack-neutron20:49
*** tomoe_ has quit IRC20:53
*** jpena has quit IRC20:53
*** gaud has quit IRC20:58
*** nlahouti has quit IRC21:02
*** padkrish_ has quit IRC21:03
*** padkrish has joined #openstack-neutron21:03
*** aranjan has joined #openstack-neutron21:05
openstackgerritDoug Wiegley proposed openstack/neutron: Flavor Framework implementation  https://review.openstack.org/13975821:06
*** padkrish has quit IRC21:08
*** padkrish has joined #openstack-neutron21:08
*** nlahouti has joined #openstack-neutron21:08
*** ajo has quit IRC21:09
*** annegent_ has quit IRC21:10
*** ajo has joined #openstack-neutron21:10
*** haramaki has quit IRC21:10
*** haramaki has joined #openstack-neutron21:11
*** claudiub has quit IRC21:12
*** bjornar_ has quit IRC21:14
*** wanjing has joined #openstack-neutron21:15
*** haramaki has quit IRC21:16
*** ChuckC_ is now known as ChuckC21:16
*** aranjan has quit IRC21:18
*** thedodd has quit IRC21:18
*** karimb has joined #openstack-neutron21:19
*** aranjan has joined #openstack-neutron21:19
*** Zemeio has quit IRC21:20
*** oreillyd has joined #openstack-neutron21:21
openstackgerritMerged openstack/neutron: Fix flake exclude matching of .*  https://review.openstack.org/15276421:23
*** aranjan has quit IRC21:24
ajomarun, ping21:24
ajohttps://bugs.launchpad.net/oslo.rootwrap/+bug/141733121:24
marunajo: hi!21:24
ajoI talked to ttx about the bug, and he had some questions, and a reference to other bug I believe we also hit in functional testing21:25
marunI have a workaround that could work21:25
ajoaha, nice, he told, he's not opposed to change that in rootwrap,21:25
*** oreillyd has quit IRC21:25
marunI'm not sure if it would be preferable to modifying secure_path21:25
ajobut that if we had a workaround, that would be better to avoid complicating the rootwrap codebase21:25
*** Networkn3rd has quit IRC21:25
ajomarun, I suppose an important case is letting the users test without installing all the neutron binaries in the system, I guess21:26
ajomarun, what's the workaround?21:26
marunajo: OS_ROOTWRAP_CMD=sudo "PATH={env:PATH}:{envbindir}" {envbindir}/neutron-rootwrap {envdir}/etc/neutron/rootwrap.conf21:26
marunajo: set in tox.ini21:26
ajoahh21:26
ajoneat21:27
marunajo: basically, I'm overriding the sudo path to include the env's bin dir21:27
marunajo: The downside is that it overrides with the non-sudo path21:27
marunajo: I don't see a way to simply add the bin dir21:27
ajooverrides the non-sudo path?21:27
marunajo: rather than adding the bin dir to the sudo PATH21:28
marunajo: it sets the sudo PATH to the non-sudo PATH + bin dir21:28
ajoahh21:28
ajook21:28
ajooverrides the sudo path in favor of the non-sudo `bindir ?21:28
marunajo: so long as we're willing to accept that, the fix is working21:28
ajoI suppose it should be acceptable for testing purposes21:29
marunajo: the bin dir would be same in both cases21:29
*** imcsk8 has quit IRC21:29
ajois not a production system... so security is not so important here21:29
*** imcsk8 has joined #openstack-neutron21:29
marunajo: fair enough21:29
openstackgerritMichael Johnson proposed openstack/python-neutronclient: Implement LBaaS object model v2  https://review.openstack.org/11147521:29
marunajo: workaround it is.  I'll remove the comment that says 'remove when bug is fixed' :)21:29
marunajo: thanks for looking into this!21:29
ajomarun: no problem, I was happy trying to help,  let's keep the link to the bug & the review for just in case we need to retake that path :)21:30
marunajo: sounds good :)21:30
*** packet has quit IRC21:30
*** achanda has quit IRC21:33
*** Marga_ has quit IRC21:33
*** Marga_ has joined #openstack-neutron21:34
pc_mmarun: ping21:35
*** thedodd has joined #openstack-neutron21:36
marunpc_m: pong21:37
pc_mmarun: As we talked before about sharing the generate_testr_results hook function, I pushed this out for review https://review.openstack.org/#/c/153053/. Can you take a look at it and let me know what you think?21:37
marunarmax: ^^21:37
marunpc_m: armax was actually the author of this, I'm hoping he'll be able to take a look.21:38
*** baoli has joined #openstack-neutron21:38
marunpc_m: I don't think the functional test prep belongs there, though.21:38
*** achanda has joined #openstack-neutron21:38
roaetdougwig: ping, wrt #148318, is it really Copyright Rackspace? That feels super weird to me21:38
marunpc_m: that's actually part of the api job.21:38
*** tiswanso has quit IRC21:38
pc_mmarun: I can add him to the review.21:38
roaetgeneral audience: when I make a file for general openstack is it still copyright <my company>?21:39
*** tiswanso has joined #openstack-neutron21:39
pc_mmarun: the devstack-gate area I put this in?21:39
*** phil_h has quit IRC21:39
marunpc_m: I think so, but I'm not honestly sure.21:39
dougwigroaet: check out the argument at the top of this file, and the quotes from infra: https://review.openstack.org/#/c/151375/6/neutron/common/repos.py,unified21:40
marunpc_m: Similar functionality existed for unit testing, for example, but I'm not sure exactly where it was defined.21:40
pc_mmarun: Wasn't really sure where to place it.21:40
marunpc_m: armax had to recreate it for us, so presumably it doesn't exist in devstack-gate.21:40
pc_mmarun: We had talked about moving to the devstack-gate repo.21:41
*** ronis has quit IRC21:41
roaetdougwig: jeeze. TIL..21:41
dougwigroaet: hehe.21:41
roaetdougwig: thanks for the insight. that's all very silly21:41
dougwigroaet: usually is when lawyers get involved.21:41
roaettbh the 'copyright' makes no sense. I'm sure it should be like when we sign the CLA we can write code for the foundation21:41
roaetif someone from another company modifies the file, what does that mean? or the foundation? I really have no idea what all the implications of this single line could mean21:42
pc_mmarun: that area is where functions.sh is located, which is overriden for neutron.21:42
pc_mmarun: In any case, I'm new to this, so any guidance as to where this script should be located, would be great.21:43
marunpc_m: it may be that it belongs in functions.sh, we'll have to see what the core reviewers have to say.21:43
roaetdougwig: our entire quark plugin is (C) foundation21:43
pc_mmarun: Yeah, I was thinking about that, although the functions there seemed to be for different purposes, so I winged it and picked a new file.21:44
pc_mmarun: I'll ask over in infra and see what they say too.21:45
marunpc_m: I think it's fine in a new file, just remove the superfluous functions and add a license to the file.21:45
*** julim has quit IRC21:45
marunpc_m: thanks for doing this!21:45
*** EmilienM|afk is now known as EmilienM21:45
pc_mmarun: Sure. NP.21:46
pc_mmarun: BTW, I did the wiki doc for the procedures I used. Sent an email on ML to provide link.21:46
marunpc_m: cool, I'll take a look.21:47
*** belmoreira has joined #openstack-neutron21:48
pc_mmarun: https://wiki.openstack.org/wiki/Neutron/FunctionalGateSetup21:51
marunpc_m: thanks!21:51
pc_mmarun: yw21:52
*** phil_h has joined #openstack-neutron21:52
*** carl_baldwin has joined #openstack-neutron21:52
*** shashankhegde has quit IRC21:54
*** jwy has left #openstack-neutron21:55
*** bradjones has quit IRC21:57
*** carl_baldwin has quit IRC21:59
*** shashankhegde has joined #openstack-neutron21:59
*** ijw_ has joined #openstack-neutron22:01
*** bradjones has joined #openstack-neutron22:01
*** karimb has quit IRC22:02
*** annegent_ has joined #openstack-neutron22:04
*** ijw has quit IRC22:04
*** s3wong has joined #openstack-neutron22:04
openstackgerritDane LeBlanc proposed openstack/neutron: Support Multiple IPv6 Prefixes and Addresses for IPv6 Network  https://review.openstack.org/14906822:05
*** annegent_ has quit IRC22:05
*** vivek-ebay has quit IRC22:05
*** vivek-ebay has joined #openstack-neutron22:05
*** achanda has quit IRC22:08
*** claudiub has joined #openstack-neutron22:08
*** nyechiel__ has joined #openstack-neutron22:09
*** aranjan has joined #openstack-neutron22:10
*** tiswanso has quit IRC22:11
*** padkrish has quit IRC22:12
*** padkrish has joined #openstack-neutron22:13
*** xgerman_ has quit IRC22:13
*** tiswanso has joined #openstack-neutron22:13
*** phil_h has quit IRC22:14
*** padkrish_ has joined #openstack-neutron22:15
*** kevinbenton_ is now known as kevinbenton22:15
*** padkrish has quit IRC22:17
*** gildub has joined #openstack-neutron22:18
*** Sukhdev has joined #openstack-neutron22:18
openstackgerritTerry Wilson proposed openstack/neutron: Stop using passed root_helper in ip_lib  https://review.openstack.org/14582922:20
openstackgerritTerry Wilson proposed openstack/neutron: Remove deprecated DEFAULT.root_helper  https://review.openstack.org/15306622:20
openstackgerritTerry Wilson proposed openstack/neutron: Use processutils.execute for process execution  https://review.openstack.org/15306722:20
*** thedodd has quit IRC22:21
*** nyechiel__ has quit IRC22:22
otherwiseguymestery: HenryG: ^ I broke the ip_lib root_helper removal into more patches, left the option in to minimize it's code/test impact and allow tests of *aaS to keep passing. *Much* easier to review this time.22:24
otherwiseguys/it's/its/ AGAIN22:24
mesteryotherwiseguy: Thanks!22:25
otherwiseguySo basically ip_lib doesn't actually *use* the root_helper passed in, but it's there to make everything else happy. Hopefully every phase this way can minimize the number of test changes so we're sure things keep working. :p22:26
otherwiseguyI'm going to give up on using the correct "its".22:26
otherwiseguyIt's (I did it!) just not happening today.22:27
*** tiswanso has quit IRC22:27
*** tellesnobrega_ has joined #openstack-neutron22:27
openstackgerritBilly Olsen proposed openstack/neutron: Refactor logging in loop to only log debug messages once  https://review.openstack.org/14978422:28
*** Xurong has quit IRC22:28
openstackgerritJuergen Brendel proposed openstack/neutron: ARP spoofing patch: Low level ebtables integration  https://review.openstack.org/14113022:28
*** phil_h has joined #openstack-neutron22:29
openstackgerritMerged openstack/neutron: Fix extra-dhcp-opt on stateless dhcpv6 subnet  https://review.openstack.org/14745522:29
*** thedodd has joined #openstack-neutron22:29
*** sripriya has joined #openstack-neutron22:32
*** padkrish_ has quit IRC22:33
*** padkrish has joined #openstack-neutron22:33
*** Xurong has joined #openstack-neutron22:34
*** Xurong has quit IRC22:34
*** s3wong has quit IRC22:36
*** padkrish has quit IRC22:38
*** manishg has quit IRC22:39
*** manishg has joined #openstack-neutron22:39
*** jobewan has quit IRC22:40
*** johnbelamaric has quit IRC22:43
openstackgerritMerged openstack/neutron: Nuke remaining service config and extensions from main repo  https://review.openstack.org/15154122:46
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Refactor _remove_unused_security_group_info  https://review.openstack.org/15251222:47
openstackgerritMiguel Angel Ajo proposed openstack/neutron: Refactor _remove_unused_security_group_info  https://review.openstack.org/15251222:49
ajosalv-orlando,  ^  check the comments from previous patchset, thanks for reviewing it22:50
ajoI only did not do the .keys() removal,22:50
ajoas we're manipulating the dictionary as we iterate22:50
ajotime to sleep22:51
ajogood afternoon/night/* everybody22:51
*** jobewan has joined #openstack-neutron22:54
openstackgerritMaru Newby proposed openstack/neutron: Automate host configuration for functional testing  https://review.openstack.org/14611322:54
ajomarun  ^  what about adding an option to run_tests.sh to make it super-easy to the untrained functional tester?22:55
ajo./run_tests.sh -f  ?22:55
openstackgerritMerged openstack/neutron-fwaas: Code cleanup for Freescale FWaaS Plugin  https://review.openstack.org/15285622:55
marunajo: the fact that it butches the system with devstack makes me question the wisdom of that.22:56
marunajo: but you're welcome to submit a patch22:56
marunbutches -> butchers22:56
ajomarun, sure, I will look at it when my neurons come back to proper status (now they are in sleepy mode) :)22:56
ajoI will re-read your patch to make sure I didn't lost the context22:57
marunajo: it's changed quite a bit since the last revision.22:57
*** dims_ has quit IRC22:57
*** jorgem has quit IRC23:00
*** ajo has quit IRC23:00
*** jaypipes has quit IRC23:00
openstackgerritMaru Newby proposed openstack/neutron: Automate host configuration for functional testing  https://review.openstack.org/14611323:01
*** aepifanov has quit IRC23:02
*** kbringard has quit IRC23:03
*** vivek-ebay has quit IRC23:04
*** pc_m has quit IRC23:04
*** signed8bit has quit IRC23:07
*** vivek-ebay has joined #openstack-neutron23:07
*** achanda has joined #openstack-neutron23:08
openstackgerritRajeev Grover proposed openstack/neutron: Log entry when no Floating IP interface present  https://review.openstack.org/15307723:09
openstackgerritMerged openstack/python-neutronclient: Implement LBaaS object model v2  https://review.openstack.org/11147523:09
*** prad has quit IRC23:10
*** aranjan has quit IRC23:10
openstackgerritDoug Wiegley proposed openstack/neutron: Flavor Framework implementation  https://review.openstack.org/13975823:13
*** achanda has quit IRC23:15
openstackgerritMaru Newby proposed openstack/neutron: Automate host configuration for functional testing  https://review.openstack.org/14611323:18
*** g4rg4m3|_ has quit IRC23:18
*** jasondotstar has joined #openstack-neutron23:19
*** belmoreira has quit IRC23:20
*** banix has quit IRC23:20
*** reed has quit IRC23:24
*** hichihara has joined #openstack-neutron23:25
*** lori is now known as lori|away23:25
*** ZZelle_ has quit IRC23:25
*** thedodd has quit IRC23:28
*** jorgem has joined #openstack-neutron23:28
*** absubram has quit IRC23:28
*** sgordon has quit IRC23:28
*** shivharis has joined #openstack-neutron23:29
shivharismarkmcclain: hi23:29
*** steveg` has joined #openstack-neutron23:31
markmcclainshivharis: hi23:32
shivharismarkmcclain: wanted to run an idea by you23:33
shivhariswould it be appropriate or useful to have a neutron/vendor directory where vendors pull in their stuff from other repos23:33
shivharisin this way one can probably use submodules of git to manage 2 distinct repos23:34
shivharisi was trying to do a POC before i get my head cut off on that23:34
shivharisopinion?23:34
*** achanda has joined #openstack-neutron23:37
markmcclainshivharis: we had considered that as a possible path as part of the vendor decomp23:37
openstackgerritMerged openstack/neutron: Imported Translations from Transifex  https://review.openstack.org/15280123:37
markmcclaingit submodules can cause some unexpected problems so we decided against them23:38
shivharissubmodules i agree need more TLC - how about just a place like neuton/vendor directory the problem i have is23:39
*** baoli has quit IRC23:39
*** jorgem has quit IRC23:39
*** ajmiller has quit IRC23:39
shivharisthat things are scattered like tests/plugins/driver are all over the place23:39
*** steveg` has quit IRC23:40
*** dims__ has joined #openstack-neutron23:40
shivharishow to have one repo and push the code upstream as a consolidated entity (the problem gets worse when there are multiple products)23:41
*** dims_ has joined #openstack-neutron23:41
*** steveg` has joined #openstack-neutron23:43
*** dims__ has quit IRC23:46
*** jobewan has quit IRC23:47
shivharisi'll assume submodules is not a good idea, but a common location like neutron/vendor?23:47
dougwiggit submodules are the work of satan.23:47
dougwignot that i feel strongly.23:47
dougwig:)23:47
shivharisdougwig: :-)23:48
*** jorgem has joined #openstack-neutron23:48
shivharismarkmcclain: i'll assume submodules is not a good idea, but a common location like neutron/vendor?23:49
shivharisdougwig: there are definitely more careful steps needed to handle submodules. i will not pursue that path23:51
markmcclainshivharis: in general the vendor split out will allow you to construct your tree how you'd like23:51
*** gildub has quit IRC23:51
markmcclainshould concentrate vendor specific code in a single location23:51
dougwigshivharis: have you had a problem using the mechanisms spelled out in the vendor decomp spec?23:52
shivharisthat is my point as well. singe location is key. right now test and drivers are in different locations23:52
*** haramaki has joined #openstack-neutron23:53
shivharisdougwig: yes, i followed it carefully, found some holes communicated those to armax23:53
dougwigshim in neutron, drivers and tests in separate repo.  i'm not sure how they're in different locations?23:53
armaxshivharis: care to elaborate?23:53
openstackgerritSalvatore Orlando proposed openstack/neutron: wip ipam reference driver  https://review.openstack.org/15048523:53
armaxshivharis: what holes are you referring to? and were they plugged? or you’re on a sinking shit?23:54
armaxoops23:54
armaxship23:54
shivharissplit.sh needed some arguments, without those it would hang, and told me to go for a coffee :-)23:54
armaxshivharis: regarding split.sh I fixed that23:54
shivharisarmax: :-)23:54
shivharisarmax: yes you did, np23:55
armaxshivharis: I was operating under the assumption that one would read the script, but I made it more user friendly23:55
armaxshivharis: anything else?23:55
*** annegent_ has joined #openstack-neutron23:55
*** markvoelker has quit IRC23:56
shivharisi got the history done right with that, but problem now is that in a dev. environment i want to push code upstream to stackforge and maybe sometimes to neutron23:56
armaxok23:56
shivharisthat is not clean, wanted a reasonable solution for that, so asking for opinions23:56
*** rm_work is now known as rm_work|away23:56
*** jorgem has quit IRC23:57
armaxshivharis: what problem are you facing?23:57
*** jorgem has joined #openstack-neutron23:57
shivharisconsolidating in one location seems to be a good idea23:57
openstackgerritDoug Wiegley proposed openstack/neutron-vpnaas: Dummy commit.  DO NOT MERGE  https://review.openstack.org/15308723:57
armaxshivharis: there are a few pointers on the git history where other people went ahead and started off the decomp on the right foot23:57
shivharisconsolidating vendor stuff in one directory/location is a good idea23:57
armaxshivharis: arista’s Sukhdev is a great resource for that23:57
shivharishe did not get the history at all :-(23:58
*** jorgem has quit IRC23:58
dougwigyou want one repo that sometimes pushes to stackforge and sometimes to neutron?  how is that one repo, ever?23:58
armaxshivharis: there’s the ODL example23:58
armaxshivharis: regardless of the history, the process is still the same23:59
shivharissome files are in repo 1 and some other files are in repo 223:59
shivharisneutron core and vendor stuff, and the files are interspersed23:59
shivharisi was suggesting using one dir/location for vendor stuff23:59

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