Tuesday, 2017-03-28

*** oomichi has quit IRC00:19
*** oomichi has joined #openstack-lbaas00:23
*** ducttape_ has joined #openstack-lbaas00:29
*** bzhao has quit IRC00:31
*** bzhao has joined #openstack-lbaas00:32
*** catintheroof has joined #openstack-lbaas00:35
*** gongysh has joined #openstack-lbaas01:01
*** amotoki has joined #openstack-lbaas01:08
*** reedip has quit IRC01:12
*** amotoki has quit IRC01:14
openstackgerritJi Chengke proposed openstack/octavia master: change genconfig output-file path  https://review.openstack.org/44990201:17
*** xgerman has quit IRC01:25
*** nakul_d has quit IRC01:25
*** ipsecguy has quit IRC01:25
*** xgerman has joined #openstack-lbaas01:27
*** ipsecguy has joined #openstack-lbaas01:27
*** nakul_d has joined #openstack-lbaas01:27
*** ducttape_ has quit IRC01:30
*** gongysh has quit IRC01:34
*** catintheroof has quit IRC01:41
*** sanfern has quit IRC01:51
*** reedip has joined #openstack-lbaas02:00
*** ducttape_ has joined #openstack-lbaas02:12
*** ducttape_ has quit IRC02:27
*** oomichi has quit IRC02:28
*** oomichi has joined #openstack-lbaas02:32
*** oomichi has quit IRC02:39
*** oomichi has joined #openstack-lbaas02:42
*** gongysh has joined #openstack-lbaas02:43
*** blogan has quit IRC02:50
*** mjblack has quit IRC03:08
*** aojea has joined #openstack-lbaas03:13
*** fnaval has joined #openstack-lbaas03:17
*** aojea has quit IRC03:18
*** mjblack has joined #openstack-lbaas03:26
*** ducttape_ has joined #openstack-lbaas03:27
*** ducttape_ has quit IRC03:32
*** links has joined #openstack-lbaas03:36
*** sanfern has joined #openstack-lbaas03:40
*** ajo has quit IRC03:50
*** ajo has joined #openstack-lbaas03:54
*** armax has quit IRC04:03
*** ducttape_ has joined #openstack-lbaas04:28
*** ducttape_ has quit IRC04:29
*** MarkMielke has quit IRC04:35
*** gcheresh_ has joined #openstack-lbaas05:16
*** aojea has joined #openstack-lbaas05:26
*** aojea has quit IRC05:27
*** aojea has joined #openstack-lbaas05:28
*** aojea has quit IRC05:32
*** rcernin has joined #openstack-lbaas05:38
*** aojea has joined #openstack-lbaas05:52
*** ducttape_ has joined #openstack-lbaas05:58
*** ducttape_ has quit IRC06:03
*** aojea has quit IRC06:03
*** bzhao has quit IRC06:27
*** bzhao has joined #openstack-lbaas06:28
*** oomichi has quit IRC06:48
*** oomichi has joined #openstack-lbaas06:52
*** amotoki has joined #openstack-lbaas07:13
*** pcaruana has joined #openstack-lbaas07:13
*** aojea has joined #openstack-lbaas07:17
*** tesseract has joined #openstack-lbaas07:18
*** amotoki has quit IRC07:18
*** rcernin has quit IRC07:29
*** rcernin has joined #openstack-lbaas07:47
*** aojea_ has joined #openstack-lbaas07:55
*** aojea has quit IRC07:58
*** rcernin has quit IRC08:06
*** rcernin has joined #openstack-lbaas08:07
*** rcernin has quit IRC08:10
*** rcernin has joined #openstack-lbaas08:10
*** amotoki has joined #openstack-lbaas08:14
*** amotoki has quit IRC08:19
*** kobis has joined #openstack-lbaas08:23
*** yamamoto has quit IRC08:31
*** bbbzhao has joined #openstack-lbaas08:43
*** bzhao has quit IRC08:46
*** rcernin has quit IRC08:46
*** rcernin has joined #openstack-lbaas08:46
*** rcernin_ has joined #openstack-lbaas08:48
*** fnaval has quit IRC08:53
*** amotoki has joined #openstack-lbaas09:15
*** amotoki has quit IRC09:19
*** yamamoto has joined #openstack-lbaas09:31
*** gongysh has quit IRC09:39
*** gongysh has joined #openstack-lbaas09:41
*** gongysh has quit IRC09:47
*** reedip has quit IRC09:51
*** yamamoto has quit IRC09:56
*** belharar has joined #openstack-lbaas09:57
*** yamamoto has joined #openstack-lbaas10:04
*** krypto has joined #openstack-lbaas10:05
*** krypto has joined #openstack-lbaas10:05
*** amotoki has joined #openstack-lbaas10:16
*** amotoki has quit IRC10:20
*** sanfern has quit IRC10:28
*** yamamoto has quit IRC11:00
*** oomichi has quit IRC11:09
*** oomichi has joined #openstack-lbaas11:13
*** amotoki has joined #openstack-lbaas11:17
*** amotoki has quit IRC11:21
*** yamamoto has joined #openstack-lbaas11:25
*** openstackgerrit has quit IRC11:33
*** krypto has quit IRC11:38
*** krypto has joined #openstack-lbaas11:39
*** krypto has quit IRC11:39
*** krypto has joined #openstack-lbaas11:39
*** openstackgerrit has joined #openstack-lbaas11:47
*** reedip has joined #openstack-lbaas11:50
*** chlong has joined #openstack-lbaas12:17
*** gongysh has joined #openstack-lbaas12:43
*** ducttape_ has joined #openstack-lbaas12:46
*** ducttape_ has quit IRC12:47
*** ducttape_ has joined #openstack-lbaas12:48
*** ducttape_ has quit IRC13:12
*** kobis has quit IRC13:16
*** amotoki has joined #openstack-lbaas13:18
*** links has quit IRC13:22
*** amotoki has quit IRC13:23
strigazihi there, we have octatia in our magnum jobs. Recently, diskimage-builder and dib-utils removed their /bin dir that used contain some shell script that could be used instead of installing.13:24
strigaziHave you address this chnage in stable/newton?13:25
*** kobis has joined #openstack-lbaas13:31
strigazifrom this job run in newton i think that is the problem: http://logs.openstack.org/98/450698/1/check/gate-functional-dsvm-magnum-api-identity-v3-only-ubuntu-xenial-nv/b9f4125/logs/devstacklog.txt.gz#_2017-03-28_13_00_50_06213:33
*** amotoki has joined #openstack-lbaas13:34
*** sanfern has joined #openstack-lbaas13:43
*** ducttape_ has joined #openstack-lbaas13:43
*** belharar has quit IRC13:51
*** bzhao_ has joined #openstack-lbaas14:04
*** yamamoto has quit IRC14:12
johnsomstrigazi It is addressed in master, but maybe we don't have all of the backports done yet.  It is very frustrating the number of breaking changes from DIB and their lack of stable branches14:25
*** armax has joined #openstack-lbaas14:27
strigazijohnsom yeah we have a similar problem but thankfully only on job. Maybe you could pin to a commit for the stable branches. i don't know if this is a good practice though14:28
johnsomRight, there are trade offs.  I will take a look at or stable branches today.14:30
*** gongysh has quit IRC14:38
dasmjohnsom: o/ i'm releasing updates for newton. neutron-lbaas was assigned to neutron team in newton. should i take care of that, or maybe you'll take it over from me?14:39
dasmhttps://github.com/openstack/releases/blob/master/deliverables/newton/neutron-lbaas.yaml14:39
johnsomdasm I have it.  We still have backports anyway14:40
dasmjohnsom: ack, thanks14:40
*** armax has quit IRC14:41
*** armax has joined #openstack-lbaas14:41
*** krypto has quit IRC14:45
*** krypto has joined #openstack-lbaas14:45
*** kobis has quit IRC14:50
*** kobis has joined #openstack-lbaas14:51
*** kobis has quit IRC14:52
*** blogan has joined #openstack-lbaas14:53
*** fnaval has joined #openstack-lbaas14:57
*** dasm has quit IRC15:08
*** gcheresh_ has quit IRC15:08
*** dasm has joined #openstack-lbaas15:13
*** yamamoto has joined #openstack-lbaas15:13
*** krypto has quit IRC15:15
*** krypto has joined #openstack-lbaas15:16
*** sanfern has quit IRC15:19
*** krypto has quit IRC15:21
*** yamamoto has quit IRC15:23
*** rcernin has quit IRC15:31
*** rcernin_ has quit IRC15:31
*** sanfern has joined #openstack-lbaas15:31
johnsomWhat the heck???  They are trying to remove our diversity tag????  https://review.openstack.org/#/c/44866715:40
*** fnaval has quit IRC15:52
*** fnaval has joined #openstack-lbaas15:59
*** aojea_ has quit IRC16:00
ankur-gupta-f4johnsom: make me a core!!! then it will become less that 50%. Quick fix. :P  I have some questions about the Pools controller16:05
xgermanha16:12
*** pcaruana has quit IRC16:26
nmagnezijohnsom, do they count neutron-lbaas?16:28
johnsomYes16:28
rm_workah i wonder if they realize brandon isn't RAX now?16:32
rm_workthat'd reduce the rax core-number a bit16:33
*** chlong has quit IRC16:34
johnsomWell, technically all of neutron are still core too.  You will see Armando and Ihar both did core reviews in octavia...16:35
johnsomThe reason given on the patch is 56% reviews, but that is adding Intel and Rackspace together, which I don't think is reasonable.16:35
nmagnezijohnsom, sorry if it is a dumb question, but are there any implications for that tag?16:37
johnsomWell, yes, they could drop us as an official OpenStack project.16:38
nmagneziwow, okay.16:38
johnsomIf it is perceived to be a pet project of one company.16:38
nmagnezibtw if you look at the Ocata statistics, RAX is only 45%, so why do they give a verdict based on data that can change by the end of the cycle?16:39
nmagnezihttp://stackalytics.com/?module=octavia&release=ocata16:39
johnsomThey are going off of Pike data16:40
xgermanok, so if jonsom and I review less we are good?16:47
johnsomI know, right....16:47
xgermanuninetended consequences16:48
*** chlong has joined #openstack-lbaas16:49
johnsomYeah, so looking at the tool output, 56% of the core reviews were from Rackspace employees.  That is the problem in the view of the tag guidelines....16:57
johnsomI guess as PTL I should register independent to not skew the numbers....16:58
*** bzhao_ has quit IRC17:09
*** krypto has joined #openstack-lbaas17:09
*** krypto has joined #openstack-lbaas17:09
rm_workankur-gupta-f4: replied to your comment on pools17:09
rm_workwho else can review that today BTW? johnsom is already +217:10
ankur-gupta-f4Yea i saw17:10
ankur-gupta-f4But we are moving away from lookup since17:10
rm_workis diltram out? haven't heard from him yet this week17:10
ankur-gupta-f4The url endpoints different17:10
ankur-gupta-f4Its no longer the giant string of <lb-id>/listener/<list-id>/pool etc...17:11
ankur-gupta-f4Which was the use case for lookup/remainder17:11
ankur-gupta-f4Right17:11
sshankankur-gupta-f4, /v2/pool/<pool_id>/member/... To find member that lookup is used.17:12
rm_workmembers have no high level endpoint, right17:13
rm_workeven in this API17:13
ankur-gupta-f4They do dont they17:14
rm_workI don't think so17:14
ankur-gupta-f4Nvm17:14
ankur-gupta-f4Ur right17:14
reedip_o17:14
ankur-gupta-f4Okay17:14
ankur-gupta-f4Makes sense17:14
ankur-gupta-f4Thx17:14
ankur-gupta-f4Ignore comment17:14
rm_workhttps://github.com/openstack/octavia/blob/master/octavia/tests/functional/api/v2/base.py#L49-L5017:15
rm_workah you found it, k17:15
ankur-gupta-f4Yea looked at nlbaas api17:15
rm_workHMs do have their own though, but i guess we are still allowing them to be addressable via that?17:15
rm_workmaybe the HM piece of _loopup() could go away?17:16
ankur-gupta-f4Yea17:16
ankur-gupta-f4In which case lookup seems like a misnomer17:16
reedipso who is becoming the new core :)17:16
ankur-gupta-f4But its okay for now17:16
ankur-gupta-f4I vote sshank17:16
rm_workcan't be rax or intel :P17:17
reedipwhy rm_work ?17:17
rm_workthat's the whole issue17:17
reedip??17:17
rm_workis that rax and intel control most of the reviews apparently17:17
reedipoh diversity tag17:18
reedipbecause of cores?17:18
rm_workyeah17:18
reedipand non-cores dont count?17:18
rm_workapparently not to them17:18
reedipbut how did they arrive at 56% ?17:20
reediphttp://stackalytics.com/report/contribution/octavia/90 shows 6 cores for octavia with 3 from RAX17:21
reedipAnd well blogan , is he there in RAX?17:22
nmagnezireedip, even better, see the whole group of lbaas related projects http://stackalytics.com/report/contribution/octavia-group/9017:29
reedipbut thats with puppet octavia and oter projects, right ?17:29
reediphey I am on number#5 :D17:30
nmagnezijohnsom, would know the answer to that. I know that includes at least neutron-lbaas neutron-lbaas-dashboard octavia17:30
*** amotoki has quit IRC17:31
johnsomhttps://governance.openstack.org/tc/reference/projects/octavia.html17:31
johnsomThat is the official list.  Also, there is an "Octavia official" group in stackalytics17:31
reedipjohnsom : yeah17:32
ankur-gupta-f4ugh just remembered i need to work on the client. thanks johnsom for the reminder17:32
reedipthe thing is they are combining neutron-lbaas17:32
reedipand other neutron-lbaas items17:32
ankur-gupta-f4reedip: you have like a lot of free time to help me out :)17:32
reedipand other neutron-lbaas projects*17:32
reedipankur-gupta-f4 : its 2 AM here17:33
rm_workwhere IS the core list? need to de-register blogan17:33
ankur-gupta-f4reedip: and...?17:33
rm_workstackalytics still has him as rax prolly17:33
reedipBut Yes I will help you out tomorrow morning !17:33
ankur-gupta-f4haha awesome thanks17:33
reediprm_work : you can go to gerrit and do that I guess17:33
reedipwait17:33
reediphttps://review.openstack.org/#/admin/groups/501,members17:34
reediphttps://review.openstack.org/#/admin/groups/1013,members17:34
reedipthe second one is the dashboard core17:34
reedipand thats the interesting one17:35
reedipIt includes Horizon cores :)17:35
johnsomYeah, octavia still includes neutron cores as well17:35
reedipbut cant see Brandon anywhere here17:36
reedipam I checking the wrong list ?17:36
reedipI can see that people can be added and deleted, but cant find brandon17:37
reedipankur-gupta-f4 : ping me with whatever work needs to be done, I am going to sleep17:37
ankur-gupta-f4okay17:37
bloganrm_work, johnsom: you can't kill me! you can't kill the undead17:40
diltramrm_work: I'm here, you need smth?17:40
diltramblogan: :P17:41
rm_workwas just wondering :P17:41
bloganhey diltram! how's the baby?17:41
diltramwhere are you? why your desk is still empty?17:41
rm_workhadn't seen you talk in a bit17:41
bloganrm_work: i'm always listening17:41
rm_worklol17:41
bloganwatching, waiting to pounce17:41
diltramblogan: great, he starting to cry on beginning of night but for the rest of day he's awesome :)17:41
blogandiltram: like all night?17:42
diltramblogan: no, max an hour17:42
diltramit's PURPLE cyring17:43
diltrambut today he asleep around 1AM and he was sleeping till 7 :D17:43
diltramrm_work: I'm lost in the cloud-init/dpdk stuff :P17:44
diltramI'm trying to make our gates functional finally :P17:44
*** aojea has joined #openstack-lbaas17:45
xgermandiltram just give it wodka17:50
diltrama baby?17:51
xgermanisn’t that what they do in Russia?17:51
xgerman“in Soviet Russia bat drank Vodka”17:52
xgermanbat=baby17:52
johnsomFYI, I already hit the big delete button in the sky on blogan's core....  (I may or may not have been laughing maniacally at the time...)17:52
johnsomThat isn't going to help the ocata stats though...17:52
blogandiltram: how old is he?17:52
*** gcheresh_ has joined #openstack-lbaas17:52
diltramxgerman: I don't know what people do in Russia, never was there17:52
bloganjohnsom: i live in! dont worry, i've got a timebomb in octavia's code17:53
johnsomI would consider letting him back in if he sings the hymn for us though.....  grin17:53
blogani mean...i probably have many of them17:53
bloganbc my code17:53
diltramblogan: 9 weeks17:53
johnsomblogan You aren't kidding....17:53
johnsomgrin17:53
* blogan glares at johnsom17:53
blogandiltram: you're a lucky asshole17:53
diltramI know :D17:54
blogani have another friend who just had a baby and they're like "yeah he's sleeping 4 hours at a time"17:54
bloganthats wonderful17:54
diltram:D :D17:54
diltrammine is sleeping for like 4 hours in day17:54
diltrambut on night he's sleeping 5-7 right now :)17:55
xgermanwow, that’s huge17:55
*** krypto has quit IRC17:55
blogandiltram: well congratulations, really, im glad its a bit easier17:56
diltramblogan: like for the first one, he's the best what I could get17:57
*** kbyrne has quit IRC18:00
johnsomFYI, heads up, I will be putting a patch in to change the service type to "load-balancer" per the discussion in the service type repo.  Even though there is zero traction in that repo, I have started the process in infra and German has put in the patch for OSA18:00
diltramjohnsom: ok18:00
*** tesseract has quit IRC18:06
*** kbyrne has joined #openstack-lbaas18:09
*** gcheresh_ has quit IRC18:10
openstackgerritMichael Johnson proposed openstack/octavia master: Changing the service-type to load-balancer  https://review.openstack.org/45091618:12
*** gcheresh_ has joined #openstack-lbaas18:19
*** aojea has quit IRC18:29
*** aojea has joined #openstack-lbaas18:29
*** aojea has quit IRC18:34
*** eandersson_ is now known as eandersson18:35
*** kbyrne has quit IRC18:50
*** kbyrne has joined #openstack-lbaas18:52
*** armax has quit IRC18:55
*** dlundqui1 has quit IRC19:03
*** dlundquist has joined #openstack-lbaas19:03
*** aojea has joined #openstack-lbaas19:32
*** armax has joined #openstack-lbaas19:50
rm_workanyone for +A on Pools? :P20:42
rm_workwant to move to members!20:42
*** ducttape_ has quit IRC20:50
*** ducttape_ has joined #openstack-lbaas20:50
*** ducttape_ has quit IRC20:51
johnsomrm_work Do you think members is ready for review?  I haven't had a minute today to look20:52
johnsomBut I want to keep going....20:52
*** gcheresh_ has quit IRC20:55
*** ducttape_ has joined #openstack-lbaas20:57
rm_workhmmm21:02
rm_worki need to revisit really quick21:02
rm_workhmm yeah i think there's a little bit still broken with members21:07
rm_worki need to take a look at it still21:07
*** ianychoi has quit IRC21:07
*** aojea has quit IRC21:07
*** ianychoi has joined #openstack-lbaas21:08
*** ducttape_ has quit IRC21:11
johnsomOk.  I'm helping a user debug an amp issue, so.... take your time grin21:11
openstackgerritAdam Harwell proposed openstack/octavia master: Introduce Octavia v2 API for Members  https://review.openstack.org/40593221:18
rm_workwell there's ... a rebase and minor fix21:18
*** aojea has joined #openstack-lbaas21:29
*** ipsecguy has quit IRC21:47
*** aojea has quit IRC21:49
*** aojea has joined #openstack-lbaas21:49
openstackgerritMerged openstack/octavia master: Changing the service-type to load-balancer  https://review.openstack.org/45091621:51
*** aojea has quit IRC21:54
*** cody-somerville has joined #openstack-lbaas22:08
*** fnaval has quit IRC22:08
rm_workPOOOOOOOLS22:29
rm_workReview it! :P22:29
* rm_work heads out for the day22:29
*** ducttape_ has joined #openstack-lbaas22:30
*** yamamoto has joined #openstack-lbaas22:30
*** ducttape_ has quit IRC22:36
*** ducttape_ has joined #openstack-lbaas22:36
*** yamamoto has quit IRC22:37
*** ducttape_ has quit IRC22:39
*** ducttape_ has joined #openstack-lbaas22:39
*** ducttape_ has quit IRC22:39
nmagnezijohnsom, are you busy? I have a question about communication between octavia-api <-> octavia-worker22:44
johnsomnmagnezi Sure, what is your question?22:44
nmagnezijohnsom, i have an env (not devstack) with Octavia deployed, and I see requests reach octavia-api just fine. but for some reason nothing is being passed to the worker. I checked and rabbitmq options seem to look okay22:46
nmagnezitrying to figure out what else can it be22:46
nmagnezithe worker log show no action some something is preventing it from getting the events22:46
nmagnezis/some/so22:46
openstackgerritOpenStack Proposal Bot proposed openstack/octavia master: Updated from global requirements  https://review.openstack.org/45104622:47
johnsomHmmm, so you see a 202 in the api log, but nothing popping off the queue in the controller worker.22:48
johnsomFirst thought, look in the API process logs with a guru report.22:51
johnsomnmagnezi Find the octavia-api process ID, do a kill -SIGUSR2 on it.  This will dump the config into the log.22:52
*** fnaval has joined #openstack-lbaas22:52
johnsomLook for "  api_handler = queue_producer"22:53
* nmagnezi take notes22:53
johnsomJust to make sure the no-op testing handler isn't being used for some reason22:53
*** yamamoto has joined #openstack-lbaas22:53
nmagnezijohnsom, yup i set queue_producer explicitly to make sure that is not the case22:53
johnsomOk, so you thought of that already22:54
nmagnezibut what you suggested above might reveal new info22:54
johnsomYeah, it dumps the "as running" configuration22:54
nmagnezii'll get to that22:54
nmagnezii knew such thing exists, but didn't know how to do it22:54
nmagnezinice :)22:54
johnsomIt helps to find when there are multiple config lines added by mistake22:54
*** yamamoto has quit IRC22:56
nmagneziack. thanks!22:56
johnsomAll of the rabbit stuff is in there too.22:58
johnsomThinking about what else I might check for that....  I would check the octavia-api and octavia-worker logs for rabbit errors, but I'm sure you have covered that already.23:00
*** yamamoto has joined #openstack-lbaas23:03
reedipankur-gupta-f4 : around?23:23
*** yamamoto has quit IRC23:24
ankur-gupta-f4reedip: heyo. Give me a second23:24
reedipseconds up ..23:24
reedip:P23:24
reediptake ur time :)23:24
ankur-gupta-f4reedip: sorry was dealing with JIRA23:29
ankur-gupta-f4good morning23:29
ankur-gupta-f4whats up23:29
reedipDid you apply Heeng with it23:29
reedipyou wanted some client side support ?23:30
ankur-gupta-f4yea. So this is going to be tricky.23:30
ankur-gupta-f4let me dig out the IRC convo23:30
reedipoh ok23:31
ankur-gupta-f4http://eavesdrop.openstack.org/irclogs/%23openstack-sdks/%23openstack-sdks.2017-03-23.log.html23:31
ankur-gupta-f4Huge convo about how to set up the initial client23:31
ankur-gupta-f4in which basically this is not the way: https://review.openstack.org/#/c/448331/23:32
ankur-gupta-f41) help understanding fully what he wants 2) help getting it started. Got too much on my plate right now and wouldn't be able to take a look at it till Thursday/Friday23:33
reedipankur-gupta-f4 : same here.. this is where our stratergy fails, when everyone is too self less :D23:33
reedipanyways, I will look at it tomorrow if all my patches dont finish up today . If they do, then today itself23:34
reedipwill give back a respnse to you when you come back online today evening( for me) /today morning( for you )23:34
reedip:)23:34
ankur-gupta-f4Okay. Will reach out tmw23:34
ankur-gupta-f4Or later this week23:35
reedipwhenever you feel like, lemme know :)23:35
reedipI will comment/work on it in case you are too busy :)23:35
ankur-gupta-f4K will add you to anything23:36
*** catintheroof has joined #openstack-lbaas23:37
reediprm_work, johnsom : moved this patch to octavia , I believe zhaobo would be working on this with me in supporting role. Lemme know if you feel this bug doesnt apply to octavia23:44
reediprm_work, johnsom : https://bugs.launchpad.net/octavia/+bug/165708923:44
openstackLaunchpad bug 1657090 in octavia "duplicate for #1657089 [RFE]Add bandwidth_limit to vip" [Wishlist,In progress] - Assigned to zhaobo (zhaobo6)23:44
johnsomreedip I closed it duplicate of https://bugs.launchpad.net/bugs/165709023:45
openstackLaunchpad bug 1657090 in octavia "[RFE]Add bandwidth_limit to vip" [Wishlist,In progress] - Assigned to zhaobo (zhaobo6)23:45
johnsomThere was more discussion already on 165709023:45
reedipjohnsom : never ming23:45
reedipyeah saw that :)23:45
reedipdont know how 2 of them were created... :)23:45
johnsomI have that spec up on my screen in the "get to this soon" pile....23:46
reedipHAHAHAHA :)23:46
johnsomI have one on top of it23:46
reedipI have it on ping zhaobo and others soon23:46
johnsomThe life of a PTL, you have no idea23:46
reedipdont want to anytime soon :)23:46
johnsomIt's 4:47 and I have an email to finish, one to start, a patch to comment on, a spec to comment on, and a spec to update.  Then I can get back to what I had planned to work on today....  grin.23:48
reedipthats too much ...23:48
reedipI only have 4 patches to work upon, 2 specs to see and about 25 tabs open for reviews ... :D23:49
reedipand then I start my office at 9 am23:49
reedipits 8 right now !23:49
johnsomHa, so you are part way there!23:49
johnsomDo you have a minute to chat about this qos spec?23:50
reedipjohnsom : sure ...23:50
johnsomDo you have a use case for this in addition to ZhaoBo?23:51
reedipfrom my side, as of now, no ... Not a specific use case but I understand his requirements technically.23:51
johnsomEither way we set this up, the end user that creates the LB could remove the restriction, either by deleting the QoS policy from neutron (which I think it would not let you do without removing it from the LB first) or by just updating the LB max_bps23:52
reedipjohnsom : yes, therefore the policy for this would be admin_or_owner23:53
reedipi mean in policy.json23:53
johnsomI'm thinking it makes this a lot easier on us if we just accept a qos_policy_id on the LB and then we blindly apply it to the VIP port.23:54
reedipbecause Ideally only those users who are admins or owner of the port where the QoS is applied should be able to modify it23:54
johnsomThat way when neutron adds some new feature/algorithm for the QoS we just get it for free and don't have to extend the API for it23:54
johnsomRight, they are the same people the can access the load balancer create/update right?23:55
reedipjohnsom: I think zhaobo was convinced with this justification when you discussed it with me, and I shared the same to him23:55
johnsomOk, so folks think just accepting the qos_policy_id is good enough?23:55
reedipso yes, the Spec should change to accept only the qos_policy_id. The rules connnected to the policy wont affect the default Octavia Driver as it doesnt accept DSCP markings23:55
reedipbut third party driver would have to ignore it23:56
reedipjohnsom : as per the last bit of discussion, yes !23:56
johnsomOr use it, a user could specify it and have a backend honor it for something, that would be up to the user.23:56
reedipjohnsom ; totally backened dependent23:57
reedipbut we would have to give them the headsup23:57
johnsomIn reality every device on the network gets to decide if it cares about those fields.  Most just ignore23:57
reediphmm...23:58
reedipsensible the devices are23:59

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