openstackgerrit | Lin Hua Cheng proposed openstack/python-openstackclient: Add ability for diplaying hypervisor statistics https://review.openstack.org/157635 | 00:04 |
---|---|---|
*** chlong has quit IRC | 00:15 | |
*** stevemar has joined #openstack-sdks | 00:18 | |
*** bknudson has joined #openstack-sdks | 00:37 | |
openstackgerrit | Brian Curtin proposed stackforge/python-openstacksdk: Add base for Proxy classes to inherit from https://review.openstack.org/160585 | 00:59 |
openstackgerrit | Brian Curtin proposed stackforge/python-openstacksdk: Add ServerDetail so list can include details https://review.openstack.org/160598 | 01:43 |
*** rmcall has joined #openstack-sdks | 02:06 | |
*** sigmavirus24 is now known as sigmavirus24_awa | 02:08 | |
*** terrylhowe has joined #openstack-sdks | 02:54 | |
openstackgerrit | Brian Curtin proposed stackforge/python-openstacksdk: Set Flavor and Image resources on Server https://review.openstack.org/160618 | 03:14 |
openstackgerrit | Brian Curtin proposed stackforge/python-openstacksdk: Set put_update for compute.v2.server.Server https://review.openstack.org/160620 | 03:17 |
openstackgerrit | Brian Curtin proposed stackforge/python-openstacksdk: Add ServerDetail so list can include details https://review.openstack.org/160598 | 03:31 |
openstackgerrit | Brian Curtin proposed stackforge/python-openstacksdk: Set Flavor and Image resources on Server https://review.openstack.org/160618 | 03:31 |
openstackgerrit | Brian Curtin proposed stackforge/python-openstacksdk: Add example to do CRUD operations on servers https://review.openstack.org/160623 | 03:31 |
openstackgerrit | Merged stackforge/python-openstacksdk: Fix assert order in test_resource https://review.openstack.org/160467 | 04:27 |
openstackgerrit | Dean Troyer proposed openstack/python-openstackclient: Add --os-cloud support https://review.openstack.org/129795 | 04:37 |
dtroyer | mordred: ^^^^ first cut at a fresh update. see how many corner cases you can find ;) | 04:45 |
dtroyer | also help w/o auth should be a bit more sane now too | 04:46 |
openstackgerrit | Dean Troyer proposed openstack/python-openstackclient: Add --os-cloud support https://review.openstack.org/129795 | 04:49 |
openstackgerrit | Brian Curtin proposed stackforge/python-openstacksdk: Use update_attrs to handle updating servers https://review.openstack.org/160635 | 04:57 |
openstackgerrit | Brian Curtin proposed stackforge/python-openstacksdk: Add ServerDetail so list can include details https://review.openstack.org/160598 | 05:00 |
openstackgerrit | Brian Curtin proposed stackforge/python-openstacksdk: Set put_update for compute.v2.server.Server https://review.openstack.org/160620 | 05:00 |
openstackgerrit | Brian Curtin proposed stackforge/python-openstacksdk: Add example to do CRUD operations on servers https://review.openstack.org/160623 | 05:00 |
openstackgerrit | Brian Curtin proposed stackforge/python-openstacksdk: Set Flavor and Image resources on Server https://review.openstack.org/160618 | 05:00 |
openstackgerrit | Brian Curtin proposed stackforge/python-openstacksdk: Add example to do CRUD operations on servers https://review.openstack.org/160623 | 05:20 |
*** redrobot has quit IRC | 06:21 | |
*** redrobot has joined #openstack-sdks | 06:25 | |
*** redrobot is now known as Guest32544 | 06:25 | |
*** terrylhowe has quit IRC | 06:52 | |
*** rmcall has quit IRC | 06:58 | |
*** dstufft has quit IRC | 07:08 | |
*** dstufft has joined #openstack-sdks | 07:15 | |
*** openstackgerrit has quit IRC | 07:22 | |
*** openstackgerrit has joined #openstack-sdks | 07:22 | |
*** stevemar has quit IRC | 08:15 | |
*** sigmavirus24_awa has quit IRC | 08:26 | |
*** stevelle has quit IRC | 09:46 | |
*** stevelle has joined #openstack-sdks | 10:00 | |
*** tellesnobrega has quit IRC | 11:27 | |
*** terrylhowe has joined #openstack-sdks | 11:27 | |
*** tellesnobrega has joined #openstack-sdks | 11:33 | |
mordred | dtroyer: oo! exciting | 11:49 |
mordred | dtroyer: if we get that working, do you want to include os-client-config in your openstackclient project proposal? | 11:50 |
mordred | dtroyer: 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 params | 11:51 |
mordred | Shrews: ^^ check out dtroyer's latest patch | 12:15 |
mordred | Shrews: its almost like we could start doing everything from one set of config! | 12:16 |
mordred | Shrews: since dtroyer is using "auth_type" for plugin name, perhaps we should change the name of that param in ansible and shade too for consistency | 12:22 |
mordred | dtroyer: oh - you know - you may already be taking car of the auth dict in os-client-config ... | 12:24 |
mordred | ah - nope - it goes the other direction | 12:25 |
openstackgerrit | Monty Taylor proposed stackforge/os-client-config: Rename auth_plugin to auth_type https://review.openstack.org/160771 | 12:32 |
mordred | dtroyer, Shrews: ^^ there ya go - that should make that part slighly less suck | 12:32 |
dtroyer | mordred: 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 |
openstackgerrit | Julien Danjou proposed openstack/python-openstackclient: telemetry: add support for alarm creation https://review.openstack.org/155822 | 12:39 |
mordred | dtroyer: no worries - I think it's actually an easy fix | 12:39 |
mordred | dtroyer: because you keep an auth dict in osc yourself | 12:39 |
dtroyer | ksc'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 URLs | 12:40 |
mordred | dtroyer: I _think_ something like this: http://paste.openstack.org/show/186310 | 12:41 |
mordred | dtroyer: will at least get real close to consuming auth dict and also doing what you're already doing | 12:41 |
mordred | (untested) | 12:41 |
mordred | still 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_options | 12:42 |
mordred | dtroyer: oh - you know what - there's another thing we could do ... | 12:43 |
mordred | nah - let me poke at this for just a second longer before I go and get hacky :) | 12:46 |
mordred | dtroyer: 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 |
dtroyer | ok. 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 night | 12:47 |
mordred | dtroyer: I'm cafinating already - so I'll take a pass and see if I can get it to work for me :) | 12:48 |
dtroyer | cool | 12:48 |
openstackgerrit | Dean Troyer proposed openstack/python-openstackclient: Handle novaclient >2.20.0 https://review.openstack.org/160473 | 13:02 |
mordred | yes. please merge that one :) | 13:02 |
dtroyer | stevemar ^^^ stevemar ^^^ stevemar | 13:03 |
dtroyer | sometimes that wakes him up... | 13:03 |
mordred | dtroyer: do you ever have one of those moments where you're trying to figure out why something is erroring | 13:04 |
mordred | dtroyer: and you put a bunch of prints in | 13:04 |
mordred | and you get a print around each side of a thing that shouldn't be able to fail | 13:04 |
mordred | and yet it does? | 13:04 |
dtroyer | all. the. time. re: last nigh and the tests for this change… | 13:05 |
dtroyer | I chalk it up to my still-not-too-deep grokking of Pythonism | 13:05 |
mordred | dtroyer: ok - well, I'm almost there with this change and the auth dict | 13:06 |
mordred | but I need to change locations | 13:06 |
mordred | so it'll be probably 2 hours before I can have a follow up patch for you | 13:06 |
dtroyer | np, I have $DAY_JOB waiting for me too | 13:06 |
mordred | (I'm very motivated for this, btw, because it will help my day-to-day immensely) | 13:06 |
*** mattfarina has joined #openstack-sdks | 14:18 | |
Shrews | mordred: neato | 14:21 |
*** chlong has joined #openstack-sdks | 14:27 | |
openstackgerrit | Julien Danjou proposed stackforge/python-openstacksdk: identity: add missing tenant options to valid options https://review.openstack.org/160821 | 14:52 |
openstackgerrit | Julien Danjou proposed openstack/python-openstackclient: telemetry: add support for alarm creation https://review.openstack.org/155822 | 14:53 |
openstackgerrit | Brian Curtin proposed stackforge/python-openstacksdk: Add base for Proxy classes to inherit from https://review.openstack.org/160585 | 14:55 |
openstackgerrit | Brian Curtin proposed stackforge/python-openstacksdk: Add base for Proxy classes to inherit from https://review.openstack.org/160585 | 15:00 |
openstackgerrit | Julien Danjou proposed stackforge/python-openstacksdk: identity: add missing tenant options to valid options https://review.openstack.org/160821 | 15:21 |
openstackgerrit | Julien Danjou proposed stackforge/python-openstacksdk: identity: use set() for valid_options https://review.openstack.org/160840 | 15:26 |
mordred | dtroyer: got it working | 15:28 |
mordred | dtroyer: let me clean up my debugging | 15:28 |
openstackgerrit | Monty Taylor proposed openstack/python-openstackclient: Add --os-cloud support https://review.openstack.org/129795 | 15:35 |
mordred | dtroyer: that works for me - I haven't dived into the test suite yet | 15:35 |
*** sigmavirus24 has joined #openstack-sdks | 15:35 | |
mordred | dtroyer: it isn't doing image list for me - but I'll dig in to that next | 15:36 |
dtroyer | mordred: I think there are some stragglers to pick up in build_auth_params yet that do not come from the plugins | 15:44 |
mordred | dtroyer: nope - it was endpoint hacking | 15:45 |
openstackgerrit | Monty Taylor proposed openstack/python-openstackclient: Don't hack the image url https://review.openstack.org/160848 | 15:46 |
mordred | dtroyer: that fixes image list for me | 15:47 |
openstackgerrit | Merged stackforge/python-openstacksdk: Add base for Proxy classes to inherit from https://review.openstack.org/160585 | 15:48 |
*** pm90_ has joined #openstack-sdks | 15:49 | |
dtroyer | I'm having issues picking up values from the environment | 15:51 |
*** stevemar has joined #openstack-sdks | 15:53 | |
*** rmcall has joined #openstack-sdks | 16:02 | |
openstackgerrit | Julien Danjou proposed stackforge/python-openstacksdk: identity: use set() for valid_options https://review.openstack.org/160840 | 16:23 |
openstackgerrit | Julien Danjou proposed openstack/python-openstackclient: telemetry: add support for alarm creation https://review.openstack.org/155822 | 16:26 |
*** Guest32544 is now known as redrobot | 16:51 | |
openstackgerrit | Brian Curtin proposed stackforge/python-openstacksdk: Add example to do CRUD operations on servers https://review.openstack.org/160623 | 17:07 |
*** chlong has quit IRC | 17:17 | |
mordred | dtroyer: kk. I'll hit that next | 17:22 |
mordred | dtroyer: k. I've replicated the env vars problem | 17:34 |
*** tellesnobrega has quit IRC | 17:43 | |
openstackgerrit | Monty Taylor proposed openstack/python-openstackclient: Pass cloud by name https://review.openstack.org/160917 | 17:44 |
mordred | dtroyer: ^^ that works for me | 17:44 |
*** tellesnobrega has joined #openstack-sdks | 18:04 | |
*** trown has joined #openstack-sdks | 18:16 | |
briancurtin | python-openstacksdk meeting in #openstack-meeting-3 in 15' | 18:45 |
sigmavirus24 | briancurtin: i'll be late | 18:57 |
sigmavirus24 | apologies in advance | 18:57 |
briancurtin | you're fired | 18:57 |
sigmavirus24 | damnit | 18:57 |
* sigmavirus24 starts knocking on cloud providers doors | 18:57 | |
openstackgerrit | Ramaraja proposed openstack/python-openstackclient: Fix help messages https://review.openstack.org/160951 | 19:36 |
*** pm90_ has quit IRC | 19:52 | |
stevemar | sigmavirus24, quick knocking on doors | 19:54 |
sigmavirus24 | stevemar: more like "Let me in!!!" | 19:57 |
*** pm90_ has joined #openstack-sdks | 20:06 | |
*** pm90_ has quit IRC | 20:06 | |
*** pm90_ has joined #openstack-sdks | 20:07 | |
mordred | dtroyer: hey! I got it all completely working along with unittests | 20:28 |
dtroyer | mordred: awesome! I'm catching up with what I missed… | 20:28 |
mordred | dtroyer: I'm rebasing and squashing a couple of things | 20:28 |
dtroyer | ok… on the images version thing…, that change breaks images for me and default devstack | 20:29 |
mordred | dtroyer: awesome. so I'll need to figure out a richer fix there | 20:29 |
dtroyer | not that default devstack is correct | 20:29 |
dtroyer | really it just needs discovery | 20:29 |
mordred | dtroyer: well, the thing I removed was a hack in the first place - yeah | 20:30 |
mordred | btw - glance discovery is bonghits | 20:30 |
dtroyer | and I haven't doen that. that's why the assumption that setting OS_IMAGE_API_VERSION was good enough | 20:30 |
mordred | dtroyer: I think what happened with me is taht setting image-api-version still was broken | 20:30 |
dtroyer | but I'm not using that…you patched my stand-alone low level API ;) | 20:30 |
mordred | because the assumption on the url structure was wrong | 20:30 |
mordred | :) | 20:30 |
mordred | hehe | 20:30 |
dtroyer | ah, ok | 20:30 |
mordred | let me send you a thing without the image-api part | 20:31 |
dtroyer | yeah, I was going to split that out | 20:31 |
openstackgerrit | Monty Taylor proposed openstack/python-openstackclient: Don't hack the image url https://review.openstack.org/160848 | 20:33 |
openstackgerrit | Monty Taylor proposed openstack/python-openstackclient: Add --os-cloud support https://review.openstack.org/129795 | 20:33 |
mordred | dtroyer: ok - there's the stack new - I'll work on the image url thing now | 20:33 |
mordred | dtroyer: I'm _very_ curious as to why the openstackclient keystone.auth entrypoints don't show up in the test runs | 20:33 |
dtroyer | I 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 bit | 20:34 |
mordred | dtroyer: yes. it works for all my stuff except for image | 20:35 |
* mordred hands dtroyer the conch | 20:36 | |
mordred | dtroyer: btw - did you see my question from way earlier about including os-client-config in your new client project proposal? | 20:36 |
dtroyer | I 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 |
mordred | cool | 20:37 |
mordred | dtroyer: 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 |
dtroyer | the 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 |
mordred | dtroyer: 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 |
dtroyer | mordred: 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 urlss | 20:54 |
openstackgerrit | Monty Taylor proposed openstack/python-openstackclient: Don't hack the image url https://review.openstack.org/160848 | 21:06 |
mordred | dtroyer: there ya go - that should still work for you, and works for me on rax and hp | 21:06 |
openstackgerrit | Monty Taylor proposed stackforge/os-client-config: Handle project_name/tenant_name in the auth dict https://review.openstack.org/160979 | 21:13 |
mordred | Shrews: ^^ also, I noticed this | 21:13 |
Shrews | mordred: will check it out in a bit | 21:16 |
openstackgerrit | Christy Fernando proposed openstack/python-openstackclient: Changed user list help. https://review.openstack.org/160981 | 21:17 |
*** pm90_ has quit IRC | 21:28 | |
*** pm90_ has joined #openstack-sdks | 21:32 | |
*** pm90_ has joined #openstack-sdks | 21:33 | |
openstackgerrit | Monty Taylor proposed stackforge/os-client-config: Handle project_name/tenant_name in the auth dict https://review.openstack.org/160979 | 21:34 |
openstackgerrit | Monty Taylor proposed stackforge/os-client-config: Rename auth_plugin to auth_type https://review.openstack.org/160771 | 21:34 |
openstackgerrit | Monty Taylor proposed stackforge/os-client-config: Add two newlines to the ends of files https://review.openstack.org/160989 | 21:34 |
*** stevemar has quit IRC | 21:41 | |
*** mattfarina has quit IRC | 21:59 | |
openstackgerrit | Dean Troyer proposed openstack/python-openstackclient: Add --os-cloud support https://review.openstack.org/161006 | 22:27 |
openstackgerrit | Dean Troyer proposed openstack/python-openstackclient: Add --os-cloud support https://review.openstack.org/129795 | 22:28 |
dtroyer | mordred: ^^^ hopefully that passes functional tests and doesn't break you | 22:28 |
mordred | dtroyer: I shall try | 22:29 |
mordred | dtroyer: (that looks like two different changes? should I just do 129795?) | 22:30 |
mordred | dtroyer: seems to work for me | 22:33 |
*** rmcall has quit IRC | 22:36 | |
dtroyer | yeah, I fubar'ed the change id, ignore 161006 | 22:42 |
openstackgerrit | Dean Troyer proposed openstack/python-openstackclient: Don't hack the image url https://review.openstack.org/160848 | 22:42 |
openstackgerrit | Merged stackforge/python-openstacksdk: identity: add missing tenant options to valid options https://review.openstack.org/160821 | 22:42 |
openstackgerrit | Merged stackforge/python-openstacksdk: identity: use set() for valid_options https://review.openstack.org/160840 | 22:42 |
*** stevemar has joined #openstack-sdks | 22:44 | |
*** openstackgerrit has quit IRC | 22:51 | |
*** openstackgerrit has joined #openstack-sdks | 22:52 | |
*** pm90_ has quit IRC | 22:53 | |
*** pm90_ has joined #openstack-sdks | 23:06 | |
*** stevemar has quit IRC | 23:29 | |
*** chlong has joined #openstack-sdks | 23:31 | |
*** sigmavirus24 is now known as sigmavirus24_awa | 23:36 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!