Monday, 2017-04-03

« Sunday, 2017-04-02 Index Tuesday, 2017-04-04 »
*** reedip has joined #openstack-lbaas00:25
*** aojea has joined #openstack-lbaas00:30
*** aojea has quit IRC00:35
*** sanfern has quit IRC01:38
*** reedip has quit IRC01:40
*** rcernin has joined #openstack-lbaas01:58
*** reedip has joined #openstack-lbaas01:59
reedipo/02:04
*** yamamoto has joined #openstack-lbaas02:29
*** amotoki has joined #openstack-lbaas02:39
*** rcernin has quit IRC02:45
*** cody-somerville has joined #openstack-lbaas02:46
*** csomerville has quit IRC02:49
*** sanfern has joined #openstack-lbaas02:53
*** catintheroof has joined #openstack-lbaas02:55
*** ducttape_ has joined #openstack-lbaas02:57
*** ducttape_ has quit IRC02:58
*** ducttape_ has joined #openstack-lbaas02:58
*** amotoki has quit IRC03:00
*** ducttape_ has quit IRC03:17
*** amotoki has joined #openstack-lbaas03:20
*** fnaval has joined #openstack-lbaas03:20
*** ducttape_ has joined #openstack-lbaas03:22
*** fnaval_ has quit IRC03:24
*** ducttape_ has quit IRC03:24
*** ducttape_ has joined #openstack-lbaas03:24
*** amotoki has quit IRC03:29
*** yamamoto has quit IRC03:37
*** amotoki has joined #openstack-lbaas03:38
*** ducttape_ has quit IRC03:41
*** ducttape_ has joined #openstack-lbaas03:48
*** reedip has quit IRC03:52
*** ducttape_ has quit IRC03:55
*** reedip has joined #openstack-lbaas04:00
*** amotoki has quit IRC04:01
*** amotoki has joined #openstack-lbaas04:08
*** catintheroof has quit IRC04:10
*** yamamoto has joined #openstack-lbaas04:16
*** ducttape_ has joined #openstack-lbaas04:48
*** cody-somerville has quit IRC04:51
*** ducttape_ has quit IRC04:53
*** rcernin has joined #openstack-lbaas04:56
*** fnaval has quit IRC04:58
*** rcernin has quit IRC05:18
*** rcernin has joined #openstack-lbaas05:31
*** gcheresh_ has joined #openstack-lbaas05:53
*** oomichi has quit IRC06:03
*** oomichi has joined #openstack-lbaas06:04
*** IRCFrEAK has joined #openstack-lbaas06:05
*** IRCFrEAK has left #openstack-lbaas06:06
*** yuanying has quit IRC06:07
*** yuanying_ has joined #openstack-lbaas06:13
*** kobis has joined #openstack-lbaas06:14
*** ducttape_ has joined #openstack-lbaas06:18
*** ducttape_ has quit IRC06:23
*** oomichi has quit IRC06:29
*** oomichi has joined #openstack-lbaas06:30
*** voelzmo has joined #openstack-lbaas06:31
*** sputnik13 has quit IRC06:32
*** sputnik13 has joined #openstack-lbaas06:34
*** tesseract has joined #openstack-lbaas06:34
*** bzhao has joined #openstack-lbaas06:35
*** bbzhao has quit IRC06:38
*** voelzmo has quit IRC06:38
*** voelzmo has joined #openstack-lbaas06:42
*** pcaruana has joined #openstack-lbaas06:57
*** fnaval has joined #openstack-lbaas06:59
*** fnaval has quit IRC07:03
*** aojea has joined #openstack-lbaas07:21
*** amotoki_ has joined #openstack-lbaas07:41
*** amotoki has quit IRC07:44
*** ducttape_ has joined #openstack-lbaas07:49
*** ducttape_ has quit IRC07:54
*** openstackgerrit has quit IRC08:03
*** belharar has joined #openstack-lbaas08:15
*** belharar has quit IRC08:16
*** belharar has joined #openstack-lbaas08:17
*** aojea_ has joined #openstack-lbaas08:21
*** aojea has quit IRC08:24
*** reedip has quit IRC08:28
*** sanfern has quit IRC08:32
*** mjblack has quit IRC08:33
*** mjblack has joined #openstack-lbaas08:38
*** sanfern has joined #openstack-lbaas09:01
*** bcafarel has quit IRC09:06
*** reedip has joined #openstack-lbaas09:08
*** belharar has quit IRC09:14
*** ducttape_ has joined #openstack-lbaas09:20
*** ducttape_ has quit IRC09:24
*** amotoki_ has quit IRC09:25
*** bcafarel has joined #openstack-lbaas10:00
*** gcheresh has joined #openstack-lbaas10:02
*** gcheresh_ has quit IRC10:02
*** rcernin has quit IRC10:05
*** krypto has joined #openstack-lbaas10:13
*** krypto has joined #openstack-lbaas10:13
*** openstackgerrit has joined #openstack-lbaas10:13
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-lbaas master: Imported Translations from Zanata  https://review.openstack.org/44903710:13
*** rcernin has joined #openstack-lbaas10:24
*** ducttape_ has joined #openstack-lbaas10:50
*** sanfern has quit IRC10:54
*** ducttape_ has quit IRC10:55
*** fnaval has joined #openstack-lbaas11:01
*** fnaval has quit IRC11:06
*** gcheresh_ has joined #openstack-lbaas11:24
*** gcheresh has quit IRC11:24
*** voelzmo has quit IRC11:52
*** voelzmo has joined #openstack-lbaas11:53
*** sanfern has joined #openstack-lbaas12:19
*** ducttape_ has joined #openstack-lbaas12:20
*** _ducttape_ has joined #openstack-lbaas12:24
*** ducttape_ has quit IRC12:28
sanfernHi All, I want to send my amphora haproxy logs to splunk/graylog, Do we have hooks to enable this?12:38
*** mjblack has quit IRC12:52
*** mjblack has joined #openstack-lbaas12:55
*** _ducttape_ has quit IRC13:02
*** voelzmo has quit IRC13:03
*** sanfern has quit IRC13:04
*** sanfern has joined #openstack-lbaas13:04
*** krypto has quit IRC13:24
xgermanno13:28
xgermanbut you are welcome to contribute those ;-)13:28
xgermanoverall, we sue oslo logging so whatever works with that should work with us13:29
*** catintheroof has joined #openstack-lbaas13:31
nmagnezixgerman, i think that specifically for haproxy logs we simply use https://github.com/openstack/octavia/blob/master/elements/haproxy-octavia-ubuntu/post-install.d/20-setup-haproxy-log , no?13:33
nmagnezixgerman, i might be wrong here.. just asking :)13:33
*** voelzmo has joined #openstack-lbaas13:33
xgermanwell, if you move all the haproxy logs into splunk, et. al. - first they are a ton and 2nd for multi tenant you need really good access controls13:37
xgermanI was thinking he wanted the octavia logs…13:38
*** ducttape_ has joined #openstack-lbaas13:38
*** ipsecguy has quit IRC13:40
*** ipsecguy has joined #openstack-lbaas13:45
*** ducttape_ has quit IRC13:51
reedip...o...13:54
*** voelzmo has quit IRC14:00
*** amotoki has joined #openstack-lbaas14:05
*** voelzmo has joined #openstack-lbaas14:08
*** voelzmo has quit IRC14:08
*** reedip has quit IRC14:13
*** voelzmo has joined #openstack-lbaas14:14
*** ducttape_ has joined #openstack-lbaas14:18
*** amotoki has quit IRC14:25
*** reedip has joined #openstack-lbaas14:26
openstackgerritMerged openstack/octavia master: Updated from global requirements  https://review.openstack.org/45247914:28
openstackgerritMerged openstack/octavia master: Move API v1, v2 hooks to API Common  https://review.openstack.org/44828814:29
*** ducttape_ has quit IRC14:30
sanfernxgerman, Thanks. I am looking for adding entry into "log {{spunk server ip : port }}   syslog info"  for haproxy.cfg file14:34
xgermanmmh, our default image does not support logging to a central server at all since we didn’t want to open up the network in that (+evertbodt logs differently)14:36
*** ducttape_ has joined #openstack-lbaas14:36
sanfernthis splunk or graylog server details define in octavia.conf14:37
xgermanoctavia conf takes oslo logging directives14:37
*** blogan_ is now known as blogan14:38
johnsomYou could add a section to octavi.conf that gets passed down into the amphora agent conf that specifies how to configure haproxy to log.  Since haproxy.cfg is generated you pretty much have to do that14:41
johnsomOr you could just add it to the jinja template to hard code it14:42
*** ipsecguy has quit IRC14:42
*** chlong has joined #openstack-lbaas14:45
xgermanwe probably should document that in some way if gaproxy logging is becoming popular14:46
sanfernjohnsom, to pass the value from octavia.conf to jinja requires change in octavia code14:47
*** ipsecguy has joined #openstack-lbaas14:47
xgermanyes14:48
johnsomCorrect14:48
xgermanbut you could make a patch that it bcomes configurable14:48
johnsomFiguring out logging is a research topic for Pike14:48
xgermannot sure if we want to log the haproxy logs… but *shrug*14:49
johnsomIf the target is fixed you can just edit the jinja, but if you need to change it, that is code14:52
sanfernserver can be changed to different depending on type of workload or cloud.14:55
*** gcheresh_ has quit IRC14:57
*** amotoki has joined #openstack-lbaas14:59
*** _ducttape_ has joined #openstack-lbaas15:00
*** voelzmo has quit IRC15:01
*** voelzmo has joined #openstack-lbaas15:02
*** voelzmo has quit IRC15:02
*** ducttape_ has quit IRC15:03
xgermanDNS?15:07
johnsomDNS is disabled in the amps15:21
*** _ducttape_ has quit IRC15:21
*** ducttape_ has joined #openstack-lbaas15:22
xgermanjoking :-) Though somebody was working on making DNS work…15:23
*** fnaval has joined #openstack-lbaas15:25
reedipwho said dns ?15:27
rm_workyeah it would be very easy to edit the jinja template that creates the haproxy.conf15:27
rm_workwe designed it to be very simple for an operator to customize those15:27
xgermanyeah, indeed15:28
xgermanor we push that etcd pie-in-the-sky OpenStack service15:29
*** tesseract has quit IRC15:31
diltramjohnsom: so we finally know that centos is still not working :P15:33
johnsomHa, is it the group setting I wonder?15:34
rm_workcentos is working for me15:34
rm_workI am using it in GD cloud15:34
diltramit's something with amphora-agent15:34
diltramrm_work: as amphora base os?15:34
rm_workyes15:35
diltramweird15:35
diltramas on our gates it's not working15:35
rm_workjust have to set the user_group thing15:35
rm_workuntil nmagnezi's patch merges15:35
diltramaaa, ok15:35
rm_workwhich we just spent many hours discussing yesterday :P15:35
xgermanwhat? On a Sunday>15:36
xgerman?15:36
diltramso I should make a patch which sets this direclty or we gonna wait for merge?15:36
xgermanyou have the change-id so we can speed up merging?15:36
johnsomLet's wait for the fix15:36
xgerman+115:37
*** kobis has quit IRC15:37
rm_workyeah I was working all weekend :/15:38
rm_workxgerman: well, per discussion, he's looking to rework it15:38
rm_workjohnsom: i don't know if you read any of that scrollback15:38
*** amotoki has quit IRC15:39
rm_workcan we merge https://review.openstack.org/#/c/405932/ today I hope?15:40
rm_workit should be good15:40
rm_workand https://review.openstack.org/#/c/406328/ is ready for review, as is https://review.openstack.org/#/c/406336/15:40
rm_workL7Policy / L7Rule15:41
xgermanok, I will look over them15:41
rm_workthere's some stuff in the migration that is a little ... mixed together15:41
rm_workbut i opted for fewer migrations15:41
rm_workbecause they are annoying to deal with15:41
rm_workjohnsom: i fixed all the stuff you mentioned, since there was enough material to justify a new patchset :/15:42
rm_workand since it was obvious we weren't merging on friday15:43
johnsomYeah, I see that.  Just looking it over now15:43
xgermanwhen are the filters showing up?15:43
rm_workxgerman: after we get all of this merged15:43
rm_workand then the pagination patch15:43
xgermanok15:43
rm_workwhich is also close15:43
rm_workbut i need to rebase it on L7Policy/Rule15:43
xgermanmy plugin-proxy doesn’t work well without filters15:43
rm_workthe faster we can get some of these merged, the faster I will get to filters :P15:44
johnsom+2 again on members from me15:44
*** aojea_ has quit IRC15:45
diltramrm_work: commented on the Nirs patch, I completely agree with your idea. It should be configured in dib on backing image stage15:45
diltramand it's all15:45
diltramwe can deprecate the config and not use it any more15:46
rm_workyes15:47
diltramjohnsom: spining up devstack with members15:47
johnsom+115:47
rm_worki am sad i didn't look at it a while back and comment, because i have been feeling that it MUST be way overcomplicated for a while15:48
rm_workit would be super neat if we could look at both L7s today too15:52
*** pcaruana has quit IRC15:53
johnsomYeah, I started on L7 policy Friday, good to know both are ready for a pass15:57
rm_workyeah i spent a lot of time this weekend getting those up to speed15:58
*** ipsecguy has quit IRC15:58
*** ipsecguy has joined #openstack-lbaas15:58
xgermansorry, rm_work16:00
rm_workno worries, i had nothing better to do16:01
johnsomHA, that global requirements patch...  The list of exclusions for DIB should say something about it's recent stability16:01
openstackgerritAnkur proposed openstack/python-octaviaclient master: Initialize plugin for OSC  https://review.openstack.org/44622316:01
xgermanrm_work found some nits on members16:01
rm_workwas stuck in an AirBnB with shitty internet so I couldn't play games or anything :P16:02
xgerman:-(16:02
rm_workjohnsom: heh yeah i was looking at that and how it'll work with the cherry-pick for dib in ocata16:02
*** cody-somerville has joined #openstack-lbaas16:02
rm_workjohnsom: wasn't going to touch it just yet16:02
rm_workxgerman: no worries i'll take a look16:02
johnsomI shouldn't touch it.  Don't want to skew the metrics16:02
xgermanlol16:03
xgermanI think contributors should weight more than people reviewing…16:03
johnsomSigh, yes.  One out of eight metrics and you drop the tag....16:05
reedipankur-gupta-f4 :any advances with the OSC?16:05
xgermanwell, TC elections are coming up - we can run on. a platform to chnage metrics ;-)16:06
johnsomHahahaha16:06
ankur-gupta-f4reedip: yea. It runs, the two tests pass. Feel free to review16:07
reedipD16:07
johnsomWell, if we mirror "other" politics, we would only change the metrics for nova/neutron/keystone and leave it for the other projects16:07
reedipso the test-requirements worked16:07
ankur-gupta-f4yea jenkins passing i think16:08
reedip:)16:08
reedipkk16:08
xgermanrm_work living out my nits; not sure how your system works but whoever tests L7 should try a position > MAX_INT and see if the error message makes sense16:16
xgermanof course I could pull the patch and look myself ;-)16:16
johnsomI tried the range validation on a previous patch and it was reasonable16:16
johnsomBut, yes, ^^^^ That....  Grin16:17
rm_workxgerman: the tests actually do this16:29
rm_workxgerman: and you can see i think just from looking at the test code16:29
rm_workit checks for the string16:29
xgermanthe tests I saw did put in like “test” when an integer value was asked16:30
xgermanso this doesn’t tell us if the integer is in the range…16:30
rm_workoh that's member16:31
rm_workyou mentioned L716:31
xgermanL7 s the same - w ehave a position16:31
xgermanand test gain “test”16:31
rm_workerr16:31
rm_workin L7 i definitely made a test that does > and <16:31
rm_worklet me find it16:32
xgermanhttps://review.openstack.org/#/c/406328/25/octavia/tests/unit/api/v2/types/test_l7policies.py - that ain’t it16:32
rm_worktest_l7policy_positions16:32
rm_workjust a little down16:32
xgermanah, true, I gave up after not seeing it in post16:33
rm_workI guess it should be in both16:33
xgermanyep16:33
rm_workso if i fix all of the things you guys commented on, xgerman / diltram16:33
rm_workthose are mergable? :P16:34
rm_workor are you going to make more comments on the next patchset? :P16:34
diltramrm_work: yes16:34
rm_workok16:34
xgermanI think so16:34
johnsomI have not tested L7 stuff yet16:34
xgermanmy stuff are nits16:34
xgermanok, besides actually testing it ;-)16:34
rm_workLOL so16:40
rm_workwhile both of you commented on needing to use a set()16:40
rm_workneither of you noticed that it is impossible for that condition to even be met <_<16:40
rm_workdon't even need a set I think16:40
rm_worklistener_ids = [l.id for l in pool.listeners]16:41
xgermanwell, then hat code should be removed16:41
rm_workyep16:41
rm_workdone16:41
rm_workgo ahead and +A it guys :P16:53
rm_worknot sure why the bot didn't report updates16:53
openstackgerritMerged openstack/octavia master: Adds a new config for soft-anti-affinity  https://review.openstack.org/45192416:57
rm_workah, umm17:03
rm_workjohnsom: do you know how/when in the API reponses, "children" is actually set to True?17:03
johnsomLike for sub paths? or for single call create type requests?17:04
rm_workhmmm17:04
rm_workI mean I want to test that L7PolicyReponse actually does child calculations properly17:05
rm_workbut ... it never has children set to True17:05
diltramrm_work: this feature doesn't work :P17:05
diltramit's buggy17:05
diltramand doesn't work really17:05
*** chlong has quit IRC17:05
johnsomAh, the from data model stuf17:05
rm_worklol17:06
diltramyeah17:06
rm_workI think it's literally never called with children=True17:06
rm_worklike, it isn't possible17:06
rm_workmaybe this was for some future thing?17:06
rm_workeven in v1 it isn't17:06
diltramI was fighting with it a few weeks ago on one of the patches17:06
diltramtrue17:06
rm_work...17:06
diltramwe need to rewrite this code17:06
rm_workok ...17:06
diltramand make this usable17:06
rm_workso should I just leave it for now?17:06
rm_workor rip it out17:06
rm_workand we can add it back later?17:07
diltramas it doesn't allow us to really use this rename mechanizm17:07
diltramyou need to leave it17:07
rm_workwhy? I mean,17:07
rm_workthe code patch is unreachable17:07
rm_work*code path17:07
diltramare you sure?17:07
rm_workyes17:08
rm_work(99%)17:08
rm_workI have searched for all usages17:08
rm_workthere is no way for it to be called with children=True17:08
rm_workunless I missed something17:08
rm_workBTW you owe me a +A on https://review.openstack.org/#/c/405932/17:08
rm_work:P17:08
diltramI would still probably leave and just mark with TODO as thing to be fixed17:08
rm_workk17:08
diltramdone17:09
diltramkeep it rolling :P17:09
rm_workL7Policy is ready for review17:10
rm_workGOGOGOGO17:10
rm_workdiltram / johnsom / xgerman ^^17:13
rm_workxgerman: already fixed your comment17:13
xgermanand I gave a +2 for members17:14
xgermanL7 I probably should install and test before I17:14
xgerman=217:14
rm_workyesplz17:15
xgermanok, I will pull that patch but my devstack is a fickle friend17:17
rm_workxgerman: i added the quota test (whoops, forgot to copy that one over) but I don't honestly know how to hit the duplicate-id thing17:18
rm_worksince we no longer allow post-by-id17:18
rm_workin v217:18
rm_workso maybe that is honestly dead code17:18
xgermank, it just looked suspicious in Sonar17:18
rm_worki mean, the quota test missing was a good catch17:19
xgermanwell, Sonar is smart ;-)17:19
*** chlong has joined #openstack-lbaas17:21
openstackgerritAdam Harwell proposed openstack/octavia master: Align Octavia API to n-lbaasv2 for L7Rules  https://review.openstack.org/40633617:23
*** armax has joined #openstack-lbaas17:25
xgermanok, vmware updates itself and can’t resume my devstack vm, restarting..17:27
*** ducttape_ has quit IRC17:27
openstackgerritMerged openstack/neutron-lbaas master: Updated from global requirements  https://review.openstack.org/45247717:36
openstackgerritMerged openstack/neutron-lbaas master: Imported Translations from Zanata  https://review.openstack.org/44903717:36
openstackgerritMerged openstack/octavia master: Tests helper function _send_requests should catch socket.error  https://review.openstack.org/45200417:37
xgermanwell, if those filters wpould be there I could use that neutron client to test :-(17:46
xgermannow I need to copy & paste from that API spec :-)17:47
*** armax has quit IRC17:53
johnsomThat API spec may not help you.  They never documented for neutron-lbaas (or it got deleted) and the v1 docs in octavia are slightly different17:54
johnsomTwo issues posted, still testing L7policy17:59
rm_workxgerman: rebasing sorting/paging now...18:00
rm_workit was ... complex18:00
rm_worklots of changes lol18:00
rm_worktons of conflicts18:00
xgermanarrgh, wish I had filtering so I can use the passthrouhj18:01
rm_workyeah I know18:06
rm_workworking on it18:06
*** gcheresh_ has joined #openstack-lbaas18:07
*** armax has joined #openstack-lbaas18:07
johnsomHmm, so update is broken for L7policy....18:11
*** harlowja has joined #openstack-lbaas18:12
johnsomUpdating the position and the admin-state doesn't actually update18:13
johnsomOk, I have done a test pass18:18
johnsomI'm going to take a quick look at what we should do about the L7policy operating status issue.  As it is right now, it will never come out of OFFLINE18:19
xgermanBug? https://usercontent.irccloud-cdn.com/file/7jEU5GUJ/Screen%20Shot%202017-04-03%20at%202.27.15%20PM.png18:27
xgermanwill file :-)18:28
johnsomThat is interesting...18:31
xgermanLP doesn’t let you upload pictures?18:31
xgermanhttps://bugs.launchpad.net/octavia/+bug/167925918:31
openstackLaunchpad bug 1679259 in octavia "[V2 API] Trying to crate a pool causes exception" [Undecided,Confirmed]18:31
xgermanFound it18:31
johnsomSo the request is missing the LB ID or Listener ID.  That exception needs some work....18:34
johnsomxgerman By "found it" does that mean you are fixing it?18:34
xgermanno, I found where to upload images18:35
xgermanI could of course try to fix…18:36
johnsomYeah, I can reproduce that18:36
rm_workerg ok18:37
rm_workyou want to fix it?18:37
rm_worki see why18:37
xgermanok, then it’s likely quicker if you do18:37
rm_workk18:38
rm_workthough i have to finish fixing this patch first18:38
rm_worki'm deeep into sorting/paging18:38
johnsomdetails -> detail ?18:38
*** cody-somerville has quit IRC18:39
*** cody-somerville has joined #openstack-lbaas18:39
*** blogan has quit IRC18:40
*** blogan has joined #openstack-lbaas18:41
rm_worki mean i literally just have to finish this and commit/review before i check out to do another change18:42
openstackgerritAdam Harwell proposed openstack/octavia master: Implement sorting and pagination for octavia  https://review.openstack.org/38214718:55
rm_workok18:55
johnsomOk, I am done picking on L7Policy.  I'm going to grab a bite for lunch and then start on L7rules18:55
openstackgerritAdam Harwell proposed openstack/octavia master: Fix pool validation exception typo  https://review.openstack.org/45289918:57
rm_workk i'll go fix policy18:57
rm_workyes johnsom it was detail(s)18:57
rm_workdumb typo18:57
johnsomIt happens.  A quick grep didn't find any others18:58
*** armax has quit IRC19:00
*** aojea has joined #openstack-lbaas19:01
xgermanok19:02
*** csomerville has joined #openstack-lbaas19:05
xgermanafk - back in a while19:07
*** csomerville has quit IRC19:07
*** csomerville has joined #openstack-lbaas19:07
*** cody-somerville has quit IRC19:08
openstackgerritMerged openstack/octavia master: Introduce Octavia v2 API for Members  https://review.openstack.org/40593219:10
rm_workjohnsom: erm, when you're back...19:12
rm_workjohnsom: IIRC the update isn't SUPPOSED to change the DB19:12
rm_workit's the handler that does it19:12
rm_workoh but the handler isn't doing it for you?19:12
rm_workhold up19:12
johnsomRight, second query same result19:14
rm_workerm hey19:18
rm_workcan you check if v1 update works19:18
rm_worki don't think it will either19:18
rm_work<_<19:18
rm_workfrom what I see19:18
rm_workI can fix it19:18
rm_workbut can you verify that?19:18
rm_workjohnsom: ^^19:20
rm_worki am 99% sure both are broken and I know why19:20
rm_workhonestly i am not clear on how member updates works either19:22
rm_workok scratch that I am not clear on how ANY of the updates work19:23
rm_workthey are ALL passing the wrong type19:23
rm_workunless I am blind19:23
rm_workOH I SEE ok19:24
rm_workwell i still do know why19:24
johnsomHmm, I have to figure out the json format for v1, I got a 404 on my first try...19:25
johnsomtypo, ok19:26
rm_workyeah nm i think v1 will work19:27
rm_worki missed one conversion layer19:27
johnsomYep, v1 works for at least admin-state19:27
rm_workthough19:27
rm_workhmm19:27
johnsomOk, back to lunch19:27
rm_workno errors in the worker log?19:27
rm_workah k19:27
rm_workworker/api19:27
johnsomno errors19:28
rm_workhmm19:32
rm_workah19:32
rm_workjust noticed the UpdatePoolInDB DB task's Revert is broken19:32
rm_workhmm19:32
rm_workfor a while19:32
rm_worklooks like from a bad merge19:34
*** chlong has quit IRC19:37
*** harlowja has quit IRC19:37
rm_workor not... hmm19:38
rm_workwell i see why admin_state_up doesn't work19:42
*** armax has joined #openstack-lbaas19:47
*** sanfern has quit IRC19:48
*** chlong has joined #openstack-lbaas19:51
xgermanback as well20:02
johnsomOk, so L7rules are ready right?20:08
rm_workshould be20:16
rm_workhold up, posting an update to policies20:16
rm_workadmin_state should work20:16
rm_workMAYBE position will work?20:16
rm_workah that was many minutes ago :P20:17
openstackgerritAdam Harwell proposed openstack/octavia master: Align Octavia API to n-lbaasv2 for L7Policy  https://review.openstack.org/40632820:18
rm_workok20:18
rm_workcomments addressed, I believe20:18
rm_workplease do retest update20:18
rm_workonly question was about showing rules on the policy response20:19
rm_workjohnsom: we don't do that currently for pools->members either20:19
rm_workI can enable it, but if I do IMO i should do it for both20:19
rm_worki honestly feel like we should always return the WHOLE TREE if we know it :P20:19
rm_workbut that isn't how the API is set up right now20:19
rm_workit's very minimal and requires traversing manually and making many calls20:20
rm_workbut that is not difficult to change20:20
rm_workah, though I just realized when paging/sorting comes into play, that will be a wrench20:22
rm_workwell, maybe not20:22
openstackgerritAdam Harwell proposed openstack/octavia master: Align Octavia API to n-lbaasv2 for L7Policy  https://review.openstack.org/40632820:28
rm_workone more tweak20:28
rm_workxgerman: so you reviewing policies?20:29
xgermanI am installing the latest20:29
johnsomWith rules thing, isn't it just the rule IDs?20:29
rm_workerm20:30
rm_workis that what neutron-lbaas does?20:30
rm_workwe didn't have it set up that way20:30
johnsomLet me look20:30
rm_workif it is, I can make it happen20:30
rm_worksindhu: are you working on the HM patch? https://review.openstack.org/#/c/406256/20:30
sindhurm_work: I was looking at functional test20:31
rm_workok20:31
sindhurm_work: but can u take a look20:31
rm_worki'd like to apply a set of generic refactors I've been doing to all of the functional test files20:31
rm_workto make them consistent20:31
sindhurm_work: sure go ahead20:31
rm_workok, you don't have any outstanding work to push up?20:31
rm_workI can wait if you do20:32
sindhurm_work: I haven't made much changes20:32
sindhurm_work: no I don't have any major changes20:32
rm_workok20:32
johnsomhttps://www.irccloud.com/pastebin/GrUxm65B/20:32
openstackgerritAdam Harwell proposed openstack/octavia master: Align Octavia API to n-lbaasv2 for L7Rules  https://review.openstack.org/40633620:33
johnsomrm_work that is a "show" for the policy, it's just IDs like we have in other parts of the API20:33
rm_workok20:33
rm_workI can do that20:33
*** armax has quit IRC20:34
*** armax has joined #openstack-lbaas20:34
*** armax has quit IRC20:35
johnsomPools does do members this way as well, FYI...20:43
*** armax has joined #openstack-lbaas20:48
rm_workyeah ok20:51
rm_worki'll fix them both20:51
rm_workoh errr20:51
rm_workyeah k20:51
johnsomOk, yeah, I see that pool has the full members, not just the IDs.  Missed that one20:52
*** harlowja has joined #openstack-lbaas20:53
johnsomOk, moving on to rules20:55
johnsomHmm, Unknown column 'l7rule.created_at' in 'field list'\20:59
johnsomI did the migration for L7policy....20:59
rm_workhmmmmmm21:00
rm_workrecently?21:00
rm_worki have updated it a few times21:00
johnsomThis morning.21:00
rm_workk21:00
rm_worknot sure i did the created_at though, fff21:00
rm_worksec21:00
rm_workok I am only partly done with this HM thing but i'm going to push it up and come back to it in a bit21:00
rm_workand fix the stuff on L7Policy21:00
johnsomOk, yeah, L7rule is missing those21:01
openstackgerritAdam Harwell proposed openstack/octavia master: Introduce Octavia v2 API for Health Monitor  https://review.openstack.org/40625621:01
*** gcheresh_ has quit IRC21:04
* xgerman doesn’t know why I not always start with a clean ubunu install when bringing up devstack21:05
nmagnezijohnsom, rm_work, diltram, thanks for the feedback :-)21:07
nmagnezidiltram, currently for centos amp, you must use user_group = haproxy21:07
rm_workjohnsom: on Pool, does it ALSO do the same thing for Health Monitors (shows a list of IDs)?21:09
johnsomhttps://www.irccloud.com/pastebin/fMZInPQU/21:10
johnsomNo, it's not a list, just one line with the HM ID21:10
rm_workk21:11
*** netchord has joined #openstack-lbaas21:12
rm_workabout to push up policy, then fix rules21:18
*** aojea has quit IRC21:20
*** aojea has joined #openstack-lbaas21:20
rm_workgiving it a separate migration >_<21:21
openstackgerritAnkur proposed openstack/python-octaviaclient master: Initialize plugin for OSC  https://review.openstack.org/44622321:23
diltramnmagnezi: we're waiting for a patch21:23
diltramI'm not gonna set it21:23
*** darrenc has quit IRC21:24
*** darrenc has joined #openstack-lbaas21:24
openstackgerritAdam Harwell proposed openstack/octavia master: Align Octavia API to n-lbaasv2 for L7Policy  https://review.openstack.org/40632821:24
*** aojea has quit IRC21:25
rm_workjohnsom: ok, actually fixed pools AND l7policy in that patch <_<21:29
johnsomOk21:29
rm_workAUGH21:30
rm_workmissed a pep8 on the policy one21:30
rm_workoh nm no i didn't21:30
rm_workwhew21:30
rm_workah yep I did21:31
rm_workfff21:31
openstackgerritAdam Harwell proposed openstack/octavia master: Align Octavia API to n-lbaasv2 for L7Rules  https://review.openstack.org/40633621:33
openstackgerritAdam Harwell proposed openstack/octavia master: Align Octavia API to n-lbaasv2 for L7Policy  https://review.openstack.org/40632821:33
rm_workk21:33
rm_workjohnsom: should be fixed21:34
rm_workbut i'm about to have to call it for the day... meant to go to sleep 3.5 hours ago21:34
johnsomOK21:34
johnsomOk, catch you tomorrow21:34
*** reedip has quit IRC21:35
openstackgerritAnkur proposed openstack/python-octaviaclient master: Initialize plugin for OSC  https://review.openstack.org/44622321:38
*** armax has quit IRC21:51
*** armax has joined #openstack-lbaas21:57
*** armax has quit IRC22:02
*** rcernin has quit IRC22:12
rm_workugh got distracted by HMs22:13
rm_workjohnsom: HMs are locked to pool_id in v122:13
rm_workbut in v2 they are distinct objects right?22:13
johnsomYes22:13
rm_workk22:14
johnsomPart of the migration just copies the uuid from pool to the HM ID field I think22:14
johnsombecause default UUID isn't a thing...22:14
johnsomBTW you added project ID in policy and in rule22:16
johnsomOh, maybe not, maybe it is just my DB state22:17
johnsomhmmm22:17
johnsomYep, fixed22:21
rm_workyeah sorry22:23
rm_workmoved it22:23
rm_workbecause... relevancy22:23
johnsomYeah, no problem22:23
*** chlong has quit IRC22:38
*** armax has joined #openstack-lbaas22:39
*** fnaval has quit IRC22:42
rm_workjohnsom: you can't change a HM type after creation?22:43
johnsomI don't think so, let me check22:43
*** catintheroof has quit IRC22:44
johnsomNope, it's a delete/recreate22:44
rm_workk22:44
johnsomGrrrr, something is borked22:47
rm_workok getting weird DB issues23:10
rm_workstrange23:10
johnsomI'm about ready to stop hacking on l7rule just to keep testing23:11
openstackgerritNir Magnezi proposed openstack/octavia master: (WIP) Auto detect haproxy user_group  https://review.openstack.org/42939823:11
rm_workerr23:17
rm_workwhat is broken with it23:17
johnsomI posted comments already.  But the paths are wrong l7rule->rule23:17
johnsomI tried hacking it in, but the change list was getting big.  I'm going to read through now and get to the other stuff I need to do today.23:18
rm_workok not done with that yet, but I NEED to sleep23:19
openstackgerritAdam Harwell proposed openstack/octavia master: Introduce Octavia v2 API for Health Monitor  https://review.openstack.org/40625623:19
rm_workkk23:19
johnsomCatch you in the morning when we can wrap this stuff up.23:19
rm_workdo rules HAVE admin_state_up?23:27
johnsomhttps://github.com/openstack/neutron-lbaas/blob/master/neutron_lbaas/extensions/l7.py#L16123:27
johnsomEvidently23:27
rm_workk23:27
johnsomGot to sleep dude.  The issues will still be here in the morning...  Grin23:28
johnsomI have some administrative junk I need to do today, so I can't hack on them.23:29
rm_workk23:31
rm_workwell23:32
rm_workumm23:32
rm_workit's cool, i'll get a 3 hour nap before class23:32
rm_worki honestly might take tomorrow off23:32
rm_worki haven't slept much in a while23:33
rm_workthis schedule is really bad with classes23:33
*** armax has quit IRC23:38
*** sticker has quit IRC23:42
rm_workseriously why does pep8 take longer to START than py27 does to finish23:46
johnsomI don't know....23:46
rm_workalright i think i almost got this23:50
johnsomAlright, on administriva done, one to go23:51
openstackgerritAdam Harwell proposed openstack/octavia master: Align Octavia API to n-lbaasv2 for L7Rules  https://review.openstack.org/40633623:53
rm_workk23:53
rm_workshould have fixed all of that23:53
rm_workanywhooooo23:53
* rm_work sleeps23:54
« Sunday, 2017-04-02 Index Tuesday, 2017-04-04 »

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