*** xgerman has quit IRC | 00:02 | |
dougwig | anyone know why openstack processes don't just use django to implement the web services/orm/api, instead of reinventing all this crap every time? | 00:02 |
---|---|---|
rm_work | yes, just separating the parts that are daemon-like from deploy-worker so it is importable library code | 00:02 |
rm_work | dougwig: <_< | 00:02 |
rm_work | the assumption being that django is something people would want to use? :/ | 00:03 |
dougwig | hahahaha. well, i would prefer rails, but i get shot when i say ruby. | 00:03 |
dougwig | but seriously, web services frameworks are a solved problem, but not in openstack-land. | 00:04 |
johnsom | Yeah, I was a bit surprised there wasn't a clear "library" for that.... | 00:05 |
*** chlong has joined #openstack-lbaas | 00:06 | |
rm_work | isn't pecan a WSF? | 00:17 |
rm_work | or is my definition of WSF not good | 00:18 |
rm_work | because pecan is the official openstack framework AFAIK | 00:18 |
dougwig | not as abstracted as django or rails, and let's face it, you'll hit the GIL of either interpreter before *any* of these solutions is really a scaling issue. | 00:18 |
rm_work | eh | 00:19 |
rm_work | the problem is probably that everyone has their favorite thing | 00:20 |
rm_work | for instance, i don't really like django | 00:20 |
rm_work | but that's just me | 00:20 |
rm_work | maybe I need to give it a try again | 00:20 |
rm_work | but I'd much rather use Flask, any day of the week | 00:20 |
dougwig | django or rails will never do things the way anyone wants. they'll just do them one way that works, and which PaaS can be wrapped around. convention and all that, and at this point, they scale well. it's a lot of glue i don't have to write, which means more beer time. but. not in openstack! :) | 00:27 |
dougwig | or like johnsom suggested, i've always been floored that there isn't just "import oslo.api_server" or something. | 00:29 |
*** xgerman has joined #openstack-lbaas | 00:32 | |
*** mlavalle has quit IRC | 00:36 | |
*** rm_mobile| has joined #openstack-lbaas | 00:49 | |
*** rm_mobile| has quit IRC | 00:49 | |
*** sbfox has quit IRC | 00:51 | |
*** rm_mobile has quit IRC | 00:52 | |
sbalukoff | On the above conversation: I would love to use a different name than 'deploy worker' as it will be doing more than just deployments. | 01:11 |
johnsom | Yeah, it has morphed a bit | 01:11 |
johnsom | MCP? | 01:11 |
sbalukoff | drone? | 01:12 |
sbalukoff | imp? | 01:12 |
sbalukoff | cherub? | 01:12 |
johnsom | cherub? really? | 01:13 |
sbalukoff | Heh! | 01:13 |
sbalukoff | artifex | 01:13 |
sbalukoff | artifex = latin for worker. | 01:13 |
sbalukoff | Since, you know, that whole ancient Rome theme... | 01:13 |
johnsom | Much better than cherub | 01:14 |
sbalukoff | I can throw a ton of terrible names at you until I herd you toward one that doesn't suck. ;) | 01:14 |
sbalukoff | I'm cool with artifex. | 01:15 |
johnsom | I could go with that too. | 01:15 |
sbalukoff | Or even, you know "worker" | 01:15 |
sbalukoff | Clearly, we need to debate this for three weeks. | 01:15 |
johnsom | Hahaha | 01:15 |
sbalukoff | I just know that 'deploy worker' is too specific. | 01:15 |
johnsom | You just want to keep me busy rolling spec updates... | 01:16 |
sbalukoff | :D | 01:17 |
sbalukoff | Go with artifex or just plain "worker" then. | 01:18 |
sbalukoff | I'm pretty sure I know what blogan would choose. | 01:18 |
sbalukoff | Apparently 'operarius' is also a valid translation. | 01:18 |
sbalukoff | But screw that. | 01:18 |
sbalukoff | That's like 5 syllables. | 01:18 |
johnsom | I think worker is too generic, we will likely have more than one type | 01:18 |
sbalukoff | artifex it is, then. ;) | 01:19 |
sbalukoff | opifex is also a valid translation, though seems to imply some level of skill at a trade. | 01:19 |
*** xgerman has quit IRC | 01:22 | |
*** sbalukoff has quit IRC | 01:28 | |
openstackgerrit | Merged openstack/neutron-lbaas: Migrate to oslo.concurrency https://review.openstack.org/147244 | 01:38 |
openstackgerrit | Carlos Garza proposed openstack/neutron-lbaas: Common TLS utilities https://review.openstack.org/142915 | 02:09 |
*** pcaruana has quit IRC | 02:41 | |
*** woodster_ has quit IRC | 02:50 | |
openstackgerrit | Carlos Garza proposed openstack/neutron-lbaas: Common TLS utilities https://review.openstack.org/142915 | 02:54 |
openstackgerrit | Carlos Garza proposed openstack/neutron-lbaas: Common TLS utilities https://review.openstack.org/142915 | 03:01 |
*** burper has joined #openstack-lbaas | 03:02 | |
*** burper has left #openstack-lbaas | 03:02 | |
*** crc32 has quit IRC | 03:07 | |
*** sbalukoff has joined #openstack-lbaas | 04:04 | |
*** chlong has quit IRC | 04:06 | |
*** woodster_ has joined #openstack-lbaas | 04:11 | |
*** chlong has joined #openstack-lbaas | 04:11 | |
*** chlong_ has joined #openstack-lbaas | 04:13 | |
*** chlong has quit IRC | 04:17 | |
*** chlong_ has quit IRC | 04:18 | |
*** chlong has joined #openstack-lbaas | 04:24 | |
*** amotoki_ has joined #openstack-lbaas | 04:25 | |
*** chlong_ has joined #openstack-lbaas | 04:26 | |
*** chlong has quit IRC | 04:27 | |
*** chlong__ has joined #openstack-lbaas | 04:27 | |
*** chlong_ has quit IRC | 04:29 | |
*** chlong_ has joined #openstack-lbaas | 04:29 | |
*** chlong_ has quit IRC | 04:32 | |
*** chlong has joined #openstack-lbaas | 04:32 | |
*** chlong__ has quit IRC | 04:33 | |
*** chlong has quit IRC | 04:34 | |
*** amotoki_ has quit IRC | 04:35 | |
*** amotoki_ has joined #openstack-lbaas | 04:37 | |
*** amotoki_ has quit IRC | 04:41 | |
rm_you| | blogan: blogan | 04:47 |
*** rm_you| is now known as rm_you | 04:51 | |
rm_you | blogan: you there? | 04:51 |
*** chlong has joined #openstack-lbaas | 05:00 | |
*** chlong_ has joined #openstack-lbaas | 05:01 | |
*** chlong has quit IRC | 05:04 | |
*** chlong__ has joined #openstack-lbaas | 05:06 | |
*** chlong_ has quit IRC | 05:09 | |
*** chlong has joined #openstack-lbaas | 05:12 | |
*** chlong__ has quit IRC | 05:12 | |
*** chlong has quit IRC | 05:12 | |
*** xgerman has joined #openstack-lbaas | 05:22 | |
*** chlong has joined #openstack-lbaas | 05:25 | |
*** Miouge has quit IRC | 05:28 | |
*** chlong has quit IRC | 05:31 | |
*** chlong has joined #openstack-lbaas | 05:31 | |
*** chlong has quit IRC | 05:34 | |
*** sbfox has joined #openstack-lbaas | 05:35 | |
openstackgerrit | Brandon Logan proposed openstack/neutron-lbaas: Simplify user workflow creating loadbalancers https://review.openstack.org/144834 | 05:38 |
*** chlong has joined #openstack-lbaas | 05:40 | |
*** chlong_ has joined #openstack-lbaas | 05:41 | |
*** sbfox has quit IRC | 05:44 | |
*** chlong has quit IRC | 05:45 | |
openstackgerrit | Brandon Logan proposed openstack/neutron-lbaas: Simplify user workflow creating loadbalancers https://review.openstack.org/144834 | 05:48 |
blogan | yo | 05:54 |
blogan | rm_you: am now | 05:54 |
blogan | sbalukoff: since i lost the amphora debate, may as well go with artifex | 05:56 |
blogan | i disagree with worker being to generic and artifex isn't though, if artifex means worker, then thats just as generic | 05:56 |
openstackgerrit | Brandon Logan proposed openstack/neutron-lbaas: Simplify user workflow creating loadbalancers https://review.openstack.org/144834 | 06:04 |
*** woodster_ has quit IRC | 06:20 | |
*** amotoki_ has joined #openstack-lbaas | 06:25 | |
sbalukoff | blogan: I agree that artifex isn't any more specific than worker. | 06:30 |
rm_you | wtf are you guys talking about | 06:30 |
*** amotoki_ has quit IRC | 06:44 | |
*** amotoki_ has joined #openstack-lbaas | 06:45 | |
*** amotoki_ has quit IRC | 06:59 | |
*** amotoki_ has joined #openstack-lbaas | 07:01 | |
*** amotoki_ has quit IRC | 07:01 | |
*** amotoki_ has joined #openstack-lbaas | 07:12 | |
*** amotoki_ has quit IRC | 07:17 | |
*** apuimedo has joined #openstack-lbaas | 07:19 | |
*** sbfox has joined #openstack-lbaas | 07:19 | |
*** kobis has joined #openstack-lbaas | 07:25 | |
*** amotoki_ has joined #openstack-lbaas | 07:30 | |
*** xgerman has quit IRC | 07:34 | |
*** amotoki_ has quit IRC | 07:36 | |
*** amotoki_ has joined #openstack-lbaas | 07:36 | |
*** Miouge has joined #openstack-lbaas | 07:38 | |
*** chlong_ has quit IRC | 07:39 | |
sbalukoff | rm_work: "Deploy worker" is too specific. So we could go with "worker" or... artifex, which is latin for 'worker' (you know, going off that whole ancient Rome theme). | 07:46 |
rm_you | uhh | 07:50 |
*** amotoki_ has quit IRC | 07:53 | |
*** amotoki_ has joined #openstack-lbaas | 07:56 | |
*** amotoki_ has quit IRC | 07:58 | |
*** amotoki_ has joined #openstack-lbaas | 07:58 | |
*** amotoki_ has quit IRC | 08:00 | |
*** amotoki_ has joined #openstack-lbaas | 08:03 | |
*** amotoki_ has quit IRC | 08:05 | |
*** amotoki_ has joined #openstack-lbaas | 08:05 | |
rm_you | so the reason we needed to use the term Amphora is because the word "container" in our context was *already used* by something specific, and that thing was NOT what we were talking about | 08:14 |
rm_you | I think in this case, the term worker is actually too generic | 08:14 |
rm_you | it should be something like "deploy_shared_codebase" | 08:15 |
rm_you | lol | 08:15 |
rm_you | not exactly that, because that is dumb | 08:15 |
rm_you | but I don't think it needs it's own proper noun kind of word <_< | 08:15 |
*** amotoki_ has quit IRC | 08:17 | |
*** amotoki_ has joined #openstack-lbaas | 08:18 | |
*** amotoki_ has quit IRC | 08:20 | |
*** amotoki_ has joined #openstack-lbaas | 08:22 | |
*** amotoki_ has quit IRC | 08:25 | |
*** amotoki_ has joined #openstack-lbaas | 08:26 | |
*** amotoki_ has quit IRC | 08:27 | |
sbalukoff | rm_you: That's fine. I think spending more than about 30 seconds on this is bikeshedding, and I'd rather just pick something less specific than "deploy worker" and move on. | 08:28 |
*** amotoki_ has joined #openstack-lbaas | 08:28 | |
rm_you | yeah T_T | 08:29 |
rm_you | i missed where the discussion actually started or was taking place >_> | 08:29 |
rm_you | ah well | 08:29 |
* rm_you heads to bed | 08:30 | |
*** amotoki_ has quit IRC | 08:30 | |
*** amotoki_ has joined #openstack-lbaas | 08:32 | |
*** amotoki_ has quit IRC | 08:33 | |
*** kbyrne has joined #openstack-lbaas | 08:57 | |
*** sbfox has quit IRC | 09:28 | |
*** evgenyf has joined #openstack-lbaas | 09:34 | |
*** amotoki has quit IRC | 09:42 | |
*** amotoki has joined #openstack-lbaas | 09:42 | |
*** Miouge_ has joined #openstack-lbaas | 09:47 | |
*** Miouge has quit IRC | 09:48 | |
*** Miouge_ is now known as Miouge | 09:51 | |
*** Miouge has quit IRC | 10:22 | |
*** amotoki has quit IRC | 10:26 | |
*** Miouge has joined #openstack-lbaas | 11:13 | |
*** amotoki has joined #openstack-lbaas | 11:25 | |
*** amotoki has quit IRC | 11:28 | |
*** amotoki has joined #openstack-lbaas | 11:47 | |
*** chlong_ has joined #openstack-lbaas | 12:08 | |
*** woodster_ has joined #openstack-lbaas | 13:00 | |
*** chlong_ has quit IRC | 13:56 | |
*** amotoki_ has joined #openstack-lbaas | 14:16 | |
*** apuimedo has quit IRC | 14:57 | |
*** mlavalle has joined #openstack-lbaas | 15:00 | |
*** mlavalle_ has joined #openstack-lbaas | 15:02 | |
*** mlavalle_ has quit IRC | 15:02 | |
*** mlavalle_ has joined #openstack-lbaas | 15:02 | |
*** mlavalle has quit IRC | 15:05 | |
*** jschwarz has joined #openstack-lbaas | 15:09 | |
*** jschwarz has quit IRC | 15:13 | |
*** jschwarz has joined #openstack-lbaas | 15:13 | |
*** mestery has quit IRC | 15:24 | |
*** fnaval has joined #openstack-lbaas | 15:24 | |
*** mestery has joined #openstack-lbaas | 15:28 | |
*** TrevorV_ has joined #openstack-lbaas | 15:28 | |
*** sbfox has joined #openstack-lbaas | 15:41 | |
*** sbfox has quit IRC | 15:41 | |
*** jorgem has joined #openstack-lbaas | 15:54 | |
*** xgerman has joined #openstack-lbaas | 15:57 | |
*** kobis has quit IRC | 16:00 | |
*** sbfox has joined #openstack-lbaas | 16:07 | |
*** sbfox has quit IRC | 16:08 | |
*** amotoki_ has quit IRC | 16:16 | |
blogan | yall sure like using the term bikeshedding now | 16:32 |
blogan | whats latin for queue consumer? | 16:33 |
TrevorV_ | blogan, http://www.wordhippo.com/what-is/the/latin-word-for-cb7a82c317871051c535dc464aab698448138aae.html | 16:52 |
blogan | TrevorV_: I was joking | 16:59 |
*** mwang2_ has joined #openstack-lbaas | 17:01 | |
*** mwang2 has quit IRC | 17:04 | |
TrevorV_ | blogan, I know :D | 17:09 |
rm_work | blogan: T_T | 17:09 |
rm_work | i still feel like there was more to that discussion that i missed | 17:09 |
rm_work | but ah well | 17:09 |
blogan | no it was basically what sbalukoff said | 17:11 |
*** jorgem has quit IRC | 17:16 | |
*** jorgem has joined #openstack-lbaas | 17:22 | |
xgerman | let's vote on names again | 17:25 |
*** sbfox has joined #openstack-lbaas | 17:25 | |
blogan | lets vote on whether we should vote on names again | 17:28 |
*** jorgem has quit IRC | 17:33 | |
*** jschwarz has quit IRC | 17:35 | |
*** enikanorov__ has joined #openstack-lbaas | 17:36 | |
*** enikanorov_ has quit IRC | 17:38 | |
*** jorgem has joined #openstack-lbaas | 17:44 | |
rm_work | #vote sandwiches (looooonch, bbl) | 17:47 |
*** mwang2__ has joined #openstack-lbaas | 17:56 | |
*** SumitNaiksatam has joined #openstack-lbaas | 17:57 | |
*** mwang2_ has quit IRC | 18:00 | |
*** barclaac has joined #openstack-lbaas | 18:02 | |
*** barclaac|2 has quit IRC | 18:05 | |
*** kbyrne has quit IRC | 18:07 | |
*** Youcef has joined #openstack-lbaas | 18:07 | |
*** barclaac has quit IRC | 18:07 | |
*** crc32 has joined #openstack-lbaas | 18:08 | |
*** kbyrne has joined #openstack-lbaas | 18:09 | |
*** evgenyf has quit IRC | 18:26 | |
dougwig | oh lord, not amphora redux 77. | 18:29 |
dougwig | can we go back to "vm" yet? | 18:29 |
*** barclaac has joined #openstack-lbaas | 18:52 | |
*** barclaac has quit IRC | 18:53 | |
*** barclaac has joined #openstack-lbaas | 18:53 | |
*** evgenyf has joined #openstack-lbaas | 19:03 | |
*** SumitNaiksatam has quit IRC | 19:13 | |
*** SumitNaiksatam has joined #openstack-lbaas | 19:17 | |
xgerman | TrevorV, blogan, rmwork: I was playing with the nova driver and here are a couple of issues: | 19:32 |
xgerman | - we (=HP) need support for regions | 19:32 |
xgerman | - we can deal with nova being v3 only but our public cloud runs v2 so that would be nice if this would be configurable | 19:33 |
xgerman | I am finding more stuff - but wondeirng if I should just make a patch... or how we should handle that... | 19:33 |
*** Miouge has quit IRC | 19:45 | |
*** [1]evgenyf has joined #openstack-lbaas | 19:46 | |
*** evgenyf has quit IRC | 19:46 | |
*** [1]evgenyf is now known as evgenyf | 19:46 | |
jorgem | rm_work: https://blueprints.launchpad.net/octavia/+spec/tls-data-security https://blueprints.launchpad.net/octavia/+spec/barbican-interface-library | 19:47 |
rm_work | johnsom: hey ^^ per above, looks like you created a BP for something I was tracking as part of another BP already | 19:53 |
rm_work | johnsom: and the thing you made the BP for is "done" | 19:53 |
johnsom | which BP? | 19:54 |
rm_work | crc32: Per-secret Policy (the Barbican alternative to Keystone Trusts): https://review.openstack.org/#/c/127353/ | 19:54 |
rm_work | johnsom: the ones jorgem linked above | 19:54 |
johnsom | I did not create tls-data-security, sbalukoff did. The barbican one was a placeholder for the controller break out. | 19:56 |
*** [1]evgenyf has joined #openstack-lbaas | 19:56 | |
johnsom | I can mark it superseded. Which one covers that? | 19:56 |
johnsom | Does the tls-data-security cover barbican-interface-library? | 19:57 |
rm_work | crc32: Spec for Identifying CAs (the basis of the CA stuff): https://review.openstack.org/#/c/129048/ and then the one we'd use (private self-signed): https://review.openstack.org/#/c/140575/ | 19:58 |
rm_work | johnsom: yes | 19:58 |
*** evgenyf has quit IRC | 19:58 | |
*** [1]evgenyf is now known as evgenyf | 19:58 | |
johnsom | rm_work: done | 19:59 |
*** Youcef has quit IRC | 20:03 | |
*** mestery_ has joined #openstack-lbaas | 20:09 | |
*** kobis has joined #openstack-lbaas | 20:09 | |
*** mestery has quit IRC | 20:11 | |
*** SumitNaiksatam has quit IRC | 20:12 | |
openstackgerrit | Carlos Garza proposed openstack/neutron-lbaas: Common TLS utilities https://review.openstack.org/142915 | 20:14 |
*** kobis has quit IRC | 20:15 | |
*** evgenyf has quit IRC | 20:18 | |
*** jorgem has quit IRC | 20:24 | |
openstackgerrit | Trevor Vardeman proposed stackforge/octavia: haproxy reference amphora API client https://review.openstack.org/145637 | 20:25 |
*** jorgem has joined #openstack-lbaas | 20:46 | |
rm_work | johnsom: thanks | 20:51 |
blogan | xgerman: yeah im pretty sure we run v2 here at rax too for nova | 20:53 |
rm_work | crc32: this is actually the CSR spec I think: https://review.openstack.org/#/c/129048/ | 20:53 |
xgerman | yeah, the code is v3 | 20:53 |
xgerman | also I found an issue with the interface list | 20:53 |
xgerman | we don't have that extension but there is a build in network property | 20:54 |
blogan | xgerman: ok we can definitely work to get a solution that works for everyone | 20:56 |
blogan | a driver for the nova driver eh? | 20:57 |
blogan | lets do it! | 20:57 |
xgerman | lol | 20:57 |
xgerman | I think we should be able to pack that in one driver | 20:57 |
blogan | yeah im sure we can | 20:58 |
xgerman | I can propose a patch later today which toggles between v2 and v3 and checks for that extension | 20:59 |
blogan | be my guest, whatever it takes to get that working | 21:00 |
xgerman | but I am wondeirng if we need the interface extension or if network is goo enough | 21:00 |
xgerman | code: | 21:00 |
xgerman | def _translate_amphora(self, nova_response): | 21:00 |
xgerman | '''Convert a nova virtual machine into an amphora object. | 21:00 |
xgerman | :param nova_response: JSON response from nova | 21:00 |
xgerman | :returns: an amphora object | 21:00 |
xgerman | ''' | 21:00 |
xgerman | # Extract information from nova response to populate desired amphora | 21:00 |
xgerman | # fields | 21:00 |
xgerman | lb_network_ip = None | 21:00 |
xgerman | for interface in nova_response.networks: | 21:00 |
xgerman | if interface == CONF.networking.lb_network_id: | 21:00 |
xgerman | lb_network_ip = nova_response.networks[interface] | 21:00 |
dougwig | pastebin, xgerman | 21:00 |
xgerman | response = models.Amphora( | 21:00 |
xgerman | compute_id=nova_response.id, | 21:00 |
xgerman | status=nova_response.status, | 21:00 |
xgerman | lb_network_ip=lb_network_ip | 21:00 |
xgerman | ) | 21:00 |
xgerman | return response | 21:00 |
dougwig | instead of toggling, shouldn't we have two drivers (one v2, one v3) ? | 21:01 |
xgerman | http://pastebin.com/LDNxSAZB | 21:01 |
*** mestery_ is now known as mestery | 21:01 | |
xgerman | well, it just changes some constant | 21:01 |
rm_work | crc32: this might also be interesting: https://review.openstack.org/#/c/142212/ | 21:04 |
xgerman | ok, I will just propose a patch later today and then we can discuss ;-) | 21:09 |
*** sbfox has quit IRC | 21:15 | |
dougwig | xgerman: then you can have one derive from the other, but the toggle is still in the driver selection. | 21:18 |
dougwig | i guess that's six and one half dozen. | 21:19 |
xgerman | ok, sounds good | 21:19 |
*** sbfox has joined #openstack-lbaas | 21:19 | |
*** evgenyf has joined #openstack-lbaas | 21:20 | |
*** chlong_ has joined #openstack-lbaas | 21:20 | |
blogan | xgerman: if you can get the ips for all the nics without that then go for it | 21:24 |
xgerman | it seems I can do that | 21:24 |
xgerman | ok, I will make a patch then ;-) | 21:25 |
blogan | xgerman: TrevorV and rm_work/rm_you worked on that together, so I would ask them why they were using that method | 21:26 |
blogan | which i just did | 21:26 |
xgerman | thanks - I was wondeirng myself | 21:26 |
TrevorV_ | woah wait what? | 21:31 |
xgerman | http://pastebin.com/LDNxSAZB | 21:32 |
xgerman | interfaces is some extension HP cloud doesn't have | 21:32 |
xgerman | so I coded it with the newtorks attrribute on the server | 21:32 |
xgerman | wonder why you guys picked interfaces? | 21:32 |
TrevorV_ | That's rm_work / rm_you 's fault... | 21:33 |
xgerman | well, I am also running nova v2 (had to change that in code, too) | 21:33 |
TrevorV_ | He told me the other one wasn't a guarantee or something like that, and interfaces was | 21:33 |
*** [1]evgenyf has joined #openstack-lbaas | 21:33 | |
TrevorV_ | xgerman, that should be a config value, did I mess that up? | 21:33 |
TrevorV_ | I probably hard-coded it | 21:33 |
TrevorV_ | :( | 21:33 |
xgerman | yep + dougwig suggested making two drivers v2 and v3 | 21:34 |
rm_work | TrevorV_: yep i told blogan already :P | 21:34 |
rm_work | I was about to look and see if we can get to the attribute that German is talking about | 21:34 |
TrevorV_ | Damnit... Now I feel bad, thought I had that already sorted out. | 21:34 |
xgerman | rm_work can you repat for people who don't have bugged RAX's office | 21:34 |
rm_work | xgerman: was referring to [15:32:53] <TrevorV_>That's rm_work / rm_you 's fault... | 21:35 |
rm_work | it is in fact my fault | 21:35 |
xgerman | ok | 21:35 |
rm_work | I told TrevorV_ to use that nova ttribute | 21:35 |
rm_work | *attribute | 21:35 |
rm_work | because in our system it was the simplest to guarantee correctness | 21:35 |
TrevorV_ | (just means I didn't think for myself, rm_work) | 21:35 |
rm_work | the other way is possible but kinda icky | 21:35 |
TrevorV_ | Lulz | 21:35 |
rm_work | I did not realize that was our own extension | 21:35 |
xgerman | yeah, glad I tried that this monring ;-) | 21:36 |
TrevorV_ | Are there significant differences between v2 and v3 clients that we'd need multiple drivers? | 21:36 |
rm_work | TrevorV_: there is no nova v3 | 21:36 |
xgerman | also all those get('id') or whatever break | 21:36 |
TrevorV_ | its documented, is it not? | 21:36 |
rm_work | they abandoned v3 | 21:36 |
TrevorV_ | Ooooh okay | 21:36 |
rm_work | there will be a nova v2.1 | 21:36 |
TrevorV_ | wait, xgerman what do you mean? | 21:36 |
*** evgenyf has quit IRC | 21:36 | |
*** [1]evgenyf is now known as evgenyf | 21:36 | |
rm_work | they decided to move to microversioning and not do a whole breaking rewrite, i think | 21:37 |
TrevorV_ | xgerman, you're saying there is no "id" field returned by nova? | 21:37 |
rm_work | xgerman: something.get('id') should by definition NOT break | 21:37 |
rm_work | that's the point of "get()" | 21:37 |
rm_work | unless "something" is None | 21:37 |
xgerman | well, it only worked on my end when I said .id | 21:37 |
rm_work | uhh | 21:37 |
TrevorV_ | That makes even less sense ha ha | 21:37 |
rm_work | it's properties and not a dict? | 21:37 |
rm_work | wtf | 21:37 |
xgerman | yep | 21:38 |
xgerman | might be a v2 thing | 21:38 |
rm_work | xgerman: can you paste me the python code necessary to spin up a nova client and get that result? | 21:38 |
rm_work | without your keys, obviously | 21:38 |
TrevorV_ | brb | 21:38 |
rm_work | I can compare it to what I was doing | 21:38 |
rm_work | xgerman: I assume you have just done it in a python CLI | 21:38 |
rm_work | to test | 21:38 |
xgerman | nope, I pliugged it into my taskflow to spin up an amphora | 21:39 |
xgerman | my code: http://pastebin.com/GYAhJ5i7 | 21:39 |
dougwig | keep in mind that "two drivers" might just be one class that derives from another ,with a different constant set in init. | 21:40 |
xgerman | yep | 21:40 |
xgerman | that seems the most sane | 21:40 |
TrevorV_ | that's fair dougwig | 21:43 |
xgerman | I can hammer out that patch (but I don't have a v3 system to test - yet) | 21:45 |
rm_work | (there is no nova v3) | 21:45 |
xgerman | that explains it ;-) | 21:45 |
rm_work | so, there's this | 21:45 |
rm_work | http://developer.openstack.org/api-ref-compute-v3.html | 21:45 |
rm_work | which was abandoned | 21:45 |
blogan | it is or will be 2.1, which means it will be backwards compatible with 2.0 | 21:45 |
rm_work | I am trying to find somewhere that actually clearly states what happened | 21:46 |
rm_work | it became this: http://developer.openstack.org/api-ref-compute-v2.1.html | 21:46 |
rm_work | per http://developer.openstack.org/api-ref.html (I guess that's the best i can do) | 21:46 |
*** chlong_ has quit IRC | 21:50 | |
xgerman | so what are we gonna do? | 21:52 |
dougwig | aren't they going to micro versions? | 21:52 |
xgerman | yep, that, too | 21:52 |
rm_work | yes | 21:53 |
*** SumitNaiksatam has joined #openstack-lbaas | 21:55 | |
TrevorV_ | xgerman, there should only be one location to choose what version it is, if I remember correctly | 21:57 |
xgerman | yep | 21:58 |
openstackgerrit | German Eichberger proposed stackforge/octavia: Patches nova client to work with HP Cloud https://review.openstack.org/147674 | 21:59 |
xgerman | ^^^ this si the ode which will fail py27 | 21:59 |
xgerman | or not | 22:00 |
xgerman | TrevorV_, rm_work | 22:01 |
rm_work | hmm | 22:01 |
rm_work | I am slightly concerned about the property vs. dictionary thing | 22:02 |
xgerman | well, now you have code to play with and let me know ;-) | 22:02 |
rm_work | yeah i will look | 22:02 |
TrevorV_ | thanks for looking at this xgerman | 22:09 |
TrevorV_ | My fault for not doing a more thorough job :( | 22:09 |
TrevorV_ | Did have one nit though | 22:09 |
xgerman | well, I reviewed it so it's my fault as well ;-) | 22:09 |
TrevorV_ | as rm_work said, that dictionary versus property thing was surprising | 22:10 |
rm_work | i am trying to get keystone to auth me right now | 22:10 |
TrevorV_ | Mostly because I thought that was interchangeable but had distinct differences... my fault for making an assumption | 22:10 |
rm_work | but once I get that | 22:10 |
rm_work | i will know wtf | 22:10 |
rm_work | damnit keystone | 22:11 |
xgerman | cool | 22:11 |
TrevorV_ | switching my computer to windows, be back shortly | 22:12 |
*** TrevorV_ has quit IRC | 22:12 | |
rm_work | ok, properties is correct | 22:15 |
rm_work | checking networks stuff now | 22:15 |
*** TrevorV_ has joined #openstack-lbaas | 22:17 | |
rm_work | xgerman: as long as the Amphora model can handle "lb_network_ip" being a list (including ipv4 and ipv6) then your code is good | 22:18 |
xgerman | we don't have ipv6/v4 mixed so i am not sure... | 22:19 |
TrevorV_ | Alright, I'm here. | 22:20 |
TrevorV_ | So what was the thing figured out about property versus dict? | 22:20 |
rm_work | a property is accessed "object.property" | 22:20 |
rm_work | object.get("thing") only works if object is a dict() | 22:20 |
TrevorV_ | Yes, thanks for the python lesson :P I was referring to what was returned in the nova-client. | 22:21 |
rm_work | nova client returns an object | 22:21 |
TrevorV_ | Its an object then? I was under the impression it WAS a dict | 22:21 |
TrevorV_ | Gotcha | 22:21 |
TrevorV_ | My fault then | 22:21 |
TrevorV_ | Misunderstood that part as well | 22:21 |
rm_work | I think when I was showing you stuff, I was showing you a dict-dump of the object | 22:21 |
TrevorV_ | Thanks for the catch xgerman | 22:21 |
xgerman | no worries -- I should have caught ti during review... | 22:22 |
rm_work | but under the (mistaken) assumption that you understood that wasn't the actual object :P | 22:22 |
rm_work | because a dict-dump was the easiest way to enumerate what all the object had | 22:22 |
TrevorV_ | Right, dictionaries CAN use the dot notation though right? | 22:22 |
rm_work | nope | 22:22 |
TrevorV_ | You sure about that? | 22:23 |
rm_work | yes | 22:23 |
rm_work | server = mynova.servers.get(server_id) | 22:23 |
rm_work | server.to_dict() | 22:23 |
rm_work | is what I showed you | 22:23 |
rm_work | may have not explained that well at the time | 22:23 |
rm_work | <_< | 22:23 |
rm_work | xgerman: reviewed | 22:26 |
xgerman | cool | 22:27 |
TrevorV_ | See I thought you me and brandon talked about the dot notation versus the .get notation... and .get was appropriate in one situation but still interchangeable somewhere | 22:27 |
rm_work | you can do either: | 22:27 |
rm_work | mydict['apples'] OR mydict.get('apples') | 22:27 |
rm_work | in the first case, you'll get a KeyError if there's no 'apples' | 22:28 |
rm_work | in the second, you' | 22:28 |
rm_work | *you'll just get None returned | 22:28 |
rm_work | sometimes you'll want a keyerror and sometimes you'll want it to be more graceful | 22:28 |
rm_work | or, you can do mydict.get('apples', 0) | 22:28 |
rm_work | to set the default return to 0 | 22:28 |
TrevorV_ | Oooooooooooooooooooooooooooooooh | 22:28 |
TrevorV_ | Okay, so it wasn't dot, it was bracket | 22:29 |
TrevorV_ | My fault | 22:29 |
TrevorV_ | Okay, my brain was a little broked there | 22:29 |
rm_work | iz cool, you pulled one over on all the reviewers :P | 22:29 |
TrevorV_ | NOT on purpose | 22:29 |
TrevorV_ | :D | 22:29 |
rm_work | :P | 22:29 |
TrevorV_ | So I won't take the "juked me out" credit, but the "you messed up" credit I'll accept | 22:29 |
rm_work | lol | 22:29 |
rm_work | review juking | 22:30 |
dougwig | i usually don't want a key error, so I think they got the [] vs method wrong (ruby[] is None if missing, which is vastly nicer.) | 22:30 |
rm_work | eh | 22:30 |
rm_work | I like it if I want to be doing try/catch | 22:30 |
rm_work | especially if None is a value that might be possible to set | 22:30 |
TrevorV_ | Like, nullable fields rm_work ? | 22:30 |
rm_work | it is important to know the difference between "unset" and None | 22:30 |
rm_work | yes | 22:31 |
dougwig | i'm not saying you shouldn't be able to do that; i'm saying which is really the common case? | 22:31 |
dougwig | and that common case should get the [] | 22:31 |
rm_work | eh. | 22:32 |
TrevorV_ | Alright, well, since python is opensource you should go in there and make it happen. We'll wait here :D | 22:32 |
rm_work | ^^ | 22:32 |
rm_work | I said that to crc32 about gerrit earlier and I just got a glare :P | 22:33 |
dougwig | ha, if I had that pull, i'd make whitespace not significant. | 22:33 |
blogan | dougwig quit extolling ruby | 22:33 |
dougwig | that last is extolling nearly every other language. | 22:33 |
rm_work | dougwig is master troll | 22:34 |
* dougwig demands a toll. | 22:34 | |
rm_work | python whitespace is actually one reason I love python so much :P | 22:34 |
blogan | i like the forced indentation in python | 22:34 |
rm_work | it FORCES good code style | 22:34 |
blogan | style = format? | 22:34 |
crc32 | rm_workspace: Considering I can't see whitespace to fix it. Its one of my pet peeves. | 22:35 |
blogan | lol rm_workspace | 22:35 |
rm_work | heh | 22:38 |
dougwig | i learned python in the bad years of space vs tab not doing sane things when inter-mixed. | 22:39 |
rm_work | it's still pretty messed up with tabs | 22:39 |
rm_work | but most IDEs auto-replace with 4x space now | 22:39 |
dougwig | plus, it's goddamn annoying when it forces a trivial one-line loop to be spread out. | 22:39 |
rm_work | that's PEP8 I thought, not technically a pure python restriction | 22:40 |
rm_work | I thought you could do: | 22:40 |
rm_work | for i in range(1,10): print(i) | 22:40 |
dougwig | yeah, i want to see that get accepted. | 22:41 |
TrevorV_ | I think there are a lot of pep8 issues I don't really care for. | 22:41 |
TrevorV_ | pep8 restrictions I mean | 22:41 |
rm_work | dougwig: my point is it *works* in python | 22:43 |
rm_work | it's just pep8 that says "don't please" | 22:43 |
*** openstack has joined #openstack-lbaas | 23:02 | |
*** sendak.freenode.net sets mode: -o rm_work | 23:06 | |
-sendak.freenode.net- *** Notice -- TS for #openstack-lbaas changed from 1421362519 to 1403021244 | 23:06 | |
*** sendak.freenode.net sets mode: +ct-s | 23:06 | |
*** omnibus7 has joined #openstack-lbaas | 23:06 | |
*** LinuxJedi_ has joined #openstack-lbaas | 23:06 | |
*** mordred has joined #openstack-lbaas | 23:06 | |
*** blogan has joined #openstack-lbaas | 23:06 | |
*** sbfox has joined #openstack-lbaas | 23:06 | |
*** pckizer__ has joined #openstack-lbaas | 23:06 | |
*** aiharos_ has joined #openstack-lbaas | 23:06 | |
*** enikanorov___ has joined #openstack-lbaas | 23:06 | |
*** dougw2 has joined #openstack-lbaas | 23:06 | |
*** enikanorov_ has joined #openstack-lbaas | 23:06 | |
*** ptoohill has joined #openstack-lbaas | 23:06 | |
*** jorgem1 has joined #openstack-lbaas | 23:06 | |
*** fnaval_ has joined #openstack-lbaas | 23:06 | |
*** ctracey_ has joined #openstack-lbaas | 23:06 | |
*** evgenyf has joined #openstack-lbaas | 23:06 | |
*** mestery has joined #openstack-lbaas | 23:06 | |
*** kbyrne has joined #openstack-lbaas | 23:06 | |
*** mwang2__ has joined #openstack-lbaas | 23:06 | |
*** enikanorov__ has joined #openstack-lbaas | 23:06 | |
*** xgerman has joined #openstack-lbaas | 23:06 | |
*** woodster_ has joined #openstack-lbaas | 23:06 | |
*** intr1nsic has joined #openstack-lbaas | 23:06 | |
*** sballe_ has joined #openstack-lbaas | 23:06 | |
*** rm_you has joined #openstack-lbaas | 23:06 | |
*** jkoelker has joined #openstack-lbaas | 23:06 | |
*** mugu has joined #openstack-lbaas | 23:06 | |
*** eren has joined #openstack-lbaas | 23:06 | |
*** LinuxJedi has joined #openstack-lbaas | 23:06 | |
*** johnsom has joined #openstack-lbaas | 23:06 | |
*** openstackgerrit has joined #openstack-lbaas | 23:06 | |
*** sendak.freenode.net changes topic to "LBaaS and Octavia reviews - https://etherpad.openstack.org/p/lbaas_reviews" | 23:06 | |
*** enikanorov_ has quit IRC | 23:07 | |
*** xgerman has quit IRC | 23:07 | |
*** sballe_ has quit IRC | 23:07 | |
*** openstackgerrit has quit IRC | 23:07 | |
*** johnsom has quit IRC | 23:07 | |
*** LinuxJedi has quit IRC | 23:07 | |
*** bedis has joined #openstack-lbaas | 23:08 | |
*** clev has joined #openstack-lbaas | 23:09 | |
*** codekobe has joined #openstack-lbaas | 23:12 | |
*** amotoki has joined #openstack-lbaas | 23:12 | |
*** TrevorV has joined #openstack-lbaas | 23:12 | |
*** redrobot has joined #openstack-lbaas | 23:12 | |
*** HenryG has joined #openstack-lbaas | 23:12 | |
*** xgerman has joined #openstack-lbaas | 23:12 | |
*** johnsom has joined #openstack-lbaas | 23:13 | |
*** mrwangmaster has joined #openstack-lbaas | 23:13 | |
*** openstackgerrit has joined #openstack-lbaas | 23:13 | |
*** barclaac has joined #openstack-lbaas | 23:13 | |
*** sballe__ has joined #openstack-lbaas | 23:13 | |
*** sbalukoff has joined #openstack-lbaas | 23:13 | |
*** BeardyMcBeards has joined #openstack-lbaas | 23:14 | |
*** redrobot is now known as Guest29310 | 23:14 | |
*** crc32 has joined #openstack-lbaas | 23:16 | |
*** ctracey_ is now known as ctracey | 23:20 | |
*** mlavalle_ has joined #openstack-lbaas | 23:21 | |
*** dougwig has joined #openstack-lbaas | 23:23 | |
rm_work | is this review still actually still being worked on? https://review.openstack.org/#/c/141948/ mwang2__ | 23:23 |
rm_work | can't quite tell, there are like 4 reviews that are updating aspects of this same thing, and most are merged but this is WIP and older? | 23:24 |
mwang2__ | thie one can be abonded | 23:25 |
rm_work | ok, you are the only one who has the "abandon" button | 23:25 |
rm_work | can you do that? | 23:26 |
mwang2__ | let me find it | 23:26 |
*** fnaval_ has quit IRC | 23:27 | |
dougwig | anyone else get killed by that net split? | 23:28 |
*** johnsom has quit IRC | 23:30 | |
*** johnsom has joined #openstack-lbaas | 23:30 | |
*** barclaac has quit IRC | 23:30 | |
*** barclaac has joined #openstack-lbaas | 23:30 | |
*** sballe__ has quit IRC | 23:31 | |
*** sballe__ has joined #openstack-lbaas | 23:31 | |
*** mlavalle_ has quit IRC | 23:31 | |
*** mlavalle_ has joined #openstack-lbaas | 23:31 | |
*** dougwig has quit IRC | 23:31 | |
*** dougwig has joined #openstack-lbaas | 23:31 | |
rm_work | mwang2__: should be in the middle of https://review.openstack.org/#/c/141948/ right below the current patchset, next to the review button | 23:31 |
*** SumitNaiksatam has joined #openstack-lbaas | 23:32 | |
*** barclaac|2 has joined #openstack-lbaas | 23:32 | |
mwang2__ | done | 23:32 |
*** barclaac has quit IRC | 23:33 | |
rm_work | cool | 23:36 |
openstackgerrit | German Eichberger proposed stackforge/octavia: Patches nova client to work with HP Cloud https://review.openstack.org/147674 | 23:38 |
*** chlong has joined #openstack-lbaas | 23:40 | |
*** jorgem1 has quit IRC | 23:44 | |
*** markmcclain has joined #openstack-lbaas | 23:47 | |
*** fnaval has joined #openstack-lbaas | 23:50 | |
*** crc32 has quit IRC | 23:54 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!