*** mlavalle has quit IRC | 00:06 | |
*** vivek-ebay has quit IRC | 00:10 | |
openstackgerrit | min wang proposed openstack/neutron-lbaas: Admin API tempest https://review.openstack.org/171011 | 00:26 |
---|---|---|
*** xgerman has quit IRC | 00:30 | |
*** mwang2 has quit IRC | 00:42 | |
*** bharath has quit IRC | 00:56 | |
*** bharath has joined #openstack-lbaas | 00:56 | |
*** vivek-ebay has joined #openstack-lbaas | 00:57 | |
*** SumitNaiksatam has joined #openstack-lbaas | 00:58 | |
*** bharath has quit IRC | 01:01 | |
*** xgerman has joined #openstack-lbaas | 01:33 | |
*** xgerman has quit IRC | 01:36 | |
*** xgerman has joined #openstack-lbaas | 01:37 | |
*** xgerman has quit IRC | 01:37 | |
*** xgerman has joined #openstack-lbaas | 01:37 | |
openstackgerrit | Madhusudhan Kandadai proposed openstack/neutron-lbaas: Introduced tempest API tests for openstack/neutron-lbaas https://review.openstack.org/165627 | 01:38 |
*** xgerman has quit IRC | 01:39 | |
*** xgerman has joined #openstack-lbaas | 01:39 | |
*** vivek-ebay has quit IRC | 01:48 | |
*** madhu_ak has quit IRC | 02:01 | |
*** madhu_ak has joined #openstack-lbaas | 02:02 | |
*** xgerman has quit IRC | 02:09 | |
*** madhu_ak has quit IRC | 02:39 | |
*** chlong has quit IRC | 02:42 | |
*** chlong has joined #openstack-lbaas | 02:44 | |
*** vivek-ebay has joined #openstack-lbaas | 02:58 | |
*** ajmiller_ has joined #openstack-lbaas | 03:08 | |
*** xgerman has joined #openstack-lbaas | 03:09 | |
*** ajmiller__ has quit IRC | 03:12 | |
openstackgerrit | Madhusudhan Kandadai proposed openstack/neutron-lbaas: Introduced tempest API tests for openstack/neutron-lbaas https://review.openstack.org/165627 | 03:16 |
*** sbfox has joined #openstack-lbaas | 03:32 | |
*** fnaval has quit IRC | 03:47 | |
*** TrevorV_ has joined #openstack-lbaas | 03:56 | |
*** TrevorV_ has quit IRC | 04:01 | |
*** fnaval has joined #openstack-lbaas | 04:06 | |
openstackgerrit | Phillip Toohill proposed openstack/neutron-lbaas: WIP: Neutron LBaaS: TLS Barbican Scenario Test https://review.openstack.org/164828 | 04:16 |
*** amotoki has joined #openstack-lbaas | 04:20 | |
*** xgerman has quit IRC | 04:28 | |
openstackgerrit | Trevor Vardeman proposed stackforge/octavia: Amphora SSH Driver https://review.openstack.org/160964 | 04:29 |
*** madhu_ak has joined #openstack-lbaas | 04:34 | |
*** bharath has joined #openstack-lbaas | 04:42 | |
*** bharath has quit IRC | 04:46 | |
*** vivek-ebay has quit IRC | 04:52 | |
*** sbfox has quit IRC | 04:56 | |
*** sbfox has joined #openstack-lbaas | 05:08 | |
*** TrevorV_ has joined #openstack-lbaas | 05:08 | |
*** TrevorV|Home has joined #openstack-lbaas | 05:19 | |
openstackgerrit | Doug Wiegley proposed openstack/neutron-lbaas: Rename imports based on neutron tests reorganization. https://review.openstack.org/170983 | 05:20 |
openstackgerrit | Doug Wiegley proposed openstack/neutron-lbaas: Rename imports based on neutron tests reorganization. https://review.openstack.org/170983 | 05:21 |
*** apuimedo has joined #openstack-lbaas | 05:22 | |
*** madhu_ak has quit IRC | 05:23 | |
*** TrevorV_ has quit IRC | 05:28 | |
openstackgerrit | Doug Wiegley proposed openstack/neutron-lbaas: WIP - Test an alternate mechanism for enabling lbaasv2 in devstack-gate https://review.openstack.org/171049 | 05:31 |
TrevorV|Home | Hey doug can I bother you a minute? | 05:35 |
TrevorV|Home | dougwig, ^^ | 05:35 |
dougwig | TrevorV|Home: sure | 05:35 |
*** Aish has quit IRC | 05:36 | |
TrevorV|Home | I'm unable to stack devstack. | 05:36 |
TrevorV|Home | AttributeError: 'InstallRequirement' object has no attribute 'url' | 05:36 |
TrevorV|Home | That's the last line of the error stacktrace bit | 05:36 |
dougwig | i need a little more context than that. :) | 05:36 |
TrevorV|Home | Yeah, I was getting it | 05:36 |
TrevorV|Home | ha ha | 05:36 |
TrevorV|Home | python update.py /blah/blah/python-neutronclient.py | 05:37 |
TrevorV|Home | (no .py on client, sorry) | 05:37 |
TrevorV|Home | That's what threw the error at least | 05:37 |
TrevorV|Home | The only change I see in my devstack directory is fixing "openrc" to have "admin" as user instead of "demo" | 05:38 |
TrevorV|Home | I should have tried fresh clone first. | 05:38 |
TrevorV|Home | Hold on | 05:38 |
TrevorV|Home | Yeah, started fresh with same issue | 05:40 |
TrevorV|Home | Want a gist of the stack? | 05:40 |
TrevorV|Home | dougwig, ^^ | 05:40 |
dougwig | yes, please | 05:41 |
TrevorV|Home | https://gist.github.com/anonymous/4b4f0e92e45a08a80082 | 05:41 |
TrevorV|Home | Once again, only change was me replacing "demo" with "admin" in openrc | 05:42 |
dougwig | i don't think i've ever done it that way before. is /opt/stack git repos or package directories? | 05:43 |
TrevorV|Home | When you run stack.sh that's where all the projects are installed | 05:44 |
TrevorV|Home | You don't think you've ever specified admin or what? | 05:45 |
dougwig | right. and those are git repos, right? | 05:45 |
TrevorV|Home | Sorry, yes | 05:45 |
dougwig | no, i meant i've always done requirements updates from dev source trees. | 05:46 |
dougwig | not that they should ever need to be done manually. what are you trying to do | 05:46 |
dougwig | ? | 05:46 |
TrevorV|Home | Literally stack. | 05:46 |
TrevorV|Home | It fails. | 05:46 |
dougwig | oh, i see. | 05:47 |
dougwig | you don't want to hear my normal answer to that kind of weird error. | 05:47 |
TrevorV|Home | What, reinstall? | 05:47 |
TrevorV|Home | I'm on my linus tower, otherwise I would :( | 05:47 |
dougwig | go to bed and try again in the morning. :) | 05:47 |
TrevorV|Home | linux*** | 05:47 |
TrevorV|Home | Ha ha ha I could, but I'm a little determined right now | 05:48 |
dougwig | is this a fresh stack, or have you done a devstack before? | 05:48 |
TrevorV|Home | I've successfully stacked previously to this, like just last week, started up today and it fails. It may be that it's too late for me right now and I can't think about how to find the error | 05:49 |
dougwig | aha, that's where you're fucked. | 05:49 |
TrevorV|Home | Yeah, exactly... hence me asking for help :P | 05:49 |
dougwig | first i'd wipe everything out of pip, then i'd do "./unstack.sh ; ./clean.sh ; git pull ; ./stack.sh" | 05:50 |
TrevorV|Home | Hmm... forgot about ./clean.sh | 05:50 |
dougwig | the downrev requirements mix isn't tested, because the single-use jenkins nodes always get the latest. | 05:50 |
TrevorV|Home | How does one... clean out pip? | 05:51 |
dougwig | pip list, and nuke the content. | 05:51 |
TrevorV|Home | Like.. one at a time?!? | 05:52 |
TrevorV|Home | dougwig, it's made it a lot farther after the clean.sh | 05:55 |
TrevorV|Home | crossing my fingers... | 05:55 |
dougwig | re-using devstack nodes is a lot like playing russian roulette with a glock. | 05:56 |
TrevorV|Home | hmm... nevermind... still failed with the same error, just must have done extra setup previous to the attempt. | 05:56 |
dougwig | you have RECLONE=yes ? does it fail the same way manually? | 05:57 |
TrevorV|Home | Yeah, fails the same way | 05:57 |
TrevorV|Home | I'm not sure what you mean by manually | 05:57 |
TrevorV|Home | Like, going into each /opt/stack/project directory and doing a "pip install -r reqs" | 05:57 |
TrevorV|Home | there? | 05:57 |
dougwig | yeah, re-create the same steps. | 05:58 |
TrevorV|Home | I can try | 05:58 |
dougwig | worst case, all the deps install, and it can skip that path on the next stack. best case, you find the error. | 05:58 |
rm_work | pip freeze | xargs pip uninstall | 05:59 |
rm_work | ;P | 05:59 |
TrevorV|Home | nice rm_work | 06:00 |
TrevorV|Home | You think doing that and then running ./stack.sh should help maybe? | 06:00 |
rm_work | eh, really not sure if your pip installed packages are going to be an issue | 06:00 |
rm_work | i missed the actual problem | 06:00 |
rm_work | other than your devstack on your local machine is f'ed | 06:01 |
TrevorV|Home | Scroll up, link above to gist | 06:01 |
rm_work | can't | 06:01 |
TrevorV|Home | I repaste | 06:01 |
rm_work | reconnected on this machine too recently | 06:01 |
TrevorV|Home | https://gist.github.com/anonymous/4b4f0e92e45a08a80082 | 06:01 |
TrevorV|Home | I debate bringing in my tower just for this to get sorted out :P | 06:01 |
rm_work | where does the "InstallRequirement" class come from | 06:02 |
rm_work | eerg possibly setuptools | 06:03 |
rm_work | distros have a funny habit of using custom, old/outdated setuptools | 06:04 |
rm_work | so like, the one that ships with ubuntu | 06:04 |
rm_work | you have pyenv on there? | 06:04 |
dougwig | yeah, i'm betting it's a package version mismatch snafu. | 06:04 |
rm_work | if so, try this (easy to revert): | 06:04 |
rm_work | pyenv install 2.7.8 | 06:05 |
TrevorV|Home | I think I have pyenv | 06:05 |
rm_work | pyenv virtualenv 2.7.8 openstack | 06:05 |
rm_work | pyenv global openstack | 06:05 |
rm_work | THEN do your devstack | 06:05 |
rm_work | that'll guarantee you no distro BS | 06:05 |
TrevorV|Home | says 2.7.8 already exists | 06:05 |
rm_work | okthat is fine | 06:05 |
rm_work | wasn't sure if you had installed it already | 06:05 |
TrevorV|Home | That's the one I use default | 06:06 |
dougwig | i hate to disappear on you guys, but i've been up since 3am. | 06:06 |
rm_work | heh | 06:06 |
dougwig | so i'm disappearing. :) | 06:06 |
dougwig | night | 06:06 |
TrevorV|Home | take it easy dougwig sorry to keep you | 06:06 |
rm_work | TrevorV|Home: so yeah you're just going to make a clean virtualenv based on pyenv's 2.7.8, and then set it global | 06:06 |
TrevorV|Home | Can't make it my global | 06:06 |
TrevorV|Home | says not installed | 06:06 |
rm_work | err | 06:06 |
TrevorV|Home | mistyped, nvmd | 06:06 |
rm_work | wait did i get the args backwards for cirtualenv | 06:07 |
rm_work | *virtualenv | 06:07 |
rm_work | ah nope good | 06:07 |
rm_work | anywho, hopefully that should work | 06:07 |
TrevorV|Home | alright, restacking with that | 06:07 |
rm_work | and that way you can clean it out by just removing that virtualenv and remaking it every time | 06:07 |
rm_work | or at least, every time you have issues | 06:07 |
TrevorV|Home | Yeah | 06:07 |
rm_work | you can do "pyenv global system" | 06:08 |
rm_work | to reset | 06:08 |
TrevorV|Home | If this doesn't fix it, then fuck. | 06:08 |
TrevorV|Home | I'm so mad | 06:08 |
TrevorV|Home | Nope. | 06:08 |
rm_work | i could WFT tomorrow morning if you wanted me to look at it :P or you could just haul it in | 06:08 |
TrevorV|Home | Same effing error | 06:08 |
TrevorV|Home | I'm probably going to haul it in | 06:08 |
TrevorV|Home | I'll haul it in every damn day just to keep a linux machine as my primary macheen :P | 06:09 |
rm_work | tracking down the InstallRequirement code now to figure out what version the url parameter disappeared/appeared in | 06:09 |
rm_work | OH | 06:09 |
rm_work | it comes from pip! | 06:10 |
TrevorV|Home | ? | 06:10 |
rm_work | pip --version | 06:10 |
TrevorV|Home | 6.1.0 | 06:10 |
rm_work | what the jesus | 06:10 |
rm_work | rofl | 06:10 |
TrevorV|Home | Is that old?? | 06:10 |
rm_work | weird, usually pyenv installs its own pip | 06:11 |
TrevorV|Home | right?! | 06:11 |
TrevorV|Home | well, it did | 06:11 |
dougwig | TrevorV|Home: given the zuul status of two simple lbaas changes right now, i'd *highly* recommend going to bed and trying again in the morning. | 06:11 |
TrevorV|Home | DAMNIT dougwig I WAS WORKING ON PROVING BRANDON WRONG | 06:11 |
rm_work | TrevorV|Home: ok so | 06:13 |
dougwig | TrevorV|Home: yep, your error is being discussed in the infra channel. | 06:13 |
rm_work | can you paste the WHOLE pip version response? | 06:13 |
dougwig | TrevorV|Home: i'm sure you can find some other way to prove brandon wrong. it's a target rich goal. | 06:13 |
rm_work | interesting | 06:13 |
TrevorV|Home | dougwig, this happens every time... EVERY time I get REALLY close to getting him proven wrong... something comes up. | 06:14 |
TrevorV|Home | Every time | 06:14 |
dougwig | TrevorV|Home: the universe hates you. with cause. | 06:14 |
rm_work | TrevorV|Home may just have been the first in here to see this issue :P | 06:14 |
rm_work | like i was the first one to suffer from the gate failure at the SA hackathon | 06:14 |
TrevorV|Home | Yeah. I just. Idk. Alright. | 06:14 |
rm_work | and was tearing my hair out for 30m | 06:14 |
rm_work | of course, this has been apparently longer :P | 06:14 |
TrevorV|Home | I'll call it a night then. Probably bring my tower into the office for funsies. See you guys later. | 06:15 |
rm_work | TrevorV|Home: still curious though about pip | 06:15 |
dougwig | you can fix the bug and be a global openstack hero. | 06:15 |
TrevorV|Home | Thanks dougwig , now go to sleep | 06:15 |
rm_work | TrevorV|Home: could you paste the whole line? | 06:15 |
dougwig | night all | 06:15 |
rm_work | WOAH | 06:15 |
blogan | TrevorV|Home: you won't prove me wrong until you fix all the other bugs to get to the point im at | 06:16 |
TrevorV|Home | What other bugs? | 06:16 |
blogan | TrevorV|Home: you need to flush after you write, that is a definite | 06:16 |
TrevorV|Home | Nope, don't have to do that, and if you'd have listened I could have proven to you why | 06:17 |
blogan | well if you could get the code running you will see i am right | 06:17 |
blogan | i've commented on the other bugs ive got so far | 06:18 |
TrevorV|Home | Second comment is not a bug. It should be changed in a different review. | 06:19 |
TrevorV|Home | First comment I agree | 06:19 |
rm_work | blogan: i think technically if you do a .close() it should force a flush | 06:19 |
rm_work | assuming you mean a socket or a file buffer | 06:19 |
blogan | rm_work: well since he's trying to read the file before the close, and the fact that delete=True on a tempfile deletes the file on close...no | 06:20 |
TrevorV|Home | If the interpreter does some weird auto-flush behavior when running manually then I can see me being wrong | 06:20 |
rm_work | TrevorV|Home: that's very very possible | 06:20 |
TrevorV|Home | However, when I open the interpreter and write to the file, then open another terminal and check that temp-file's contents, its populated by what I write to it | 06:20 |
TrevorV|Home | Without having closed it. | 06:21 |
blogan | TrevorV|Home: you should change it to what the data_model has it currently, another review can go into your code and change it to container_id if it is wanted | 06:21 |
TrevorV|Home | I thought it was currently named container_id blogan | 06:21 |
blogan | TrevorV|Home: and it does not do that for me in an interpreter, nor the code running, so it is possible that it works for you and not me, but flushing it after the write should still be done | 06:22 |
blogan | TrevorV|Home: in neutron-lbaas, not in octavia | 06:22 |
TrevorV|Home | oh seriously?!?! | 06:22 |
TrevorV|Home | My bad | 06:22 |
blogan | yeah, i didn't explain that well enough, but yeah you can't run your code unless that is fixed, and fixed in the jinja code as well | 06:23 |
blogan | TrevorV|Home: are you doing a with tempfile.NamedTempFile(delete=True) as temp | 06:24 |
blogan | in your interpreter? | 06:24 |
TrevorV|Home | no, and that's not in the code, is it? | 06:24 |
blogan | nope | 06:24 |
TrevorV|Home | The steps I've done in the interpreter are as follows | 06:24 |
TrevorV|Home | import tempfile stuffs | 06:25 |
blogan | and you're doing it in ubuntu right? | 06:25 |
TrevorV|Home | yes | 06:25 |
TrevorV|Home | import tempfile stuffs | 06:25 |
TrevorV|Home | tmp = NamedTempFile(delete=True) | 06:25 |
TrevorV|Home | tmp.write("some data") | 06:25 |
TrevorV|Home | print tmp.name | 06:25 |
TrevorV|Home | Then, I open a new terminal and run "cat tmp.name" | 06:26 |
TrevorV|Home | Boom, "some data" is printed to the terminal | 06:26 |
rm_work | yep | 06:27 |
rm_work | found the change | 06:27 |
rm_work | https://github.com/pypa/pip/commit/e8e2566279879b7df04394edfcaa9c63c0ce9e67 | 06:27 |
TrevorV|Home | Woah... See now it didn't work | 06:28 |
TrevorV|Home | wtf | 06:28 |
blogan | TrevorV|Home: yep i do the exact same thing, and the file exists, without any data | 06:28 |
TrevorV|Home | Yeah, just got that. | 06:28 |
blogan | flush will work all the time | 06:28 |
TrevorV|Home | It wrote on my cloud vm though. Weird. | 06:28 |
TrevorV|Home | Idk what its doing different there. | 06:28 |
TrevorV|Home | Different version of python maybe? | 06:28 |
blogan | ? no idea | 06:28 |
TrevorV|Home | 2.7.6 | 06:28 |
*** kobis has joined #openstack-lbaas | 06:28 | |
TrevorV|Home | Yeah, earlier version | 06:29 |
TrevorV|Home | Shit | 06:29 |
TrevorV|Home | Alright, so you're right, flush will work every time. | 06:29 |
TrevorV|Home | It should be done right after the write though right? | 06:29 |
rm_work | yeah file handles are unpredictable | 06:29 |
rm_work | yeah | 06:29 |
rm_work | write -> flush | 06:29 |
rm_work | the writes depend on the OS and disk caching | 06:31 |
rm_work | so if you are doing it manually, often there is a delay and some other writes queue up on your system from other things | 06:32 |
rm_work | and by the time you check, it's gone through | 06:32 |
blogan | also your sftp.put assumes that directory exists | 06:32 |
TrevorV|Home | Yeah, I saw that one | 06:32 |
rm_work | but if it's happening quickly in code, it will be too fast and miss | 06:32 |
rm_work | ugh, fucking directory making is BS | 06:32 |
blogan | TrevorV|Home: so what have we learned today? | 06:32 |
blogan | TrevorV|Home: ill start it off 1) brandon is always right | 06:33 |
TrevorV|Home | blogan, that sometimes the environment can skew results, and 2 people can be right at the same time | 06:33 |
rm_work | because os.mkdir behaves differently in just about EVERY python version | 06:33 |
blogan | TrevorV|Home: no, you were not right | 06:33 |
blogan | rm_work: paramiko sftp has a mkdir command | 06:33 |
rm_work | does it behave correctly? :P | 06:33 |
TrevorV|Home | Pfff | 06:33 |
blogan | so at least that is abstracted away | 06:33 |
blogan | we can assume so | 06:33 |
rm_work | because os.mkdir should die in a fire | 06:33 |
* blogan crosses fingers | 06:33 | |
TrevorV|Home | Either way, I can't say I care about that, so I'll figure it out. | 06:33 |
blogan | care about what? | 06:34 |
TrevorV|Home | Worst case I'll just toss in another exec line to make the directory | 06:34 |
TrevorV|Home | I'll make the changes here in a minute and push the review again | 06:34 |
TrevorV|Home | I'll update to tls_certificate_id or whatever | 06:34 |
TrevorV|Home | I was adlibbing because I hadn't read it yet | 06:35 |
rm_work | exec should be used as little as physically possible | 06:35 |
TrevorV|Home | Goddamn my cat is all over me right now... | 06:35 |
TrevorV|Home | Desk is her space apparently | 06:35 |
rm_work | it's one of those things to have a good level of paranoia about | 06:35 |
TrevorV|Home | Judging by the fact that the ssh_driver is BUILT around the exec function, I can't say I'll agree with you in this case. | 06:36 |
rm_work | well, i helped them fix the gate | 06:36 |
TrevorV|Home | Wait, so stacking should work now or no | 06:36 |
blogan | same issue TrevorV|Home was running into? | 06:37 |
rm_work | yeah | 06:37 |
rm_work | TrevorV|Home: not yet | 06:37 |
rm_work | there's a patch up | 06:37 |
rm_work | I linked the cause -- it was pip 6.1.0 release today | 06:37 |
TrevorV|Home | Yeah, thanks for that rm_work | 06:37 |
rm_work | https://github.com/pypa/pip/commit/e8e2566279879b7df04394edfcaa9c63c0ce9e67 | 06:37 |
rm_work | funtimes | 06:37 |
TrevorV|Home | Guess I probably don't need to bring in my tower then | 06:38 |
rm_work | yeah :) | 06:38 |
rm_work | unless you reaaaally want to | 06:38 |
TrevorV|Home | I might do it anyway... I miss my linuxes | 06:38 |
rm_work | hehe | 06:38 |
rm_work | man, i am jealous of the BYOD plan thing | 06:38 |
blogan | linux mint works remarkably well on mac | 06:39 |
rm_work | bet carlos is sad he just missed that | 06:39 |
blogan | at least my mbp | 06:39 |
rm_work | yeah <_< your model | 06:39 |
TrevorV|Home | I'm actually thinking about getting a specific computer from the ibuypower website that's super tiny but has a good VC so I can use multiple monitors | 06:39 |
rm_work | ptoohill and I are boned | 06:39 |
TrevorV|Home | I went through bestbuy on their pre-built models... none with 2 vid ports except one vga and one dvi or hdmi... | 06:39 |
TrevorV|Home | Lame | 06:39 |
rm_work | lol worst buy | 06:39 |
blogan | or you could build your own | 06:39 |
TrevorV|Home | Its the tower, not the laptop | 06:39 |
TrevorV|Home | ibuypower is "building my own" without the replacing of the parts | 06:40 |
TrevorV|Home | Meaning, I don't get any DOA stuffs | 06:40 |
TrevorV|Home | rm_work, did you use my gist or get your own? | 06:40 |
TrevorV|Home | nvmd | 06:40 |
TrevorV|Home | hja ha | 06:40 |
TrevorV|Home | ha ha*** | 06:40 |
rm_work | yeah, tracked down the commit | 06:41 |
rm_work | thanks git blame | 06:41 |
rm_work | it's like they knew what it was going to be used for when they named the command :P | 06:42 |
Santosh_NS | A out of context query: for client--->extension-->plugin----->dbplugin flow , I was expecting there should be validation for db_data/say empty tenant-id etc at Plugin layer. Plz correct me . Can you plz point me where are we doing validation for resource data,For example create listener where we are verifying tenant_id is uuid | 06:43 |
blogan | Santosh_NS: tenant_id is not validated, the other fields are, and they're defined in the extension module in the massive RESOURCE_ATTR_MAP dictionary | 06:48 |
TrevorV|Home | sorry rm_work, I'm going to be using exec_command again. | 06:48 |
rm_work | for what? >_> | 06:48 |
TrevorV|Home | Otherwise I have to make recursive "mkdir" method calls. | 06:48 |
rm_work | err | 06:48 |
TrevorV|Home | Well not recursive, a loop | 06:48 |
TrevorV|Home | But a series of requests | 06:48 |
TrevorV|Home | Apparently if the parent directory doesn't already exist it will fail | 06:48 |
rm_work | yeah | 06:48 |
rm_work | that is what os.makedirs is for | 06:49 |
rm_work | it's a pain but it's still 100% better than exec | 06:49 |
blogan | can't do it over ssh | 06:49 |
rm_work | and paramiko's mkdir has no -p equiv? | 06:49 |
TrevorV|Home | If it has it, its not on their documentation | 06:50 |
*** woodster_ has quit IRC | 06:50 | |
blogan | doing the exec_command will just pass in ids that have no user input, so its not bad bad | 06:50 |
rm_work | it's super paranoia, but, things like that can be used as part of overflow attacks / etc | 06:51 |
rm_work | best to just avoid exec entirely | 06:51 |
rm_work | http://stackoverflow.com/questions/14819681/upload-files-using-sftp-in-python-but-create-directories-if-path-doesnt-exist | 06:51 |
rm_work | several options there | 06:51 |
TrevorV|Home | Just looked at that... | 06:52 |
rm_work | most of those mkdir_p commands look sane | 06:52 |
TrevorV|Home | best answer is recursive requests... sorry not on my list of things to do for this. | 06:52 |
rm_work | they straight up give you the code | 06:52 |
rm_work | copy/paste it | 06:52 |
TrevorV|Home | Yes, I understand, but where I can make one request versus several, I'll pick one. Thanks | 06:53 |
rm_work | ugh | 06:53 |
rm_work | i already -1'd german several times for exec | 06:53 |
rm_work | when i -1 you, it's not going to be personal >_> | 06:53 |
blogan | well i can't rightl +2 something another core -1's | 06:54 |
blogan | rightly | 06:54 |
blogan | i mean i can if it neither budges | 06:54 |
TrevorV|Home | I don't think its problematic to be cautious, but we're talking about one extra exec. It already execs. | 06:55 |
rm_work | to be fair, that's true | 06:55 |
TrevorV|Home | If it makes you feel better I can fore-go the SFTP command and just semicolon delineate a list of commands in one request. | 06:55 |
rm_work | we'll see when i actually get to look at the CR | 06:55 |
rm_work | that helps me very little T_T | 06:56 |
blogan | TrevorV|Home: what does the -sf switch do when invoking the haproxy command? | 06:56 |
rm_work | though if you can exec without using a variable at all inside exec() | 06:56 |
rm_work | like if it's ALWAYS "service haproxy restart" | 06:56 |
TrevorV|Home | Well, that never happens, rm_work. | 06:56 |
TrevorV|Home | blogan, it has to do with program exit/execution | 06:57 |
blogan | TrevorV|Home: yeah just read it | 06:58 |
TrevorV|Home | Sorry, I couldn't remember off-hand, but its essentially the "draining" state change thing | 06:58 |
blogan | finally got update to fully complete | 07:00 |
TrevorV|Home | rm_work, can you access http://www.ibuypower.com/? | 07:01 |
TrevorV|Home | no question mark sorry | 07:01 |
rm_work | http://isup.me/http://www.ibuypower.com | 07:02 |
rm_work | down for me and probably everyone else too :P | 07:02 |
TrevorV|Home | I didn' | 07:02 |
TrevorV|Home | I didn't know you could do that... Why I didn't know that, I don't know, but I didn't. | 07:02 |
TrevorV|Home | Makes sense | 07:02 |
rm_work | TrevorV|Home: so if you need to execute commands on the system, at least use subprocess.call() | 07:03 |
rm_work | slightly less prone to security nightmares | 07:03 |
TrevorV|Home | rm_work, are you fully aware of what I'm doing in the ssh_driver? | 07:04 |
rm_work | nope | 07:04 |
rm_work | barely partially aware | 07:04 |
TrevorV|Home | Okay... Well, I'm not going to ssh to the amphora, and then start a python interpreter to use subprocess.... | 07:04 |
rm_work | err, then how would you use exec | 07:05 |
TrevorV|Home | paramiko.SSHClient().exec_command('command goes here') | 07:05 |
*** kiran_ has joined #openstack-lbaas | 07:05 | |
TrevorV|Home | (except its just a client I initialize and use the method) | 07:05 |
rm_work | ah, that is a different exec | 07:05 |
TrevorV|Home | It could still be insecure to some degree, but its what I'm doin | 07:06 |
rm_work | also: lol yeah, probably want to just && or ; | 07:06 |
rm_work | so you don't have to make multiple calls -- but i assume it's a session anyway | 07:06 |
rm_work | so probably not significant | 07:06 |
rm_work | IE: do whatev | 07:06 |
*** kiran_ is now known as kiran-r | 07:07 | |
TrevorV|Home | yeah, its a session | 07:08 |
TrevorV|Home | I think | 07:08 |
TrevorV|Home | Maybe not | 07:08 |
TrevorV|Home | It only establishes the connection once | 07:08 |
TrevorV|Home | Then you interact | 07:08 |
TrevorV|Home | Then close the connection | 07:08 |
TrevorV|Home | Whatever, just check the review at some point and you'll get caught up | 07:09 |
*** TrevorV|Home has quit IRC | 07:09 | |
*** fnaval has quit IRC | 07:24 | |
*** chlong has quit IRC | 07:34 | |
rm_work | oh man, this rabbit hole just keeps going | 07:37 |
*** sbfox has quit IRC | 08:06 | |
*** apuimedo has quit IRC | 08:06 | |
*** kobis has quit IRC | 08:25 | |
*** kobis has joined #openstack-lbaas | 08:26 | |
*** chlong has joined #openstack-lbaas | 10:07 | |
*** tobberydberg has joined #openstack-lbaas | 10:47 | |
openstackgerrit | Jakub Libosvar proposed openstack/neutron-lbaas: Add HaproxyNSDriver to lbaas entry points https://review.openstack.org/171127 | 11:07 |
*** bedis has quit IRC | 11:29 | |
*** bedis has joined #openstack-lbaas | 11:29 | |
*** amotoki has quit IRC | 11:38 | |
*** enikanorov has quit IRC | 11:42 | |
*** enikanorov has joined #openstack-lbaas | 11:43 | |
*** kiran-r has quit IRC | 11:43 | |
*** chlong has quit IRC | 11:49 | |
*** chlong has joined #openstack-lbaas | 11:51 | |
*** woodster_ has joined #openstack-lbaas | 12:10 | |
*** kobis has left #openstack-lbaas | 12:12 | |
openstackgerrit | Susanne Balle proposed stackforge/octavia: WIP: haproxy reference amphora API client https://review.openstack.org/170993 | 12:53 |
openstackgerrit | Susanne Balle proposed stackforge/octavia: WIP: HaProxy rest ref driver implementation https://review.openstack.org/171172 | 12:53 |
*** amotoki has joined #openstack-lbaas | 13:00 | |
*** openstackgerrit has quit IRC | 13:07 | |
*** openstackgerrit has joined #openstack-lbaas | 13:07 | |
*** chlong has quit IRC | 13:51 | |
openstackgerrit | Thierry Carrez proposed openstack/neutron-lbaas: Open Liberty development https://review.openstack.org/171205 | 13:54 |
*** mlavalle has joined #openstack-lbaas | 13:59 | |
*** jorgem has joined #openstack-lbaas | 14:09 | |
*** jorgem1 has joined #openstack-lbaas | 14:16 | |
*** jorgem1 has quit IRC | 14:16 | |
*** jorgem1 has joined #openstack-lbaas | 14:17 | |
*** jorgem has quit IRC | 14:18 | |
*** Aish has joined #openstack-lbaas | 14:24 | |
*** devlaps has quit IRC | 14:38 | |
*** fnaval has joined #openstack-lbaas | 14:38 | |
*** jorgem1 has quit IRC | 14:56 | |
*** jorgem has joined #openstack-lbaas | 14:58 | |
*** jorgem1 has joined #openstack-lbaas | 15:03 | |
*** jorgem has quit IRC | 15:05 | |
*** vivek-ebay has joined #openstack-lbaas | 15:11 | |
*** vivek-ebay has quit IRC | 15:11 | |
*** xgerman has joined #openstack-lbaas | 15:14 | |
*** vivek-ebay has joined #openstack-lbaas | 15:16 | |
*** jorgem1 has quit IRC | 15:18 | |
*** jorgem has joined #openstack-lbaas | 15:25 | |
*** ajmiller_ has quit IRC | 15:27 | |
*** ajmiller has joined #openstack-lbaas | 15:33 | |
*** amotoki has quit IRC | 15:34 | |
*** madhu_ak has joined #openstack-lbaas | 15:34 | |
*** devlaps has joined #openstack-lbaas | 15:40 | |
*** kiran has joined #openstack-lbaas | 15:45 | |
*** madhu_ak has quit IRC | 15:49 | |
*** vivek-ebay has quit IRC | 16:06 | |
*** SumitNaiksatam has quit IRC | 16:06 | |
openstackgerrit | Phillip Toohill proposed stackforge/octavia: Adds rise threshold to configuration generation https://review.openstack.org/171268 | 16:14 |
*** jorgem has quit IRC | 16:14 | |
*** madhu_ak has joined #openstack-lbaas | 16:16 | |
*** Aish has quit IRC | 16:18 | |
*** jorgem has joined #openstack-lbaas | 16:31 | |
openstackgerrit | German Eichberger proposed stackforge/octavia: Make PEM file confirm with SSH Driver https://review.openstack.org/171283 | 16:34 |
*** devlaps has quit IRC | 16:39 | |
*** jorgem has quit IRC | 16:42 | |
openstackgerrit | German Eichberger proposed stackforge/octavia: Make PEM file confiorm with SSH Driver https://review.openstack.org/171283 | 16:49 |
openstackgerrit | German Eichberger proposed stackforge/octavia: Make PEM file conform with SSH Driver https://review.openstack.org/171283 | 16:50 |
*** vivek-ebay has joined #openstack-lbaas | 16:50 | |
xgerman | not my day today | 16:50 |
rm_work | heh | 16:50 |
rm_work | I was trying to comment on it before you got 3 up | 16:50 |
rm_work | to slow | 16:50 |
rm_work | *too slow | 16:50 |
xgerman | no worries | 16:50 |
*** _kiran_ has joined #openstack-lbaas | 16:54 | |
*** kiran has quit IRC | 16:57 | |
*** Aish has joined #openstack-lbaas | 16:59 | |
*** mlavalle has quit IRC | 17:01 | |
*** _kiran_ has quit IRC | 17:01 | |
*** kiran has joined #openstack-lbaas | 17:01 | |
*** bharath has joined #openstack-lbaas | 17:04 | |
dougwig | morning | 17:06 |
*** kiran has quit IRC | 17:07 | |
*** smcgough1 has quit IRC | 17:10 | |
openstackgerrit | German Eichberger proposed stackforge/octavia: Implements the haproxy amphora agent api server https://review.openstack.org/160034 | 17:11 |
*** smcgough has joined #openstack-lbaas | 17:12 | |
Aish | blogan: hi | 17:14 |
*** sbfox has joined #openstack-lbaas | 17:23 | |
*** ajmiller_ has joined #openstack-lbaas | 17:43 | |
*** ajmiller has quit IRC | 17:47 | |
xgerman | hi | 17:48 |
*** Varun_Lodaya has joined #openstack-lbaas | 17:49 | |
*** bharath has quit IRC | 17:57 | |
*** jorgem has joined #openstack-lbaas | 17:58 | |
*** bharath has joined #openstack-lbaas | 17:58 | |
*** jorgem has quit IRC | 17:58 | |
*** smcgough1 has joined #openstack-lbaas | 18:26 | |
*** mlavalle has joined #openstack-lbaas | 18:27 | |
*** smcgough has quit IRC | 18:28 | |
*** bharath_ has joined #openstack-lbaas | 18:36 | |
*** bharath has quit IRC | 18:37 | |
openstackgerrit | Trevor Vardeman proposed stackforge/octavia: Amphora SSH Driver https://review.openstack.org/160964 | 18:40 |
*** ajmiller has joined #openstack-lbaas | 18:44 | |
*** ajmiller_ has quit IRC | 18:44 | |
*** raj___ has joined #openstack-lbaas | 18:55 | |
*** enikanorov has quit IRC | 19:01 | |
openstackgerrit | Doug Wiegley proposed openstack/neutron-lbaas: WIP - Test an alternate mechanism for enabling lbaasv2 in devstack-gate https://review.openstack.org/171049 | 19:11 |
*** bharath_ has quit IRC | 19:11 | |
*** bharath has joined #openstack-lbaas | 19:12 | |
*** openstackgerrit has quit IRC | 19:22 | |
*** openstackgerrit has joined #openstack-lbaas | 19:23 | |
openstackgerrit | Trevor Vardeman proposed stackforge/octavia: Amphora SSH Driver https://review.openstack.org/160964 | 19:23 |
*** madhu_ak has quit IRC | 19:35 | |
*** sbfox has quit IRC | 19:36 | |
*** madhu_ak has joined #openstack-lbaas | 19:43 | |
openstackgerrit | min wang proposed stackforge/octavia: Add health check amphora-driver-interface https://review.openstack.org/170599 | 19:57 |
ptoohill | all sni certs are combined? question regarding the change xgerman | 20:08 |
xgerman | will answer in 10 minutes | 20:08 |
rm_work | xgerman: yeah, curious where that came from -- I mixed your thing up with a change ptoohill was explaining to me last week about certs/keys/intermediates all being concatenated into one file, but don't know where it's specified that SNI is done like this | 20:21 |
rm_work | haproxy docs somewhere? | 20:21 |
johnsom | http://cbonte.github.io/haproxy-dconv/configuration-1.5.html#5.1-crt-list | 20:22 |
johnsom | Is that what you are asking about? | 20:22 |
*** vivek-ebay has quit IRC | 20:26 | |
rm_work | johnsom: https://review.openstack.org/171283 | 20:29 |
rm_work | it says it is to conform with the SSH driver, but the SSH driver apparently doesn't do anything like that | 20:29 |
johnsom | Doesn't the LBaaS v2 TLS use a single file? | 20:30 |
ptoohill | no | 20:30 |
ptoohill | It uses individual files for each cert/keypair | 20:30 |
ptoohill | I followed what baptiste said was the proper way to handle sni in Haproxy | 20:31 |
*** kobis has joined #openstack-lbaas | 20:31 | |
johnsom | Ok, I think he mis-read that code then. I remember he was talking about this last week when we were in Seattle | 20:31 |
ptoohill | you have a default cert, and a directory with all the non-default certs | 20:31 |
xgerman | now you have my attention... rrading... | 20:32 |
xgerman | ptoohill, so we are not putting them in one file? | 20:33 |
xgerman | it looked to me like https://review.openstack.org/#/c/160964/17/octavia/amphorae/drivers/haproxy/ssh_driver.py was concatentaing all SNI certs | 20:34 |
ptoohill | xgerman, no, each cert/keypair is one pem | 20:34 |
ptoohill | the ssh_driver is doing the same as v2 nlbaas | 20:35 |
ptoohill | it build individual pems for each container | 20:35 |
xgerman | gotcha, so for SNI we are using the cert directory option? | 20:36 |
ptoohill | correct | 20:36 |
ptoohill | for ex: | 20:36 |
ptoohill | bind:1000 ssl crt /certs/default.pem crt /certs | 20:36 |
ptoohill | haproxy will look through the certs dir and match the CN to the sni request | 20:37 |
ptoohill | now i was looking into the sni filter thing which we havnt implemented in v2 | 20:38 |
xgerman | yep, it looked to us like you cna get the same by just concatenating the whole file ;-) | 20:39 |
ptoohill | you can, but why? | 20:39 |
ptoohill | or atleast it looks like you can, i havnt used this way | 20:39 |
xgerman | well, using one file makes things simpler... but the LBaaS V2 spec indicated that the order of the certificates plays some role | 20:40 |
ptoohill | if we need to add/remove one cert how does regenerating a potential huge file simpler? | 20:41 |
ptoohill | we can just save the data from one container and not reload all cert data | 20:41 |
xgerman | yep | 20:42 |
xgerman | so in the ssh driver you don't use different cert directories per listener | 20:42 |
ptoohill | now if thats the only way to apply sni-filter then we should look into it. im trying to find good examples for it but coming up short | 20:42 |
xgerman | yeah, probably somethign we need to ask bedis about | 20:42 |
xgerman | "good examples" | 20:42 |
ptoohill | whats wrong with a few additional directories in order to not reload 10+ tls containers | 20:42 |
xgerman | nothing wrong | 20:43 |
xgerman | and I don't really have a strong opinion one way or the other -- I just misread your driver (actually johnsom misread it as well) | 20:43 |
johnsom | Guilt | 20:43 |
johnsom | guilty | 20:44 |
xgerman | yeah, we should ask for more comments ;-) | 20:44 |
johnsom | Personally, I would lean towards the files | 20:44 |
xgerman | single file? | 20:45 |
johnsom | Separate files | 20:46 |
xgerman | ok, let's call that directory | 20:46 |
xgerman | then there was the "mapping file" | 20:47 |
ptoohill | mapping file? | 20:52 |
ptoohill | Right now it seems everything is blown away and regenerated anyhow. So i dont really have an opinion on one way or the other unless we plan on ever needing to trouble shoot a particular cert.. for whatever reason | 20:53 |
ptoohill | then i would say individual files are better. But i still need to figure out what this sni-filter is and if we need it | 20:53 |
rm_work | i'm interested in how SNI cert ordering happens if it's individual files vs one large file | 20:54 |
ptoohill | think its that sni-filter thing | 20:55 |
xgerman | rm_work: | 20:55 |
ptoohill | I only set it up the way it is now because that was the suggested 'proper' way. But if we need to we can totally redo it | 20:55 |
xgerman | * directory: will be loaded in alphabetcial order | 20:55 |
xgerman | * one file - probably anyones guess | 20:56 |
xgerman | * "mapping file": controls the order | 20:56 |
rm_work | ok | 20:56 |
ptoohill | oh, thought you meant mapping file in driver code | 20:56 |
xgerman | that is my understanding, which might be wrong | 20:56 |
rm_work | so either we make a mapping file, or we name things similar to like, rc.d | 20:56 |
rm_work | C001-host1.mysite.com.pem | 20:57 |
rm_work | C002-wildcard.mysite.com.pem | 20:57 |
rm_work | ? | 20:57 |
xgerman | do the crt-list Michael posted might be safest | 21:00 |
xgerman | on the other hand the order is sort of unusual and only applies to edgecases | 21:01 |
xgerman | since in most cases the computer is smart enough to go from the most specific to the least specific | 21:02 |
*** kobis has quit IRC | 21:03 | |
*** raj___ has quit IRC | 21:07 | |
xgerman | so, it looks like the people who run haproxy.com like certificate directories | 21:11 |
ptoohill | i.e /something/certs/ | 21:13 |
ptoohill | ? | 21:13 |
xgerman | yep | 21:14 |
*** tobberydberg has quit IRC | 21:17 | |
xgerman | ok, I better revert then... | 21:21 |
*** chlong has joined #openstack-lbaas | 21:25 | |
*** vivek-ebay has joined #openstack-lbaas | 21:27 | |
*** vivek-eb_ has joined #openstack-lbaas | 21:30 | |
*** vivek-ebay has quit IRC | 21:31 | |
openstackgerrit | Doug Wiegley proposed openstack/neutron-lbaas: Rename imports based on neutron tests reorganization. https://review.openstack.org/170983 | 21:35 |
openstackgerrit | min wang proposed stackforge/octavia: Add health check amphora-driver-interface https://review.openstack.org/170599 | 21:36 |
*** openstackgerrit has quit IRC | 21:37 | |
*** openstackgerrit has joined #openstack-lbaas | 21:37 | |
openstackgerrit | Brandon Logan proposed stackforge/octavia: Added neutron allowed address pairs network driver https://review.openstack.org/167441 | 21:40 |
openstackgerrit | German Eichberger proposed stackforge/octavia: Implements the haproxy amphora agent api server https://review.openstack.org/160034 | 21:46 |
johnsom | I have put a line item on tomorrow's agenda to review the API/Controller database design. I am about to take another pass through the controller worker code to update for the database work. | 21:47 |
johnsom | It would be great if whoever is working on the consumer worker/api code could be at the meeting. I'm not sure if Jorge is still working on it or if someone else is going to finish it up. | 21:48 |
ptoohill | jorgem should be in meeting tomorrow if thats still his | 21:52 |
*** sbfox has joined #openstack-lbaas | 21:54 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/neutron-lbaas: Updated from global requirements https://review.openstack.org/171398 | 21:54 |
johnsom | Cool, thanks! | 21:54 |
openstackgerrit | OpenStack Proposal Bot proposed stackforge/octavia: Updated from global requirements https://review.openstack.org/171405 | 22:00 |
*** mestery has quit IRC | 22:11 | |
*** mestery has joined #openstack-lbaas | 22:12 | |
*** xgerman has quit IRC | 22:13 | |
*** xgerman has joined #openstack-lbaas | 22:15 | |
openstackgerrit | Merged stackforge/octavia: Updated from global requirements https://review.openstack.org/171405 | 22:45 |
openstackgerrit | Al Miller proposed stackforge/octavia: Add devstack plugin for octavia https://review.openstack.org/167796 | 22:53 |
*** mlavalle has quit IRC | 22:55 | |
*** Aish has left #openstack-lbaas | 23:06 | |
*** TrevorV|Home has joined #openstack-lbaas | 23:12 | |
*** sbfox has quit IRC | 23:13 | |
*** TrevorV|Home has quit IRC | 23:17 | |
*** mwang2 has joined #openstack-lbaas | 23:25 | |
*** bharath has quit IRC | 23:51 | |
openstackgerrit | Michael Johnson proposed stackforge/octavia: Adding post_network_plug to the noop amphora driver https://review.openstack.org/171434 | 23:51 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!