| *** chlong has quit IRC | 00:01 | |
| *** chlong has joined #openstack-lbaas | 00:02 | |
| *** chlong has quit IRC | 00:06 | |
| *** jorgem has quit IRC | 00:11 | |
| dougwig | crc32: you need to grab a clean master, rebase it into your topic branch, and then just re-run "git review". | 00:19 |
|---|---|---|
| dougwig | your dependency is actually up to date. | 00:19 |
| crc32 | git checkout master; git pull; git checkout mybranch;git merge master? | 00:20 |
| crc32 | like that dougwig? | 00:21 |
| openstackgerrit | Carlos Garza proposed stackforge/octavia: Add nsCertType and ExtendedKey usage extensions to CertGenerator https://review.openstack.org/144670 | 00:22 |
| rm_work | dougwig: his dependency's dependency is not | 00:22 |
| crc32 | oh wait. git rebase master. | 00:22 |
| rm_work | dougwig: the review in question was like the third on a review chain | 00:22 |
| rm_work | and the first two were not up to date... he wouldn't want to pull master into his | 00:22 |
| *** vivek-ebay has quit IRC | 00:26 | |
| crc32 | https://review.openstack.org/#/c/144831 <- patch in merge conflict. | 00:26 |
| crc32 | can I fix it. | 00:27 |
| crc32 | its just a one liner thats duplicated | 00:27 |
| crc32 | or rather how would I fix it. | 00:27 |
| dougwig | you can fix that if you want. get a fresh master, cut a topic branch, cherry-pick that in, git review | 00:28 |
| crc32 | I don't think I could do the merge conflict my self as only gerrit can do merges. | 00:28 |
| dougwig | no, you can do it locally and push it up. | 00:28 |
| dougwig | i can do it quick if it's blocking you, but it sounds like a good thing for you to take a crack at. :) | 00:29 |
| *** chlong has joined #openstack-lbaas | 00:29 | |
| crc32 | yea I want to try to fix it. I don't understand the cut a topic branch though | 00:30 |
| rm_work | there are several ways to do it | 00:31 |
| rm_work | that is one | 00:31 |
| rm_work | but you don't have to | 00:31 |
| crc32 | but I'm confused. It doesn't look like any of the commits in my dependency chain touch the loadbalancer_db file. | 00:31 |
| rm_work | yeah i do not get a merge conflict there | 00:32 |
| rm_work | he might just need to click the "rebase" button on it again | 00:32 |
| *** vivek-ebay has joined #openstack-lbaas | 00:33 | |
| dougwig | clicking rebase isn't an option if you don't own the review. but anyone can do a manual merge/update | 00:33 |
| rm_work | at which point my guess is that it'll all trickle down the chain and be fine | 00:33 |
| crc32 | it looks like it says its conflicted with https://review.openstack.org/#/c/141247/11 but patchset 12 was already merged | 00:33 |
| rm_work | right, brandon needs to do it | 00:33 |
| rm_work | it's not crc32's problem | 00:33 |
| dougwig | rm_work: nope, not if you do it locally. | 00:33 |
| rm_work | dougwig: yeah but i would just let brandon do it | 00:33 |
| rm_work | it's not like his review will go in before that one | 00:34 |
| rm_work | he's stuck behind it anyway, let brandon fix his own review :P | 00:34 |
| dougwig | unless he's blocked, in which case i'm sure it's a nonsense conflict. | 00:34 |
| dougwig | sheesh, we have talked about this longer than it'd take to fix. :) | 00:34 |
| rm_work | yeah but it's also a WIP | 00:35 |
| rm_work | if either of us upload the fixed review, it'd wipe that until brandon got online again :P | 00:35 |
| xgerman | dougwig that's why we never get anyhting done :-) | 00:35 |
| rm_work | which is why i'd just wait and let brandon do it :P | 00:35 |
| rm_work | we should all nag him about it, of course | 00:35 |
| openstackgerrit | Doug Wiegley proposed openstack/neutron-lbaas: Implement synchronous haproxy driver methods https://review.openstack.org/144831 | 00:35 |
| dougwig | done. | 00:35 |
| dougwig | took <60 seconds. | 00:35 |
| rm_work | ugh | 00:36 |
| rm_work | now it's not WIP | 00:36 |
| xgerman | lol | 00:36 |
| dougwig | https://www.irccloud.com/pastebin/61eYs5qj | 00:36 |
| rm_work | yeah, i had it done 5 minutes ago, but i didn't want to mess with his review :P | 00:36 |
| dougwig | you were saying it's not WIP? :) | 00:36 |
| dougwig | look again | 00:36 |
| dougwig | he lives to be messed with. | 00:36 |
| xgerman | I might need the same to kae my vpn review work | 00:37 |
| dougwig | someone want to do it for 144832 as a learning experiment? | 00:37 |
| rm_work | rofl | 00:37 |
| rm_work | dougwig: there's no real reason to bother with doing a new branch | 00:37 |
| rm_work | I essentially do the same thing but with one less line :P | 00:38 |
| dougwig | copying the topic branch from the review keeps the Topic in gerrit identical to the original submitter. | 00:38 |
| dougwig | it's not required, but i like to mess with as little as possible. | 00:38 |
| crc32 | so now I do the same thing? | 00:40 |
| dougwig | yep, but use the topic and cherry-pick from 144832 instead. | 00:40 |
| rm_work | you can... again, i don't see a reason to be in a hurry | 00:40 |
| rm_work | but | 00:40 |
| rm_work | if you really feel like taking over his commit :P | 00:40 |
| dougwig | after that, you'll be able to just hit the rebase button in your review. | 00:40 |
| rm_work | I generally let people handle their own stuff | 00:40 |
| dougwig | rm_work: the gerrit owner and author stay as the original. being the committer confers *nothing*. no free pass, no stackalytics, no nothing. :) | 00:40 |
| dougwig | i'd rather see people move forward than sit around waiting. hell, blogan rebased one of my patches yesterday. | 00:41 |
| rm_work | yeah, i still feel like it's rude to take action on other people's commits though | 00:41 |
| *** dougwig is now known as dougwig_the_rude | 00:41 | |
| dougwig_the_rude | that's me. | 00:41 |
| *** dougwig_the_rude is now known as dougwig | 00:42 | |
| rm_work | to be fair, rebasing is probably the least impacting | 00:42 |
| rm_work | so long as there actually isn't any conflict | 00:42 |
| dougwig | sbalukoff: what happened to octavia's docs job? or did it never have one? | 00:43 |
| dougwig | rm_work: if i or someone else is blocked, and a simple rebase is in the way, you can be guaranteed that i'll be rude every single time. | 00:43 |
| crc32 | what branch where you on when you did "git pull"? | 00:43 |
| dougwig | not that i consider it rude. | 00:43 |
| dougwig | crc32: master | 00:44 |
| xgerman | you can keep your handle | 00:45 |
| xgerman | blogan would say it's the true doug :-) | 00:45 |
| dougwig | ha. | 00:45 |
| *** pattabi has joined #openstack-lbaas | 00:48 | |
| pattabi | in devstack, the db tables for the v2 data model are not getting created ... any pointers/help on this would be really helpful | 00:49 |
| dougwig | pattabi: the patch for that went in yesterday. when is the last time you refreshed your devstack? | 00:51 |
| pattabi | i did today afternoon | 00:51 |
| dougwig | hmm, that's not good. and you have q-lbaas in your conf file? | 00:52 |
| pattabi | actually, i did not refresh devstack .. i only refreshed the neutron and neutron-lbaas. | 00:52 |
| johnsom | rm_work: I see your confusion on the controller spec. I have commented here: https://review.openstack.org/#/c/136540/ see if that aligns with your thinking. Jorge should get an comment notification. | 00:52 |
| pattabi | i will clean up the devstack completely and try to reinstall from scratch one more time and get in touch with you if i still face the issue | 00:53 |
| openstackgerrit | Doug Wiegley proposed openstack/neutron-lbaas: Rename array of LB constants to better align with usage https://review.openstack.org/145648 | 00:55 |
| openstackgerrit | Carlos Garza proposed openstack/neutron-lbaas: Implement managers for synchronous haproxy driver https://review.openstack.org/144832 | 00:57 |
| crc32 | Did I do that one right? | 00:57 |
| *** Putns has quit IRC | 01:00 | |
| *** xgerman has quit IRC | 01:02 | |
| dougwig | crc32: is your Common TLS Utilities review the same as the "New common util module for TLS Certificate management" review from the neutron feature branch? | 01:04 |
| *** dank has quit IRC | 01:04 | |
| crc32 | dougwig it was intended to be moved over from neutron - | 01:05 |
| crc32 | neutron -> neutron-lbaas | 01:05 |
| crc32 | so I guess it should be abandoned in neutron | 01:05 |
| dougwig | right, i'm just accounting for all of the abandoned reviews. | 01:05 |
| dougwig | subject line changed, but that's the same one? | 01:05 |
| crc32 | can you send me the neutron change request URL? | 01:06 |
| crc32 | so I can verify? | 01:06 |
| dougwig | https://review.openstack.org/#/c/123492/ | 01:07 |
| crc32 | thats the tricky part. | 01:07 |
| crc32 | 142915 only has the cert_parser functions we still need to move over the barbican get_cert delete_cert and store_cert methods. | 01:08 |
| *** SumitNaiksatam has quit IRC | 01:09 | |
| dougwig | looks like adam is the original author. can you or adam take on moving the rest of it? | 01:09 |
| crc32 | yes we talked about that today. I still need to discuss with him who's migrating the code. | 01:10 |
| openstackgerrit | Carlos Garza proposed openstack/neutron-lbaas: Common TLS utilities https://review.openstack.org/142915 | 01:14 |
| *** sbfox has joined #openstack-lbaas | 01:18 | |
| *** markmcclain has quit IRC | 01:18 | |
| *** markmcclain has joined #openstack-lbaas | 01:20 | |
| *** vivek-ebay has quit IRC | 01:36 | |
| openstackgerrit | Carlos Garza proposed stackforge/octavia: Common TLS utilies https://review.openstack.org/145092 | 01:39 |
| *** sbfox has quit IRC | 01:43 | |
| *** vivek-ebay has joined #openstack-lbaas | 01:44 | |
| *** sbfox has joined #openstack-lbaas | 01:48 | |
| *** SumitNaiksatam has joined #openstack-lbaas | 01:53 | |
| openstackgerrit | Carlos Garza proposed stackforge/octavia: Common TLS utilies https://review.openstack.org/145092 | 01:53 |
| *** markmcclain has quit IRC | 01:55 | |
| openstackgerrit | Xu Han Peng proposed openstack/neutron-lbaas: Add subnet extra routes to lbaas namespace https://review.openstack.org/144931 | 02:03 |
| *** sbfox has quit IRC | 02:07 | |
| sbalukoff | dougwig: I know we have a tox docs task, but I don't think we have anything in CI which actually builds documentation that gets put online somewhere. | 02:09 |
| sbalukoff | I thought that was a privilege reserved for, you know, projects that are in the OpenStack canon. But I could be wrong about that. | 02:10 |
| *** sbfox has joined #openstack-lbaas | 02:10 | |
| *** chlong has quit IRC | 02:11 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-lbaas: Updated from global requirements https://review.openstack.org/145670 | 02:30 |
| *** sbfox has quit IRC | 02:33 | |
| *** sbfox has joined #openstack-lbaas | 02:37 | |
| *** sbfox has quit IRC | 02:50 | |
| *** vivek-ebay has quit IRC | 02:50 | |
| *** pattabi has quit IRC | 02:57 | |
| *** markmcclain has joined #openstack-lbaas | 03:05 | |
| *** SumitNaiksatam has quit IRC | 03:53 | |
| *** chlong has joined #openstack-lbaas | 03:57 | |
| blogan | i am back | 04:02 |
| blogan | sorry i missed some of that crc32 | 04:02 |
| blogan | you get it sorted out? | 04:02 |
| *** chlong has quit IRC | 04:02 | |
| blogan | johnsom: ping | 04:04 |
| *** chlong has joined #openstack-lbaas | 04:18 | |
| *** xgerman has joined #openstack-lbaas | 04:23 | |
| *** markmcclain has quit IRC | 04:33 | |
| *** markmcclain has joined #openstack-lbaas | 05:05 | |
| *** xgerman has quit IRC | 05:07 | |
| *** sbfox has joined #openstack-lbaas | 05:08 | |
| *** sbfox has quit IRC | 05:09 | |
| *** sbfox has joined #openstack-lbaas | 05:10 | |
| *** markmcclain has quit IRC | 05:23 | |
| *** vivek-ebay has joined #openstack-lbaas | 05:29 | |
| *** jschwarz has joined #openstack-lbaas | 05:52 | |
| *** vivek-ebay has quit IRC | 06:05 | |
| *** kobis has joined #openstack-lbaas | 06:08 | |
| *** sbfox has quit IRC | 06:16 | |
| *** woodster_ has quit IRC | 07:10 | |
| openstackgerrit | Phillip Toohill proposed stackforge/octavia: HaProxy reference driver implementation https://review.openstack.org/144348 | 07:29 |
| *** mestery has quit IRC | 07:33 | |
| *** apuimedo has joined #openstack-lbaas | 07:37 | |
| openstackgerrit | Phillip Toohill proposed stackforge/octavia: HaProxy reference driver implementation https://review.openstack.org/144348 | 07:45 |
| *** chlong has quit IRC | 07:47 | |
| openstackgerrit | Phillip Toohill proposed stackforge/octavia: HaProxy reference driver implementation https://review.openstack.org/144348 | 08:00 |
| *** mestery has joined #openstack-lbaas | 08:14 | |
| *** crc32 has quit IRC | 08:23 | |
| *** vivek-ebay has joined #openstack-lbaas | 09:06 | |
| *** vivek-ebay has quit IRC | 09:10 | |
| *** sbalukoff has quit IRC | 09:23 | |
| *** sbalukoff has joined #openstack-lbaas | 10:27 | |
| *** kobis has quit IRC | 10:50 | |
| *** jschwarz has quit IRC | 11:20 | |
| *** chlong has joined #openstack-lbaas | 11:43 | |
| *** Miouge has quit IRC | 12:02 | |
| *** Miouge has joined #openstack-lbaas | 12:03 | |
| *** Miouge has quit IRC | 12:08 | |
| *** woodster_ has joined #openstack-lbaas | 12:29 | |
| *** Putns has joined #openstack-lbaas | 12:56 | |
| *** Putns has left #openstack-lbaas | 12:57 | |
| *** Miouge has joined #openstack-lbaas | 13:03 | |
| *** kobis has joined #openstack-lbaas | 13:10 | |
| *** kobis1 has joined #openstack-lbaas | 13:39 | |
| *** kobis has quit IRC | 13:39 | |
| *** Miouge has quit IRC | 15:09 | |
| *** vivek-ebay has joined #openstack-lbaas | 15:22 | |
| *** vivek-ebay has quit IRC | 15:25 | |
| *** Miouge has joined #openstack-lbaas | 15:33 | |
| *** apuimedo has quit IRC | 15:51 | |
| *** xgerman has joined #openstack-lbaas | 15:56 | |
| *** kobis1 has quit IRC | 16:03 | |
| *** mlavalle has joined #openstack-lbaas | 16:11 | |
| *** jschwarz has joined #openstack-lbaas | 16:12 | |
| *** chlong has quit IRC | 16:34 | |
| *** sbfox has joined #openstack-lbaas | 16:56 | |
| rm_work | when is the neutron-lbaas midcycle meetup again> | 17:05 |
| rm_work | ? | 17:05 |
| xgerman | rm_work: https://etherpad.openstack.org/p/lbaas-kilo-meetup | 17:17 |
| rm_work | thx | 17:17 |
| rm_work | LinuxJedi: I actually like protobuf a lot, but I think we're trying to follow the "OpenStack Way" for this, for better or for worse… though since the use of JSON you're talking about is completely internal and everything is controlled by the operator, I think most concerns don't really apply -- it should be fine | 17:21 |
| rm_work | (just catching up on Octavia emails from the holidays) | 17:21 |
| *** vivek-ebay has joined #openstack-lbaas | 17:21 | |
| rm_work | was octavia.io down? :( I guess it is up now, but it's sad if it was down | 17:21 |
| rm_work | also lol sbalukoff: "Please also note that this is a fruitless discussion: Nobody working on Octavia is interested in going back on the months of discussion, design, and work we've done on Octavia" sounds exactly like what happened when all of us walked into a certain neutron-lbaas meeting when we started all of this T_T | 17:23 |
| rm_work | very deja-vu | 17:23 |
| xgerman | well, sbalukoff quenches all dissenters | 17:23 |
| rm_work | heh | 17:23 |
| rm_work | where is LinuxJedi located? I remember talking with him when you guys were all on the libra team … but is he remote from everyone else at HP on this project? seems like getting together in person with one of the 6-8 other HP people working on octavia might be useful if possible :P | 17:24 |
| xgerman | he is in London but moved to some more R than D team in HP | 17:25 |
| rm_work | ah | 17:25 |
| rm_work | err, about to head out to lunch/errand, back in a while | 17:25 |
| LinuxJedi | well, I'm a few hours away from London | 17:25 |
| sballe | rm_work: Also remember HP is big and not all group will have the same goals. | 17:25 |
| LinuxJedi | I think the last count for HP was 250,000 employees. Several overlaps in different departments. | 17:26 |
| rm_work | sballe: yes :( | 17:26 |
| rm_work | LinuxJedi: yeah but I remembered you guys were on the same team before, figured you might be locationally similar :P | 17:27 |
| rm_work | guess not | 17:27 |
| LinuxJedi | no, I used to run the Libra team when I was in HP Cloud (Helion). Myself and several others spun up a new department which is only very loosly related to cloud | 17:28 |
| rm_work | ha, nice, got your own department now | 17:28 |
| LinuxJedi | well, Brian Aker runs it, but yea, it is a small department with a lot of freedom | 17:28 |
| rm_work | what makes you guys interested in Octavia now? | 17:29 |
| rm_work | if you're not part of Helion | 17:29 |
| rm_work | have a use for something like it internally? | 17:29 |
| LinuxJedi | I won't actually know until next week, jut that Brian is thinking of using it as a part of an upcoming project so asked me to find out as much as I can for meetings next week | 17:29 |
| rm_work | alright, feel free to ignore me if it's top secret classified info :P | 17:29 |
| LinuxJedi | lol! You now know about as much as I do right now :) | 17:30 |
| a2hill | rm_work, it was 'down' because of a failure of pep8 :P | 17:30 |
| rm_work | blah | 17:30 |
| rm_work | stupid pep8 | 17:30 |
| a2hill | Then i brought it down trying to test out graphviz versions >< | 17:30 |
| rm_work | lol | 17:30 |
| LinuxJedi | the last 6 months or so I've been working on high performance database connectors so I've been out of touch with a lot of things :) | 17:30 |
| rm_work | oh, right, i said i was going to lunch -- err, bbl | 17:31 |
| rm_work | but yeah, IRC is probably the easiest place to get answers, over the mailing list | 17:31 |
| * rm_work scurries off | 17:31 | |
| LinuxJedi | cool, will note that for future :) | 17:31 |
| *** kobis has joined #openstack-lbaas | 17:33 | |
| *** kobis has quit IRC | 17:43 | |
| *** jorgem has joined #openstack-lbaas | 17:49 | |
| *** jschwarz has quit IRC | 17:57 | |
| dougwig | morning all | 17:57 |
| a2hill | Mornin' | 18:00 |
| *** vivek-ebay has quit IRC | 18:00 | |
| *** sbalukoff has quit IRC | 18:01 | |
| *** vivek-ebay has joined #openstack-lbaas | 18:01 | |
| *** vivek-ebay has quit IRC | 18:02 | |
| *** vivek-eb_ has joined #openstack-lbaas | 18:02 | |
| *** SumitNaiksatam has joined #openstack-lbaas | 18:06 | |
| *** Youcef has joined #openstack-lbaas | 18:08 | |
| *** vivek-eb_ has quit IRC | 18:17 | |
| *** vivek-ebay has joined #openstack-lbaas | 18:17 | |
| *** vivek-ebay has quit IRC | 18:22 | |
| *** vivek-ebay has joined #openstack-lbaas | 18:22 | |
| *** jorgem has quit IRC | 18:24 | |
| openstackgerrit | Pauline Yeung proposed openstack/neutron-lbaas: Block subnet gateway IP to be used as LB VIP https://review.openstack.org/145877 | 18:38 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-lbaas: Updated from global requirements https://review.openstack.org/145670 | 18:45 |
| xgerman | ^^ proposal-bot -- OpenStack is self aware and self modifying... | 18:53 |
| dougwig | a2hill: just confirming that you or carlos are going to re-propose all of this change to the new repo? | 18:53 |
| dougwig | https://www.irccloud.com/pastebin/rtVsHRbU | 18:53 |
| dougwig | a2hill: and are you re-proposing this one? | 18:53 |
| dougwig | https://www.irccloud.com/pastebin/aHiq2kUM | 18:53 |
| a2hill | the above? | 18:54 |
| a2hill | yea, thats been done | 18:54 |
| a2hill | i believe | 18:54 |
| a2hill | https://review.openstack.org/#/c/145151/ | 18:54 |
| a2hill | im not sure about the tls utils stuff | 18:55 |
| dougwig | did you merge two commits, because there was also a review with that subject in the feature branch (there were two) | 18:55 |
| dougwig | https://review.openstack.org/#/q/openstack/neutron++branch:feature/lbaasv2,n,z | 18:55 |
| a2hill | yea, it was moved a couple times | 18:56 |
| dougwig | ahh, yes, the common tis was a question for rm_work and carlos. | 18:56 |
| dougwig | ok, i'll mark both of those as moved/dealt with. thanks. | 18:57 |
| *** sbalukoff has joined #openstack-lbaas | 19:22 | |
| *** vivek-ebay has quit IRC | 19:27 | |
| *** kobis has joined #openstack-lbaas | 19:56 | |
| *** SumitNaiksatam has quit IRC | 19:56 | |
| *** sbfox has quit IRC | 20:02 | |
| *** jorgem has joined #openstack-lbaas | 20:02 | |
| rm_work | dougwig: there's half of the TLS work that might need to be ported still to neutron-lbaas | 20:23 |
| rm_work | dougwig: but it depends somewhat on the velocity of the Castellan project | 20:23 |
| rm_work | dougwig: I'm planning to start on it today probably, put it in review WIP, and then find out next week whether I should just abandon it in favor of Castellan integration or not | 20:24 |
| *** vivek-ebay has joined #openstack-lbaas | 20:28 | |
| *** vivek-ebay has quit IRC | 20:32 | |
| *** vivek-ebay has joined #openstack-lbaas | 20:44 | |
| dougwig | rm_work: what is castellan? | 20:45 |
| *** vivek-eb_ has joined #openstack-lbaas | 20:47 | |
| *** vivek-ebay has quit IRC | 20:47 | |
| rm_work | dougwig: https://github.com/openstack/castellan | 20:48 |
| *** kobis has quit IRC | 20:48 | |
| *** sbfox has joined #openstack-lbaas | 20:49 | |
| rm_work | dougwig: essentially what I was writing previously, but now it's official and in its own project | 20:50 |
| *** blogan_ has joined #openstack-lbaas | 20:57 | |
| blogan_ | arg | 20:58 |
| *** jorgem has quit IRC | 21:06 | |
| *** blogan_ has quit IRC | 21:08 | |
| xgerman | rm_work: documentation->Not Found | 21:26 |
| rm_work | <_< | 21:26 |
| xgerman | http://docs.openstack.org/developer/castellan | 21:26 |
| rm_work | yeah possibly | 21:26 |
| rm_work | the repo JUST got merged recently | 21:27 |
| rm_work | The one-liner description is the important part | 21:27 |
| *** mwang2 has joined #openstack-lbaas | 21:27 | |
| rm_work | I'm the one writing the spec for the certificate part, so I'm not worried about it being good for our use-case :P | 21:27 |
| xgerman | neat | 21:28 |
| *** jorgem has joined #openstack-lbaas | 21:33 | |
| *** crc32 has joined #openstack-lbaas | 22:06 | |
| rm_work | ugh, slight changes to controller.dot and it just looks horrible now T_T | 22:07 |
| *** apuimedo has joined #openstack-lbaas | 22:16 | |
| openstackgerrit | min wang proposed stackforge/octavia: amphora driver interface v0.5 changes https://review.openstack.org/141948 | 22:22 |
| dougwig | rm_work: when will you know which direction we take? if castellan (good job, btw) might not make kilo, or even kilo-2, we might want a workable prototype code in place for people to be working with/testing. | 22:23 |
| rm_work | dougwig: yeah, I will find out more Monday | 22:25 |
| rm_work | it's possible we'll make a big push during the Barbican meetup in Feb (the week after our meetup) | 22:25 |
| dougwig | what's missing from lbaas v2 right now for working terminated_https support? | 22:26 |
| rm_work | but like I said, I'll try to get the alternative ready in a WIP review in case it looks like it won't make Kilo | 22:26 |
| rm_work | the second half of the TLS code, basically | 22:26 |
| rm_work | which was in the review on the old branch, but when we did the new repo, carlos split out his half and I need to do my half still | 22:26 |
| dougwig | ok | 22:27 |
| dougwig | ty | 22:27 |
| rm_work | Was looking at fixing controller.dot / controller.rst but I think I will prioritize that right now | 22:27 |
| rm_work | err, I will prioritize TLS, and switch to that presently | 22:27 |
| johnsom | rm_work What changes are you thinking for the controller.rst? | 22:28 |
| rm_work | johnsom: API Manager shouldn't need to communicate with the DB (so remove that line) | 22:29 |
| rm_work | and API Manager should have a link directly to Deploy Worker (which should either be a different color or shape, to highlight the fact that it is just shared code / thread, not a daemon) | 22:30 |
| rm_work | and the text needs to be updated to reflect that Deploy Worker will be responsible for CUD, not just C | 22:30 |
| rm_work | API Manager is just a thin layer that pulls from the queue and schedules | 22:30 |
| rm_work | (either with TaskFlow directly or not, doesn't really matter) | 22:31 |
| rm_work | per jorgem's comments yesterday on the merged review | 22:31 |
| johnsom | Yeah, I commented on that. I will make an update. | 22:32 |
| jorgem | johnsom: oh yeah I put another comment today :) | 22:32 |
| rm_work | ok, you'll handle it? | 22:32 |
| rm_work | then I don't have to worry about this stupid graphviz issue where it shifts everything around to look absolutely horrible? :P | 22:32 |
| johnsom | Yeah, I was just waiting for all of the feedback to come in. I think blogan wanted to comment too | 22:32 |
| rm_work | k, jorgem told me to take a look at it today but I am perfectly happy to leave it to you | 22:33 |
| rm_work | so I can concentrate on the remainder of my TLS work | 22:33 |
| johnsom | Well, since you mentioned graphviz.... What is up with it? | 22:33 |
| rm_work | I already tried making said changes, and it looks... hideous | 22:33 |
| rm_work | it shifts everything around so the lines are all ridiculous | 22:34 |
| rm_work | it's just... bad | 22:34 |
| johnsom | I will take a look and hate it more as well | 22:34 |
| rm_work | :) | 22:35 |
| blogan | johnsom: i have no other comments left, jorgem can now ask for himself | 22:38 |
| johnsom | Ok, cool | 22:38 |
| rm_work | hmm there's no neutron_lbaas/common ? >_> | 22:39 |
| a2hill | yea? | 22:40 |
| a2hill | there is | 22:40 |
| a2hill | oh | 22:40 |
| a2hill | looks like there is after carlos' commit | 22:40 |
| rm_work | yeah I guess I'll do my work on top of his | 22:40 |
| openstackgerrit | min wang proposed stackforge/octavia: amphora driver interface v0.5 changes https://review.openstack.org/141948 | 22:41 |
| a2hill | is this for the get_container stuffs? | 22:41 |
| rm_work | https://review.openstack.org/#/c/123492/11/neutron/common/certmgr/certmgr.py | 22:44 |
| rm_work | that interface and the implementation | 22:45 |
| a2hill | guess im curious where the work is that the TLS extention/reference impl is waiting on to retrieve the container | 22:46 |
| rm_work | yeah, this | 22:46 |
| rm_work | but to you it isn't a "container", it's just a "cert" | 22:47 |
| a2hill | whats the cert_ref is that the container_id? | 22:47 |
| rm_work | yes | 22:48 |
| a2hill | ah, kk cool | 22:48 |
| a2hill | I forget, is the driver responsible for storing the cert or is that done further up? its been a while | 22:50 |
| *** apuimedo has quit IRC | 22:51 | |
| dougwig | the driver. | 22:52 |
| dougwig | neutron lbaas just stores the barbican id | 22:52 |
| a2hill | but if it has a barbican_id did it not store it already? | 22:52 |
| a2hill | sorry, can ignore these questions, ill have to dig back into it all | 22:53 |
| a2hill | derp | 22:58 |
| sbalukoff | Just call me the dissenter crusher. | 23:03 |
| sbalukoff | Or quencher. | 23:03 |
| sbalukoff | Whatever. | 23:03 |
| a2hill | youre a horrible person sbalukoff | 23:04 |
| sbalukoff | Thank you. | 23:04 |
| a2hill | :P | 23:04 |
| a2hill | In either case you are less horrible then blogan, so i dont know if thats a win for you or not | 23:05 |
| a2hill | oh, hahah, is blogan even in here? Hes so bad even his computer gave up on him | 23:05 |
| blogan | ive hobbled in | 23:05 |
| a2hill | :P | 23:06 |
| *** crc32 has quit IRC | 23:06 | |
| *** jorgem has quit IRC | 23:18 | |
| sbalukoff | Haha | 23:18 |
| rm_work | a2hill: neutron-lbaas stores tls_certificate_id in the DB, the driver is responsible for pulling it from Barbican and passing it to the haproxy system | 23:20 |
| rm_work | or, it should be | 23:20 |
| rm_work | I am not sure what state we left the interface in | 23:20 |
| rm_work | possibly right now the driver interface expects the cert to be passed directly? but that is not what we want to do | 23:20 |
| a2hill | yea, that makes sense, i think i asked the wrong questions at first | 23:20 |
| *** ajmiller_ has joined #openstack-lbaas | 23:30 | |
| *** ajmiller has quit IRC | 23:33 | |
| dougwig | i'm apparently very rude, so where does that put me on the horrible scale? | 23:39 |
| *** ajmiller_ has quit IRC | 23:39 | |
| *** mlavalle has quit IRC | 23:45 | |
| *** sbfox has quit IRC | 23:48 | |
| *** sbfox has joined #openstack-lbaas | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!