Tuesday, 2015-03-03

openstackgerritLin Hua Cheng proposed openstack/python-openstackclient: Add ability for diplaying hypervisor statistics  https://review.openstack.org/15763500:04
*** chlong has quit IRC00:15
*** stevemar has joined #openstack-sdks00:18
*** bknudson has joined #openstack-sdks00:37
openstackgerritBrian Curtin proposed stackforge/python-openstacksdk: Add base for Proxy classes to inherit from  https://review.openstack.org/16058500:59
openstackgerritBrian Curtin proposed stackforge/python-openstacksdk: Add ServerDetail so list can include details  https://review.openstack.org/16059801:43
*** rmcall has joined #openstack-sdks02:06
*** sigmavirus24 is now known as sigmavirus24_awa02:08
*** terrylhowe has joined #openstack-sdks02:54
openstackgerritBrian Curtin proposed stackforge/python-openstacksdk: Set Flavor and Image resources on Server  https://review.openstack.org/16061803:14
openstackgerritBrian Curtin proposed stackforge/python-openstacksdk: Set put_update for compute.v2.server.Server  https://review.openstack.org/16062003:17
openstackgerritBrian Curtin proposed stackforge/python-openstacksdk: Add ServerDetail so list can include details  https://review.openstack.org/16059803:31
openstackgerritBrian Curtin proposed stackforge/python-openstacksdk: Set Flavor and Image resources on Server  https://review.openstack.org/16061803:31
openstackgerritBrian Curtin proposed stackforge/python-openstacksdk: Add example to do CRUD operations on servers  https://review.openstack.org/16062303:31
openstackgerritMerged stackforge/python-openstacksdk: Fix assert order in test_resource  https://review.openstack.org/16046704:27
openstackgerritDean Troyer proposed openstack/python-openstackclient: Add --os-cloud support  https://review.openstack.org/12979504:37
dtroyermordred: ^^^^   first cut at a fresh update. see how many corner cases you can find ;)04:45
dtroyeralso help w/o auth should be a bit more sane now too04:46
openstackgerritDean Troyer proposed openstack/python-openstackclient: Add --os-cloud support  https://review.openstack.org/12979504:49
openstackgerritBrian Curtin proposed stackforge/python-openstacksdk: Use update_attrs to handle updating servers  https://review.openstack.org/16063504:57
openstackgerritBrian Curtin proposed stackforge/python-openstacksdk: Add ServerDetail so list can include details  https://review.openstack.org/16059805:00
openstackgerritBrian Curtin proposed stackforge/python-openstacksdk: Set put_update for compute.v2.server.Server  https://review.openstack.org/16062005:00
openstackgerritBrian Curtin proposed stackforge/python-openstacksdk: Add example to do CRUD operations on servers  https://review.openstack.org/16062305:00
openstackgerritBrian Curtin proposed stackforge/python-openstacksdk: Set Flavor and Image resources on Server  https://review.openstack.org/16061805:00
openstackgerritBrian Curtin proposed stackforge/python-openstacksdk: Add example to do CRUD operations on servers  https://review.openstack.org/16062305:20
*** redrobot has quit IRC06:21
*** redrobot has joined #openstack-sdks06:25
*** redrobot is now known as Guest3254406:25
*** terrylhowe has quit IRC06:52
*** rmcall has quit IRC06:58
*** dstufft has quit IRC07:08
*** dstufft has joined #openstack-sdks07:15
*** openstackgerrit has quit IRC07:22
*** openstackgerrit has joined #openstack-sdks07:22
*** stevemar has quit IRC08:15
*** sigmavirus24_awa has quit IRC08:26
*** stevelle has quit IRC09:46
*** stevelle has joined #openstack-sdks10:00
*** tellesnobrega has quit IRC11:27
*** terrylhowe has joined #openstack-sdks11:27
*** tellesnobrega has joined #openstack-sdks11:33
mordreddtroyer: oo! exciting11:49
mordreddtroyer: if we get that working, do you want to include os-client-config in your openstackclient project proposal?11:50
mordreddtroyer: oh - also - I don't know if you saw - but auth stuff goes into an auth dict now - which is to support auth plugins and the variability of auth params11:51
mordredShrews: ^^ check out dtroyer's latest patch12:15
mordredShrews: its almost like we could start doing everything from one set of config!12:16
mordredShrews: since dtroyer is using "auth_type" for plugin name, perhaps we should change the name of that param in ansible and shade too for consistency12:22
mordreddtroyer: oh - you know - you may already be taking car of the auth dict in os-client-config  ...12:24
mordredah - nope - it goes the other direction12:25
openstackgerritMonty Taylor proposed stackforge/os-client-config: Rename auth_plugin to auth_type  https://review.openstack.org/16077112:32
mordreddtroyer, Shrews: ^^ there ya go - that should make that part slighly less suck12:32
dtroyermordred: This was a pretty direct rebase of what I did last fall.  I have to keep control of the auth bits in OSC because I have two places to use different plugins that Jamie didn't want in ksc.12:38
openstackgerritJulien Danjou proposed openstack/python-openstackclient: telemetry: add support for alarm creation  https://review.openstack.org/15582212:39
mordreddtroyer: no worries - I think it's actually an easy fix12:39
mordreddtroyer: because you keep an auth dict in osc yourself12:39
dtroyerksc's token/url plugin is not the token/endpoint that we've grown to love so to stay compatible I needed to add that, then there's a workaround for keystone's default config that has localhost in the service caalog URLs12:40
mordreddtroyer: I _think_ something like this: http://paste.openstack.org/show/18631012:41
mordreddtroyer: will at least get real close to consuming auth dict and also doing what you're already doing12:41
mordred(untested)12:41
mordredstill walking through the code - I think there are 1 or 2 other places in clientmanager that need to talk to _auth_params instead of _cli_options12:42
mordreddtroyer: oh - you know what - there's another thing we could do ...12:43
mordrednah - let me poke at this for just a second longer before I go and get hacky :)12:46
mordreddtroyer: also, I was thinking about doing something like what you do wrt domains and default_domain over in shade - so perhaps that's another data massage that os-client-config wants to grow ...12:46
dtroyerok.  I think the auth dict should be doable, but I'm not sure about the validate function.  no caffeine yet so I don't remember why I couldn't use it last night12:47
mordreddtroyer: I'm cafinating already - so I'll take a pass and see if I can get it to work for me :)12:48
dtroyercool12:48
openstackgerritDean Troyer proposed openstack/python-openstackclient: Handle novaclient >2.20.0  https://review.openstack.org/16047313:02
mordredyes. please merge that one :)13:02
dtroyerstevemar ^^^ stevemar ^^^ stevemar13:03
dtroyersometimes that wakes him up...13:03
mordreddtroyer: do you ever have one of those moments where you're trying to figure out why something is erroring13:04
mordreddtroyer: and you put a bunch of prints in13:04
mordredand you get a print around each side of a thing that shouldn't be able to fail13:04
mordredand yet it does?13:04
dtroyerall. the. time.     re: last nigh and the tests for this change…13:05
dtroyerI chalk it up to my still-not-too-deep grokking of Pythonism13:05
mordreddtroyer: ok - well, I'm almost there with this change and the auth dict13:06
mordredbut I need to change locations13:06
mordredso it'll be probably 2 hours before I can have a follow up patch for you13:06
dtroyernp, I have $DAY_JOB waiting for me too13:06
mordred(I'm very motivated for this, btw, because it will help my day-to-day immensely)13:06
*** mattfarina has joined #openstack-sdks14:18
Shrewsmordred: neato14:21
*** chlong has joined #openstack-sdks14:27
openstackgerritJulien Danjou proposed stackforge/python-openstacksdk: identity: add missing tenant options to valid options  https://review.openstack.org/16082114:52
openstackgerritJulien Danjou proposed openstack/python-openstackclient: telemetry: add support for alarm creation  https://review.openstack.org/15582214:53
openstackgerritBrian Curtin proposed stackforge/python-openstacksdk: Add base for Proxy classes to inherit from  https://review.openstack.org/16058514:55
openstackgerritBrian Curtin proposed stackforge/python-openstacksdk: Add base for Proxy classes to inherit from  https://review.openstack.org/16058515:00
openstackgerritJulien Danjou proposed stackforge/python-openstacksdk: identity: add missing tenant options to valid options  https://review.openstack.org/16082115:21
openstackgerritJulien Danjou proposed stackforge/python-openstacksdk: identity: use set() for valid_options  https://review.openstack.org/16084015:26
mordreddtroyer: got it working15:28
mordreddtroyer: let me clean up my debugging15:28
openstackgerritMonty Taylor proposed openstack/python-openstackclient: Add --os-cloud support  https://review.openstack.org/12979515:35
mordreddtroyer: that works for me  - I haven't dived into the test suite yet15:35
*** sigmavirus24 has joined #openstack-sdks15:35
mordreddtroyer: it isn't doing image list for me - but I'll dig in to that next15:36
dtroyermordred: I think there are some stragglers to pick up in build_auth_params yet that do not come from the plugins15:44
mordreddtroyer: nope - it was endpoint hacking15:45
openstackgerritMonty Taylor proposed openstack/python-openstackclient: Don't hack the image url  https://review.openstack.org/16084815:46
mordreddtroyer: that fixes image list for me15:47
openstackgerritMerged stackforge/python-openstacksdk: Add base for Proxy classes to inherit from  https://review.openstack.org/16058515:48
*** pm90_ has joined #openstack-sdks15:49
dtroyerI'm having issues picking up values from the environment15:51
*** stevemar has joined #openstack-sdks15:53
*** rmcall has joined #openstack-sdks16:02
openstackgerritJulien Danjou proposed stackforge/python-openstacksdk: identity: use set() for valid_options  https://review.openstack.org/16084016:23
openstackgerritJulien Danjou proposed openstack/python-openstackclient: telemetry: add support for alarm creation  https://review.openstack.org/15582216:26
*** Guest32544 is now known as redrobot16:51
openstackgerritBrian Curtin proposed stackforge/python-openstacksdk: Add example to do CRUD operations on servers  https://review.openstack.org/16062317:07
*** chlong has quit IRC17:17
mordreddtroyer: kk. I'll hit that next17:22
mordreddtroyer: k. I've replicated the env vars problem17:34
*** tellesnobrega has quit IRC17:43
openstackgerritMonty Taylor proposed openstack/python-openstackclient: Pass cloud by name  https://review.openstack.org/16091717:44
mordreddtroyer: ^^ that works for me17:44
*** tellesnobrega has joined #openstack-sdks18:04
*** trown has joined #openstack-sdks18:16
briancurtinpython-openstacksdk meeting in #openstack-meeting-3 in 15'18:45
sigmavirus24briancurtin: i'll be late18:57
sigmavirus24apologies in advance18:57
briancurtinyou're fired18:57
sigmavirus24damnit18:57
* sigmavirus24 starts knocking on cloud providers doors18:57
openstackgerritRamaraja proposed openstack/python-openstackclient: Fix help messages  https://review.openstack.org/16095119:36
*** pm90_ has quit IRC19:52
stevemarsigmavirus24, quick knocking on doors19:54
sigmavirus24stevemar: more like "Let me in!!!"19:57
*** pm90_ has joined #openstack-sdks20:06
*** pm90_ has quit IRC20:06
*** pm90_ has joined #openstack-sdks20:07
mordreddtroyer: hey! I got it all completely working along with unittests20:28
dtroyermordred: awesome!  I'm catching up with what I missed…20:28
mordreddtroyer: I'm rebasing and squashing a couple of things20:28
dtroyerok…  on the images version thing…, that change breaks images for me and default devstack20:29
mordreddtroyer: awesome. so I'll need to figure out a richer fix there20:29
dtroyernot that default devstack is correct20:29
dtroyerreally it just needs discovery20:29
mordreddtroyer: well, the thing I removed was a hack in the first place - yeah20:30
mordredbtw - glance discovery is bonghits20:30
dtroyerand I haven't doen that.  that's why the assumption that setting OS_IMAGE_API_VERSION was good enough20:30
mordreddtroyer: I think what happened with me is taht setting image-api-version still was broken20:30
dtroyerbut I'm not using that…you patched my stand-alone low level API  ;)20:30
mordredbecause the assumption on the url structure was wrong20:30
mordred:)20:30
mordredhehe20:30
dtroyerah, ok20:30
mordredlet me send you a thing without the image-api part20:31
dtroyeryeah, I was going to split that out20:31
openstackgerritMonty Taylor proposed openstack/python-openstackclient: Don't hack the image url  https://review.openstack.org/16084820:33
openstackgerritMonty Taylor proposed openstack/python-openstackclient: Add --os-cloud support  https://review.openstack.org/12979520:33
mordreddtroyer: ok - there's the stack new - I'll work on the image url thing now20:33
mordreddtroyer: I'm _very_ curious as to why the openstackclient keystone.auth entrypoints don't show up in the test runs20:33
dtroyerI haven't looked at that yet…  but I've accumulated some fixes for the functional test in the mean time that I'm going to merge into 129795 if you're done poking at it for a bit20:34
mordreddtroyer: yes. it works for all my stuff except for image20:35
* mordred hands dtroyer the conch20:36
mordreddtroyer: btw - did you see my question from way earlier about including os-client-config in your new client project proposal?20:36
dtroyerI did… I'm fine with that, looks like we'll have cliff too, but I'm waiting for dhellmann to declare that final before sending the proposal.20:37
mordredcool20:37
mordreddtroyer: any hints as to what broke with the image thing? is it that the default catalog returns an endpoint that is different than what needs to get passed in?20:48
dtroyerthe usual (devstack) catalog has URLs without a version.   IIRC glanceclient adds that itself in the version-specific code.  My hack was a placeholder until real discovery is added.20:51
mordreddtroyer: I think my problem is that my catalog has a v1 in it ... perhaps I should just extend the hack to only append the version if there is not a version in the url already?20:52
dtroyermordred: we do something similar for Identity, but the other way around…remove /v2 if it is in auth_url so proper discovery works with legacy auth urlss20:54
openstackgerritMonty Taylor proposed openstack/python-openstackclient: Don't hack the image url  https://review.openstack.org/16084821:06
mordreddtroyer: there ya go - that should still work for you, and works for me on rax and hp21:06
openstackgerritMonty Taylor proposed stackforge/os-client-config: Handle project_name/tenant_name in the auth dict  https://review.openstack.org/16097921:13
mordredShrews: ^^ also, I noticed this21:13
Shrewsmordred: will check it out in a bit21:16
openstackgerritChristy Fernando proposed openstack/python-openstackclient: Changed user list help.  https://review.openstack.org/16098121:17
*** pm90_ has quit IRC21:28
*** pm90_ has joined #openstack-sdks21:32
*** pm90_ has joined #openstack-sdks21:33
openstackgerritMonty Taylor proposed stackforge/os-client-config: Handle project_name/tenant_name in the auth dict  https://review.openstack.org/16097921:34
openstackgerritMonty Taylor proposed stackforge/os-client-config: Rename auth_plugin to auth_type  https://review.openstack.org/16077121:34
openstackgerritMonty Taylor proposed stackforge/os-client-config: Add two newlines to the ends of files  https://review.openstack.org/16098921:34
*** stevemar has quit IRC21:41
*** mattfarina has quit IRC21:59
openstackgerritDean Troyer proposed openstack/python-openstackclient: Add --os-cloud support  https://review.openstack.org/16100622:27
openstackgerritDean Troyer proposed openstack/python-openstackclient: Add --os-cloud support  https://review.openstack.org/12979522:28
dtroyermordred: ^^^  hopefully that passes functional tests and doesn't break you22:28
mordreddtroyer: I shall try22:29
mordreddtroyer: (that looks like two different changes? should I just do 129795?)22:30
mordreddtroyer: seems to work for me22:33
*** rmcall has quit IRC22:36
dtroyeryeah, I fubar'ed the change id, ignore 16100622:42
openstackgerritDean Troyer proposed openstack/python-openstackclient: Don't hack the image url  https://review.openstack.org/16084822:42
openstackgerritMerged stackforge/python-openstacksdk: identity: add missing tenant options to valid options  https://review.openstack.org/16082122:42
openstackgerritMerged stackforge/python-openstacksdk: identity: use set() for valid_options  https://review.openstack.org/16084022:42
*** stevemar has joined #openstack-sdks22:44
*** openstackgerrit has quit IRC22:51
*** openstackgerrit has joined #openstack-sdks22:52
*** pm90_ has quit IRC22:53
*** pm90_ has joined #openstack-sdks23:06
*** stevemar has quit IRC23:29
*** chlong has joined #openstack-sdks23:31
*** sigmavirus24 is now known as sigmavirus24_awa23:36

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