Friday, 2015-06-12

*** bharath_ has joined #openstack-lbaas00:03
*** bharath has quit IRC00:05
*** ypraveen has joined #openstack-lbaas00:07
openstackgerritmin wang proposed openstack/neutron-lbaas: Admin API tempest for healthmonitor  https://review.openstack.org/17354200:15
*** Aish has quit IRC00:30
*** Aish has joined #openstack-lbaas00:32
*** ganeshna has joined #openstack-lbaas00:32
*** bharath_ has quit IRC00:37
*** ganeshna has quit IRC00:40
*** bharath has joined #openstack-lbaas00:43
*** bharath has quit IRC00:47
*** Aish has quit IRC00:48
*** madhu_ak has quit IRC00:49
*** madhu_ak has joined #openstack-lbaas00:52
madhu_akblogan: around?00:53
madhu_akblogan: after rebasing fnaval's scenario tests with correct imports module and adding 'scenario' in tox.ini, when I run the test, I am getting an error. Any idea about this: https://gist.github.com/akmadhusudhan/d0faaee37599a06b137b00:56
madhu_akscenario's patch : https://review.openstack.org/#/c/158904/00:56
*** KunalGandhi has quit IRC01:14
*** madhu_ak has quit IRC01:33
*** fnaval has quit IRC01:45
*** Aish has joined #openstack-lbaas01:53
*** fnaval has joined #openstack-lbaas02:01
*** ypraveen has quit IRC02:20
*** Kunal has joined #openstack-lbaas02:29
*** KunalGandhi has joined #openstack-lbaas02:29
*** KunalGan_ has joined #openstack-lbaas02:34
*** KunalGandhi has quit IRC02:35
*** Kunal has quit IRC02:35
*** Djinnn has joined #openstack-lbaas02:50
*** Djinnn has quit IRC02:55
*** SumitNaiksatam_ has joined #openstack-lbaas03:05
*** SumitNaiksatam has quit IRC03:07
*** SumitNaiksatam_ has quit IRC03:10
*** kiran-r has joined #openstack-lbaas03:12
*** SumitNaiksatam has joined #openstack-lbaas03:18
*** SumitNaiksatam has quit IRC03:24
*** kiran-r has quit IRC03:45
*** ganeshna has joined #openstack-lbaas03:46
*** kiran-r has joined #openstack-lbaas03:51
*** kiranr has joined #openstack-lbaas03:55
*** kiran-r has quit IRC03:55
*** kiranr has quit IRC04:02
*** KunalGan_ has quit IRC04:34
*** ypraveen has joined #openstack-lbaas04:44
*** chlong is now known as chlong_mtg04:59
*** kiran-r has joined #openstack-lbaas05:17
*** kiran-r has quit IRC05:21
*** kiran-r has joined #openstack-lbaas05:21
*** ganeshna has quit IRC05:40
*** ganeshna has joined #openstack-lbaas05:41
*** chlong_mtg is now known as chlong06:06
*** ypraveen has quit IRC06:11
*** amotoki has joined #openstack-lbaas06:16
*** Aish has quit IRC06:23
*** KunalGandhi has joined #openstack-lbaas06:32
*** Kunal has joined #openstack-lbaas06:34
*** KunalGandhi has quit IRC06:39
*** KunalGandhi has joined #openstack-lbaas06:39
*** KunalGandhi has quit IRC06:43
*** Kunal has quit IRC06:44
*** amotoki has quit IRC06:47
*** amotoki has joined #openstack-lbaas06:50
*** kiranr has joined #openstack-lbaas08:15
*** kiran-r has quit IRC08:15
*** kiranr has quit IRC08:15
*** kiranr has joined #openstack-lbaas08:16
*** chlong is now known as chlong_afk08:16
*** kiranr is now known as kiran-r08:18
*** haigang has joined #openstack-lbaas08:25
*** haigang has quit IRC08:26
*** fnaval has quit IRC08:38
*** fnaval has joined #openstack-lbaas08:41
*** Alex_Stef has joined #openstack-lbaas08:48
*** woodster_ has quit IRC09:01
*** chlong_afk has quit IRC09:03
*** Alex_Stef has quit IRC09:27
*** numan has joined #openstack-lbaas09:48
*** numan has quit IRC09:49
*** numan has joined #openstack-lbaas09:49
*** Miouge has joined #openstack-lbaas10:20
*** ganeshna has quit IRC10:27
*** ganeshna has joined #openstack-lbaas10:33
openstackgerritJian Wen proposed openstack/neutron-lbaas: Run as root for killing haproxy  https://review.openstack.org/19100810:35
*** Miouge has left #openstack-lbaas10:40
*** ganeshna has quit IRC11:23
*** Aish has joined #openstack-lbaas13:00
*** Aish has quit IRC13:05
*** bradjones has quit IRC13:35
*** bradjones has joined #openstack-lbaas13:38
*** bradjones has quit IRC13:38
*** bradjones has joined #openstack-lbaas13:38
*** kiran-r has quit IRC13:49
*** localloo1 has joined #openstack-lbaas13:50
*** woodster_ has joined #openstack-lbaas13:51
*** localloo1 has quit IRC14:02
*** TrevorV has joined #openstack-lbaas14:15
*** fnaval has quit IRC14:49
openstackgerritTrevor Vardeman proposed stackforge/octavia: Fix the delete pool flow  https://review.openstack.org/19078015:21
*** mlavalle has joined #openstack-lbaas15:37
openstackgerritTrevor Vardeman proposed stackforge/octavia: Fix the delete loadbalancer flow  https://review.openstack.org/18723715:40
*** mixos has joined #openstack-lbaas15:43
TrevorVjohnsom, I addressed your concerns my man15:46
johnsomYep, thanks!  Working on those reviews now15:47
*** Aish has joined #openstack-lbaas15:48
johnsomDon't we have tests for the models under the functional tests?  octavia/tests/functional/db/test_models.py15:48
*** pck has quit IRC15:49
TrevorVthose are for the models, not the data_models15:50
TrevorVModels are the database objects15:50
johnsomAh yes15:51
TrevorVOnce again, I can add a class to test it, I just don't know that it's worth it since it literally just makes whatever is there "None"... Does it really provide that much?15:52
TrevorVI'm not against it, its not a hard test obviously, just seems silly15:52
johnsomIt was a nit, let's pass15:54
johnsomI agree, it would be a silly test15:54
TrevorVAlright, I was just checking.  I mean, if we start adding some other methods that do things like that throughout the data_models then we should add a test class, just for tracking changes, but for one simple method, I feel that's not really necessary15:56
*** numan has quit IRC15:58
johnsomCool stuff.  Now you just need to bother one of your co-workers to kick them through15:58
*** pck has joined #openstack-lbaas16:00
*** KunalGandhi has joined #openstack-lbaas16:03
*** Kunal has joined #openstack-lbaas16:03
TrevorVjohnsom, I'm the only dev in the office today... blogan is out through Monday, back Tuesday16:03
TrevorVWe'll see if anyone here can get to it ha ha16:03
johnsomMaybe Adam or Doug can hit them16:04
*** Aish has quit IRC16:08
*** kiran-r has joined #openstack-lbaas16:14
*** KunalGandhi has quit IRC16:16
*** Kunal has quit IRC16:20
*** fnaval has joined #openstack-lbaas16:33
*** sbalukoff has quit IRC16:34
*** Aish has joined #openstack-lbaas16:35
*** KunalGandhi has joined #openstack-lbaas16:45
*** madhu_ak has joined #openstack-lbaas16:50
*** jorgem has joined #openstack-lbaas16:55
xgermanare we good to +A16:57
xgerman?16:57
xgermanlast I saw was a pep8 error16:58
xgermanbut maybe I am looking at the wrong thing16:58
xgermanTrevorV ping me and I can +2 or whatever is needed16:58
openstackgerritmin wang proposed openstack/neutron-lbaas: Introduced tempest API tests for openstack/neutron-lbaas  https://review.openstack.org/16843917:04
*** jorgem has quit IRC17:07
openstackgerritDoug Wiegley proposed openstack/neutron-lbaas: Add v1 quota tests in-tree  https://review.openstack.org/19115317:08
*** ypraveen has joined #openstack-lbaas17:10
*** kiran-r has quit IRC17:10
*** fnaval_ has joined #openstack-lbaas17:14
*** fnaval has quit IRC17:16
*** fnaval has joined #openstack-lbaas17:16
*** fnaval_ has quit IRC17:20
*** jorgem has joined #openstack-lbaas17:26
*** fnaval has quit IRC17:34
openstackgerritDoug Wiegley proposed openstack/neutron-lbaas: Add neutron to requirements  https://review.openstack.org/19116117:40
*** amotoki has quit IRC17:41
*** kiran-r has joined #openstack-lbaas17:43
openstackgerritMerged openstack/neutron-lbaas: Admin API tempest for healthmonitor  https://review.openstack.org/17354217:49
*** sbalukoff has joined #openstack-lbaas17:51
*** jorgem has quit IRC17:51
openstackgerritMerged openstack/neutron-lbaas: Adding code to prevent vip port deletion from port api  https://review.openstack.org/17601618:00
*** bharath has joined #openstack-lbaas18:00
*** mixos is now known as mixos-away18:04
TrevorVxgerman, I swear I ran tox... but I guess I fudged it up.18:10
TrevorVAre we okay with the only +2's being from HP?  I thought we wanted cross-company +2's?18:11
*** mixos-away is now known as mixos18:15
openstackgerritTrevor Vardeman proposed stackforge/octavia: Fix the delete loadbalancer flow  https://review.openstack.org/18723718:18
*** kiran-r has quit IRC18:30
TrevorVjohnsom, that test already exists, I made a duplicate.18:30
TrevorVOn accident18:30
johnsomOh, ok, let me take a look again18:31
TrevorVSure thing18:31
TrevorVThe original one is on current line 361 if that helps18:31
TrevorVSorry, 334, wrong line18:31
johnsomThat test runs MarkAmphoraDeletedInDB() vs. your MarkLBAmphoraeDeletedInDB()18:35
TrevorVOoooooooooh... I misread that... DAMNIT... Alright I'll add it back and change the name18:37
johnsomYep, it's a different test18:43
TrevorVI just checked the one I removed, its exactly the same with just differently mocked parameters to one18:45
TrevorVthey both test this: MarkAmphoraDeletedInDB18:45
*** mwang2_ has quit IRC18:47
TrevorVI probably copied it and forgot to mod it for the testing of marking ALL amphorae deleted.  Which I'm about to do.18:47
TrevorVjohnsom, ^^18:47
*** mwang2 has joined #openstack-lbaas18:48
openstackgerritmin wang proposed openstack/neutron-lbaas: Admin_state_up tempest test for health monitor  https://review.openstack.org/17882718:55
openstackgerritTrevor Vardeman proposed stackforge/octavia: Fix the delete loadbalancer flow  https://review.openstack.org/18723719:09
*** bharath has quit IRC19:26
openstackgerritMadhusudhan Kandadai proposed openstack/neutron-lbaas: Tempest API tests using testscenarios for create/update pool  https://review.openstack.org/18001919:44
*** TrevorV has quit IRC19:45
*** bharath has joined #openstack-lbaas19:51
openstackgerritAishwarya Thangappa proposed openstack/neutron-lbaas: Moving the base-admin-state-up.py to a separate patch.  https://review.openstack.org/19120519:53
*** Aish has quit IRC20:07
*** pck has quit IRC20:07
*** SlickNik has quit IRC20:07
*** redrobot has quit IRC20:07
*** pck has joined #openstack-lbaas20:07
*** redrobot has joined #openstack-lbaas20:08
*** SlickNik has joined #openstack-lbaas20:08
*** redrobot is now known as Guest6707420:08
*** Aish has joined #openstack-lbaas20:08
johnsomAll good Trevor, have a good weekend20:16
*** ebagdasa has quit IRC20:59
*** ebagdasa has joined #openstack-lbaas21:01
openstackgerritAishwarya Thangappa proposed openstack/neutron-lbaas: Moving base_admin_state_up.py to a new patch  https://review.openstack.org/19121721:13
*** ebagdasa has quit IRC21:22
*** ebagdasa has joined #openstack-lbaas21:24
madhu_akfnaval: you around?21:35
madhu_akptoohill, blogan: ^^21:36
*** madhu_ak has quit IRC21:37
*** madhu_ak has joined #openstack-lbaas21:38
*** mixos has quit IRC23:04
johnsomFYI, I have a patchset ready for the octavia rename to update the devstack samples, etc.23:19
johnsomJust waiting for infra to finish23:19
*** openstackgerrit has quit IRC23:22
*** openstackgerrit has joined #openstack-lbaas23:22
openstackgerritAishwarya Thangappa proposed openstack/neutron-lbaas: Tempest tests for Members using testscenarios.  https://review.openstack.org/18043623:24
openstackgerritAishwarya Thangappa proposed openstack/neutron-lbaas: Tempest tests for Listener using testscenarios.  https://review.openstack.org/17981823:26
*** Aish has quit IRC23:33
*** ajmiller has quit IRC23:48
openstackgerritJeremy Stanley proposed openstack/octavia: Update .gitreview file for project rename  https://review.openstack.org/19128223:50

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