Thursday, 2017-09-21

« Wednesday, 2017-09-20 Index Friday, 2017-09-22 »
*** tongl has quit IRC00:12
openstackgerritAdam Harwell proposed openstack/octavia master: Switch to using PKCS12 for TLS Term certs  https://review.openstack.org/50417500:24
kongjohnsom: do you think it's appropriate that octavia can provide a maintanence api that do amphora upgrade? or it's out of octavia scope and will reply on the scripts of operators?00:31
kongi think it's a so common task, i can's see there is any difference between scripts00:32
johnsomWe already did I think.  Maybe it didn't merge?00:34
johnsomLook for failover API00:34
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-lbaas-dashboard master: Updated from global requirements  https://review.openstack.org/50466000:37
*** openstackgerrit has quit IRC00:41
*** openstackgerrit has joined #openstack-lbaas00:42
openstackgerritAdam Harwell proposed openstack/octavia master: Switch to using PKCS12 for TLS Term certs  https://review.openstack.org/50417500:42
kongjohnsom: hah, nice!00:43
kongdoes that mean, we just need to build a new image and upload to glance, then just call that API?00:43
kongi didn't notice that before00:44
kongi even wrote a script one year ago...00:44
rm_workwoo! works, johnsom00:51
rm_workkong: yeah, that's the idea00:51
rm_workthough we need an API endpoint that does more than just *one*00:51
rm_worklike, "do a list of them" or "do all of them for a project" or "do ALL of them"00:51
rm_worksome basic level of orchestration00:52
rm_workjohnsom: the client fix seems to work in my devstack, and the pkcs12 patch seems to work in my devstack :)00:53
kongthanks, rm_work, i will test the api today00:55
johnsomNice00:55
kongbtw, what's your time now?00:55
rm_workwith both old (container) and new (single pkcs12 secret)00:55
rm_workboth of us are in PST00:55
kongok00:56
kongglad to know we have working time overlap for discussion00:58
rm_workjohnsom: ok to be fair, what I tested was the barbican driver -- i don't have a real way to test with the Castellan driver yet :/ but it SHOULD work...00:58
rm_workxgerman_: responded to your comment on https://review.openstack.org/#/c/477034/1101:01
johnsomYeah, I am usually around now.  I just had to run to the store after 5 for dinner fixings.01:05
rm_workok, added more tests xgerman_01:15
rm_worknot sure what's up with the bot, but i just pushed a new patchset01:15
rm_workjohnsom: you'll need to re-review i guess01:15
rm_workjust a rebase and more tests tho01:15
rm_workhttps://review.openstack.org/#/c/477034/11..12 (controller-worker stuff isn't mine)01:16
johnsomYeah, still looks good to me.  +1 for more tests!01:18
rm_workopenstackgerrit: hmm01:22
openstackgerritChenghui Yu proposed openstack/neutron-lbaas master: Update link in README when using devstack to install neutron-lbaas plugin  https://review.openstack.org/49520201:26
openstackgerritMichael Johnson proposed openstack/neutron-lbaas master: Update link in README when using devstack to install neutron-lbaas plugin  https://review.openstack.org/49520201:31
johnsomHmm, something strange happened there01:32
rm_workeh, I've seen Stranger Things01:33
* rm_work hides01:33
*** fnaval has quit IRC01:35
*** fnaval has joined #openstack-lbaas01:36
*** tongl has joined #openstack-lbaas01:39
johnsomDang it, we need to merge that requirements fix for octavia-dashboard.  Somehow in the changes yesterday we dropped the openstacksdk.01:42
johnsomI need to go eat, but I will fix that merge conflict after dinner.  A quick review would be appreciated.01:42
rm_workjohnsom: k. uhh, does d_api.get_session() take cycles or is it just a singleton?01:48
rm_workwondering if i should bother doing like ... session = db_api.get_session() if i'm going to use it 2-3 times01:48
rm_workjohnsom: ummm, WHICH requirements fix?01:49
rm_workhttps://review.openstack.org/#/c/505414/ is f'd01:49
rm_worki'm not sure I understand what you're referring to01:50
*** amotoki has quit IRC01:55
*** amotoki has joined #openstack-lbaas01:56
*** strigazi has quit IRC01:56
*** strigazi has joined #openstack-lbaas01:58
*** cody-somerville has joined #openstack-lbaas02:15
johnsomIf I remember correctly, get_session is one per thread, so you should be good to go02:22
johnsomHmmm, gerritbot is broken02:24
johnsomYeah, it was https://review.openstack.org/#/c/505414/02:24
johnsomHmmm, the members patch blew up too, ovh couldn't get to packages02:25
johnsomMan, OVH hosts have a bunch of problems these days02:26
openstackgerritChenghui Yu proposed openstack/neutron-lbaas master: Update link in README when using devstack to install neutron-lbaas plugin  https://review.openstack.org/49520202:32
openstackgerritHengqing Hu proposed openstack/octavia master: Handle stopping the health manager and ejecting out of current health failovers  https://review.openstack.org/46274802:44
*** tongl has quit IRC02:49
*** armax has quit IRC03:22
*** gans has joined #openstack-lbaas03:24
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-lbaas-dashboard master: Updated from global requirements  https://review.openstack.org/50466003:25
*** ankur-gupta-f4 has quit IRC03:45
*** cody-somerville has quit IRC04:11
*** cody-somerville has joined #openstack-lbaas04:12
*** cody-somerville has quit IRC04:12
*** cody-somerville has joined #openstack-lbaas04:12
*** cody-somerville has quit IRC04:26
*** eezhova has joined #openstack-lbaas04:33
*** tongl has joined #openstack-lbaas04:39
*** links has joined #openstack-lbaas04:43
*** eezhova has quit IRC04:55
*** pcaruana has joined #openstack-lbaas05:09
*** aojea has joined #openstack-lbaas05:31
*** aojea has quit IRC05:35
*** tongl has quit IRC05:58
*** aojea has joined #openstack-lbaas06:05
*** rcernin has joined #openstack-lbaas06:28
*** reedip has quit IRC07:01
*** sshank has joined #openstack-lbaas07:04
*** pcaruana has quit IRC07:05
*** reedip has joined #openstack-lbaas07:15
*** ChanServ sets mode: +o openstack07:16
*** tesseract has joined #openstack-lbaas07:19
*** sshank has quit IRC07:19
*** pcaruana has joined #openstack-lbaas07:23
*** fnaval has quit IRC08:09
*** gcheresh has joined #openstack-lbaas08:34
openstackgerritOpenStack Proposal Bot proposed openstack/octavia-dashboard master: Imported Translations from Zanata  https://review.openstack.org/50555409:06
*** gans has quit IRC09:16
*** yamamoto has quit IRC09:21
*** yamamoto has joined #openstack-lbaas09:22
*** salmankhan has joined #openstack-lbaas09:24
*** slaweq has joined #openstack-lbaas09:37
openstackgerritAdam Harwell proposed openstack/octavia master: WIP: Floating IP Network Driver (spans L3s)  https://review.openstack.org/43561209:39
*** tesseract has quit IRC09:41
*** tesseract has joined #openstack-lbaas09:41
*** bbzhao has quit IRC09:58
*** eezhova has joined #openstack-lbaas10:07
*** eezhova_ has joined #openstack-lbaas10:09
*** eezhova has quit IRC10:12
*** salmankhan has quit IRC10:17
*** salmankhan has joined #openstack-lbaas10:17
*** gcheresh has quit IRC10:29
*** sapd has quit IRC10:59
*** salmankhan has quit IRC11:08
*** salmankhan has joined #openstack-lbaas11:10
*** gcheresh has joined #openstack-lbaas11:13
*** gcheresh has quit IRC11:18
*** salmankhan has quit IRC11:19
*** sapd has joined #openstack-lbaas11:32
*** sapd has quit IRC11:39
*** salmankhan has joined #openstack-lbaas11:41
*** sapd has joined #openstack-lbaas11:43
*** dayou has quit IRC11:51
*** chlong has joined #openstack-lbaas11:52
*** sapd has quit IRC11:54
*** sapd has joined #openstack-lbaas11:55
*** sapd has quit IRC11:56
*** sapd has joined #openstack-lbaas11:56
openstackgerritMerged openstack/octavia master: Update links in CONTRIBUTING.rst  https://review.openstack.org/50546311:57
*** atoth has joined #openstack-lbaas12:03
openstackgerritMerged openstack/neutron-lbaas master: Don't forget to call the super's resource_cleanup  https://review.openstack.org/50414012:39
openstackgerritMerged openstack/neutron-lbaas master: Fix logging for lb timeout operations  https://review.openstack.org/50433112:39
openstackgerritMerged openstack/neutron-lbaas master: Use flake8-import-order plugin  https://review.openstack.org/48054312:40
*** catintheroof has joined #openstack-lbaas12:41
*** fnaval has joined #openstack-lbaas12:53
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-lbaas-dashboard master: Updated from global requirements  https://review.openstack.org/50466013:05
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-lbaas-dashboard master: Updated from global requirements  https://review.openstack.org/50466013:33
openstackgerritOpenStack Proposal Bot proposed openstack/octavia master: Updated from global requirements  https://review.openstack.org/50617013:34
openstackgerritzhangyangyang proposed openstack/neutron-lbaas-dashboard master: Cleanup test-requirements  https://review.openstack.org/50617613:41
*** slaweq has quit IRC13:55
openstackgerritMichael Johnson proposed openstack/octavia-dashboard master: Add subnet_id to member details page  https://review.openstack.org/50542714:05
*** armax has joined #openstack-lbaas14:14
openstackgerritMerged openstack/octavia-dashboard master: Imported Translations from Zanata  https://review.openstack.org/50555414:15
*** eezhova_ has quit IRC14:17
*** fnaval has quit IRC14:20
*** fnaval has joined #openstack-lbaas14:21
*** fnaval has quit IRC14:25
*** openstackgerrit has quit IRC14:33
*** gcheresh has joined #openstack-lbaas14:37
*** fnaval has joined #openstack-lbaas14:40
johnsomFYI, our gates are broken as we are pulling master DIB and they have an issue with the networkx module update.14:43
*** gcheresh has quit IRC14:46
*** leitan has joined #openstack-lbaas14:53
*** links has quit IRC15:07
*** openstackgerrit has joined #openstack-lbaas15:26
openstackgerritHengqing Hu proposed openstack/octavia master: Handle stopping the health manager and ejecting out of current health failovers  https://review.openstack.org/46274815:26
openstackgerritzhangyangyang proposed openstack/python-octaviaclient master: Cleanup test-requirements  https://review.openstack.org/50625015:30
*** tongl has joined #openstack-lbaas15:48
*** eezhova has joined #openstack-lbaas16:21
*** tesseract has quit IRC16:50
*** rcernin has quit IRC16:52
*** salmankhan has quit IRC17:05
*** salmankhan has joined #openstack-lbaas17:05
rm_workjoooooy networkx17:43
*** aduarte has joined #openstack-lbaas17:44
rm_workjohnsom: did they have an ETA for you?17:50
johnsomNope17:50
rm_workok, thoughts?17:50
rm_workwe could try to pin DIB17:50
rm_workto a specific patch in our gates17:50
*** ianychoi has quit IRC17:57
*** ianychoi has joined #openstack-lbaas17:58
rm_workjohnsom: ^^17:58
rm_workjohnsom: i really need to get some stuff merging this week... what do you want to do to fix this17:59
johnsomYeah, I was kind of giving it some time to see what they came up with.  I see the revert is now abandoned17:59
rm_workhmm17:59
johnsomCan you poke them?17:59
rm_workDIB? yes17:59
johnsomI'm in a meeting17:59
johnsomMaybe this: https://review.openstack.org/#/c/506162/18:01
*** SumitNaiksatam has joined #openstack-lbaas18:02
*** tonygunk has joined #openstack-lbaas18:15
-openstackstatus- NOTICE: Gerrit OpenIDs have been accidentally overwritten and are in the process of being restored18:17
rm_workalso: ^^ joy18:19
rm_workwhelp i'm gonna go take a shower and maybe do lunch or something <_<18:20
-openstackstatus- NOTICE: The Gerrit service on review.openstack.org is being taken offline briefly to perform database repair work but should be back up shortly18:22
*** ChanServ changes topic to "The Gerrit service on review.openstack.org is being taken offline briefly to perform database repair work but should be back up shortly"18:22
*** tonygunk has quit IRC18:25
johnsomYeah, looking like lunch time....18:27
*** armax has quit IRC18:44
*** armax has joined #openstack-lbaas18:44
xgerman_rm_work since nobody reviews my stuff for months…18:49
rm_workwhich stuff? I finally have the ability/time to review again18:50
rm_worki've been moving through most anything that looked ready18:50
xgerman_and just now gerrit is down18:51
xgerman_I like some eyes on the Active-Active noop driver18:51
xgerman_and https://review.openstack.org/50189018:52
rm_workah, yes18:52
rm_workthe active/active stuff has unfortunately been a bit lower priority previously18:53
rm_workbut will have a chance to look at it once we get this current cluster@*$#% of stuff through18:53
xgerman_in my experience things fix themselves in a few days ;-)18:54
xgerman_but it’s annoying that we are dead in the wayer18:55
xgerman_rm_work: gerrit is back - https://review.openstack.org/#/c/313006/19:09
johnsomOh, oh, I want to play too....  https://review.openstack.org/#/c/493273/ Grin19:13
johnsomhttps://review.openstack.org/50541419:13
johnsomhttps://review.openstack.org/#/c/505427/19:13
rm_workjohnsom: comment but no vote yet on https://review.openstack.org/#/c/493273/519:16
*** salmankhan has quit IRC19:17
rm_workxgerman_: i really just think i'm going to have to clear like.... two or three days to go through all of that active-active stuff19:17
rm_worki can't just flip gears to that right now and hope to understand wtf all is going on there19:18
johnsomResponded19:18
johnsomYeah, most of the rest of those patches in the chain still need help.  We are going to have to take them one-by-one19:19
johnsomAfter I make some progress on openstacksdk for dashboard (Might have a long review cycle), I plan to pivot to attacking act/act19:20
*** ChanServ changes topic to "Welcome to LBaaS / Octavia - Queens development is now open."19:20
-openstackstatus- NOTICE: OpenIDs for the Gerrit service have been restored from a recent backup and the service is running again; before/after table states are being analyzed now to identify any remaining cleanup needed for changes made to accounts today19:20
openstackgerritMerged openstack/python-octaviaclient master: Improve error reporting for the octavia plugin  https://review.openstack.org/49327319:23
xgerman_we are just 3 cores so reviewing should be our priority19:24
xgerman_we can’t assume people working on a patch hold their attention span for weeks19:24
rm_workyes, for smaller stuff i can pivot around and get reviews in19:25
rm_workas I have19:25
rm_workbut active-active is a lower priority than 90% of the other stuff out there IMO, and it's HUUUUGE19:25
jnieszyes, but there are resources we have available now to use : )19:26
johnsomjniesz non-core reviews would really help us out....19:26
rm_worki'd love to get it in19:26
rm_workbut i honestly don't *understand* what it's doing to any real degree yet19:26
johnsomHelps folks learn the code too19:26
rm_workand it's a big pile of complexity19:26
jnieszyes, I know Jacky has been helping with the dashboard stuff as well19:26
rm_workwhich i can get to eventually, but it's going to seriously take me the better part of a day or two of direct focus19:26
rm_workand i can't spare that this week19:27
rm_workif you want me to review something that i can walk through the code, understand, and test inside of a couple hours, i can do that whenever19:27
rm_workespecially if it's a bug/improvement for existing features19:28
xgerman_anyhow, we need to get our priorities traight19:29
xgerman_if we just want to do feature improvements that’s cool with me but then we shouldn’t make big goals19:30
*** leitan has quit IRC19:40
*** leitan has joined #openstack-lbaas19:40
*** pcaruana has quit IRC19:41
*** leitan has quit IRC19:41
*** leitan has joined #openstack-lbaas19:41
johnsomFYI, I dropped the broken integration tests from dashboard projects.  Horizon team has dropped integration tests for now and advised us to do the same.  That should speed up our dashboard gate runs.19:49
openstackgerritAdam Harwell proposed openstack/octavia master: Add admin endpoint for amphora info  https://review.openstack.org/50540419:51
*** fnaval has quit IRC20:01
*** eezhova has quit IRC20:11
*** fnaval has joined #openstack-lbaas20:12
*** eezhova has joined #openstack-lbaas20:17
*** armax has quit IRC20:18
johnsomxgerman_ Have a second?20:22
*** eezhova has quit IRC20:28
xgerman_sure20:32
xgerman_@johnsom20:32
johnsomSo, two things.20:32
johnsominfo/diag/details20:32
johnsomhttps://docs.openstack.org/octavia/latest/contributor/api/haproxy-amphora-api.html#get-amphora-info20:33
johnsomWe already have a spec for the amp api with info and details20:33
xgerman_yes20:33
xgerman_but we want to make that more accessible through the api-server20:33
johnsomI think the diag stuff should be a different patch20:34
xgerman_k20:34
johnsomNow the path thing20:34
johnsomI waffle on that a bit.20:34
xgerman_yeah, it doesn’t feel right to have amphora as a top-level-resource since only we have that20:35
johnsom /v2.0/lbaas/amphorae doesn't bother me that much.  It would just return [] if there are no amps20:35
rm_workyep20:36
*** salmankhan has joined #openstack-lbaas20:36
johnsomIt keeps the REST hierarchy kind of clear20:36
rm_workthough if you give me a sec, i already have the patch to make it /v2.0/octavia/amphorae20:36
rm_workso, either is ok with me20:36
xgerman_if I am a user it wouldn’t be clear to me why amphora returns [] if I have an F520:36
xgerman_so UX wise the /octavia/ makes that better20:37
johnsomAre you arguing we should extend the amphora abstraction to include vendor drivers?  That seems, ummm, bad20:37
xgerman_I am saying we should put it under a path where a user sees it’s Octavia specific20:38
johnsom /v2.0/octavia/amphorae implies it's not an lbaas path or not part of the lbaas API we are pulling forward....20:39
*** salmankhan has quit IRC20:40
xgerman_yep, amphora are octavia specific - lbaas is for every vendor20:40
johnsomWould, in the future, if we allowed drivers to add /v2.0/lbaas/F5...  that would be a bad thing?20:40
xgerman_well, they could add /v2.0/F5/20:41
johnsomOk, yeah, I get it.  /v2.0/octavia/amphorae is better20:41
johnsomrm_work?20:41
xgerman_he said he had that patch ;-)20:42
johnsomYeah, but we should have the conversation.  My opinion changed, so....20:42
xgerman_+120:42
*** leitan has quit IRC20:48
rm_workyeah waiting on tests20:49
rm_workkk20:49
rm_workalso added a release note20:49
openstackgerritAdam Harwell proposed openstack/octavia master: Add admin endpoint for amphora info  https://review.openstack.org/50540420:49
johnsomOk, so we are moving forward with /v2.0/octavia/amphorae20:50
rm_workIt seemed like you two agreed20:50
rm_workand i already said i was fine with it20:50
xgerman_splendid20:54
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-lbaas-dashboard master: Updated from global requirements  https://review.openstack.org/50466020:55
openstackgerritOpenStack Proposal Bot proposed openstack/octavia master: Updated from global requirements  https://review.openstack.org/50617020:56
*** sshank has joined #openstack-lbaas20:58
johnsomjniesz Just reviewed again.  Gerrit did something strange with the patch set 10 comments, ignore those.21:01
johnsomI think just storing that network info and I'm good21:02
openstackgerritAdam Harwell proposed openstack/octavia master: Add admin endpoint for amphora info  https://review.openstack.org/50540421:04
*** catintheroof has quit IRC21:07
openstackgerritOpenStack Proposal Bot proposed openstack/neutron-lbaas-dashboard master: Updated from global requirements  https://review.openstack.org/50466021:12
openstackgerritOpenStack Proposal Bot proposed openstack/octavia master: Updated from global requirements  https://review.openstack.org/50617021:13
johnsomrm_work reviewed21:21
rm_workyeah working on making it still able to filter on project_id21:21
jnieszjohnsom thanks for reviewing active/active l3 spec21:22
jnieszfor the front-end subnet, are you ok storing that on the amp table?21:22
*** aojea has quit IRC21:22
rm_workjohnsom: i'm a little concerned that the wrong line in the policy didn't seem to affect the policy queries at all?21:22
rm_worksince the tests were passing21:22
*** aojea has joined #openstack-lbaas21:23
johnsomThat path is just for the auto generated docs21:23
johnsomjniesz Yeah, I think that is fine21:24
jnieszcool, I will make the change21:25
*** aojea has quit IRC21:27
*** armax has joined #openstack-lbaas21:28
rm_workjohnsom: ok sooooo that's a TINY bit of a hack21:39
rm_workbut it works21:39
rm_work(once again, openstackgerrit is sitting it out when i'd expect a patch notification)21:40
rm_workjohnsom: https://review.openstack.org/#/c/505404/7/octavia/api/common/pagination.py21:40
rm_work... and when I say "a tiny bit of a hack" ...21:40
rm_worki don't see another way unless i can figure out how to expose that field ON the object21:42
johnsomrm_work so if you made it work... Should we put the _auth_get_all and project_id back?  That way it is consistent?21:42
rm_workno21:42
rm_workit can't be21:42
johnsomAh, because of the injected project ID query string?21:44
johnsomI mean it could all go back except the addition of the query filter21:45
*** Aju has quit IRC21:45
*** redondo-mk has quit IRC21:45
*** coreycb has quit IRC21:46
*** amitry has quit IRC21:46
*** rm_work has quit IRC21:47
*** dasanind has quit IRC21:48
*** coreycb has joined #openstack-lbaas21:48
*** amotoki has quit IRC21:48
*** amitry has joined #openstack-lbaas21:48
*** amotoki has joined #openstack-lbaas21:51
*** dasanind has joined #openstack-lbaas21:52
*** afranc has joined #openstack-lbaas21:52
openstackgerritJason Niesz proposed openstack/octavia master: blueprint: l3-active-active  https://review.openstack.org/45300521:55
konghi guys, what's the status of octavia-dashboard? available from Pike? or still work in progress?21:58
johnsomkong It didn't make Pike, but is working in Queens.  You can run the queens code against Pike Octavia though21:58
kongjohnsom: cool21:59
johnsomI don't think I have cut a release yet for it to pypi.  I should probably look at that21:59
kongthat's ok, it's not a hard requirement for us for the phrase 122:00
kongjust want to know if there are people working on that22:00
johnsomYeah, I will release it with the Queens MS1 for sure22:00
johnsomYeah, queens octavia-dashboard is going to get some love....  More if you help!  grin22:00
johnsomhttps://review.openstack.org/#/q/project:openstack/octavia-dashboard+status:open22:01
*** redondo-mk has joined #openstack-lbaas22:01
kongactually, i am scared of reading horizon code22:02
*** rm_work has joined #openstack-lbaas22:02
*** rm_work has quit IRC22:02
*** rm_work has joined #openstack-lbaas22:02
kong:-)22:03
rm_workmy bouncer exploded apparently22:04
rm_workjohnsom: i mean, i can't put project_id back in the signature22:05
rm_workwhich means i can't use it22:05
rm_workwhich means using the old auth method is weird because it expects project_id22:05
johnsomWhy can't you put it back in?  Does it strip it from the params?22:06
johnsomkong Me too, but I'm going to give it a try....22:07
*** sshank has quit IRC22:28
*** kbyrne has quit IRC22:33
kongI have contacted Jude and he agreed I can take over his patch https://review.openstack.org/#/c/48677522:36
kongscenario test is a hard requirement for us22:36
*** sshank has joined #openstack-lbaas22:48
rm_workjohnsom: yes22:57
rm_workjohnsom: if you include it in the signature of the method it's stripped from the hook params22:57
rm_workwhich is technically before i can even get at it, deep in pecan22:58
johnsomOk22:58
*** tongl has quit IRC23:46
« Wednesday, 2017-09-20 Index Friday, 2017-09-22 »

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