Wednesday, 2017-02-08

« Tuesday, 2017-02-07 Index Thursday, 2017-02-09 »
*** fnaval has quit IRC00:04
*** ducttape_ has quit IRC00:11
*** ducttape_ has joined #openstack-lbaas00:16
*** hjensas has joined #openstack-lbaas00:21
*** cody-somerville_ has joined #openstack-lbaas00:25
*** ducttape_ has quit IRC00:26
rm_workjohnsom: for translations for strings returning from API calls... do we use _() ?00:31
johnsomI think so00:31
rm_workk00:32
*** ducttape_ has joined #openstack-lbaas00:32
johnsom_() is preferred for any user facing message, even if it is also going to a log file. This ensures that the translated version of the message will be available to the user.00:32
johnsomhttp://docs.openstack.org/developer/oslo.i18n/guidelines.html#choosing-a-marker-function00:33
rm_workk00:36
*** cody-somerville_ has quit IRC00:43
rm_workupdating my network_id patch shortly: now with infinity% more validation00:59
openstackgerritAdam Harwell proposed openstack/octavia master: Add network_id to VIP to match neutron-lbaas  https://review.openstack.org/42992601:28
*** yamamoto has quit IRC01:34
*** yamamoto has joined #openstack-lbaas01:38
*** yamamoto has quit IRC01:43
*** ipsecguy has quit IRC01:51
*** ipsecguy has joined #openstack-lbaas01:51
*** amotoki has joined #openstack-lbaas02:07
*** cody-somerville has quit IRC02:19
*** gongysh has joined #openstack-lbaas02:31
*** gongysh has quit IRC02:32
*** yamamoto has joined #openstack-lbaas02:40
*** cody-somerville has joined #openstack-lbaas02:55
*** cody-somerville has quit IRC02:55
*** cody-somerville has joined #openstack-lbaas02:55
*** amotoki has quit IRC03:00
*** yamamoto has quit IRC03:01
*** amotoki has joined #openstack-lbaas03:10
*** amotoki has quit IRC03:15
*** amotoki has joined #openstack-lbaas03:19
*** armax has quit IRC03:35
*** armax has joined #openstack-lbaas03:36
*** armax has quit IRC03:36
*** amotoki has quit IRC03:40
*** ducttape_ has quit IRC03:47
*** dileepr has quit IRC03:49
*** fnaval has joined #openstack-lbaas03:49
*** links has joined #openstack-lbaas03:50
*** fnaval has quit IRC03:50
*** fnaval has joined #openstack-lbaas03:51
*** sanfern has joined #openstack-lbaas04:02
*** beardedeagle has joined #openstack-lbaas04:10
*** amotoki has joined #openstack-lbaas04:11
*** ducttape_ has joined #openstack-lbaas04:12
*** ducttape_ has quit IRC04:27
*** ducttape_ has joined #openstack-lbaas04:28
*** faizy has joined #openstack-lbaas04:30
*** csomerville has joined #openstack-lbaas04:35
*** cody-somerville has quit IRC04:35
*** ducttape_ has quit IRC04:38
*** fnaval has quit IRC04:45
*** gongysh has joined #openstack-lbaas05:03
*** amotoki has quit IRC05:03
*** amotoki has joined #openstack-lbaas05:04
*** csomerville has quit IRC05:20
*** ducttape_ has joined #openstack-lbaas05:39
*** sanfern has quit IRC05:41
*** ducttape_ has quit IRC05:48
reedipHi, any reviews for https://review.openstack.org/#/c/404774/ ?05:50
*** amotoki_ has joined #openstack-lbaas06:13
*** amotoki has quit IRC06:14
*** anilvenkata has joined #openstack-lbaas06:25
openstackgerritZhaoBo proposed openstack/octavia master: Not allow to create vip in lb-mgmt-net  https://review.openstack.org/43057806:25
openstackgerritZhaoBo proposed openstack/octavia master: Not allow to create vip in lb-mgmt-net  https://review.openstack.org/43057806:31
*** kobis has joined #openstack-lbaas06:39
*** cody-somerville_ has joined #openstack-lbaas06:40
*** eezhova has joined #openstack-lbaas06:52
*** rcernin has joined #openstack-lbaas06:59
*** kobis has quit IRC07:03
*** beardedeagle has quit IRC07:04
*** hjensas has quit IRC07:15
*** kobis has joined #openstack-lbaas07:24
*** pcaruana has joined #openstack-lbaas07:29
*** zoukeke has joined #openstack-lbaas07:32
*** tesseract has joined #openstack-lbaas07:35
*** eezhova has quit IRC07:43
*** ducttape_ has joined #openstack-lbaas07:45
*** ducttape_ has quit IRC07:52
*** yamamoto has joined #openstack-lbaas07:56
*** yamamoto has quit IRC07:56
*** hjensas has joined #openstack-lbaas07:58
*** eezhova has joined #openstack-lbaas08:16
*** yamamoto has joined #openstack-lbaas08:20
*** yamamoto has quit IRC08:26
*** chlong has quit IRC08:30
*** chlong has joined #openstack-lbaas08:31
openstackgerritZhaoBo proposed openstack/octavia master: Not allow to create vip in lb-mgmt-net  https://review.openstack.org/43057808:42
*** hjensas has quit IRC09:03
*** amotoki_ has quit IRC09:09
*** zoukeke has quit IRC09:21
*** zoukeke has joined #openstack-lbaas09:21
*** zoukeke has quit IRC09:21
*** zoukeke has joined #openstack-lbaas09:21
*** ipsecguy has quit IRC09:32
*** openstackgerrit has quit IRC09:34
*** links has quit IRC09:35
*** zw_ has joined #openstack-lbaas09:40
*** med_` has joined #openstack-lbaas09:44
*** coreycb` has joined #openstack-lbaas09:45
*** anilvenkata has quit IRC09:45
*** zw has quit IRC09:45
*** med_ has quit IRC09:45
*** coreycb has quit IRC09:45
*** ducttape_ has joined #openstack-lbaas09:48
*** links has joined #openstack-lbaas09:51
*** anilvenkata has joined #openstack-lbaas09:52
*** ducttape_ has quit IRC09:56
*** zoukeke has quit IRC10:07
*** openstackgerrit has joined #openstack-lbaas10:13
*** yamamoto has joined #openstack-lbaas10:17
*** hjensas has joined #openstack-lbaas10:20
*** yamamoto has quit IRC10:22
*** ipsecguy has joined #openstack-lbaas10:26
openstackgerritLi Jingjing proposed openstack/neutron-lbaas master: Fix typo in .pylintrc  https://review.openstack.org/43070910:27
*** links has quit IRC10:50
*** ipsecguy has quit IRC10:59
*** links has joined #openstack-lbaas11:06
openstackgerritZhixin Li proposed openstack/neutron-lbaas master: Keep user_group option of haproxy section in lbaas_agent.ini  https://review.openstack.org/43083211:07
openstackgerritZhixin Li proposed openstack/neutron-lbaas master: Keep user_group option of haproxy section in lbaas_agent.ini  https://review.openstack.org/43083211:09
*** yamamoto has joined #openstack-lbaas11:20
*** yamamoto has quit IRC11:26
*** Redlook has joined #openstack-lbaas11:33
*** ipsecguy has joined #openstack-lbaas11:46
*** ipsecguy has quit IRC11:46
*** ducttape_ has joined #openstack-lbaas11:47
*** gongysh has quit IRC11:53
*** hjensas has quit IRC12:05
*** ipsecguy has joined #openstack-lbaas12:09
*** faizy has quit IRC12:16
*** ipsecguy has quit IRC12:16
*** ipsecguy has joined #openstack-lbaas12:17
*** ducttape_ has quit IRC12:20
*** ducttape_ has joined #openstack-lbaas12:32
*** _ducttape_ has joined #openstack-lbaas12:34
*** yamamoto has joined #openstack-lbaas12:35
*** _ducttape_ has quit IRC12:36
*** ipsecguy has quit IRC12:36
*** _ducttape_ has joined #openstack-lbaas12:36
*** ipsecguy has joined #openstack-lbaas12:37
*** ducttape_ has quit IRC12:37
*** ipsecguy has quit IRC12:41
*** coreycb` has quit IRC12:48
*** yamamoto has quit IRC12:48
*** coreycb has joined #openstack-lbaas12:49
*** _ducttape_ has quit IRC12:51
*** ducttape_ has joined #openstack-lbaas12:51
*** gongysh has joined #openstack-lbaas12:55
*** catintheroof has joined #openstack-lbaas13:08
*** hjensas has joined #openstack-lbaas13:22
*** hjensas has quit IRC13:22
*** hjensas has joined #openstack-lbaas13:22
*** ducttape_ has quit IRC13:31
*** links has quit IRC13:39
*** AlexeyAbashkin has joined #openstack-lbaas13:42
*** AlexeyAbashkin has quit IRC13:42
*** yamamoto has joined #openstack-lbaas13:49
*** faizy has joined #openstack-lbaas13:52
*** yamamoto has quit IRC13:55
openstackgerritNir Magnezi proposed openstack/octavia master: Auto detect haproxy user_group  https://review.openstack.org/42939814:03
openstackgerritNir Magnezi proposed openstack/octavia master: Auto detect haproxy user_group  https://review.openstack.org/42939814:09
*** ducttape_ has joined #openstack-lbaas14:10
*** AlexeyAbashkin has joined #openstack-lbaas14:13
*** matt-borland has joined #openstack-lbaas14:24
*** ducttape_ has quit IRC14:31
*** ducttape_ has joined #openstack-lbaas14:31
*** _ducttape_ has joined #openstack-lbaas14:37
*** ducttape_ has quit IRC14:38
*** fnaval has joined #openstack-lbaas14:54
*** AlexeyAbashkin has quit IRC15:05
*** ipsecguy has joined #openstack-lbaas15:08
*** kobis1 has joined #openstack-lbaas15:13
*** kobis has quit IRC15:14
*** kobis1 has quit IRC15:15
*** fnaval has quit IRC15:50
*** fnaval has joined #openstack-lbaas15:51
*** armax has joined #openstack-lbaas16:03
*** kobis has joined #openstack-lbaas16:31
*** anilvenkata has quit IRC16:32
openstackgerritMerged openstack/neutron-lbaas-dashboard master: Imported Translations from Zanata  https://review.openstack.org/42919316:38
*** rcernin has quit IRC16:39
*** beardedeagle has joined #openstack-lbaas16:42
*** eezhova has quit IRC16:51
*** tesseract has quit IRC17:05
*** _ducttape_ has quit IRC17:19
*** ducttape_ has joined #openstack-lbaas17:20
*** armax has quit IRC17:23
*** armax has joined #openstack-lbaas17:24
*** armax has quit IRC17:32
*** eezhova has joined #openstack-lbaas17:39
*** armax has joined #openstack-lbaas17:39
openstackgerritNakul Dahiwade proposed openstack/octavia master: Align Octavia API to n-lbaasv2 for L7Policy  https://review.openstack.org/40632817:42
openstackgerritShashank Kumar Shankar proposed openstack/octavia master: Introduce Octavia v2 API for pools  https://review.openstack.org/40592217:50
*** cpuga has joined #openstack-lbaas18:20
*** cpuga has quit IRC18:22
*** cpuga has joined #openstack-lbaas18:23
*** robcresswell has quit IRC18:35
*** amitry has quit IRC18:35
*** amit213 has quit IRC18:35
*** dasanind has quit IRC18:35
*** sindhu has quit IRC18:35
*** kong has quit IRC18:37
*** ducttape_ has quit IRC18:39
*** ctracey has quit IRC18:45
*** johnsom has quit IRC18:45
*** dlundquist has quit IRC18:45
openstackgerritMerged openstack/octavia master: Fix py3x gates and functional  https://review.openstack.org/42489019:01
*** faizy has quit IRC19:08
*** hjensas has quit IRC19:10
*** kobis has quit IRC19:13
*** amitry has joined #openstack-lbaas19:17
*** ctracey has joined #openstack-lbaas19:28
*** johnsom-alt has joined #openstack-lbaas19:29
johnsom-altJoy, IRCcloud is down.  Using the web client for now.  Hope it is back in time for the meeting19:31
*** ducttape_ has joined #openstack-lbaas19:40
xgermanmmh, works for me19:55
*** robcresswell has joined #openstack-lbaas19:55
johnsom-althttps://twitter.com/irccloud19:56
johnsom-altI guess I am in the unlucky half...19:56
*** kong has joined #openstack-lbaas19:59
*** dlundquist has joined #openstack-lbaas20:02
*** ducttape_ has quit IRC20:04
*** ducttape_ has joined #openstack-lbaas20:05
*** johnsom has joined #openstack-lbaas20:06
*** amit213 has joined #openstack-lbaas20:11
*** dasanind has joined #openstack-lbaas20:14
*** sindhu has joined #openstack-lbaas20:16
*** cody-somerville has joined #openstack-lbaas20:25
*** cody-somerville has quit IRC20:25
*** cody-somerville has joined #openstack-lbaas20:25
*** pcaruana has quit IRC20:43
openstackgerritNakul Dahiwade proposed openstack/octavia master: Introduce Octavia v2 API for Members  https://review.openstack.org/40593220:44
*** ducttape_ has quit IRC20:47
*** johnsom-alt has quit IRC20:48
openstackgerritShashank Kumar Shankar proposed openstack/octavia master: Introduce Octavia v2 API for pools  https://review.openstack.org/40592220:56
*** cpuga_ has joined #openstack-lbaas20:59
*** cpuga_ has quit IRC21:01
openstackgerritAdam Harwell proposed openstack/octavia master: Allow VIP on lb_mgmt_subnet  https://review.openstack.org/43117921:08
rm_workjohnsom: I think that's *it*21:08
rm_worknot complex21:08
rm_workit was way shittier just now but I think I fixed it to be reliable (though that means I haven't technically tested THIS code)21:08
*** cody-somerville has quit IRC21:09
rm_workbut something like this worked for me in our environment21:09
*** cody-somerville has joined #openstack-lbaas21:16
*** cody-somerville has quit IRC21:16
*** cody-somerville has joined #openstack-lbaas21:16
*** ducttape_ has joined #openstack-lbaas21:27
openstackgerritAdam Harwell proposed openstack/octavia master: Allow a configurable maximum Amphora name length  https://review.openstack.org/43118621:32
rm_work^^ will be nice to have that upstream too21:32
rm_workoh, I should add a test21:32
*** eezhova has quit IRC21:51
openstackgerritAdam Harwell proposed openstack/octavia master: Allow a configurable maximum Amphora name length  https://review.openstack.org/43118621:52
rm_workwoo a test21:54
rm_workneed to figure out how to add a test for the VIP-on-mgmt one21:54
*** Redlook has left #openstack-lbaas22:10
*** yamamoto has joined #openstack-lbaas22:10
openstackgerritSindhu Devale proposed openstack/octavia master: DB migration to add ID column to HM  https://review.openstack.org/43045122:12
*** cpuga has quit IRC22:21
openstackgerritMichael Johnson proposed openstack/octavia master: Fix Octavia v1 API project_id for POST  https://review.openstack.org/43120722:23
*** cody-somerville has quit IRC22:28
*** beardedeagle has quit IRC22:28
*** matt-borland has quit IRC22:29
rm_workjohnsom: is that a WIP? ^^22:47
johnsomYes22:48
*** dileepr has joined #openstack-lbaas22:51
rm_workk22:54
rm_workI'd like your thoughts on my other two patches22:54
rm_workI need a test on the first one, but the other is just a "do we mind accepting this" thing22:54
*** cody-somerville has joined #openstack-lbaas22:54
johnsomI have already started giving comments, just had to pause for a 1-122:55
rm_workah k22:56
johnsomNot a fan of the random...22:57
johnsomWhat is your limit?22:57
johnsomYou couldn't do one of the hash games?22:57
johnsomHow are you going to not run into collisions?  Not that the name is really that important or key here, but for supportability it is nice to be able to traceback/generate the name.23:00
rm_workjohnsom: so, I asked in crypto about that23:03
rm_workthe random choice there is what they recommended23:04
rm_workoriginally i was using a hash system and just clipping it23:04
rm_workbut due to the way some of those work, it's really not much better23:04
johnsomOur base64_sha1_string method didn't get you down to a size you needed?23:05
rm_workour setting is 15 characters, which with that patch makes it 14 characters of random, which is (26+10)^14?23:05
johnsomWell, you are adding a to it23:05
johnsomwhich I thought you didn't like + for strings23:06
rm_workpfft23:06
rm_workyeah I should fix that23:06
johnsomWell, let me give some comments23:06
rm_workbut yeah like I said, 14 characters of random23:06
rm_worksince I add an a23:06
rm_workbase64_sha1_string isn't even close23:07
rm_workand it's not as random actually23:07
johnsomYeah, it's not meant to be random, it's meant to be reproducable23:08
johnsomSolving different problems23:08
rm_workyeah23:09
rm_workbut I mean, for collision purposes, it's not useful23:09
bloganoye23:11
bloganrm_work, johnsom: https://github.com/openstack/octavia/blob/master/octavia/tests/functional/api/v1/test_pool.py#L25323:12
blogansshank and ndahiwade found that gem23:13
rm_workuhh23:13
johnsomlol23:13
rm_workwut23:13
johnsomWhew, it wasn't my code...  grin23:13
rm_workit's ... MAKING SURE we can't name pools?23:14
blogando a blame on that and it'll make sense23:14
rm_worklol23:14
rm_worki did :P23:14
bloganits not as easy as saying put should update it in the api layer, bc i just remember we had this discussion a while back about how we needed to keep the old version of the data and also ahve the new version...so we just didn't update the db at the api layer, we send it off to the controller to do that23:15
bloganso it works as intended23:15
bloganbut, these tests don't use the controller23:16
johnsomSo, um, why is that even passing23:16
rm_workah yeah i see23:16
rm_workthis is lulz tho23:17
blogani know, i think the update stuff needs a bit of an overhaul at some point, to where the db updates happen at the api layer23:17
bloganbut if we ever need to roll back on a failure, how would we do that?23:18
bloganthat was the problem23:18
johnsomRight, I remember that.23:18
*** fnaval has quit IRC23:18
bloganyea its slowly coming back23:18
johnsomI remember arguing for something like that, but I thought that was INSIDE the flows, not here23:19
johnsomBut, yes, I remember now23:19
johnsomIt's the whole async thing.23:19
blogani think we can still do the update here, btu first grab the old version, then update, then send the old version and new version to the controller worker, and if it ever fails do a diff on the two models and update the db back23:19
johnsomIf the handler call blows up, we had no way to undo the changes23:19
bloganoh yeah, that assumes we're using the cw too...23:20
johnsomYeah, probably23:20
bloganbut nlbaas just tossed it into error soooo...we have to maintain taht behavior :)23:20
rm_work>_<23:20
rm_workWTB not maintaining bad behavior23:21
blogantoo expensive of a buy23:21
rm_workT_T23:21
johnsomnlbaas didin't care what the result was from octavia.  Even if we throw 500 it still told the user 20223:21
johnsomSo, back at you....23:21
johnsomgrin23:21
rm_workf@#& n-lbaas23:21
rm_workcan't it die in a fire already23:21
bloganso octavia should just say success even on failure? i like that23:21
bloganalternative statuses?23:22
*** cody-somerville has quit IRC23:22
johnsomAnyhow, we are fixing that23:22
johnsom"alternative status" I like that23:22
rm_work^^ by removing n-lbaas? >_>23:22
rm_workfffff23:22
*** cody-somerville has joined #openstack-lbaas23:22
*** cody-somerville has quit IRC23:22
*** cody-somerville has joined #openstack-lbaas23:22
johnsomYes, by removing nlbaas23:22
blogancan we print out all the lines of code in nlbaas and set it on fire?23:22
johnsomSure23:23
johnsomHey, to make it fun you should print out v1 too23:23
bloganif that's still somewhere23:23
johnsomI have visions of rube goldberg with a paper shredder feeding a fire23:24
johnsomIt's still in mitaka23:24
rm_workfffff23:24
johnsomWell, until April23:24
rm_workyou guys are ruining my day23:24
*** catintheroof has quit IRC23:25
*** ajo has quit IRC23:26
johnsomSo what you are advocating for, blogan, is to change the API to synchronous?23:26
* johnsom Waits for rm_work's head to explode23:26
rm_worknope, too late, n-lbaas is frozen as of one minute ago23:27
rm_work+2 +A it's official23:27
johnsomHa, darn, I forgot to bring that up today23:27
*** ajo has joined #openstack-lbaas23:27
johnsomI really do want to freeze it23:27
bloganjohnsom: not synchronous, just update the db changes immediately23:27
bloganit'll still be in pending_update23:28
rm_work#vote When do we freeze the n-lbaas codebase? [Now, Five Minutes Ago]23:28
blogani guess if a rollback happens that my get confusing...so we need a ROLLBACK status23:28
blogan:)23:28
johnsomEMAIL-BLOGAN-DOH-STATUS23:29
*** cpuga has joined #openstack-lbaas23:41
johnsomrm_work Have you tried this AAP patch?23:42
johnsomIt shouldn't actually work due to the namespace right?23:42
johnsomOh, I think I get it now23:44
johnsomArgh, the spares pool failover patch broke the docs flow generation23:48
*** ducttape_ has quit IRC23:48
johnsomrm_work commented on both23:52
« Tuesday, 2017-02-07 Index Thursday, 2017-02-09 »

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