Friday, 2017-03-31

rm_workk np00:04
johnsomSo, setting this pool status to pending update....  Do we really need this for member?00:07
johnsomCurrently there is nothing to set it back...00:07
rm_workdid i miss resetting it?00:13
rm_worki mean, i think we need to set stuff to pending-update, no?00:13
rm_workit'll trigger a provisioning00:13
rm_workthe handler should set it back?00:13
johnsomYeah, we probably should.00:13
rm_workoh ffff except the handler is used to v1 pools00:13
johnsomThat is our current model yes00:13
rm_workoh but they still have prov status00:14
rm_workso it should work, right?00:14
johnsomThat shouldn't matter00:14
johnsomI have some comments inbound00:14
johnsomOk, that is what I see at first glance for the types00:21
rm_workk i'll deal with thaat later today i think00:24
*** ducttape_ has quit IRC00:27
*** gongysh has joined #openstack-lbaas00:47
*** fnaval has quit IRC00:49
openstackgerritMerged openstack/python-octaviaclient master: Update Docs for more information and dummy commands
*** armax has quit IRC00:54
*** Dave has quit IRC00:54
*** yuanying has quit IRC00:58
*** Dave has joined #openstack-lbaas01:02
*** bzhao_ has quit IRC01:09
*** ducttape_ has joined #openstack-lbaas01:10
*** yuanying has joined #openstack-lbaas01:10
*** fnaval has joined #openstack-lbaas01:11
*** gongysh has quit IRC01:16
*** reedip has quit IRC01:20
*** fnaval_ has joined #openstack-lbaas01:30
*** fnaval has quit IRC01:34
*** fnaval has joined #openstack-lbaas01:58
*** fnaval_ has quit IRC02:02
*** ducttape_ has quit IRC02:04
*** reedip has joined #openstack-lbaas02:24
*** cody-somerville has quit IRC02:43
*** csomerville has joined #openstack-lbaas02:43
*** fnaval_ has joined #openstack-lbaas02:49
*** fnaval has quit IRC02:51
*** ducttape_ has joined #openstack-lbaas02:53
*** yuanying has quit IRC03:11
*** yuanying has joined #openstack-lbaas03:14
*** sanfern has joined #openstack-lbaas03:17
*** ducttape_ has quit IRC03:20
*** amotoki has quit IRC03:22
*** reedip has quit IRC03:25
*** reedip has joined #openstack-lbaas03:28
*** ducttape_ has joined #openstack-lbaas03:31
*** armax has joined #openstack-lbaas03:33
*** ducttape_ has quit IRC03:34
*** armax has quit IRC03:36
*** _ducttape_ has joined #openstack-lbaas03:39
*** ducnc has quit IRC03:44
*** _ducttape_ has quit IRC03:48
*** amotoki has joined #openstack-lbaas03:50
*** ducttape_ has joined #openstack-lbaas03:51
*** amotoki has quit IRC03:56
*** amotoki has joined #openstack-lbaas04:00
*** yamamoto has joined #openstack-lbaas04:01
*** ducttape_ has quit IRC04:05
*** reedip has quit IRC04:17
*** ducttape_ has joined #openstack-lbaas04:19
*** fnaval has joined #openstack-lbaas04:21
*** amotoki has quit IRC04:22
*** fnaval_ has quit IRC04:24
*** armax has joined #openstack-lbaas04:26
*** reedip has joined #openstack-lbaas04:26
*** ducttape_ has quit IRC04:27
*** ducttape_ has joined #openstack-lbaas04:28
*** fnaval_ has joined #openstack-lbaas04:33
*** ducttape_ has quit IRC04:33
*** fnaval has quit IRC04:35
*** yamamoto has quit IRC04:42
*** yamamoto has joined #openstack-lbaas04:44
*** yamamoto has quit IRC05:09
*** reedip has quit IRC05:14
*** gcheresh_ has joined #openstack-lbaas05:17
*** yamamoto has joined #openstack-lbaas05:28
*** ducttape_ has joined #openstack-lbaas05:29
*** gcheresh_ has quit IRC05:32
*** ducttape_ has quit IRC05:35
*** fnaval_ has quit IRC05:35
*** sticker has quit IRC05:41
*** yamamoto has quit IRC05:41
*** rcernin has joined #openstack-lbaas05:44
*** amotoki has joined #openstack-lbaas06:05
*** amotoki_ has joined #openstack-lbaas06:08
*** amotoki has quit IRC06:10
*** voelzmo has joined #openstack-lbaas06:19
*** voelzmo has quit IRC06:23
*** voelzmo has joined #openstack-lbaas06:29
*** yamamoto has joined #openstack-lbaas06:42
voelzmo@rm_work thanks for the clarification06:46
voelzmo@xgerman I filed bug for PROXY protocol support as RfE06:46
openstackLaunchpad bug 1677987 in octavia "[RfE] Support PROXY protocol Header for TCP/SSL" [Undecided,New]06:46
*** yamamoto has quit IRC06:49
*** armax has quit IRC06:56
*** ducttape_ has joined #openstack-lbaas07:00
*** ducttape_ has quit IRC07:05
*** tesseract has joined #openstack-lbaas07:08
*** Guest96478 has joined #openstack-lbaas07:11
*** aojea has joined #openstack-lbaas07:12
*** Guest96478 has quit IRC07:16
*** reedip has joined #openstack-lbaas07:23
*** Guest27815 has joined #openstack-lbaas07:27
*** pcaruana has joined #openstack-lbaas07:27
*** Guest27815 has quit IRC07:38
-openstackstatus- NOTICE: Jobs in gate are failing with POST_FAILURE. Infra roots are investigating07:43
*** ChanServ changes topic to "Jobs in gate are failing with POST_FAILURE. Infra roots are investigating"07:43
*** rcernin has quit IRC07:54
*** tesseract has quit IRC07:54
*** rcernin has joined #openstack-lbaas07:55
*** tesseract has joined #openstack-lbaas07:55
*** gcheresh_ has joined #openstack-lbaas08:01
*** aojea_ has joined #openstack-lbaas08:17
*** aojea has quit IRC08:20
-openstackstatus- NOTICE: has corrupted disks, it's being repaired. Please avoid rechecking until this is fixed08:23
*** ChanServ changes topic to " has corrupted disks, it's being repaired. Please avoid rechecking until this is fixed"08:23
*** ipsecguy has quit IRC08:51
*** ipsecguy has joined #openstack-lbaas08:51
*** fnaval has joined #openstack-lbaas09:48
*** fnaval has quit IRC09:53
*** sputnik13 has joined #openstack-lbaas10:02
*** bbbzhao has quit IRC10:03
*** bbbzhao has joined #openstack-lbaas10:03
*** amotoki_ has quit IRC10:15
*** sanfern has quit IRC10:34
*** amotoki has joined #openstack-lbaas11:32
rm_workok, the member tests were in a little worse shape than I thought T_T11:41
*** fnaval has joined #openstack-lbaas11:49
*** fnaval has quit IRC11:53
*** ducttape_ has joined #openstack-lbaas12:17
*** ducttape_ has quit IRC12:30
*** catintheroof has joined #openstack-lbaas12:36
*** chlong has joined #openstack-lbaas12:40
*** HW-Peter has joined #openstack-lbaas12:55
*** HW-Peter has quit IRC12:56
*** HW-Peter has joined #openstack-lbaas12:56
*** aojea_ has quit IRC13:05
*** ducttape_ has joined #openstack-lbaas13:30
*** mariusv has quit IRC13:34
*** ducttape_ has quit IRC13:35
openstackgerritGerman Eichberger proposed openstack/octavia master: Adds a new config for soft-anti-affinity
*** ducttape_ has joined #openstack-lbaas13:38
*** gcheresh_ has quit IRC13:42
rm_workoh man, such tedium13:49
*** amotoki has quit IRC13:59
rm_workuhhh and i'm fixing some previous API bugs on the members patch, as I discovered them when making members do things ... if you want them split out, feel free to do it yourself ;)14:00
xgermanI am cool with that; are you using the Cllse-Bug tag?14:02
rm_worknot sure i filed bugs for them14:03
rm_worki just ... discovered them and fixed them14:03
rm_workalready just spent two hours fixing functional tests to not be dumb14:03
rm_workDRY refactoring and such14:03
rm_worki will stab my eyes out with a rusty spoon before I go back in and split those all out or try to make bugs for everything T_T14:04
rm_work 10 files changed, 823 insertions(+), 744 deletions(-)14:05
rm_work^^ 99% test code14:05
rm_workthat's *uncommitted*14:05
openstackgerritGerman Eichberger proposed openstack/octavia master: Failover logic doesn't handle anti-affinity
openstackgerritAdam Harwell proposed openstack/octavia master: Introduce Octavia v2 API for Members
rm_workmembers SHOULD BE ready for review now, for real14:08
openstackgerritGerman Eichberger proposed openstack/octavia master: Adds the ability to failover a load balancer
rm_workjohnsom: WTB re-review, not sure what you want exactly with the repositories locking method change14:17
johnsomOk, not in the office yet, but will look.  If we are leaving the pool status change, we at least need to update the members flows to set it back active14:20
*** yamamoto has joined #openstack-lbaas14:21
rm_workdoes it not?14:22
* rm_work checks14:22
rm_workjohnsom: in taskflow can you have an optional provides?14:26
* rm_work looks14:26
xgermanof all people you should know ;-)14:26
rm_worki hated taskflow and avoided it at all costs until recently ;P14:27
rm_workbut i figured that'd be bad form if I'm sharing an office with harlowja14:27
rm_workI blame blogan's influence14:27
johnsomYou can have a provides that doesn't get used, but these should be simple tasks right?14:27
rm_workyeah i just want to add "pool" to MarkLBAndListenersActiveInDB14:30
rm_workand do something with it if it's there, otherwise ignore14:30
rm_workis that not how taskflow works? >_>14:31
rm_workdo I need to duplicate that whole thing and make another one that also takes a pool?14:31
rm_workseems WET to me14:31
reedipkonichiwa ...14:31
rm_workおはようございます ^_^14:32
reediprm_work : stop flaunting !14:32
* rm_work gets this morning's UTF-8 check out of the way14:32
rm_workraise your hand if you saw boxes :P14:32
reedipI saw "Hello , Good Morning :14:33
reedipI saw "Hello , Good Morning "14:33
rm_workyeah it mostly just gets windows folks14:33
reedipI am Ubuntu :D14:33
reedipyou know the meaning of Ubuntu , right ?14:34
rm_workI did at one point14:40
rm_worksomething like ... people14:40
rm_workah, humanity14:41
*** ducttape_ has quit IRC14:44
*** ducttape_ has joined #openstack-lbaas14:44
*** ducttape_ has quit IRC14:45
*** ducttape_ has joined #openstack-lbaas14:45
*** ducttape_ has quit IRC14:46
reedipI am what I am because we all are :D14:46
*** ducttape_ has joined #openstack-lbaas14:46
*** ducttape_ has quit IRC14:47
rm_workyep, great sentiment :)14:56
johnsomrm_work It's an interesting line with the Taskflow model between making tasks individual code blocks with single tasks that are reused across flows and consolidating methods14:56
rm_workwell it's fine14:56
rm_worki just added another flow14:57
johnsomflow?  You mean task right?14:57
*** voelzmo has quit IRC14:57
rm_worksince there is already a MarkPoolActiveInDB14:57
rm_worki mean task14:57
rm_workadded that additional existing task to the flow14:57
johnsomAh, yeah, cool14:57
johnsomDo we need it on update flow as well?14:57
*** ducttape_ has joined #openstack-lbaas14:58
rm_worki got it14:59
rm_workalso delete15:00
rm_workrunning last tests15:01
rm_workjohnsom: hope that's right15:07
openstackgerritAdam Harwell proposed openstack/octavia master: Introduce Octavia v2 API for Members
openstackgerritAnkur proposed openstack/python-octaviaclient master: Initialize plugin for OSC
*** rcernin has quit IRC15:08
reediphi ankur-gupta-f4 , sorry couldnt verify this patch earlier15:10
reediplooking into it15:10
rm_workhopefully I don't cause anyone too much rebase trouble with that patch <_<15:10
ankur-gupta-f4np. Im nearing the end somehow. Need to figure out why how test discovery is working. Locally its not running the tests :|15:12
reedipok lemme take that up : this is the one right ?
reedipOops you abandonded it15:13
reedipwait lemme download an run the patch15:15
reedipIs the OSC CLI running?15:16
*** fnaval has joined #openstack-lbaas15:18
*** ducttape_ has quit IRC15:19
*** yamamoto has quit IRC15:21
*** armax has joined #openstack-lbaas15:22
*** krypto has joined #openstack-lbaas15:22
ankur-gupta-f4reedip: its at the point where you can list loadbalancers with OSC15:24
ankur-gupta-f4with the 'openstack loadbalancer list' command15:24
reedipso where is it failing15:24
ankur-gupta-f4The tests15:24
reedipok.. I have created a vitualenv, let me take a look at the tests15:25
*** yamamoto has joined #openstack-lbaas15:25
ankur-gupta-f4it needs more tests, but the issue right now is that I couldn't figure how to get tox to pick up the one unit/api/test i had15:25
reedipOk, can see the Loadbalancers in OSC15:25
reedipokay ?15:25
rm_workoh god the April 1 stuff is starting here15:36
reediptest now :D15:37
openstackgerritReedip proposed openstack/python-octaviaclient master: Initialize plugin for OSC
reedipokay , sorry15:42
reedipsomething weird happening15:42
reedipchuck that15:42
*** pcaruana has quit IRC15:46
reedipyeah, sorry15:46
ankur-gupta-f4just pulled it down15:46
reedipchecking again15:46
ankur-gupta-f4playing with it now15:46
reedipgo play :D15:47
reediptests in this patch work smoothly15:47
*** yamamoto has quit IRC15:47
*** yamamoto has joined #openstack-lbaas15:48
ankur-gupta-f4patchset 7?15:49
*** armax has quit IRC15:49
*** armax has joined #openstack-lbaas15:50
reedipI am just not in shape :P15:50
*** yamamoto has quit IRC15:50
*** yamamoto has joined #openstack-lbaas15:50
*** yamamoto has quit IRC15:50
rm_worksindhu: Can you rebase HMs on top of members?15:55
rm_worksindhu: it looks like it is on master now, right?15:55
rm_workah I guess I can do it15:56
rm_workif you don't have any work in progress15:56
rm_workbut it's not a super hurry15:56
sindhurm_work: actually, HM needs some work15:56
rm_workhmm k15:56
rm_workare you doing it?15:56
sindhurm_work: I m working on it today15:56
*** belharar has joined #openstack-lbaas15:56
rm_workwhat about L715:56
rm_workis someone actively working on that?15:56
rm_workthose are the remaining ones15:57
sindhunakul_d: is working on that15:57
openstackgerritGerman Eichberger proposed openstack/neutron-lbaas master: Octavia Proxy Plugin
xgerman^^ with the latest list lbs works15:57
rm_worknakul_d: how are the L7 patches looking?15:57
*** armax has quit IRC15:57
rm_workyeah, wanted to hear from the author15:58
*** armax has joined #openstack-lbaas15:58
rm_worknakul_d: if you are actively working on the L7 patches, awesome. if not, let me know, and I will start getting them up to date15:58
rm_worklooks like they're in merge conflict presently, and will be in worse shape once members lands, probably15:59
rm_workah, not too bad15:59
*** armax has quit IRC15:59
rm_workonly touches base and test_pool15:59
nakul_drm_work: I didn't take a look for some time16:02
nakul_drm_work: Feel free to get them up to date if you have time :)16:02
rm_workxgerman: commented on your anti-affinity patch16:03
rm_worknakul_d: ok, I can do that16:03
rm_workxgerman: err, the soft-anti-affinity one16:04
*** ducttape_ has joined #openstack-lbaas16:05
*** ducttape_ has quit IRC16:05
*** belharar has quit IRC16:09
xgermanyeah, I needed the soft-anti-affinity to test the failover one with my limited hardware16:10
openstackgerritGerman Eichberger proposed openstack/octavia master: Adds a new config for soft-anti-affinity
*** ducttape_ has joined #openstack-lbaas16:12
rm_workxgerman: whoops, misread your delete as an add16:13
rm_workon the constant16:13
rm_workLGTM now anyway, thanks for fixing the nit :P16:13
rm_workoh though I was asking if you could add the line to etc/octavia.conf16:13
rm_workthat was in my top-level comment, heh16:14
xgermanok, shouldn’t take long16:14
openstackgerritGerman Eichberger proposed openstack/octavia master: Adds a new config for soft-anti-affinity
xgermanhere we go16:17
xgermanknowing me I probably have a typo16:18
rm_workoh, was there not even an anti-affinity section to begin with?!16:20
*** voelzmo has joined #openstack-lbaas16:20
*** belharar has joined #openstack-lbaas16:21
rm_worklol yeah one more nit incoming T_T16:21
reedipankur-gupta-f4: does this seem relatd ?16:25
reedip  /usr/local/lib/python2.7/dist-packages/nose/plugins/ RuntimeWarning: Unable to load plugin openstack.nose_plugin = openstack.nose_plugin:Openstack: No module named nose_plugin16:25
*** voelzmo has quit IRC16:25
ankur-gupta-f4where are you seeing that error16:26
openstackgerritGerman Eichberger proposed openstack/octavia master: Adds a new config for soft-anti-affinity
*** voelzmo has joined #openstack-lbaas16:26
reediptestr list-tests16:26
reedipcant find it anywhere..16:27
ankur-gupta-f4ran it. don't see an error16:27
reedipthats weird16:27
reedipdoes it list any tests?16:27
reedipok, this is showing me error in octaviaclient16:28
reedipbut not in openstackclient16:28
reedipit listed all the tests in openstackclint16:28
ankur-gupta-f4I added this to my octaviaclient .testr.conf "${PYTHON:-python} -m discover -t ./ ${OS_TEST_PATH:-./octaviaclient/tests/unit} $LISTOPT $IDOPTION"16:28
reediphow does this help?16:29
rm_workxgerman: please don't kill me, I'm debating whether that comment would be better as just "# Anti-affinity configuration" lol16:29
xgermanat one point we need to call in johnsom to referee16:30
xgermanor blogan since he is now impartial16:31
*** voelzmo has quit IRC16:31
ankur-gupta-f4reedip: it doesn't16:31
johnsomYou are calling me not impartial?  Adam wins....16:31
johnsomI'm still in meetings, so haven't had a chance to look16:32
rm_workxgerman: yes I vote for johnsom telling us what it should say :P16:32
reedipankur-gupta-f4 :hahahaha16:32
reedipgood joke :)16:32
ankur-gupta-f4pulling at straws now16:32
reedipbut I think it is telling something16:33
ankur-gupta-f4tox russian roulette16:33
reedipdid u look at my patchset for TaaS?16:33
reedipthe only difference is the client which I am using there .probably16:33
reedipand the fakes, but the fakes are irrelevant16:33
ankur-gupta-f4its something related to how it discovers tests16:33
ankur-gupta-f4or runs tests16:33
reedipwhy dont u have openstackclient as a requirement ???????16:36
*** ducttape_ has quit IRC16:36
reedipfor Octaviaclient :D16:36
*** ducttape_ has joined #openstack-lbaas16:39
johnsomOk, xgerman rm_work are there comments on the patch or something else we are "discussing"?16:41
rm_workwe're discussing ...  a comment16:41
rm_workin the config file16:41
rm_workthe wording :P16:42
rm_work"# Enable anti-affinity in nova" or maybe actually just "# Anti-affinity settings" or something like that16:42
johnsomYeah, ok.  I would drop "nova" as that ties us to a compute driver.  I'm sure there will be a new shiny ball at some point in the future that might support anti-affinity16:43
xgermanjust wrap it in a comment and I will fix it16:43
* xgerman thought os-ansible was a tough crowd16:44
johnsomDifference here is we turn our reviews around...  grin16:46
*** ducttape_ has quit IRC16:50
johnsomHmmm, ok, I see what you did here.  You are doing a per-driver setting.  Hmmm.16:50
*** yamamoto has joined #openstack-lbaas16:51
*** ducttape_ has joined #openstack-lbaas16:53
harlowjarm_work whattttt16:55
johnsomharlowja How is it going?16:56
harlowjait goes16:56
johnsomHope you enjoyed the family skiing...16:56
harlowjatrying to automate all the things16:56
johnsomAh the robot overlords eh?16:57
harlowjawe call him 'padre' currently16:57
*** yamamoto has quit IRC16:57
xgermanharlowja: how do we know it's you and not some bot?16:58
johnsomOk, I had two other issues with that patch, but probably ended up siding with german16:59
johnsomYeah, I guess you could automate the "lol"s16:59
xgermanI think they give it a away16:59
harlowjarandomLol() func17:00
johnsomIf it starts asking us "How do you feel about that?" we will know.....17:00
xgermanI think they made some progress since Eliza17:01
johnsomHahaha, someone got the old timer reference....17:01
rm_workCleverBot is still just as CleverBot as ever...17:01
xgermanAfk and then I will fix17:02
*** ducttape_ has quit IRC17:03
harlowjaso ya, i'm wrangling rm_work to get involved in padre17:03
harlowjato make it deploy octavia17:03
rm_workI'm wriggling away17:03
rm_workbut if you notice, I did do a test-build command earlier :P17:04
johnsomharlowja No, he has to keep working on Octavia17:04
rm_workyes they have been keeping me dreadfully busy here >_>17:04
johnsomharlowja We create OSA for octavia, what more do you need?  Grin17:04
harlowjathats rackspace ansible blah blah17:05
harlowjanot padre17:05
harlowjapadre is self-aware17:05
harlowjasomeone must teach padre17:05
johnsomOk, ok, so I also hear tripleo will have support soon, that is more GoDaddy style right?17:05
* harlowja not sure who uses tripleo17:06
johnsomIt's a RedHat thing right?17:06
harlowjaya, but we just use centos17:06
*** krypto has quit IRC17:06
harlowjanot all of redhat blah blah17:07
johnsomI hear the latest thing is just ask Alexa to create a stack....17:07
harlowjapadre support for alexa17:07
johnsomMaybe padre and Alexa should talk....17:07
* harlowja adds note to self to do that17:07
johnsomI get an invite to the wedding....17:07
reedipankur-gupta-f4 ; any luck?17:08
harlowja johnsom17:08
harlowjait is triggering jenkins there17:08
harlowjait also is plugged into the whole of openstack review firehose17:09
harlowjaand various other things, lol17:09
harlowjait is learning at an exponential rate17:09
johnsomOh geez, that would be chatty annoying.  Where is the mute button????  Grin17:09
harlowjaits in a subthread17:09
harlowjasomething irc doesn't have :-P17:09
harlowjabut slack does17:09
*** armax has joined #openstack-lbaas17:10
johnsomYeah, we had all kinds of cool bots/hacks for HipChat at HP....17:10
harlowjanone as great as padre17:10
johnsomrm_work had a IRC bot at RackSpace17:10
rm_workand I think Slack bots are awesome17:11
rm_workI just don't have time right now or I would be ALLLLL over it17:11
harlowjamy goal is to replace all operational procedures with it17:11
harlowja*long term goal17:11
johnsomI will be impressed when it says, "Shoot partner, QE just used up all of the Jenkins slaves.  Go bake some beans, we're going to be here a while".17:11
harlowjawill add that17:12
harlowjawhat's QE17:12
harlowjapadre is the QE17:12
*** krypto has joined #openstack-lbaas17:13
*** armax has quit IRC17:16
*** krypto has quit IRC17:23
*** krypto has joined #openstack-lbaas17:23
ankur-gupta-f4reedip: not yet17:24
reediplol ... I am also tired now17:24
reedipwould give it half an hour more befire I call quits and move to Legends Of Tomorrow17:25
openstackgerritAnkur proposed openstack/python-octaviaclient master: Initialize plugin for OSC
reedipankur-gupta-f4 : did it work /17:33
reedipthe OSC17:33
ankur-gupta-f4no. pushed up a WIP just now so i can get help from others.17:33
reedipput a WF -1 then, else people might review :P17:34
reedipI am downloading ur code now17:34
*** gcheresh_ has joined #openstack-lbaas17:40
reedipankur-gupta-f4 : now there are no test cases :)17:44
ankur-gupta-f4have to match the right test to the right file and structure17:44
ankur-gupta-f4note: nosetests discovers test. tox isn't right now17:45
reedipwhy api ?17:46
reedipwhy not osc?17:46
*** krypto has quit IRC17:47
ankur-gupta-f4Two different things being tested17:49
ankur-gupta-f4One is the actual test of hitting api17:49
ankur-gupta-f4Other is test for whether it returns the list17:50
*** armax has joined #openstack-lbaas17:51
reedipankur-gupta-f4:maybe also add os-testr in the test-requirements.17:55
rm_workjohnsom: L7Policies have ... project_ids, right?17:55
reedipI am leaving, see u in a few hours :)17:55
*** reedip is now known as reedip_afk17:56
johnsomrm_work It should now inherit from LB, but on the types...   Yes, tenant_id and project_id are present17:56
rm_workI didn't mean the types17:57
rm_workeh i'll figure this out17:57
johnsomIn the controller it should be the same as say pools17:57
johnsomIt will just inherit from the LB17:57
johnsomThough it looks like the DB doesn't have a column for that...17:59
rm_worki'm running into issues at the model creation layer17:59
rm_workfiguring it out18:00
rm_workgo do something else for now :P18:00
johnsomGrin, trying to help ankur with this odd testr problem....18:00
rm_workhmm this is kindof a mess IMO18:02
rm_workdb_prepare.create_l7policy() runs validate.sanitize_l7policy_api_args() first thing, does stuff, then is immediately passed to self.repositories.l7policy.create() which runs validate.sanitize_l7policy_api_args()18:03
*** gcheresh_ has quit IRC18:10
*** aojea has joined #openstack-lbaas18:10
rm_workjohnsom: L7Policies should also have created/updated timestamps, right?18:18
johnsomYes, I think so18:18
*** tesseract has quit IRC18:21
openstackgerritGerman Eichberger proposed openstack/octavia master: Adds a new config for soft-anti-affinity
rm_workuhhh EVERYTHING has both statuses right?18:34
johnsomEvery top level object should have both, yes18:38
*** aojea has quit IRC18:40
*** csomerville has quit IRC19:05
*** HW-Peter has quit IRC19:06
*** ducttape_ has joined #openstack-lbaas19:22
*** aojea has joined #openstack-lbaas19:31
*** ducttape_ has quit IRC19:31
rm_workwhelp didn't mean to work this long today <_<19:40
rm_workwhelp i failed to actually USE that fancy context_manager correctly19:55
rm_workit's ok though, fixing it19:55
-openstackstatus- NOTICE: will be offline from 20:00 to 23:00 UTC for planned upgrade maintenance19:58
*** gcheresh_ has joined #openstack-lbaas20:04
rm_workugh l7 ALSO suffers from this random mock bug20:05
rm_workso it's just pools and l7handlers20:05
*** ducttape_ has joined #openstack-lbaas20:11
rm_workjohnsom: what do you think of members?20:32
johnsomI will start looking again now20:34
*** csomerville has joined #openstack-lbaas20:48
rm_workOK FFFF20:59
rm_workAbout to upload this20:59
rm_workwould be nice to merge members :P20:59
johnsomLooking at it20:59
*** gcheresh_ has quit IRC21:05
openstackgerritAdam Harwell proposed openstack/octavia master: Align Octavia API to n-lbaasv2 for L7Policy
rm_worki'm gonna go fall over now21:09
johnsomHave a good one.  I WILL finish my testing on members today21:10
rm_work... L7Rules might be this weekend'21:11
rm_workfollowed by a refresh of Paging/Sorting21:11
johnsomShould I look at L7 policy too?21:11
rm_workyeah if you have time21:11
rm_workit's in a lot better shape21:11
rm_workI'd be amazed if I got even close to everything the first pass through though21:12
rm_workbut definitely got a lot21:12
rm_workgot everything *I* saw21:12
rm_workit has a migration for the columns on L7Policy :/21:12
rm_workwas missing ... most of them21:13
rm_workoh yeah, wanted your opinion on that21:15
rm_workhave to non-null a column21:15
rm_workchoosing what to set everything to21:15
rm_workwell, you'll see it21:16
rm_workI commented in the migration21:16
*** aojea has quit IRC21:21
johnsomStill there?21:22
johnsomHelp me with the lock session decorator21:23
johnsomIn, under the with, it is still using context.session, shouldn't it use lock_session or something like that?21:23
rm_workjohnsom: lol21:23
rm_worki was hoping you wouldn't catch that21:23
rm_worki fixed it in the L7Policy patch21:24
rm_worki mentioned it in channel earlier21:24
rm_workthe same exists for pools21:24
rm_workalso fixed in the L7Policy patch21:25
johnsomSorry I missed it on pools....21:25
rm_workme too :P21:25
rm_workI'm sorry *I* missed it21:25
rm_workI wrote the thing21:25
rm_workbut i've pretty much been coding myself to exhaustion every day, so21:26
johnsomOk, It's ugly to fix it this way, but not enough I will -1 it21:26
rm_workit's not that surprising <_<21:26
johnsomYeah, take a break man!21:26
rm_workwell, my schedule is ... fun21:26
johnsomBTW, I will be in Portland tomorrow, but might be around off and on the day after21:27
rm_workI mean, tomorrow IS Saturday21:27
rm_workso you're allowed :P21:27
rm_worklooks like everything is still broken in infra though anyway21:39
rm_workso no merging today :/21:40
rm_workoh, i think maybe they're just wrapping up the fix21:40
johnsomYeah, I was going to poke you on those gate fixes, but no point at the moment21:41
rm_workwhich ones?21:42
johnsomYou can't look at the logs though, so...21:44
-openstackstatus- NOTICE: The upgrade maintenance for has been completed and it is back online.21:50
johnsomFYI, that does not me the logs are back yet...21:51
rm_work^^ we really need "all"?22:09
rm_workmaybe that doesn't mean what I think it means?22:09
johnsomWell, we have been running with "all" until now.22:09
rm_worklol. ok then22:09
rm_workI wonder what happens if I +2+A22:09
rm_workprolly... breaks?22:09
johnsomIt's that, or a list of stuff we have to babysit22:10
rm_worklet's find out22:10
rm_workanyone can always recheck it later22:10
rm_workon I STILL wanted to see at least one pass of octavia scenarios...22:10
rm_workit failed AGAIN on my recheck but i can't look at the logs to see why22:11
johnsomYeah, ok22:11
rm_workjust seems a little fishy that ONLY those failed, twice in a row22:12
rm_workjohnsom: if you do end up -1 on members and i need a new patchset anyway, i'll port back the changes for the lock_session thing, it isn't too bad22:14
johnsomOk, np22:14
johnsomI put a comment in for other reviews about our discussion.22:14
rm_workyou almost done with that patch?22:15
rm_workif so i'll stay up22:15
johnsomThough that review just goes against my diversity number...  Uh22:15
johnsomNo, it will be a while.  Get some sleep22:16
rm_workit is OK I'll -1 it too22:16
rm_workI have been22:16
*** fnaval has quit IRC22:29
*** catintheroof has quit IRC22:36
*** ducttape_ has quit IRC22:42
*** ducttape_ has joined #openstack-lbaas22:43
*** ducttape_ has quit IRC22:43
*** ducttape_ has joined #openstack-lbaas22:47
openstackgerritSindhu Devale proposed openstack/octavia master: Introduce Octavia v2 API for Health Monitor
sindhurm_work: johnsom : health monitor endpoints are working :)22:50
johnsomCool!  Thank you22:50
sindhurm_work: johnsom haven't worked on the tests yet22:50
sindhurm_work: if it is a priority, please feel free to work on it22:51
*** ducttape_ has quit IRC22:51
sindhurm_work: else, i'll do it on Monday :)22:51
sindhuHappy weekend y'all :)22:51
* johnsom waves22:52
johnsomOk, there is a bug in members, it comes up in OFFLINE, I think it should be NO_MONITOR OR OFFLINE depending on the HM status of the pool.  I'm going to open a bug for that22:56
johnsomI added that feature to both octavia and nlbaas so I knew to check for it.23:00
*** fnaval has joined #openstack-lbaas23:06
johnsommembers passes my tests23:15
*** ducttape_ has joined #openstack-lbaas23:26
*** ducttape_ has quit IRC23:28
*** ducttape_ has joined #openstack-lbaas23:28
*** ducttape_ has quit IRC23:29
*** ducttape_ has joined #openstack-lbaas23:30
*** fnaval_ has joined #openstack-lbaas23:43
*** fnaval has quit IRC23:46

Generated by 2.14.0 by Marius Gedminas - find it at!