Friday, 2016-01-29

*** swormke has joined #openstack-lbaas00:00
*** longstaff has joined #openstack-lbaas00:01
*** yamamoto_ has joined #openstack-lbaas00:02
swormkelongstaff: hey00:02
*** frzn_ has joined #openstack-lbaas00:02
openstackgerritMerged openstack/neutron-lbaas: Don't always run Octavia amphroa image build  https://review.openstack.org/26779800:05
openstackgerritBharath M proposed openstack/octavia: Use signing digest defined in config to sign CSR  https://review.openstack.org/27382500:06
bharathmrmwork: I fixed the patch.. also pushed a change to Master as you suggested00:11
*** ducttape_ has joined #openstack-lbaas00:18
*** diogogmt has quit IRC00:21
*** frzn_ has quit IRC00:22
*** yamamoto_ has quit IRC00:27
*** swormke has quit IRC00:31
*** lunarlamp has quit IRC00:37
*** longstaff has quit IRC00:48
*** chlong has joined #openstack-lbaas00:49
*** _cjones_ has quit IRC00:56
*** _cjones_ has joined #openstack-lbaas00:56
*** bdrich has quit IRC00:57
*** SumitNaiksatam has quit IRC00:57
*** drjones has joined #openstack-lbaas01:01
*** _cjones_ has quit IRC01:02
*** drjones has quit IRC01:06
*** _cjones_ has joined #openstack-lbaas01:06
*** ducttape_ has quit IRC01:06
*** bdrich has joined #openstack-lbaas01:09
*** amotoki has joined #openstack-lbaas01:12
*** amotoki has quit IRC01:15
*** piet_ has quit IRC01:17
*** yamamoto_ has joined #openstack-lbaas01:23
openstackgerritmin wang proposed openstack/octavia: Implements: blueprint anti-affinity server group  https://review.openstack.org/27234401:24
*** piet has joined #openstack-lbaas01:25
*** amotoki has joined #openstack-lbaas01:28
*** minwang2 has quit IRC01:30
*** hockeynut has quit IRC01:31
*** _cjones_ has quit IRC01:32
*** hockeynut has joined #openstack-lbaas01:34
*** yamamoto_ has quit IRC01:37
*** Aish has quit IRC01:43
*** ducttape_ has joined #openstack-lbaas01:43
*** jaff_cheng has joined #openstack-lbaas01:44
*** amotoki has quit IRC01:48
*** longstaff has joined #openstack-lbaas01:48
*** longstaff has quit IRC01:53
*** ducttape_ has quit IRC02:01
*** amotoki has joined #openstack-lbaas02:08
*** amotoki_ has joined #openstack-lbaas02:11
*** amotoki has quit IRC02:13
*** amotoki_ has quit IRC02:16
*** SumitNaiksatam has joined #openstack-lbaas02:16
*** woodster_ has quit IRC02:16
*** amotoki has joined #openstack-lbaas02:19
*** hang has joined #openstack-lbaas02:20
*** fnaval has quit IRC02:21
*** yamamoto_ has joined #openstack-lbaas02:23
*** jaff_cheng has quit IRC02:23
*** bdrich has quit IRC02:24
*** madhu_ak has quit IRC02:27
*** amotoki has quit IRC02:29
*** ducttape_ has joined #openstack-lbaas02:31
*** fnaval has joined #openstack-lbaas02:36
*** Aish has joined #openstack-lbaas02:47
*** Aish has left #openstack-lbaas02:47
*** ducttape_ has quit IRC03:00
*** ducttape_ has joined #openstack-lbaas03:01
*** bana_k has quit IRC03:03
*** bana_k has joined #openstack-lbaas03:10
*** amotoki has joined #openstack-lbaas03:11
*** ducttape_ has quit IRC03:11
*** ducttape_ has joined #openstack-lbaas03:18
*** Tiancheng has joined #openstack-lbaas03:37
*** yuanying_ has joined #openstack-lbaas03:38
*** yuanying has quit IRC03:40
*** fnaval has quit IRC03:53
*** allan_h has quit IRC03:56
*** Tiancheng_ has joined #openstack-lbaas04:00
*** Tiancheng has quit IRC04:00
*** ducttape_ has quit IRC04:02
*** yuanying has joined #openstack-lbaas04:06
*** yuanying_ has quit IRC04:06
*** piet has quit IRC04:07
xgermanblogan so you think somebody is running a cloud with a different endpoint_type for barbican? Our cloud has all the endpoint types for all services04:07
xgermanI am just not sure if somebody would set up a cloud where he would mix and match...04:08
*** yuanying has quit IRC04:11
bloganxgerman: you don't think someone would use an internal barbican endpoint type?04:14
blogani've heard that being a use case, where they don't want barbican to be exposed to the end user04:14
xgermanwell, in our cloud all endpoints are internal, admin, and public04:14
xgermanservices (like octavia) use internal04:14
xgermanblogan, gotcha so barbican would be internal, but nova public (and no internal endpoint?)04:15
bloganxgerman: there might be an internal endpoint, but for our staging environments we'll have some mixed and match ones for sure04:15
blogangranted the neutron-lbaas endpoint type for barbican is an edge case, its still posisble04:16
xgermanok, yeah I also commented on different endpoints for nova/neutron — not sure if we need to support different users as well04:17
*** yuanying has joined #openstack-lbaas04:17
xgermanthis certainly is adding complexity I am not sure we need...04:18
*** ducttape_ has joined #openstack-lbaas04:22
*** piet has joined #openstack-lbaas04:26
bloganmeh its not too complex, it breaks it down between services, we'll definitely need service_name different for nova and neutron04:28
blogani suppose the credentials could be all for one if that was the case, but especially for testing environments, the endpoint_type, and service_name will need to be per service04:29
xgermanyeah, I am with you04:29
xgermanI just was hoping it would be an easy patch (but is there ever one)04:30
bloganwell for this one you can just change the endpoint_type to be for nova and neutron, and a later patch can add the service_name, or does that not simplify it enough?04:32
xgermanyeah, I will do that04:36
*** ducttape_ has quit IRC04:36
*** ducttape_ has joined #openstack-lbaas04:36
bloganthanks!04:37
blogani envision even the need for overriding the nova and neutron url one day, but that day is not today04:37
*** numans has quit IRC04:41
xgermanwhich other service offers that ;-)04:44
bloganwhich other service uses other services like we do?04:46
*** Tiancheng_ has quit IRC04:48
*** Tiancheng_ has joined #openstack-lbaas04:49
*** prabampm has joined #openstack-lbaas04:52
*** ducttape_ has quit IRC04:56
*** minwang2 has joined #openstack-lbaas04:59
*** amotoki_ has joined #openstack-lbaas05:11
*** chlong has quit IRC05:12
*** piet has quit IRC05:21
*** mixos has joined #openstack-lbaas05:25
*** numans has joined #openstack-lbaas05:31
*** chlong has joined #openstack-lbaas05:33
*** hang has quit IRC05:33
*** jaff_cheng has joined #openstack-lbaas05:33
*** amotoki_ has quit IRC05:47
*** hang has joined #openstack-lbaas06:08
openstackgerritReedip proposed openstack/neutron-lbaas: [WIP]Add Timeout on Listeners- DO NOT REVIEW  https://review.openstack.org/27389606:09
*** jaff_cheng has quit IRC06:09
*** longstaff has joined #openstack-lbaas06:23
*** longstaff has quit IRC06:27
*** hang has quit IRC06:27
*** chlong has quit IRC06:28
*** hang has joined #openstack-lbaas06:28
*** rcernin has quit IRC06:29
*** jaff_cheng has joined #openstack-lbaas06:35
*** hang has quit IRC06:36
*** hang has joined #openstack-lbaas06:36
*** amotoki has quit IRC06:39
*** chlong has joined #openstack-lbaas06:39
*** amotoki has joined #openstack-lbaas06:39
*** jaff_cheng has quit IRC06:41
*** amotoki has quit IRC06:43
*** amotoki has joined #openstack-lbaas06:43
*** kobis has joined #openstack-lbaas07:06
openstackgerritBrandon Logan proposed openstack/octavia: Let drivers decide to use region or not  https://review.openstack.org/27391507:08
*** cgross has joined #openstack-lbaas07:16
*** amotoki has quit IRC07:22
*** amotoki has joined #openstack-lbaas07:24
*** rcernin has joined #openstack-lbaas07:25
*** minwang2 has quit IRC07:26
*** chlong has quit IRC07:32
openstackgerritBrandon Logan proposed openstack/octavia: Add service name conf option for neutron and nova  https://review.openstack.org/27392807:47
*** amotoki has quit IRC07:49
*** amotoki has joined #openstack-lbaas07:51
*** kobis has quit IRC07:56
*** eezhova has joined #openstack-lbaas07:59
*** kobis has joined #openstack-lbaas08:01
*** eranra has joined #openstack-lbaas08:06
*** kobis has quit IRC08:08
*** kobis has joined #openstack-lbaas08:20
*** mixos has quit IRC08:20
*** kobis has quit IRC08:43
*** jaff_cheng has joined #openstack-lbaas08:45
*** amotoki has quit IRC08:46
*** hang has quit IRC08:48
*** hang has joined #openstack-lbaas08:49
*** jaff_cheng has quit IRC08:52
*** amotoki has joined #openstack-lbaas09:00
*** diogogmt has joined #openstack-lbaas09:00
*** amotoki has quit IRC09:08
*** reedip is now known as outofmemory09:09
*** amotoki has joined #openstack-lbaas09:17
*** amotoki has quit IRC09:20
*** amotoki has joined #openstack-lbaas09:20
*** mariusv has joined #openstack-lbaas09:23
*** amotoki has quit IRC09:24
*** links has joined #openstack-lbaas09:38
*** links has quit IRC09:38
*** jaff_cheng has joined #openstack-lbaas09:40
*** hang has quit IRC09:42
*** yamamoto_ has quit IRC10:16
*** longstaff has joined #openstack-lbaas10:27
*** longstaff has quit IRC10:32
openstackgerritStephen Balukoff proposed openstack/octavia: Optimizations for tempest tests  https://review.openstack.org/27399010:33
*** Tiancheng_ has quit IRC10:40
*** Tiancheng has joined #openstack-lbaas10:43
*** Tiancheng has quit IRC10:49
*** Tiancheng has joined #openstack-lbaas10:58
*** Tiancheng has quit IRC11:03
*** kobis has joined #openstack-lbaas11:05
*** sbalukoff has quit IRC11:06
*** sbalukoff has joined #openstack-lbaas11:22
*** jaff_cheng has quit IRC11:24
*** doug-fish has quit IRC11:51
*** rcernin has quit IRC12:18
*** rcernin has joined #openstack-lbaas12:18
*** rtheis has joined #openstack-lbaas12:27
*** yamamoto has joined #openstack-lbaas12:39
*** yamamoto has quit IRC12:43
*** yamamoto has joined #openstack-lbaas12:48
*** yamamoto has quit IRC12:59
*** yamamoto has joined #openstack-lbaas13:05
*** ducttape_ has joined #openstack-lbaas13:10
*** piet has joined #openstack-lbaas13:24
*** longstaff has joined #openstack-lbaas13:30
*** yamamoto has quit IRC13:31
*** doug-fish has joined #openstack-lbaas13:31
*** ducttape_ has quit IRC13:31
*** doug-fish has quit IRC13:32
*** doug-fish has joined #openstack-lbaas13:32
*** longstaff has quit IRC13:34
*** yamamoto has joined #openstack-lbaas13:36
*** yamamoto has quit IRC13:38
*** yamamoto has joined #openstack-lbaas13:47
*** diogogmt_ has joined #openstack-lbaas13:47
*** diogogmt has quit IRC13:48
*** diogogmt_ is now known as diogogmt13:48
*** yamamoto has quit IRC13:53
*** yamamoto has joined #openstack-lbaas13:55
*** Aish has joined #openstack-lbaas14:00
*** yamamoto has quit IRC14:05
*** ducttape_ has joined #openstack-lbaas14:06
*** diogogmt has quit IRC14:06
*** yamamoto has joined #openstack-lbaas14:14
*** yamamoto has quit IRC14:17
*** diogogmt has joined #openstack-lbaas14:21
*** yamamoto has joined #openstack-lbaas14:22
*** diogogmt has quit IRC14:25
*** longstaff has joined #openstack-lbaas14:26
*** neelashah has joined #openstack-lbaas14:26
*** piet has quit IRC14:27
*** yamamoto has quit IRC14:29
*** piet has joined #openstack-lbaas14:39
*** yamamoto has joined #openstack-lbaas14:43
*** yamamoto has quit IRC14:50
*** ducttape_ has quit IRC15:01
*** yamamoto has joined #openstack-lbaas15:03
*** yamamoto has quit IRC15:03
*** mixos has joined #openstack-lbaas15:07
*** yamamoto has joined #openstack-lbaas15:08
*** yamamoto has quit IRC15:08
*** mixos has quit IRC15:16
*** bdrich has joined #openstack-lbaas15:17
*** amotoki has joined #openstack-lbaas15:19
*** mixos has joined #openstack-lbaas15:19
*** mixos has quit IRC15:23
*** ducttape_ has joined #openstack-lbaas15:27
*** liamji has joined #openstack-lbaas15:37
liamjihello, I have a question about using lbaas v2. I find the return of the command 'neutron lbaas-pool-show <pool-id>' has no status property. How can I get the status of Pool? thanks15:39
*** amotoki has quit IRC15:42
*** amotoki has joined #openstack-lbaas15:50
*** allan_h has joined #openstack-lbaas15:51
johnsomsbalukoff around?15:55
*** alhu_ has joined #openstack-lbaas15:55
*** allan_h has quit IRC15:57
*** amotoki has quit IRC15:58
*** alhu_ has quit IRC16:02
*** allan_h has joined #openstack-lbaas16:05
*** mixos has joined #openstack-lbaas16:12
*** minwang2 has joined #openstack-lbaas16:12
*** numans has quit IRC16:13
*** mgarza has joined #openstack-lbaas16:14
*** amotoki has joined #openstack-lbaas16:17
*** alejandrito has joined #openstack-lbaas16:19
*** diogogmt has joined #openstack-lbaas16:22
*** diogogmt has quit IRC16:26
*** rcernin has quit IRC16:27
*** bana_k has quit IRC16:31
*** rtheis_ has joined #openstack-lbaas16:34
*** rtheis has quit IRC16:36
*** piet has quit IRC16:38
*** minwang2 has quit IRC16:44
*** piet has joined #openstack-lbaas16:59
*** minwang2 has joined #openstack-lbaas17:10
*** liamji has quit IRC17:11
*** _cjones_ has joined #openstack-lbaas17:13
*** ducttape_ has quit IRC17:18
*** _cjones_ has quit IRC17:20
*** _cjones_ has joined #openstack-lbaas17:20
*** woodster_ has joined #openstack-lbaas17:29
*** fnaval has joined #openstack-lbaas17:34
*** ducttape_ has joined #openstack-lbaas17:41
*** ducttape_ has quit IRC17:42
*** ducttape_ has joined #openstack-lbaas17:43
*** madhu_ak has joined #openstack-lbaas17:45
*** bana_k has joined #openstack-lbaas17:50
*** Aish has quit IRC17:55
*** amotoki has quit IRC18:06
*** neelashah has quit IRC18:11
*** SumitNaiksatam has quit IRC18:17
*** diogogmt has joined #openstack-lbaas18:21
*** diogogmt has quit IRC18:26
dougwiganyone have a spare minute to peek at this? http://logs.openstack.org/63/274163/1/check/gate-neutron-lbaasv2-dsvm-minimal/18e3932/console.html18:27
dougwiggate breakage18:27
*** jwarendt has joined #openstack-lbaas18:37
*** bana_k has quit IRC18:42
*** bana_k has joined #openstack-lbaas18:43
sbalukoffjohnsom: I'm sorta-around. Have some training to attend today, but thankfully it's all online web-cast stuff.18:52
johnsomsbalukoff You will be assimilated...  I had a question about the bug you filed: https://bugs.launchpad.net/bugs/153943918:53
openstackLaunchpad bug 1539439 in octavia "TestHeartbeatUDP.test_check intermittently fails tox tests" [Undecided,New]18:53
sbalukoffSure! what's the question?18:54
johnsomsbalukoff what branch and commit # were you seeing it on?18:54
sbalukoffI have verified it happening on master.18:54
sbalukoffIs no-one else seeing that test fail occasionally?  (I'm thinking about 1/10th of the time--  enough to be annoying but not seriously gum up the works.)18:55
johnsomHmmm, I ask because there is this patch for that issue: https://review.openstack.org/#/c/259592/18:55
*** pcaruana has joined #openstack-lbaas18:57
*** SumitNaiksatam has joined #openstack-lbaas18:58
*** markd_ has joined #openstack-lbaas18:59
sbalukoffOk, let me look at that.19:00
sbalukoffAlso, in the mean time, trust me, y'all want to review and merge this:  https://review.openstack.org/#/c/273990/19:00
johnsomYeah, could be I missed something, but it resolved it for me.19:00
*** markd_ has quit IRC19:04
*** bana_k has quit IRC19:06
sbalukoffAlso, if I could get a +2 on this little non-functionality-impacting patch, that'd get it off my radar and fix a pet peeve of mine that I've been seeing as I run some of these tox tests in a loop: https://review.openstack.org/#/c/272910/19:07
sbalukoff(Noticing this trying to track down that damned intermitent session persistence tempest test bug-- there's an extra space in that message!)19:09
*** bana_k has joined #openstack-lbaas19:15
*** pcaruana has quit IRC19:24
*** ducttape_ has quit IRC19:31
*** ducttape_ has joined #openstack-lbaas19:34
*** pcaruana has joined #openstack-lbaas19:37
sbalukoffThanks for the quick review, johnsom! I have comments.19:40
*** crc32 has joined #openstack-lbaas19:47
*** crc32 has quit IRC19:47
*** crc32 has joined #openstack-lbaas19:47
johnsomblogan Can you take a look at the discussion on https://review.openstack.org/#/c/273990/1/octavia/common/config.py?19:54
johnsomI think you had made some comments about being careful with the rate we hit the APIs in the past.19:55
bloganjohnsom: welp, obiously depends on how many threads are going, we do have bursts of many creates that would probably cause our service account to get rate limited by nova unless they set our account to allow that much19:58
sbalukoffblogan: Are you talking production, or in devstack?19:58
bloganproduction of course, and it is a config option so we'd have to change it (and would) so i'm not too worried about it being 119:59
bloganthough i'm not sure we should making the default config options always be assuming devstack, neutron has had some issues with that20:00
sbalukoffI'm happy to put a compromise in place: Make the devstack/plugin.sh adjust those values to be more optimal for devstack runs (ie. jenkins tempest tests), but leave the defaults where they are for regular instals.20:00
blogansbalukoff: oh great idea, that is appropriate place to set the devstack defaults20:01
johnsomYeah, that works for me too20:01
sbalukoffOk, I'll see about getting a new patchset up within the hour.20:01
bloganthanks20:01
sbalukoff(I was surprised at how much this sped stuff up for this continuous run where I'm trying to get that stupid session persistence tempest bug to trigger.)20:02
sbalukoff(That's the reason I'm a bit hot to trot on that particular patch.)20:03
*** pcaruana has quit IRC20:06
johnsomCurious why you need that hook script....20:06
johnsomAh, I see why20:07
*** pcaruana has joined #openstack-lbaas20:09
openstackgerritStephen Balukoff proposed openstack/octavia: Optimizations for tempest tests  https://review.openstack.org/27399020:13
sbalukoffBecause otherwise dhclient grabs DNS server information from DHCP. :P20:14
johnsomI saw that you are blowing away the main script that collects that data.  The new empty one was the question20:18
*** pcaruana has quit IRC20:19
*** jwarendt has quit IRC20:21
*** crc32 has quit IRC20:21
*** crc32 has joined #openstack-lbaas20:22
sbalukoffOh! Yes-- I'm unsure what dhclient will do if it doesn't have a make_resolv_conf() since I suspect it's hard-coded to call said function. This way it can still call a function (which does nothing) and be happy.20:22
*** ducttape_ has quit IRC20:31
*** kobis has quit IRC20:32
*** kobis has joined #openstack-lbaas20:33
bana_kHi I was working on the this https://bugs.launchpad.net/octavia/+bug/153422320:39
openstackLaunchpad bug 1534223 in octavia "compile_amphora_details hardcodes topology as consts.TOPOLOGY_SINGLE" [Medium,Confirmed] - Assigned to Banashankar (bkalebe)20:40
bana_kbut I have couple of questions regarding how to handle this in case of pre created amphora pool.20:40
bana_kas of now I see, we are looking for vrrp or some other config file to determine the topology.20:41
*** crc32 has quit IRC20:42
bana_kwhat if we change the config to have single topology and we will be still using the amps from the pool, which were spawned when the config was active-standby20:42
*** crc32 has joined #openstack-lbaas20:43
sbalukoffbana_k: I'm not that familiar with the code there, but can you answer: Is there a difference between amphorae spawned for the spares pool when the topology is different (ie. are active-standby-spawned spare amphorae different from stand-alone-spawned spares in any way?)20:44
bana_kyes it is, I am trying to find the code.20:45
bana_ksbalukoff : gimme 5-10 mins :)20:45
sbalukoffOk, thanks.20:46
sbalukoff(I have a meeting in 15 minutes, so I might not be responsive then.)20:46
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-lbaas: Updated from global requirements  https://review.openstack.org/27280520:47
bana_koh ok.20:47
*** ducttape_ has joined #openstack-lbaas20:49
*** kobis has quit IRC20:51
*** piet has quit IRC20:54
*** kobis has joined #openstack-lbaas20:55
*** kobis has quit IRC20:56
bana_ksbalukoff: going for lunch, will look for the code once back.20:57
*** piet has joined #openstack-lbaas20:57
*** kobis has joined #openstack-lbaas20:59
*** crc32 has quit IRC21:03
*** crc32 has joined #openstack-lbaas21:04
elarsondoes anyone have a link to the lbaas talk from the tokyo summit?21:04
*** ajmiller has quit IRC21:04
*** ajmiller has joined #openstack-lbaas21:06
*** kobis has quit IRC21:06
*** mixos has quit IRC21:23
johnsomelarson Yes, just a sc21:25
elarsonfound it!21:26
elarsonjohnsom: ^^^21:26
johnsomhttps://www.openstack.org/summit/tokyo-2015/videos/presentation/load-balancing-as-a-service-liberty-and-beyond21:26
elarsonhad to search for "load balancing" not lbaas :)21:26
johnsomYep21:26
elarsonjohnsom: thanks for finding it!21:26
johnsomNP21:26
fnavalsbalukoff: hi sbalukoff thanks for the review; question on the deprecated paths: how would the paths change with shared pools?  would 'v1' be replaced with 'v2'?21:30
*** SumitNaiksatam has quit IRC21:32
*** rcernin has joined #openstack-lbaas21:36
sbalukofffnaval:  No, pools and all their children are no longer accessed from within a listener's context.  So in otherwords this deprecated URL:  /v1/loadbalancers/<uuid>/listeners/<uuid>/pools/<uuid>/members/<uuid>  becomes this:   /v1/loadbalancers/<uuid>/pools/<uuid>/members/<uuid>21:38
sbalukofffnaval:  Though of course if your patch lands before the shared pools patch, the URLs you have in it are appropriate.21:39
sbalukoff...and honestly, if your patch lands after the shared pools patch, those URLs will still work; They'll just be deprecated.21:40
fnavalah ok understood.  if I make the change now, will it still work?  or only work when the shared pools land? sbalukoff21:41
johnsomsbalukoff No worries on the gate, it's a gate wide issue21:41
sbalukoffjohnsom: Oh, ok!21:41
sbalukoffThanks for saving me the trouble of spending a couple hours trying to reproduce a phantom problem. ;)21:42
johnsomdougwig was looking for someone to take a look at it earlier21:42
sbalukofffnaval:  If you make the change now, you are effectively making your patch dependent on the shared pools patch.  So... if you make the change now, it won't work with master right now. :)21:42
johnsomunfortunately I'm tied up on an internal deliverable today21:43
sbalukofffnaval:  So yes, you might want to hold off on my suggestion there.21:43
sbalukoffjohnsom: "at it"?  fnaval's patch? That's a good idea-- Octavia needs independent tempest tests. :)21:43
fnavalk understood, sbalukoff - so I think I won't have to make any changes to the patch; but I'll keep note of the change as soon as shared pools land.  I can make the required changes.21:43
sbalukofffnaval: Sounds good!21:43
sbalukofffnaval: On my other note about your patch....21:44
fnavalsbalukoff: cool thanks sir.21:44
fnavalsbalukoff: oh theres another change needed?21:44
sbalukofffnaval: Is there a really basic tempest test that can be run right now with your patch?  (I'd just like to make sure the "skeleton" is complete, as it were.)21:44
johnsomsbalukoff Didn't follow that last question21:44
fnavalsbalukoff: this would be worked on next: https://review.openstack.org/#/c/182554/21:44
sbalukoffjohnsom: You said:  "dougwig was looking for someone to take a look at it earlier"  I was asking what the "it" was in that sentence.21:45
johnsomsbalukoff ah, it = the gate issue21:45
dougwiglbaas gate is bombed.21:45
fnavalthat next patch will have the scenario test sbalukoff21:46
sbalukofffnaval: Holy moly! That's 15000 lines of code!21:46
fnavalsbalukoff: yeah, that's gonna be trimmed a lot21:46
sbalukoffjohnsom: Oh, ok!21:46
sbalukoffdougwig: D'oh!21:46
fnavali'm probably going to just split it up.  the original PR had a couple hundred lines.21:46
sbalukofffnaval: I highly recommend splitting it up.  But I assume that monster patch is dependent on this skeleton patch, right?21:47
sbalukoff(Adding meat to the bones, as it were?)21:47
fnavalyes, it's dependent on it.21:48
fnavalyep meat to the bones21:48
sbalukoffOk, then that makes me less reluctant to merge the skeleton patch.21:48
sbalukoff:)21:48
fnaval=)21:48
fnavalso, we jumped into doing the scenario tests vs. functional basic tests since a lot of that would be done via the neutron-lbaas tests.21:49
fnavalin a blackbox indirect way21:49
fnavalthat next patch would be just an initial basic scenario test, but it seems that others kept piling more code on top of it.21:49
fnavalI'm going to clean it up and have it do just the basic scenario test with subsequent separate PRs to add more tests to it.21:50
*** diogogmt has joined #openstack-lbaas21:51
*** xilch has joined #openstack-lbaas21:51
sbalukofffnaval: Honestly, I'd prefer to get an initial basic scenario test in first, then we can back-fill more complicated tests. This would also mean that we are more quickly in a position to harass people to write scenario tests for new functionality they add. (Right now, we can't enforce that.)21:51
*** diogogmt has quit IRC21:52
sbalukofffnaval: Awesome!21:52
fnavalYep!21:52
*** diogogmt has joined #openstack-lbaas21:53
*** alejandrito has quit IRC21:54
*** xilch has quit IRC21:56
fnavalsbalukoff: https://review.openstack.org/#/c/172199   here's the change again for your convenience!  thanks for the help!21:57
sbalukofffnaval: Thanks!21:57
*** madhu_ak has quit IRC22:00
*** ducttape_ has quit IRC22:01
openstackgerritStephen Balukoff proposed openstack/octavia: Optimizations for tempest tests  https://review.openstack.org/27399022:02
*** crc32 has quit IRC22:05
*** doug-fis_ has joined #openstack-lbaas22:10
*** doug-fi__ has joined #openstack-lbaas22:14
*** doug-fish has quit IRC22:14
openstackgerritDoug Wiegley proposed openstack/neutron-lbaas: Janky hack to get gate passing  https://review.openstack.org/27427222:14
openstackgerritCedric Brandily proposed openstack/neutron-lbaas: Delete tempest fork  https://review.openstack.org/27381722:16
*** doug-fis_ has quit IRC22:17
*** doug-fi__ has quit IRC22:18
johnsomdougwig Thanks for that patch.  Note, Cedric's patch nukes that code, so maybe we want to merge it instead22:20
dougwigjohnsom: it's still marked as a wip.  if it's ready, then certainly yes.22:21
dougwigjohnsom: plus, it's not up to me whether my patch merges.  :)22:21
johnsomAh, screen refresh timing.  Got it.22:21
johnsomTrue...  But a nice dialog catches things I miss, like the WIP flag.  Anyway, watching your patch, will +2 when it gets through the gate22:23
*** diogogmt has quit IRC22:25
*** bdrich has quit IRC22:26
*** doug-fish has joined #openstack-lbaas22:26
*** madhu_ak has joined #openstack-lbaas22:27
dougwigjohnsom: i don't actually have a ton of hope for my patch. if the v3 identity client has never been used, what are the odds that one patch is all it needs?22:28
*** sbalukoff has quit IRC22:29
*** doug-fish has quit IRC22:31
ajmillerdoug-fish ping I just put a -1 on the listener pane patch.22:32
ajmillercould you please have a look and let me know if I'm thinking about this wrong.22:33
*** rtheis_ has quit IRC22:34
*** ducttape_ has joined #openstack-lbaas22:34
johnsomdougwig Missing urllib import22:45
dougwigdang, i knew that was going to bite me.22:46
openstackgerritDoug Wiegley proposed openstack/neutron-lbaas: Janky hack to get gate passing  https://review.openstack.org/27427222:47
*** sbalukoff has joined #openstack-lbaas22:57
*** alejandrito has joined #openstack-lbaas23:01
*** yamamoto has joined #openstack-lbaas23:08
*** yamamoto has quit IRC23:12
bana_ksbalukoff: https://github.com/openstack/octavia/blob/81befc2e7d1593b3acaa40bac214735ee0ec8515/octavia/controller/worker/flows/load_balancer_flows.py#L9523:16
bana_khttps://github.com/openstack/octavia/blob/81befc2e7d1593b3acaa40bac214735ee0ec8515/octavia/controller/worker/flows/load_balancer_flows.py#L5323:17
bana_khttps://github.com/openstack/octavia/blob/5e44f2f1b93faee12043fb5b933f01cf375e9717/octavia/amphorae/backends/agent/api_server/listener.py#L17023:17
bana_khttps://github.com/openstack/octavia/blob/8daef76effd4ea77ecfe35148f500c21221ce8f1/octavia/amphorae/drivers/haproxy/jinja/templates/haproxy_proxies.template#L7323:18
johnsombana_k Hi23:18
bana_kHi johnsom23:19
johnsomSo, when we create spare amphora, we call controller_worker create_amphora, which runs the amphora_flows.get_create_amphora_flow23:19
johnsomIt doesn't call that post method23:20
johnsomget_post_lb_amp_association_flow23:20
johnsomWe don't give the amp a "topology role" until failover time23:20
johnsomOr amp boot time23:21
bana_kthats only role you mean master/backup right ?23:21
johnsomRight23:21
bana_kwhat about the topology, like single or active-standby23:22
bana_kI thought we decide that at the time of amp creation23:22
johnsomCurrently that is globaly set per controller (not the right final answer, but what we have)23:23
johnsomIt's decided based on the config at load balancer create time23:23
openstackgerritGerman Eichberger proposed openstack/octavia: Replace depcricated Nova networks with Nova interfaces  https://review.openstack.org/27373323:23
bana_kyea so, once we have spare amphora and then we change the config to have a topology type how we should handle that23:24
bana_kas spare will be having the old topology type, right ?23:24
johnsomSpare has no topology type23:24
johnsomThis patch may help, it updates the failover flow for act/standby.  There you can see how I stamp a topology role on an amp(spare or booted)23:26
johnsomAh, now I remember the issue you are working on....23:26
johnsomRight, config file.23:26
bana_kwhat do we this for https://github.com/openstack/octavia/blob/81befc2e7d1593b3acaa40bac214735ee0ec8515/octavia/controller/worker/flows/load_balancer_flows.py#L5323:26
bana_kyes config file23:26
johnsomagent.conf gets created at nova boot time and passed via config drive so that the amp agent has it's settings for things like health manager23:27
bana_koh ok.23:30
johnsomThis has to do with plug vip timing.  plug vip needs to know if it is a act/standby or not.23:30
*** mgarza has quit IRC23:30
bana_kyea23:30
johnsomI think I'm getting close to caught up23:30
*** longstaff has quit IRC23:30
bana_k2 bugs, one is hardcoded topology type in compile_amphora_details and other is ifconfig up/down for vrrp port23:31
johnsomYep23:32
johnsomHere is an idea.  It's a bit of work however....23:32
bana_khttps://bugs.launchpad.net/octavia/+bug/152738823:32
openstackLaunchpad bug 1527388 in octavia "Active/Standby vip interface is live on both instances" [Medium,New] - Assigned to Banashankar (bkalebe)23:32
johnsomI would consider adding a method to the amphora API that allows you to update the amphora-agent.conf.  Should be easy to copy the way we update the haproxy conf file.23:33
johnsomThen I would add a task that renders and pushes down a new agent.conf to the amp(s).23:34
johnsomThen add that task to the create load balancer flow, before the networking tasks23:34
bana_kok sir, got it :)23:34
johnsomWe will probably want this later anyway to push down updates for the list of health manger IPs.23:34
bana_kll try to do23:35
johnsomDoes that make sense?23:35
johnsomI'm just "thinking out loud" here....23:35
bana_kstill need to figure out why vrrp standby's port is not up after master fails23:35
johnsomAre you talking about what it logs?23:36
bana_kNo, if you remember we did test whether the standbys port become active after the master fails, which did happen.23:37
bana_kn we were thinking of explore the vrrp conf to add the correct interface name n al23:37
bana_kdidn*23:38
johnsomAh23:38
johnsomRight, the interface name thing...23:38
bana_kyea, just got some free cycles to work on it, so ll try to explore that part23:38
*** alejandrito has quit IRC23:41
*** Aish has joined #openstack-lbaas23:43
*** ducttape_ has quit IRC23:52
openstackgerritGerman Eichberger proposed openstack/octavia: Fixes intermittend udp heartbeat test failure  https://review.openstack.org/27429423:58
xgerman^^ sbalukoff23:59

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