Friday, 2015-09-18

*** SumitNaiksatam has quit IRC00:00
*** vivek-eb_ has joined #openstack-lbaas00:18
*** vivek-ebay has quit IRC00:18
*** bana_k has quit IRC00:29
*** mlavalle has quit IRC00:32
*** Aish has joined #openstack-lbaas00:42
*** Aish has quit IRC00:51
*** vivek-eb_ has quit IRC00:53
*** ganeshna has joined #openstack-lbaas01:10
*** Aish has joined #openstack-lbaas01:12
*** Aish has quit IRC01:12
*** bharathm has quit IRC01:17
*** minwang2 has quit IRC01:18
*** vjay15 has joined #openstack-lbaas01:20
*** ganeshna has quit IRC01:22
*** madhu_ak has quit IRC01:23
*** apuimedo has quit IRC01:25
*** minwang2 has joined #openstack-lbaas01:28
*** SumitNaiksatam has joined #openstack-lbaas01:45
*** vjay15 has quit IRC01:52
*** KunalGandhi has joined #openstack-lbaas02:01
*** KunalGan_ has joined #openstack-lbaas02:13
*** KunalGandhi has quit IRC02:15
*** ganeshna has joined #openstack-lbaas02:40
*** sbalukoff has quit IRC02:42
xgermanblogan doubt we have stats in octavia — should we skip that test? neutron_lbaas.tests.tempest.v2.api.test_load_balancers_non_admin.LoadBalancersTestJSON.test_get_load_balancer_stats[smoke]02:54
xgermanhttp://logs.openstack.org/65/223265/11/check/gate-neutron-lbaasv2-dsvm-api/4546426/console.html02:54
*** johnsom_ has joined #openstack-lbaas03:09
*** ganeshna has quit IRC03:18
*** ganeshna has joined #openstack-lbaas03:19
*** minwang2 has quit IRC03:28
*** minwang2 has joined #openstack-lbaas03:35
openstackgerritMichael Johnson proposed openstack/octavia: Don't build amphora image if one is available  https://review.openstack.org/22392203:45
johnsom_The stats issue could be caused by controller_ip_port_list not being filled in.  https://bugs.launchpad.net/octavia/+bug/149003303:56
openstackLaunchpad bug 1490033 in octavia "Devstack scripts need to enable lb-network controller IP" [Critical,New] - Assigned to Brandon Logan (brandon-logan)03:56
johnsom_I'm going to dig through the logs a bit and see03:56
*** Guest58973 has quit IRC04:04
*** Brian has joined #openstack-lbaas04:12
*** Brian is now known as Guest4138704:13
rm_workhmm so we're closer04:24
rm_workbut octavia is still failing some of the scenario and load_balancer tests04:24
rm_worknot to mention the ones we don't have jobs for yet04:25
*** minwang2 has quit IRC04:27
johnsom_Yep.  I think on the load balancer tests it is just the stats test04:28
*** minwang2 has joined #openstack-lbaas04:28
rm_workdo you know how to decode what this means for gate job selection?04:29
rm_workbranch: ^(?!stable/(?:juno|kilo)).*$04:29
johnsom_Passed: 37 Skipped: 4 Failed: 104:29
rm_workyeah on the scenario there were a few but it looked like a cascade04:29
rm_workpossibly from a missing WAIT or something04:29
johnsom_Looks like regex04:29
rm_workyes04:30
rm_worklol04:30
rm_worki can kinda parse it04:30
rm_workbut04:30
rm_worki am not sure exactly what it means for zuul04:30
rm_worki will ask in infra04:30
*** crc32 has joined #openstack-lbaas04:32
rm_workcool04:36
rm_workjohnsom_: https://review.openstack.org/#/c/225000/04:36
rm_workthere's the other side04:36
rm_workrealized i forgot to push that up before because i was trying to verify the regex was right04:36
johnsom_Cool04:36
rm_workonce "various fixes" and the "split up gate tests" patches merge, we can get that merged04:37
rm_workand then continue testing "set octavia as the reference"04:37
*** vivek-ebay has joined #openstack-lbaas04:39
johnsom_Looks good to me04:41
rm_workwe still never found out if people in infra would actually ... refuse to do this04:43
rm_workhopefully shouldn't be an issue04:43
rm_workbut we did just EXPLODE the number of instances our changes take04:43
johnsom_Yes, but at least they turn around faster...04:43
johnsom_stats error is a sqlalchemy error: InvalidRequestError: This session is in 'prepared' state; no further SQL can be emitted within this transaction.04:46
*** crc32 has quit IRC04:52
*** minwang2 has quit IRC05:00
*** kiran-r has joined #openstack-lbaas05:01
*** minwang2 has joined #openstack-lbaas05:02
*** kiran-r has quit IRC05:07
*** sbalukoff has joined #openstack-lbaas05:15
*** vivek-ebay has quit IRC05:22
*** ganeshna has quit IRC05:29
*** ganeshna has joined #openstack-lbaas05:29
*** johnsom_ has quit IRC05:29
*** Youcef has quit IRC05:31
*** openstackgerrit has quit IRC05:31
*** openstackgerrit has joined #openstack-lbaas05:31
*** Youcef has joined #openstack-lbaas05:32
*** KunalGandhi has joined #openstack-lbaas05:32
*** minwang2 has quit IRC05:33
*** numans has joined #openstack-lbaas05:34
*** KunalGan_ has quit IRC05:35
*** KunalGandhi has quit IRC05:37
*** Youcef has quit IRC05:42
*** Youcef has joined #openstack-lbaas05:46
*** kiran-r has joined #openstack-lbaas05:48
*** Youcef has quit IRC05:59
*** Youcef has joined #openstack-lbaas06:05
*** clev-away has quit IRC06:27
*** sbalukoff has quit IRC06:27
*** clev has joined #openstack-lbaas06:27
*** kiran-r has quit IRC06:32
*** kiran-r has joined #openstack-lbaas06:33
openstackgerritAdam Harwell proposed openstack/neutron-lbaas: Split up gate tests for LBaaSv2  https://review.openstack.org/22434506:57
*** woodster_ has quit IRC07:09
*** HenryG has quit IRC07:21
*** bharathm has joined #openstack-lbaas07:23
*** ganeshna has quit IRC07:30
*** ganeshna has joined #openstack-lbaas07:30
*** Youcef has quit IRC07:34
*** Youcef has joined #openstack-lbaas07:38
*** sbalukoff has joined #openstack-lbaas08:10
*** HenryG has joined #openstack-lbaas08:12
*** HenryG has quit IRC08:16
*** HenryG has joined #openstack-lbaas08:30
*** chlong has quit IRC08:47
*** Youcef has quit IRC08:47
*** Youcef has joined #openstack-lbaas08:50
*** kiran-r has quit IRC08:54
*** Youcef has quit IRC08:59
*** kiran-r has joined #openstack-lbaas09:01
*** ganeshna has quit IRC09:04
*** Youcef has joined #openstack-lbaas09:05
*** ganeshna has joined #openstack-lbaas09:06
*** Youcef has quit IRC09:20
*** Youcef has joined #openstack-lbaas09:27
rm_workjohnsom: not sure what is wrong with https://review.openstack.org/#/c/225000/09:41
rm_workI am heading to bed now, will take another crack at fixing it when I wake up (lunchtime or so for you) unless you need it sooner09:41
*** Youcef has quit IRC10:26
*** Youcef has joined #openstack-lbaas10:36
*** Guest41387 has quit IRC10:44
*** johnsom_ has joined #openstack-lbaas11:02
*** johnsom has quit IRC11:06
*** amotoki has joined #openstack-lbaas11:14
*** kiran-r has quit IRC11:22
*** ganeshna has quit IRC11:25
*** chlong has joined #openstack-lbaas12:06
*** chlong has quit IRC12:07
*** chlong has joined #openstack-lbaas12:07
*** Brian has joined #openstack-lbaas12:13
*** Brian is now known as Guest9551012:14
*** Guest95510 has quit IRC12:18
*** denghui has joined #openstack-lbaas12:48
*** apuimedo has joined #openstack-lbaas13:09
*** denghui has quit IRC13:26
*** denghui has joined #openstack-lbaas13:28
*** woodster_ has joined #openstack-lbaas13:32
*** openstackgerrit has quit IRC13:46
*** clev is now known as clev-away13:46
*** openstackgerrit has joined #openstack-lbaas13:46
*** johnsom_ has quit IRC14:30
*** johnsom has joined #openstack-lbaas14:30
*** vjay15 has joined #openstack-lbaas14:48
*** ganeshna has joined #openstack-lbaas14:56
*** denghui has quit IRC15:05
*** numans has quit IRC15:10
*** vjay15 has quit IRC15:19
*** vjay15 has joined #openstack-lbaas15:34
*** vjay15 has quit IRC15:40
*** akamyshnikova has quit IRC15:42
*** mlavalle has joined #openstack-lbaas15:43
*** minwang2 has joined #openstack-lbaas15:49
xgermanjohnsom, rm_work, blogan: If we don’t get the issues resolved by Monday we are out :-(15:49
*** vjay15 has joined #openstack-lbaas15:53
*** johnsom_ has joined #openstack-lbaas15:57
*** minwang2 has quit IRC16:00
*** SumitNaiksatam has quit IRC16:02
*** minwang2 has joined #openstack-lbaas16:03
*** ganeshna has quit IRC16:05
*** ganeshna has joined #openstack-lbaas16:06
*** ganeshna has quit IRC16:09
*** ganeshna has joined #openstack-lbaas16:09
johnsom_rm_work I have pushed up a patchset with the job not defined issue fixed16:16
*** ganeshna_ has joined #openstack-lbaas16:22
*** ganeshna has quit IRC16:24
bloganhello, sorry was away yesterday, had to go out of town for a funeral16:24
*** vjay15 has quit IRC16:25
*** ganeshna has joined #openstack-lbaas16:26
bloganso whats the status on the jobs right now?16:27
bloganintermitten api test failures?16:27
*** ganeshna_ has quit IRC16:28
*** ganeshna has quit IRC16:33
*** johnsom__ has joined #openstack-lbaas16:35
*** johnsom_ has quit IRC16:36
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-lbaas: Updated from global requirements  https://review.openstack.org/22522716:39
*** minwang2_ has joined #openstack-lbaas16:39
*** minwang2 has quit IRC16:42
*** minwang2_ is now known as minwang216:42
*** KunalGandhi has joined #openstack-lbaas16:42
johnsom__blogan Sorry to hear that.16:43
*** barclaac has joined #openstack-lbaas16:43
johnsom__blogan The stats test is failing with a SQL alchemy issue.  I didn't have an environment to test with last night so didn't go after it16:43
bloganlink for logs?16:44
xgermanI think at that point we should just skip that test and file a bug16:46
xgermanstats are not that important16:46
*** vivek-ebay has joined #openstack-lbaas16:46
johnsom__http://logs.openstack.org/65/223265/11/check/gate-neutron-lbaasv2-dsvm-api/4546426/logs/screen-q-svc.txt.gz16:46
bloganoh its teh stats call?16:46
johnsom__InvalidRequestError: This session is in 'prepared' state; no further SQL can be emitted within this transaction.16:47
bloganfound it16:47
johnsom__I think it just needs a with session, but not positive16:47
*** ajmiller has joined #openstack-lbaas16:47
bloganhmm16:49
*** madhu_ak has joined #openstack-lbaas16:49
bloganill try to run a quick test on it16:49
johnsom__Cool, thanks16:49
bloganwe could skip it too if its not easily fixable16:49
johnsom__The split stuff passes now: https://review.openstack.org/#/c/225000/216:50
xgermanthanks — I still think @skip would be ok for the stats, too16:50
bloganjohnsom__: passes with octavia?16:50
mesteryblogan: https://review.openstack.org/#/c/225205/, easy one, and it can merge I think because we need it and also https://review.openstack.org/#/c/213380/ and then we fold pecan back in. Yay!16:50
bloganxgerman: indeed, not super improtant16:50
johnsom__Just the gates to get it merged, not that lbaas tests yet16:50
bloganmestery: thats ready for +A?16:51
bloganmestery: teh first?16:51
mesteryblogan: +2 it for me, I'm watching and I'll merge it later16:51
mesterySoudn ok?16:51
bloganokay16:51
mesterythanks blogan !16:51
bloganmestery: i +2'ed the 2nd one but didnt +A bc of hte jenkins failure, though it looks unrelated16:52
mesteryblogan: Yup, I'm watching them, just want a second +2 so I can merge them both when ready16:52
mesteryThanks!16:52
bloganjohnsom__: so have we run the split tests against the octavia driver yet?16:53
johnsom__No, just the load balancer one16:53
bloganjohnsom__: oh okay, how long did that take?16:53
bloganthat one should be the shortest16:53
rm_workjohnsom__: AAAAH16:55
rm_workjohnsom__: I see16:55
rm_workI was lamenting to dougwig that i didn't understand how the heck those template regexes were supposed to work16:56
rm_worknow i see: they are explicitly defined <_<16:56
johnsom__blogan About an hour, but the image build still ran in that one.  I have fixed that now, so should be quicker16:56
rm_workjohnsom__: thanks for fixing that16:56
bloganjohnsom__: awesome16:56
johnsom__NP16:56
bloganjohnsom__: i do tink the member and health monitors will go over an hour then, but it shouldnt be too bad16:56
rm_workso do we want to try to push infra to push https://review.openstack.org/#/c/225000/ ASAP or do we not even care16:56
rm_workcould we go ahead with only the LB tests passing?16:57
xgermanmmh, we should have all tests16:57
johnsom__Yeah, we should get the full test suite going16:58
rm_workok, dougwig can you help move this, or mestery16:58
blogani actually wonder what mestery's opinion is of that16:59
rm_workheh17:00
rm_workmestery: still around to review our ridiculous solution? https://review.openstack.org/#/c/225000/17:00
johnsom__Hahaha17:00
xgermanor dougwig or armax17:00
blogandougwig already gave his lukewarm approval17:00
bloganhe can't go back now17:01
xgermanwell, he didn’t go on record with a +117:01
bloganverbal contract, unbreakable17:01
bloganwell technically written contract17:01
bloganin irc17:01
bloganhold up in a court of law17:01
mesteryrm_work: Looking17:03
mesteryrm_work: I'd like armax to look at that one too, added him there17:03
*** johnsom__ has quit IRC17:05
minwang2does anyone edit wiki page in openstack, such as https://wiki.openstack.org/wiki/Neutron17:09
minwang2i am wondering can we just edit directly on the page or need to edit in github and submit patach17:10
minwang2?17:10
bloganminwang2: wiki is just edit in page17:11
rm_workmaybe just need to be logged in correctly?17:11
rm_workor is there a group that has permissions to edit those17:11
minwang2ok, so we dont need to submit patches17:11
*** ajmiller has quit IRC17:12
minwang2after  logging in, i found the edit button, so i am thinking to edit it directly there :)17:12
*** minwang2_ has joined #openstack-lbaas17:13
bloganjohnsom, xgerman: think i got the stats thing figured out17:15
bloganrunning tests to verify first17:15
xgermanyeah!!17:15
*** minwang2 has quit IRC17:16
*** minwang2_ is now known as minwang217:16
rm_workcool17:18
rm_workis it ok that we're going without having operating statuses updating yet in Neutron-LBaaS?17:19
minwang2folks, this patch needs some love https://review.openstack.org/#/c/224929/ :)17:19
rm_workmestery / dougwig: +1 the patch again plox: https://review.openstack.org/#/c/225000/17:19
mesteryrm_work: yes sir17:20
rm_work:)17:20
*** bana_k has joined #openstack-lbaas17:20
rm_workis this patch dead/obsolete? https://review.openstack.org/#/c/209675/17:22
rm_workthat is essentially what is being addressed by https://review.openstack.org/#/c/223265/ right?17:23
minwang2rm_work do you have any patch to edit the gate_hook and post_gate_hook17:24
rm_workI think https://review.openstack.org/#/c/223265/ does it17:24
rm_workjohnsom: oh wow the LB test*only* still took 1:56!?17:25
minwang2then i think we dont need mine17:25
rm_workok17:26
rm_workcan you abandon it?17:26
*** amotoki has quit IRC17:26
rm_workI think johnsom just stole your work and ported it into the new patch :P lol17:26
rm_workmake him give you a co-authored-by line, lol17:27
minwang2hahaha, it is ok, we worked together for my patch too17:27
rm_workyeah just abandon it and reference the new patch if you can17:28
minwang2rm_work https://review.openstack.org/#/c/225000/4/zuul/layout.yaml, you may want to delete 2844-284517:28
minwang2i will do17:28
johnsomWe need to keep the other one as it will become the namespace test I suspect17:29
rm_workit doesn't hurt anyway, it's experimental and non-voting17:29
rm_workthe next time someone needs to do anything experimental, they can edit it17:30
minwang2also for name “ gate-neutron-lbaasv2-dsvm-healthmonitor”, do you want to mention something like gate-neutron-lbaasv2-octavia-dsvm-healthmonitor17:30
rm_worktechnically it is only octavia because octavia will be set as defauly17:30
rm_work*default17:31
rm_workright now it isn't even testing those against octavia, it's testing them against namespace17:31
minwang2hmm, ok17:31
rm_workor would be, if it merged17:31
rm_workit's not really a "hard linkage"17:31
johnsomRight17:32
rm_workso is blogan updating https://review.openstack.org/#/c/223345/17:33
johnsomSo, just to summarize, let's not do anything to the experimental gate job right now.17:33
rm_workso that it fixes the stats issue17:33
rm_workyeah i also don't want to wipe that +2 :P17:33
minwang2i have abandoned my patch https://review.openstack.org/#/c/209675/17:34
rm_workcool thanks17:34
*** minwang2 has quit IRC17:34
rm_workis good to get stuff cleaned up17:34
*** minwang2 has joined #openstack-lbaas17:35
johnsomrm_work This is good for review now: https://review.openstack.org/#/c/223922/ (don't build image)17:36
rm_workok17:36
rm_workdid anything change since i tested it?17:36
johnsomI did a couple of experimental tests, but it should be basically where it was17:37
rm_workjohnsom: what is -s 217:37
rm_workto diskimage create17:38
johnsomImage size 2GB17:38
rm_workoh right17:38
rm_worknot a huge fan of the comments sitting around17:38
rm_workbut17:38
rm_workdeadlines :P17:38
johnsomComments?17:40
rm_workthe tempfix there17:40
rm_workwith the -o17:40
rm_workspinning a test VM17:41
rm_workwill +W soon17:41
johnsomYou must be looking at a different patchset than I think you are17:41
johnsomOh ptoohill17:41
johnsom's old comment17:42
johnsomThat can be a separate patch17:43
rm_worklike i said, deadlines, don't even care17:43
rm_workshould be ~1200s17:44
rm_workso, in 20m i'll +W :P17:44
*** minwang2 has quit IRC17:46
ptoohillThey changed the behaviour or the function. So, my comment is more of a verification question. But I think that can be used as is and comment removed. But yea, can take a look at that for another patch.17:53
ptoohillof*17:53
*** minwang2 has joined #openstack-lbaas17:53
rm_workso the other thing i noticed that is killing me about a ton of neutron-lbaas / octavia -- we don't use context managers for our db sessions17:54
rm_workI think we are leaking db-sessions all over the place17:54
rm_workwith db.session() as db_session:17:55
rm_workis so good17:55
rm_workit also takes care of making sure commits happen after the session is over17:55
johnsomI thought we had that covered in the repositories.py17:56
rm_workhttps://gist.github.com/rm-you/1450b1267d636ecb4dac17:57
rm_workhmm, i don't think so17:57
rm_worki saw a lot of stuff that looked like people making sessions and just forgetting them17:57
rm_workbut17:57
rm_workmaybe!?17:57
rm_workespecially in octavia17:58
*** harlowja has quit IRC17:58
*** harlowja has joined #openstack-lbaas17:58
johnsomYeah, I am talking about octavia.  Most everything goes through repositories and subtransactions on one session17:58
rm_workah hmm17:58
rm_worklet me try to look17:58
johnsomI don't remember who set all of that up17:59
rm_workwhat about https://review.openstack.org/#/c/215359/15/octavia/controller/housekeeping/house_keeping.py,cm18:01
rm_workline 100 or so18:01
rm_workthat is the first example i can think of18:01
rm_workwhere i noticed it, but i think i have seen similar elsewhere18:01
rm_workah18:02
rm_workreturn facade.get_session(expire_on_commit=expire_on_commit)18:02
rm_workinteresting18:02
rm_workthey've abstracted this so many levels deep18:03
johnsomYes, you are feeling my pain now18:03
rm_workcontext managers still seem a lot cleaner to me :/18:04
rm_workjohnsom:18:17
rm_work2015-09-18 17:58:31.566 | +++ nova image-list18:17
rm_work2015-09-18 17:58:31.566 | +++ awk '/ amphora-x64-haproxy / {print $4}'18:17
rm_work2015-09-18 17:58:31.941 | ERROR (CommandError): You must provide a username or user id via --os-username, --os-user-id, env[OS_USERNAME] or env[OS_USER_ID]18:17
rm_work2015-09-18 17:58:31.959 | ++ AMPHORA_IMAGE_NAME=18:17
rm_workjohnsom: not sure it worked18:17
rm_worktime was 1720s18:17
rm_worklonger than expected if it downloaded18:17
rm_worklooking through stack.sh log to see if i can find it DL but only found that error so far18:18
johnsomDo you have the latest global requirements updates?  I thought that error was from those upstream issues.18:18
rm_workand18:18
rm_work2015-09-18 18:01:50.047 | +++ nova image-list18:18
rm_work2015-09-18 18:01:50.047 | +++ awk '/ amphora-x64-haproxy / {print $4}'18:18
rm_work2015-09-18 18:01:50.506 | ERROR (ConnectionRefused): Unable to establish connection to http://23.253.20.111:8774/v2.1/18:18
rm_work2015-09-18 18:01:50.524 | ++ AMPHORA_IMAGE_NAME=18:18
rm_workthis is literally built just now18:18
rm_workthe whole VM is fresh18:18
johnsomOk, so your keystone is broken18:18
rm_workhmm18:19
johnsomAlso note, that octavia patch does not download the image.  You have to put that in your localrc or in your gate_hook18:19
rm_workoh18:19
johnsomas we did18:19
rm_worki thought i tested it before and it totally did Download it18:19
johnsomThat patch just doesn't build if the image came in from one of those other paths18:19
rm_workin fact i know it did18:19
rm_workdid you change it?18:20
*** crc32 has joined #openstack-lbaas18:20
johnsomMaybe you tested one of the experiments when I was having trouble getting the gate to pull down the image18:20
rm_workmy last test a day or two ago i did the exact same thing and it was downloaded for me18:20
rm_workhmm ok18:20
rm_workah yeah i guess i see18:21
johnsomI didn't want download to be the "default" option for everyone.  That is a bigger decision than having the gate download an image18:21
rm_worknothing in this patch about downloading. so yeah18:21
rm_workah and by just including it in IMAGE_URLS18:21
rm_workglance grabs it automagically18:22
johnsomYep18:22
rm_workok, i'm good then18:22
rm_work+W18:22
rm_workyeah i see, this way changes *inside* octavia will always try to build the image and it will not hit the issue i was concerned about before18:23
rm_workbut in neutron-lbaas the gate specifies the prebuilt18:24
rm_workneat18:24
*** Brian has joined #openstack-lbaas18:24
*** Brian is now known as Guest5270618:24
rm_workyou rebased back off of my changes?18:24
rm_workerr nm18:25
rm_workwas on the wrong patch18:25
crc32hey rm_work whats the hostname for the mvn repo for CLB 1.0 again?18:26
rm_workcrc32: you on internal IRC?18:26
crc32yea.18:26
crc32But I'm publicly exposed too/.18:26
rm_worklol18:28
*** Guest52706 has quit IRC18:28
johnsomcrc32 I'm not sure that is legal in Texas?18:30
openstackgerritMerged openstack/octavia: Don't build amphora image if one is available  https://review.openstack.org/22392218:31
crc32yea its totally illegal now.18:31
crc32I'm just as suprised as you.18:32
crc32s I guess they never got the Nexus jar running?18:36
crc32ping me. Am I lagged?18:39
johnsomrm_work Your change broke the jenkins test18:39
rm_workjohnsom: err18:41
rm_workjohnsom: whichwat18:41
johnsomJob gate-neutron-lbaasv2-dsvm-api not defined18:41
rm_workwat18:42
rm_workwhat still calls it18:42
rm_workoh that's right18:43
rm_workit's the neutron test >_>18:43
rm_workalright fff18:43
rm_workfixing this18:44
crc32don't forget about me rm_work.18:44
crc32IRC FTW right18:44
rm_worksorry, this stuff is really high priority T_T18:44
crc32ok I'll be screwing around with it.18:44
crc32do get back to me though18:45
*** bharathm has quit IRC18:47
openstackgerritAdam Harwell proposed openstack/neutron-lbaas: Split up gate tests for LBaaSv2  https://review.openstack.org/22434518:54
rm_workjohnsom: ok18:54
rm_workjohnsom: updated both sides again18:54
rm_workjohnsom: i THINK that should do it -- renamed "-api" to "-minimal" instead of removing it, since it is being used by neutron18:54
rm_workjohnsom: and updated our test hook to make a new case for that18:54
rm_worksince that was our plan18:54
rm_worki'll wait for the jenkins +1 before doing anything18:55
rm_work*before poking people for reviews18:55
crc32I didn't really need your help with maven. Just wanted to know how far you got before I started working on it.18:55
crc32rm_work ^^18:56
rm_workah18:56
rm_workyeah not far18:56
rm_workwell18:56
rm_workcrc32: can we talk on RAX?18:56
crc32yea18:56
*** apuimedo has quit IRC18:57
*** Youcef has quit IRC19:08
openstackgerritBrandon Logan proposed openstack/neutron-lbaas: Various fixes for tempest tests with octavia  https://review.openstack.org/22334519:34
xgermanyeah?19:36
*** bharathm has joined #openstack-lbaas19:39
*** madhu_ak has quit IRC19:41
*** minwang2 has quit IRC20:00
*** sballe has joined #openstack-lbaas20:01
*** minwang2 has joined #openstack-lbaas20:02
*** vivek-ebay has quit IRC20:20
*** apuimedo has joined #openstack-lbaas20:20
openstackgerritBrandon Logan proposed openstack/neutron-lbaas: Various fixes for tempest tests with octavia  https://review.openstack.org/22334520:21
openstackgerritBrandon Logan proposed openstack/neutron-lbaas: Split up gate tests for LBaaSv2  https://review.openstack.org/22434520:25
openstackgerritBrandon Logan proposed openstack/neutron-lbaas: Set Octavia as the reference LBaaSv2 driver  https://review.openstack.org/22326520:25
*** intr1nsic has joined #openstack-lbaas20:31
bloganbtw rm_work, johnsom is correct the repository does take care of the session using context manager20:32
rm_workyeah20:32
rm_worki see the facade thing20:32
bloganoh well thats boilerplate from oslo_db docs20:32
rm_workyeah20:33
*** minwang2 has quit IRC20:37
openstackgerritCarlos Garza proposed openstack/octavia: Implementing EventStreamer  https://review.openstack.org/21873520:38
*** minwang2 has joined #openstack-lbaas20:41
*** minwang2 has quit IRC20:50
*** vivek-ebay has joined #openstack-lbaas20:50
*** minwang2 has joined #openstack-lbaas20:57
*** vivek-ebay has quit IRC20:57
*** vivek-ebay has joined #openstack-lbaas20:57
*** Brian has joined #openstack-lbaas21:04
*** Brian is now known as Guest2092421:05
ptoohillof*https://review.openstack.org/#/c/224435/21:08
*** Guest20924 has quit IRC21:09
*** minwang2 has quit IRC21:09
johnsomI already hooked you up on that one21:10
rm_workjohnsom: i think everything is looking ok? so i am going to switch gears to putting out internal fires now21:12
rm_workping me if you need anything21:12
johnsomOk.  The last gate run is just about done21:12
johnsomThanks!21:12
*** minwang2 has joined #openstack-lbaas21:14
*** chlong has quit IRC21:14
johnsomHmmm, one more question, looking at Ubuntu's liberty cloud archive, I don't see octavia in there....21:20
johnsomDoes anyone know if it's bundled into something else?21:20
johnsomI'm looking here: http://ubuntu-cloud.archive.canonical.com/ubuntu/dists/trusty-proposed/liberty/21:21
xgermanwe probably need to make sure it gets bundled21:22
xgermanmestery?21:22
openstackgerritCarlos Garza proposed openstack/octavia: Implementing EventStreamer  https://review.openstack.org/21873521:24
*** minwang2 has quit IRC21:27
*** diogogmt has joined #openstack-lbaas21:29
mesteryxgerman johnsom: Yes, that needs to be packaged up for Ubuntu (and Fedora).21:33
mesteryNot sure who typically does that, likely someone from the distros21:33
*** minwang2 has joined #openstack-lbaas21:33
johnsomI see some evidence that RDO is packaging it up.  Just don't see it in Ubuntu's cloud archive21:33
*** manishg has joined #openstack-lbaas21:34
johnsomblogan rm_work We had an LB create timeout.  This host was slow anyway, with devstack bring up over 20 minutes too21:35
johnsomwaiting on full logs....21:35
blogangrr21:36
bloganwell i wonder if the amp went into ERROR pretty quickly21:36
bloganbc we dont have that "bug" fixed where it should recognize that and put the LB in error immediately21:37
johnsomThe log I can see shows "PENDING_CREATE" and "OFFLINE"21:37
bloganyeah for the LB, which would happen in the amp went into ERROR21:37
johnsomYeah.  Really anxious for the full log set21:38
*** minwang2 has quit IRC21:43
bloganso to hedge our bets, im working on having the tests create only one or two LBs for most of our tests, see how that goes21:44
blogansomething we should try to do anyway i think21:44
bloganwill require skipping some tests though21:44
bloganbut at this point i dont care about taht21:44
bloganwe can re-enable when we get containers or vt-x enabled21:44
*** manishg has quit IRC21:46
*** manishg has joined #openstack-lbaas21:46
*** minwang2 has joined #openstack-lbaas21:50
*** manishg has quit IRC21:50
xgermanyeah, anything which works by Monday is goodness21:51
rm_workyep21:56
johnsomhttp://logs.openstack.org/65/223265/12/check/gate-neutron-lbaas-python27/fe0f128/console.html21:57
rm_workthat's ... a weird failure?21:59
johnsomUnit test failure21:59
rm_workthe error is ERROR [neutron.api.extensions] Extension path 'neutron/tests/unit/extensions' doesn't exist! ??22:00
rm_workwat22:00
*** minwang2 has quit IRC22:02
*** manishg has joined #openstack-lbaas22:06
*** adam_g has joined #openstack-lbaas22:09
bloganthat error is always there, its something else22:11
johnsomUnit test failure is:22:20
johnsomAssertionError: Expected call: put('/v1/loadbalancers/test_id/listeners/test_id/pools/test_id', {'description': None, 'name': None, 'lb_algorithm': None, 'enabled': None, 'protocol': None})22:20
johnsomActual call: put('/v1/loadbalancers/test_id/listeners/test_id/pools/test_id', {'description': None, 'name': None, 'lb_algorithm': None, 'enabled': None, 'id': 'test_id', 'protocol': None})22:20
johnsomThe id is new22:20
bloganoh god it must be my changes22:22
bloganim dumb22:22
johnsomThere is still some other problem or it was a slow node22:23
bloganthat one is just a unittest22:24
ptoohillhttps://review.openstack.org/#/c/224435/ <-- it's an easy review :)22:26
rm_worki'm heading to dinner, but i will be online most of tonight ... and this weekend, probably, at least monitoring those changes22:27
bloganrm_work: isnt this like lunch time for you22:28
rm_workblogan: yeah but i don't make a big deal out of it :P22:28
rm_worktechnically it's breakfast since i haven't eaten yet22:28
openstackgerritPhillip Toohill proposed openstack/octavia: Updates for containers functionality  https://review.openstack.org/19995422:28
*** ptoohill is now known as pc-pothole22:29
bloganargh such a dumb oversight22:34
bloganpushing up a fix in a few minutes22:34
openstackgerritMerged openstack/octavia: Make keystone verify configurable  https://review.openstack.org/22443522:35
johnsomWould your fix address why the lb create failed?22:36
openstackgerritBrandon Logan proposed openstack/neutron-lbaas: Various fixes for tempest tests with octavia  https://review.openstack.org/22334522:36
bloganjohnsom: probably not, but it fixes the unit test22:38
johnsomOk22:38
bloganjohnsom: you happen to find the logs for it?22:38
johnsomI'm just not seeing anything in the CW log other than long times for it to come up22:39
johnsomhttp://logs.openstack.org/65/223265/12/check/gate-neutron-lbaasv2-dsvm-api/bf441ef/22:39
blogani see "No more IP Addresses avaialble on network"22:39
johnsomYeah, but I think that is after we went off the rails22:40
bloganoh okay22:40
bloganwell that lb did finally go active22:43
bloganafter nearly 8 minutes22:43
bloganoh looking at the wrong lb22:44
blogandamn wish we logged what nova was responding with22:45
*** openstackgerrit has quit IRC22:46
*** openstackgerrit has joined #openstack-lbaas22:46
johnsomNova had to report ready or we wouldn't go into "Could not talk to instance"22:47
bloganoh good point22:47
johnsomIt's just the kernel boot and service start times after that22:47
bloganso services didnt start for some reason22:47
johnsomOr the boot time was too long.22:48
johnsomI think those SSLContext warnings are a successful connect to the agent22:48
bloganthat may not be for the same lb though22:52
bloganthere's another LB getting updated at the same time, and deleted22:52
blogando we connect to the amphora on delete?22:52
*** minwang2 has joined #openstack-lbaas22:52
blogandoesnt look like it22:53
bloganso nvm22:53
bloganwell im not sure22:53
johnsomYeah, I struggle to come up with anything that went wrong other than timeout waiting for the amp to boot22:55
bloganwell one thing this does show, we need better logging :)22:55
johnsomYes.  That I know as well22:55
bloganthough i think we all knew that already22:55
johnsomWe have great DEBUG logging...  grin22:56
johnsomWhich is usually all I look at22:56
bloganha so in this you see like 2 lines of log22:56
bloganoh wait22:57
bloganthe config isn't set to debug22:57
bloganso its not logging debug messages22:57
johnsomRight, that is my point22:57
bloganlol22:57
blogansorry over my head22:57
johnsomEverything we want to know is in debug22:57
*** manishg has quit IRC22:57
xgermanok, so we should just modify our gates to switch that on?22:57
*** minwang2 has quit IRC23:04
*** minwang2 has joined #openstack-lbaas23:04
johnsomI don't think it is going to tell us anything more than the amps took longer than the test timeout of five minutes to come up.23:14
johnsomblogan Thoughts?  My vote is to push that timeout up to ten minutes23:15
*** minwang2 has quit IRC23:16
*** manishg has joined #openstack-lbaas23:17
*** minwang2 has joined #openstack-lbaas23:17
xgerman+!23:19
xgerman+123:19
openstackgerritMichael Johnson proposed openstack/neutron-lbaas: Split up gate tests for LBaaSv2  https://review.openstack.org/22434523:20
openstackgerritMichael Johnson proposed openstack/neutron-lbaas: Set Octavia as the reference LBaaSv2 driver  https://review.openstack.org/22326523:20
johnsomI might as well rebase these are get another go started23:20
xgermanyep23:23
bloganjohnsom: fine by me23:30
johnsomGrin.  Now we have to figure out how to do that...23:30
xgermanok, at least the tempest fixes are approved by Jenkins23:34
*** manishg has quit IRC23:41
*** crc32 has quit IRC23:45
openstackgerritMichael Johnson proposed openstack/neutron-lbaas: Various fixes for tempest tests with octavia  https://review.openstack.org/22334523:47
openstackgerritMichael Johnson proposed openstack/neutron-lbaas: Split up gate tests for LBaaSv2  https://review.openstack.org/22434523:47
openstackgerritMichael Johnson proposed openstack/neutron-lbaas: Set Octavia as the reference LBaaSv2 driver  https://review.openstack.org/22326523:47
johnsomOk, giving that a go23:47
xgermansuspense is killing me23:50
johnsomSorry to say, we have a long wait....23:50
xgermanI was afraid of that23:51
johnsomFor those playing along at home:23:52
johnsomhttps://jenkins03.openstack.org/job/gate-neutron-lbaasv2-dsvm-api/1353/console23:52
*** bharathm has quit IRC23:55

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