Friday, 2015-06-19

*** sdake_ has joined #openstack-neutron00:00
*** ZZelle_ has joined #openstack-neutron00:01
*** ZZelle_ has quit IRC00:01
*** nlahouti has quit IRC00:02
*** roeyc has quit IRC00:02
*** sdake has quit IRC00:03
*** sdake_ is now known as sdake00:06
*** padkrish has joined #openstack-neutron00:10
*** lnicolas has joined #openstack-neutron00:12
openstackgerritDoug Wiegley proposed openstack/neutron: Updated from global requirements  https://review.openstack.org/19337000:15
openstackgerritSridhar Ramaswamy proposed openstack/neutron-specs: Add Dynamic Multipoint VPN to Neutron VPNaaS  https://review.openstack.org/18156300:15
*** armax has quit IRC00:16
*** amotoki_ has joined #openstack-neutron00:23
*** apuimedo has quit IRC00:23
*** yamamoto has joined #openstack-neutron00:24
*** hitalia has quit IRC00:25
*** sridhar_ram has quit IRC00:26
*** amotoki_ has quit IRC00:28
*** carl_baldwin has quit IRC00:28
*** Marga_ has quit IRC00:29
*** yamamoto has quit IRC00:29
*** ecbanks has quit IRC00:30
*** r-daneel has quit IRC00:34
*** Marga_ has joined #openstack-neutron00:34
*** dims has joined #openstack-neutron00:39
*** ecbanks has joined #openstack-neutron00:40
*** thorst has joined #openstack-neutron00:40
*** sdake_ has joined #openstack-neutron00:41
*** banix has quit IRC00:42
*** nati_ueno has quit IRC00:43
*** nati_ueno has joined #openstack-neutron00:44
*** sdake has quit IRC00:44
*** merooney has quit IRC00:46
*** tongl has quit IRC00:46
*** dims has quit IRC00:47
*** dims has joined #openstack-neutron00:47
*** dsneddon has quit IRC00:48
*** spandhe has joined #openstack-neutron00:49
*** jwy has quit IRC00:50
*** banix has joined #openstack-neutron00:52
*** zzzeek has joined #openstack-neutron00:52
*** carl_baldwin has joined #openstack-neutron00:53
*** aranjan has quit IRC00:53
*** openstack has joined #openstack-neutron00:56
*** markvoelker has joined #openstack-neutron00:56
*** shwetaap has quit IRC00:58
*** shwetaap has joined #openstack-neutron00:59
*** carl_baldwin has quit IRC00:59
*** carl_baldwin has joined #openstack-neutron01:00
*** markvoelker has quit IRC01:02
*** padkrish has quit IRC01:02
*** carl_baldwin has quit IRC01:04
*** shivrao has quit IRC01:04
*** Marga_ has quit IRC01:08
*** erkules_ has joined #openstack-neutron01:09
*** sthillma has quit IRC01:09
*** Marga_ has joined #openstack-neutron01:10
*** enikanorov2 has quit IRC01:10
*** erkules has quit IRC01:11
HenryGkevinbenton: pong01:13
*** yamamoto has joined #openstack-neutron01:14
*** jwy has joined #openstack-neutron01:17
*** yamamoto has quit IRC01:18
*** ivar-lazzaro has quit IRC01:19
*** ianbrown has quit IRC01:22
*** enikanorov2 has joined #openstack-neutron01:23
*** merooney has joined #openstack-neutron01:24
*** ecbanks has quit IRC01:24
*** ianbrown has joined #openstack-neutron01:25
HenryGdougwig: ping01:27
dougwigHenryG: ack01:27
HenryGdougwig: Do you remember how the *aas alembic migrations branches were started? I need to know which ducks need to be in a row when doing the same for 3rd party repos.01:28
dougwigHenryG: one sec, let me get off phone.01:29
*** shwetaap has quit IRC01:31
HenryGdougwig: I am looking at https://review.openstack.org/140545 for example01:31
*** Longgeek has joined #openstack-neutron01:31
HenryGbut we didn't have Depends-On in those days, so wondering if there was some other prep work01:32
*** dave-mcc_ has quit IRC01:33
*** dsanders1 has quit IRC01:33
dougwigyou can make your own separate chain without touching neutron.  the trick comes in *running* the migration.  right now, you just have to know to do it, as as stuff decomps, it's getting a bit untenable.01:34
dougwigi've long pondered adding a way to register your outside chain, so the standard upgrades will know to run them, in addition to automatically allowing neutron-db-manage to have a command-line option to trigger them.01:34
dougwigHenryG: ^^01:34
HenryGYep I know about that bit. I have some wording on it in https://review.openstack.org/18726701:35
HenryGline 47901:35
*** stanzgy has joined #openstack-neutron01:35
dougwigas an alternative to the steps you suggest, you can also just have the decomp repo start it's chain at the moment of decomp.  i.e., the neutron migrations get frozen for the old models.01:36
dougwigi can see how leaving that cruft would be suckful, though.01:37
dougwigwe did it for lbaas v1 to make the split easier to digest, and because those models are deprecated anyway.01:37
HenryGWe've agreed to leave the cruft. It's too painful and error prone to remove.01:38
*** Lufus has joined #openstack-neutron01:38
dougwigthe missing piece is that you have to make your own alembic upgrade script, IMO. the neutron one doesn't hook anything but the *aas repos.01:38
dougwigand there's no automatic tracking of those, either.  the distros just have to know to always do all four01:39
HenryGYes. Line 473 in my review is sorely lacking in detail. That is what I am trying to buff up on.01:39
dougwigHenryG: oh, gotcha.  you want to talk to blogan01:39
HenryGok, ping blogan :)01:40
openstackgerritHuang Rui proposed openstack/neutron: Add zvm agent type and vif type  https://review.openstack.org/19298001:40
*** yeungp1 has joined #openstack-neutron01:41
*** yeungp has quit IRC01:41
HenryGdougwig: I guess what I need to know is if the downrev in the start revision can be a neutron revision (instead of None).01:41
dougwigno, because it's a separate chain. you want it independent.01:42
dougwigHenryG: ^^01:42
dougwigkeep forgetting your prefix01:42
dougwigotherwise alembic will try to make a tree and it gets hairy.01:42
*** debedb has joined #openstack-neutron01:43
*** _cjones_ has quit IRC01:44
*** shivrao has joined #openstack-neutron01:45
HenryGdougwig: I see. But this "separate" chain assumes a whole bunch of tables already exist.01:45
dougwigHenryG: leap of faith, man, leap of faith.01:45
HenryGdougwig: heh01:45
dougwigmore seriously, in a sane world, you'd depend on a >= version of neutron in requirements.txt, which would guarantee that your tables exist.01:46
dougwigso you kinda have to pretend that part is there for it to all hang together.01:46
HenryGdougwig: and also trusting the neutron core alembic branch to never touch those tables after the split01:46
dougwigHenryG: yes, that is hella important, and i was only comfortable with it because the tables we left were deprecated.01:47
*** debedb has quit IRC01:48
HenryGdougwig: I guess such migration would be found pretty quick, and then nuked from orbit01:48
dougwigyou could also add a test in neutron/tests/db that enshrined the current state, with big hairy warnings not to touch it.01:48
dougwigHenryG: ^^01:48
*** debedb has joined #openstack-neutron01:49
HenryGdougwig: yep, I am looking at the tests now01:49
HenryGdougwig: they are enshrined in neutron/db/migration/alembic_migrations/external.py01:50
*** banix has quit IRC01:50
HenryGdougwig: the untouchables, that is01:50
*** Longgeek has quit IRC01:50
HenryGdougwig: as each 3rd party model is moved out of tree, it must be added to that list01:51
dougwigright, but does a test error if a newer migration actually touches them?01:51
dougwigi didn't write such a thing.01:51
HenryGdougwig: still checking. akamyshnikova may have done one01:52
dougwiga grep says no.  it's only used in the test tree to exclude it from the migration check test01:53
*** shwetaap has joined #openstack-neutron01:53
HenryGdougwig: I don't see it. Filing a bug.01:53
HenryGaye01:53
*** s3wong has quit IRC01:53
Lufushttps://bugs.launchpad.net/neutron/+bug/146568701:53
openstackLaunchpad bug 1465687 in neutron "use fixed ip address cannot ping/ssh instance from controller node " [Undecided,New] - Assigned to Khayam Gondal (khayam-gondal)01:53
HenryGdougwig: The tricky part is knowing when a table was "frozen".01:54
dougwigHenryG: maybe the final version should be added to external.py?  convert those lists to dicts?01:55
Lufuswhether we can ping/ssh the instance fixed ip from controller node, I found in instance, we can ping/ssh controlelr node, but controller node cannot ping/ssh instance ip01:55
*** thorst has quit IRC01:55
Lufuswho can give some advice about this quesiton01:55
HenryGdougwig: my thoughts exactly01:56
*** markvoelker has joined #openstack-neutron01:57
*** britthouser has joined #openstack-neutron01:58
*** merooney has quit IRC01:59
*** enikanorov2 has quit IRC01:59
*** mitchjameson has quit IRC02:00
*** ypraveen has quit IRC02:00
*** nijaba has joined #openstack-neutron02:00
*** britthou_ has quit IRC02:02
*** aix has quit IRC02:02
*** yamamoto has joined #openstack-neutron02:02
*** markvoelker has quit IRC02:02
*** Aish has joined #openstack-neutron02:04
*** Aish has quit IRC02:04
*** pavel_bondar has quit IRC02:04
*** ecbanks has joined #openstack-neutron02:04
*** banix has joined #openstack-neutron02:06
*** ecbanks has quit IRC02:09
*** shivrao has quit IRC02:10
HenryGdougwig: bug 146670402:11
openstackbug 1466704 in neutron "Frozen (external) DB tables not prevented from alteration" [Undecided,New] https://launchpad.net/bugs/146670402:11
*** Marga_ has quit IRC02:12
dougwigHenryG: that can probably get a low hanging fruit tag.02:12
HenryGdougwig: sounds good. Please also set it to confirmed02:12
*** daneyon_ has quit IRC02:13
HenryGdougwig: Although I suspect Ann will have the code out for review before I wake up tomorrow :)02:13
dougwigHenryG: lol, likely true02:13
*** shwetaap has quit IRC02:14
*** shwetaap has joined #openstack-neutron02:15
*** Marga_ has joined #openstack-neutron02:16
HenryGdougwig: the *aaSes have not removed their downgrade()s02:16
HenryGNow that is a fruit rotting on the ground ;)02:17
dougwigHenryG: lol, indeed.02:17
dougwigHenryG: i have just the person for that one.02:17
HenryGdougwig: cool02:17
*** shwetaap has quit IRC02:23
*** shwetaap has joined #openstack-neutron02:23
*** wwriverrat has quit IRC02:23
*** ypraveen has joined #openstack-neutron02:24
*** amotoki_ has joined #openstack-neutron02:24
*** tflynn_ has joined #openstack-neutron02:25
HenryGdougwig: do you know what grenade's problem is with neutron today?02:26
*** yeungp1 has quit IRC02:27
*** tflynn has quit IRC02:29
*** amotoki_ has quit IRC02:30
*** tflynn_ has quit IRC02:30
*** claudiub has quit IRC02:31
*** chem has joined #openstack-neutron02:31
*** kiran-r has joined #openstack-neutron02:33
*** sdake_ has quit IRC02:34
*** prad has quit IRC02:35
*** cburgess has quit IRC02:39
*** cburgess has joined #openstack-neutron02:44
*** spandhe has quit IRC02:45
*** jamielennox is now known as jamielennox|away02:46
*** tianhuan has joined #openstack-neutron02:47
*** Longgeek has joined #openstack-neutron02:47
*** Marga_ has quit IRC02:49
*** jamielennox|away is now known as jamielennox02:53
*** tomoe_ has joined #openstack-neutron03:00
*** dsanders has joined #openstack-neutron03:02
*** wuhg has joined #openstack-neutron03:03
*** cburgess has quit IRC03:04
*** cburgess has joined #openstack-neutron03:05
*** dims has quit IRC03:06
wuhghere:the from grouter namespace ,i can't ping the external gateway, any help03:06
*** cburgess has quit IRC03:07
*** cburgess has joined #openstack-neutron03:10
*** nati_uen_ has joined #openstack-neutron03:15
*** armax has joined #openstack-neutron03:16
*** nati_ueno has quit IRC03:19
*** tomoe_ has quit IRC03:19
*** Longgeek has quit IRC03:19
*** carl_baldwin has joined #openstack-neutron03:21
*** rm_work has quit IRC03:25
*** amotoki_ has joined #openstack-neutron03:25
*** carl_baldwin has quit IRC03:26
*** debedb has quit IRC03:26
*** shwetaap has quit IRC03:27
*** shwetaap has joined #openstack-neutron03:28
*** coolsvap|away is now known as coolsvap03:30
*** amotoki_ has quit IRC03:30
*** rm_work|away has joined #openstack-neutron03:31
*** rm_work|away is now known as rm_work03:31
*** rm_work has joined #openstack-neutron03:31
*** crc32 has quit IRC03:31
*** sdake has joined #openstack-neutron03:33
*** amit213 has joined #openstack-neutron03:34
*** amit213 has quit IRC03:36
*** Marga_ has joined #openstack-neutron03:38
*** amit213 has joined #openstack-neutron03:39
*** norman has joined #openstack-neutron03:39
normanHi all, anyone ever run into the issue that tap device changed to be down , and then no connection to the VM?03:40
*** amit213 has quit IRC03:41
*** jwy has quit IRC03:42
*** kiran-r has quit IRC03:44
*** Marga_ has quit IRC03:44
*** markvoelker has joined #openstack-neutron03:46
openstackgerritshihanzhang proposed openstack/neutron: Add conntrack-tool to manage security groups  https://review.openstack.org/14771303:46
*** markvoelker has quit IRC03:51
*** ildikov_ has quit IRC03:55
*** debedb has joined #openstack-neutron04:00
*** Longgeek has joined #openstack-neutron04:00
*** zz_jgrimm has quit IRC04:05
*** ypraveen_ has joined #openstack-neutron04:05
*** debedb has quit IRC04:05
*** banix has quit IRC04:06
*** Longgeek has quit IRC04:06
*** zz_jgrimm has joined #openstack-neutron04:06
*** ypraveen has quit IRC04:06
*** ypraveen_ is now known as ypraveen04:06
*** aranjan has joined #openstack-neutron04:07
*** sdake_ has joined #openstack-neutron04:07
*** Longgeek has joined #openstack-neutron04:08
*** debedb has joined #openstack-neutron04:10
*** sdake has quit IRC04:10
*** Longgeek_ has joined #openstack-neutron04:12
*** fedexo has joined #openstack-neutron04:12
*** veena has joined #openstack-neutron04:13
*** Longgeek has quit IRC04:15
openstackgerritMerged openstack/neutron: Updated from global requirements  https://review.openstack.org/19337004:19
*** ypraveen has quit IRC04:19
*** carl_baldwin has joined #openstack-neutron04:21
*** Networkn3rd has quit IRC04:24
*** Networkn3rd has joined #openstack-neutron04:24
*** amotoki_ has joined #openstack-neutron04:26
*** jbell8 has joined #openstack-neutron04:30
*** rossella_s has quit IRC04:30
*** rossella_s has joined #openstack-neutron04:31
*** amotoki_ has quit IRC04:31
*** achanda has joined #openstack-neutron04:33
*** miyagishi_t has joined #openstack-neutron04:34
*** debedb_ has joined #openstack-neutron04:36
*** debedb has quit IRC04:37
*** rm_you has joined #openstack-neutron04:37
*** debedb has joined #openstack-neutron04:38
openstackgerritHirofumi Ichihara proposed openstack/neutron-specs: Add availability_zone support  https://review.openstack.org/16961204:40
*** hitalia has joined #openstack-neutron04:40
*** debedb_ has quit IRC04:41
*** Marga_ has joined #openstack-neutron04:42
*** irenab has joined #openstack-neutron04:43
*** _cjones_ has joined #openstack-neutron04:45
*** markvoelker has joined #openstack-neutron04:47
*** Marga__ has joined #openstack-neutron04:47
*** ianbrown has quit IRC04:48
*** _cjones_ has quit IRC04:49
openstackgerritHirofumi Ichihara proposed openstack/neutron-specs: Add availability_zone support  https://review.openstack.org/16961204:51
*** Marga_ has quit IRC04:51
*** markvoelker has quit IRC04:51
*** tflynn has joined #openstack-neutron04:52
*** carl_baldwin has quit IRC04:53
*** ijw_ has joined #openstack-neutron04:55
openstackgerritHirofumi Ichihara proposed openstack/neutron-specs: Add availability_zone support  https://review.openstack.org/16961204:58
*** trinaths has joined #openstack-neutron04:59
*** achanda has quit IRC04:59
*** anilvenkata has joined #openstack-neutron05:00
*** tflynn_ has joined #openstack-neutron05:00
*** ianbrown has joined #openstack-neutron05:00
*** chem has quit IRC05:01
kevinbentonHenryG: hey, just wanted to make sure my comment made sense on the db retry patch05:03
*** tflynn has quit IRC05:04
*** boris-42 has quit IRC05:12
*** dsneddon has joined #openstack-neutron05:14
*** jprovazn has joined #openstack-neutron05:16
*** vCloudernBeer has joined #openstack-neutron05:17
*** irenab has quit IRC05:18
*** garyk has joined #openstack-neutron05:19
*** rm_you has quit IRC05:19
*** jbell8 has quit IRC05:20
*** yuanying has quit IRC05:24
*** Lufus has quit IRC05:25
*** amotoki_ has joined #openstack-neutron05:27
hichiharaamotoki: ping05:28
*** Networkn3rd has quit IRC05:31
*** amotoki_ has quit IRC05:31
*** sdake_ has quit IRC05:33
*** miyagishi_t has quit IRC05:40
*** miyagishi_t has joined #openstack-neutron05:41
*** SridharG has joined #openstack-neutron05:46
openstackgerritSean M. Collins proposed openstack/neutron: [DO NOT MERGE] Patch to test the new linux bridge job  https://review.openstack.org/18723505:46
*** shwetaap has quit IRC05:49
vCloudernBeerrossella_s: ping05:52
*** miyagishi_t has quit IRC05:54
*** absubram has joined #openstack-neutron06:01
*** absubram has quit IRC06:01
openstackgerritHirofumi Ichihara proposed openstack/neutron-specs: Add enable_new_agents to agent conf in network node  https://review.openstack.org/17077406:01
*** kiran-r has joined #openstack-neutron06:02
*** jbell8 has joined #openstack-neutron06:05
*** numan has joined #openstack-neutron06:05
*** ianbrown has quit IRC06:11
*** vijaykc4 has joined #openstack-neutron06:15
openstackgerritshihanzhang proposed openstack/neutron: Don't delete DVR namespace if there are still ports on this node  https://review.openstack.org/19167706:18
*** armax has quit IRC06:22
*** neeti has joined #openstack-neutron06:32
*** lpetrut has joined #openstack-neutron06:34
*** anshul has joined #openstack-neutron06:35
*** markvoelker has joined #openstack-neutron06:36
*** yog_ has joined #openstack-neutron06:38
*** markvoelker has quit IRC06:40
*** tflynn_ has quit IRC06:44
*** moshele has joined #openstack-neutron06:47
*** gildub has quit IRC06:48
*** tianhuan has left #openstack-neutron06:53
*** sudipto has joined #openstack-neutron06:53
*** fedexo has quit IRC06:55
*** vijaykc4 has quit IRC06:59
*** Marga__ has quit IRC07:00
*** jlanoux has joined #openstack-neutron07:04
*** Marga_ has joined #openstack-neutron07:04
*** oomichi has joined #openstack-neutron07:05
*** oomichi_ has joined #openstack-neutron07:05
*** Marga_ has quit IRC07:05
*** chlong has quit IRC07:06
openstackgerritSean M. Collins proposed openstack/neutron: [DO NOT MERGE] Patch to test the new linux bridge job  https://review.openstack.org/18723507:10
*** kawa2014 has joined #openstack-neutron07:16
*** fzdarsky has joined #openstack-neutron07:16
*** ihrachyshka has joined #openstack-neutron07:17
*** jlanoux has quit IRC07:18
*** oda-g has quit IRC07:19
*** jlanoux has joined #openstack-neutron07:19
*** ronis has joined #openstack-neutron07:21
openstackgerritshihanzhang proposed openstack/neutron: Add conntrack-tool to manage security groups  https://review.openstack.org/14771307:22
*** oomichi has quit IRC07:24
*** oomichi_ has quit IRC07:24
*** devvesa has joined #openstack-neutron07:26
openstackgerritYAMAMOTO Takashi proposed openstack/neutron: Reject router-interface-add with a port which doesn't have any addresses  https://review.openstack.org/19344007:31
*** otherwiseguy has quit IRC07:36
*** dsanders has quit IRC07:41
*** matrohon has joined #openstack-neutron07:42
*** lukasa has joined #openstack-neutron07:43
*** akamyshnikova has quit IRC07:44
*** ygbo has joined #openstack-neutron07:45
*** alex_klimov has joined #openstack-neutron07:45
*** veena has quit IRC07:48
*** akamyshnikova has joined #openstack-neutron07:50
*** dsanders has joined #openstack-neutron07:50
*** vCloudernBeer has quit IRC07:51
*** safchain has joined #openstack-neutron07:54
*** claudiub has joined #openstack-neutron07:58
*** jamielennox is now known as jamielennox|away08:00
*** otherwiseguy has joined #openstack-neutron08:02
*** irenab has joined #openstack-neutron08:03
*** ZZelle has quit IRC08:03
*** ZZelle has joined #openstack-neutron08:03
*** mickeys has joined #openstack-neutron08:04
*** vijaykc4 has joined #openstack-neutron08:05
*** irenab has quit IRC08:08
*** dsanders has quit IRC08:08
openstackgerritThomas Morin proposed openstack/neutron: Precision networking-bagpipe-l2 subproject  https://review.openstack.org/19344708:14
*** mickeys has quit IRC08:15
*** ildikov_ has joined #openstack-neutron08:15
*** boris-42 has joined #openstack-neutron08:17
*** lpetrut has quit IRC08:24
ihrachyshkasc68cal, hey. I look at lb agent code and is a bit surprised that we make attempt there to plug a tap device into a qbr- bridge if it's not plugged yet. it seems like work to do for VIF driver on nova side only, isn't it?08:24
*** markvoelker has joined #openstack-neutron08:25
openstackgerritElena Ezhova proposed openstack/neutron-fwaas: [WIP] Switch to oslo.service  https://review.openstack.org/19345308:25
*** afazekas has joined #openstack-neutron08:25
openstackgerritElena Ezhova proposed openstack/neutron-vpnaas: [WIP] Switch to oslo.service  https://review.openstack.org/19345508:26
*** aranjan has quit IRC08:29
*** amotoki_ has joined #openstack-neutron08:29
*** pavel_bondar has joined #openstack-neutron08:29
*** markvoelker has quit IRC08:29
openstackgerritElena Ezhova proposed openstack/neutron: [WIP] Awitch to oslo.service  https://review.openstack.org/19299908:31
*** aix has joined #openstack-neutron08:32
*** alexpilotti has joined #openstack-neutron08:33
*** amotoki_ has quit IRC08:34
*** hichihara has quit IRC08:35
*** miyagishi_t has joined #openstack-neutron08:38
openstackgerritPavel Bondar proposed openstack/neutron: Remove _check_ip_in_allocation_pool  https://review.openstack.org/19273808:39
*** vijaykc4 has quit IRC08:41
*** debedb has quit IRC08:44
*** devvesa has quit IRC08:46
ZZelleihrachyshka, hi08:47
*** debedb has joined #openstack-neutron08:47
ihrachyshkaZZelle, hi08:48
*** Longgeek_ has quit IRC08:51
*** Longgeek has joined #openstack-neutron08:51
*** lpetrut has joined #openstack-neutron08:56
ajokevinbenton, shz08:57
shzyes08:57
ajoI respectfully disagree about the use of @staticmethod for just the shake of not referencing self :)08:57
*** debedb has quit IRC08:57
ajoIMHO we may only use @staticmethod when a method is truthfully going to be used as static :), otherwise we add confusion to the reader08:57
ajoI had that sort of discussion in other patches and not even cores agreed :D, so I keep repeating myself :D08:58
kevinbentonajo: what do you mean if it's going to be used as static?08:58
ajoihrachyshka, do you have an opinion on this ?  :)08:58
kevinbentonit is being used as static!08:58
kevinbentonnobody is referencing self08:58
ajokevinbenton, I mean a caller will use it as Class.method(xxxx)08:58
ajothat's a true static use :)08:59
kevinbentonright, the unit tests should call it that way08:59
ajooutside the class methods themselves :)08:59
kevinbentonthere just are no unit tests for that method right now08:59
ajokevinbenton my point is only about not adding noise to the reader,08:59
ajowhat's the gain of making it @staticmethod if nobody will call Class.method(...)08:59
ihrachyshkaajo, I'm agnostic08:59
kevinbentona unit test should be calling it09:00
ajoihrachyshka: lol09:00
ihrachyshkareally, there are better fights to fight09:00
*** john-davidge has joined #openstack-neutron09:00
ajokevinbenton: real code, unit tests can instanciate the class very cheaply :)09:00
kevinbentonhalf of the reason our unit tests are big complex messes is because we can't call most of the methods without a big complicated init of an object that mocks a million things09:00
ihrachyshkaneither I think it's worth a -1, but that's just me09:00
ajobut ok, kevinbenton , that's a point, unit tests are cleaner...09:00
*** devvesa has joined #openstack-neutron09:01
ajook, I'm agnostic now too ;)09:01
kevinbentonajo: look at the tests for the agents and how much junk has to be mocked out09:01
ajoI will not -1 for a @staticmethod used just for notselfreference :P09:01
ihrachyshkahonestly, I usually don't bind a function to a class if it does not depend on state09:01
ihrachyshkawhat is so wrong about functions? :)09:01
kevinbentonihrachyshka: +109:01
ajoihrachyshka, well... grouping them logically?09:01
*** pask81 has joined #openstack-neutron09:02
kevinbentoni would rather see it be a module function than unnecessary instance methods09:02
ihrachyshkaajo, in a pile of 5000+ lines of code, yeah. it's a lot better than a helper submodule09:02
ajoihrachyshka, but submodules are ok for generic stuff09:02
ihrachyshka /sarcasm off09:02
ajoihrachyshka, the kind of function I'm talking about is the one that only serves the purpose of a class... , and it's just done to subdivide work in functions instead of having a 500L function :)09:02
ajodoesn't make sense to reuse it anywhere else09:03
*** wznoinsk has quit IRC09:03
ihrachyshkathen make it private global helper function09:03
kevinbentonajo: but that's still a function that should be tested09:03
kevinbentoncheck this out09:03
ajokevinbenton, yes, I totally agree on the testing case09:03
kevinbentonhttps://github.com/openstack/neutron/blob/master/neutron/tests/unit/plugins/openvswitch/agent/test_ovs_neutron_agent.py#L11709:03
kevinbentonthat's so disgusting that pep8 gave up and didn't complain about the line longer than 80 characters09:03
ajokevinbenton, I'm having breakfast, do you want to make me dizzy? :D09:03
ajoooouchcrap09:04
kevinbenton:)09:04
ajogot pep8 dizzy ... ;)09:04
kevinbentonajo: i agree with you if object construction is really simple09:04
*** john-davidge has quit IRC09:04
kevinbentonajo: which it actually doesn't seem to be too bad in this particular patch09:04
kevinbentonajo: but we have a tendancy to have mega objects that do all the things09:05
*** wznoinsk has joined #openstack-neutron09:05
*** john-davidge has joined #openstack-neutron09:05
*** nati_uen_ has quit IRC09:05
ajokevinbenton, I agree, I tend to ask for object splits, where it makes sense09:05
ihrachyshkaoh hilarious code09:05
ajoshz can tell I asked him to separate conntrack manager :)09:05
ajobtw shz , good work,09:06
ihrachyshkaVeni, vidi, vomiti09:06
ajolet's go back to reviewing your patch https://review.openstack.org/#/c/147713/18/  you changed a lot from PS16 to PS18 so I need to re-read most of it :)09:06
ihrachyshkakevinbenton, yes, those piles I referred. our objects are generally overflowed by stuff that could be split logically09:06
kevinbentonihrachyshka: :) i learned a new latin phrase to describe a friday night09:07
ajolol09:08
ihrachyshkakevinbenton, by any chance, do you know who is responsible for bridge cleanup? nova vif too?09:09
kevinbentonihrachyshka: you mean deleting the iptables bridge?09:09
ajoihrachyshka, yes, it's nova vif AFAIK09:09
ihrachyshkayes09:09
kevinbentonyeah09:09
kevinbentonnova09:09
kevinbentonvif unplug09:09
ajowe delete iptables entries, btw09:09
kevinbentonaye09:09
ajoopenvswitch-agent specifically09:09
ihrachyshkait's funny that lb agent tries to plug and unplug tap to qbr itself09:10
ajoihrachyshka, really?09:10
kevinbentonload balancer agent?09:10
ihrachyshkaI'm going to kill that crap. racy nonsense09:10
kevinbentonor linux bridge agent?09:10
ihrachyshkakevinbenton, linuxbridge09:10
ajodo we have LinuxBridge testing in the end ?09:10
kevinbentonah, yeah, i think we have well decided that it's up to the creator to destroy things09:10
ihrachyshkaajo, all kinds of overmocked unit tests. isn't it enough??09:11
ajokevinbenton, yes otherwise responsibilities are disperse, and code is a mess :)09:11
ajowhat if I mock.patch('mock.patch') as patcherpatcher09:11
ajo(sorry... ':D ;)09:11
kevinbentonwe should monkey patch mock.patch to make sure it can only be called twice during a unit test09:12
ajothe syntax is probably not correct, but it wouldn't do too much fun... just09:12
ajokevinbenton, lol, that would be nice09:12
kevinbentonif you require more than two patches, needs a rewrite09:12
ajokevinbenton: if somebody does, it will hit him with a loud noise09:12
*** devvesa has quit IRC09:13
ajokevinbenton, then we first need to rewrite /neutron/tests/unit almost at whole ;)09:13
kevinbentonyeah09:13
ajokevinbenton, or we can build a karma system based in number of mocks09:13
openstackgerritSridhar Gaddam proposed openstack/neutron: Add sanity_check for keepalived ipv6 support  https://review.openstack.org/16878109:13
ajoyou introduce a mock :  -1 on karma09:13
ajoyou remove a mock: +1 on karma09:14
ajomockarma09:14
ajoI have a funny morning ... ':D09:14
kevinbentonajo: write a test without using mock should also gain karma09:14
ajobut it'd be cool to see the stats09:14
ajokevinbenton: true!,09:14
ajokevinbenton, sounds like an interesting project, in fact ;D09:15
ajowith reporting to the irc channel ;D09:15
kevinbentonajo: there are some nice uses of mocking to inject failures09:15
kevinbentonmight be hard to test exception paths without them09:15
ajoof course :)09:15
matrohonkevinbenton : hi09:17
kevinbentonmatrohon: hey, what's up?09:17
matrohonkevinbenton : I was reviewing the spec : https://review.openstack.org/#/c/94612/09:17
matrohonkevinbenton : I'm concerned that the reference implementation details are not in the spec anymore....09:18
kevinbentonmatrohon: that's fine, we are trying to move away from that because it was almost always wrong in the end anyway09:19
kevinbentonmatrohon: if there was a specific concern about an implementation detail that might break compatiability, that should be noted09:19
matrohonkevinbenton : I understand and I think it's the good path to onboard more people09:20
kevinbentonmatrohon: agreeing on the high-level data model and how users will interact is what's important09:20
*** lori|away is now known as lori09:20
kevinbentonmatrohon: then once we get into implementing things, we may find that one approach works much better than another09:21
matrohonkevinbenton : however we're ending with spec that we don't even know if a ref implementation will be applicable09:21
kevinbentonmatrohon: what do you mean?09:21
kevinbentonmatrohon: if there is no reference implementation, it will be rejected09:21
kevinbentonmatrohon: there still has to be a working implementation09:22
matrohonkevinbenton : I mean that we might have API and use acses that can only be handled by sdn controllers09:22
matrohonkevinbenton : ok that's what I wanted to hear :)09:22
kevinbentonmatrohon: oh, no. we wont' allow that09:23
kevinbentonmatrohon: we just don't to spend a bunch of time designing an ML2 extension when it might work better as a service plugin or something09:23
matrohonkevinbenton : but it seems weird to have so many discussions on a spec, and not having a parallel discussion on the implementation09:24
ajomatrohon, we're changing that, to discuss that kind of detail later on a devref during implementation09:25
kevinbentonmatrohon: the implementation discussions were basically pointless09:25
ajoso we get also a good developer documentation , and we can maintain it with code09:25
*** vijaykc4 has joined #openstack-neutron09:25
kevinbentonmatrohon: because usually an issue would creep up that required a change away from the original design09:25
kevinbentonmatrohon: so you would have to go back and update the spec09:25
ajomatrohon, like: https://review.openstack.org/#/c/190635/09:25
ajoI started by devref, discuss in there, then code along with the devref09:26
*** jbell8 has quit IRC09:26
ajoimplementation detail discussion is better left for later IMO,09:26
*** norman has quit IRC09:27
kevinbentonmatrohon: basically we are trying to get away from complete waterfall design because it doesn't fit well with how many ongoing changes there are to the codebase09:27
matrohonkevinbenton, ajo : fine thanks for clarification, hope it'll work better09:27
kevinbentonmatrohon: and in the time it would take to churn all of the implementation details out in the spec, you could just write the implementation09:28
ajoand discuss over code09:28
ajoless bike shedding09:28
ajoand ... more or less yak shaving ? (It's unclear to me if shaving yaks is actually useful)09:28
matrohonkevinbenton, ajo : make sens, I'm too used to spec with implementation details I think. It's always hard to understand a spec that you don't even know how it will works behind the hood09:29
matrohonkevinbenton, ajo : it's probably an integrator/developper POI09:29
*** devvesa has joined #openstack-neutron09:29
*** aranjan has joined #openstack-neutron09:29
*** coolsvap is now known as coolsvap|away09:30
*** amotoki_ has joined #openstack-neutron09:30
kevinbentonmatrohon: yeah, ultimately we are trying to separate the "product manager/user" components from the implementation details09:30
matrohonkevinbenton, ajo : POI -> POV point of view :)09:30
matrohonajo : concerning https://review.openstack.org/#/c/190635/, is there a corresponding rfe bug?09:31
ajomatrohon: it's implementation detail of the QoS spec09:32
ajomatrohon, I guess I should link the bp :)09:32
matrohonajo : +109:32
ajomatrohon I started a mail list thread about it, but nobody responded :)09:32
ajomatrohon, let me look for the topic09:32
ajomatrohon: [neutron] slimming down (the RFE) process for small internal design stuff09:33
ajokevinbenton ^09:33
matrohonajo : didn't see the ML thread, I have to answer since I need such a framework for bgpvpn09:33
ajoahh, true matrohon , you told me09:33
matrohonajo : thanks09:33
ajomatrohon, if it's reusable for you too, we can move it to the master branch then09:34
*** hexlibris has joined #openstack-neutron09:34
kevinbentonihrachyshka: ping09:34
*** eandersson has joined #openstack-neutron09:34
matrohonajo : oh, I didn't realize taht you were having your own feature branch!09:34
*** amotoki_ has quit IRC09:34
kevinbentonihrachyshka: can you +2/+A  https://review.openstack.org/#/c/160230/09:34
*** aranjan has quit IRC09:34
kevinbentonihrachyshka: it's been sitting for a gross amount of time09:35
*** yatin has joined #openstack-neutron09:36
matrohonajo : I think it should be moved to the master branch since it's a generic framework, and at least sriov implem want to use it09:37
*** coolsvap|away is now known as coolsvap09:37
ihrachyshkakevinbenton, done09:38
kevinbentonihrachyshka: thx09:38
*** moshele has quit IRC09:40
*** jlibosva has joined #openstack-neutron09:40
*** jlanoux_ has joined #openstack-neutron09:40
*** abregman has quit IRC09:41
*** abregman has joined #openstack-neutron09:41
SridharGajo: ihrachyshka: kevinbenton: Can you please share your views regarding the following bug and suggest, thanks. https://bugs.launchpad.net/neutron/+bug/146551309:41
openstackLaunchpad bug 1465513 in neutron "VM get incorrect default gateway when attached port DG manually to router " [Undecided,New] - Assigned to Sridhar Gaddam (sridhargaddam)09:41
*** jasondotstar has joined #openstack-neutron09:41
*** jlanoux has quit IRC09:42
kevinbentonSridharG: that's expected09:42
kevinbentonSridharG: the gateway_ip property of the subnet is what is given out via DHCP09:43
kevinbentonSridharG: regardless of where any router interfaces are09:43
SridharGso kevinbenton, can we consider that the expectation from DHCP to advertise the port-ip (which is now associated to router) as gateway_ip is an invalid use-case?09:45
*** miyagishi_t has quit IRC09:45
kevinbentonSridharG: yeah, just because it's a router IP doesn't mean it should be the default gateway09:46
ZZelleihrachyshka, could you restore https://review.openstack.org/#/c/160699 or i create a new change?09:46
kevinbentonSridharG: you can have several routers attached to a network09:46
kevinbentonSridharG: and only one is the default gateway09:46
kevinbentonSridharG: then extra routes might point some traffic at the other routers09:47
ihrachyshkaZZelle, restored09:47
ZZelleihrachyshka, thx09:47
SridharGkevinbenton: aah ok.09:47
ihrachyshkaZZelle, thanks on working on it09:47
*** dsneddon has quit IRC09:47
SridharGkevinbenton: thanks very much :-) I'll mark the bug as invalid09:47
SridharGkevinbenton: cool, you already did it :-)09:48
kevinbentonSridharG: no prob!09:48
kevinbentonSridharG: thanks for asking09:48
*** abregman has quit IRC09:49
openstackgerritSalvatore Orlando proposed openstack/neutron: Reservations support  https://review.openstack.org/16365909:53
openstackgerritCyril Roelandt proposed openstack/neutron: Python3: do not use im_self/im_func  https://review.openstack.org/19347609:53
*** fkobzik has joined #openstack-neutron09:57
matrohonajo?09:58
ihrachyshkakevinbenton, it does not seem that nova cleans the lb bridges though http://git.openstack.org/cgit/openstack/nova/tree/nova/virt/libvirt/vif.py#n61110:00
ihrachyshkaI suspect they rely on lb agent to do it: http://git.openstack.org/cgit/openstack/neutron/tree/neutron/plugins/linuxbridge/agent/linuxbridge_neutron_agent.py#n94910:01
*** trinaths has quit IRC10:04
openstackgerritIhar Hrachyshka proposed openstack/neutron: lb: avoid doing nova VIF work plumbing tap to qbr  https://review.openstack.org/19348510:04
openstackgerritIhar Hrachyshka proposed openstack/neutron: lb: stop handling Havana device updates  https://review.openstack.org/19348610:04
openstackgerritLucas Alvares Gomes proposed openstack/neutron: DHCP agent: Set an "ipxe" tag to work with Ironic  https://review.openstack.org/17204010:05
*** amotoki_ has joined #openstack-neutron10:07
*** matrohon has quit IRC10:07
*** dims has joined #openstack-neutron10:09
*** trinaths has joined #openstack-neutron10:09
*** mmdurrant has quit IRC10:09
*** coolsvap is now known as coolsvap|away10:11
*** markvoelker has joined #openstack-neutron10:13
*** markvoelker has quit IRC10:18
*** bobmel has quit IRC10:21
*** bobmel has joined #openstack-neutron10:21
*** anshul has quit IRC10:24
openstackgerritMerged openstack/neutron: Remove _check_ip_in_allocation_pool  https://review.openstack.org/19273810:24
*** anshul has joined #openstack-neutron10:24
*** abehl has joined #openstack-neutron10:24
openstackgerritMerged openstack/neutron: Ensure tests run under python2.7  https://review.openstack.org/16023010:24
*** anshul has quit IRC10:24
*** abehl has quit IRC10:24
*** anshul_ has joined #openstack-neutron10:25
ajoihrachyshka: if you have some time to check the conversation/comments here: https://review.openstack.org/#/c/190635/10:25
ajoI'd like more opinions before updating the spec10:25
ajonow10:25
* ajo splits patches! (or trie too...) it seems there's always that comes in the middle of it (procratination--)10:25
*** anshul_ has quit IRC10:26
*** dave-mccowan has joined #openstack-neutron10:27
*** anshul has joined #openstack-neutron10:28
*** anshul has quit IRC10:29
*** anshul has joined #openstack-neutron10:29
*** anshul has quit IRC10:29
*** anshul has joined #openstack-neutron10:30
*** aranjan has joined #openstack-neutron10:30
*** aranjan has quit IRC10:35
openstackgerritPavel Bondar proposed openstack/neutron: Move _add_auto_addrs_on_network_ports  https://review.openstack.org/19307510:38
*** dsneddon has joined #openstack-neutron10:40
*** vbel has quit IRC10:40
*** vbel has joined #openstack-neutron10:41
ihrachyshkaajo, done. also, speaking of that tiny patch of mine about ml2 qos rule list detection... I guess it should be exposed into base plugin, right? it's not ml2 specific (so basically, there should be two patches: one for base plugin to get a way to determine supported types; and another for ml2 to implement it in a custom way that considers multi-driver-ness.10:41
*** vbel has quit IRC10:41
ihrachyshkathe reason why we need it on base plugin level is that we'll want to expose the list to users thru API10:41
ihrachyshkaat least that's my understanding10:41
*** vijaykc4 has quit IRC10:41
ihrachyshkacomments?10:41
openstackgerritPavel Bondar proposed openstack/neutron: Decompose create_port and save_subnet  https://review.openstack.org/19309710:41
ajoihrachyshka, yes, but we don't need to add a call to base plugin10:42
ajoihrachyshka, plugins could, as they initialize, report to qos the list of rules they support10:42
ajoihrachyshka, but then... well, that doesn't work for the ml2 case10:42
ajoihrachyshka, ok, I need to think :)10:42
ajothanks :)10:42
ihrachyshkaajo, I don't think you want your plugin to be aware of a service_plugin, but it's philosophical10:43
ajoihrachyshka, may be ml2 can report it too, as it knows the mechanism drivers it has registered10:43
*** nyechiel has joined #openstack-neutron10:43
ajoihrachyshka, how is the plugin going to support qos if it's not aware of it? :)10:44
ihrachyshkaeveryone can report. qos plugin should just get core plugin and issue a single call to get the list of types10:44
ihrachyshkawell, I mean, you don't want plugin to push data into service_plugins, since plugin is always there, but qos plugin may be missing10:44
*** afazekas has quit IRC10:44
ajoihrachyshka, that makes sense10:44
ihrachyshkaso instead we can be lazy and allow qos plugin to fetch data on demand10:44
ajogood point10:44
*** roeyc has joined #openstack-neutron10:45
ajoihrachyshka, on other topic, any idea on how to filter CI messages from gerrit?10:45
ajoI can see new patchsets by headers10:45
ajobut...10:45
ihrachyshkaso think about it. I'll continue on that on Mon.10:45
ajoCI messages come as "normal comments"10:45
ajoX-Gerrit-MessageType: comment10:45
ajo:(10:46
ihrachyshkaajo, I ended up ignoring everything that does not vote (unless I'm really suspicious)10:46
ihrachyshkaajo, btw you get email notifs for them, so you can apply filtering there10:46
ihrachyshkaah, sorry, I misunderstood you initially10:46
trinathsajo: few ci msgs come as comments10:46
ihrachyshkawell, they all have "CI" in their names10:46
ajotrinaths: ryu, hp, hyperv10:47
ajoall those are comments10:47
*** vijaykc4 has joined #openstack-neutron10:47
ajohmmm good hint ihrachyshka10:47
ajoI hope no michael CI will ever come and join openstack because I'll be filtering him from gerrit ;D10:47
ajoanything "CI (Code Review)" or "Jenkins (Code Review)" goes to *archive!10:49
ajoohhhh10:49
trinathsajo: filter that way.10:49
ajobeautiful!10:49
*** stanzgy has quit IRC10:50
openstackgerritJakub Libosvar proposed openstack/neutron: Refactor NetcatTester class  https://review.openstack.org/18883610:50
openstackgerritJakub Libosvar proposed openstack/neutron: ip_lib: Add flush() command to IpNeigh to clean arp cache  https://review.openstack.org/19227210:50
openstackgerritJakub Libosvar proposed openstack/neutron: Introduce connection testers module  https://review.openstack.org/19114710:50
openstackgerritJakub Libosvar proposed openstack/neutron: Move NetcatTester to common/net_helpers  https://review.openstack.org/19280110:50
openstackgerritJakub Libosvar proposed openstack/neutron: RootHelperProcess: kill can consume signal number  https://review.openstack.org/19319610:50
openstackgerritJakub Libosvar proposed openstack/neutron: Add test cases to testing firewall drivers  https://review.openstack.org/19114810:50
openstackgerritJakub Libosvar proposed openstack/neutron: Introduce yet another Pinger object  https://review.openstack.org/19280210:50
*** Marga_ has joined #openstack-neutron10:50
ajojlibosva++10:50
ajo^10:50
ajo:)10:50
*** Marga_ has quit IRC10:50
jlibosvaajo: :)10:51
jlibosvaajo: do review! :D10:51
ajojlibosva: I'll split a patch (finally) and review it again :)10:51
ajojlibosva, is iptables passing now? :)10:51
jlibosvaajo: you mean firewall blink?10:52
ajojlibosva: yup :)10:53
jlibosvaajo: it is10:53
ajojlibosva: https://twitter.com/mangel_ajo/status/61184934673322803210:54
ajo:_)10:54
jlibosvaajo: so coooool!!!10:54
jlibosvaajo: where are you?10:55
*** nyechiel has quit IRC10:55
ajoIt's like a short of shopping center nearby, it was a railway station reconverted into restaurants + shopping10:55
trinathsajo: :)10:56
*** dims has quit IRC10:57
*** pc_m has joined #openstack-neutron11:00
*** ajo is now known as ajo\picksupfood11:01
*** trinaths has quit IRC11:02
openstackgerritSalvatore Orlando proposed openstack/neutron: NSX QoS ext: RXTX factor can be decimal  https://review.openstack.org/18966511:03
*** wuhg has quit IRC11:04
*** dims has joined #openstack-neutron11:06
*** afazekas has joined #openstack-neutron11:07
*** trinaths has joined #openstack-neutron11:10
*** jprovazn has quit IRC11:10
*** jprovazn has joined #openstack-neutron11:12
*** thing1 has joined #openstack-neutron11:14
*** thing1 has left #openstack-neutron11:15
*** neeti has quit IRC11:21
*** merooney has joined #openstack-neutron11:22
*** roeyc has quit IRC11:23
*** moshele has joined #openstack-neutron11:27
*** markvoelker has joined #openstack-neutron11:29
*** mwagner_afk has quit IRC11:31
*** aranjan has joined #openstack-neutron11:31
openstackgerritIhar Hrachyshka proposed openstack/neutron: lb: stop handling Havana device updates  https://review.openstack.org/19348611:31
openstackgerritIhar Hrachyshka proposed openstack/neutron: lb: avoid doing nova VIF work plumbing tap to qbr  https://review.openstack.org/19348511:31
openstackgerritPeter V. Saveliev proposed openstack/neutron-specs: VLAN aware VMs  https://review.openstack.org/9461211:33
*** markvoelker has quit IRC11:34
*** aranjan has quit IRC11:36
*** banix has joined #openstack-neutron11:42
*** jasondotstar has quit IRC11:43
*** thorst has joined #openstack-neutron11:45
*** abregman has joined #openstack-neutron11:46
*** banix has quit IRC11:47
*** trinaths has left #openstack-neutron11:47
*** hitalia has quit IRC11:50
*** boris-42 has quit IRC11:52
*** moshele has quit IRC11:56
*** mmdurrant has joined #openstack-neutron11:59
*** erkules_ is now known as erkules11:59
*** erkules has joined #openstack-neutron11:59
openstackgerritIhar Hrachyshka proposed openstack/neutron: cleanup openstack-common.conf and sync updated files  https://review.openstack.org/18911711:59
openstackgerritJakub Libosvar proposed openstack/neutron: Introduce connection testers module  https://review.openstack.org/19114711:59
openstackgerritJakub Libosvar proposed openstack/neutron: Add test cases to testing firewall drivers  https://review.openstack.org/19114811:59
openstackgerritPavel Bondar proposed openstack/neutron: Move _delete_port  https://review.openstack.org/19310612:00
*** markvoelker has joined #openstack-neutron12:05
SridharGbradjones: ianw: hello, I have a question regarding the MTU selection and advt support in Neutron.12:10
*** ajo\picksupfood is now known as ajo12:10
SridharGbradjones: ianw: I hope Phase1 and 2 changes are currently implemented.12:12
*** coolsvap|away is now known as coolsvap12:12
*** yatin has quit IRC12:13
SridharGbradjones: ianw: according to the spec "Correct the network control and plugging code to correctly set the MTU on all software network elements that respect it" seems to be part of Phase1. Looking at the code, i feel this is not yet implemented. Is my understanding correct or am i missing something?12:14
*** yamamoto has quit IRC12:19
openstackgerritPeter V. Saveliev proposed openstack/neutron-specs: VLAN aware VMs  https://review.openstack.org/9461212:20
*** Sanjeev has joined #openstack-neutron12:21
*** coolsvap is now known as coolsvap|away12:22
*** Sanjeev has quit IRC12:24
*** jbell8 has joined #openstack-neutron12:27
*** Marga_ has joined #openstack-neutron12:29
*** rossella_s has quit IRC12:30
*** rossella_s has joined #openstack-neutron12:31
*** jbell8 has quit IRC12:31
openstackgerritPavel Bondar proposed openstack/neutron: Add request factory for pluggable IPAM  https://review.openstack.org/19266312:32
*** aranjan has joined #openstack-neutron12:32
*** sudipto has quit IRC12:32
*** coolsvap|away is now known as coolsvap12:34
*** aranjan has quit IRC12:37
*** marrusl has joined #openstack-neutron12:37
*** Marga_ has quit IRC12:39
*** Marga_ has joined #openstack-neutron12:40
*** raildo has joined #openstack-neutron12:41
*** Longgeek has quit IRC12:41
*** Longgeek has joined #openstack-neutron12:42
*** jasondotstar has joined #openstack-neutron12:43
*** dims has quit IRC12:46
*** dims has joined #openstack-neutron12:46
*** zhenguo has quit IRC12:51
*** jaypipes has quit IRC12:51
openstackgerritElena Ezhova proposed openstack/neutron-fwaas: Switch to oslo.service  https://review.openstack.org/19345312:53
openstackgerritElena Ezhova proposed openstack/neutron-vpnaas: Switch to oslo.service  https://review.openstack.org/19345512:54
*** yamamoto has joined #openstack-neutron12:57
*** yamamoto has quit IRC12:57
*** yamamoto has joined #openstack-neutron12:57
*** Longgeek has quit IRC12:57
*** Longgeek has joined #openstack-neutron12:59
*** coolsvap is now known as coolsvap|away12:59
*** numan has quit IRC13:00
*** dboik has joined #openstack-neutron13:03
openstackgerritCyril Roelandt proposed openstack/neutron: Python3: use dict.keys() instead of dict.iterkeys()  https://review.openstack.org/19352413:03
*** thurloat_isgone is now known as thurloat13:04
*** julim has joined #openstack-neutron13:05
*** sdake has joined #openstack-neutron13:05
*** HenryG has quit IRC13:05
*** dboik_ has joined #openstack-neutron13:05
*** HenryG has joined #openstack-neutron13:05
*** HenryG has quit IRC13:06
*** julim has quit IRC13:07
*** sdake_ has joined #openstack-neutron13:07
*** HenryG has joined #openstack-neutron13:07
*** dane_leblanc has joined #openstack-neutron13:07
*** pbrooko has joined #openstack-neutron13:08
*** julim has joined #openstack-neutron13:08
*** dboik has quit IRC13:08
*** devvesa has quit IRC13:08
*** claudiub has quit IRC13:08
*** sdake has quit IRC13:10
openstackgerritCyril Roelandt proposed openstack/neutron: Python3: do not use im_self/im_func/func_closure  https://review.openstack.org/19347613:11
*** vijaykc4 has quit IRC13:11
*** baoli has joined #openstack-neutron13:14
*** AndroUser has joined #openstack-neutron13:15
*** sudipto has joined #openstack-neutron13:16
*** baoli_ has joined #openstack-neutron13:16
*** yamamoto has quit IRC13:17
*** baoli has quit IRC13:19
*** sdake has joined #openstack-neutron13:20
*** achanda has joined #openstack-neutron13:23
*** sdake_ has quit IRC13:24
*** amuller has joined #openstack-neutron13:26
*** jaypipes has joined #openstack-neutron13:26
openstackgerritElena Ezhova proposed openstack/neutron: Switch to oslo.service  https://review.openstack.org/19299913:26
*** dguerri` is now known as dguerri13:26
*** AndroUser has quit IRC13:27
*** yamamoto has joined #openstack-neutron13:27
*** devvesa has joined #openstack-neutron13:29
*** abregman has quit IRC13:30
*** abregman has joined #openstack-neutron13:30
*** dannywilson has joined #openstack-neutron13:32
*** aranjan has joined #openstack-neutron13:33
eezhovaihrachyshka, could you help me one more time? How can I disable pylint checks for the neutron.common.rpc code that I copied to neutron_lbaas?13:33
rossella_samuller, HenryG hi! do you think we can merge https://review.openstack.org/#/c/140042/ ? tests ran 5 times successfully13:33
*** saju_m has joined #openstack-neutron13:34
ihrachyshkaeezhova, hm, why do you need to disable it?13:34
*** kiran-r has quit IRC13:34
eezhovaihrachyshka, there are pylint failures for it13:34
ihrachyshkaeezhova, can you show them to me?13:35
eezhovaihrachyshka, here https://jenkins05.openstack.org/job/gate-neutron-lbaas-pep8/271/console13:35
ihrachyshkaeezhova, there are ways to disable specific pylint checks, but I'd like to see them first13:35
*** ferest has joined #openstack-neutron13:35
*** shwetaap has joined #openstack-neutron13:35
HenryGrossella_s: I am OK with it. amuller what do you think?13:37
amullerHenryG: rossella_s: Let's merge it. If it starts failing too many times we'll have to revert it and figure out what's wrong.13:37
*** aranjan has quit IRC13:38
HenryGrossella_s: amuller: I see that ZZelle pointed out some possible duplicate id/ip issues. We can try to put in checks for dups if we see failures.13:38
ZZelleHenryG, could you point the review?13:39
HenryGZZelle: https://review.openstack.org/14004213:39
ihrachyshkaeezhova, hm, how about fixing those in neutron first? at least two of those seem legit.13:39
amullerrossella_s: HenryG: ZZelle: Rossella, I think Cedric's points are valid and should take a couple of mins to fix13:40
*** dguerri is now known as dguerri`13:40
amullermaybe we'd want to merge that version instead13:40
rossella_samuller: ok, let me update it13:41
eezhovaihrachyshka, can I just fix these issues right in neutron_lbaas and make an independent patch to neutron? Otherwise there will be yet another dependency :(13:42
ihrachyshkaeezhova, yeah, sure. I don't mind since we kill the file afterwards13:42
eezhovaihrachyshka, great, thanks!13:43
ihrachyshkaeezhova, btw http://docs.pylint.org/faq.html#is-it-possible-to-locally-disable-a-particular-message13:43
ihrachyshkaor even check 4.213:43
ihrachyshkait may be the easiest13:43
* pavel_bondar would like to bring some reviewer's attention to https://review.openstack.org/#/c/190623 since there were no new feedback for two days13:44
*** lnicolas has quit IRC13:44
ihrachyshkaor even #pylint: skip-file13:44
*** jckasper has joined #openstack-neutron13:44
*** prad has joined #openstack-neutron13:44
eezhovaihrachyshka, aha, thanks for a hint13:45
*** dguerri` is now known as dguerri13:45
*** dansmith is now known as superdan13:45
*** dannywilson has quit IRC13:46
*** dannywilson has joined #openstack-neutron13:46
*** yamamoto_ has joined #openstack-neutron13:47
*** igordcard_ has quit IRC13:47
*** rpothier has joined #openstack-neutron13:49
*** jaypipes is now known as leakypipes13:50
*** yamamoto has quit IRC13:51
*** beagles is now known as beagles_brb13:53
*** subscope has quit IRC13:56
*** shwetaap has quit IRC13:56
*** mwagner_afk has joined #openstack-neutron13:57
*** dims is now known as dimsum__13:57
*** merooney has quit IRC13:57
openstackgerritJakub Libosvar proposed openstack/neutron: Disable keepalived process in keepalived func test  https://review.openstack.org/19355813:57
openstackgerritCyril Roelandt proposed openstack/neutron: Python3: do not use '+' on dict_items objects  https://review.openstack.org/19355913:58
*** Marga_ has quit IRC13:58
*** merooney has joined #openstack-neutron14:01
*** dannywilson has quit IRC14:02
*** claudiub has joined #openstack-neutron14:02
*** fredlhsu has joined #openstack-neutron14:03
openstackgerritRossella Sblendido proposed openstack/neutron: Adds base in-tree functional testing of the ovs_neutron_agent  https://review.openstack.org/14004214:03
*** f13o has joined #openstack-neutron14:05
*** jagter has quit IRC14:06
*** morganfainberg is now known as morgan-devserver14:06
openstackgerritElena Ezhova proposed openstack/neutron-vpnaas: Switch to oslo.service  https://review.openstack.org/19345514:06
*** pbrooko has quit IRC14:06
*** vijaykc4 has joined #openstack-neutron14:07
*** lori is now known as lori|away14:07
*** roeyc has joined #openstack-neutron14:07
*** morgan-devserver is now known as CaptainMorgan14:08
*** jlibosva has quit IRC14:09
*** ihrachyshka has quit IRC14:09
*** jlibosva has joined #openstack-neutron14:09
*** johnbelamaric has joined #openstack-neutron14:09
*** ajmiller has joined #openstack-neutron14:10
*** beagles_brb is now known as beagles14:10
*** roeyc has quit IRC14:12
*** pbrooko has joined #openstack-neutron14:13
*** ronis has quit IRC14:14
*** localloop127 has joined #openstack-neutron14:14
*** krtaylor has quit IRC14:15
*** shwetaap has joined #openstack-neutron14:15
*** vijaykc4 has quit IRC14:16
*** aranjan has joined #openstack-neutron14:16
*** pbrooko has quit IRC14:17
openstackgerritElena Ezhova proposed openstack/neutron: Switch to oslo.service  https://review.openstack.org/19299914:17
*** yfried_ has quit IRC14:20
*** ferest has quit IRC14:20
*** zz_jgrimm is now known as jgrimm14:20
*** akamyshnikova1 has joined #openstack-neutron14:21
*** Networkn3rd has joined #openstack-neutron14:27
*** r-daneel has joined #openstack-neutron14:27
*** archers has joined #openstack-neutron14:29
openstackgerritCyril Roelandt proposed openstack/neutron: Python3: use itertools.zip instead of itertools.izip  https://review.openstack.org/19357214:29
*** alaski is now known as lascii14:29
*** josecastroleon has quit IRC14:31
*** carl_baldwin has joined #openstack-neutron14:33
*** chem has joined #openstack-neutron14:34
*** dboik_ has quit IRC14:34
*** jasondotstar has quit IRC14:35
*** yamahata has joined #openstack-neutron14:35
*** shwetaap has quit IRC14:35
*** akamyshnikova1 has quit IRC14:35
*** akamyshnikova1 has joined #openstack-neutron14:35
*** achanda has quit IRC14:36
*** jasondotstar has joined #openstack-neutron14:37
*** carl_baldwin has quit IRC14:37
*** yamahata has quit IRC14:38
*** yamahata has joined #openstack-neutron14:38
*** Marga_ has joined #openstack-neutron14:38
openstackgerritOleg Bondarev proposed openstack/neutron: Fix l3 agent to not create already deleted router  https://review.openstack.org/18351914:40
*** carl_baldwin has joined #openstack-neutron14:40
*** carl_baldwin has quit IRC14:41
*** roeyc has joined #openstack-neutron14:43
*** rushil has joined #openstack-neutron14:43
*** dguerri is now known as dguerri`14:44
*** tomoe_ has joined #openstack-neutron14:44
*** yamahata has quit IRC14:44
*** krtaylor has joined #openstack-neutron14:45
*** yamahata has joined #openstack-neutron14:45
*** carl_baldwin has joined #openstack-neutron14:46
openstackgerritBrian Haley proposed openstack/neutron-specs: [WIP] "Get me a network" spec  https://review.openstack.org/18485714:46
*** carl_baldwin has quit IRC14:47
*** Marga_ has quit IRC14:47
*** Marga_ has joined #openstack-neutron14:47
*** yamamoto_ has quit IRC14:48
*** carl_baldwin has joined #openstack-neutron14:48
*** archers has quit IRC14:50
*** dguerri` is now known as dguerri14:50
openstackgerritOleg Bondarev proposed openstack/neutron: l3 agent: do router cleanup for unknown routers  https://review.openstack.org/19062614:50
*** spandhe has joined #openstack-neutron14:51
*** daneyon has joined #openstack-neutron14:52
*** absubram has joined #openstack-neutron14:52
*** yamamoto has joined #openstack-neutron14:53
*** f13o has quit IRC14:53
*** krtaylor has quit IRC14:53
*** Longgeek has quit IRC14:53
*** f13o has joined #openstack-neutron14:54
*** Longgeek has joined #openstack-neutron14:54
*** absubram has quit IRC14:54
*** absubram has joined #openstack-neutron14:55
HenryGjlibosva: ping14:57
jlibosvaHenryG: hi14:57
*** Bhavya has joined #openstack-neutron14:57
HenryGjlibosva: sorry for always turning to you for grenade questions ...14:57
jlibosvaHenryG: np, I'm happy to help14:57
HenryGjlibosva: Is the grenade job very unstable? I see it failing often14:58
jlibosvaHenryG: I noticed that this morning too. But I haven't looked into it yet.14:58
*** lukasa has quit IRC14:58
HenryGjlibosva: OK. I poked around in a few failing logs but as usual I could not find much14:59
*** daneyon_ has joined #openstack-neutron14:59
*** carl_baldwin has quit IRC14:59
jlibosvaHenryG: can you send me a link please to any failing job? I'll try to have a look too15:00
*** mlavalle has joined #openstack-neutron15:00
*** Marga_ has quit IRC15:00
HenryGjlibosva: this one I just looked at: https://review.openstack.org/19352415:00
jlibosvaHenryG: thanks15:00
jlibosvaHenryG: hm, I see tempest failing15:00
*** TrevorV has joined #openstack-neutron15:00
*** Marga_ has joined #openstack-neutron15:01
*** zhhuabj has joined #openstack-neutron15:01
HenryGjlibosva: oh, right. Hang on, let me look for another one from last night15:02
jlibosvaHenryG: for me it doesn't seem like Neutron issue. Do you have any info if other grenade jobs that don't use neutron fail too?15:02
*** daneyon has quit IRC15:02
* jlibosva thinks there is a page with job statistics but can't find it15:02
HenryGjlibosva: try this one,  http://logs.openstack.org/70/193370/1/check/check-grenade-dsvm-neutron/2a15551/15:03
*** zzzeek has joined #openstack-neutron15:03
*** amuller is now known as amuller_afk15:03
*** agireud has joined #openstack-neutron15:04
openstackgerritPaul Michali proposed openstack/neutron-vpnaas: VPNaaS: DevRef for multiple local subnets  https://review.openstack.org/19194415:04
*** carl_baldwin has joined #openstack-neutron15:05
*** saju_m has quit IRC15:05
*** yamamoto has quit IRC15:06
*** jbell8 has joined #openstack-neutron15:06
jlibosvaHenryG: I would say it has to do something either with glance or keystone15:06
jlibosvaHenryG: openstack image list --property status=active15:07
jlibosvaERROR: openstack Unauthorized (HTTP 401)15:07
*** jbell8 has quit IRC15:07
*** jbell8 has joined #openstack-neutron15:07
HenryGjlibosva: ok, thanks15:07
*** Bhavya has quit IRC15:07
jlibosvaHenryG: I can't find keystone logs there, can you see it? I might be blind15:08
HenryGjlibosva: I am blinder than you15:08
jlibosvaHenryG: this is in glance logs http://logs.openstack.org/70/193370/1/check/check-grenade-dsvm-neutron/2a15551/logs/new/screen-g-api.txt.gz#_2015-06-19_00_52_34_46715:09
*** rushil has quit IRC15:09
*** banix has joined #openstack-neutron15:09
openstackgerritSalvatore Orlando proposed openstack/neutron: Add DB support for resource usage tracking  https://review.openstack.org/18860815:09
HenryGjlibosva: then other projects should be having gate issues also?15:09
openstackgerritMerged openstack/neutron: DHCP agent: Set an "ipxe" tag to work with Ironic  https://review.openstack.org/17204015:10
jlibosvaHenryG: that's what I asked, I don't know where can I find this info15:10
*** carl_baldwin has quit IRC15:10
jlibosvaHenryG: maybe kibana can help us15:10
*** dboik has joined #openstack-neutron15:10
*** daneyon_ has quit IRC15:12
*** armax has joined #openstack-neutron15:13
*** yamamoto has joined #openstack-neutron15:13
HenryGotherwiseguy: ping15:13
jlibosvaHenryG: in last 48h it happned 8 times and all of failures were on neutron grenade, so seems like our fault15:13
openstackgerritMerged openstack/neutron: Python3: do not use im_self/im_func/func_closure  https://review.openstack.org/19347615:13
HenryGjlibosva: how can neutron cause keystone failures????15:14
* jlibosva shrugs15:14
jlibosvaHenryG: but when I was working with grenade I saw really weird influences there15:15
jlibosvaHenryG: so I found out keystone logs are hidden in apache directory :) I'm not very familiar with keystone logs http://logs.openstack.org/70/193370/1/check/check-grenade-dsvm-neutron/2a15551/logs/apache/keystone.txt.gz15:19
jlibosvaHenryG: for me it seems like keystone is trying to start in a loop and fails at the end15:20
*** Bhavya has joined #openstack-neutron15:21
openstackgerritPavel Bondar proposed openstack/neutron: Decompose _create_subnet_from_pool  https://review.openstack.org/19359515:21
HenryGjlibosva: shall we hop over to the keystone channel and ask them to help us look for clues in the log?15:21
jlibosvaHenryG: I just did :)15:21
openstackgerritMerged openstack/neutron: Fix 'router_gateway' port status can't be updated  https://review.openstack.org/15150115:23
*** eandersson has quit IRC15:23
*** yamamoto has quit IRC15:23
jlibosvaHenryG: it's https://bugs.launchpad.net/keystone/+bug/146689315:23
openstackLaunchpad bug 1466485 in Keystone "duplicate for #1466893 keystone fails with: ArgsAlreadyParsedError: arguments already parsed: cannot register CLI option" [Critical,Confirmed]15:23
*** sudipto_ has joined #openstack-neutron15:24
HenryGjlibosva: yep, I saw that. Thanks!15:24
jlibosvaHenryG: and by the time I joined channel there was an ongoing discussion on this matter15:24
HenryGindeed15:24
*** sudipto has quit IRC15:24
*** roeyc has quit IRC15:26
*** jasondotstar has quit IRC15:26
*** rushil has joined #openstack-neutron15:26
*** absubram has quit IRC15:26
jlibosvaHenryG: np, have a great weekend :)15:27
HenryGjlibosva: You too! Thanks again.15:27
*** jasondotstar has joined #openstack-neutron15:28
openstackgerritMerged openstack/neutron: Fix cisco_csr_identifier_map.ipsec_site_conn_id  https://review.openstack.org/19015415:28
openstackgerritMerged openstack/neutron: fix DHCP port changed when dhcp-agent restart  https://review.openstack.org/16252215:29
*** edmondsw has joined #openstack-neutron15:30
*** aranjan has quit IRC15:32
*** mitchjameson has joined #openstack-neutron15:32
*** mitchjameson has quit IRC15:33
*** wwriverrat has joined #openstack-neutron15:34
*** yamamoto has joined #openstack-neutron15:35
*** carl_baldwin has joined #openstack-neutron15:35
openstackgerritMerged openstack/python-neutronclient: Updated from global requirements  https://review.openstack.org/19238815:37
vishwanathjamotoki, hi15:38
openstackgerritMerged openstack/neutron-fwaas: Updated from global requirements  https://review.openstack.org/19040615:38
*** carl_baldwin has quit IRC15:38
*** abregman has quit IRC15:38
*** _cjones_ has joined #openstack-neutron15:38
otherwiseguyHenryG: pong15:38
*** mitchjameson has joined #openstack-neutron15:39
amotoki_vishwanathj: hi15:39
*** brad[] has joined #openstack-neutron15:40
*** dsanders has joined #openstack-neutron15:40
vishwanathjamotoki_, it would be great if you could review my updates and the test cases for the patch set https://review.openstack.org/#/c/190336/15:40
vishwanathjand provide your feedback15:40
*** thurloat is now known as thurloat_isgone15:41
HenryGotherwiseguy: I looked at the ovs python package15:41
otherwiseguyHenryG: and noticed how very not python3-friendly it is? :)15:41
vishwanathjamotoki_, also, for the bug https://bugs.launchpad.net/neutron/+bug/1465440 , should we handle resolution in Horizon instead of neutron?15:41
openstackLaunchpad bug 1465440 in neutron "Firewall attribute "Shared" is set to None by default instead of 'False'" [High,Confirmed] - Assigned to vishwanath jayaraman (vishwanathj)15:41
HenryGotherwiseguy: that's what I was going to say :)15:41
amotoki_vishwanathj: looking the review15:42
vishwanathjamotoki_, thanks15:42
otherwiseguyHenryG: The reason is that they have been trying to support Python 2.4+. There is a patch to move that to 2.7+. I have some preliminary work for py3-compatibility (it just fixes the stuff that running setup.py finds). Plan on doing full support soon.15:42
otherwiseguyHenryG: Apparently Xen ships with py2.4 or something. But if you install py2.7 it works as well.15:43
amotoki_vishwanathj: re bug 1465440, after reading comment from Sumit, it should be fixed in Horizon.15:44
openstackbug 1465440 in neutron "Firewall attribute "Shared" is set to None by default instead of 'False'" [High,Confirmed] https://launchpad.net/bugs/1465440 - Assigned to vishwanath jayaraman (vishwanathj)15:44
otherwiseguyHenryG: Doing both py2.4 and py3 in the same tree is, if not impossible, very hard.15:44
HenryGotherwiseguy: OK, sounds like you are on top of this15:44
amotoki_vishwanathj: in neutron fwaas, it may be nice to have a comment to reflect Sumit's comment. It would help other developers.15:44
otherwiseguyHenryG: Trying to be, anyway. :)15:44
HenryGotherwiseguy: I don't want to derail Neutron's py3k efforts15:45
vishwanathjamotoki_, that makes sense, should I also refer to the bug in the code comments15:45
HenryGotherwiseguy: Let me know if I can help on the ovs side15:45
*** daneyon has joined #openstack-neutron15:46
amotoki_vishwanathj: related-bug loosk good to me, or we can just mention the bug in the commit message.15:46
openstackgerritMerged openstack/neutron: Python3: do not use '+' on dict_items objects  https://review.openstack.org/19355915:47
*** tomoe_ has quit IRC15:47
otherwiseguyHenryG: Will do. So far: http://openvswitch.org/pipermail/dev/2015-May/055202.html and http://openvswitch.org/pipermail/dev/2015-May/055310.html15:47
*** krtaylor has joined #openstack-neutron15:48
ajohi armax, morning ;)15:49
ajodo you have time to chat about the remote-callback thing? :)15:49
armaxajo: moring15:49
armaxajo: going into a meeting now :(15:50
armaxand I have 3 meetings back-to-back :(15:50
ajoarmax, ack, I will update the spec then, and let's keep discussion over there15:50
armaxajo: we should sync up here15:50
armaxthough15:50
ajoarmax: I'll try to reconnect at night when I get the beasts (kids) to sleep15:50
armaxajo: what suits you best15:51
*** kiran-r has joined #openstack-neutron15:51
ajoarmax, otherwise I'll retry on monday afternoon ;)15:51
armaxajo: thanks15:51
armaxajo: aye15:51
ajothank you armax15:51
armaxthank you ajo!15:52
*** ygbo has quit IRC15:52
*** thurloat_isgone is now known as thurloat15:52
amotoki_vishwanathj: one question on status INACTIVE.15:54
vishwanathjamotoki_, go ahead15:54
*** yog_ has quit IRC15:55
amotoki_vishwanathj: when update_firewall is called and router_ids or firewall['router_ids'] are empty, status stays to INACTIVE when reaching to L.34815:55
* pavel_bondar is looking for the second core reviewer for https://review.openstack.org/#/c/193075/ have already one +215:55
amotoki_vishwanathj: is it a possible case?15:56
*** hitalia has joined #openstack-neutron15:56
*** alex_klimov has quit IRC15:56
*** dsanders has quit IRC15:56
vishwanathjamotoki_, it might so happen that an end user creates a firewall with associated policy with rules and no routers attached.....15:57
*** jlibosva has quit IRC15:58
vishwanathjlater the end user updates either the firewall rule attributes or firewall policy attributes and the call will finally propogate to the update_firewall hitting the L325 condition being false....15:59
*** dsanders has joined #openstack-neutron15:59
*** lascii is now known as alaski15:59
vishwanathjat which time the firewall status has to continue to be INACTIVE16:00
*** tongl has joined #openstack-neutron16:00
amotoki_vishwanathj: so, does "if router_ids or firewall['router_ids']:  <current logic>  else: status = INACTIVE" work?16:00
openstackgerritSalvatore Orlando proposed openstack/neutron: Introduce usage data tracking for Neutron  https://review.openstack.org/18888316:01
openstackgerritSalvatore Orlando proposed openstack/neutron: Enable resource usage tracking for reference plugins.  https://review.openstack.org/18888416:01
openstackgerritSalvatore Orlando proposed openstack/neutron: Create packages for quota modules  https://review.openstack.org/18861816:01
*** tongl has quit IRC16:01
sc68calHey, has anyone else noticed that sometimes the check-grenade-dsvm-neutron fails - http://logstash.openstack.org/#eyJzZWFyY2giOiJtZXNzYWdlOlwiRm91bmQgbm8gdmFsaWQgaW1hZ2VzIHRvIHVzZSFcIiBmaWxlbmFtZTpcImdyZW5hZGUuc2gudHh0Lmd6XCIiLCJmaWVsZHMiOltdLCJvZmZzZXQiOjAsInRpbWVmcmFtZSI6IjE0NDAwIiwiZ3JhcGhtb2RlIjoiY291bnQiLCJ0aW1lIjp7InVzZXJfaW50ZXJ2YWwiOjB9LCJzdGFtcCI6MTQzNDcyOTcwOTU5OH0= ?16:01
*** pask81 has quit IRC16:02
*** john-davidge has quit IRC16:02
*** dsanders has quit IRC16:02
*** jlanoux_ has quit IRC16:02
vishwanathjamotoki_, yes it would work, having the default status set to INACTIVE avoids having the else statement16:02
*** yamahata has quit IRC16:03
amotoki_vishwanathj: setting the default status value means that status will be kept unless any logic touches it.16:04
*** tongl has joined #openstack-neutron16:05
amotoki_vishwanathj: in this case we have more clear condition to set the status to INACTIVE. Perhaps it is the reason I was a bit confused and it lead to my question.16:05
*** hitalia has quit IRC16:06
vishwanathjamotoki_, what is your recommendation?16:07
amotoki_vishwanathj: my vote is to use if-else. I am now writing a comment in the review.16:07
vishwanathjamotoki_, sounds good16:08
*** nlahouti has joined #openstack-neutron16:09
*** carl_baldwin has joined #openstack-neutron16:10
*** dannywilson has joined #openstack-neutron16:11
*** crose has joined #openstack-neutron16:11
amotoki_vishwanathj: thanks for your patience.16:11
vishwanathjamotoki_, appreciate your time and comments, thanks. I will shortly upload a new patch16:12
amotoki_vishwanathj: at now, we have only two fwaas cores, so you need to grab both. I am not a core of fwaas now.16:13
vishwanathjwho are the two fwaas cores?16:14
*** apuimedo has joined #openstack-neutron16:14
amotoki_vishwanathj: mestery and dougwig16:14
vishwanathjamotoki_, I will approach them after I address your comments and get a +1 from you16:14
mesteryvishwanathj amotoki_: Thanks!16:15
*** carl_baldwin has quit IRC16:16
*** merooney has quit IRC16:16
*** anilvenkata has left #openstack-neutron16:17
*** spandhe has quit IRC16:19
*** kiran-r has quit IRC16:21
openstackgerritSridhar Gaddam proposed openstack/neutron: Wait for radvd process to become active  https://review.openstack.org/19362416:21
*** mitchjameson has quit IRC16:23
*** achanda has joined #openstack-neutron16:23
*** SridharG has left #openstack-neutron16:23
*** ildikov_ has quit IRC16:23
*** roeyc has joined #openstack-neutron16:24
openstackgerritvishwanath jayaraman proposed openstack/neutron-fwaas: Fixes firewall going to error state on an update  https://review.openstack.org/19033616:26
*** sridhar_ram has joined #openstack-neutron16:26
*** sdake_ has joined #openstack-neutron16:27
*** raildo has quit IRC16:29
*** dguerri is now known as dguerri`16:29
*** aranjan has joined #openstack-neutron16:29
*** rossella_s has quit IRC16:30
*** rossella_s has joined #openstack-neutron16:30
*** raildo has joined #openstack-neutron16:30
*** sdake has quit IRC16:31
*** dannywilson has quit IRC16:31
*** devvesa has quit IRC16:31
*** dannywilson has joined #openstack-neutron16:32
*** kiran-r has joined #openstack-neutron16:32
*** Djinnn has joined #openstack-neutron16:32
*** fedexo has joined #openstack-neutron16:32
*** roeyc has quit IRC16:33
openstackgerritKyle Mestery proposed openstack/neutron: DHCP: Add default DNS servers to Neutron DHCP agent  https://review.openstack.org/19271216:34
*** Djinnn has quit IRC16:36
*** yamamoto has quit IRC16:36
*** Djinnn has joined #openstack-neutron16:36
*** dannywilson has quit IRC16:36
*** roeyc has joined #openstack-neutron16:37
*** rm_work is now known as rm_work|away16:37
*** fredlhsu has quit IRC16:38
*** sthillma has joined #openstack-neutron16:38
openstackgerritTerry Wilson proposed openstack/neutron: Add support for PluginWorker and Process creation notification  https://review.openstack.org/18939116:39
*** edmondsw has quit IRC16:39
*** thurloat is now known as thurloat_isgone16:42
*** zhhuabj has quit IRC16:43
openstackgerritSalvatore Orlando proposed openstack/neutron: Enable resource usage tracking for reference plugins.  https://review.openstack.org/18888416:44
*** dguerri` is now known as dguerri16:45
*** dguerri is now known as dguerri`16:46
*** tflynn has joined #openstack-neutron16:47
*** aranjan has quit IRC16:47
*** aranjan has joined #openstack-neutron16:47
*** roeyc has quit IRC16:47
*** ZZelle_ has joined #openstack-neutron16:48
*** safchain has quit IRC16:48
*** tflynn_ has joined #openstack-neutron16:48
*** f13o has quit IRC16:49
*** sthillma has quit IRC16:50
*** ZZelle__ has joined #openstack-neutron16:51
*** tflynn_ has quit IRC16:51
*** tflynn has quit IRC16:51
*** ijw_ has quit IRC16:52
*** sdake has joined #openstack-neutron16:52
*** tflynn has joined #openstack-neutron16:52
*** roeyc has joined #openstack-neutron16:52
*** Marga_ has quit IRC16:53
*** yamahata has joined #openstack-neutron16:53
*** Marga_ has joined #openstack-neutron16:53
*** rm_work|away is now known as rm_work16:53
*** thedodd has joined #openstack-neutron16:54
*** thurloat_isgone is now known as thurloat16:55
*** sdake_ has quit IRC16:55
*** raildo has quit IRC16:55
*** numan has joined #openstack-neutron16:56
*** raildo has joined #openstack-neutron16:56
*** Swami has joined #openstack-neutron16:57
*** bradjones has quit IRC16:58
*** bradjones has joined #openstack-neutron16:58
*** bradjones has quit IRC16:58
*** bradjones has joined #openstack-neutron16:58
*** Djinnn_ has joined #openstack-neutron16:59
*** yamahata has quit IRC16:59
*** mwagner_afk has quit IRC16:59
*** Marga_ has quit IRC17:00
*** Marga_ has joined #openstack-neutron17:01
*** fredlhsu has joined #openstack-neutron17:02
*** Djinnn has quit IRC17:02
*** rushil has quit IRC17:02
*** jagter has joined #openstack-neutron17:04
*** sdake_ has joined #openstack-neutron17:05
vishwanathjamotoki_, hi, with regards to the bug https://bugs.launchpad.net/neutron/+bug/1465440, I still think that in neutron-fwaas code we would need to update the db with the 'shared' attribute value? What are your thoughts? Thanks17:05
openstackLaunchpad bug 1465440 in neutron "Firewall attribute "Shared" is set to None by default instead of 'False'" [High,Confirmed] - Assigned to vishwanath jayaraman (vishwanathj)17:05
openstackgerritMartin Roy proposed openstack/neutron: Use string exception casting everywhere  https://review.openstack.org/19324517:09
*** sdake has quit IRC17:09
leakypipessc68cal: nice work on the LB devstack stuff.17:09
*** merooney has joined #openstack-neutron17:09
*** Djinnn_ has quit IRC17:10
sc68calleakypipes: thanks :) - although like I said in the e-mail, it's mostly work that I borrowed from Jens17:10
* sc68cal was like the ape in the movie 2001 - touching the monolith17:11
leakypipeshehe17:11
*** bradjones has quit IRC17:11
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Fix FloatingIP Namespace creation in DVR for Late Binding  https://review.openstack.org/17750717:12
*** ijw_ has joined #openstack-neutron17:12
*** bradjones has joined #openstack-neutron17:12
*** bradjones has quit IRC17:12
*** bradjones has joined #openstack-neutron17:12
*** Bhavya has quit IRC17:13
*** rushil has joined #openstack-neutron17:14
*** claudiub has quit IRC17:15
*** kawa2014 has quit IRC17:15
*** roeyc has quit IRC17:18
Swamiobondarev: hi17:18
*** spandhe has joined #openstack-neutron17:19
*** idevops has joined #openstack-neutron17:21
*** dsanders has joined #openstack-neutron17:22
*** lpetrut has quit IRC17:23
*** idevops has quit IRC17:23
*** shivrao has joined #openstack-neutron17:24
anteayasc68cal: you just have the visuals today17:24
anteayasc68cal: laughed for a full minute after you and monty did an outtake from gone with the wind17:24
Swamifolks, can anyone help me in resolving the "tox" issue after the "Liberty" version change in setup.cfg.17:25
*** aranjan_ has joined #openstack-neutron17:25
*** aranjan has quit IRC17:25
*** sthillma has joined #openstack-neutron17:26
Swamirkukura_: ping17:27
*** Marga_ has quit IRC17:27
*** diegows has joined #openstack-neutron17:29
*** Marga_ has joined #openstack-neutron17:29
*** Djinnn has joined #openstack-neutron17:31
*** rushil has quit IRC17:31
*** rushil has joined #openstack-neutron17:32
*** ZZelle_ has quit IRC17:32
*** jagter has quit IRC17:33
*** ZZelle__ has quit IRC17:33
*** jagter has joined #openstack-neutron17:33
*** SumitNaiksatam has quit IRC17:36
*** yamamoto has joined #openstack-neutron17:37
*** akamyshnikova1 has quit IRC17:39
openstackgerritKyle Mestery proposed openstack/neutron: DHCP: Add default DNS servers to Neutron DHCP agent  https://review.openstack.org/19271217:39
*** mitchjameson has joined #openstack-neutron17:39
*** yamamoto has quit IRC17:42
*** ronis has joined #openstack-neutron17:42
*** breitz has quit IRC17:42
*** breitz has joined #openstack-neutron17:42
*** agireud has quit IRC17:44
*** akamyshnikova1 has joined #openstack-neutron17:44
*** Longgeek has quit IRC17:45
*** ildikov has joined #openstack-neutron17:48
*** jwy has joined #openstack-neutron17:50
*** amuller_afk is now known as amuller17:51
Swamiamuller: do you have a minute17:52
*** ferest has joined #openstack-neutron17:55
*** jasondotstar has quit IRC17:56
*** mickeys has joined #openstack-neutron17:59
*** jasondot_ has joined #openstack-neutron17:59
*** SumitNaiksatam has joined #openstack-neutron18:01
*** jprovazn has quit IRC18:01
*** prad_ has joined #openstack-neutron18:01
openstackgerritElena Ezhova proposed openstack/neutron-fwaas: Switch to oslo.service  https://review.openstack.org/19345318:03
*** sridhar_ram1 has joined #openstack-neutron18:04
amullerSwami: just entered an hour long meeting18:04
Swamiamuller: just wanted to check with you on the status of the patch that you uploaded for the DVR functional tests.18:04
*** sridhar_ram has quit IRC18:05
Swamiamuller: #link https://review.openstack.org/#/c/178880/18:05
amullerSwami: yeah I have to get back to that18:07
*** thurloat is now known as thurloat_isgone18:07
Swamiamuller: Let me know when we can merge that patch. So that I can start writing my tests based on that file.18:08
amullerSwami: OK I'll try to get back to that patch sooner rather than later18:09
Swamiamuller: thanks that would help.18:09
*** jagter has quit IRC18:09
*** dannywilson has joined #openstack-neutron18:10
*** dannywilson has quit IRC18:10
*** dannywilson has joined #openstack-neutron18:10
*** s3wong has joined #openstack-neutron18:11
*** jagter has joined #openstack-neutron18:15
*** hitalia has joined #openstack-neutron18:16
*** thurloat_isgone is now known as thurloat18:16
*** alexpilotti has quit IRC18:18
*** Marga_ has quit IRC18:19
*** neophy has joined #openstack-neutron18:20
*** Marga_ has joined #openstack-neutron18:20
*** mickeys has quit IRC18:22
*** Djinnn has quit IRC18:22
*** tflynn has quit IRC18:27
*** aranjan_ has quit IRC18:28
*** kiran-r has quit IRC18:29
*** agireud has joined #openstack-neutron18:30
*** mwagner_afk has joined #openstack-neutron18:31
*** SumitNaiksatam has quit IRC18:32
*** roeyc has joined #openstack-neutron18:33
*** SumitNaiksatam has joined #openstack-neutron18:33
*** roeyc has quit IRC18:33
*** sudipto_ has quit IRC18:34
*** lpetrut has joined #openstack-neutron18:37
openstackgerritMichael Bayer proposed openstack/neutron-specs: Propose scripted online schema migrations  https://review.openstack.org/19293718:38
*** diegows has quit IRC18:39
*** merooney has quit IRC18:39
*** shwetaap has joined #openstack-neutron18:42
*** nlahouti has quit IRC18:43
*** carl_baldwin has joined #openstack-neutron18:45
*** rushil has quit IRC18:45
*** Longgeek has joined #openstack-neutron18:46
HenryGsc68cal: the grenade failures are keystone related, see bug 1466893 and/or bug 146648518:47
openstackbug 1466485 in Keystone "duplicate for #1466893 keystone fails with: ArgsAlreadyParsedError: arguments already parsed: cannot register CLI option" [Critical,Incomplete] https://launchpad.net/bugs/146648518:47
openstackbug 1466485 in Keystone "keystone fails with: ArgsAlreadyParsedError: arguments already parsed: cannot register CLI option" [Critical,Incomplete] https://launchpad.net/bugs/146648518:47
*** Marga_ has quit IRC18:48
openstackgerritElena Ezhova proposed openstack/neutron: Switch to oslo.service  https://review.openstack.org/19299918:51
*** jagter has quit IRC18:51
*** diegows has joined #openstack-neutron18:51
*** rushil has joined #openstack-neutron18:51
*** diegows has quit IRC18:52
*** merooney has joined #openstack-neutron18:54
*** agireud has quit IRC18:54
*** ZZelle_ has joined #openstack-neutron18:54
*** neophy has quit IRC18:55
*** nlahouti has joined #openstack-neutron18:57
*** agireud has joined #openstack-neutron18:57
openstackgerritSwaminathan Vasudevan proposed openstack/neutron: Fix FloatingIP Namespace creation in DVR for Late Binding  https://review.openstack.org/17750718:59
*** regXboi has joined #openstack-neutron19:00
* regXboi pads in, checks the clock and looks for carl_baldwin...19:00
carl_baldwinregXboi: hi19:00
regXboicarl_baldwin: did you have a chance to look at stuff?19:01
regXboiI admit, I didn't get a chance to look at the mpe code as closely as I would have liked :(19:02
carl_baldwinregXboi: I did have a chance to look at it.19:03
carl_baldwinregXboi: I also made some updates to my spec based on your comments.19:03
regXboiI saw the updates - I was going to re-review over the weekend19:03
carl_baldwinHonestly, I don’t much commonality except in the term segment.19:03
regXboialong with a bunch of the other items19:04
carl_baldwinmpe seems to be about multiple L2 segments.  What I want is L3 segments with L3 boundaries.19:04
regXboiwill that be enough to confuse people?19:04
carl_baldwinYes, I’m sure it will do a nice job confusing people.  :)19:04
regXboiok, so let's break this down19:04
regXboifirst mpe vs the rfe19:05
regXboibecause my understanding of the rfe was that they wanted L2 segments19:05
carl_baldwinregXboi: Good to know.  My understanding was that they want L3 segments.19:05
* regXboi rereads the rfe19:05
carl_baldwinBut, maybe we’re just looking at it differently.19:06
regXboiugh19:06
regXboi" A "segment" could mean a L2 domain, IP block boundary, or other partition. "19:06
regXboiso the first part is done19:06
regXboiL2 domain = mpe19:06
carl_baldwinregXboi: That doesn’t jive with “IP block boundary"19:06
regXboiIP block boundary sounds like what you are doing19:07
regXboiyeah - it sounds like a catch all19:07
carl_baldwinEach segment is an L2 domain and L2 doesn’t cross the boundary.  On the other hand, my understanding of mpe is that L2 crosses the boundaries.19:07
*** ijw__ has joined #openstack-neutron19:07
regXboiwell... let's look at the examples19:08
carl_baldwinregXboi: ^ This is why I think of it as L3 segments.19:08
*** SumitNaiksatam_ has joined #openstack-neutron19:08
carl_baldwinklindgren has done some work on the etherpad which we should look at:  https://etherpad.openstack.org/p/Network_Segmentation_Usecases19:08
rkukura_Swami: did you ping me?19:09
Swamirkukura_: yes19:09
rkukura_Swami: what’s up?19:09
Swamirkukura_: I am seeing an issue with DVR when I try to update a dvr router iterface.19:09
* regXboi goes and looks19:09
*** jagter has joined #openstack-neutron19:09
openstackgerritMerged openstack/neutron-vpnaas: Trim some unused test requirements  https://review.openstack.org/19329419:10
Swamirkukura_: it seems that during update, it tries to call the port_update_postcommit, and while checking for the "status" it fails, since it is not returning the DVRContext, but the PortContext.19:10
openstackgerritMerged openstack/neutron: Python3: use dict.keys() instead of dict.iterkeys()  https://review.openstack.org/19352419:10
Swamirkukura_: check the bug description and the logs #link https://bugs.launchpad.net/neutron/+bug/146543419:10
openstackLaunchpad bug 1465434 in neutron "DVR issues with supporting multiple subnets per network on DVR routers" [Undecided,New] - Assigned to Swaminathan Vasudevan (swaminathan-vasudevan)19:10
*** SumitNaiksatam has quit IRC19:10
*** SumitNaiksatam_ is now known as SumitNaiksatam19:10
*** merooney has quit IRC19:10
*** ijw_ has quit IRC19:11
regXboicarl_baldwin: I'm not sure I agree with you, I think that for proper values of the provider_type mpe is L2 segments19:11
*** sridhar_ram1 has quit IRC19:11
regXboion the other hand, for proper values of the provider_type mpe is more like L3 segments :(19:12
carl_baldwinregXboi: The mpe model doesn’t work for what we want here.  The segments fit under a network and then the subnets associate with the network.  There is no way to associate a subnet with only one segment.  This is what I believe is the important distinction.19:12
rkukura_Swami: I’ll take a look19:12
Swamirkukura_: thanks.19:13
regXboicarl_baldwin: I'll buy that what you are doing is different from mpe19:13
regXboicarl_baldwin: I'm looking at mpe vs what the LDT was asking for19:13
carl_baldwinregXboi: The only conclusion I can draw is that mpe segments fit in the same L2 domain and then the L3 on top.19:13
*** sridhar_ram has joined #openstack-neutron19:14
regXboicarl_baldwin: then I guess my question is what is an mpe of type flat? bridged segments?19:14
*** apuimedo has quit IRC19:14
regXboioh lord19:15
*** yfauser has joined #openstack-neutron19:15
* regXboi reads the etherpad and understands why they want segments19:15
*** amit213 has joined #openstack-neutron19:16
*** yfauser1 has joined #openstack-neutron19:16
*** yfauser1 has left #openstack-neutron19:16
regXboicarl_baldwin: ok, I agree that these are all different, we'll just have to find terminology that doesn't overload the senses19:17
carl_baldwinklindgren:  Please confirm my understanding:  network is L3 only north of the TOR.  Each rack has its own L2 isolated from the other racks.  You want users to select the L3 network without having to know about racks/L2 segments/etc.19:18
klindgrenregXboi, - Ours (Godaddy) was the most extreem use case of wanting segments.19:18
*** ijw__ has quit IRC19:18
*** hitalia has quit IRC19:18
klindgrenOthers L2 segments sizes were larger - say 4-5 racks worth of gear19:19
carl_baldwinklindgren: Actually, your network topology with TOR accessing the L3 network is what I have been working on too.19:19
klindgrencarl_baldwin, correct.19:19
*** yfauser has quit IRC19:19
regXboiklindgren: I assume that you've looked at mpe and concluded it doesn't work for you?19:19
*** sdake_ has quit IRC19:19
*** bradjones has quit IRC19:20
klindgrenI assume MPE is the thing that Carl is working on for supporting routed networks?19:21
carl_baldwinregXboi: I agree we need to discuss the terminology.  Calling them both segments, at least without some other qualifier, is begging to confuse people.19:21
*** achanda has quit IRC19:21
regXboiklindgren: no, MPE is not what Carl is working on - it is already part of the Neutron API19:21
carl_baldwinklindgren: mpe is different.  It doesn’t work for my use case.19:21
*** jckasper has quit IRC19:22
carl_baldwinklindgren: mpe allows segmenting an L2 network across different types of overlays, I think.  The model provides no way to associate an IP block with an L2 segment.19:22
*** daneyon has quit IRC19:22
*** bradjones has joined #openstack-neutron19:22
*** bradjones has quit IRC19:22
*** bradjones has joined #openstack-neutron19:22
carl_baldwinklindgren: ^ This led me to conclude that the segments probably come together into a continuous L2 domain.19:22
*** ijw_ has joined #openstack-neutron19:23
carl_baldwinklindgren: Neither of us want continuous L2.  We want L3 routing between segments.19:23
regXboiwhat I see about DHCP running on each L2 in the etherpad makes me now agree with carl_baldwin19:23
*** nmagnezi has joined #openstack-neutron19:23
klindgrenok - that was my understanding - part of the reason why we went to ml2 in the first place was the idea that the same L2 could be provided by multiple ways.  We were hoping that we could extend that.  But the issue is we dont have a continuos layer2 - we have may many l2 zones - that all fit together via a routed network19:24
carl_baldwinklindgren: “many l2 zones - that all fit together via a routed network” describes what I’m after to.19:25
carl_baldwintoo.19:25
regXboicarl_baldwin: so it sounds like you need to generalize to not just external networks?19:25
* regXboi stops for a moment and muses that there is a difference19:26
carl_baldwinregXboi: right.  Provider networks too.19:26
*** achanda has joined #openstack-neutron19:27
*** merooney has joined #openstack-neutron19:27
regXboinevermind - there isn't a difference19:27
regXboiboth of these are not the undercloud19:27
klindgrenThe other thing is that for the most part we also don't use any of the other parts of neutron networking.  IE L3_agents19:27
*** rushil has quit IRC19:27
*** Sukhdev has joined #openstack-neutron19:28
regXboiok... I think I'm happy now - carl_baldwin, I'll look over the weekend and in the meantime, I'll think about a term other than segments19:28
carl_baldwinklindgren: Right, I imagine your VMs booting on the provider network with no Neutron virtual router.19:28
*** slogan621 has joined #openstack-neutron19:28
klindgrencarl_baldwin, exactly19:28
*** numan has quit IRC19:28
slogan621are there examples of plugins that directly (or indirectly) publish to the telemetry (ceilometer) notification bus that someone can point me at?19:29
*** merooney has quit IRC19:29
*** csoukup has joined #openstack-neutron19:29
carl_baldwinregXboi: I’m inclined to call them l3 segments.  A continuous L3 network that is segmented in to isolated L2 pieces.  It is very much like an mpe segment is a continuous L2 that is segmented in to isolated physical/encap pieces.19:29
regXboiok, does the model use "segements" anywhere? if so how about "l3-segments" instead19:30
regXboier "segments" that is19:30
regXboiif not, it's just terminology and we are good19:30
carl_baldwinregXboi: I’ll await your review.  I’m sure we’ll find something.  Thanks for the discussion.19:31
regXboicarl_baldwin: and thank you for taking the time19:31
* regXboi is much happier now :)19:32
* carl_baldwin is too19:32
carl_baldwinklindgren: Let me take another pass over the etherpad to see if I want to know anything else.19:33
klindgrencarl_baldwin, sure.  More than happy to give details to make discussion next wednesday easier19:34
carl_baldwinklindgren: It is very much appreciated.19:34
*** Marga_ has joined #openstack-neutron19:34
carl_baldwinklindgren: The DHCP agents and metadata services is something I hadn’t come around to thinking about yet.  It is a very good point.19:35
*** rdo has quit IRC19:35
*** ihrachyshka has joined #openstack-neutron19:36
klindgrencarl_baldwin, we are hoping to solve it other ways - such as running dhelper on the switch and point it to a known neutron dhcp server19:37
*** rdo has joined #openstack-neutron19:37
klindgrenmetadata - we were probabbly jsut going to run that on each compute node.  However it becomes an issue if we cant use iptables to redirect the requests to the local server19:37
klindgrenthough I am sure we could do something similar via openflow rules in the vswitch19:38
*** pc_m has quit IRC19:38
* carl_baldwin afk, brb19:39
*** rushil has joined #openstack-neutron19:41
*** amotoki_ has quit IRC19:41
*** hitalia has joined #openstack-neutron19:43
*** Marga_ has quit IRC19:43
*** Marga_ has joined #openstack-neutron19:43
*** yamahata has joined #openstack-neutron19:48
*** yamahata has quit IRC19:48
*** slogan621 has quit IRC19:48
*** yamahata has joined #openstack-neutron19:49
carl_baldwinklindgren: I’m a little fuzzy on the nova/neutron interplay.19:49
klindgrenSure19:49
carl_baldwinklindgren: I think we are in full agreement for how these segments should work.  I think we achieve a pretty good mutual understanding.19:49
klindgrenSo when our users boot a vm they do not provide a -net statement19:50
carl_baldwinklindgren: ^ I was just going to ask if they provide a -net.19:50
*** Aish has joined #openstack-neutron19:50
klindgrenthe scheduler filters down to a set of compute nodes.  We have configured host aggregates that contain a network metadata field.  We implemented a filter in the nova scheduler to look to see what networks are associated with a compute host19:51
*** fredlhsu has quit IRC19:51
carl_baldwinklindgren: What do the users provide in lieu of -net?19:52
*** amit213 has quit IRC19:53
klindgrenand we check to make sure that network has IP's (its also possible that 2 L2 networks are associated with a compute node - but we dont have any configured like that today).  Assuming it has IP address spacing we will modified the boot request to use the network associated with that compute host19:53
*** ecbanks has joined #openstack-neutron19:53
*** Marga_ has quit IRC19:53
*** Marga_ has joined #openstack-neutron19:53
klindgrenif someone boots a vm with a specific network defined - we will filter hosts to only those that have that network associated to it via the host-aggregate metadata19:55
klindgrenwe have this code public somewhere...19:56
* klindgren trying to find19:56
openstackgerritDavanum Srinivas (dims) proposed openstack/neutron: cleanup openstack-common.conf and sync updated files  https://review.openstack.org/18911720:01
*** achanda has quit IRC20:03
*** ecbanks has quit IRC20:04
*** merooney has joined #openstack-neutron20:04
*** boris-42 has joined #openstack-neutron20:04
spandheHi folks, is GPLv2 OK to integrate with OpenStack?20:04
*** amit213 has joined #openstack-neutron20:04
*** mickeys has joined #openstack-neutron20:05
*** spandhe has left #openstack-neutron20:05
*** amit213 has quit IRC20:05
*** merooney has quit IRC20:06
*** amit213 has joined #openstack-neutron20:06
*** spandhe has joined #openstack-neutron20:07
*** Aish has left #openstack-neutron20:07
carl_baldwinklindgren: What I was thinking is that you wanted user to specify a -net when booting a vm.  But, you want them to specifiy the L3 routed network and not one of the segments.  Is that right?20:07
spandheGot disconnected. Let me ask again:20:07
spandheHi folks, is GPLv2 OK to integrate with OpenStack?20:07
ZZelle_spandhe, it depends what you mean by integrate20:07
spandheZZelle_: I am specifically talking about this utility: http://freecode.com/projects/omcmd20:08
klindgrencarl_baldwin, that would be the ideal use case.  But to make that totally work - nova and neutron would have to know what lives where20:08
spandheZZelle_: This is a commandline utility20:08
*** Marga_ has quit IRC20:09
ZZelle_spandhe, still, what to mean by integrate?20:09
*** Marga_ has joined #openstack-neutron20:09
openstackgerritAdolfo Duarte proposed openstack/neutron: Preserve DVR FIP rule priority over Agent restarts  https://review.openstack.org/19371120:09
ZZelle_s/to/you/20:09
spandheZZelle_: I am planning to use this commandline utility in dhcpd driver.20:09
spandheZZelle_: e.g. firing a command: omcmd -s 127.0.0.1 find host name=<xyz>20:10
jrollspandhe: is there a code repository for that?20:10
carl_baldwinklindgren: ok.20:10
*** krtaylor has quit IRC20:10
ZZelle_spandhe, if you are calling an external binary, it should be fine20:10
spandhejroll: no, we wont be importing any library or adding any code to our repo.. we will be using this just like we use Dnsmasq20:10
spandheas a commandline utility20:11
jrollspandhe: I'm just curious, besides the fact of if you can integrate it :)20:11
*** thurloat is now known as thurloat_isgone20:11
*** akamyshnikova1 has quit IRC20:12
*** marrusl has quit IRC20:12
*** akamyshnikova1 has joined #openstack-neutron20:13
ZZelle_spandhe, it's fine to call GPL binary like dnsmasq/bash/etc ... really more complicated to import python gpl lib20:14
*** dan is now known as Guest8780820:14
*** dan_ has joined #openstack-neutron20:14
*** Djinnn has joined #openstack-neutron20:14
spandheZZelle_: ok.. good to know.. thanks! :)20:14
*** dan_ is now known as Guest9629920:14
*** rushil has quit IRC20:15
*** akamyshnikova1 has quit IRC20:24
*** prad_ has quit IRC20:26
*** ronis has quit IRC20:28
*** tidwellr has left #openstack-neutron20:28
*** rossella_s has quit IRC20:30
*** rossella_s has joined #openstack-neutron20:31
*** julim has quit IRC20:31
*** CaptainMorgan is now known as morgan20:41
*** Marga_ has quit IRC20:41
*** Marga_ has joined #openstack-neutron20:41
*** amotoki_ has joined #openstack-neutron20:42
*** daneyon has joined #openstack-neutron20:43
*** aranjan has joined #openstack-neutron20:45
*** fzdarsky has quit IRC20:46
*** ecbanks has joined #openstack-neutron20:46
*** amotoki_ has quit IRC20:47
*** ihrachyshka has quit IRC20:47
*** Djinnn has quit IRC20:48
*** hitalia has quit IRC20:48
*** daneyon has quit IRC20:48
*** sripriya has joined #openstack-neutron20:49
*** aranjan has quit IRC20:49
*** britthouser has quit IRC20:50
sc68calanteaya: haha - sometimes my references/humor is a little quirky :)20:51
openstackgerritRawlin Peters proposed openstack/neutron: Optimize ipset usage in IptablesFirewallDriver  https://review.openstack.org/19325920:51
*** Djinnn has joined #openstack-neutron20:53
*** raildo has quit IRC20:53
*** Djinnn has quit IRC20:54
*** Djinnn has joined #openstack-neutron20:55
*** rushil has joined #openstack-neutron20:56
*** jbell8 has quit IRC20:57
openstackgerritCedric Brandily proposed openstack/neutron: Switch to oslo_utils.uuidutils  https://review.openstack.org/16069921:00
anteayasc68cal: it works for me21:00
anteayasc68cal: I was tearing up21:01
*** regXboi has left #openstack-neutron21:01
sc68calanteaya: haha21:01
anteaya:)21:01
openstackgerritMartin Roy proposed openstack/neutron: Use string exception casting everywhere  https://review.openstack.org/19324521:06
*** krtaylor has joined #openstack-neutron21:08
*** rpothier has quit IRC21:12
*** aranjan has joined #openstack-neutron21:13
*** Djinnn has quit IRC21:13
*** daneyon has joined #openstack-neutron21:14
*** thorst has quit IRC21:15
*** lpetrut has quit IRC21:15
*** Djinnn has joined #openstack-neutron21:20
*** bradjones has quit IRC21:20
*** Marga_ has quit IRC21:21
*** bradjones has joined #openstack-neutron21:22
*** bradjones has quit IRC21:22
*** bradjones has joined #openstack-neutron21:22
*** fredlhsu has joined #openstack-neutron21:23
johnbelamariccarl_baldwin: ping21:24
*** Sukhdev has quit IRC21:27
*** localloop127 has quit IRC21:28
*** Marga_ has joined #openstack-neutron21:30
*** pck is now known as pckizer21:31
*** dboik has quit IRC21:33
openstackgerritCarl Baldwin proposed openstack/neutron: Create a routing table manager  https://review.openstack.org/19203221:34
carl_baldwinjohnbelamaric: pong21:34
johnbelamariccarl_baldwin: hi, just saw you set workflow on a patch but it depends on one that hasn't had workflow set yet - what will happen?21:35
*** csoukup has quit IRC21:35
carl_baldwinjohnbelamaric: It will wait.  I probably didn’t notice, which patch is that?21:36
johnbelamarichttps://review.openstack.org/#/c/193075/21:37
johnbelamaricdepends on21:37
johnbelamarichttps://review.openstack.org/#/c/192663/21:37
*** sridhar_ram1 has joined #openstack-neutron21:37
carl_baldwinIt doesn’t show a depends for me.21:37
johnbelamariccarl_baldwin: oh. ok. maybe I am reading the new gerrit interface incorrectly21:38
carl_baldwinSometimes, gerrit forgets the depends-on relationship.21:38
carl_baldwinIn this case, it might not wait.  This happened to me once before, I don’t remember exactly what happened.21:38
johnbelamaricit looks like PS1 shows the dependency but PS2 doesn't21:39
*** chem has quit IRC21:39
carl_baldwinDid pavel_bondar reorganize the deps?21:39
carl_baldwinOkay, I just pulled down 193075 and there is no longer a dependency.21:39
*** sridhar_ram has quit IRC21:40
johnbelamaricyes, maybe it got confused21:40
johnbelamaricok21:40
carl_baldwinSo, gerrit got it right this time.21:40
*** crose has quit IRC21:40
johnbelamaricok, just wanted to make sure, thanks21:40
*** dyydj has joined #openstack-neutron21:40
johnbelamariccarl_baldwin: yeah, totally different files they are modifying. probably something further down the chain depends on both so I misread the chain21:41
carl_baldwinThere is no way for a gerrit patch down the line to depend on both in this project.  Gerrit will reject merge patches.21:42
*** dboik has joined #openstack-neutron21:43
carl_baldwinBut, maybe there was no real dependency there.21:43
johnbelamaricyeah, i don't think there was21:43
johnbelamaricthanks - i am heading out. ttyl21:43
carl_baldwinjohnbelamaric: ttyl21:44
*** johnbelamaric has quit IRC21:44
*** nmagnezi has quit IRC21:45
openstackgerritCarl Baldwin proposed openstack/neutron: Decompose DVR CSNAT L3 Agent from Compute Node L3 Agent  https://review.openstack.org/18536821:48
*** sridhar_ram1 has quit IRC21:48
*** Marga_ has quit IRC21:51
*** dboik has quit IRC21:53
*** Marga_ has joined #openstack-neutron21:53
*** aranjan has quit IRC21:53
*** Marga_ has quit IRC21:53
*** Marga_ has joined #openstack-neutron21:54
*** padkrish has joined #openstack-neutron21:56
*** daneyon has quit IRC21:57
*** hitalia has joined #openstack-neutron21:58
*** Bhavya has joined #openstack-neutron22:00
*** TrevorV has quit IRC22:00
*** aranjan has joined #openstack-neutron22:03
*** rushil has quit IRC22:07
*** aranjan has quit IRC22:08
*** isq has joined #openstack-neutron22:11
*** padkrish has quit IRC22:12
*** padkrish has joined #openstack-neutron22:13
*** dane_leblanc has quit IRC22:13
*** yamahata has quit IRC22:14
*** rawat has quit IRC22:14
*** chem has joined #openstack-neutron22:16
*** Marga_ has quit IRC22:17
*** Marga_ has joined #openstack-neutron22:18
*** sbalukoff has quit IRC22:18
*** dimsum__ has quit IRC22:19
*** sridhar_ram has joined #openstack-neutron22:19
*** dimsum__ has joined #openstack-neutron22:20
*** bobmel has quit IRC22:21
*** bobmel has joined #openstack-neutron22:21
openstackgerritCarl Baldwin proposed openstack/neutron: Create a routing table manager  https://review.openstack.org/19203222:22
*** alex_klimov has joined #openstack-neutron22:23
*** carl_baldwin has quit IRC22:24
*** banix has quit IRC22:27
*** ijw_ has quit IRC22:27
*** sbalukoff has joined #openstack-neutron22:29
*** anshul has quit IRC22:30
*** fredlhsu has quit IRC22:30
*** jasondot_ has quit IRC22:32
*** Djinnn has quit IRC22:32
*** dimsum__ has quit IRC22:32
*** rawlin has quit IRC22:34
*** merooney has joined #openstack-neutron22:38
*** agireud has quit IRC22:39
*** thorst has joined #openstack-neutron22:39
*** alex_klimov has quit IRC22:41
openstackgerritMerged openstack/neutron: Move _add_auto_addrs_on_network_ports  https://review.openstack.org/19307522:42
*** amotoki_ has joined #openstack-neutron22:43
kevinbentonmestery: you still around?22:48
*** amotoki_ has quit IRC22:49
*** merooney has quit IRC22:54
*** Djinnn has joined #openstack-neutron22:54
*** merooney has joined #openstack-neutron22:55
*** zzzeek has quit IRC22:58
*** Swami has quit IRC23:00
*** zzzeek has joined #openstack-neutron23:01
openstackgerritMerged openstack/neutron: Add request factory for pluggable IPAM  https://review.openstack.org/19266323:03
*** aranjan has joined #openstack-neutron23:04
*** tomoe_ has joined #openstack-neutron23:04
*** dsneddon is now known as dsneddon_away23:05
*** thurloat_isgone has quit IRC23:05
*** zigo has quit IRC23:05
*** lori|away has quit IRC23:05
*** ondergetekende has quit IRC23:06
*** mickeys has quit IRC23:06
*** zigo has joined #openstack-neutron23:07
*** thurloat_isgone has joined #openstack-neutron23:07
*** thurloat_isgone is now known as thurloat23:07
*** ondergetekende has joined #openstack-neutron23:07
*** lori has joined #openstack-neutron23:08
*** thedodd has quit IRC23:08
*** Djinnn has quit IRC23:09
*** aranjan has quit IRC23:09
*** Marga_ has quit IRC23:09
*** jgrimm is now known as zz_jgrimm23:10
*** merooney has quit IRC23:12
*** zzzeek has quit IRC23:15
*** moha_hunt has joined #openstack-neutron23:15
*** amit213 has quit IRC23:16
*** amit213 has joined #openstack-neutron23:17
*** lori has quit IRC23:18
*** amit213 has quit IRC23:19
*** lori has joined #openstack-neutron23:20
*** amit213 has joined #openstack-neutron23:20
*** krtaylor has quit IRC23:21
*** ianw has quit IRC23:21
*** rstarr has quit IRC23:22
*** fredlhsu has joined #openstack-neutron23:22
*** krtaylor has joined #openstack-neutron23:22
*** sripriya has quit IRC23:23
openstackgerritHenry Gessau proposed openstack/neutron: Switch to oslo_utils.uuidutils  https://review.openstack.org/16069923:23
*** aranjan has joined #openstack-neutron23:25
*** arif-ali has quit IRC23:26
openstackgerritCedric Brandily proposed openstack/neutron-vpnaas: Switch to oslo_utils.uuidutils  https://review.openstack.org/19375423:27
openstackgerritCedric Brandily proposed openstack/neutron-fwaas: Switch to oslo_utils.uuidutils  https://review.openstack.org/19375623:27
*** merooney has joined #openstack-neutron23:27
openstackgerritCedric Brandily proposed openstack/neutron: Switch to oslo_utils.uuidutils  https://review.openstack.org/16069923:27
*** arif-ali has joined #openstack-neutron23:29
openstackgerritCedric Brandily proposed openstack/neutron: Switch to oslo_utils.uuidutils  https://review.openstack.org/16069923:29
*** rstarr has joined #openstack-neutron23:29
HenryGZZelle: sorry, I thought you would be asleep by now :)23:30
ZZelle_HenryG, i am waiting for an awfully late flight :s23:31
*** fedexo has quit IRC23:31
*** fedexo has joined #openstack-neutron23:32
*** ianw has joined #openstack-neutron23:34
*** Marga_ has joined #openstack-neutron23:34
HenryGZZelle: what about lbaas?23:38
HenryGZZelle: nevermind I see it23:38
HenryGNot sure why it didn't show in IRC23:39
ZZelle_HenryG, ?23:41
ZZelle_HenryG, ah, openstackgerrit messages23:42
*** sridhar_ram has quit IRC23:42
HenryGZZelle: yes, 193755 is missing in my IRC client23:42
*** tomoe_ has quit IRC23:42
*** ajmiller has quit IRC23:42
ZZelle_me too23:43
*** tomoe_ has joined #openstack-neutron23:43
HenryGZZelle: also not in eavesdrop23:43
*** vivek-ebay has joined #openstack-neutron23:43
*** tomoe_ has quit IRC23:43
HenryGZZelle: I think you pushed too many too fast after each other :)23:43
ZZelle_O_o23:43
*** padkrish has quit IRC23:44
*** SumitNaiksatam has quit IRC23:45
*** aranjan has quit IRC23:48
*** padkrish has joined #openstack-neutron23:51
openstackgerritMerged openstack/neutron: Change ensure_dir to not check directory exists first  https://review.openstack.org/18865923:52
*** Marga_ has quit IRC23:54
*** ZZelle_ has quit IRC23:57
*** dannywilson has quit IRC23:57
*** tomoe_ has joined #openstack-neutron23:58
*** tomoe__ has joined #openstack-neutron23:59
*** tomoe_ has quit IRC23:59

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