Friday, 2015-06-19

« Thursday, 2015-06-18 Index Saturday, 2015-06-20 »
openstackgerritMadhusudhan Kandadai proposed openstack/neutron-lbaas: Neutron_LBaaS: Update README for Tests  https://review.openstack.org/16134600:08
madhu_akblogan, xgerman, dougwig, fnaval: Have also reflect the readme on how to run the scenario tests00:08
openstackgerritMadhusudhan Kandadai proposed openstack/neutron-lbaas: Neutron_LBaaS: Update README for Tests  https://review.openstack.org/16134600:13
*** amotoki_ has joined #openstack-lbaas00:23
*** apuimedo has quit IRC00:23
openstackgerritMadhusudhan Kandadai proposed openstack/neutron-lbaas: Neutron LBaaS: Load Balancer Basic Scenario Test  https://review.openstack.org/15890400:27
*** amotoki_ has quit IRC00:28
*** madhu_ak has quit IRC00:40
*** openstack has joined #openstack-lbaas00:54
*** madhu_ak has joined #openstack-lbaas01:29
*** woodster_ has quit IRC01:31
*** bharath has quit IRC01:36
*** fnaval has quit IRC01:45
*** ypraveen has quit IRC02:00
*** Aish has joined #openstack-lbaas02:04
*** Aish has quit IRC02:04
*** ypraveen has joined #openstack-lbaas02:24
*** amotoki_ has joined #openstack-lbaas02:24
*** amotoki_ has quit IRC02:30
*** kiran-r has joined #openstack-lbaas02:33
*** ganeshna has joined #openstack-lbaas03:23
*** rm_work has quit IRC03:25
*** amotoki_ has joined #openstack-lbaas03:25
*** amotoki_ has quit IRC03:30
*** rm_work|away has joined #openstack-lbaas03:31
*** rm_work|away is now known as rm_work03:31
*** rm_work has joined #openstack-lbaas03:31
*** crc32 has quit IRC03:31
*** kiran-r has quit IRC03:44
*** ypraveen_ has joined #openstack-lbaas04:05
*** ypraveen has quit IRC04:06
*** ypraveen_ is now known as ypraveen04:06
*** ypraveen has quit IRC04:19
*** amotoki_ has joined #openstack-lbaas04:26
*** amotoki_ has quit IRC04:31
*** rm_you has joined #openstack-lbaas04:37
*** ganeshna has quit IRC04:37
*** ganeshna has joined #openstack-lbaas04:47
*** ganeshna_ has joined #openstack-lbaas04:51
*** ganeshna has quit IRC04:52
*** rm_you has quit IRC05:19
*** amotoki_ has joined #openstack-lbaas05:27
*** amotoki_ has quit IRC05:31
*** madhu_ak has quit IRC05:41
*** madhu_ak has joined #openstack-lbaas05:42
*** kiran-r has joined #openstack-lbaas06:02
*** numan has joined #openstack-lbaas06:05
*** ganeshna_ has quit IRC06:07
*** madhu_ak has quit IRC06:08
*** ganeshna has joined #openstack-lbaas06:10
*** chlong has quit IRC07:06
*** ganeshna has quit IRC07:23
rm_workjohnsom: now you've got me wanting to fix https://review.openstack.org/#/c/184868/ again -- were you going to rebase it? or should I get it fixed back up?07:33
*** ganeshna has joined #openstack-lbaas07:55
openstackgerritElena Ezhova proposed openstack/neutron-lbaas: [WIP] Switch to oslo.service  https://review.openstack.org/19345408:25
*** amotoki_ has joined #openstack-lbaas08:29
*** amotoki_ has quit IRC08:34
*** amotoki_ has joined #openstack-lbaas09:30
*** amotoki_ has quit IRC09:34
*** amotoki_ has joined #openstack-lbaas10:07
*** rdekel has joined #openstack-lbaas10:37
*** rdekel has quit IRC11:09
*** ganeshna has quit IRC11:14
*** woodster_ has joined #openstack-lbaas12:40
*** numan has quit IRC13:00
*** HenryG has quit IRC13:05
*** HenryG has joined #openstack-lbaas13:05
*** HenryG has quit IRC13:06
*** HenryG has joined #openstack-lbaas13:07
openstackgerritElena Ezhova proposed openstack/neutron-lbaas: Switch to oslo.service  https://review.openstack.org/19345413:21
*** kiran-r has quit IRC13:34
openstackgerritElena Ezhova proposed openstack/neutron-lbaas: Switch to oslo.service  https://review.openstack.org/19345413:49
*** f13o has joined #openstack-lbaas14:05
*** ajmiller has joined #openstack-lbaas14:10
*** localloop127 has joined #openstack-lbaas14:14
*** f13o has quit IRC14:53
*** f13o has joined #openstack-lbaas14:54
*** mlavalle has joined #openstack-lbaas15:00
*** TrevorV has joined #openstack-lbaas15:00
*** ganeshna_ has joined #openstack-lbaas15:01
*** ganeshna_ has joined #openstack-lbaas15:02
*** fnaval has joined #openstack-lbaas15:02
*** kiran-r has joined #openstack-lbaas15:51
TrevorVjohnsom, online yet?15:55
xgermanI don’t see him — but he should be online soon15:56
TrevorVI was just curious if I could assign myself to the bug for "fix delete member" since I took that over.15:56
TrevorVI forgot to ask earlier this week15:56
xgermanyeah, just go ahead15:56
TrevorVAlrighty15:57
TrevorVQuoted here folks :P15:57
openstackgerritTrevor Vardeman proposed openstack/octavia: Fix the delete member flow  https://review.openstack.org/19361015:57
TrevorVHey xgerman do bugs automatically close when the fix has been merged?15:58
TrevorVOr are they supposed to?15:58
xgermannot sure --15:58
xgermanif they don’t you will need to close them manually15:58
xgerman(guessing)15:58
TrevorVYeah15:59
TrevorVThe other delete fixes have been merged (except member and health monitor) but the bugs still say "Fix Committed"15:59
TrevorVSo was just wondering.15:59
TrevorVI'll ask brandon15:59
bloganbugs status will be changed when you push to gerrit16:11
bloganwell when they get merged16:11
bloganthey'll be changed to Fix Committed16:11
*** jorgem has joined #openstack-lbaas16:13
*** apuimedo has joined #openstack-lbaas16:14
*** madhu_ak has joined #openstack-lbaas16:15
*** madhu_ak_ has joined #openstack-lbaas16:15
TrevorVWhat closes them blogan ?16:20
blogandefine clos16:21
blogane16:21
*** kiran-r has quit IRC16:21
TrevorVblogan, I guess I am looking for when it changes to "fix released" but we don't really have one of those ha ha.16:23
TrevorVSo nvmd16:23
TrevorVidk what I wanted16:23
bloganthey'll get changed when teh milestone octavia 0.5 gets released16:24
TrevorVYeah, I read that just now16:24
xgermanblogan: https://review.openstack.org/#/c/161346/16:26
xgerman11 revisions for a readme...16:26
blogantypos?16:28
xgermanthe spell check gate?16:28
*** madhu_ak_ has quit IRC16:29
bloganwhy is there a README and a README.rst?16:29
xgermanyeah, that confused me, too16:30
madhu_akthats a good question.. was not sure whether we need README...it was available before..not sure whether we can remove it16:30
madhu_akso that we can maintain README.rst?16:31
blogani think they should merged into README.rst16:32
*** kiran-r has joined #openstack-lbaas16:32
blogan2 readmes are a bit odd16:32
TrevorVWe could be the first, the pioneers on a new frontier16:32
madhu_akokay, I shall modify the patch and remove README and have only README.rst?16:32
bloganbut one second on that too im checking something out, whether you have to copy the tempest.conf or if it will automatically pick it p16:32
madhu_akwhen running jenkins, it is automatically picked up, for running the tests locally, we need to copy them though16:33
bloganmadhu_ak: yeah just remove the README and keep the README.rst, they're saying the same information16:33
madhu_akawesome, will incorporate the changes..16:33
bloganmadhu_ak: i think i can make a change in the tox.ini so it automatically picks it up16:33
bloganthough that can just be a separate review16:33
madhu_akmakes sense to me..will do the necessary change then16:34
madhu_akoh16:34
bloganso yeah go ahead and make your changes16:34
bloganunless you want to do this16:34
bloganup to you16:34
madhu_akfor tox change, I made it here though: https://review.openstack.org/#/c/158904/16:34
madhu_akexcept the tempest.conf copy information16:34
bloganmadhu_ak: i think if we change the TEMPEST_CONFIG_DIR in all of those to this:16:35
bloganTEMPEST_CONFIG_DIR={env:TEMPEST_CONFIG_DIR:/opt/stack/tempest/etc}16:35
bloganthat way it pulls from the environment variable that i believe devstack sets16:36
madhu_aksure.. will do the required change and submit it in the scenario patch itself?16:37
*** rm_you|wtf has joined #openstack-lbaas16:37
*** rm_work is now known as rm_work|away16:37
bloganmadhu_ak: need to test it out first :)16:40
madhu_ak:) sounds good blogan16:40
*** rm_you| has quit IRC16:40
xgermanTrevorV, bogan I am puzzled by 2015-06-18 23:47:41.247 | ERROR: InvocationError: could not find executable ‘doc8’ in Octavia16:49
*** f13o has quit IRC16:49
xgermanblogan16:49
TrevorVI can't say I've ever seen that error.16:50
bloganhavent seen it either16:50
bloganxgerman: getting it locally?16:50
xgermanmmh - recheck?16:50
TrevorVI just did a fresh stack probably 30 minutes ago, no problems for me.16:50
xgermanwell, it’s a jenkins job16:50
TrevorVHmm....16:50
xgermanhttp://logs.openstack.org/72/171172/11/check/gate-octavia-pep8/c1c1b23/console.html16:50
TrevorVxgerman, new version of pep8 just released maybe?16:53
*** rm_work|away is now known as rm_work16:53
johnsomTrevorV No worries, grab that patchset16:53
xgermanI am worrying16:53
TrevorVjohnsom, already have a review out :)16:54
johnsomI will have a look16:54
*** numan has joined #openstack-lbaas16:56
rm_workjohnsom: are you taking over that py34 CR?16:58
*** bradjones has quit IRC16:58
johnsomYeah, I was going to update it for my comments and rebase as a test of my e-mail address change.16:58
*** bradjones has joined #openstack-lbaas16:58
*** bradjones has quit IRC16:58
*** bradjones has joined #openstack-lbaas16:58
johnsomJust didn't get to it yesterday16:59
rm_workkk just curious17:01
rm_worki am pretty busy so go for it17:01
rm_workwould like to see it get in17:01
*** bradjones has quit IRC17:11
*** bradjones has joined #openstack-lbaas17:12
*** bradjones has quit IRC17:12
*** bradjones has joined #openstack-lbaas17:12
openstackgerritGerman Eichberger proposed openstack/octavia: haproxy reference amphora REST API client  https://review.openstack.org/17117217:15
*** ganeshna_ has quit IRC17:16
xgermanTrevorV, blogan adding doc8 to test-requirements fixes17:24
xgermannow I better make an independent fix with that17:24
xgermanyep, is already in master - wonder why ir didn’t bring it in as part of the review rebase17:25
TrevorVHmm.  Not sure xgerman good catch though :D17:27
*** SumitNaiksatam has quit IRC17:36
openstackgerritAishwarya Thangappa proposed openstack/neutron-lbaas: Moving contents of base_admin_state_up.py to base.py  https://review.openstack.org/19121717:48
openstackgerritAishwarya Thangappa proposed openstack/neutron-lbaas: Tempest tests for Members using testscenarios.  https://review.openstack.org/18043617:49
openstackgerritAishwarya Thangappa proposed openstack/neutron-lbaas: Tempest tests for Listener using testscenarios.  https://review.openstack.org/17981817:50
*** jorgem has quit IRC17:51
*** bharath has joined #openstack-lbaas17:53
openstackgerritPhillip Toohill proposed openstack/octavia: Fixes missing admin/user for devstack plugin  https://review.openstack.org/19366217:57
openstackgerritElena Ezhova proposed openstack/neutron-lbaas: Switch to oslo.service  https://review.openstack.org/19345417:58
*** SumitNaiksatam has joined #openstack-lbaas18:01
*** kiran-r has quit IRC18:29
*** SumitNaiksatam has quit IRC18:32
*** SumitNaiksatam has joined #openstack-lbaas18:33
openstackgerritElena Ezhova proposed openstack/neutron-lbaas: Switch to oslo.service  https://review.openstack.org/19345418:44
openstackgerritMadhusudhan Kandadai proposed openstack/neutron-lbaas: Neutron_LBaaS: Update README for Tests  https://review.openstack.org/16134618:58
*** SumitNaiksatam_ has joined #openstack-lbaas19:08
xgermanblogan: pls review: https://review.openstack.org/#/c/171172/ — I am especially interested in your take how we should handle the flow — I really need to prototype some configurable flow thing19:09
*** SumitNaiksatam has quit IRC19:10
*** SumitNaiksatam_ is now known as SumitNaiksatam19:10
*** apuimedo has quit IRC19:14
bloganxgerman: hmm yeah having that cert_task in the flow when right now its only for this amphora driver is a bit odd19:17
bloganxgerman: same thing we are trying to solve for with containers needing a different task as well19:18
xgermanyep19:18
xgermanbut I think configurable flows are not that hard19:18
xgerman:-)19:18
johnsomThe conditional flows might help us here too.  Not sure if that has merged yet19:20
bloganxgerman: no but the implementation of them beign configurable is another matter19:20
*** bradjones has quit IRC19:20
bloganxgerman: what are your ideas on what way you make the configurable?19:20
xgermanmy idea is that you can say in the config: create_amphora_flow=database_tasks.CreateAmphoraInDB, compute_tasks.ComputeCreate, ...19:21
xgermanand then the system does something like:19:21
*** bharath has quit IRC19:22
*** bradjones has joined #openstack-lbaas19:22
*** bradjones has quit IRC19:22
*** bradjones has joined #openstack-lbaas19:22
bloganxgerman: so doesn't that mean the deployer has to know what order to call the driver methods?19:22
*** nmagnezi has joined #openstack-lbaas19:23
xgermanfor t in CONF(create_amphora_flow): create_amhora_flow.add(getattr(tasks, t)(provides=constants.AMPHORA_ID)19:23
xgermanwell, the deployer could use one if the flows we provide or learn how to make their own19:24
xgermanalternatively we can do parameters and do conditional flows as johnsom suggests19:24
bloganthat sounds like that would be best made as another, dare i say, abstracted layer19:24
johnsomMaybe we should have loadable drivers that contain the flows.  Seems like a bit of overhead though19:25
bloganjohnsom: yeah thats what it seems like that putting the flows in the config is trying to do but not as ideal19:25
bloganbut man abstraction layer after abstraction layer gets confusing19:26
johnsomAgreed19:26
xgermanyep19:26
xgermanon the other hand we can always restrict the alternative flows and go conditional19:27
*** numan has quit IRC19:28
openstackgerritMadhusudhan Kandadai proposed openstack/neutron-lbaas: Neutron LBaaS: Load Balancer Basic Scenario Test  https://review.openstack.org/15890419:30
openstackgerritMadhusudhan Kandadai proposed openstack/neutron-lbaas: Modified tox.ini  https://review.openstack.org/19369719:30
*** kobis has joined #openstack-lbaas19:32
bloganyeah i am not sure on it, i feel like the abstraction layer is the right way to do ita nd provides the most flexibility, but very worried about that confusion part19:33
*** kobis1 has joined #openstack-lbaas19:35
xgermantrue19:36
xgermanso you like putting the loading of flows into drivers?19:37
*** kobis has quit IRC19:37
xgermanor we can try to have Spring for Python… just saying19:37
bloganlol plz no19:40
*** kobis1 has quit IRC19:40
*** amotoki_ has quit IRC19:41
bloganso im talking about ahving like a WorkflowManager type of interface that does have get_create_loadbalancer_flow, get_update_loadbalancer_flow, etc and then implementations for that interface19:43
openstackgerritMadhusudhan Kandadai proposed openstack/neutron-lbaas: Modified tox.ini  https://review.openstack.org/19370619:43
bloganthen again, its hard to be specific as to what that flow does and be generic enough to handle all drivers, and at that point its gets confusing19:44
bloganoye19:44
bloganjust spit balling at this point19:44
openstackgerritMadhusudhan Kandadai proposed openstack/neutron-lbaas: Neutron LBaaS: Load Balancer Basic Scenario Test  https://review.openstack.org/15890419:46
bloganthis comes back to my driver decoupling option of boiling the ocean as an option, make the controller worker abstracted with create_lb, update_lb, delete_lb, create_listener, etc methods19:47
xgermanyeah, I also pulled calling other drivers from one driver but in the cert case that’s not convincing (since I also then need to code nova boot)19:48
bloganand then we keep the compute drivers how they are, make the network driver just a straight wrapper of neutron methods instead of the unholy mess i've made it, and then the amphora driver would be the more often implemented interface19:48
xgermanyep, or we go back to make each driver returns flows ;-)19:49
xgermanbut I think making the amphora driver the main integration point might be good19:50
*** Aish has joined #openstack-lbaas19:50
xgermanwith network I am not sure since we wanted to support multiple topologies19:50
bloganthe drivers returning flows woudln't be much different right now honestly, you could execute the the same tasks in the driver methods now without flows taht you could with flows19:52
xgermanwell, I could return flows involving other drivers tasks IHMO19:52
xgermansomewhere we need to control the order of tasks and which tasks are run in a flow to adapt to different use cases19:54
xgermanbut yeah, I have no good answer where that should live nor how we can do that without confusing people19:55
*** bharath has joined #openstack-lbaas19:55
bloganxgerman: well you could call other driver methods right now if you wanted and it'd still risk the same circular problem19:59
xgermanyep20:00
*** Aish has left #openstack-lbaas20:07
openstackgerritAishwarya Thangappa proposed openstack/neutron-lbaas: Added admin/non_admin api tests  https://review.openstack.org/17342320:16
*** amotoki_ has joined #openstack-lbaas20:42
*** amotoki_ has quit IRC20:47
openstackgerritAishwarya Thangappa proposed openstack/neutron-lbaas: Moving contents of base_admin_state_up.py to base.py  https://review.openstack.org/19121720:47
openstackgerritAishwarya Thangappa proposed openstack/neutron-lbaas: Tempest tests for Members using testscenarios.  https://review.openstack.org/18043620:55
openstackgerritAishwarya Thangappa proposed openstack/neutron-lbaas: Tempest tests for Listener using testscenarios.  https://review.openstack.org/17981820:55
*** bradjones has quit IRC21:20
*** bradjones has joined #openstack-lbaas21:22
*** bradjones has quit IRC21:22
*** bradjones has joined #openstack-lbaas21:22
openstackgerritMadhusudhan Kandadai proposed openstack/neutron-lbaas: Neutron LBaaS: Load Balancer Basic Scenario Test  https://review.openstack.org/15890421:24
*** localloop127 has quit IRC21:28
*** pck is now known as pckizer21:31
madhu_akblogan: addressed your comments. Thanks for your findings!21:31
*** mwang2 has joined #openstack-lbaas21:35
*** nmagnezi has quit IRC21:45
openstackgerritTrevor Vardeman proposed openstack/neutron-lbaas: Add vip_network_id to load balancer  https://review.openstack.org/19373621:57
*** TrevorV has quit IRC22:00
madhu_akblogan, dougwig: i think we can resume this job: 'check-neutron-lbaasv2-dsvm-api' as voting22:04
madhu_akthe above job is back to normal and is good to go22:05
madhu_akxgerman ^^22:06
xgermancool22:06
*** sbalukoff has quit IRC22:18
*** sbalukoff has joined #openstack-lbaas22:29
openstackgerritAishwarya Thangappa proposed openstack/neutron-lbaas: Tempest tests for Members using testscenarios.  https://review.openstack.org/18043622:31
openstackgerritAishwarya Thangappa proposed openstack/neutron-lbaas: Tempest tests for Listener using testscenarios.  https://review.openstack.org/17981822:42
*** amotoki_ has joined #openstack-lbaas22:43
openstackgerritAishwarya Thangappa proposed openstack/neutron-lbaas: Moving contents of base_admin_state_up.py to base.py  https://review.openstack.org/19121722:47
*** amotoki_ has quit IRC22:49
openstackgerritAishwarya Thangappa proposed openstack/neutron-lbaas: Tempest tests for Members using testscenarios.  https://review.openstack.org/18043622:50
openstackgerritAishwarya Thangappa proposed openstack/neutron-lbaas: Tempest tests for Listener using testscenarios.  https://review.openstack.org/17981822:51
openstackgerritAishwarya Thangappa proposed openstack/neutron-lbaas: Tempest tests for Listener using testscenarios.  https://review.openstack.org/17981823:02
openstackgerritAishwarya Thangappa proposed openstack/neutron-lbaas: Tempest tests for Members using testscenarios.  https://review.openstack.org/18043623:05
*** zigo has quit IRC23:05
*** rm_you|wtf has quit IRC23:06
*** zigo has joined #openstack-lbaas23:07
*** rm_you|wtf has joined #openstack-lbaas23:07
openstackgerritAishwarya Thangappa proposed openstack/neutron-lbaas: Tempest tests for Listener using testscenarios.  https://review.openstack.org/17981823:07
openstackgerritAishwarya Thangappa proposed openstack/neutron-lbaas: Tempest tests for Members using testscenarios.  https://review.openstack.org/18043623:07
*** rm_you|wtf has quit IRC23:16
*** rm_you|wtf has joined #openstack-lbaas23:17
openstackgerritCedric Brandily proposed openstack/neutron-lbaas: Switch to oslo_utils.uuidutils  https://review.openstack.org/19375523:27
*** ajmiller has quit IRC23:42
*** vivek-ebay has joined #openstack-lbaas23:43
*** SumitNaiksatam has quit IRC23:45
*** madhu_ak has quit IRC23:53
« Thursday, 2015-06-18 Index Saturday, 2015-06-20 »

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