| *** amotoki has joined #openstack-lbaas | 00:16 | |
| *** allan_h has quit IRC | 00:22 | |
| *** chlong has joined #openstack-lbaas | 00:26 | |
| *** allan_h has joined #openstack-lbaas | 00:28 | |
| *** chlong has quit IRC | 01:03 | |
| *** yamamoto_ has joined #openstack-lbaas | 01:48 | |
| *** chlong has joined #openstack-lbaas | 02:06 | |
| *** yamamoto_ has quit IRC | 02:07 | |
| *** yamamoto_ has joined #openstack-lbaas | 02:08 | |
| *** yamamoto_ has quit IRC | 02:10 | |
| *** yamamoto_ has joined #openstack-lbaas | 02:10 | |
| *** yamamoto_ has quit IRC | 02:10 | |
| *** yamamoto_ has joined #openstack-lbaas | 02:58 | |
| *** reedip_away is now known as reedip | 03:03 | |
| *** amotoki has quit IRC | 04:13 | |
| *** links has joined #openstack-lbaas | 04:14 | |
| *** amotoki has joined #openstack-lbaas | 04:27 | |
| *** jidar_ has joined #openstack-lbaas | 04:38 | |
| *** jidar has quit IRC | 04:38 | |
| *** jidar_ is now known as jidar | 04:38 | |
| *** amotoki has quit IRC | 04:39 | |
| *** amotoki has joined #openstack-lbaas | 04:40 | |
| *** ducttape_ has joined #openstack-lbaas | 04:43 | |
| *** neelashah has joined #openstack-lbaas | 04:47 | |
| *** neelashah has quit IRC | 04:55 | |
| *** amotoki has quit IRC | 04:56 | |
| *** amotoki has joined #openstack-lbaas | 05:11 | |
| *** ducttape_ has quit IRC | 05:18 | |
| *** prabampm has joined #openstack-lbaas | 05:32 | |
| *** amotoki_ has joined #openstack-lbaas | 06:08 | |
| *** amotoki has quit IRC | 06:11 | |
| openstackgerrit | Aishwarya Thangappa proposed openstack/octavia: Adding "region and endpoint_type" parameters to barbican_acl.py https://review.openstack.org/272861 | 06:33 |
|---|---|---|
| *** numans has joined #openstack-lbaas | 06:34 | |
| *** allan_h has quit IRC | 06:39 | |
| *** chlong has quit IRC | 07:03 | |
| *** nmagnezi has joined #openstack-lbaas | 07:49 | |
| openstackgerrit | Merged openstack/neutron-lbaas: Enable debug logging in octavia scenario tests https://review.openstack.org/276900 | 08:07 |
| *** rcernin has joined #openstack-lbaas | 08:09 | |
| *** chlong has joined #openstack-lbaas | 08:48 | |
| reedip | blogan: ping? | 08:54 |
| *** amotoki_ has quit IRC | 08:55 | |
| *** jschwarz has joined #openstack-lbaas | 09:00 | |
| *** openstackgerrit has quit IRC | 09:02 | |
| *** openstackgerrit has joined #openstack-lbaas | 09:03 | |
| *** amotoki has joined #openstack-lbaas | 09:14 | |
| *** reedip is now known as outofmemory | 09:24 | |
| *** yuanying is now known as yuanying_influen | 09:25 | |
| *** yuanying_influen is now known as yuanying_flu | 09:25 | |
| *** amotoki has quit IRC | 09:32 | |
| *** Alex_Stef has joined #openstack-lbaas | 09:46 | |
| *** ebagdasa has quit IRC | 10:10 | |
| *** jschwarz is now known as jschwarz|lunch | 11:11 | |
| *** rcernin has quit IRC | 11:19 | |
| *** rcernin has joined #openstack-lbaas | 11:22 | |
| *** rcernin has quit IRC | 11:25 | |
| *** rcernin has joined #openstack-lbaas | 11:26 | |
| *** yamamoto_ has quit IRC | 11:27 | |
| *** prabampm has quit IRC | 11:37 | |
| *** prabampm has joined #openstack-lbaas | 11:39 | |
| *** prabampm1 has joined #openstack-lbaas | 11:51 | |
| *** prabampm has quit IRC | 11:51 | |
| *** numans has quit IRC | 12:03 | |
| *** amotoki has joined #openstack-lbaas | 12:13 | |
| *** yamamoto has joined #openstack-lbaas | 12:21 | |
| *** rtheis has joined #openstack-lbaas | 12:30 | |
| *** amotoki has quit IRC | 12:35 | |
| *** yamamoto has quit IRC | 12:42 | |
| *** mdavidson has joined #openstack-lbaas | 12:43 | |
| *** yamamoto has joined #openstack-lbaas | 12:43 | |
| *** yamamoto has quit IRC | 12:48 | |
| *** amotoki has joined #openstack-lbaas | 12:58 | |
| *** yamamoto has joined #openstack-lbaas | 12:58 | |
| *** ducttape_ has joined #openstack-lbaas | 13:13 | |
| *** amotoki has quit IRC | 13:15 | |
| *** links has quit IRC | 13:20 | |
| *** localloop127 has joined #openstack-lbaas | 13:27 | |
| *** yamamoto has quit IRC | 13:27 | |
| *** ducttape_ has quit IRC | 13:29 | |
| *** Alex_Stef has quit IRC | 13:35 | |
| *** yamamoto has joined #openstack-lbaas | 13:58 | |
| *** neelashah has joined #openstack-lbaas | 14:00 | |
| *** yamamoto has quit IRC | 14:28 | |
| *** Aish has joined #openstack-lbaas | 14:49 | |
| *** ducttape_ has joined #openstack-lbaas | 14:51 | |
| *** jschwarz|lunch is now known as jschwarz | 14:58 | |
| *** TrevorV has joined #openstack-lbaas | 15:00 | |
| *** nmagnezi has quit IRC | 15:02 | |
| *** piet has joined #openstack-lbaas | 15:03 | |
| openstackgerrit | Trevor Vardeman proposed openstack/octavia: Amphora create will add a 'fake' heartbeat on creation https://review.openstack.org/271512 | 15:10 |
| *** ajmiller has joined #openstack-lbaas | 15:14 | |
| *** Bjoern has joined #openstack-lbaas | 15:17 | |
| *** amotoki has joined #openstack-lbaas | 15:24 | |
| *** rcernin has quit IRC | 15:42 | |
| *** doug-fish has joined #openstack-lbaas | 15:42 | |
| *** amotoki has quit IRC | 15:46 | |
| *** links has joined #openstack-lbaas | 15:49 | |
| *** links has quit IRC | 15:57 | |
| *** ajmiller_ has joined #openstack-lbaas | 16:05 | |
| *** ajmiller has quit IRC | 16:09 | |
| *** _ducttape_ has joined #openstack-lbaas | 16:19 | |
| *** armax has joined #openstack-lbaas | 16:20 | |
| *** ducttape_ has quit IRC | 16:21 | |
| *** sbalukoff has quit IRC | 16:24 | |
| *** numans has joined #openstack-lbaas | 16:32 | |
| *** allan_h has joined #openstack-lbaas | 16:35 | |
| openstackgerrit | Trevor Vardeman proposed openstack/octavia: Amphora create will add a 'fake' heartbeat on creation https://review.openstack.org/271512 | 16:37 |
| *** _ducttape_ has quit IRC | 16:40 | |
| *** ducttape_ has joined #openstack-lbaas | 16:41 | |
| *** armax has quit IRC | 16:46 | |
| *** jschwarz has quit IRC | 16:47 | |
| *** fnaval has joined #openstack-lbaas | 17:03 | |
| *** madhu_ak has joined #openstack-lbaas | 17:07 | |
| *** Aish has quit IRC | 17:18 | |
| *** yamamoto has joined #openstack-lbaas | 17:21 | |
| *** yamamoto has quit IRC | 17:22 | |
| openstackgerrit | Adam Harwell proposed openstack/octavia: Revert "Fix for ssl context wrapper spam", add warning filter https://review.openstack.org/277506 | 17:33 |
| *** Aish has joined #openstack-lbaas | 17:34 | |
| *** yamamoto has joined #openstack-lbaas | 17:43 | |
| *** armax has joined #openstack-lbaas | 17:56 | |
| xgerman | rm_work I had to rewrite that endpoint filter logic three times to appease blogan | 17:58 |
| rm_work | lol | 17:59 |
| rm_work | poor xgerman | 17:59 |
| xgerman | yeah, we need to streamline simple changes... | 18:00 |
| xgerman | guess time to fork octavia so I can add an endpoint type | 18:01 |
| rm_work | xgerman: is "publicURL" the correct endpoint_type for devstack? | 18:03 |
| xgerman | yes | 18:03 |
| rm_work | k | 18:03 |
| rm_work | then it is wrong in https://review.openstack.org/#/c/272861/ | 18:03 |
| rm_work | wanted to check before I left a -1 there | 18:03 |
| xgerman | and it’s the default in the keystone client if you don’t specify one | 18:04 |
| rm_work | Yeah I thought yours was correct | 18:04 |
| xgerman | in our cloud we need to use “internal” but people also have “admin" | 18:04 |
| rm_work | but before I told Aish to fix it I wanted to make sure | 18:04 |
| xgerman | you can also say “public" | 18:04 |
| rm_work | ah | 18:04 |
| xgerman | both work | 18:04 |
| rm_work | well | 18:04 |
| rm_work | I'd like it to be consistent | 18:04 |
| rm_work | so, sorry Aish T_T | 18:05 |
| xgerman | yeah, agreed on that one | 18:05 |
| rm_work | confusing for deployers elsewise | 18:05 |
| xgerman | if they will even find those settings since they are now per service and not for all | 18:05 |
| rm_work | heh | 18:06 |
| rm_work | well, it matters | 18:06 |
| rm_work | though for barbican it technically doesn't matter, until BarbicanCertGenerator works | 18:06 |
| rm_work | because the client doesn't use those values for retrieval | 18:06 |
| xgerman | it does not? | 18:06 |
| rm_work | it uses the HREF for the container | 18:06 |
| rm_work | ignoring the service catalog values | 18:06 |
| xgerman | I thought Barbican can’t do certs but if it could we could use the generator | 18:06 |
| rm_work | right | 18:07 |
| rm_work | but ... I honestly don't know if I ever see that happening | 18:07 |
| rm_work | I think Anchor will be the way forward | 18:07 |
| xgerman | yep | 18:07 |
| *** armax has quit IRC | 18:07 | |
| rm_work | I am tempted to remove the dead code there | 18:07 |
| rm_work | delete the LocalCertManager and BarbicanCertGenerator classes | 18:08 |
| *** doug-fish has quit IRC | 18:08 | |
| xgerman | ok | 18:08 |
| rm_work | might do that now | 18:08 |
| xgerman | yeah, you have my +2 | 18:08 |
| *** doug-fish has joined #openstack-lbaas | 18:09 | |
| *** armax has joined #openstack-lbaas | 18:12 | |
| *** doug-fish has quit IRC | 18:13 | |
| rm_work | who else runs on OSX? | 18:18 |
| rm_work | I'm getting a couple of tox failures on py27/py34 that look like false positives and only happen to me on my OSX box | 18:18 |
| *** yamamoto has quit IRC | 18:26 | |
| xgerman | I filed a bug | 18:26 |
| xgerman | rm_work | 18:26 |
| rm_work | for whichwhat | 18:27 |
| xgerman | for the OS X error | 18:28 |
| *** armax has quit IRC | 18:31 | |
| xgerman | can’t find it somebody must have closed it | 18:32 |
| rm_work | ah | 18:34 |
| rm_work | octavia.tests.functional.amphorae.backend.agent.api_server.test_server.ServerTestCase.test_get_interface and octavia.tests.functional.amphorae.backend.agent.api_server.test_server_sysvinit.ServerTestCase.test_get_interface | 18:34 |
| rm_work | uhh | 18:34 |
| johnsom | I don't remember an OS/X bugs in the octavia bug list. Maybe it was filed in a different project? | 18:36 |
| *** prometheanfire has left #openstack-lbaas | 18:41 | |
| rm_work | umm | 18:41 |
| rm_work | address_format = netifaces.AF_INET6 | 18:41 |
| rm_work | on OSX this is "30" | 18:41 |
| xgerman | yeah, I thought if filed | 18:43 |
| rm_work | on linux this is 10 | 18:43 |
| rm_work | wtf | 18:43 |
| johnsom | The constants are different???? | 18:43 |
| rm_work | yes | 18:43 |
| johnsom | Evil | 18:43 |
| rm_work | so need to use the constants in the test | 18:43 |
| rm_work | it's fixable | 18:43 |
| rm_work | i'll do it now | 18:43 |
| xgerman | cool | 18:43 |
| xgerman | ok, I guess that interface patch attracts all the nit-pickers in the universe | 18:43 |
| johnsom | That is as bad as debian not taking eth1.cfg in interfaces.d but ubuntu requires .cfg | 18:44 |
| *** armax has joined #openstack-lbaas | 18:44 | |
| openstackgerrit | Adam Harwell proposed openstack/octavia: Remove dead code around certificate handling https://review.openstack.org/277532 | 18:46 |
| xgerman | rm_work +2 | 18:46 |
| rm_work | wtf is "17"? | 18:47 |
| rm_work | for type | 18:47 |
| rm_work | ah, AF_ROUTE | 18:47 |
| *** piet has quit IRC | 18:47 | |
| rm_work | ok, easy peasyf ix | 18:48 |
| TrevorV | rm_work https://review.openstack.org/#/c/271512 | 18:48 |
| rm_work | kk | 18:48 |
| TrevorV | xgerman johnsom https://review.openstack.org/#/c/271512 | 18:48 |
| rm_work | will look momentarily | 18:48 |
| TrevorV | Preez! | 18:48 |
| rm_work | that session persistence bug is really ... persistent | 18:48 |
| rm_work | is session persistence actually *broken* or does that intermittent bug just hit 90% of the time <_< | 18:48 |
| TrevorV | I noticed that. | 18:48 |
| TrevorV | I have zero idea what's even wrong... ha ha | 18:49 |
| johnsom | Not sure. I'm also tracking the ovh hosts seem to be failing out our scenario test where the rax hosts aren't failing. May be related, may not | 18:50 |
| johnsom | Well, TrevorV's patch was on rax and failed too. | 18:51 |
| rm_work | filed https://bugs.launchpad.net/octavia/+bug/1543248 xgerman | 18:51 |
| openstack | Launchpad bug 1543248 in octavia "OSX has tox test failures due to netifaces inconsistency" [Undecided,New] | 18:51 |
| TrevorV | Oh god please someone tackle that bug, please... | 18:52 |
| openstackgerrit | Adam Harwell proposed openstack/octavia: Use netifaces constants correctly in testing https://review.openstack.org/277533 | 18:52 |
| rm_work | ^^ resolved | 18:52 |
| johnsom | Hmmm, there is still a problem with you patch TrevorV and a failover bug, but I'm really puzzled by that, as it was fixed in the patch that merged | 18:53 |
| TrevorV | johnsom wait what? | 18:53 |
| *** bank_ has joined #openstack-lbaas | 18:53 | |
| *** bana_k has joined #openstack-lbaas | 18:53 | |
| rm_work | as soon as https://review.openstack.org/#/c/277533/ gets through Zuul I will be bugging you guys for a quick merge :P | 18:53 |
| xgerman | +2 | 18:54 |
| johnsom | TrevorV it still tried to failover an amp. It failed because of some failover flow issue that puzzles me (looking at that now). | 18:54 |
| TrevorV | Oh. Really? Maybe a different topology issue? | 18:55 |
| johnsom | It claims there is no "amphora" object in the flow for markallocatedindb, which is strange as the steps before that get the amphora | 18:56 |
| rm_work | O_o | 18:57 |
| TrevorV | So the flow is complaining, not a task specifically failing? | 18:58 |
| johnsom | right | 18:58 |
| TrevorV | Oh. I find that more interesting. Weird. Would it throw that error if "amphora=None"? | 18:59 |
| johnsom | What the heck, the line above it, in the flow also requires 'amphora' but that didn't errorr | 19:00 |
| TrevorV | Hah... are you sure you don't have different code running than what you're looking at? | 19:01 |
| rm_work | xgerman / johnsom can you take a look at my comments on https://review.openstack.org/#/c/271476/8 and tell me if i'm wrong? | 19:03 |
| johnsom | Yep, I'm looking at master vs. the error you got | 19:03 |
| *** neelashah has quit IRC | 19:03 | |
| *** _ducttape_ has joined #openstack-lbaas | 19:08 | |
| *** ducttape_ has quit IRC | 19:11 | |
| *** doug-fish has joined #openstack-lbaas | 19:16 | |
| *** sbalukoff has joined #openstack-lbaas | 19:16 | |
| *** fnaval has quit IRC | 19:16 | |
| *** doug-fish has quit IRC | 19:21 | |
| *** _ducttape_ has quit IRC | 19:22 | |
| *** neelashah has joined #openstack-lbaas | 19:22 | |
| *** ducttape_ has joined #openstack-lbaas | 19:22 | |
| *** doug-fish has joined #openstack-lbaas | 19:22 | |
| *** armax has quit IRC | 19:26 | |
| openstackgerrit | Brandon Logan proposed openstack/octavia: Fix / and /v1 API uris https://review.openstack.org/277544 | 19:34 |
| *** armax has joined #openstack-lbaas | 19:34 | |
| *** armax has quit IRC | 19:36 | |
| rm_work | blogan / xgerman / johnsom / anyone: have you see this failure before? http://logs.openstack.org/06/277506/1/check/gate-neutron-lbaasv2-dsvm-scenario/33327b1/console.html#_2016-02-08_19_26_57_622 | 19:37 |
| blogan | rm_work: yes | 19:37 |
| blogan | rm_work: one of them intermittent ones we're not sure about right now | 19:37 |
| blogan | rm_work: could you put a launchpad bug in for this one? we should start keeping track of this one as well, we already have one for the session persistence one and i think there's another one that doesn't have a bug for it that causes the listener test to break | 19:38 |
| blogan | i have a feeling they're all related though | 19:39 |
| blogan | rm_work: ^ the patch i just pushed up, it was all staticmethods fault! | 19:39 |
| rm_work | lol | 19:41 |
| rm_work | staticmethods don't kill people, people kill people | 19:41 |
| *** piet has joined #openstack-lbaas | 19:42 | |
| *** fnaval has joined #openstack-lbaas | 19:42 | |
| rm_work | looks super ugly but: https://bugs.launchpad.net/octavia/+bug/1543264 | 19:48 |
| openstack | Launchpad bug 1543264 in octavia "intermittent scenario failure in test_health_monitor_basic" [Undecided,New] | 19:48 |
| rm_work | blogan: ^^ | 19:48 |
| sbalukoff | Woot! My Monday's going great-- got my first -1 of the week in! ;) | 19:49 |
| rm_work | heh | 19:50 |
| sbalukoff | blogan: Seriously though, good catch on that. I never would have guessed. (Also, I do seriously think we should change it to "v1", then I'm good with it.) | 19:50 |
| rm_work | Yeah switched to -1 :P | 19:50 |
| rm_work | Since it isn't JUST me being pedantic | 19:50 |
| sbalukoff | It's both of us being pedantic. | 19:51 |
| rm_work | Yes, I'm ok with being in a pedantic mob | 19:51 |
| rm_work | but it's lonely being a solitary pedant | 19:51 |
| *** neelashah1 has joined #openstack-lbaas | 19:51 | |
| sbalukoff | BBIAB, gotta grab some lunch before this stupid audit today. (Thankfully the last of these I'll have to do in a long while.) | 19:51 |
| *** neelashah has quit IRC | 19:52 | |
| blogan | rm_work: thats good enough, be sure to recheck bug 1543264 when you need a recheck bc of this | 19:52 |
| openstack | bug 1543264 in octavia "intermittent scenario failure in test_health_monitor_basic" [Undecided,New] https://launchpad.net/bugs/1543264 | 19:52 |
| rm_work | k | 19:52 |
| rm_work | I mean, technically NEVER need a recheck for scenario failures <_< | 19:52 |
| blogan | sbalukoff: johnsom hunted me down for it, he threatened me if i didnt help figure it out | 19:52 |
| rm_work | blogan: https://review.openstack.org/#/c/277506/ | 19:53 |
| blogan | sbalukoff: and what do you mean change it to "v1"? ohhh its V! | 19:53 |
| blogan | V1 | 19:53 |
| blogan | no its lowercase, at least the uri is | 19:53 |
| rm_work | <_< | 19:53 |
| blogan | sbalukoff: you'll have to clear it up for me when youg et back | 19:54 |
| rm_work | blogan: lowercase the V1 to v1 | 19:54 |
| rm_work | that's it :P | 19:54 |
| blogan | but it is lowercase in the uri | 19:54 |
| rm_work | is .lower() being run on it? | 19:54 |
| blogan | oh he put comments | 19:55 |
| blogan | nvm | 19:55 |
| blogan | thats just the body | 19:55 |
| blogan | that gets returned | 19:55 |
| rm_work | lol | 19:55 |
| blogan | it was just a "ill put something here" | 19:55 |
| rm_work | is it even used | 19:55 |
| blogan | ha no | 19:55 |
| rm_work | lol | 19:55 |
| rm_work | I *wasn't* going to -1 | 19:55 |
| rm_work | as I said | 19:55 |
| rm_work | but | 19:55 |
| blogan | that calls has been broken for probably 6 mos to a year, 500, and no one has really noticed | 19:55 |
| rm_work | mob mentality | 19:55 |
| rm_work | I noticed! | 19:55 |
| rm_work | and complained | 19:55 |
| blogan | prove it! | 19:56 |
| rm_work | and didn't bother to fix it because who cares <_< | 19:56 |
| * blogan watches rm_work spend all day going through the eavesdop logs for the past year | 19:56 | |
| rm_work | lol | 19:56 |
| rm_work | I was about to start grepping | 19:56 |
| rm_work | decided I don't care enough | 19:56 |
| blogan | perhaps if someone in particualr challenged you to it you would | 19:57 |
| rm_work | <_< | 19:57 |
| rm_work | I wonder if I can get Korean food delivered.... | 19:57 |
| *** neelashah1 has quit IRC | 19:58 | |
| blogan | air mail? | 19:58 |
| rm_work | it's the Lunar New Year today, and thus is https://en.wikipedia.org/wiki/Korean_New_Year :) | 19:59 |
| johnsom | FYI, I also put this one in: https://bugs.launchpad.net/neutron/+bug/1542467 | 19:59 |
| openstack | Launchpad bug 1542467 in neutron "LBAAS Intermittent gate failure TestHealthMonitorBasic.test_health_monitor_basic" [Critical,Opinion] | 19:59 |
| blogan | i figured the moon would have the same new year as earth, the thing it orbits | 19:59 |
| rm_work | was going to get some Tteokguk | 20:00 |
| blogan | johnsom: ah so thats a duplicate of what rm_work put in | 20:00 |
| *** numans has quit IRC | 20:00 | |
| rm_work | ah k so it did exist | 20:00 |
| blogan | or the other way around | 20:00 |
| rm_work | ah | 20:00 |
| blogan | surprised launchpad didn't tell you there was one that looked similar | 20:00 |
| blogan | or did you ignore that suggestin? | 20:00 |
| rm_work | well it searches Octavia | 20:00 |
| rm_work | not Neutron | 20:00 |
| rm_work | also no, didn't see anything | 20:00 |
| blogan | ah | 20:00 |
| blogan | yeah didnt see it was in neutron and octavia respectively | 20:01 |
| rm_work | will dupe mine | 20:01 |
| rm_work | done | 20:01 |
| johnsom | Well, the gate is technically a neutron-lbaas gate/test so I put it over there | 20:01 |
| rm_work | yeah | 20:02 |
| rm_work | i agree it makes sense | 20:02 |
| *** woodster_ has joined #openstack-lbaas | 20:02 | |
| rm_work | now it will show up in both places because mine is in octavia and marked as a dupe of yours | 20:02 |
| rm_work | blogan: comments on https://review.openstack.org/#/c/271476/ ? | 20:03 |
| *** neelashah has joined #openstack-lbaas | 20:03 | |
| johnsom | I didn't threaten anyone BTW. Just asked for help as it was not obvious to me what was wrong. | 20:05 |
| blogan | rm_work: good catch on that, though i think the original intention of the get_client methods were nto to eb a factory but to be a singleton, so perhaps that needs to be changed | 20:05 |
| johnsom | It became relevant for API endpoint monitoring | 20:05 |
| blogan | johnsom: no witnesses, i felt my life was in danger | 20:05 |
| TrevorV | johnsom | 20:05 |
| blogan | johnsom: cornered me in a dark corner of irc | 20:05 |
| rm_work | blogan: you think a different patch is fine? | 20:06 |
| TrevorV | johnsom did you set the new environment variable on my patch? | 20:06 |
| openstackgerrit | Aishwarya Thangappa proposed openstack/octavia: Adding "region and endpoint_type" parameters to barbican_acl.py https://review.openstack.org/272861 | 20:06 |
| rm_work | I am just worried it'll be forgotten | 20:06 |
| blogan | rm_work: yeah basically bc it does what is needed for the original intention of the clients.py code | 20:06 |
| rm_work | T_T | 20:06 |
| rm_work | so if the first thing to initialize it happens to want an admin client... | 20:07 |
| rm_work | EVERYTHING gets an admin client | 20:07 |
| blogan | yep | 20:07 |
| rm_work | good game | 20:07 |
| blogan | i know | 20:07 |
| blogan | so bug report? | 20:07 |
| rm_work | Guess so | 20:07 |
| johnsom | blogan Nobody is buying it. Especially when I'm how many miles away???? 2,259 | 20:07 |
| blogan | johnsom: haven't you seen the movie hackers? they were able to sink a oil tanker from that distance! | 20:08 |
| johnsom | Sigh | 20:08 |
| blogan | i'm sure you could hack into my chair and make it roll down the escalator | 20:08 |
| johnsom | I'll just blind you by making your IRC client blink | 20:09 |
| johnsom | Yeah.... | 20:09 |
| * blogan keeps a history of that threat | 20:09 | |
| johnsom | TrevorV Environment variable??? I was looking at the gate job logs | 20:10 |
| TrevorV | johnsom lemme PM you | 20:10 |
| johnsom | blogan Want me to appease the spoil sports and change the text to v1? At least for now, or are you feeling ambitious and going to do the endpoint thing? | 20:12 |
| rm_work | https://bugs.launchpad.net/octavia/+bug/1543269 | 20:12 |
| openstack | Launchpad bug 1543269 in octavia "Nova and Neutron client factories can return incorrect clients on second call" [Undecided,New] | 20:12 |
| *** armax has joined #openstack-lbaas | 20:13 | |
| blogan | johnsom: i could claim that V1 -> v1 is outside the scope of that review :) | 20:15 |
| blogan | bc it is | 20:15 |
| blogan | is it worth that battle? nope | 20:15 |
| johnsom | Yeah, I was mostly thinking about a way to clear the -1's | 20:15 |
| johnsom | grin | 20:15 |
| rm_work | heh | 20:16 |
| rm_work | yeah | 20:16 |
| openstackgerrit | Brandon Logan proposed openstack/octavia: Fix / and /v1 API uris https://review.openstack.org/277544 | 20:16 |
| sbalukoff | Delicous scope... | 20:16 |
| blogan | johnsom, rm_work: the endpoint thing is going to require some discussion | 20:16 |
| johnsom | Agreed | 20:17 |
| blogan | main question, is it abd to instantiate a new client every time? or does it need to be smart enough to keep track of the options as rm_work suggested | 20:17 |
| blogan | s/abd/bad | 20:17 |
| rm_work | it isn't that hard to track them | 20:17 |
| rm_work | I could probably do that and submit it | 20:17 |
| *** armax has quit IRC | 20:17 | |
| rm_work | in like an hour (90% of which would be testing) | 20:18 |
| blogan | but is it worth that effort if its not even a big deal to instantiate a new one everytime? | 20:18 |
| *** armax has joined #openstack-lbaas | 20:18 | |
| blogan | johnsom: is that what you meant by the endpoint thing? i assume dyou were talking about what came out of that review | 20:19 |
| rm_work | uhh, well | 20:20 |
| rm_work | it has to *hit keystone* every time, right? | 20:20 |
| rm_work | to get the URL | 20:20 |
| rm_work | so that's a request to keystone every time we use a client | 20:20 |
| rm_work | when we could just as easily *not* | 20:20 |
| rm_work | unless that stuff is cached at a different level/ | 20:21 |
| blogan | thats what the session does though, if you instantiate the client with the session then it shouldn't hit keystone everytime | 20:21 |
| blogan | but honeslty i dont know that for fact, just what i've assumed | 20:21 |
| rm_work | hmm | 20:21 |
| rm_work | we can try just removing cacheing | 20:21 |
| rm_work | can't have cache invalidation if you don't cache | 20:21 |
| rm_work | problem solved | 20:21 |
| johnsom | blogan I was talking about your comment when I asked about the V1->v1 thing | 20:21 |
| rm_work | whelp http://www.mysanantonio.com/news/local/article/San-Antonio-fire-crews-respond-to-fire-at-parking-6815580.php | 20:22 |
| johnsom | So, since it's +A'd, nevermind | 20:22 |
| blogan | johnsom: ahh so different, my best guess right now is just to return a body with the /loadbalancers uri | 20:22 |
| blogan | lol | 20:22 |
| blogan | ah the rivercenter parking lot! | 20:22 |
| blogan | garage | 20:22 |
| rm_work | heh | 20:23 |
| rm_work | "Aly Zamora said she was driving past the mall when the fire began and initially thought it may have been a terrorist attack and was worried." | 20:23 |
| rm_work | >_> | 20:23 |
| sbalukoff | ... | 20:23 |
| sbalukoff | Wow | 20:23 |
| sbalukoff | Well, everyone knows that 99% of fires in the U.S. are started by terrorists. | 20:23 |
| rm_work | I mean, they may as well just clasify Arson as Terrorism, right? >_> | 20:24 |
| blogan | anytime i see a dead animal in the road i immediately think terrorist | 20:24 |
| rm_work | (not that this was arson) | 20:24 |
| johnsom | I'm sure they already have... | 20:24 |
| blogan | well they can already just say you're a terrorist and kill you without any courts involved | 20:25 |
| blogan | or some secret court is involved | 20:26 |
| rm_work | is this an episode of Twilight Zone or something where it pans out and they ask "and who are the REAL terrorists?" | 20:27 |
| blogan | and then they show ... aliens? | 20:27 |
| *** armax has quit IRC | 20:30 | |
| rm_work | I was thinking "the government", but, could be aliens :P | 20:30 |
| sbalukoff | Why not both? | 20:32 |
| rm_work | https://www.youtube.com/watch?v=jEUJfEBNgmM | 20:33 |
| johnsom | I gave a pass at the e-mail to the openstack-dev octavia list from Major Hayden about security stuff. | 20:35 |
| johnsom | Feel free to join in the fun | 20:35 |
| rm_work | blogan: https://review.openstack.org/#/c/277533/ | 20:35 |
| rm_work | oh I guess it's waiting on zuul | 20:36 |
| rm_work | <_< | 20:36 |
| rm_work | so close to done | 20:36 |
| rm_work | also: https://review.openstack.org/#/c/277506/ | 20:37 |
| rm_work | tried to keep the warning squelch as local in scope as possible | 20:38 |
| blogan | that looks like something i wrote :) | 20:38 |
| blogan | though i'm not sure | 20:39 |
| rm_work | ok now https://review.openstack.org/#/c/277533/ passed | 20:40 |
| *** ducttape_ has quit IRC | 20:41 | |
| *** madhu_ak has quit IRC | 20:43 | |
| blogan | scenario tests failed | 20:43 |
| blogan | session persistence | 20:43 |
| johnsom | Yeah. I looked at that and +A'd it anyway | 20:45 |
| openstackgerrit | German Eichberger proposed openstack/octavia: Adds a parameter to specify endpoint type https://review.openstack.org/271476 | 20:48 |
| xgerman | hope it pleases the crowd... | 20:51 |
| blogan | xgerman: is this the first time you've done a patch in openstack? | 20:52 |
| blogan | xgerman: you seem surprised by the many opinions :) | 20:52 |
| xgerman | well, I am not doing pools… I was thinking maybe 4 reviews… but 8 or 9 is getting silly | 20:53 |
| rm_work | yeah I really hope we aren't expected to recheck for the nonvoting scenario tests every time they fail due to the several outstanding intermittent bugs :P | 20:53 |
| blogan | rm_work: i look to see what the failure is, but if it sone of those intermitten failures i don't request to wait for it ot pass | 20:54 |
| rm_work | speaking of pools/L7, isn't this one ready? https://review.openstack.org/#/c/265430/ | 20:54 |
| johnsom | Yeah, I dig into the failures because that gate has caught a number of valid issues too | 20:55 |
| rm_work | yep | 20:55 |
| *** ducttape_ has joined #openstack-lbaas | 20:56 | |
| openstackgerrit | Doug Wiegley proposed openstack/neutron-lbaas: WIP - do not merge, negative CI test https://review.openstack.org/277557 | 20:56 |
| *** jwarendt has joined #openstack-lbaas | 20:58 | |
| *** madhu_ak has joined #openstack-lbaas | 20:58 | |
| *** armax has joined #openstack-lbaas | 20:59 | |
| *** crc32 has joined #openstack-lbaas | 21:04 | |
| openstackgerrit | Merged openstack/octavia: Remove dead code around certificate handling https://review.openstack.org/277532 | 21:07 |
| openstackgerrit | Merged openstack/octavia: Use netifaces constants correctly in testing https://review.openstack.org/277533 | 21:12 |
| *** bank_ has quit IRC | 21:13 | |
| *** bana_k has quit IRC | 21:13 | |
| *** Bjoern has quit IRC | 21:20 | |
| openstackgerrit | Merged openstack/neutron-lbaas: Updated from global requirements https://review.openstack.org/277239 | 21:22 |
| openstackgerrit | Adam Harwell proposed openstack/octavia: Do not use __builtin__ in python3 https://review.openstack.org/262516 | 21:24 |
| rm_work | taking that over since no one wants to own it | 21:24 |
| johnsom | rm_work thanks! | 21:28 |
| rm_work | I *love* having time to focus on octavia | 21:28 |
| rm_work | been cooped up with internal stuff for a while | 21:28 |
| johnsom | yeah, I am trying to balance | 21:29 |
| rm_work | and I am OCD about having too many open bugs / CRs so I am trying to get some of this older stuff cleaned up... | 21:29 |
| rm_work | TrevorV: did you manage to replicate johnsom's problem on https://review.openstack.org/#/c/271512/ | 21:30 |
| TrevorV | Not so far, but scenario tests are still running | 21:31 |
| TrevorV | It got past the test that WAS failing though... | 21:31 |
| *** rooney_ has joined #openstack-lbaas | 21:32 | |
| TrevorV | johnsom I don't understand... the tests are passing on my devstack instance in our Rackspace cloud. | 21:34 |
| johnsom | I don't understand that gate failure either. Maybe we should just recheck? | 21:34 |
| TrevorV | Maybe... Lemme finish my scenario test run first just in case | 21:35 |
| johnsom | I really worry about the timing dimension to this patch | 21:35 |
| TrevorV | You mean the amount of time to wait before a failover occurs, or with "how fast we merge it"? | 21:35 |
| TrevorV | I'm lost | 21:35 |
| rm_work | yeah, that is why suggested a long timeout value for default | 21:36 |
| rm_work | which SHOULD be plenty, though maybe we need even longer? | 21:36 |
| rm_work | are there actually failover tests that run though in scenario? | 21:36 |
| johnsom | Timeouts | 21:36 |
| johnsom | No, the scenario should not trigger a failover. That is really an Octavia concept. | 21:37 |
| rm_work | yeah, that is what I thought | 21:37 |
| johnsom | It would go in Octavia scenario tests | 21:37 |
| rm_work | so where are you talking about? | 21:37 |
| rm_work | what failed? | 21:37 |
| johnsom | With this patch, it triggered a failover in the scenario gate | 21:37 |
| TrevorV | The problem isn't that the test is triggering a failover, its that a failover is happening during the testing | 21:37 |
| TrevorV | The distinction is the "unexpectedness" of the failover. | 21:38 |
| TrevorV | It SHOULDN'T be failing over anything during the testing. | 21:38 |
| rm_work | ah rofl | 21:42 |
| TrevorV | Alright, so I guess a failover did occur on my scenario test stuffs | 21:44 |
| TrevorV | Lemme check logs | 21:44 |
| TrevorV | Wait, nope. | 21:44 |
| TrevorV | Idk why | 21:44 |
| TrevorV | I don't understand... its only failing during the cleanups... | 21:46 |
| rm_work | blogan / johnsom: is http://logs.openstack.org/06/277506/1/check/gate-neutron-lbaasv2-dsvm-scenario/856b8fd/console.html#_2016-02-08_21_36_46_298 also expected? | 21:46 |
| rm_work | test_listener_basic | 21:46 |
| blogan | rm_work: yep | 21:47 |
| blogan | well not expected but a known failure | 21:47 |
| johnsom | Expected? No. Happening, sadly yes | 21:47 |
| rm_work | that doesn't look like the session persistence one | 21:47 |
| rm_work | or the HM one | 21:47 |
| rm_work | >_> | 21:47 |
| blogan | its not | 21:47 |
| rm_work | how many of these are there | 21:48 |
| rm_work | jesus | 21:48 |
| blogan | 3 | 21:48 |
| rm_work | >_< | 21:48 |
| blogan | maybe 4 | 21:48 |
| rm_work | ok I am going to start looking at one or more of these | 21:48 |
| johnsom | Yeah, there are three or four of these similar failures | 21:48 |
| blogan | they're definitely happening much more frequently | 21:48 |
| rm_work | who is working on any of them? | 21:48 |
| rm_work | anyone? | 21:48 |
| johnsom | +1 | 21:48 |
| blogan | sbalukoff did for the session persistence one, but couldn't reproduce | 21:48 |
| johnsom | I am assigned one, but my time is hit and miss | 21:48 |
| johnsom | Take it if you want it | 21:49 |
| rm_work | hmm | 21:49 |
| rm_work | which one do we see the most | 21:49 |
| johnsom | session persistence | 21:49 |
| johnsom | IMHO | 21:49 |
| TrevorV | I just... I don't even right now... | 21:50 |
| TrevorV | johnsom it failed in the cleanup of the test_session_persistence test, right after deleting all the security groups... | 21:51 |
| TrevorV | Just like in the log in the job, just an entirely different test... | 21:51 |
| openstackgerrit | Merged openstack/octavia: Fixes Octavia Amphora RHEL based images https://review.openstack.org/272905 | 21:52 |
| openstackgerrit | Aishwarya Thangappa proposed openstack/octavia: Adding "region and endpoint_type" parameters to barbican_acl.py https://review.openstack.org/272861 | 21:53 |
| *** bana_k has joined #openstack-lbaas | 21:59 | |
| *** bank_ has joined #openstack-lbaas | 22:00 | |
| *** rtheis has quit IRC | 22:07 | |
| rm_work | am I just BAD at searching these bugs? can't seem to find the session persistence failure bug | 22:12 |
| *** localloop127 has quit IRC | 22:15 | |
| blogan | rm_work: neutron | 22:17 |
| blogan | rm_work: check it out in neutron's launchpad | 22:17 |
| rm_work | yeah i am in neutron's launchpad | 22:18 |
| rm_work | searching for "lbaas session" and nothing | 22:18 |
| rm_work | well, nothing relevant | 22:19 |
| rm_work | https://bugs.launchpad.net/neutron?field.searchtext=lbaas+session&search=Search&field.status%3Alist=NEW&field.status%3Alist=INCOMPLETE_WITH_RESPONSE&field.status%3Alist=INCOMPLETE_WITHOUT_RESPONSE&field.status%3Alist=CONFIRMED&field.status%3Alist=TRIAGED&field.status%3Alist=INPROGRESS&field.status%3Alist=FIXCOMMITTED&field.assignee=&field.bug_reporter=&field.omit_dupes=on&field.has_patch=&field.has_no_package= | 22:19 |
| rm_work | that's one of like 10 queries i tried | 22:19 |
| rm_work | lbaas gate; lbaas intermittent | 22:19 |
| blogan | rm_work: if i find it in 30 seconds will murder you | 22:19 |
| rm_work | "lbaas gate" is funny because it doesn't even bring up the bug I have open in the other window | 22:19 |
| blogan | https://bugs.launchpad.net/octavia/+bug/1541675 | 22:20 |
| openstack | Launchpad bug 1541675 in octavia "intermittent session persistence scenario test failure" [Critical,New] | 22:20 |
| blogan | rm_work: ^ | 22:20 |
| blogan | it was in octavia | 22:20 |
| rm_work | which is titled "LBAAS Intermittent gate failure TestHealthMonitorBasic.test_health_monitor_basic" | 22:20 |
| rm_work | rofl | 22:20 |
| rm_work | awesome | 22:20 |
| blogan | guess we'll have to standardize on where bugs should be reported | 22:20 |
| rm_work | i think search is broken anyway | 22:20 |
| * blogan murders rm_work | 22:20 | |
| rm_work | "LBAAS Intermittent gate failure TestHealthMonitorBasic.test_health_monitor_basic" isn't found by "lbaas gate" or "lbaas intermittent" | 22:20 |
| blogan | i couldn't be a liar | 22:20 |
| blogan | yeah that search sucks | 22:21 |
| rm_work | AH it's because it's an "opinion" what | 22:22 |
| rm_work | and that's filtered off by default in search | 22:22 |
| rm_work | >_< | 22:22 |
| blogan | what? | 22:22 |
| rm_work | so I am missing a ton of bugs possibly when i search | 22:22 |
| rm_work | https://bugs.launchpad.net/neutron/+bugs?field.searchtext=lbaas+gate&orderby=-importance&search=Search&field.status%3Alist=NEW&field.status%3Alist=OPINION&field.status%3Alist=CONFIRMED&field.status%3Alist=TRIAGED&field.status%3Alist=INPROGRESS&field.status%3Alist=FIXCOMMITTED&field.status%3Alist=INCOMPLETE_WITH_RESPONSE&field.status%3Alist=INCOMPLETE_WITHOUT_RESPONSE&assignee_option=any&field.assignee=&field.bug_reporter=&field.bug_c | 22:22 |
| rm_work | ommenter=&field.subscriber=&field.structural_subscriber=&field.tag=&field.tags_combinator=ANY&field.has_cve.used=&field.omit_dupes.used=&field.omit_dupes=on&field.affects_me.used=&field.has_patch.used=&field.has_branches.used=&field.has_branches=on&field.has_no_branches.used=&field.has_no_branches=on&field.has_blueprints.used=&field.has_blueprints=on&field.has_no_blueprints.used=&field.has_no_blueprints=on | 22:22 |
| rm_work | oh god | 22:22 |
| blogan | new bugs are opinions? | 22:22 |
| rm_work | https://bugs.launchpad.net/neutron/+bug/1542467 | 22:22 |
| openstack | Launchpad bug 1542467 in neutron "LBAAS Intermittent gate failure TestHealthMonitorBasic.test_health_monitor_basic" [Critical,Opinion] | 22:22 |
| rm_work | this one is | 22:22 |
| rm_work | marked that way by Nate | 22:22 |
| rm_work | ... can i file that as a bug? :P | 22:23 |
| blogan | ahh | 22:23 |
| blogan | Nate is the bug deputy | 22:23 |
| blogan | i asked him to join our channel | 22:24 |
| johnsom | Yeah, that was really strange. Not sure why that was mark opinion | 22:24 |
| *** bana_k has left #openstack-lbaas | 22:25 | |
| openstackgerrit | Merged openstack/octavia: Fix / and /v1 API uris https://review.openstack.org/277544 | 22:25 |
| xgerman | thanks blogan!! | 22:28 |
| rm_work | this is an interesting bug | 22:28 |
| rm_work | first of the app cookie session persistence requests goes to server1, everything else to server2 -- is it possible haproxy takes a moment after being configured to use cookie based persistence to actually reload? like, our agent returns OK before it's actually reloaded the config fully? | 22:31 |
| johnsom | rm_work Oh! That is probably the bug stephen reported and I assigned to myself. | 22:32 |
| johnsom | Darn! | 22:32 |
| johnsom | Sherif added https://github.com/openstack/octavia/blob/master/octavia/controller/worker/tasks/amphora_driver_tasks.py#L58 | 22:32 |
| johnsom | Which is wrong and broken | 22:32 |
| sbalukoff | Yep, that sounds like the bug I was seeing... | 22:32 |
| rm_work | hmm | 22:33 |
| johnsom | It causes the first listener update to have the wrong data | 22:33 |
| rm_work | so this is an ACTUAL bug | 22:33 |
| rm_work | yeah | 22:33 |
| rm_work | that would make sense I think for what i'm seeing? | 22:33 |
| rm_work | it's one update behind the whole time? | 22:33 |
| johnsom | The strange thing is that has been there since last summer, so why is it hitting us now??? | 22:33 |
| johnsom | Yeah, one update behind | 22:34 |
| johnsom | https://bugs.launchpad.net/octavia/+bug/1542120 | 22:34 |
| openstack | Launchpad bug 1542120 in octavia "Changes made via API do not immediately get pushed to amphorae" [Critical,New] - Assigned to Michael Johnson (johnsom) | 22:34 |
| rm_work | LOL | 22:34 |
| rm_work | that thing >_< | 22:35 |
| rm_work | didn't sbalukoff essentially have the fix already | 22:35 |
| sbalukoff | It's a work-around. | 22:35 |
| rm_work | just was going to move it out to another patchset? | 22:35 |
| rm_work | ah | 22:35 |
| johnsom | Fixed a different way. | 22:35 |
| sbalukoff | The work-around is detailed in my bug report. | 22:35 |
| johnsom | So, the fix is to remove that database line from the code | 22:35 |
| sbalukoff | Though johnsom is right in that it's probably not the *right* solution to the problem. | 22:35 |
| johnsom | Problem is I'm not sure if the issue Sherif saw is still an issue or not. Either way it should be fixed a better way | 22:36 |
| sbalukoff | johnsom: And then revisit the old bug and try to figure out why we needed it in the first place? | 22:36 |
| johnsom | Yep | 22:36 |
| johnsom | I suspect it is a sqlalchemy model invalidation bug | 22:36 |
| sbalukoff | Fun! | 22:37 |
| johnsom | model/cache whatever they call it in sqlalchemy | 22:37 |
| rm_work | AH, i see, yeah | 22:37 |
| *** neelashah has quit IRC | 22:37 | |
| rm_work | alright | 22:37 |
| sbalukoff | Well, if the bug is still there (and unlikely to go away, since SQLAlchemy is... obtuse), there are other work-arounds we can try. it'd be nice not to have to use a work-around, though. | 22:38 |
| rm_work | what's more surprising is that this ever DOESN'T happen | 22:38 |
| sbalukoff | rm_work: You and me both. | 22:38 |
| sbalukoff | rm_work: That tells me the current tempest tests are... well, not thorough. | 22:38 |
| johnsom | Yeah, me three | 22:38 |
| rm_work | <_< | 22:38 |
| sbalukoff | which is one of the reason's I'm looking forward to fnaval's patch. | 22:39 |
| sbalukoff | :) | 22:39 |
| rm_work | yeah, very tired of cringing every time I +A anything | 22:39 |
| rm_work | because the tests passing often means nothing because either it isn't tested or the test is wrong or imcomplete <_< | 22:39 |
| rm_work | *incomplete | 22:39 |
| *** doug-fish has quit IRC | 22:41 | |
| johnsom | So, if you want I can push a patch that takes that line out. We just need to put some test cycles behind it to find any other issues that pop out. | 22:42 |
| johnsom | That line *is* wrong | 22:42 |
| rm_work | yeah | 22:42 |
| rm_work | I'd say we go that route | 22:42 |
| rm_work | I am decently comfortable looking into sqlalchemy issues | 22:43 |
| rm_work | moreso than futzing with flows | 22:43 |
| johnsom | Yeah, I know those flows worked prior to that change | 22:43 |
| *** doug-fish has joined #openstack-lbaas | 22:44 | |
| blogan | even when octavia gets its own tests, we're still going to be under teh 2 hour restrictin | 22:44 |
| blogan | so at that point should the db have the correct information and sqlalchemy has just cached it? | 22:45 |
| blogan | johnsom, rm_work, sbalukoff: ^^? | 22:45 |
| johnsom | blogan You have confused me | 22:45 |
| rm_work | I will need to see what exactly is happening | 22:46 |
| rm_work | if it still is | 22:46 |
| blogan | johnsom: in the line you linked above, in the amphora_driver_tasks | 22:46 |
| *** doug-fis_ has joined #openstack-lbaas | 22:46 | |
| blogan | yall mentioned it could be a sqlalchemy issue | 22:46 |
| rm_work | blogan: no at that point the DB has the WRONG info | 22:46 |
| rm_work | it doesn't have the info from the update that is being made | 22:46 |
| rm_work | so | 22:46 |
| rm_work | >_> | 22:46 |
| blogan | rm_work: the actual db has the wrong informatin or sqlalchemy is returning hte wrong information as if its being cached? | 22:46 |
| rm_work | the db actually has the wrong info | 22:47 |
| rm_work | it still has the old info | 22:47 |
| rm_work | DB is updated from the request in the next flow | 22:47 |
| blogan | ah okay bc we went the hacky route of not updating the db until it got finished so we could roll back to a known state | 22:47 |
| rm_work | yes | 22:47 |
| rm_work | johnsom: it's literally just ... removing that line, eh? you want to propose it? | 22:48 |
| johnsom | Working on it right onw | 22:48 |
| *** doug-fish has quit IRC | 22:48 | |
| blogan | if that bug that is in the comments comes up, we can probably do a session.refresh | 22:49 |
| rm_work | yeah i have a bunch of thoughts too | 22:49 |
| rm_work | but it all depends on what actually is the problem or if it even has a problem | 22:49 |
| blogan | you're the problem | 22:50 |
| blogan | and the solution! | 22:50 |
| rm_work | your FACE is the problem | 22:50 |
| blogan | but mostly the problem | 22:50 |
| *** crc32 has quit IRC | 22:50 | |
| blogan | my FACADE is the problem | 22:50 |
| blogan | behind the facade is a face that is the solution | 22:50 |
| *** doug-fis_ has quit IRC | 22:50 | |
| rm_work | https://www.youtube.com/watch?v=x9kJ6IJnBz0 | 22:50 |
| *** crc32 has joined #openstack-lbaas | 22:51 | |
| johnsom | Ok, something strange is going on. When I git commit on a clean checked out directory I get a commit message and "modifed" files that aren't modified in the repo | 22:54 |
| johnsom | directory | 22:54 |
| rm_work | <_< | 22:54 |
| johnsom | I have never seen this before | 22:55 |
| rm_work | uhh | 22:55 |
| rm_work | what steps did you do that i can repro | 22:55 |
| rm_work | and what is the output of git status | 22:55 |
| johnsom | Oh, nevermind, I know what my issue is. Doing 12 things at once like an idiot | 22:56 |
| rm_work | heh | 22:56 |
| rm_work | you mean doing 12 things at once like someone who is a natural multitasker? :P | 22:56 |
| rm_work | i honestly bet all of these intermittent issues are related to this same thing | 22:57 |
| rm_work | old data | 22:57 |
| *** doug-fish has joined #openstack-lbaas | 22:59 | |
| *** allan_h has quit IRC | 22:59 | |
| openstackgerrit | Michael Johnson proposed openstack/octavia: Reverts a workaround that is impacting amp updates https://review.openstack.org/277602 | 23:00 |
| rm_work | alright, will look forward to seeing results from that -- and will start building my own env with that patch now | 23:01 |
| johnsom | I think there is a delete member patch that probably needs to be reverted and the model update code applied as well. | 23:02 |
| *** TrevorV has quit IRC | 23:03 | |
| openstackgerrit | Merged openstack/octavia: Adds a parameter to specify endpoint type https://review.openstack.org/271476 | 23:03 |
| rm_work | johnsom: ah yeah i think i remember that | 23:03 |
| *** doug-fish has quit IRC | 23:03 | |
| rm_work | they did the flow-flip thing, right? | 23:03 |
| rm_work | during the midcycle to fix that bug | 23:04 |
| johnsom | Yeah | 23:04 |
| *** doug-fish has joined #openstack-lbaas | 23:04 | |
| rm_work | i'll try to figure this out | 23:04 |
| johnsom | I have it open, if this works out I will submit that too | 23:04 |
| rm_work | and then we can try to include that in this patch too | 23:04 |
| rm_work | yeah | 23:04 |
| johnsom | It's this one: https://bugs.launchpad.net/octavia/+bug/1514510 | 23:04 |
| openstack | Launchpad bug 1514510 in octavia "Delete of last member does not remove it from haproxy config" [Critical,Fix released] - Assigned to Eran Raichstein (eranra) | 23:04 |
| johnsom | It looks like the model update code was missing originally | 23:04 |
| rm_work | yep, i remember helping look at that | 23:04 |
| rm_work | should just be the flow order reversal, since it uses listenerupdate as well | 23:05 |
| johnsom | Yeah, but the model update code will need to be added to delete the member from the model | 23:06 |
| rm_work | ah | 23:06 |
| johnsom | Like this: https://github.com/openstack/octavia/blob/master/octavia/controller/worker/flows/pool_flows.py#L45 | 23:07 |
| rm_work | that's already there, isn't it? | 23:07 |
| rm_work | line 56 here: https://review.openstack.org/#/c/260605/5/octavia/controller/worker/flows/member_flows.py | 23:08 |
| johnsom | Oh, it is there | 23:08 |
| johnsom | Ok | 23:08 |
| rm_work | ... in like half an hour I'll have an environment :P | 23:09 |
| rm_work | blogan: https://review.openstack.org/#/c/277506/ | 23:10 |
| rm_work | sbalukoff: ^^ you're a +2 now :) | 23:11 |
| sbalukoff | Heh! | 23:11 |
| sbalukoff | (Sorry, I'm being pulled in 5 directions at once at the moment.) | 23:12 |
| rm_work | lol no worries if you're busy just ignore me | 23:12 |
| *** doug-fish has quit IRC | 23:14 | |
| *** doug-fish has joined #openstack-lbaas | 23:14 | |
| *** doug-fish has quit IRC | 23:15 | |
| *** doug-fish has joined #openstack-lbaas | 23:15 | |
| sbalukoff | rm_work: Just had a look and +A'ed. Just out of curiosity though: was there a problem with pyasn1? | 23:17 |
| rm_work | it isn't used | 23:17 |
| rm_work | it's just required as part of the security thing | 23:17 |
| rm_work | IF we use ndg-httpsclient | 23:17 |
| rm_work | it's ... that whole thing is super weird with the auto-injecting | 23:17 |
| rm_work | I'm hoping we can get back on the global-reqs checking soon <_< | 23:19 |
| sbalukoff | Aah. | 23:19 |
| rm_work | I think that was the only one besides the taskflow version | 23:19 |
| xgerman | task flow got +A I thoiught | 23:19 |
| rm_work | just now | 23:19 |
| rm_work | waiting for it to merge still | 23:19 |
| rm_work | https://review.openstack.org/#/c/276549/ | 23:19 |
| xgerman | yep | 23:20 |
| rm_work | well, hours ago | 23:20 |
| rm_work | but still in the pipe | 23:20 |
| xgerman | the gozers of this world | 23:20 |
| johnsom | Yeah, I poked them in the oslo meeting this morning | 23:20 |
| rm_work | lol | 23:20 |
| rm_work | thanks for the +As guys :P | 23:20 |
| *** ajmiller_ has quit IRC | 23:20 | |
| blogan | rm_work: you asked for it | 23:20 |
| rm_work | johnsom: still time for you to +A that | 23:21 |
| johnsom | Ha! | 23:21 |
| johnsom | I would do no such vote stacking | 23:21 |
| johnsom | I have to be a role model | 23:22 |
| johnsom | grin | 23:22 |
| rm_work | lol | 23:22 |
| rm_work | such integrity | 23:22 |
| * blogan starts the smear compaign against johnsom | 23:23 | |
| blogan | superPAC | 23:23 |
| johnsom | I will laugh if it fails in the merge gate | 23:23 |
| blogan | what skeletons does johnsom have in his closet | 23:23 |
| *** doug-fish has quit IRC | 23:23 | |
| johnsom | Plenty I'm sure | 23:23 |
| blogan | my closet is full of pre-skeletons... | 23:24 |
| blogan | and my crawlspaces | 23:25 |
| johnsom | Mmmm, I bet it smells lovely | 23:25 |
| *** doug-fish has joined #openstack-lbaas | 23:26 | |
| *** doug-fish has quit IRC | 23:27 | |
| *** doug-fish has joined #openstack-lbaas | 23:28 | |
| *** ajmiller has joined #openstack-lbaas | 23:29 | |
| rm_work | waaat https://review.openstack.org/#/c/276549/ | 23:35 |
| *** piet has quit IRC | 23:35 | |
| johnsom | Great, everyone's gates are broken.... | 23:36 |
| sbalukoff | Smells like Monday, eh! | 23:37 |
| *** doug-fish has quit IRC | 23:38 | |
| rm_work | guess a recheck should fix it according to clarkb, giving that a shot | 23:38 |
| *** piet has joined #openstack-lbaas | 23:38 | |
| *** doug-fish has joined #openstack-lbaas | 23:40 | |
| *** piet has quit IRC | 23:43 | |
| *** doug-fish has quit IRC | 23:46 | |
| johnsom | sbalukoff Let me help you make it even more of a Monday: | 23:50 |
| johnsom | https://bugs.launchpad.net/octavia/+bug/1543356 | 23:50 |
| openstack | Launchpad bug 1543356 in octavia "Errors found for octavia api----get listener stats" [Undecided,New] | 23:50 |
| johnsom | https://bugs.launchpad.net/octavia/+bug/1543366 | 23:50 |
| openstack | Launchpad bug 1543366 in octavia "Errors found for octavia api----create health monitor on deprecated endpoint Edit" [Undecided,New] | 23:50 |
| johnsom | https://bugs.launchpad.net/octavia/+bug/1543364 | 23:50 |
| openstack | Launchpad bug 1543364 in octavia "Errors found for octavia api----create health monitor on new endpoint" [Undecided,New] | 23:50 |
| johnsom | Related to shared pools patch testing | 23:50 |
| *** doug-fish has joined #openstack-lbaas | 23:50 | |
| *** doug-fish has quit IRC | 23:50 | |
| *** doug-fish has joined #openstack-lbaas | 23:50 | |
| *** ducttape_ has quit IRC | 23:53 | |
| openstackgerrit | Merged openstack/octavia: Revert "Fix for ssl context wrapper spam", add warning filter https://review.openstack.org/277506 | 23:53 |
| *** doug-fish has quit IRC | 23:54 | |
| *** rooney_ has quit IRC | 23:56 | |
| rm_work | ok i need to be off for a while | 23:58 |
| rm_work | catch most of you tomorrow and maybe some of you tonight :P | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!