| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2017-03-29 | |||
| 22:28:52 | dtroyer | I need to make version decisions in commands before I get that far | |
| 22:28:57 | jamielennox | something that did major and microversion discovery - and probably capabilities on the same request | |
| 22:29:00 | dtroyer | oooh, that would be helpful at some point :) | |
| 22:29:37 | jamielennox | ya, if we just put it on a new content-type and say ksa works if you implement this i think we might actually get some traction as opposed to last time i did it | |
| 22:33:05 | jamielennox | dtroyer: anyway i can wait and make sure to put everything we actually need in there first | |
| 22:33:27 | jamielennox | dtroyer: so on another front did you have a look at the os-c-c-2 WIP i put up | |
| 22:33:30 | jamielennox | ? | |
| 22:33:37 | jamielennox | i cannot figure out what i want that interface to look like | |
| 22:33:50 | dtroyer | I did last week and wasn't exactly sure I understood it all | |
| 22:34:19 | jamielennox | yea, i didn't make it easy, it was all experimenting and no comments | |
| 22:35:21 | dtroyer | I am ok with the current interface, basically decomposing get_one_cloud() so I can insert things and leave out other things is what I'm looking for | |
| 22:35:47 | dtroyer | that auth plugin loading bit me again this week | |
| 22:36:12 | dtroyer | it happens waaaaaay to early in o-c-c | |
| 22:37:05 | jamielennox | dtroyer: agreed, i'm not sure i want people overriding it | |
| 22:37:12 | jamielennox | my thought was it should be simply a datastore | |
| 22:37:29 | jamielennox | but i got way into the weeds | |
| 22:39:19 | dtroyer | that would be my preference | |
| 22:40:25 | dtroyer | I would love to hand it an argparse Namespace, have it merge that with clouds.*, do all of the hacky things to make interop work and be done | |
| 22:41:49 | jamielennox | that's part of where i was getting caught up, does o-c-c control the parser, what about envs? | |
| 22:43:28 | jamielennox | how do you handle the AllClouds object vs the get_one_cloud case | |
| 22:43:49 | dtroyer | OSC does the parser and env (via argparse) | |
| 22:43:55 | dtroyer | I don't look at AllClouds at all | |
| 22:44:06 | jamielennox | yea, but o-c-c has to maintain that | |
| 22:44:10 | dtroyer | altough commands to work with clouds.yaml might | |
| 22:44:29 | dtroyer | right, that's fine | |
| 22:45:15 | dtroyer | I would be OK with there being a common set of argparse declaration in o-c-c, but I want control over it | |
| 22:46:52 | dtroyer | working with this again Monday I realized just how f-ed up OSC's use of the auth plugins is. IS it work fixing that before this rewrite is done? do those go together? | |
| 22:51:05 | jamielennox | dtroyer: i've attempted to mess with osc's auth plugins before and can never find a way to exactly maintain compatibility | |
| 22:51:14 | jamielennox | but i don't think you should wait for o-c-c | |
| 22:51:41 | dtroyer | we can't avoid breaking some things. I'm preparing for a 4.0 where we can make some changes there | |
| 22:52:36 | dtroyer | since _everything else_ is doing some things differently than OSC, maybe its time OSC adjusts a bit | |
| 22:53:28 | jamielennox | would be nice to have the o-c-c fixed up for then | |
| 22:53:33 | jamielennox | not sure how likely that is though | |
| 22:54:09 | dtroyer | yeah, I expected some things to change there. I don't knwo what the timing of 4.0 is, June/July at the absolute earliest | |
| 22:56:32 | dtroyer | one other thing that's bugging me: you mentioned a long time ago that I should be making requests adapters rather than sessions. | |
| 22:57:03 | dtroyer | I'm thinking I need to fix that too | |
| 23:04:40 | jamielennox | dtroyer: so adapters were a direct response to how you create a client once and then reuse it with the same parameters | |
| 23:05:16 | jamielennox | it meant you could do things like set the service_type on an object and make calls via that rather than set it every time on a session.request | |
| 23:05:51 | jamielennox | it's _probably_ easier give that you do like a create_client thing that returns something specific to that usage | |
| 23:06:07 | jamielennox | but if you know what you're doing with a session you may not need it | |
| 23:06:46 | jamielennox | it really depends on how you want to use it | |
| 23:07:30 | dtroyer | ok. I was looking at shade and wanting to converge some of the lower-level stuff a bit more. I've got to implement some nova-net stuff that's being removed from novaclient so I was looknig again at OSC' api layer stuff | |
| #openstack-sdks - 2017-03-30 | |||
| 00:31:57 | openstackgerrit | OpenStack Proposal Bot proposed openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/451606 | |
| 01:52:06 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: start using api.versions https://review.openstack.org/451618 | |
| 01:52:43 | dtroyer | jamielennox: ^^^ is a very quick attempt at microversion handling in osc. openstackclient.api.versions is the bit that I'm thinking about for ksa | |
| 02:28:32 | reedip | stevemar ; there ? | |
| 04:18:39 | reedip | RuiChen : there? | |
| 04:40:15 | openstackgerrit | Merged openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/451606 | |
| 06:22:17 | openstackgerrit | Thomas Bechtold proposed openstack/python-openstacksdk master: Fix doc build if git is absent https://review.openstack.org/451662 | |
| 06:42:22 | reedip | RuiChen : ping | |
| 08:36:27 | rabel | good morning! | |
| 08:36:48 | reedip | its evening ! :D | |
| 08:37:38 | rabel | :D good evening, reedip | |
| 10:05:07 | openstackgerrit | Thomas Bechtold proposed openstack/python-openstacksdk master: Fix doc build if git is absent https://review.openstack.org/451662 | |
| 12:15:58 | openstackgerrit | Merged openstack/python-openstacksdk master: Fix doc build if git is absent https://review.openstack.org/451662 | |
| 13:01:41 | RuiChen | we have IRC meeting now? | |
| 13:01:52 | reedip | we do ? | |
| 13:03:22 | RuiChen | I'm not sure, but today is thursday | |
| 13:05:04 | dtroyer | it's later today, 1900 UTC | |
| 13:05:21 | reedip | whats the time now ?? | |
| 13:05:25 | reedip | :D | |
| 13:06:10 | dtroyer | 1306 UTC | |
| 13:06:21 | dtroyer | so six hours from now | |
| 13:06:48 | RuiChen | I always fail to identify odd and even weeks :( | |
| 13:06:54 | reedip | thats better thanks :) | |
| 13:07:41 | dtroyer | maybe this can help: http://eavesdrop.openstack.org/calendars/oslo-team-meeting.ics | |
| 13:07:47 | reedip | :D | |
| 13:07:54 | dtroyer | http://eavesdrop.openstack.org/calendars/openstackclient-team-meeting.ics is the right one | |
| 13:08:24 | RuiChen | ah, thank you dtroyer :) | |
| 13:10:26 | RuiChen | today IRC meeting is too late for me, I will join next time | |
| 13:11:29 | dtroyer | RuiChen: did you see my last comemnt on https://review.openstack.org/#/c/450452/? If you have no objection I'll merge it today | |
| 13:11:55 | dtroyer | the OSC companion to that is https://review.openstack.org/#/c/450453/ | |
| 13:12:30 | dtroyer | I'm doing both so we don't have to wait on the osc-lib release cycle to fix the problem | |
| 13:12:50 | RuiChen | dtroyer: not yet, I will review it now | |
| 13:20:04 | RuiChen | dtroyer: I saw your comments, but the current patch will break network help command in no auth case, I face it in my devstack | |
| 13:32:17 | dtroyer | do you have both of them in place? | |
| 13:32:56 | RuiChen | yes, I do | |
| 13:33:11 | RuiChen | pip install -e osc-lib and osc | |
| 13:33:33 | dtroyer | ok, I'll get back to it then, I'm in the middle of microversion stuff ATM and can't switch back mentally that quickly | |
| 13:33:35 | dtroyer | thanks | |
| 13:35:04 | RuiChen | np, would like to see microversion land in osc :) | |
| 13:36:42 | dtroyer | RuiChen: https://review.openstack.org/451618 is the start if you're interested. I'm working on the set of stuff to put into ksa for support | |
| 13:39:12 | RuiChen | dtroyer: yeah, I push it in my review queue, I have a nova keypair related commands depends on osc microversion, thank you | |
| 13:47:17 | dtroyer | RuiChen: one more thought on the help/auth issue, try it with master os-client-config. That's why I need to do this, and if it doesn't work on release of o-c-c that's my problem | |
| 13:47:41 | dtroyer | this is attempting to make out functional-tips jobs pass | |
| 13:51:59 | RuiChen | hmmm... I used the latest o-c-c, but not master, I can try it in tommrow, I'm ok to merge that patch first | |
| 13:52:17 | dtroyer | ok, thanks | |
| 13:53:33 | RuiChen | np, bye, I should go | |
| 16:10:51 | openstackgerrit | Ankur proposed openstack/python-openstackclient master: Trivial Fix https://review.openstack.org/451890 | |
| 16:12:13 | dtroyer | ankur-gupta-f1: ^^^ Please bit something slightly more informational in the first line so you can know what it is from the gerrit queue screen | |
| 16:12:46 | dtroyer | no worries this time | |
| 16:13:10 | ankur-gupta-f4 | sorry. Was just a quick thing. It was bothering me so threw it up quickly | |
| 16:15:49 | ankur-gupta-f4 | dtroyer: any reviews needed for OSC? | |
| 16:15:51 | dtroyer | yeah, I know how that is. | |
| 16:16:52 | dtroyer | have you weighed in on the osc-side auth thing yet? https://review.openstack.org/#/c/450453/? | |
| 16:17:52 | ankur-gupta-f4 | not yet. will pull both the auth patches down. | |
| 16:18:02 | ankur-gupta-f4 | no better way to ruin a fresh env | |
| 16:18:07 | dtroyer | RuiChen was reporting some problems with it and current release os-client-config. I'm deep in microvarsionland and don't want to stop right now. It is meant to fix master so master o-c-c should also be checked, which is what I had done. I think it's the combination master o-c-c and osc-lib with release osc that is broken | |
| 16:19:39 | ankur-gupta-f4 | oh good. And yesterday the global req for sdk was bumped | |
| 16:19:41 | dtroyer | thanks. ah, I see, yes, the new functional-tips job in osc-lib fails, so that's the bit. | |