| *** sbalukoff has joined #openstack-lbaas | 00:08 | |
| *** banix has quit IRC | 00:20 | |
| *** sbalukoff has quit IRC | 00:21 | |
| *** busterswt has joined #openstack-lbaas | 00:37 | |
| *** ajmiller has quit IRC | 00:49 | |
| *** banix has joined #openstack-lbaas | 00:56 | |
| *** woodster_ has quit IRC | 01:10 | |
| *** banix has quit IRC | 01:35 | |
| *** banix has joined #openstack-lbaas | 01:41 | |
| *** sbalukoff has joined #openstack-lbaas | 01:51 | |
| *** banix has quit IRC | 01:59 | |
| *** banix has joined #openstack-lbaas | 02:31 | |
| *** Krast has joined #openstack-lbaas | 02:32 | |
| *** enikanorov has quit IRC | 02:40 | |
| *** enikanorov has joined #openstack-lbaas | 02:40 | |
| *** sbalukoff has quit IRC | 03:04 | |
| *** banix has quit IRC | 03:29 | |
| openstackgerrit | Brandon Logan proposed openstack/neutron-lbaas: Pulling in tempest files https://review.openstack.org/162445 | 03:36 |
|---|---|---|
| *** vivek-ebay has joined #openstack-lbaas | 03:50 | |
| *** sbfox has joined #openstack-lbaas | 03:53 | |
| *** woodster_ has joined #openstack-lbaas | 04:37 | |
| *** sbfox has quit IRC | 04:46 | |
| *** sbfox has joined #openstack-lbaas | 04:50 | |
| *** SumitNaiksatam has quit IRC | 05:15 | |
| *** Varun_Lodaya has joined #openstack-lbaas | 05:37 | |
| *** sbalukoff has joined #openstack-lbaas | 05:40 | |
| *** vivek-ebay has quit IRC | 05:40 | |
| *** Varun_Lodaya has quit IRC | 06:29 | |
| *** woodster_ has quit IRC | 06:40 | |
| openstackgerrit | Brandon Logan proposed openstack/neutron-lbaas: Pulling in tempest files https://review.openstack.org/162445 | 06:43 |
| openstackgerrit | Brandon Logan proposed openstack/neutron-lbaas: Neutron LBaaS: Initial Load Balancer Tests https://review.openstack.org/153016 | 06:43 |
| *** apuimedo has joined #openstack-lbaas | 07:07 | |
| *** sbfox has quit IRC | 07:42 | |
| *** Miouge has joined #openstack-lbaas | 07:44 | |
| *** chlong has quit IRC | 08:07 | |
| openstackgerrit | Kobi Samoray proposed openstack/neutron-lbaas: VMWare Edge loadbalancer v1 driver https://review.openstack.org/153370 | 08:31 |
| *** amotoki has joined #openstack-lbaas | 08:33 | |
| *** enikanorov_ has joined #openstack-lbaas | 08:35 | |
| *** vivek-ebay has joined #openstack-lbaas | 08:41 | |
| *** vivek-ebay has quit IRC | 08:45 | |
| *** BeardyMcBeards has quit IRC | 09:04 | |
| *** BeardyMcBeards has joined #openstack-lbaas | 09:06 | |
| *** chlong has joined #openstack-lbaas | 11:03 | |
| *** woodster_ has joined #openstack-lbaas | 11:18 | |
| *** kobis has joined #openstack-lbaas | 11:53 | |
| *** Miouge_ has joined #openstack-lbaas | 12:01 | |
| *** Miouge has quit IRC | 12:01 | |
| *** Miouge_ is now known as Miouge | 12:01 | |
| *** Miouge has quit IRC | 12:50 | |
| *** Miouge has joined #openstack-lbaas | 12:50 | |
| *** chlong has quit IRC | 13:04 | |
| *** Miouge has quit IRC | 13:04 | |
| *** amotoki has quit IRC | 13:04 | |
| *** amotoki has joined #openstack-lbaas | 13:14 | |
| *** jschwarz has joined #openstack-lbaas | 13:15 | |
| *** chlong has joined #openstack-lbaas | 13:17 | |
| *** woodster_ has quit IRC | 13:30 | |
| *** banix has joined #openstack-lbaas | 13:52 | |
| *** sballe__ has joined #openstack-lbaas | 14:01 | |
| *** woodster_ has joined #openstack-lbaas | 14:07 | |
| *** jamiem has joined #openstack-lbaas | 14:15 | |
| *** vivek-ebay has joined #openstack-lbaas | 14:33 | |
| *** vivek-ebay has quit IRC | 14:47 | |
| *** vivek-ebay has joined #openstack-lbaas | 14:54 | |
| *** juliancash_ has quit IRC | 14:54 | |
| *** vivek-ebay has quit IRC | 14:54 | |
| *** Varun_Lodaya has joined #openstack-lbaas | 15:04 | |
| *** xgerman has joined #openstack-lbaas | 15:20 | |
| sballe__ | Morning | 15:43 |
| *** Varun_Lodaya has quit IRC | 15:49 | |
| *** kobis has quit IRC | 16:03 | |
| *** woodster_ has quit IRC | 16:10 | |
| *** apuimedo has quit IRC | 16:11 | |
| *** mlavalle has joined #openstack-lbaas | 16:13 | |
| jschwarz | dougwig, ping. can I bother you with an lbaas v1 question? :) | 16:28 |
| dougwig | jschwarz: shoot | 16:31 |
| *** Varun_Lodaya has joined #openstack-lbaas | 16:32 | |
| jschwarz | dougwig, what's the wanted behavior of the following scenario: | 16:32 |
| jschwarz | dougwig, create a network, create an lbaas pool attached to the network, remove the network | 16:33 |
| dougwig | oh, and it'll delete because it has no ports attached, right? | 16:33 |
| jschwarz | dougwig, should the pool still be there even if the network and subnet it was connected to no longer exist? | 16:33 |
| jschwarz | dougwig, yep - only a pool, no ports | 16:33 |
| dougwig | i would think the network delete should fail, if the pool is really associated to it. | 16:34 |
| jschwarz | dougwig, was afraid you'd say that :) | 16:34 |
| jschwarz | dougwig, alright, will have a look at the code and suggest a patch. thanks | 16:34 |
| *** ajmiller has joined #openstack-lbaas | 16:35 | |
| *** sbfox has joined #openstack-lbaas | 16:38 | |
| *** fnaval has joined #openstack-lbaas | 16:42 | |
| blogan | jschwarz: i believe the reason for that is when you create a pool a port is not actually created on that subnet/network, unless a driver does it | 16:45 |
| blogan | jscwarz: which i dont believe the ref impl driver creates a port either | 16:46 |
| jschwarz | blogan, yes, I agree. but logically the pool is attached to a subet | 16:46 |
| blogan | jschwarz: yeah but i dont believe there is even a foreign key to it | 16:47 |
| jschwarz | blogan, removing such a subnet (as I described) pops up errors on horizon's side | 16:47 |
| jschwarz | blogan, I'm fine with going for a horizon-fix, but am conflicted if the fix shouldn't be at the source | 16:47 |
| blogan | jschwarz: i havent been in v1 land in a while, but i bet horizon has its own validation | 16:48 |
| blogan | jschwarz: that type of thing should be done at the source | 16:48 |
| jschwarz | blogan, well, on my setup (and on a client's) this raises an error | 16:48 |
| jschwarz | blogan, let me pastebin, sec | 16:48 |
| blogan | jschwarz: what does? trying to delete the network with an attached pool through the API? or through horizon? | 16:49 |
| jschwarz | blogan, http://pastebin.com/fHBvdF3D | 16:49 |
| jschwarz | blogan, opening the pool's page gives this error on upstream | 16:49 |
| jschwarz | blogan, but the client says that on earlier versions the error causes the entire 'load balancer' tab to be unaccessible | 16:50 |
| *** vivek-ebay has joined #openstack-lbaas | 16:50 | |
| blogan | yeah thats an odd error, not really too informative other than its not expecting a list and getting it somewhere | 16:51 |
| blogan | well in that location | 16:51 |
| blogan | which i have 0 experience with horizon unfortunately | 16:51 |
| jschwarz | blogan, client reported this in icehouse IMO | 16:51 |
| jschwarz | blogan, upstream allows deleting the pool as long as this is done from the load balancer screen | 16:52 |
| jschwarz | blogan, either way i agree that this should be treated at the source | 16:52 |
| blogan | well that is failing on a GET | 16:52 |
| blogan | is that after a DELETE of a pool? | 16:52 |
| jschwarz | that is when trying to enter the page of a pool | 16:52 |
| jschwarz | so 'pool show' probably | 16:53 |
| jschwarz | (CLI doesn't cause problems obviously) | 16:53 |
| blogan | yeah, so thats a horizon issue, but what actions did yout ake to get this to start? | 16:53 |
| jschwarz | blogan, I created a network, created a pool, attached a healthmonitor to the pool (probably not needed), deleted the network | 16:53 |
| jschwarz | blogan, then, tried to "view" the pool's information on the loadbalancer tab | 16:54 |
| jschwarz | ie. press on the '<pool name>' link | 16:54 |
| blogan | jschwarz: i see, but the cli will show you the pool's information without error? | 16:54 |
| blogan | jschwarz: i wonder if horizion is trying to get the network/subnet information that you deleted and its failing there | 16:55 |
| *** Krast has quit IRC | 16:55 | |
| *** Krast has joined #openstack-lbaas | 16:56 | |
| jschwarz | blogan, yes re: cli question | 16:56 |
| jschwarz | blogan, neutron lb-pool-show shows that the pool's subnet_id field is non-empty (pointing to old subnet) | 16:57 |
| jschwarz | probably horizon is trying to show the subnet's name or something... let me check one sec | 16:57 |
| jschwarz | blogan, yeah, horizion is displaying mininalistic data on the subnet | 16:58 |
| jschwarz | name and actual subnet CIDR | 16:58 |
| jschwarz | (with a link) | 16:58 |
| blogan | jschwarz: yeah there's the issue, of course the root of the issue is the pool registering itself in some form or fashion as being on that subnet | 16:59 |
| blogan | jschwarz: or lack of | 16:59 |
| jschwarz | blogan, yep. | 17:00 |
| jschwarz | blogan, so I'll look into patching db_base_plugin (on neutron's side) as it seems to be the code should go in there | 17:00 |
| blogan | jschwarz: are you going to check the pools table on delete or something? | 17:01 |
| jschwarz | blogan, hopefully - try to retrieve all the pools connected to the subnet which is deleted and if there is raise an exception | 17:02 |
| blogan | jschwarz: i'm not sure that is the right way to go as the pool technically has no links to that subnet, other than a db entry. i think the two ways to solve it are make sure the pool creates a port on that subnet, or the pool's subnet_id has a foreign key to the subnets table | 17:03 |
| blogan | the 2nd one i'm not sure about either bc we're at the point where we are starting to think about having neutron-lbaas being in its own db, which obviously can't have a foreign key to the neutron db | 17:04 |
| jschwarz | blogan, I also dislike the foreign key. backporting this would be hell and I don't think that's the correct solution here | 17:04 |
| jschwarz | blogan, also I think that adding a new port for each pool is a bit of an overkill.. why do you think that's needed? | 17:05 |
| blogan | jschwarz: well replace "create port" with "register with", some mechanism to do that | 17:06 |
| jschwarz | blogan, we could achieve the same functionality through simply checking if a pool is associated to a subnt | 17:06 |
| jschwarz | s/subnt/subnet/g | 17:06 |
| blogan | jschwarz: but if neutron-lbaas is in its own db that check may not be possible | 17:06 |
| blogan | though v1 tables will most likely remain in neutron | 17:06 |
| blogan | btw the thought of neutron-lbaas in its own db is something nto discussed much, and probably only applicable to v2, we'll be discussing it more in vancouver | 17:07 |
| jschwarz | I see | 17:07 |
| blogan | jschwarz: as you can tell, i'm uncertain about the way forward | 17:07 |
| jschwarz | blogan, I still don't see why creating a port for each pool is needed here though | 17:08 |
| blogan | jschwarz: i have a feeling that neutron wouldn't accept code that checks the pools table though. i coudl be wrong | 17:08 |
| jschwarz | blogan, uhm... | 17:08 |
| blogan | jschwarz: oh i agree its overkill, but i said above, there needs to be some way to register, creating a port was a simplistic way | 17:08 |
| jschwarz | blogan, well there's an API to check exactly these kind of things | 17:09 |
| jschwarz | blogan, but I see your point now | 17:10 |
| blogan | jschwarz: i would say ask the neutron cores about modifying db_base_plugin to check the pools table | 17:11 |
| jschwarz | blogan, the l3_db.py file has checks to the VPNaaS | 17:11 |
| jschwarz | blogan, specifically "vpnservice.check_router_in_use" | 17:12 |
| blogan | jschwarz: i have a sneaking feeling they have some other way to do this | 17:12 |
| jschwarz | blogan, so it isn't unheard of | 17:12 |
| blogan | jschwarz: i have very limited knowledge of vpnaas, but i think there's something exceptional behind that, don't quote me on that though | 17:13 |
| blogan | jschwarz: pc_m would probably know about that | 17:13 |
| jschwarz | blogan, nevermind, I've been looking at an outdated code - armax added merged a patch 5 days that removed exactly this: https://github.com/openstack/neutron/commit/e1a9eb1f92169b8683364019c62efbafa3f63291 | 17:14 |
| *** sbalukoff has quit IRC | 17:15 | |
| blogan | ah yeah thats what i was thinking about when i said "some other way to do this", the event registry | 17:15 |
| jschwarz | blogan, lol :) | 17:15 |
| jschwarz | what is this registry? where did it come from? <.< | 17:16 |
| blogan | jschwarz: i remember reading a spec and the code, it got merged recently | 17:16 |
| jschwarz | blogan, well, better do some digging | 17:17 |
| jschwarz | I'll report back when I have something, though it might take a few days | 17:17 |
| blogan | jschwarz: https://review.openstack.org/#/c/153774/ | 17:18 |
| jschwarz | blogan, thanks :) | 17:20 |
| *** Varun_Lodaya has quit IRC | 17:20 | |
| jschwarz | blogan, so if I use that API, will that be an acceptable approach for you? | 17:21 |
| jschwarz | blogan, or do you think we should add a port regardless? | 17:22 |
| blogan | jschwarz: i need to read up on it in detail, but thats a form of registring that i believe works no matter where neutron-lbaas's db is located. but i need to read in detail into it | 17:23 |
| jschwarz | ACK | 17:23 |
| blogan | jschwarz: TL;DR: yes, as long as i undersrtand it how i think i do | 17:23 |
| jschwarz | :) | 17:23 |
| *** madhu_ has joined #openstack-lbaas | 17:34 | |
| jschwarz | blogan, the registry code seems very straightforward and I think we can use it | 17:48 |
| *** madhu_ has quit IRC | 17:48 | |
| jschwarz | blogan, it will however make backport a huge pain since this is new code :< | 17:49 |
| *** madhu_ has joined #openstack-lbaas | 17:49 | |
| jschwarz | blogan, I'm inclined to just create a port when attaching a pool to a subnet :) | 17:49 |
| blogan | jschwarz: lol coming full circle. so for v1 that would work, though thinking about it more, I wonder if that would break existing deployments in some way | 17:51 |
| jschwarz | blogan, gah. | 17:51 |
| *** banix has quit IRC | 17:52 | |
| blogan | jschwarz: lol i know, never now if someone relies on that lack of functionality | 17:52 |
| blogan | s/now/know | 17:52 |
| blogan | jschwarz: if horizon handled it gracefully would that suit your needs? | 17:52 |
| blogan | jschwarz: but thats just duct tape, bleh | 17:52 |
| jschwarz | maybe a db migrate fix? :) | 17:52 |
| jschwarz | can add a port for each pool... | 17:53 |
| blogan | that has a lot of different ramifications | 17:53 |
| blogan | quotas, can a subnet handle another port, things im not evne away of | 17:53 |
| blogan | aware of | 17:53 |
| jschwarz | oh yey!!! | 17:54 |
| jschwarz | well we could duct tape it nicely ^_^ | 17:55 |
| blogan | jschwarz: that might be the best bad idea we have | 17:55 |
| jschwarz | blogan, let me sleep on this for a few days. I'll get back to you | 17:57 |
| blogan | jschwarz: okay thanks, also keep in mind we're doing a v2 of lbaas, but that can use the event registry code | 17:57 |
| jschwarz | blogan, aye, I heard such a rumor | 17:58 |
| jschwarz | blogan, doubt though that the v2 will make the same mistake twice ;-) | 17:58 |
| blogan | jschwarz: haha famous last words | 17:59 |
| jschwarz | XD | 17:59 |
| *** sbfox has quit IRC | 18:00 | |
| *** crc32 has joined #openstack-lbaas | 18:05 | |
| *** chlong has quit IRC | 18:05 | |
| *** amotoki has quit IRC | 18:09 | |
| *** amotoki has joined #openstack-lbaas | 18:10 | |
| *** mwang2 has joined #openstack-lbaas | 18:11 | |
| *** sbfox has joined #openstack-lbaas | 18:13 | |
| *** jschwarz has quit IRC | 18:14 | |
| *** sbalukoff has joined #openstack-lbaas | 18:23 | |
| *** banix has joined #openstack-lbaas | 18:32 | |
| *** juliancash has joined #openstack-lbaas | 18:49 | |
| juliancash | s/nd/and/ | 18:55 |
| juliancash | ^ oops | 18:55 |
| madhu_ | blogan: https://review.openstack.org/#/c/162445/ for this patch, where can I see test_loadbalancer.py in api/ directory in-order to run the tests? | 19:08 |
| *** SumitNaiksatam has joined #openstack-lbaas | 19:09 | |
| *** busterswt has quit IRC | 19:11 | |
| madhu_ | or should I need to run the tempest tests in this way? 1. Pull this https://review.openstack.org/#/c/162445/ and pull https://review.openstack.org/#/c/153016/ on top of it? | 19:12 |
| madhu_ | blogan, fnaval : ^^ | 19:12 |
| *** vivek-ebay has quit IRC | 19:13 | |
| madhu_ | xgerman, fnaval,juliancash, blogan: I am able to run the tests using blogan patch( https://review.openstack.org/#/c/162445) and fnaval patch (https://review.openstack.org/#/c/153016/) | 19:20 |
| *** jorgem has joined #openstack-lbaas | 19:20 | |
| madhu_ | fyi: https://gist.github.com/akmadhusudhan/6f8a5e7b47a5e96072dc | 19:21 |
| xgerman | thanks madhu_ | 19:25 |
| *** Aish has joined #openstack-lbaas | 19:31 | |
| *** Varun_Lodaya has joined #openstack-lbaas | 19:37 | |
| *** woodster_ has joined #openstack-lbaas | 19:41 | |
| fnaval | madhu_: awesome to hear that | 19:43 |
| fnaval | sorry for the late reply | 19:43 |
| blogan | madhu_: i may end up going a different route than that review, im finalizing it right now, looks like it'll work and be a lot easier | 19:49 |
| Aish | blogan: i tried the same. But, why should we run the lbaasv2 agent manually? and while doing that i am getting this error.. https://gist.github.com/AishwaryaT/6eb3d092dc86bff36d60 | 20:00 |
| fnaval | Aish: I don't believe that it needs to be run manually now. though blogan would know | 20:00 |
| fnaval | or ptoohill | 20:00 |
| ptoohill | ? | 20:01 |
| fnaval | does the agent still need to be run manually? | 20:01 |
| fnaval | with the latest code? | 20:01 |
| ptoohill | the latest devstack? | 20:01 |
| blogan | it does until devstack automatically starts it, which i believe ajmiller has a patch up for | 20:02 |
| madhu_ | I ran the lbaasv2 haproxy agent manually, then hitting the tests... (but I didnt want to pull ptoohills patch though as it is merged in master) | 20:02 |
| blogan | https://review.openstack.org/#/c/160644/ | 20:02 |
| blogan | that review will get the agent going automatically | 20:03 |
| blogan | not merged yet | 20:03 |
| fnaval | ah ok. then it has to run manually | 20:03 |
| Aish | fnaval blogan: thnks.. So it has to be run manually until this patch is merged | 20:03 |
| blogan | Aish: correct | 20:03 |
| blogan | Aish: or you can point your localrc to pull that review down and it'll do it for you | 20:03 |
| ptoohill | And no need to worry about the driver review either, thats been merged | 20:04 |
| Aish | oh.. thnks blogan ptoohill | 20:05 |
| *** sbfox has quit IRC | 20:05 | |
| ajmiller | blogan pthooil fnaval There are two neutron-lbaas patches and 1 devstack patch make things work automatically | 20:05 |
| ajmiller | The neutron-lbaas patches are ready for merge (need some final reviews). | 20:05 |
| ptoohill | Ah, so a few things need to be checked out/merged | 20:05 |
| blogan | ajmiller: what two neutron-lbaas patches? | 20:05 |
| ajmiller | The devstack patch will depend on a grenade patch that i'm working on. | 20:06 |
| ajmiller | just a sec.. | 20:06 |
| blogan | ajmiller: oh you mean the devstack repo patch, and your service_provider neutron-lbaas patch and the agent patch | 20:06 |
| blogan | gotcha | 20:06 |
| ajmiller | https://review.openstack.org/#/c/160625/ | 20:07 |
| *** sbfox has joined #openstack-lbaas | 20:07 | |
| blogan | yeah that service_provider one will need to be added as well, so Aish it probably won't automatically start the v2 agent up wiht just that one patch | 20:07 |
| ajmiller | https://review.openstack.org/#/c/160644/ | 20:07 |
| ajmiller | https://review.openstack.org/#/c/161436/ | 20:07 |
| blogan | well it'll start it up, but the neutron-server won't be loaded with the driver that would send data to the agent | 20:07 |
| ajmiller | BTW, Ive been traveling Friday - tomrrow, not very well connected to work, but trying to keep things moving along | 20:09 |
| ajmiller | blogan I think that with all of these patches, the server will load the driver. If that isn't the case, let me know. | 20:10 |
| blogan | ajmiller: i'll have to test it out to be sure, it looks like it will but whenever ig et time ill let you know, gotta focus on the tempest stuff first though | 20:11 |
| ajmiller | NP. I'll be back full time tomorrow afternoon. I will also be testing some of this as part of the grenade work too... | 20:11 |
| *** vivek-ebay has joined #openstack-lbaas | 20:14 | |
| *** vivek-ebay has quit IRC | 20:19 | |
| *** sballe__ has quit IRC | 20:28 | |
| madhu_ | fnaval: how to run client tests? | 20:29 |
| *** bharath has joined #openstack-lbaas | 20:29 | |
| fnaval | the clients are used in teh actual tests | 20:30 |
| fnaval | they're used to make calls to the api | 20:30 |
| madhu_ | ah, I see..thanks for correcting | 20:30 |
| fnaval | yep - btw, it's time | 20:30 |
| fnaval | o/ | 20:30 |
| madhu_ | o/ | 20:31 |
| bharath | o/ | 20:31 |
| juliancash | o/ | 20:31 |
| juliancash | fnaval - Are you up for diving into google hangout quickly on Mondays? We'll keep it short. | 20:31 |
| fnaval | ah yes | 20:32 |
| fnaval | 1 sec | 20:32 |
| fnaval | whats the room | 20:32 |
| juliancash | Thanks! :-) | 20:32 |
| fnaval | link | 20:32 |
| juliancash | http://bit.ly/LBaaS_HP_Hangout | 20:32 |
| fnaval | k thank | 20:32 |
| fnaval | s | 20:32 |
| *** sbfox has quit IRC | 20:45 | |
| *** sbfox has joined #openstack-lbaas | 20:50 | |
| *** apuimedo has joined #openstack-lbaas | 20:59 | |
| openstackgerrit | Doug Wiegley proposed openstack/neutron-lbaas: Empty hooks for testing experimental lbaas job https://review.openstack.org/162757 | 21:04 |
| *** vivek-ebay has joined #openstack-lbaas | 21:04 | |
| *** apuimedo has quit IRC | 21:05 | |
| *** jorgem has quit IRC | 21:06 | |
| crc32 | is brandon around? | 21:08 |
| *** apuimedo has joined #openstack-lbaas | 21:19 | |
| *** ajmiller has quit IRC | 21:28 | |
| openstackgerrit | min wang proposed stackforge/octavia: change specs for amphora manager interface https://review.openstack.org/162771 | 21:39 |
| openstackgerrit | Brandon Logan proposed openstack/neutron-lbaas: Neutron LBaaS: Initial Load Balancer Tests https://review.openstack.org/153016 | 21:42 |
| openstackgerrit | Brandon Logan proposed openstack/neutron-lbaas: Added tox tempest environment https://review.openstack.org/162772 | 21:42 |
| *** sbfox has quit IRC | 21:43 | |
| dougwig | juliancash: sorry i missed the hangout. anything that needs my attention or status or whatnot? | 21:45 |
| *** vivek-ebay has quit IRC | 21:45 | |
| *** vivek-ebay has joined #openstack-lbaas | 21:45 | |
| *** vivek-ebay has quit IRC | 21:45 | |
| *** chlong has joined #openstack-lbaas | 21:45 | |
| *** vivek-ebay has joined #openstack-lbaas | 21:45 | |
| madhu_ | blogan: reviewed: https://review.openstack.org/#/c/153016/ | 21:46 |
| blogan | madhu_: did you cherry-pick or checkout that review? | 21:48 |
| madhu_ | checkout | 21:48 |
| blogan | did you run tox -e tempest? | 21:49 |
| blogan | and have devstack running? and have the correct tempest.conf? | 21:49 |
| blogan | i just added a README | 21:49 |
| blogan | bout to push that up | 21:49 |
| madhu_ | nope. I did run in this way in neutron_lbaas directory: nosetests -sv tests/tempest/v2/api/test_load_balancers.py | 21:49 |
| madhu_ | yes | 21:49 |
| madhu_ | btw the previous patch 21st set did work for me, checkedout 22nd patch and the ran the same I did for 21st set | 21:50 |
| blogan | yeah no more nosetests | 21:50 |
| madhu_ | oh, why is that so? | 21:51 |
| *** vivek-ebay has quit IRC | 21:51 | |
| *** vivek-eb_ has joined #openstack-lbaas | 21:51 | |
| blogan | tox -etempest will take care of pulling the right tempest down, and running the tests | 21:51 |
| madhu_ | okay, can I see the readme.txt? | 21:51 |
| blogan | give me a second to push it up | 21:51 |
| madhu_ | maybe I need to stop the devstack running, do some changes | 21:51 |
| madhu_ | sure | 21:51 |
| openstackgerrit | Brandon Logan proposed openstack/neutron-lbaas: Added tox tempest environment https://review.openstack.org/162772 | 21:52 |
| openstackgerrit | Brandon Logan proposed openstack/neutron-lbaas: Neutron LBaaS: Initial Load Balancer Tests https://review.openstack.org/153016 | 21:52 |
| *** chlong has quit IRC | 21:57 | |
| *** chlong has joined #openstack-lbaas | 21:57 | |
| *** vivek-eb_ has quit IRC | 22:00 | |
| *** vivek-ebay has joined #openstack-lbaas | 22:01 | |
| *** crc32 has quit IRC | 22:02 | |
| openstackgerrit | Brandon Logan proposed openstack/neutron-lbaas: Neutron LBaaS: Initial Listeners API tests https://review.openstack.org/155431 | 22:02 |
| openstackgerrit | Brandon Logan proposed openstack/neutron-lbaas: Neutron LBaaS: Creating Pool API Tests https://review.openstack.org/153001 | 22:04 |
| *** sbfox has joined #openstack-lbaas | 22:04 | |
| *** chlong has quit IRC | 22:04 | |
| openstackgerrit | Brandon Logan proposed openstack/neutron-lbaas: Neutron LBaaS: Adding test cases for Members. https://review.openstack.org/154237 | 22:06 |
| openstackgerrit | Brandon Logan proposed openstack/neutron-lbaas: Neutron LBaaS: Creating Health Monitor API Tests https://review.openstack.org/156773 | 22:07 |
| *** SumitNaiksatam has quit IRC | 22:11 | |
| blogan | madhu_: when you get a chance, could you do what that README says and let me know if it works or not for you? | 22:14 |
| Aish | blogan: I did a git checkout of 162772, and then tried running tox -etempest. but, it resulted in ERROR: InvocationError: '/bin/sh tools/pretty_tox.sh ' | 22:19 |
| blogan | hmm | 22:20 |
| blogan | that exists | 22:20 |
| Aish | I could run the tests manually though.. | 22:20 |
| blogan | how so manually? | 22:20 |
| Aish | using nosetests | 22:21 |
| blogan | with these new reviews? | 22:21 |
| Aish | nope.. | 22:21 |
| Aish | I just tried the initial listeners test with the new patch. And it ran fine | 22:23 |
| Aish | using nosetests | 22:23 |
| Aish | blogan: | 22:23 |
| *** openstack has joined #openstack-lbaas | 22:24 | |
| Aish | :) | 22:25 |
| Aish | Oh I am sry, i have tried with patch 11 not 12. | 22:27 |
| Aish | blogan: ^^ | 22:27 |
| madhu_ | blogan: will check that out | 22:28 |
| madhu_ | blogan: https://review.openstack.org/#/c/162772/ | 22:30 |
| madhu_ | Just reviewed | 22:30 |
| openstackgerrit | Brandon Logan proposed openstack/neutron-lbaas: Neutron LBaaS: Load Balancer Basic Scenario Test https://review.openstack.org/158904 | 22:30 |
| blogan | madhu_: it got added in that review you commented on | 22:31 |
| blogan | https://review.openstack.org/#/c/162772/2/neutron_lbaas/tests/tempest/etc/tempest.conf | 22:31 |
| madhu_ | oh, I see you changed the dev_tempconf to tempest.conf | 22:32 |
| blogan | yeah | 22:33 |
| blogan | didnt see the point in having it called dev | 22:33 |
| blogan | would just be confusing | 22:33 |
| madhu_ | ah I will recheck again and let you know then | 22:34 |
| madhu_ | yes, its confusin to have that dev tempest config file | 22:35 |
| blogan | if i can just find an environment variable to point tempest to read a config from, we wont have to copy that over | 22:36 |
| madhu_ | blogan: I have a question. Could you let me know what does tusker means? | 22:37 |
| madhu_ | Is that a service? | 22:37 |
| blogan | lol that config file is the first ive ever heard of it too | 22:38 |
| madhu_ | ha ha | 22:38 |
| blogan | ah tripleo | 22:38 |
| blogan | https://wiki.openstack.org/wiki/TripleO/Tuskar | 22:38 |
| madhu_ | oh nice, thanks for the link | 22:39 |
| openstackgerrit | min wang proposed stackforge/octavia: change specs for amphora manager interface https://review.openstack.org/162771 | 22:40 |
| *** apuimedo has quit IRC | 22:44 | |
| madhu_ | blogan: running tox taking longer than expected.. Is that so? | 22:45 |
| blogan | madhu_: listener tests take me 5 seconds | 22:45 |
| madhu_ | okay | 22:46 |
| madhu_ | let me revisit | 22:46 |
| *** bharath has quit IRC | 22:46 | |
| madhu_ | blogan: wow.. nicely put this after running the tests: tempest: commands succeeded | 22:48 |
| madhu_ | congratulations :) | 22:48 |
| madhu_ | it working for us | 22:48 |
| blogan | ah good | 22:49 |
| blogan | tox automatically puts that in :) | 22:49 |
| madhu_ | (thumbsup) | 22:56 |
| Aish | blogan: the initial listeners test ran fine with tox. :) | 22:59 |
| openstackgerrit | German Eichberger proposed stackforge/octavia: Implements the haproxy amphora agent api server https://review.openstack.org/160034 | 23:00 |
| madhu_ | blogan: I think we need to include "public_network_id = bb9e0fc9-0144-41a3-b17d-a63f829a1aba" in tempest.conf for scenarios tests to be successful | 23:21 |
| blogan | madhu_: more complex than that, that id will be different every time you run devstack | 23:22 |
| blogan | madhu_: when you run devstack with tempest, the tempest.conf is automatically built each time | 23:22 |
| madhu_ | exactly.. maybe assigning a variable to neutron net-list and use awk for printing the first first columns and second row | 23:22 |
| madhu_ | yeah, I see, even that works too | 23:23 |
| blogan | will probably have to do that, but that should go in with the scenario review | 23:23 |
| madhu_ | exactly | 23:23 |
| *** openstack has joined #openstack-lbaas | 23:24 | |
| madhu_ | let me know whether I can do the change or just wait for the changes to happen | 23:25 |
| blogan | well i wish there was just an environment variable i could point to to say this is where you can find the tempest config | 23:25 |
| blogan | but doesn't appear to be | 23:26 |
| madhu_ | oh | 23:26 |
| *** woodster_ has quit IRC | 23:30 | |
| blogan | madhu_, Aish: would yall say those tests (other than scenario) are good to go? | 23:31 |
| madhu_ | yes | 23:31 |
| madhu_ | it is good to go! | 23:31 |
| Aish | yes.. | 23:32 |
| madhu_ | I am bout to type this | 23:32 |
| blogan | great, just need it to get through the large check queue now and it can be merged | 23:32 |
| *** bharath has joined #openstack-lbaas | 23:33 | |
| madhu_ | absolutely | 23:33 |
| xgerman | +1 | 23:35 |
| *** chlong has joined #openstack-lbaas | 23:43 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!