Monday, 2017-11-27

openstackgerritBar RH proposed openstack/python-octaviaclient master: Complement Octavia client with a set of features  https://review.openstack.org/52266600:27
openstackgerritBar RH proposed openstack/python-octaviaclient master: Complement Octavia client with a set of features  https://review.openstack.org/52266600:37
openstackgerritBar RH proposed openstack/python-octaviaclient master: Complement Octavia client with a set of features  https://review.openstack.org/52266600:38
*** yuanying has quit IRC00:59
*** bar_ has quit IRC01:03
*** yuanying has joined #openstack-lbaas01:12
*** yamamoto has joined #openstack-lbaas01:29
sapd_When I use a normal user with user role, I can't get loadbalancer in octavia using this command "openstack loadbalancer list". With the error: "Forbidden (HTTP 403) (Request-ID: req-1681ffa9-8622-4712-b827-c8d6b2034a25)"02:30
*** sam_dw has quit IRC02:31
johnsomo/02:31
johnsomSo, by default Octavia has RBAC enabled and users need to be a member of the loadbalancer member role02:32
johnsomThere is a doc that explains this in the octavia documentation under configuration and policy config02:32
johnsomThere is also a sample policy.json file in the etc folder that disables the “new” OpenStack RBAC scheme and sets it back to admin and owner02:33
johnsomAlso, fyi, the Octavia API reference describes those error codes and would have pointed you towards the policy config02:35
johnsomsapd_: https://docs.openstack.org/octavia/pike/configuration/policy.html02:38
sapd_johnsom, I can't find policy.json in etc folder. Could you share it for me?02:47
johnsomsapd_: https://github.com/openstack/octavia/blob/master/etc/policy/admin_or_owner-policy.json02:48
*** threestrands_ has joined #openstack-lbaas02:49
*** threestrands_ has quit IRC02:49
*** threestrands_ has joined #openstack-lbaas02:49
*** threestrands_ has quit IRC02:50
*** threestrands has quit IRC02:50
*** threestrands_ has joined #openstack-lbaas02:50
*** threestrands_ has quit IRC02:50
*** threestrands_ has joined #openstack-lbaas02:50
sapd_Thanks johnsom,. So Octavia still have not finished code with "new" Openstack RBAC scheme.02:53
johnsomNo, Octavia is complete for RBAC02:54
johnsomYou just need to give your users the role for the “new” as described on the docs page02:56
*** atoth_ has quit IRC03:00
sapd_Thanks johnsom , Awesome :D03:06
*** links has joined #openstack-lbaas04:26
*** yamamoto has quit IRC04:43
*** yamamoto has joined #openstack-lbaas04:49
*** longkb has joined #openstack-lbaas05:08
longkbHi everyone,05:08
longkbI am deploying devstack which include neutron-loadbancer with octavia. However, when I tried to create a loadbalancer with command "neutron lbaas-loadbalancer-create --name lb subnet_mgmt", the creation status always is "PENDING_CREATE"05:11
longkbCould anyone help me to figure out what is an error? Thank you very much05:11
johnsomWhat are you running devstack on?  My bet is virtualbox05:13
longkbI am using vmware05:13
longkbIs there any difference between vmware and virtualbox?05:14
johnsomOk good. Usually this is caused by virtualbox taking 10+ minutes to boot a vm05:15
johnsomIn vmware, make sure your CPU config has virtualization enabled05:15
johnsomIn workstation it is under settings, cpu, a checkbox for vt-x05:16
longkbYes, I enabled the CPU virtualization.05:16
longkbI can create VM sucessfully05:16
johnsomOk, so the typical issue is ruled out05:16
johnsomYeah, if that isn’t on it can take 10+ minutes for the LB to go active05:17
johnsomSo, next step would be to do a nova list to see it the amphora is booted up05:18
longkbYes, I have checked. I amphora is still ACTIVE05:19
johnsomNext, check the octavia worker process log.  Assuming you are running master branch devstack that is “journalctl -u devstack@o-cw | less”05:20
longkbI got an error05:21
johnsomI would also note, with pike or newer you no longer need neutron-lbaas05:21
longkbCould you explain more about "no longer need neutron-lbaas" :)05:22
johnsomNeutron-lbaas is heading towards deprecation. Octavia is replacing it05:22
johnsomOctavia had it’s own API endpoit05:23
longkboh. I see.05:23
johnsomCan you pastebin your error so I can look at it for you?05:24
longkbThis is my error05:25
longkb"WARNING octavia.amphorae.drivers.haproxy.rest_api_driver [-] Could not connect to instance. Retrying.: ConnectionError: HTTPSConnectionPool(host='192.1 68.0.12', port=9443): Max retries exceeded with url: /0.5/plug/vip/192.168.120.10 (Caused by NewConnectionError('<urllib3.connection.VerifiedHTTPSConnection object at 0x7fb3b5aea3d0>: Failed to esta blish a new connection: [Errno 113] No route to host',))"05:25
longkbplease help me to fix it!05:27
johnsomSo, those warnings are normal while Octavia waits for nova/qemu to finish booting the vm.  It could also mean there is a network configuration issue which would be very odd with devstack.05:28
longkbas I can see. My o-cw thread went to an infinitive loop05:30
johnsomCan you do a “nova list | grep amphora”, look at the lb-mgmt-net IP address.  Make sure this does not conflict with the network your vmware is using.05:31
johnsomNo infinite loop, it will keep trying connect to the VM until the timeout expires, then it will go to ERROR status05:32
*** sticker has quit IRC05:33
longkbmy lb-mgmt-net address is 192.168.0.1205:34
longkbAnd there is no existing network is confict with lb-mgmt-net05:34
johnsomHmmm.  Ok, get the nova uuid for the instance.  Cd to /opt/stack/data/nova/instances/<uuid>/05:36
johnsomThere is a console.log file, look at the bottom of the file.  It should be a login prompt if nova/qemu finished booting it05:37
longkbI got "Permission denided"05:38
johnsomSudo05:39
longkbthis console.log is empty05:39
*** eN_Guruprasad_Rn has joined #openstack-lbaas05:40
johnsomSo, nova is broken.05:40
*** eN_Guruprasad_Rn has joined #openstack-lbaas05:40
longkbshould I restart nova service?05:41
johnsomThat log should be huge05:41
johnsomTry “openstack server show <uuid>”05:42
johnsomThere is usually an error message section for the instance05:42
*** yamamoto has quit IRC05:44
longkbhttps://drive.google.com/file/d/1v7djaiNP_Y6ILq5zbuDJ35ofakBCd-48/view?usp=sharing05:45
longkbThe instance stattus is still ACTIVE05:45
longkbYou can see in the link05:46
*** yamamoto has joined #openstack-lbaas05:48
johnsomCan you run “top” and see if qemu is at 100% cpu usage?06:00
longkbit over 100%06:03
longkbIt is 132% CPU06:03
johnsomOk, I know the problem.06:04
johnsomThe qemu is for a newer version than the OS you are running and you need to set a nova setting to make qemu boot a vm06:05
johnsomOne minute, I will post the setting06:06
longkbThank you very much06:06
*** Alex_Staf has joined #openstack-lbaas06:06
johnsomAssuming you are using ubuntu xenial for your devstack host:06:08
johnsomhttps://www.irccloud.com/pastebin/z7E7oUTq/06:08
johnsomThis is my "fixnova.sh"06:08
johnsomOnce that runs, you can create a new load balancer and it will come up.06:09
johnsomThe current one will need to go into ERROR status, then you can delete it.06:09
johnsomIt probably already is in ERROR by now.06:09
johnsomThe 100% CPU is the clue that qemu didn't actually boot the instance and it's just CPU spinning06:11
longkbIn my devstack, there is no file source /opt/stack/devstack/inc/ini-config06:12
johnsomHmm06:13
johnsomWell, you can manually edit the files06:13
johnsomit's file: /etc/nova/nova.conf, section [libvirt] key hw_machine_type value x86_64=pc-i440fx-xenial06:14
johnsomSame for the other file06:14
johnsomThe source line just pulls in the the "iniset" command06:15
longkbThanks you very much06:20
longkbI can create loadbalancer sucessfully with your instruction06:20
johnsomExcellent! I hope you enjoy Octavia06:21
longkbThanks for your helping06:22
longkbI am implementing a bp that related to clustering serivce for VNF in Tacker06:22
johnsomTime for me to sign off for the night.  Catch you tomorrow06:23
longkbThank you very much! Johnsom ^_^06:23
longkbgood nice!06:23
longkbgood night!06:23
*** annp has joined #openstack-lbaas06:39
*** spectr has joined #openstack-lbaas06:44
*** rcernin has quit IRC06:47
*** rcernin has joined #openstack-lbaas06:49
*** spectr has quit IRC06:55
*** rcernin has quit IRC07:28
*** AlexeyAbashkin has joined #openstack-lbaas07:57
*** aojea has joined #openstack-lbaas08:01
*** yamamoto has quit IRC08:04
*** aojea has quit IRC08:06
*** pcaruana has joined #openstack-lbaas08:11
openstackgerritNir Magnezi proposed openstack/octavia master: [DO NOT MERGE]: CI TEST  https://review.openstack.org/52305308:19
*** slaweq has joined #openstack-lbaas08:24
*** kobis has joined #openstack-lbaas08:24
*** yamamoto has joined #openstack-lbaas08:33
*** yamamoto has quit IRC08:51
openstackgerritZhaoBo proposed openstack/octavia master: Support UDP load balance  https://review.openstack.org/50360609:15
*** yamamoto has joined #openstack-lbaas09:31
*** spectr has joined #openstack-lbaas09:37
*** spectr has quit IRC09:37
*** bar_ has joined #openstack-lbaas09:38
*** yamamoto has quit IRC09:39
*** annp has quit IRC10:03
*** spectr has joined #openstack-lbaas10:22
*** spectr has quit IRC10:23
*** yamamoto has joined #openstack-lbaas10:27
*** yamamoto has quit IRC10:32
*** yamamoto has joined #openstack-lbaas10:53
*** yamamoto has quit IRC10:55
*** pcaruana has quit IRC10:55
*** cgoncalves has quit IRC10:56
*** cgoncalves has joined #openstack-lbaas10:57
*** cgoncalves has quit IRC10:57
*** cgoncalves has joined #openstack-lbaas10:58
*** pcaruana has joined #openstack-lbaas10:59
*** salmankhan has joined #openstack-lbaas10:59
*** yamamoto has joined #openstack-lbaas11:01
openstackgerritBernard Cafarelli proposed openstack/octavia master: Allow amphora agent install from distribution packages  https://review.openstack.org/52262611:03
*** sri_ has joined #openstack-lbaas11:06
*** yamamoto has quit IRC11:06
*** salmankhan has quit IRC11:10
*** yamamoto has joined #openstack-lbaas11:31
*** spectr has joined #openstack-lbaas11:35
*** spectr has quit IRC11:35
*** yamamoto has quit IRC11:36
*** threestrands_ has quit IRC11:42
*** atoth has joined #openstack-lbaas11:49
bcafarelnmagnezi: don't be surprised if you build a CentOS/RDO amphora with ^, it won't do much lbaas as selinux is enabled with the '-p' flag11:50
*** kong has quit IRC11:53
*** ying_zuo has quit IRC11:53
*** fyxim has quit IRC11:54
*** xgerman_ has quit IRC11:54
*** redondo-mk has quit IRC11:54
*** kong has joined #openstack-lbaas11:55
*** fyxim has joined #openstack-lbaas11:56
*** redondo-mk has joined #openstack-lbaas11:56
*** xgerman_ has joined #openstack-lbaas11:56
*** yamamoto has joined #openstack-lbaas12:01
*** yamamoto has quit IRC12:06
*** links has quit IRC12:29
*** links has joined #openstack-lbaas12:43
*** yamamoto has joined #openstack-lbaas12:47
*** yamamoto has quit IRC12:51
*** krypto has joined #openstack-lbaas13:19
*** krypto has quit IRC13:19
*** krypto has joined #openstack-lbaas13:19
*** eN_Guruprasad_Rn has quit IRC13:41
*** yamamoto has joined #openstack-lbaas13:45
*** yamamoto has quit IRC13:45
*** yamamoto has joined #openstack-lbaas13:49
*** ying_zuo has joined #openstack-lbaas13:51
*** yamamoto has quit IRC13:57
*** fnaval has joined #openstack-lbaas14:16
*** links has quit IRC14:32
*** krypto has quit IRC14:35
openstackgerritBar RH proposed openstack/octavia master: Rename 'remove' to 'reset' in Quota API  https://review.openstack.org/52240214:53
*** bar_ has quit IRC14:55
*** salmankhan has joined #openstack-lbaas14:57
*** yamamoto has joined #openstack-lbaas14:57
*** yamamoto has quit IRC15:04
*** gcheresh has quit IRC15:12
xgerman_o/15:13
*** Alex_Staf has quit IRC15:13
*** salmankhan has quit IRC15:34
*** salmankhan has joined #openstack-lbaas15:51
*** slaweq has quit IRC15:58
*** slaweq has joined #openstack-lbaas15:59
dayouhttps://review.openstack.org/#/c/522962/15:59
*** slaweq has quit IRC15:59
*** slaweq has joined #openstack-lbaas16:00
dayouIt seems there are some mis alignment between our v2 api and the openstack sdks16:00
*** slaweq has quit IRC16:00
dayouNot sure there are more of this and my way to fix it is correct or not16:00
*** slaweq has joined #openstack-lbaas16:00
*** kobis has quit IRC16:01
*** slaweq has quit IRC16:05
*** bbzhao has quit IRC16:09
*** bzhao has quit IRC16:10
*** bzhao has joined #openstack-lbaas16:14
*** bbzhao has joined #openstack-lbaas16:15
johnsomdayou No, someone mistakenly added neutron-lbaas objects to openstacksdk.  We just need to delete them16:16
johnsomdayou Oh, whoops. Monday morning. You are correct, I was thinking openstackclient not sdk.16:19
johnsomIgnore me until I get my coffee16:19
*** longstaff has joined #openstack-lbaas16:35
*** eN_Guruprasad_Rn has joined #openstack-lbaas16:52
*** salmankhan has quit IRC16:59
*** AlexeyAbashkin has quit IRC16:59
*** slaweq has joined #openstack-lbaas17:08
*** pcaruana has quit IRC17:09
*** slaweq has quit IRC17:13
nmagnezijohnsom, o/17:21
johnsomnmagnezi o/17:21
nmagnezijohnsom, tested your patch again against CentOS and it looked okay to me17:21
nmagnezibut not sure what makes it to fail CI17:22
johnsomOk, I will take a look.  It was passing last week....17:22
johnsomHa, yeah, that is odd, passes on centos, but fails on ubuntu17:25
*** harlowja has joined #openstack-lbaas17:47
*** armax has joined #openstack-lbaas17:53
*** longstaf_ has joined #openstack-lbaas17:59
*** longstaff has quit IRC18:03
johnsomnmagnezi If you have a few minutes: https://review.openstack.org/#/c/517455/18:05
johnsomI know there is a user looking forward to that patch18:05
*** sshank has joined #openstack-lbaas18:17
*** sshank has quit IRC18:24
*** eN_Guruprasad_Rn has quit IRC18:25
*** sshank has joined #openstack-lbaas18:31
*** sshank has quit IRC18:33
*** salmankhan has joined #openstack-lbaas18:33
*** sshank has joined #openstack-lbaas18:33
*** sshank has quit IRC18:33
*** sshank has joined #openstack-lbaas18:34
*** sshank has quit IRC18:34
*** sshank has joined #openstack-lbaas18:34
*** longstaff has joined #openstack-lbaas18:35
*** sshank has quit IRC18:37
*** longstaf_ has quit IRC18:38
*** sshank has joined #openstack-lbaas18:40
*** AlexeyAbashkin has joined #openstack-lbaas18:52
*** gcheresh has joined #openstack-lbaas18:52
*** AlexeyAbashkin has quit IRC18:56
*** longstaff has quit IRC19:01
*** longstaff has joined #openstack-lbaas19:02
*** pcaruana has joined #openstack-lbaas19:16
*** AlexeyAbashkin has joined #openstack-lbaas19:17
*** sanfern has joined #openstack-lbaas19:22
*** AlexeyAbashkin has quit IRC19:22
*** bar_ has joined #openstack-lbaas19:34
*** sshank has quit IRC19:35
openstackgerritSanthosh Fernandes proposed openstack/octavia master: [WIP] ACTIVE-ACTIVE rename vrrp_ to frontend_  https://review.openstack.org/52113819:47
*** sanfern has quit IRC19:47
*** sanfern has joined #openstack-lbaas19:47
*** sanfern has quit IRC19:48
*** sanfern has joined #openstack-lbaas19:48
*** sanfern has quit IRC19:49
*** sanfern has joined #openstack-lbaas19:49
*** sanfern has quit IRC19:49
*** sanfern has joined #openstack-lbaas19:50
*** sanfern has quit IRC19:50
*** salmankhan has quit IRC19:54
*** atoth has quit IRC19:56
rm_workhey all :) back19:56
rm_workany status update?19:56
johnsomo/19:56
johnsomLots of patches up for review.  I'm just getting back into things.  Caught up on my 150 emails this morning...19:57
rm_worki JUST caught up on slack and IRC19:57
rm_workthough i'm not sure i've fully processed everything I read...19:57
johnsomI'm going to heat up some lunch and work on some reviews.19:59
rm_workk19:59
rm_worki'm going to make sure my deploy didn't explode while i was away19:59
*** AlexeyAbashkin has joined #openstack-lbaas20:17
*** slaweq has joined #openstack-lbaas20:19
*** slaweq has quit IRC20:20
*** slaweq has joined #openstack-lbaas20:20
*** AlexeyAbashkin has quit IRC20:21
xgerman_o/20:24
*** threestrands_ has joined #openstack-lbaas20:32
*** threestrands_ has quit IRC20:32
*** threestrands_ has joined #openstack-lbaas20:32
*** threestrands_ has quit IRC20:33
*** threestrands_ has joined #openstack-lbaas20:34
*** threestrands has joined #openstack-lbaas20:34
*** threestrands has quit IRC20:34
*** threestrands has joined #openstack-lbaas20:34
*** sshank has joined #openstack-lbaas20:43
*** threestrands_ has quit IRC20:44
openstackgerritAdam Harwell proposed openstack/octavia master: Optimize update_health process  https://review.openstack.org/50487520:49
openstackgerritAdam Harwell proposed openstack/octavia master: WIP: Floating IP Network Driver (spans L3s)  https://review.openstack.org/43561220:50
rm_work^^ rebases20:50
openstackgerritAdam Harwell proposed openstack/octavia master: Merge amphora-agent elements  https://review.openstack.org/51899220:54
openstackgerritAdam Harwell proposed openstack/octavia master: Merge amphora-agent elements  https://review.openstack.org/51899220:55
johnsomrm_work I think something went wrong  with that last change...20:57
rm_workhmmm20:57
rm_workthis install-type stuff is interesting20:57
rm_workreally we should just wipe out the rhel element i think, and fix it to use this20:57
johnsomThe build-essentials got dropped20:57
rm_workyeah i did that20:57
johnsomYes, that is my argument and -120:57
rm_workit doesn't need to be pkg-mapped20:57
johnsomAh20:57
rm_workas bernard said, i was overriding the default pkg-mapping20:58
rm_workwhich i didn't know was a thing20:58
johnsomGot it20:58
rm_workbut it is -- so trying to futz with build-essential is just a bad idea20:58
rm_workhmm though maybe I should have left it for the *specific* rhel7 distro20:58
johnsomYeah.  But I think the install type path is probably best for the pre-packaged bits20:58
rm_workyep that is awesome functionality i didn't know existed :P20:59
rm_workI might turn that patch over to them20:59
rm_workor just abandon and have them write the merge that way20:59
rm_workjohnsom: i'm pretty close on the gentoo thing, just need to do the osutils bits21:02
rm_worki think it will boot a lot faster21:02
rm_workand we'll get newer HAProxy from it21:02
rm_workI think21:02
xgerman_I think - famous last words21:03
rm_workbut not sure if i'll have time to finish in the next few weeks, sadly21:03
rm_worklol yes21:03
johnsomCool.21:03
johnsomFYI, Q2 is next week....21:04
xgerman_so soon…21:04
johnsomYeah, no kidding21:04
bar_hey, I suspect loadbalancer-failover is broken (loadbalancer is stuck in ERROR). Is there test/gate to prove me otherwise?21:11
*** sshank has quit IRC21:12
johnsombar_ It's here: https://review.openstack.org/#/c/501559/21:14
bar_johnsom, :-(21:14
johnsomWhat do the logs have to say?21:14
bar_I've just came across this while extending octavia client21:16
*** sshank has joined #openstack-lbaas21:17
bar_failover calls in v2 cause the loadbalancers to ERROR21:17
johnsomIt would be a very recent change if it is broken21:18
johnsomIf you post the o-cw and o-hm logs I will take a look21:18
bar_johnsom, cool. thanks21:19
rm_workjohnsom: that failover test doesn't really work with the upstream driver yet :/21:22
rm_worki haven't figured out how to make it work21:23
rm_workbecause i rely on knowing the state21:23
rm_workFLIP driver updates master/backup state in the DB21:23
rm_workthe main driver doesn't21:23
rm_workbut, is bar_ talking about "failovers" in general, or the Failover API calls?21:23
rm_workkp21:23
rm_work[=\[=21:23
rm_work[=21:23
rm_workp\=21:23
rm_workdamnit21:23
rm_worksomething just fell on my keyboard and rolled around >_<21:24
johnsomlol21:25
bar_rm_work, originally, I was asking about the failover protocol in general. I tried to trigger failover using v2 api call, and since the loadbalancer broke while the call itself was successful, I looked for tests21:25
rm_workhmmm21:27
rm_workin my experience, it seems to be working...21:27
rm_workbut i still need to test against the very latest21:27
bar_ok... it fails in cw due to ImmutableObject.21:30
johnsombar_ I really need to see the logs21:30
bar_I know21:30
bar_just a sec21:31
johnsomok21:31
bar_http://paste.openstack.org/show/627510/21:31
bar_sorry for keeping you waiting21:31
bar_johnsom, rm_work ^21:32
rm_workyep21:33
rm_workthat makes sense21:33
rm_workthat's what is supposed to happen21:33
rm_workwhen your LB is pending-something, it won't do a failover21:33
rm_workwere you in the middle of updating the LB?21:34
bar_no21:34
johnsomRight, some other thread has "ownership" of the load balancer at the moment21:34
johnsomHmm, brings up an interesting point though, maybe the revert to error should be after the TestLBStatusSetPendingInDB task so it doesn't get set to ERROR if some other thread has ownership.21:35
johnsomSince it was in PENDING it likely would have been set to another state from ERROR when it completed anyway, but.21:36
bar_sorry, do you believe it to be PENDING due to another action that is not specified in the log?21:38
johnsomIt should be there somewhere if you search for that LB ID21:39
bar_I created that LB, waited on the lines for it to go active, and then charged on it with the failover call21:39
bar_and it's very consistent... happened with all my LBs21:40
*** pcaruana has quit IRC21:41
*** gcheresh has quit IRC21:43
xgerman_cores with Q2 looming: https://review.openstack.org/#/c/418530/21:45
johnsomxgerman_ It looks like your failover API might not be working correctly.  ^^^21:45
johnsomYeah, ok, it looks like we do have a conflict with failover API21:49
johnsomThis is a problem: https://github.com/openstack/octavia/blob/master/octavia/api/v2/controllers/load_balancer.py#L55121:49
xgerman_my API?21:50
xgerman_didn’t you work on it, too21:50
johnsomBecause this came in: https://github.com/openstack/octavia/commit/8baf0ca1d7a9a64c2ae80b5d8731b36b0d16a90121:50
johnsomYeah, the failover API21:50
johnsomHa, maybe21:50
bar_lol21:51
xgerman_so we just back out rm_work’s patch ;-)21:51
johnsomNo, we can just delete that _test_lb_status line21:51
xgerman_BUT the idea was you can’t fialover a pending/error LB21:51
johnsomYeah, he moved that into the actual flow, so this line is duplicating it from the flow21:52
xgerman_his is wrong then21:53
xgerman_because the admin hits fialover and we mark the LB pending — tyhen we pout it on the queue21:53
xgerman_so if we do it in the flow we igth get an update in the middle and then not faioover?21:53
johnsomYeah, I follow you21:55
nmagnezijohnsom, re: https://review.openstack.org/#/c/517455/ , will catch this tomorrow morning my timezone21:56
xgerman_now, if we do a health related failover we probably should do it in any case, too21:56
nmagnezijohnsom, but I just read tht story and I don't get how to reproducer Jason Niesz results an incorrect haproxy config21:57
johnsomOye, we are re-setting this on the cw side too21:57
nmagnezi(the cmd he uses: openstack loadbalancer listener create --name jason-test3-listener --protocol HTTP --protocol-port 80 jason-test3)21:57
rm_workjohnsom / xgerman_: IIRC I was the one arguing that failover SHOULDN'T bother checking and should "just work", but I think I was convinced to author the patch somehow22:02
johnsomnmagnezi Yeah, he locally changed the jinja2 template for the haproxy config which broke it.  It will be hard to exactly reproduce his situation22:03
nmagnezijohnsom, yeah i figured that. just double-checked that i don't miss anything here (it's late..)22:04
johnsomrm_work It's just where we do it that is the issue here. There are two different entry points, one via the API handler queue and one via the health manager22:04
nmagnezijohnsom, at a first glance the good looks okay. I will give it a second look tomorrow morning22:04
nmagnezigood night :)22:04
rm_workso it needs to be done in the health manager itself, not in the flow?22:04
rm_workbefore triggering the flow22:04
johnsomThe API path sets it to PENDING_UPDATE twice before the flow starts22:04
johnsomYep22:04
rm_workfine by me22:05
johnsomI will whip up a patch22:05
rm_worki don't even use the upstream failover flow :P22:05
bar_shouldn't we LOCK the flow somehow?22:05
bar_checking the db from two different places, could result in race condition, no?22:06
johnsombar_ This is the locking code22:07
bar_ok, i missed that22:07
*** rtjure has quit IRC22:08
johnsomWe lock the objects using the DB and the provisioning_status.   In this case the object was already locked and we had an addition check/lock attempt in the flow that got rejected because it was already locked.22:08
bar_and once it got into the diabolical ERROR status, is there a way the get rid of it?22:09
rm_workjohnsom: erm, looking at the way this keystone auth config is split up into two sections... does this make it possible to have one auth endpoint/account for the frontend API (verifying requests) and a totally different endpoint/account for communicating with backends (nova/neutron)?22:10
johnsomYes, you can delete it22:10
*** aojea has joined #openstack-lbaas22:10
johnsomrm_work Correct, that is how it is setup22:10
rm_workexcellent22:10
bar_johnsom, it did not work for me too good last time. I was left with zombies. (I'm redeploying right now)22:11
*** sshank has quit IRC22:11
johnsombar_ network stuff left?22:12
bar_loadbalancers22:12
johnsomThat should not happen unless you are using neutron-lbaas22:12
bar_nope. pure openstack client22:12
johnsomdelete should clean up ERROR objects.  I do have a patch up for review that fixes some network cleanup code that was not right and could block things depending on how the ERROR failure occured22:14
bar_it could be ramification of the weird state the worker got into.22:14
johnsombar_ reviews here are welcome: https://review.openstack.org/#/c/517455/ grin22:14
bar_:-) I think you meant Nir.22:15
bar_you need +222:16
johnsomNo, non-core reviews are super valuable too22:16
bar_ok, I'll try to get to it.22:16
johnsomIt saves the cores time if others find stuff before they get around to reviewing...22:17
bar_between, here's a brief one: https://review.openstack.org/#/c/522689/22:17
mnaserfyi https://review.openstack.org/#/c/522114/ is tested and working in our env here (didnt want to run a local patch but time constraints :()22:17
johnsomYeah, it's on my list.  There was a bunch of stuff to look at after the holiday22:18
bar_johnsom, legit, thanks.22:19
*** sshank has joined #openstack-lbaas22:22
*** rcernin has joined #openstack-lbaas22:34
openstackgerritMichael Johnson proposed openstack/octavia master: Fix the failover API to not fail with immutable LB  https://review.openstack.org/52324222:48
*** sshank has quit IRC22:49
*** salmankhan has joined #openstack-lbaas22:51
*** Alex_Staf has joined #openstack-lbaas22:56
*** sticker has joined #openstack-lbaas23:00
openstackgerritMerged openstack/neutron-lbaas master: Imported Translations from Zanata  https://review.openstack.org/52143523:02
*** ipsecguy has quit IRC23:04
openstackgerritMerged openstack/octavia-dashboard master: Imported Translations from Zanata  https://review.openstack.org/52244823:07
Alex_StafHi guys, I have some question regarding the HA architecture and some other stuff, it would be great if u could assist me with that http://paste.openstack.org/show/627523/23:08
johnsomHa, just a few....23:08
xgerman_Alex_Staf can you out that on an etherpad so we cna comment23:12
xgerman_put that on an etherpad23:12
*** salmankhan has quit IRC23:12
johnsomI was just going to use the "reply" option23:12
Alex_Stafjohnsom, haha yep . I will 1 sec23:13
Alex_Stafhttps://etherpad.openstack.org/p/AlexQUestions_)23:14
johnsomI am not sure that link came through correctly.23:17
johnsomI am working on this etherpad: https://etherpad.openstack.org/p/AlexQUestions_23:17
Alex_Stafhttps://etherpad.openstack.org/p/AlexQUestions_)23:21
Alex_Stafu lack a ")" in the end23:21
Alex_StafOk I see your23:21
openstackgerritMerged openstack/octavia master: Rename 'remove' to 'reset' in Quota API  https://review.openstack.org/52240223:21
johnsomUgh, ok, I will move my answers.23:22
Alex_Stafjohnsom, xgerman_ thanks guys. Hope I will have bunch of documentation for those questions soon23:22
*** slaweq has quit IRC23:25
openstackgerritMerged openstack/octavia master: Make the allowed_address_pairs driver better  https://review.openstack.org/51745523:25
*** slaweq has joined #openstack-lbaas23:27
*** aojea has quit IRC23:31
*** slaweq has quit IRC23:33
*** sshank has joined #openstack-lbaas23:35
openstackgerritMerged openstack/octavia master: Handle race condition deleting security group rule  https://review.openstack.org/52211423:37
dayou@johnsom,https://github.com/openstack/octavia-dashboard/blob/master/octavia_dashboard/api/rest/lbaasv2.py#L2423:42
dayouIt seems octavia-dashboard still use the neutron lbaas v2 api from openstack sdk23:43
dayouShould we update the code to use python-octaviaclient instead?23:43
johnsomdayou openstack SDK only supports Octavia, it does not support neutron-lbaas.23:44
johnsomAt least officially.23:44
johnsomIt could be hacked to since the APIs are similar23:44
johnsomAlex_Staf I think I have answered everything23:45
dayouI just see from the comment https://github.com/openstack/octavia-dashboard/blob/master/octavia_dashboard/api/rest/lbaasv2.py#L1423:45
dayouSo I am wondering whether we should update the code of octavia-dashboard to make use of python-octaviaclient23:46
johnsomdayou My mistake for not fixing that when I set it up for openstack sdk / Octavia23:46
dayouSince python-openstacksdk seems to be buggy23:46
dayouWhat's the correct way to fix it?23:47
johnsomI would argue no since python-octaviaclient is mostly setup for OpenStack Client more than a python SDK. We really should just fix openstacksdk23:47
dayouBut the https://github.com/openstack/octavia-dashboard/blob/master/octavia_dashboard/api/rest/barbican.py#L1723:49
dayoubarbican api is using barbican client23:49
johnsomYep23:49
dayouSo that's wh I am wondering23:49
dayouAlso this really bugs me23:49
dayouSince I am working on some changes on dashboard23:50
johnsomIt was just a call I made to use the openstacksdk.  In theory the python-octaviaclient could also use the SDK23:50
*** longstaff has quit IRC23:51
johnsomfor example, the neutron commands in python-openstackclient use the openstacksdk23:51
dayou/usr/local/lib/python2.7/dist-packages/openstack/load_balancer/v2/health_monitor.py23:51
dayouI have to manually edit this file to get healthmonitor resource return the correct field23:52
johnsomhttps://github.com/openstack/python-openstackclient/blob/master/openstackclient/network/v2/floating_ip.py#L1923:52
dayouOther wise for healthmoniotr resource it's not returing the parent resource indicators, which make the listing of it hard23:52
dayouWhat's the quickst way to get this fixed?23:53
dayoueither openstack sdk use octavia correctly or other way23:54
johnsomWe probably just need to put up a patch for openstacksdk23:54
*** ipsecguy has joined #openstack-lbaas23:56
dayouCould you start one with the correct way to fix it? I can follow up on it if you don't have enough time on this.23:56
johnsomSure, can you give me an example or a dashboard patch that isn't working?  I'm not sure I fully understand the problem.23:57
openstackgerritZhaoBo proposed openstack/octavia master: Support UDP load balance  https://review.openstack.org/50360623:59
dayouI can put my dashboard patch on, so you can try it. It still took a few days work for me to get it to a point to be abled to reviewd.23:59

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