Monday, 2017-12-11

« Sunday, 2017-12-10 Index Tuesday, 2017-12-12 »
*** yamamoto has joined #openstack-lbaas00:32
*** bar_ has joined #openstack-lbaas00:35
*** yamamoto has quit IRC00:40
*** openstackgerrit has joined #openstack-lbaas01:49
openstackgerrithuangshan proposed openstack/python-octaviaclient master: Add listener stats client api and osc  https://review.openstack.org/52661001:49
*** sapd_ has joined #openstack-lbaas01:50
*** sapd__ has quit IRC01:50
openstackgerritBar RH proposed openstack/octavia master: Reorder ComputeCreate arguments  https://review.openstack.org/52696501:53
*** annp has joined #openstack-lbaas02:15
*** oanson has quit IRC02:19
*** oanson has joined #openstack-lbaas02:21
openstackgerritBar RH proposed openstack/octavia master: Reorder ComputeCreate arguments  https://review.openstack.org/52696502:39
*** bar_ has quit IRC02:48
*** yamamoto has joined #openstack-lbaas03:14
*** links has joined #openstack-lbaas04:07
*** links has quit IRC04:07
bzhaohello? Sunday night. :)04:10
*** armax has joined #openstack-lbaas04:36
*** armax has quit IRC04:41
*** Alex_Staf has joined #openstack-lbaas05:04
openstackgerritHengqing Hu proposed openstack/octavia-dashboard master: Make breadcrumb same as other pages  https://review.openstack.org/52698906:01
*** yamamoto has quit IRC06:08
*** ianychoi_ is now known as ianychoi06:12
openstackgerritHengqing Hu proposed openstack/octavia-dashboard master: Make breadcrumb same as other pages  https://review.openstack.org/52698906:16
*** threestrands has quit IRC06:17
*** yamamoto has joined #openstack-lbaas06:22
*** gcheresh has joined #openstack-lbaas06:23
*** yamamoto has quit IRC06:32
openstackgerritZhaoBo proposed openstack/octavia master: Extend api to accept qos_policy_id  https://review.openstack.org/45830806:51
*** eN_Guruprasad_Rn has joined #openstack-lbaas06:51
*** Alex_Staf has quit IRC07:15
*** yamamoto has joined #openstack-lbaas07:33
*** codenamelxl has joined #openstack-lbaas07:37
*** yamamoto has quit IRC07:38
*** yamamoto has joined #openstack-lbaas07:38
codenamelxlHello everyone, I'm kinda confused between Octavia and Neutron LBaaSv2. Is Neutron LBaaSv2 going to be deprecated to honor Octavia? Does LBaaSv2 support l7 policy? What does Octavia have more to offer than LBaaSv2 ? Thanks in advance.07:40
*** rcernin has quit IRC07:43
*** AlexeyAbashkin has joined #openstack-lbaas07:46
*** yamamoto has quit IRC07:52
nmagnezirm_work, o/07:58
openstackgerritSanthosh Fernandes proposed openstack/octavia master: [WIP] ACTIVE-ACTIVE ExaBGP rest api driver  https://review.openstack.org/52700907:58
openstackgerritSanthosh Fernandes proposed openstack/octavia master: [WIP] ACTIVE-ACTIVE ExaBGP rest api driver  https://review.openstack.org/52700907:59
*** yamamoto has joined #openstack-lbaas08:06
*** b_bezak has joined #openstack-lbaas08:13
*** Alex_Staf has joined #openstack-lbaas08:45
*** rcernin has joined #openstack-lbaas08:47
*** eranra_ has joined #openstack-lbaas08:48
*** eranra_ has left #openstack-lbaas08:48
*** aojea has joined #openstack-lbaas09:19
*** aojea has quit IRC09:24
*** sticker has quit IRC09:25
*** sri_ has joined #openstack-lbaas09:31
*** yamamoto has quit IRC09:33
*** yamamoto has joined #openstack-lbaas09:34
Alex_Stafkong, ping09:36
*** eN_Guruprasad_Rn has quit IRC09:39
*** eN_Guruprasad_Rn has joined #openstack-lbaas09:46
*** dokua has joined #openstack-lbaas09:55
*** codenamelxl has quit IRC10:29
*** annp has quit IRC10:35
*** yamamoto has quit IRC10:47
*** yamamoto has joined #openstack-lbaas10:47
*** yamamoto has quit IRC10:52
*** kobis has joined #openstack-lbaas11:24
*** AlexeyAbashkin has quit IRC11:25
*** eN_Guruprasad_Rn has quit IRC11:37
*** ianychoi has quit IRC11:37
*** AlexeyAbashkin has joined #openstack-lbaas11:45
*** ianychoi has joined #openstack-lbaas11:50
*** sanfern has joined #openstack-lbaas11:52
*** yamamoto has joined #openstack-lbaas12:02
openstackgerritBernard Cafarelli proposed openstack/octavia master: Force SELinux context for amphora keepalived proces  https://review.openstack.org/52707312:18
*** eN_Guruprasad_Rn has joined #openstack-lbaas12:29
*** atoth has joined #openstack-lbaas12:37
*** eN_Guruprasad_Rn has quit IRC12:40
*** sanfern has quit IRC12:41
*** sanfern has joined #openstack-lbaas12:42
*** rcernin has quit IRC13:23
*** ianychoi has quit IRC13:40
*** bar_ has joined #openstack-lbaas14:01
*** sanfern has quit IRC14:41
*** sanfern has joined #openstack-lbaas14:42
*** yamamoto has quit IRC14:53
*** dmellado has quit IRC14:53
*** b_bezak has quit IRC14:59
*** b_bezak has joined #openstack-lbaas15:00
*** b_bezak has quit IRC15:04
*** dmellado has joined #openstack-lbaas15:05
openstackgerritBernard Cafarelli proposed openstack/octavia master: Force SELinux context for amphora keepalived process  https://review.openstack.org/52707315:05
*** yamamoto has joined #openstack-lbaas15:06
*** dmellado has quit IRC15:10
*** yamamoto has quit IRC15:12
*** dmellado has joined #openstack-lbaas15:12
*** Alex_Staf has quit IRC15:19
*** dmellado has quit IRC15:29
*** dmellado has joined #openstack-lbaas15:30
*** fnaval has joined #openstack-lbaas15:41
*** gcheresh has quit IRC15:46
*** dmellado has quit IRC15:52
*** yamamoto has joined #openstack-lbaas15:52
*** yamamoto has quit IRC15:54
*** yamamoto has joined #openstack-lbaas15:54
*** yamamoto has quit IRC15:54
*** dmellado has joined #openstack-lbaas15:55
*** armax has joined #openstack-lbaas15:56
*** dmellado has quit IRC15:58
*** dmellado has joined #openstack-lbaas16:01
xgerman_o/16:09
*** kobis has quit IRC16:18
*** AlexeyAbashkin has quit IRC16:43
*** kobis has joined #openstack-lbaas16:50
*** yamamoto has joined #openstack-lbaas16:54
sanferno/16:55
sanfernhello johnsom, Good Morning16:55
johnsomsanfern Hi16:55
sanfernregarding amphora image patch I am using exabgp cli so I can not disable it - https://review.openstack.org/#/c/491016/12/octavia/amphorae/backends/agent/api_server/exabgp.py16:56
sanfernThank you for review16:56
johnsomOk, yeah, I haven't reviewed the other code yet, just the image element.16:57
sanfernok16:57
johnsomThe documentation for exabgp is a bit light....16:57
sanfernyeah I agree. I have tested agent part and demo'ed to the team using curl, it worked pretty well.16:59
sanfernpip install is not required since we have in requirements.txt17:00
johnsomYeah, I remember you mentioning that.  That is good news, I am just trying to understand everything going on there and making sure we do a good implementation.17:00
sanfernhttps://review.openstack.org/#/c/490164/21/elements/exabgp-speaker/post-install.d/77-exabgp17:00
sanfernThanks johnsom,17:00
*** yamamoto has quit IRC17:03
*** tongl has joined #openstack-lbaas17:04
*** sanfern has quit IRC17:20
*** dokua has quit IRC17:22
*** tongl has quit IRC17:37
*** dmellado has quit IRC17:40
*** dmellado has joined #openstack-lbaas17:40
*** sanfern has joined #openstack-lbaas17:48
*** kobis has quit IRC17:53
*** dmellado has quit IRC17:59
openstackgerritBar RH proposed openstack/octavia master: Improve Neutron driver _get_resource()  https://review.openstack.org/52719918:05
*** bar_ has quit IRC18:08
*** dmellado has joined #openstack-lbaas18:09
*** SumitNaiksatam has quit IRC18:10
*** Alex_Staf has joined #openstack-lbaas18:12
Alex_StafHi guys , I have devstack with octavia and tempest . testr list-tests see the octavia tests . am I ready to go or should I know anything else ?18:16
johnsomI think you are good.  I usually set the TEMPEST_CONFIG_DIR=/opt/stack/tempest/etc variable18:18
*** tongl has joined #openstack-lbaas18:19
*** kobis has joined #openstack-lbaas18:19
*** AlexeyAbashkin has joined #openstack-lbaas18:28
*** kobis has quit IRC18:29
*** eN_Guruprasad_Rn has joined #openstack-lbaas18:30
*** kobis has joined #openstack-lbaas18:31
*** AlexeyAbashkin has quit IRC18:32
*** SumitNaiksatam has joined #openstack-lbaas18:43
*** gcheresh has joined #openstack-lbaas18:52
*** kobis has quit IRC19:00
*** AlexeyAbashkin has joined #openstack-lbaas19:03
*** AlexeyAbashkin has quit IRC19:07
*** eN_Guruprasad_Rn has quit IRC19:10
*** dmellado has quit IRC19:23
*** dmellado has joined #openstack-lbaas19:25
*** Alex_Staf has quit IRC19:28
*** gcheresh has quit IRC19:41
*** dmellado has quit IRC19:45
*** tongl has quit IRC19:51
*** dmellado has joined #openstack-lbaas19:51
*** dmellado has quit IRC19:51
*** kobis has joined #openstack-lbaas19:54
*** dmellado has joined #openstack-lbaas19:57
*** dmellado has quit IRC19:57
*** kobis has quit IRC20:01
*** dmellado has joined #openstack-lbaas20:03
*** dmellado has quit IRC20:04
*** tongl has joined #openstack-lbaas20:04
*** dmellado has joined #openstack-lbaas20:11
*** dmellado has quit IRC20:11
*** dmellado has joined #openstack-lbaas20:13
*** dmellado has quit IRC20:14
*** atoth has quit IRC20:15
*** Swami has joined #openstack-lbaas20:19
*** dmellado has joined #openstack-lbaas20:24
*** dmellado has quit IRC20:24
*** aojea has joined #openstack-lbaas20:29
*** dmellado has joined #openstack-lbaas20:34
*** aojea has quit IRC20:35
*** dmellado has quit IRC20:52
*** gcheresh has joined #openstack-lbaas20:58
*** tongl has quit IRC21:01
*** sanfern has quit IRC21:06
*** dmellado has joined #openstack-lbaas21:17
*** bar_ has joined #openstack-lbaas21:23
*** eN_Guruprasad_Rn has joined #openstack-lbaas21:28
*** dmellado has quit IRC21:28
*** dmellado has joined #openstack-lbaas21:31
*** sanfern has joined #openstack-lbaas21:39
*** threestrands has joined #openstack-lbaas21:41
*** threestrands has quit IRC21:41
*** threestrands has joined #openstack-lbaas21:41
rm_worknmagnezi: keep missing you >_>21:45
rm_worknmagnezi: maybe i'll try to catch you tonight21:45
*** SumitNaiksatam_ has joined #openstack-lbaas21:49
*** strigazi_ has joined #openstack-lbaas21:51
*** tongl has joined #openstack-lbaas21:52
*** mestery_ has joined #openstack-lbaas21:54
*** b3nt_pin has joined #openstack-lbaas21:54
*** gcheresh has quit IRC21:55
*** dokua has joined #openstack-lbaas21:56
*** SumitNaiksatam has quit IRC21:56
*** dayou has quit IRC21:56
*** mestery has quit IRC21:56
*** strigazi has quit IRC21:56
*** beagles has quit IRC21:56
*** SumitNaiksatam_ is now known as SumitNaiksatam21:56
*** mestery_ is now known as mestery21:56
rm_workxgerman_ / nmagnezi https://review.openstack.org/#/c/525295/21:57
rm_workeasy merge21:58
*** dmellado has quit IRC21:58
rm_workjohnsom / xgerman_: could use review again: https://review.openstack.org/#/c/525790/21:58
rm_workthe next two in that chain could as well21:58
rm_workbut i figure begging for two small reviews per day is probably enough :P21:59
xgerman_the 2nd one still has a -121:59
*** dayou has joined #openstack-lbaas21:59
*** dayou has quit IRC21:59
*** dayou has joined #openstack-lbaas21:59
rm_workyeah and it is invalid IMO21:59
rm_workI detailed why21:59
rm_workbar_: would be good if you could re-review21:59
bar_looking22:00
xgerman_ok22:00
bar_rm_work, You didn't change the code since we talked, only rebased it22:00
rm_workyes22:00
rm_worki believe your original comment is invalid -- it does not require a change on my part22:01
rm_worki provided some examples22:01
rm_workor if not invalid, not explicitly valid either22:01
rm_worknot worthy of a -1, at the least22:01
bar_rm_work, I see. Then we do not understand each other, because I think your example proves otherwise.22:02
rm_workif you saw the gist i linked, *you are technically correct* per line 27-28 that it doesn't get filled there22:02
rm_workbut it isn't ONLY a class variable22:02
rm_workit is also carried into instances22:02
rm_workuniquely22:02
rm_workwhich is what the rest of that code proves22:03
*** dokua has quit IRC22:03
bar_But why declare a class member in the first place? just declare it inside an __init__22:03
rm_workand it doesn't really matter if it's forever null on the class ... because ... no one uses that22:03
bar_I agree that your code *works*, but you shouldn't declare a class member if you never write to it.22:04
bar_Don't you agree?22:04
rm_workI think messing with the __init__ when you can just declare a var there is wasteful22:05
rm_workbut i think it comes down to style maybe22:05
rm_workif you're going to keep your -1 until it's changed... i don't have any specific reason not to change it22:05
rm_workso i guess i would22:05
rm_workthat's why i adjusted what I said to "if not invalid, not explicitly valid or worthy of a -1"22:06
rm_worki mean it's a 15 second fix, probably arguing about it wasn't worth the effort I guess22:06
johnsomGerman already voted a +2.  I think it's a bit odd, but works.22:07
rm_workbar_: do you care enough that you really want me to roll another patch?22:07
*** dmellado has joined #openstack-lbaas22:08
rm_workI mean, partly I am thinking we're looking at *making it* a class variable, and we're halfway there this way too, lol22:08
rm_workbut on the other hand, since it still isn't, maybe this has the potential to be confusing22:08
bar_rm_work, lol, the answer is yea :-)22:08
rm_workkk22:08
*** eN_Guruprasad_Rn has quit IRC22:08
bar_You declare a class member and then assign to an instance member you should have declared of instead, I think it deserves -1 :-)22:09
openstackgerritAdam Harwell proposed openstack/octavia master: Move loading the network driver into the flows  https://review.openstack.org/52579022:09
rm_workthere you go, as stated, 15 second change22:09
rm_worklol22:09
*** dmellado has quit IRC22:10
rm_workpoint of me having this argument: probably 0%22:10
*** dmellado has joined #openstack-lbaas22:10
rm_workso really, I just wasted my own time >_<22:10
bar_rm_work, ha. you got a +122:11
rm_workjohnsom: now you :P22:11
rm_workah and xgerman_ again T_T22:11
bar_rm_work, Please take a look at my recent patches, I added you as a reviewer mostly because it might conflict with your patches, so your opinion is significant22:12
rm_workhmm k need to figure out which ones those are22:12
bar_top 2: https://review.openstack.org/#/q/project:openstack/octavia+owner:%22Bar+RH+%253Cbar%2540redhat.com%253E%2222:13
rm_workah already i am in the middle of reviewing the second one22:14
rm_workI honestly didn't care that the error messages say like "Qos_idNotFound" lol22:14
rm_workbut yeah, i can rebase22:14
rm_workno real worry there, these look minimal22:15
bar_rm_work, good then22:15
*** rcernin has joined #openstack-lbaas22:19
rm_workbar_: although you do appear to be reordering args for ... no real purpose that I can see?22:22
rm_workthey're all used as keyword args anyway22:22
rm_workthe order doesn't matter in any of these cases22:22
rm_work(the way taskflow calls things is explicitly by keyword, not by ordered args)22:22
rm_workI guess the whole purpose of this patch is just a cleanup refactor22:23
rm_workso ... the intent is to make them somehow more logically ordered?22:23
bar_rm_work, I wrote it under the impression that order could matter, I guess there isn't really a case for it other than aesthetics, as you say...22:24
johnsomOrder doesn't matter22:24
bar_johnsom, yeah, well... You can -2 if you feel it doesn't make sense to merge it.22:25
bar_johnsom, oh, you already approved, thx22:25
johnsomArgh, I was looking at one of the L3 Act/Act patches, stacked, then realized it's parent is so out of date it doesn't run...  Sigh.22:27
openstackgerritMichael Johnson proposed openstack/octavia master: [WIP] ACTIVE-ACTIVE with exabgp-speaker - Octavia agent  https://review.openstack.org/49101622:27
rm_workbar_: was just making sure you were just doing cleanup and hadn't intended to make any actual change22:28
bar_about the qos policy octavia patch, is it a bug that it does not propagate the qos_policy applied on the VIP to the port object?22:28
rm_workit's fine, merging it now22:28
bar_rm_work, thx!22:28
rm_worklooking at the other22:28
nmagnezirm_work, o/ yeah,, we need to discuss some stuff as a follow up to the last weekly meeting. I was partially OOO and came back today and had some internal stuff to handle. tomorrow morning my time I'm going to re-visit the providers spec (I know xgerman_ made some comments on it)22:29
nmagnezibar_, you are up late :-)22:29
bar_nmagnezi, :-) you too, it seems22:30
johnsombar_ What do you mean on the QoS patch?22:30
bar_johnsom, https://review.openstack.org/#/q/project:openstack/octavia+owner:%22Bar+RH+%253Cbar%2540redhat.com%253E%2222:31
bar_woops22:31
*** dmellado has quit IRC22:31
bar_johnsom, https://review.openstack.org/#/c/458308/22:31
johnsomYeah, I have looked through that patch, was just trying to understand your concern22:32
bar_johnsom, he never updates the port with the chosen qos_policy22:33
johnsomI'm not following, he has a task that applies the qos ID to the neutron port.22:34
bar_johnsom, which he never applies on the vip_port. assume a port was given in creation of the loadbalancer, and later the qos_policy had been updated via loadbalancer set call22:36
*** dmellado has joined #openstack-lbaas22:36
johnsombar_ Maybe you should post a review vote on the patch.22:38
bar_johnsom, I will. My deployment is down for a while, so I haven't reproduced it yet.22:39
rm_workbar_: couple comments on https://review.openstack.org/#/c/527199/122:40
bar_rm_work, thanks! Did you notice I changed the function name?22:42
rm_workah for the thing on the tests... if those objects are missing something ... check to see if I already added it in https://review.openstack.org/#/c/525353/6/octavia/tests/common/constants.py22:42
johnsombar_ Ok.  If the concern is that he is applying it to the base port and not the VIP port, I think that is accurate to how neutron works.  The VIP port is a fake "allowed-address-pairs" port. I haven't loaded it up yet, but I have completed the code read through.  I plan to test it today and vote.22:42
rm_workbar_: ah actually i did not, tricksy 's' in the middle there22:43
bar_johnsom, Does he apply it on the base port? I haven't find that call...22:43
rm_workbar_: so ... ok... while I am not a huge fan of the old method disappearing, it *is* "private" so there's no contract and anyone using that directly is at their own risk... and at least it breaks cleanly22:43
rm_workso yeah, I'm less concerned22:43
bar_rm_work, yay!22:44
rm_workmy comment on the test still applies :P22:45
rm_workI think I fixed the MOCK objects for you in my patch22:45
rm_workbut that may not merge for a while, so...22:45
rm_workif you want to take my changes to those objects and try to put them in the same way, it'd make my rebase a lot easier when yours merges first :)22:46
bar_rm_work, what patch are you referring to?22:46
rm_workI linked it just now, but: https://review.openstack.org/#/c/525353/6/octavia/tests/common/constants.py22:46
rm_workIf you fixed MOCK_NEUTRON_PORT like that, and added a second one with ID2/IP222:46
rm_workyou can just use those directly22:46
*** dmellado has quit IRC22:48
bar_rm_work, MOCK_NEUTRON_PORT -> MOCK_NEUTRON_PORT_1, cool?22:51
rm_workhmm22:52
rm_workjust looking at the rest of that file, mostly we don't underscore the number22:52
bar_oh22:52
rm_workand OFTEN we don't rename the #1, though that is somewhat inconsistent22:52
bar_which do you prefer, with #1 or without?22:53
rm_workI think without, just because it makes more sense for the tests where we don't care about multiple (which is almost all of them)22:53
rm_workand of course it makes my rebase easier :P22:53
bar_rm_work, done22:53
rm_workwaiting for my two patches to merge and your two patches to merge and then i'll rebase all my stuff >_<22:58
rm_workhopefully not too bad22:58
rm_workso whenever you post that one I can +222:58
*** threestrands_ has joined #openstack-lbaas23:04
*** threestrands_ has quit IRC23:04
*** threestrands_ has joined #openstack-lbaas23:04
rm_worksoooo, we do need to look at the PKCS12 thing before Q-3 I guess: https://review.openstack.org/#/c/504175/23:05
rm_workIdeally sooner so I have a chance to make changes if necessary...23:05
rm_workI'm prolly going to be out the last two weeks of December23:06
xgerman_ok23:06
*** dmellado has joined #openstack-lbaas23:06
*** threestrands has quit IRC23:06
johnsomYeah, I think we should work on reviewing that.  I wrote it into the driver spec...  grin23:08
xgerman_rm_work: Q what happend with an existing system, e,g, where somebody didn’t stire pkcs12 certs in BBQ — dod they need to mugrate before that patch?23:08
rm_worknope23:09
rm_workso, it is designed to allow for both23:09
rm_workit will try to use the new system first (use the ref to get a secret), but if that fails, it will fall-back to trying the old driver23:10
rm_workif THAT fails, it will throw an error as it should23:10
rm_workI tried to make it the least pain possible for people already migrating23:10
rm_workoh, but I WAS going to adjust this patch so it will do the ACLs for you >_>23:10
bar_rm_work, does L69 in constants (above) make sense to you?23:11
rm_workshould I do that? :thumbsup: or :thumbsdown:23:11
rm_workbar_: in MY patch?23:11
rm_workor yours23:11
bar_In master I presume23:11
rm_workah, device_id = MOCK_COMPUTE_ID?23:11
bar_yeah, there is MOCK_DEVICE_ID there...23:11
rm_workyeah the "device" is the compute instance I thought23:11
rm_workhmmmm that is interesting tho23:11
rm_workjohnsom: when we get back a device_id on a port, is that referring to the VM it's bound to?23:12
rm_worki can check that actually, one sec23:12
johnsomDon't trust it23:12
rm_worki mean this is for a test23:13
rm_workwe literally just ... named the thing possibly wrong23:13
rm_workbut i think it's fine23:13
rm_workyes23:13
rm_workdevice_id on a port is the server_id for what it's bound to23:13
rm_workso in that mock, it makes sense, bar_23:13
rm_worki mean it's a little weird still to name it like that when it's all just a mock setup23:14
rm_workbut it IS logically the truth23:14
johnsomBut in real life that field can be unpredictable23:14
johnsomJust FYI23:14
rm_workk23:14
rm_workit seems to work in our env23:14
openstackgerritMerged openstack/octavia master: Update compute noop driver/tests (many were disabled/useless)  https://review.openstack.org/52529523:19
*** jappleii__ has joined #openstack-lbaas23:20
*** jappleii__ has quit IRC23:21
*** jappleii__ has joined #openstack-lbaas23:22
*** dmellado has quit IRC23:22
bar_rm_work, ports still use tenant_id?23:22
rm_workyes :/23:22
rm_workAFAICT23:22
*** threestrands_ has quit IRC23:22
rm_workcan you do a get on one in your cloud and check?23:23
bar_rm_work, unfortunately not today. It's down.23:23
rm_workthat's still how we're loading them in via the methods23:23
rm_workhttps://review.openstack.org/#/c/525353/6/octavia/network/drivers/neutron/utils.py23:23
rm_workexpand up a bit and see line 4823:23
rm_worki tested in my cloud and verified that is what we get back, but my cloud is very old :P so I can't say for sure it's true in Pike23:24
rm_workbut, if Pike returns something else, then we've got a bug23:24
bar_rm_work, couldn't it return both?23:25
rm_workmaybe23:25
rm_workin which case, we could clean it up and even future-proof by just *looking* for either23:25
rm_workfeel free :P23:25
johnsomI'm stacking, I can look in a few23:25
rm_worki'm still waiting for you to push your other change tho23:26
bar_johnsom, thx, looks like my machine is back, too. It would take it a while, though...23:26
bar_rm_work, I know.23:26
openstackgerritBar RH proposed openstack/octavia master: Improve Neutron driver _get_resource()  https://review.openstack.org/52719923:27
bar_rm_work, note that sleazy MOCK_SUBNET223:27
rm_worklol23:27
openstackgerritBar RH proposed openstack/octavia master: Improve Neutron driver _get_resource()  https://review.openstack.org/52719923:27
rm_worksure, seems fine23:30
johnsomshow port has project_id and tenant_id23:30
bar_johnsom, thx23:32
*** dmellado has joined #openstack-lbaas23:36
openstackgerritMerged openstack/octavia master: Reorder ComputeCreate arguments  https://review.openstack.org/52696523:46
*** sshank has joined #openstack-lbaas23:47
*** johnsom has quit IRC23:56
*** amitry has quit IRC23:57
*** AlexeyAbashkin has joined #openstack-lbaas23:58
*** johnsom has joined #openstack-lbaas23:58
« Sunday, 2017-12-10 Index Tuesday, 2017-12-12 »

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