| *** woodster_ has joined #openstack-lbaas | 00:22 | |
| *** Tiancheng has joined #openstack-lbaas | 00:48 | |
| *** amotoki has joined #openstack-lbaas | 01:25 | |
| openstackgerrit | Merged openstack/neutron-lbaas: Switch from neutron.common.log.log to oslo_log.helpers.log_method_call https://review.openstack.org/172484 | 02:25 |
|---|---|---|
| *** woodster_ has quit IRC | 03:20 | |
| *** woodster_ has joined #openstack-lbaas | 03:47 | |
| *** ajmiller has quit IRC | 03:50 | |
| *** crc32 has joined #openstack-lbaas | 03:54 | |
| *** crc32 has quit IRC | 03:58 | |
| *** rm_you has quit IRC | 04:08 | |
| *** rm_you has joined #openstack-lbaas | 04:12 | |
| *** rm_you has quit IRC | 04:12 | |
| *** rm_you has joined #openstack-lbaas | 04:12 | |
| *** _kiran_ has joined #openstack-lbaas | 04:15 | |
| *** _kiran_ is now known as kiran-r | 04:15 | |
| *** rm_work|away is now known as rm_work | 04:17 | |
| *** Varun_Lodaya has joined #openstack-lbaas | 05:02 | |
| *** kobis has joined #openstack-lbaas | 05:55 | |
| Santosh_NS | Hi | 05:55 |
| Santosh_NS | ping | 05:56 |
| openstackgerrit | Brandon Logan proposed stackforge/octavia: Integration tests to test out driver https://review.openstack.org/172800 | 06:17 |
| openstackgerrit | Brandon Logan proposed stackforge/octavia: Fixed sessionpersistence and healthmonitor refs https://review.openstack.org/172759 | 06:22 |
| blogan | hi Santosh_NS | 06:29 |
| rm_work | blogan: 172759 passed tests this time, and looks good to me -- was there anything else you were planning to do with it or is that good to go? | 06:34 |
| *** apuimedo has joined #openstack-lbaas | 06:45 | |
| *** Varun_Lodaya has quit IRC | 06:52 | |
| rm_work | blogan: still around? | 06:56 |
| *** blogan_ has joined #openstack-lbaas | 07:01 | |
| blogan_ | rm_you: im here now | 07:01 |
| blogan_ | rm_work: thats all i wanted to do with it | 07:01 |
| rm_work | ok | 07:02 |
| rm_work | +2 coming then | 07:02 |
| rm_work | blogan_: can you just…. fix https://review.openstack.org/#/c/172556/ please | 07:05 |
| rm_work | so I can +2 with a clean conscience? >_> | 07:05 |
| rm_work | those over-indented docstrings are a scourge on our codebase | 07:06 |
| rm_work | already went through and took care of some myself in another doc | 07:06 |
| blogan_ | rm_work: bleh! | 07:06 |
| rm_work | hey, johnsom -1'd as well :P | 07:06 |
| rm_work | just shift-tab a bit T_T | 07:07 |
| rm_work | and no linebreak on the first line <_< | 07:07 |
| rm_work | in fact if you'd fix ALL of them in that file | 07:07 |
| rm_work | that'd be super | 07:07 |
| blogan_ | is there 80 char limit on these specs? | 07:08 |
| rm_work | …….. there SHOULD be | 07:08 |
| rm_work | but no | 07:08 |
| blogan_ | these are just docs, so they're not real docstrings you know? | 07:08 |
| rm_work | those docstrings trigger my OCD really badly | 07:09 |
| blogan_ | since i can understand that feeling, i will fix | 07:09 |
| rm_work | also i could show you where people have copy/pasted from them in the past, leading to incorrectness in actual code :P | 07:10 |
| rm_work | but i appreciate it | 07:10 |
| blogan_ | well those should be caught in review! | 07:11 |
| rm_work | yes but apparently other people care less, and some things get workflowed without me looking at them and -1ing T_T | 07:11 |
| blogan_ | actually thsoe should be caught by pep8/hacking rules | 07:12 |
| blogan_ | i think octavia's pep8 hacking rules are fubarred htough | 07:13 |
| rm_work | yeah as i have pointed out, hacking isn't running right on octavia | 07:13 |
| rm_work | yeah | 07:13 |
| rm_work | trevor kept trying to violate hacking rules for import order last week, and showing me how it wasn't complaining so he must be right :P | 07:14 |
| rm_work | which is when i first noticed they weren't running correctly | 07:14 |
| blogan_ | ive noticed for a bit but been too lazy to investigate | 07:14 |
| blogan_ | it was but a blip on my care-o-meter | 07:14 |
| rm_work | if i have any extra care to spare tomorrow i'll look at it | 07:15 |
| rm_work | but i am somewhat afraid of all the things i might have to fix to merge it T_T | 07:15 |
| openstackgerrit | Brandon Logan proposed stackforge/octavia: Added post_vip_plug method to the amphora driver https://review.openstack.org/172556 | 07:18 |
| blogan_ | ^^ | 07:18 |
| blogan_ | better to do it now than later when its even worse | 07:19 |
| rm_work | thx | 07:19 |
| blogan_ | if i lived up to that mantra, it would have been fixed long ago | 07:19 |
| openstackgerrit | Brandon Logan proposed stackforge/octavia: Added post_vip_plug method to the amphora driver https://review.openstack.org/172556 | 07:19 |
| blogan_ | rm_you: ^^ | 07:19 |
| rm_work | k | 07:20 |
| rm_work | was gonna say | 07:20 |
| rm_work | you missed a few | 07:20 |
| blogan_ | had a misspelling | 07:20 |
| blogan_ | no i didnt | 07:20 |
| rm_work | ah, maybe still missed a few | 07:20 |
| rm_work | update/stop/start/delete | 07:20 |
| blogan_ | FFFFFF | 07:20 |
| openstackgerrit | Brandon Logan proposed stackforge/octavia: Added post_vip_plug method to the amphora driver https://review.openstack.org/172556 | 07:21 |
| blogan_ | rm_work: ^^^ | 07:22 |
| * blogan_ kicks rm_work | 07:22 | |
| rm_work | :P | 07:22 |
| blogan_ | if someone complains about the fixes being out of scope of the review i will kick them too | 07:22 |
| * blogan_ waits for dougwig to complain about it | 07:22 | |
| rm_work | technically "update" is a bit wonky because the first line is supposed to be a summary and then a blank and then more detail, not just a blank and then the continuation of the summary sentence | 07:23 |
| rm_work | BUT | 07:23 |
| rm_work | this is where i draw the line of caring | 07:23 |
| blogan_ | i didnt write it | 07:23 |
| blogan_ | so you go to hell | 07:23 |
| rm_work | at least it's technically formatted properly | 07:23 |
| rm_work | so +2 :P | 07:23 |
| blogan_ | first line is actually a good summary | 07:23 |
| blogan_ | the next line just continues | 07:23 |
| rm_work | yes. technically fine. | 07:24 |
| blogan_ | don't see anything in the formatting rules that the summary can't work as a standalone sentence and then also be continued on | 07:24 |
| *** chlong has quit IRC | 07:25 | |
| rm_work | lol | 07:25 |
| rm_work | guess not | 07:25 |
| blogan_ | loophole! | 07:25 |
| rm_work | it just violates the spirit of the thing :P | 07:25 |
| blogan_ | all my docstrings now will start with "This is a method | 07:26 |
| rm_work | see: https://review.openstack.org/#/c/172568/ | 07:26 |
| blogan_ | -1 for those whitespace fixes being out of scope | 07:26 |
| rm_work | >_< | 07:27 |
| *** Tiancheng has quit IRC | 07:27 | |
| blogan_ | alright bed time, gotta get up early | 07:29 |
| blogan_ | talk to you later | 07:29 |
| blogan_ | prob won't be on tomorrow until after lunch | 07:29 |
| rm_work | kk | 07:30 |
| rm_work | later | 07:30 |
| *** blogan_ has quit IRC | 07:30 | |
| *** Tiancheng has joined #openstack-lbaas | 07:33 | |
| *** woodster_ has quit IRC | 07:40 | |
| *** Tiancheng has quit IRC | 08:42 | |
| *** haigang has joined #openstack-lbaas | 08:46 | |
| *** Tiancheng has joined #openstack-lbaas | 08:54 | |
| *** haigang has quit IRC | 09:13 | |
| *** haigang has joined #openstack-lbaas | 09:15 | |
| *** haigang has quit IRC | 09:27 | |
| *** haigang has joined #openstack-lbaas | 09:28 | |
| *** haigang has quit IRC | 09:54 | |
| *** haigang has joined #openstack-lbaas | 09:55 | |
| *** haigang has quit IRC | 10:00 | |
| *** Tiancheng has quit IRC | 10:18 | |
| *** john-davidge has joined #openstack-lbaas | 10:54 | |
| john-davidge | Hi all o/ | 11:17 |
| john-davidge | Currently trying to track down what's causing all of these failures in the neutron lbaas CI http://logs.openstack.org/97/158697/21/check/check-neutron-lbaas-dsvm-api/8adba15/logs/testr_results.html.gz for patch https://review.openstack.org/#/c/158697/ | 11:18 |
| john-davidge | If anyone could give me some pointers as to what chnage in Neutron could be causing these lbaas failues that would be great | 11:18 |
| *** rdekel has joined #openstack-lbaas | 11:28 | |
| rdekel | Does anyone have a link for a document containing LBaaS API v2? Or perhaps which features will be implemented with it? | 11:37 |
| *** amotoki has quit IRC | 11:45 | |
| *** Tiancheng has joined #openstack-lbaas | 12:59 | |
| *** openstackgerrit has quit IRC | 13:00 | |
| *** openstackgerrit has joined #openstack-lbaas | 13:03 | |
| *** yfried has joined #openstack-lbaas | 13:13 | |
| *** yfried has left #openstack-lbaas | 13:13 | |
| openstackgerrit | Evgeny Fedoruk proposed openstack/neutron-lbaas: Use listener and pool UUIDs in Radware LBaaS https://review.openstack.org/172911 | 13:16 |
| *** woodster_ has joined #openstack-lbaas | 13:33 | |
| *** ajmiller has joined #openstack-lbaas | 14:15 | |
| *** Tiancheng has quit IRC | 14:54 | |
| *** kiran-r has quit IRC | 14:58 | |
| *** TrevorV_ has joined #openstack-lbaas | 15:13 | |
| *** rdekel has quit IRC | 15:17 | |
| *** sbalukoff has quit IRC | 15:18 | |
| *** xgerman_ has joined #openstack-lbaas | 15:19 | |
| *** TrevorV_ has quit IRC | 15:19 | |
| *** Varun_Lodaya has joined #openstack-lbaas | 15:20 | |
| openstackgerrit | Adam Harwell proposed stackforge/octavia: Fix arg order for Update functions in Controller-Worker https://review.openstack.org/172568 | 15:20 |
| *** Varun_Lodaya has quit IRC | 15:49 | |
| *** kobis has quit IRC | 15:50 | |
| *** mlavalle has joined #openstack-lbaas | 15:51 | |
| openstackgerrit | Trevor Vardeman proposed stackforge/octavia: Amphora SSH Driver https://review.openstack.org/160964 | 15:51 |
| TrevorV | xgerman_ you online my man? | 15:57 |
| *** jorgem has joined #openstack-lbaas | 15:57 | |
| xgerman_ | yep | 15:57 |
| TrevorV | you commented the word "clarification" but nothing along with it... Did you mean to post a comment as well? | 15:58 |
| TrevorV | on the ssh_driver | 15:58 |
| xgerman_ | did it not post :-( | 15:58 |
| TrevorV | Just the word "clarification"... It might be on the previous patch set though. | 15:59 |
| xgerman_ | Ok, I see it's still draft --- must have been your commit while I was reviewing | 15:59 |
| xgerman_ | posted it as a comment. I am basiclaly with Adam but I like the ability to put all the certs in one place on the file system and the rest somewhere else | 16:01 |
| TrevorV | Right right.. So let me ask you this, would it be impossible to mount a ramdisk directory IN that directory? | 16:02 |
| TrevorV | For example: | 16:02 |
| xgerman_ | not if you have like thre listeners | 16:02 |
| TrevorV | Really? | 16:03 |
| xgerman_ | your directory structure is | 16:03 |
| xgerman_ | <listener-i>/certs | 16:03 |
| xgerman_ | <listener-id>/haproxy.cfg, etc. | 16:03 |
| xgerman_ | so for each listener certs would need to be a link to the ramdisk | 16:03 |
| xgerman_ | I would prefer /mnr/ramdisk/certs/... | 16:04 |
| xgerman_ | and then /var/haproxy/... for the congigs and the rest | 16:04 |
| TrevorV | So what if I just added another base_path variable for the certs... like: self.amp_config.cert_base_path or something? | 16:05 |
| xgerman_ | that would be perfect | 16:05 |
| TrevorV | Then, if someone didn't want to do ramdisk for some reason, they could just set both to the same location and it wouldn't have a problem, right? | 16:05 |
| xgerman_ | yep | 16:05 |
| ptoohill | +1 | 16:06 |
| xgerman_ | yeah, but as rm_work is pointing out we are splitting hairs | 16:06 |
| xgerman_ | since we could as well throw the 3kb haproxy config on the ramdisk | 16:06 |
| xgerman_ | but I like to have it on disk for debugging (if the thing crashes) | 16:07 |
| TrevorV | Isn't that what logs are for xgerman_ :P | 16:07 |
| xgerman_ | yep, but seeing the config might be good, too | 16:07 |
| ptoohill | https://review.openstack.org/#/c/166487/6 Cant run tempest test that live within our codebase as it stands right now. Just thoughts others would like to know. | 16:08 |
| TrevorV | So xgerman_ are you a -1 unless I put in the extra base path, or since its "splitting hairs" it really doesn't matter? | 16:09 |
| xgerman_ | I am 0 | 16:09 |
| TrevorV | So really doesn't matter. Alright, we can always refactor right? | 16:10 |
| rm_work | yeah I think we move on | 16:10 |
| ptoohill | This extra path wouldnt be needed by jinja, just the driver. So thats one less thing to worry about if you do update it | 16:10 |
| rm_work | the SSH driver is otherwise pretty damn close to ready, right? | 16:10 |
| xgerman_ | yeah, there are still a few edges but it's good enough for the demo | 16:11 |
| TrevorV | rm_work I think it IS ready. blogan was going through all the individual methods in a full end to end run, and hadn't gone through everything yet, but as far as I know its ready to get reviewed and merged | 16:11 |
| TrevorV | xgerman_ what edges, if you don't mind me asking? | 16:11 |
| TrevorV | Like the lack of a "get_details" implementation? :P | 16:11 |
| *** vivek-ebay has joined #openstack-lbaas | 16:12 | |
| xgerman_ | more the lack of the upstart script to start haproxy | 16:12 |
| xgerman_ | get_details was optional anyway + if you have ssh you cam have ops script that anyway | 16:13 |
| TrevorV | Yep. | 16:13 |
| ptoohill | what upstart script? | 16:13 |
| ptoohill | If its what im thinking would that not be something handled by the image? | 16:13 |
| xgerman_ | we have seen haproxy crash and I like it to be automticlaly restrated in that case | 16:13 |
| TrevorV | I totally forgot about the upstart script... I was looking at your review one day for how you did it, and then got side-tracked on something else and totally forgot | 16:13 |
| ptoohill | oh | 16:13 |
| rm_work | we need to install an upstart/systemd/init/WHATEVER script to make haproxy start automatically right? | 16:13 |
| rm_work | heh yeah | 16:14 |
| xgerman_ | restart | 16:14 |
| rm_work | so one interesting thing about that though | 16:14 |
| xgerman_ | you can start with haproxy -f just fine | 16:14 |
| rm_work | is that it kinda depends on the image | 16:14 |
| rm_work | since everything is using its own freaking init system | 16:14 |
| rm_work | unless everyone has finally agreed to use a common one while i wasn't looking | 16:15 |
| xgerman_ | yep, if you are running redhat | 16:15 |
| openstackgerrit | Merged stackforge/octavia: Added post_vip_plug method to the amphora driver https://review.openstack.org/172556 | 16:15 |
| *** Varun_Lodaya has joined #openstack-lbaas | 16:15 | |
| TrevorV | Shit, I also forgot that I'd need that method... the post_vip_plug method in the SSH driver... guess its not done yet | 16:16 |
| TrevorV | Oh, and the network one... damnit damnit damnit | 16:16 |
| xgerman_ | well, you can probably merge without the init script. Haproxy crashing has been rare but when it happens it's not juct one... | 16:16 |
| *** apuimedo has quit IRC | 16:17 | |
| openstackgerrit | Trevor Vardeman proposed stackforge/octavia: Amphora SSH Driver https://review.openstack.org/160964 | 16:17 |
| *** kiran has joined #openstack-lbaas | 16:17 | |
| TrevorV | xgerman_ the startup script can be done, like I said I just forgot about it. | 16:18 |
| xgerman_ | ok, cool | 16:18 |
| TrevorV | But those other two methods are definitely not implemented yet, so its got a bit to go | 16:18 |
| rm_work | i guess you'd do the startup script assuming Ubuntu 14.04? | 16:22 |
| TrevorV | I could, sure. | 16:23 |
| TrevorV | Or figure out some kind of "what os are you" method that picks a way to do a restart dynamically based on OS... :P | 16:23 |
| rm_work | YEAH | 16:23 |
| rm_work | err caps | 16:23 |
| *** kiran is now known as kiran-r | 16:24 | |
| rm_work | yeah, maybe use jinja templates for the script | 16:24 |
| TrevorV | You're a jinja template | 16:24 |
| rm_work | MAYBE I AM, WHAT OF IT | 16:24 |
| TrevorV | caps | 16:24 |
| * rm_work thinks TrevorV is racist against Jinja templates | 16:24 | |
| johnsom | The element should handle that in the image builder (not that folks are using it) | 16:24 |
| TrevorV | The element? | 16:24 |
| ptoohill | triple0 | 16:25 |
| rm_work | johnsom: the FIFTH element? | 16:25 |
| rm_work | ah | 16:25 |
| ptoohill | seventh | 16:25 |
| TrevorV | I don't understand the context still. | 16:25 |
| ptoohill | the thing used to build image | 16:25 |
| TrevorV | Except the fifth element, I get that one | 16:25 |
| rm_work | :P | 16:25 |
| *** madhu_ak has joined #openstack-lbaas | 16:25 | |
| johnsom | Yeah, we actually have seven elements now.... girn | 16:25 |
| ptoohill | i was thinking that too, it should be able to handle scripts like that | 16:25 |
| ptoohill | :D | 16:25 |
| johnsom | girn, this is how my Monday morning is going. Coffee infusion is slow today | 16:26 |
| TrevorV | I still can't say I follow... The thing used to build the image shouldn't have any knowledge of config values. | 16:26 |
| ptoohill | restarting haproxy? | 16:27 |
| ptoohill | whats config values are specific? | 16:27 |
| TrevorV | How can you writ a restart script if you don't know where haproxy is getting its cnofigs. | 16:27 |
| TrevorV | configs*** | 16:27 |
| TrevorV | write* | 16:27 |
| ptoohill | hmm | 16:27 |
| xgerman_ | yep, that would only work if you are not doing one haproxy per listener but ONE haproxy to rule all listeners | 16:27 |
| ptoohill | good point i suppose. if it were in common location | 16:27 |
| ptoohill | well dern | 16:28 |
| johnsom | The restart script is really tied to the haproxy package installed and the base OS, so both are selected and install at disk image time | 16:28 |
| johnsom | Yeah, what xgerman_ said is true too | 16:28 |
| TrevorV | ptoohill what is different between haproxy listener configs? | 16:29 |
| ptoohill | ? | 16:29 |
| johnsom | You could probably get fancy with the script and some /etc/defaults though. | 16:29 |
| TrevorV | In other words, why do we NOT have one single config file? | 16:29 |
| TrevorV | Its differentiated on listener values that the customer requests right? | 16:30 |
| xgerman_ | there was a vote we johnsom + I lost | 16:30 |
| TrevorV | Nvmd | 16:30 |
| ptoohill | because we are doing a process per listener | 16:30 |
| TrevorV | Yeah, I processed that ha ha | 16:30 |
| johnsom | Yep, bedis has asked the same question | 16:30 |
| TrevorV | So yeah, we'd have to start said upstart script with the program name and parameters provided. | 16:31 |
| TrevorV | Granted, if we have a script that allows for those parameteres to be passed in, then we're gucci ya know? | 16:31 |
| ptoohill | it will be multiple scripts in this case i believe | 16:32 |
| TrevorV | brb checking out phase 5 of rax | 16:32 |
| dougwig | morning | 16:32 |
| ptoohill | mornin' | 16:32 |
| *** dougwig has quit IRC | 16:35 | |
| *** dougwig has joined #openstack-lbaas | 16:35 | |
| *** dougw2 has joined #openstack-lbaas | 16:36 | |
| openstackgerrit | Phillip Toohill proposed stackforge/octavia: Preparing for tempest testing https://review.openstack.org/172199 | 16:38 |
| *** jorgem has quit IRC | 16:42 | |
| openstackgerrit | Al Miller proposed stackforge/octavia: Add devstack plugin for octavia https://review.openstack.org/167796 | 16:47 |
| ajmiller | ptoohill I posted a patch addressing your comments. Everyone else: ^^ that patch adds a set of sample config scripts that automatically set up a running loadbalancer in devstack. | 16:57 |
| ptoohill | Was checking them out, looks good! Thank you! | 16:58 |
| *** jorgem has joined #openstack-lbaas | 16:58 | |
| ptoohill | Its a +1 from me after jenkins passes. Thanks again! | 16:58 |
| *** xgerman_ has quit IRC | 16:59 | |
| *** xgerman__ has joined #openstack-lbaas | 16:59 | |
| *** xgerman__ has quit IRC | 16:59 | |
| *** sbfox has joined #openstack-lbaas | 17:00 | |
| ajmiller | Funny story, I had been using those scripts on my own for a while, just keeping them on my devstack machine. Then I stumbled on a bug in devstack that caused me to accidentally delete them when running clean.sh. Had to get them back from a snapshot (thankfully had one). So I decided if they were useful to me, I'd actually pretty them up and make them available to everyone, and store them in a safe place... | 17:01 |
| ptoohill | :D | 17:02 |
| ptoohill | Indeed useful, i appreciate it :) | 17:02 |
| rm_work | blogan / dougwig / dougw2 https://review.openstack.org/#/c/172568/ | 17:06 |
| rm_work | brb making some loonch | 17:06 |
| *** sbfox has quit IRC | 17:12 | |
| *** kiran-r has quit IRC | 17:14 | |
| *** sbfox has joined #openstack-lbaas | 17:17 | |
| *** kiran has joined #openstack-lbaas | 17:19 | |
| *** xgerman has joined #openstack-lbaas | 17:21 | |
| *** SumitNaiksatam has joined #openstack-lbaas | 17:28 | |
| openstackgerrit | min wang proposed stackforge/octavia: Add network noop driver https://review.openstack.org/172601 | 17:35 |
| openstackgerrit | John Davidge proposed openstack/neutron-lbaas: Update subnet data model to include ipv6_pd_enabled https://review.openstack.org/173005 | 17:35 |
| openstackgerrit | John Davidge proposed openstack/neutron-lbaas: Update subnet data model to include ipv6_pd_enabled https://review.openstack.org/173005 | 17:36 |
| *** jorgem has quit IRC | 17:42 | |
| openstackgerrit | min wang proposed stackforge/octavia: Add compute noop driver and test case https://review.openstack.org/172606 | 17:43 |
| *** john-davidge has quit IRC | 17:43 | |
| *** bharath has joined #openstack-lbaas | 17:51 | |
| xgerman | rm_work, blogan: https://review.openstack.org/#/c/172606/ | 17:53 |
| johnsom | https://review.openstack.org/#/c/172601/ as well | 17:53 |
| *** madhu_ak_ has joined #openstack-lbaas | 17:54 | |
| *** Aish has joined #openstack-lbaas | 17:55 | |
| *** kiran has quit IRC | 17:56 | |
| *** madhu_ak has quit IRC | 17:58 | |
| *** madhu_ak__ has joined #openstack-lbaas | 18:02 | |
| *** jorgem has joined #openstack-lbaas | 18:05 | |
| *** madhu_ak_ has quit IRC | 18:05 | |
| *** SumitNaiksatam has quit IRC | 18:07 | |
| *** sbalukoff has joined #openstack-lbaas | 18:09 | |
| *** madhu_ak__ has quit IRC | 18:11 | |
| *** madhu_ak has joined #openstack-lbaas | 18:12 | |
| *** ajmiller_ has joined #openstack-lbaas | 18:29 | |
| openstackgerrit | Merged stackforge/octavia: Fix arg order for Update functions in Controller-Worker https://review.openstack.org/172568 | 18:30 |
| *** SumitNaiksatam has joined #openstack-lbaas | 18:31 | |
| *** ajmiller has quit IRC | 18:33 | |
| rm_work | xgerman: comment on 172606, needs to be fixed | 18:33 |
| rm_work | also | 18:34 |
| rm_work | One or the other of those should be dependent reviews? but it looks like they're both independent? | 18:34 |
| rm_work | sorry nm | 18:34 |
| rm_work | network / compute | 18:35 |
| rm_work | different | 18:35 |
| johnsom | Darn, we both missed those %s | 18:35 |
| johnsom | Yes, different | 18:35 |
| xgerman | +! | 18:36 |
| xgerman | glad I didn't +A | 18:36 |
| rm_work | workflow'd the other one tho | 18:36 |
| xgerman | thanks | 18:36 |
| johnsom | Min is going to fix 172606 now | 18:37 |
| openstackgerrit | Merged stackforge/octavia: Add network noop driver https://review.openstack.org/172601 | 18:39 |
| rm_work | kk | 18:41 |
| rm_work | I think otherwise it is fine | 18:41 |
| xgerman | cool | 18:42 |
| rm_work | or else can be fixed quickly once we have everything in a "running" state | 18:42 |
| rm_work | I am willing to let a lot of stuff slide right now I think, for the purposes of a working end-to-end demo, which we can clean up as we need if something is broken | 18:42 |
| xgerman | well, we should still aim for quality | 18:44 |
| openstackgerrit | min wang proposed stackforge/octavia: Add compute noop driver and test case https://review.openstack.org/172606 | 18:45 |
| rm_work | well, sure :P | 18:45 |
| rm_work | but some of this stuff just isn't obvious whether it's right or not until we get the machine in motion, I think | 18:46 |
| rm_work | and IMO we should fast-track that happening | 18:46 |
| rm_work | tempted to get the HUGE controller-worker CR merged even if it might have a couple of issues still, just so we can get stuff spinning and fix those in some nice, small, easily digestible followup CRs | 18:47 |
| rm_work | johnsom: I think that is the largest CR I've ever reviewed. Thanks for that. :P | 18:48 |
| rm_work | johnsom: ah, but the arg order spec update merged, so you can fix that now, if you'd like :) | 18:49 |
| xgerman | neat. merging the controller is good | 18:51 |
| *** fnaval has joined #openstack-lbaas | 18:51 | |
| *** ajmiller_ has quit IRC | 19:00 | |
| *** mwang2 has joined #openstack-lbaas | 19:01 | |
| *** ajmiller has joined #openstack-lbaas | 19:07 | |
| openstackgerrit | Merged stackforge/octavia: Add compute noop driver and test case https://review.openstack.org/172606 | 19:08 |
| *** jorgem has quit IRC | 19:09 | |
| openstackgerrit | Merged stackforge/octavia: Fixed sessionpersistence and healthmonitor refs https://review.openstack.org/172759 | 19:26 |
| *** vivek-ebay has quit IRC | 19:31 | |
| *** sbalukoff has quit IRC | 19:36 | |
| *** mwang2 has quit IRC | 19:37 | |
| *** sbalukoff has joined #openstack-lbaas | 19:48 | |
| *** madhu_ak has quit IRC | 19:59 | |
| *** mwang2 has joined #openstack-lbaas | 19:59 | |
| johnsom | I am thinking we get the update/db stuff in and clean up a few of the TODOs and push for merge. We are close to being about to run it on devstack, so getting a pass on that would be good. | 20:00 |
| rm_work | yep | 20:04 |
| xgerman | +! | 20:10 |
| *** sbfox has quit IRC | 20:24 | |
| *** madhu_ak has joined #openstack-lbaas | 20:28 | |
| *** sbfox has joined #openstack-lbaas | 20:30 | |
| *** vivek-ebay has joined #openstack-lbaas | 20:31 | |
| *** HenryG has quit IRC | 20:33 | |
| *** vivek-ebay has quit IRC | 20:36 | |
| *** vivek-ebay has joined #openstack-lbaas | 20:36 | |
| *** HenryG has joined #openstack-lbaas | 20:37 | |
| blogan | hello everybody | 20:39 |
| *** apuimedo has joined #openstack-lbaas | 20:41 | |
| johnsom | Afternoon | 20:42 |
| xgerman | blogan, good afternoon. | 20:45 |
| xgerman | Got towed today? | 20:46 |
| *** sbfox has quit IRC | 20:46 | |
| blogan | luckily no, but i literally drove 100 miles around the city | 20:47 |
| xgerman | 100 miles? You went to Dallas? | 20:50 |
| rm_work | lol | 20:51 |
| rm_work | that's more like 500 | 20:51 |
| blogan | people from little states | 20:51 |
| blogan | though california is a long state | 20:51 |
| * rm_work has to make that drive at least a couple times a year | 20:51 | |
| xgerman | yep, LA is about 100 miles from me | 20:51 |
| blogan | dallas is more like 350 | 20:52 |
| blogan | maybe even 300 | 20:52 |
| ptoohill | houston is 300 | 20:52 |
| blogan | no way | 20:52 |
| ptoohill | dallas is 500 | 20:52 |
| ptoohill | yea | 20:52 |
| xgerman | you are truly in the middle ;-) | 20:52 |
| blogan | houston is like 150-200 | 20:52 |
| blogan | 200 miles | 20:52 |
| ptoohill | from sa? | 20:53 |
| blogan | dallas is 275 | 20:53 |
| blogan | yes from sa | 20:53 |
| ptoohill | i must have drove that waaay too slow then | 20:53 |
| ptoohill | too 3 hrs to get to houston | 20:53 |
| ptoohill | took | 20:53 |
| blogan | 70 mph for 3 hours = 210 miles | 20:54 |
| blogan | .... | 20:54 |
| blogan | so you drove the speed limit | 20:54 |
| blogan | do you even math bro? | 20:54 |
| ptoohill | hmmm | 20:55 |
| ptoohill | thats the problem, guess i was only doing 70 | 20:56 |
| ptoohill | probs 60 in jeep | 20:56 |
| ptoohill | >< | 20:56 |
| ptoohill | el paso is the 500 miles then yea | 20:57 |
| blogan | yes it is | 20:57 |
| blogan | you people dont even know your own state, im ashamed | 20:57 |
| ptoohill | lol | 20:57 |
| ptoohill | stuck in middle, dont get out much :( | 20:57 |
| *** jorgem has joined #openstack-lbaas | 20:57 | |
| *** mwang2 has quit IRC | 21:00 | |
| *** mwang2 has joined #openstack-lbaas | 21:00 | |
| rm_work | yeah, 500 is if he went to dallas and back | 21:01 |
| rm_work | I put about 550 round-trip miles on my car every trip up, I think | 21:01 |
| blogan | you were the originator of the 500 to dallas theory | 21:02 |
| blogan | you have no credibility | 21:02 |
| dougwig | you have no face | 21:05 |
| rm_work | err | 21:05 |
| blogan | not sure who you are talking to dougwig | 21:05 |
| blogan | but if you're talking about me, i have a giant face actually | 21:06 |
| rm_work | going to dallas and back is about 500 :P which is what I said | 21:06 |
| dougwig | it was meant as utter nonsense, so it doesn't matter. | 21:06 |
| rm_work | you guys went off on some weird tangents :P | 21:06 |
| *** jorgem has quit IRC | 21:06 | |
| fnaval | sohcahtoa | 21:08 |
| ptoohill | ? | 21:11 |
| ptoohill | yes | 21:12 |
| dougwig | i bet you can go login to franklin's workstation with that | 21:17 |
| rm_work | heh | 21:18 |
| rm_work | nah, pretty sure his actual password is hunter2 | 21:18 |
| rm_work | hah, I was wondering what i remembered sohcahtoa from | 21:19 |
| rm_work | woo math order of ops | 21:19 |
| rm_work | err rather, geometry | 21:19 |
| rm_work | ops is pemdas | 21:19 |
| blogan | whats sohcahtoa? | 21:19 |
| blogan | i cant remember | 21:20 |
| rm_work | Sine=Opposite/Hypotenuse, Cosine=Adjacent/Hypotenuse, Tangent=Opposite/Adjacent | 21:20 |
| dougwig | that almost looks like a DN | 21:28 |
| blogan | you had to look that up | 21:29 |
| rm_work | ALMOST | 21:29 |
| rm_work | blogan: also, yes of course :P | 21:30 |
| rm_work | I am so bad at math | 21:30 |
| blogan | not as bad as ptoohill apparently | 21:30 |
| dougwig | such a nurturing environment that our rax brethren bring to us. | 21:31 |
| xgerman | lol | 21:32 |
| ptoohill | hey | 21:32 |
| rm_work | <3 | 21:33 |
| ptoohill | big tires makes miles vs time blurry. normally im staring at bluebonnets anyways. And kids in the backseat make it take that much longer | 21:33 |
| xgerman | yeah, mine start asking after 10 minutes "are we there yet?" | 21:34 |
| xgerman | so my new plan is to drive only over night ;-) | 21:34 |
| ptoohill | :P | 21:34 |
| ptoohill | not a bad idea :) | 21:34 |
| *** madhu_ak has quit IRC | 21:34 | |
| johnsom | On another note, ajmiller is wondering about when he can startup a consumer worker on devstack. Any news from jorge? | 21:35 |
| ptoohill | jorgem? | 21:35 |
| xgerman | yep, that guy -- i he coding right now? | 21:35 |
| blogan | he's filling up spreadsheets | 21:36 |
| xgerman | you told him we needed P{ythpon and not vb script? | 21:36 |
| blogan | he didn't fix that review today? | 21:36 |
| ptoohill | was trying to ping him :P | 21:36 |
| blogan | gotta manage the manager tsk tsk | 21:36 |
| ptoohill | Think hes taken on some new responsibilities. We may never hear from him again | 21:36 |
| * blogan crosses fingers | 21:37 | |
| ptoohill | :P | 21:37 |
| blogan | boom! if only he were here to see that | 21:37 |
| ptoohill | lolol | 21:37 |
| blogan | johnsom: if he can't get to it tomorrow, ill do it | 21:37 |
| ptoohill | hed be burned like so bad breh | 21:37 |
| rm_work | heh | 21:39 |
| johnsom | Ok, thanks for the update. | 21:40 |
| *** apuimedo has quit IRC | 21:43 | |
| *** xgerman has quit IRC | 21:56 | |
| johnsom | TrevorV mentioned in a comment that he thinks I can get an amp via 'amps = health_mon.pool.listener.load_balancer.amphorae' but I don't see the relationship on pool that gets me to listener. Is that a missing relationship or am I missing something? | 21:56 |
| *** xgerman has joined #openstack-lbaas | 21:56 | |
| rm_work | backrefs aren't specified on every object | 21:56 |
| rm_work | you can TRY IT? and see if it works | 21:56 |
| rm_work | johnsom: you specify on one object that a backref works, and SQLAlchemy magically fills in the links | 21:57 |
| *** madhu_ak has joined #openstack-lbaas | 21:57 | |
| johnsom | Ok, more sqlalchemy vodoo | 21:57 |
| *** xgerman has quit IRC | 21:58 | |
| johnsom | voodoo. I'm working toward being able to try out that code block, but not there yet. | 21:58 |
| rm_work | k | 22:02 |
| *** xgerman has joined #openstack-lbaas | 22:03 | |
| blogan | johnsom: https://github.com/stackforge/octavia/blob/master/octavia/db/models.py#L216 | 22:10 |
| blogan | there's the backref, basically put load_balancer as an attribute of the amphora | 22:10 |
| johnsom | blogan I was asking about pool and listener | 22:11 |
| johnsom | I guess there kind of is with default_pool | 22:12 |
| blogan | johnsom: line 284 same file then | 22:12 |
| rm_work | johnsom: yeah, it's just SQLAlchemy voodoo | 22:12 |
| rm_work | you get kinda used to it <_< | 22:12 |
| johnsom | yeah, working on that. | 22:12 |
| rm_work | better than Spring AOP voodoo, which I NEVER got used to | 22:12 |
| blogan | me either | 22:13 |
| blogan | instantiation through xml | 22:13 |
| johnsom | Being someone that has spent a lot of time with SQL, ORM tools bend my mind sometimes. | 22:13 |
| rm_work | yeah it's definitely different | 22:14 |
| blogan | johnsom: yeah it took me a while to figure all that out, also spent a lot of time trying to use those as our data models we pass around instead of making complete static copies, but the relationships in the ORM never stayed static | 22:14 |
| blogan | and im sure it was just somethign i wasnt understanding, but #sqlalchemy couldn't help either, so i gave up and bit the bullet | 22:15 |
| fnaval | lol | 22:16 |
| rm_work | hmm, going to do some rowing -- bbl | 22:18 |
| madhu_ak | can someone review: https://review.openstack.org/#/c/171832/ please? | 22:19 |
| blogan | madhu_ak: looks like the a10 ci failed | 22:25 |
| madhu_ak | blogan, dougwig: I am not sure of that error though | 22:27 |
| dougwig | madhu_ak: yep, that looks like another driver bug. | 22:28 |
| blogan | dougwig: do you remember the python data driven tests library maru is using? | 22:28 |
| madhu_ak | thanks though | 22:29 |
| blogan | madhu_ak: anyway, i think these two tests files are prime candidates for data driven tests because the tests are essentially the same | 22:31 |
| madhu_ak | may I know which two test files you are referring? | 22:33 |
| madhu_ak | oh okay you meant admin.py and non_admin.py files.. | 22:35 |
| *** rm_work is now known as rm_work|away | 22:37 | |
| *** mlavalle has quit IRC | 22:56 | |
| openstackgerrit | Michael Johnson proposed stackforge/octavia: Corrected some __init__() parameters https://review.openstack.org/173111 | 23:03 |
| *** Aish has quit IRC | 23:05 | |
| johnsom | Removing more of the log object passing into drivers stuff | 23:09 |
| *** chlong has joined #openstack-lbaas | 23:14 | |
| *** Aish has joined #openstack-lbaas | 23:15 | |
| *** Aish has quit IRC | 23:20 | |
| *** Aish has joined #openstack-lbaas | 23:25 | |
| openstackgerrit | Al Miller proposed stackforge/octavia: Add devstack plugin for octavia https://review.openstack.org/167796 | 23:42 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!