Wednesday, 2017-04-26

johnsomOther than that it worked for all of my tests just fine.  The quota thing was the only part missing00:00
johnsomSorry, didn't get a chance to look at single call create today.  Maybe tomorrow00:00
rm_worknp00:00
johnsomIf have a sec to take a look at the boiler plate for tempest-plugin repo: https://review.openstack.org/44608200:03
johnsomOk, ping me if something is blocking you, otherwise I'm off for the evening.00:09
rm_workkk00:12
rm_workgo! have fun00:12
*** JudeC has quit IRC00:27
*** sticker has joined #openstack-lbaas00:32
*** nmagnezi has quit IRC01:00
*** nmagnezi has joined #openstack-lbaas01:00
*** yuanying_ has joined #openstack-lbaas01:04
*** blogan has quit IRC01:04
*** blogan has joined #openstack-lbaas01:05
*** oomichi has quit IRC01:05
*** yuanying has quit IRC01:06
*** oomichi has joined #openstack-lbaas01:07
*** blogan_ has joined #openstack-lbaas01:10
*** blogan has quit IRC01:14
rm_workummmmmmmmm OK so, quotas are a wrench in sorting/paging01:30
rm_workbecause they aren't normal objects01:30
rm_workthey don't have ids01:30
rm_workor create times01:30
*** JudeC has joined #openstack-lbaas02:40
openstackgerritcheng proposed openstack/octavia master: Add new PROXY protocol to lbaas pool  https://review.openstack.org/45875502:54
*** Brenda has joined #openstack-lbaas03:26
*** amotoki has joined #openstack-lbaas03:27
*** reedip has joined #openstack-lbaas03:31
*** blogan__ has joined #openstack-lbaas03:48
*** JudeC has quit IRC03:50
*** blogan_ has quit IRC03:50
*** blogan__ has quit IRC03:51
*** amotoki has quit IRC03:54
*** reedip has quit IRC03:56
*** reedip has joined #openstack-lbaas03:56
*** amotoki has joined #openstack-lbaas04:10
*** amotoki has quit IRC04:15
*** amotoki has joined #openstack-lbaas04:20
*** gongysh has joined #openstack-lbaas04:46
*** gongysh has quit IRC04:49
*** amotoki has quit IRC04:49
*** amotoki has joined #openstack-lbaas04:58
*** reedip has quit IRC05:02
*** sticker_ has joined #openstack-lbaas05:25
*** darrenc has quit IRC05:28
*** darrenc has joined #openstack-lbaas05:28
*** sticker has quit IRC05:29
*** JudeC has joined #openstack-lbaas05:34
*** cody-somerville has quit IRC05:49
*** cody-somerville has joined #openstack-lbaas05:49
*** csomerville has joined #openstack-lbaas05:50
*** bzhao has quit IRC05:50
*** bzhao has joined #openstack-lbaas05:50
*** cody-somerville has quit IRC05:53
*** kobis has joined #openstack-lbaas06:07
*** amotoki has quit IRC06:30
*** amotoki has joined #openstack-lbaas06:31
*** armax has quit IRC06:33
*** gcheresh_ has joined #openstack-lbaas06:46
*** amotoki has quit IRC06:47
*** tesseract has joined #openstack-lbaas07:04
*** voelzmo has joined #openstack-lbaas07:13
*** pcaruana has joined #openstack-lbaas07:19
*** voelzmo has quit IRC07:47
*** voelzmo_ has joined #openstack-lbaas07:50
openstackgerritJude Cross proposed openstack/python-octaviaclient master: [WIP] Add rest of load balancer commands  https://review.openstack.org/45451607:53
*** JudeC has quit IRC07:54
*** amotoki has joined #openstack-lbaas07:58
*** voelzmo_ has quit IRC08:03
*** voelzmo has joined #openstack-lbaas08:08
*** voelzmo has quit IRC08:17
*** voelzmo has joined #openstack-lbaas08:21
*** reedip has joined #openstack-lbaas08:31
*** sticker_ is now known as sticker08:33
*** aojea has joined #openstack-lbaas08:55
*** aojea_ has joined #openstack-lbaas09:19
*** aojea has quit IRC09:23
*** zoukeke has joined #openstack-lbaas09:32
*** krypto has joined #openstack-lbaas09:41
*** reedip has quit IRC10:00
*** zoukeke has quit IRC10:10
openstackgerritZhaoBo proposed openstack/octavia master: Extend api to accept qos_policy_id  https://review.openstack.org/45830810:47
nmagnezio/12:02
*** reedip has joined #openstack-lbaas13:20
*** krypto has quit IRC13:24
*** krypto has joined #openstack-lbaas13:25
*** krypto has quit IRC13:25
*** krypto has joined #openstack-lbaas13:25
xgermano/13:36
*** voelzmo has quit IRC13:37
*** kobis has quit IRC13:44
*** krypto has quit IRC13:48
*** reedip has quit IRC13:54
*** catintheroof has joined #openstack-lbaas13:56
*** chlong has joined #openstack-lbaas13:58
*** armax has joined #openstack-lbaas13:58
*** amotoki has quit IRC14:07
*** amotoki has joined #openstack-lbaas14:21
*** krypto has joined #openstack-lbaas14:22
*** krypto has quit IRC14:28
*** krypto has joined #openstack-lbaas14:29
*** krypto has quit IRC14:29
*** krypto has joined #openstack-lbaas14:29
*** amotoki has quit IRC14:38
*** amotoki has joined #openstack-lbaas14:43
*** amotoki has quit IRC14:53
*** gcheresh_ has quit IRC15:05
*** fnaval has joined #openstack-lbaas15:07
*** catintheroof has quit IRC15:10
*** amotoki has joined #openstack-lbaas15:11
*** reedip has joined #openstack-lbaas15:11
*** catintheroof has joined #openstack-lbaas15:11
*** catintheroof has quit IRC15:11
johnsomo/15:13
*** blogan has joined #openstack-lbaas15:14
*** tonygunk has joined #openstack-lbaas15:28
*** aojea_ has quit IRC15:29
*** tonygunk has quit IRC15:53
*** reedip has quit IRC15:55
*** tonygunk has joined #openstack-lbaas15:57
*** tonygunk has quit IRC16:01
*** tonygunk has joined #openstack-lbaas16:10
*** krypto has quit IRC16:13
*** reedip has joined #openstack-lbaas16:47
*** tesseract has quit IRC16:51
*** krypto has joined #openstack-lbaas16:57
*** krypto has quit IRC16:57
*** krypto has joined #openstack-lbaas16:57
rm_worko/16:59
*** cpuga has joined #openstack-lbaas16:59
rm_workjohnsom: quotas really throws a wrench in pagination/sorting16:59
rm_worki'm working through the issues there16:59
rm_workbut... it's really odd16:59
johnsomUgh17:00
johnsomCouldn't we just add a created_at column?17:00
*** cpuga_ has joined #openstack-lbaas17:01
*** cpuga has quit IRC17:04
rm_workuhh17:04
rm_workmaybe17:04
rm_workalso ids :/17:04
rm_workok i guess it wasn't that bad, i was just tired17:07
rm_workwell, also tired now, but17:07
rm_workposting an update17:07
rm_workhad to move some stuff around interestingly17:07
rm_workshould be fine17:08
*** aojea has joined #openstack-lbaas17:20
openstackgerritAdam Harwell proposed openstack/octavia master: Implement sorting and pagination for octavia  https://review.openstack.org/38214717:24
openstackgerritAssaf Muller proposed openstack/neutron-lbaas master: Use Tempest TimeoutException from tempest.lib  https://review.openstack.org/46022717:26
rm_workjohnsom: sorry the single-create CR seems to have become a mammoth, a little bit17:26
rm_workbut, a lot of the LINES changed are just docs and tests17:26
rm_workover half, actually17:26
johnsomYeah, no problem17:26
johnsomI am restacking to finish testing the proxy protocol patch, then back to sorting, then single call17:27
rm_worklol k well17:27
rm_worksingle-call is ahead of sorting on the chain17:27
rm_workFYI17:27
johnsomYeah, I know17:27
johnsomIt's just what I'm focusing on.  I have already read through sorting and done some testing, so wrapping it up17:28
rm_workkk17:28
rm_workbrb nap17:28
openstackgerritGerman Eichberger proposed openstack/octavia master: [WIP] Adds status tree to V2 API  https://review.openstack.org/45988017:32
*** pcaruana has quit IRC17:40
*** gcheresh_ has joined #openstack-lbaas17:41
*** csomerville has quit IRC17:49
*** csomerville has joined #openstack-lbaas17:49
*** krypto has quit IRC17:51
*** reedip has quit IRC17:51
*** reedip has joined #openstack-lbaas17:57
*** aojea has quit IRC18:01
*** tonygunk has quit IRC18:02
*** aojea has joined #openstack-lbaas18:04
*** tonygunk has joined #openstack-lbaas18:11
*** reedip has quit IRC18:16
*** aojea has quit IRC18:24
*** aojea has joined #openstack-lbaas18:25
*** aojea has quit IRC18:29
*** JudeC has joined #openstack-lbaas18:39
*** voelzmo has joined #openstack-lbaas18:39
*** aojea has joined #openstack-lbaas18:58
*** cpuga_ has quit IRC19:05
*** cpuga has joined #openstack-lbaas19:05
*** cpuga has quit IRC19:05
*** cpuga has joined #openstack-lbaas19:05
johnsomArgh, our amphora agent log is messed up.  Some messages are not going to the file log, just the systemd log system19:14
*** cpuga has quit IRC19:40
*** mdurrant has joined #openstack-lbaas19:40
*** cpuga has joined #openstack-lbaas19:40
*** mdurrant__ has quit IRC19:42
xgermanwonder why cover doesn’t like me19:52
xgerman^^ johnsom if py27 and py35 pass cover should only fail if i decrease coverage by a ton.19:58
xgermanSo not sure19:58
xgermanhttps://review.openstack.org/45988019:58
johnsomOctavia meeting starting soon on #openstack-meeting-alt20:00
*** cody-somerville has joined #openstack-lbaas20:04
*** cody-somerville has quit IRC20:04
*** cody-somerville has joined #openstack-lbaas20:04
*** csomerville has quit IRC20:06
*** aojea has quit IRC20:08
*** voelzmo has quit IRC20:13
*** aojea has joined #openstack-lbaas20:17
*** cpuga has quit IRC20:19
*** cpuga has joined #openstack-lbaas20:19
*** cpuga has left #openstack-lbaas20:20
*** cpuga has joined #openstack-lbaas20:20
johnsomxgerman It failed because functional tests failed.  Coverage runs both py27 and functional test suites20:46
xgermanoh, ok20:47
*** gcheresh_ has quit IRC20:58
*** cpuga has quit IRC21:02
*** chlong has quit IRC21:48
*** aojea has quit IRC22:10
*** fnaval has quit IRC22:21
openstackgerritGerman Eichberger proposed openstack/octavia master: Adds status tree to V2 API  https://review.openstack.org/45988022:42
stickerHey all23:22
stickertrying to figure out why https://github.com/openstack/neutron-lbaas/blob/master/neutron_lbaas/services/loadbalancer/data_models.py#L96 is "< 2"23:23
stickeri can't see a reason why it wouldn't be "== 0"23:23
stickerit's making a heap more database calls than it needs to if it's "< 2".  In one of my environments, it does around 20,000 mysql queries for a single lookup23:24
stickerwith "== 0" it does maybe 20023:24
stickerand I can't see reason why you would need it to pass over the same class more than once23:25
stickercan anyone enlighten me?23:25
sticker:)23:25
stickersame goes for https://github.com/openstack/neutron-lbaas/blob/master/neutron_lbaas/services/loadbalancer/data_models.py#L11123:26
johnsomLet me take a look23:27
johnsomOh, neutron-lbaas...23:28
stickeryeah.. is that still this channel?23:28
johnsomYes23:28
johnsomsticker I am not super familiar with this code.  My suspicion is that it is < 2 for the case where pools are shared23:35
stickerhmm okay.. maybe there needs to be some additional logic in it23:36
johnsomHowever, this should be working against the sqlalchemy model and not actually making database calls23:37
johnsomIt should be an all in-memory operation23:37
stickerweird.. i'm seeing a lot of db traffic flying around23:37
johnsomMaybe it is late loading that sa model for some reason23:40
stickerand i don't think it'll scale well.  For instance, it looks at a listener object, then the pools that are linked to it, which have references to the listener object so it goes back to that again.. etc.  It loops around a lot with "< 2" as it is happy to go Listener -> Pool -> Listener, etc..23:40
johnsomYeah, there are definitely issues with the model handling in neutron-lbaas.  The octavia models that are replacing this stuff are in better shape23:41
stickeroh cool.. will that be done by Pike do you think?23:41
*** cpuga has joined #openstack-lbaas23:41
johnsomIt depends on the driver you are using.  If you are using the octavia driver, yes, we are pretty close to on track for Pike.  If you are using another driver, I don't think we are going to make Pike.23:42
*** cpuga_ has joined #openstack-lbaas23:42
rm_workjohnsom / anyone: I'm adding a config opts to specify the full URI of the API for purposes ot pagination links23:46
rm_workright now I was detecting it from request, but you might be through a LB or something or using some other DNS23:46
rm_workso I need to have it configurable I think23:46
*** cpuga has quit IRC23:46
johnsomI was wondering about that.23:46
rm_workright now we have an option "host" for the hostname23:46
rm_workso I can either add another option for the port and for http_type23:46
johnsomCan they be relative?23:46
rm_work(HTTP/HTTPS)23:46
rm_workthey're supposed to be full23:47
johnsomWell, host is used for the oslo messaging (I hate that config opt)23:47
rm_workor I can add ANOTHER opt that you put the entire thing including "http://myurl:port"23:47
johnsomIt should be the local hostname of the message receiver23:47
rm_workok so another opt?23:47
johnsomYeah, I think so.23:47
johnsomHow does neutron do it, do you know?23:48
johnsomYeah, I see others have an opt23:49
rm_workI do not23:49
johnsomhttps://www.irccloud.com/pastebin/1q6T8QFP/23:49
rm_workwhat do they name it23:49
rm_workok23:49
johnsomHa, in good OpenStack fashion I also see:23:53
johnsomnevermind, I was looking at the wrong thing.23:55

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