| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-sdks - 2017-04-27 | |||
| 21:30:41 | mordred | efried: ++ | |
| 21:30:55 | mordred | yah. I think the register_adapter_options in ksa is key | |
| 21:31:11 | dtroyer | ksa is a good example of us actually pulling this off. it'll take that level of discipline for dependencies and interfaces | |
| 21:31:17 | mordred | and then using session.get_endpoint | |
| 21:31:37 | efried | mordred With the exception of the "guess service_types for your group if not specified" thing, I have everything I need with the ksa patch. | |
| 21:31:45 | efried | That part is still the hole. | |
| 21:31:54 | mordred | because there's a few of the logic things nova needs that are in occ now but I think we may be close to being able to say that ksa should understand | |
| 21:32:11 | efried | I guess, back to baby steps, I can make service_type required for now...? | |
| 21:32:33 | efried | I don't think that'll make jaypipes happy, tho | |
| 21:32:34 | mordred | oh - yes - service_type is/should be required | |
| 21:32:38 | mordred | oh - wait | |
| 21:32:43 | mordred | hang on - context switching | |
| 21:32:51 | efried | ...in the conf | |
| 21:32:58 | mordred | I think we can put the thing you're talking about in like you have it now in the nova patches | |
| 21:33:16 | efried | okay, just a hardcoded dict of group: service_types in nova.utils? | |
| 21:33:17 | mordred | I _eventually_ think we should move that little bit to ksa - but we need a broader buy-in on that | |
| 21:33:19 | mordred | yah | |
| 21:33:23 | efried | Okay, cool. | |
| 21:33:35 | efried | So then the remaining gap is fleshing that guy out. | |
| 21:33:44 | mordred | it's basically a duplicate of code we have in occ too - but it's very small code - so we can improve the duplication over time | |
| 21:34:13 | efried | I don't begin to know what the comprehensive mapping should be there. Also didn't see it (except for some funky cinder stuff) in occ. Can you point me to it? | |
| 21:34:29 | mordred | that's pretty much it | |
| 21:34:36 | mordred | it's the funky cinder stuff | |
| 21:34:47 | mordred | there's also a mapping for mistral - but you don't need to worry about that one | |
| 21:34:49 | efried | (I gather than "funky cinder stuff" is often redundant) | |
| 21:34:57 | mordred | all the rest of them have one and only one service-type | |
| 21:35:11 | mordred | dtroyer: you saw they're talking about a new service-typefor cinder right? | |
| 21:35:20 | efried | would that be block-storage? | |
| 21:35:22 | mordred | yah | |
| 21:35:52 | dtroyer | mordred: I recall that from long ago, didn't know it was active again. | |
| 21:36:02 | dtroyer | I'm hoping it'll never gain a version? | |
| 21:36:31 | efried | dtroyer https://review.openstack.org/#/c/436178/ | |
| 21:36:52 | efried | Active today | |
| 21:44:16 | mordred | dtroyer: I did make a new patch to the consume-discovery patch covering the sad pile of multiple types | |
| 21:44:55 | mordred | dtroyer: https://review.openstack.org/#/c/460654/ | |
| 22:06:30 | jamielennox | mordred, efried: sorry was in a meeting, yea, it's probably too much to make every client subclass something from keystoneauth | |
| 22:06:41 | jamielennox | though the idea of a base useful client has come up a few times before | |
| 22:07:12 | jamielennox | mostly i was just trying to say i'm fine to have the pure adapter loading stuff because i know mordred wants that for os-c-c and shade | |
| 22:07:51 | jamielennox | but for most uses we just need a way for novaclient to say that it passes all these options through from Client.__init__ to keystoneauth | |
| 22:19:55 | mordred | jamielennox: yup - totally agree | |
| 22:20:03 | mordred | jamielennox: an I _think_ that's the intent here | |
| 22:28:12 | mordred | jamielennox: btw - you are probably one of the only other people on planet who can actually review this: https://review.openstack.org/#/c/459405/ for accuracy or intent | |
| 22:28:44 | mordred | jamielennox: so I'd love your feedback - although for the love of all that is holy don't start reading it until you're in a frame of mind that wants to bepunched in the face repeatedly for quite a while | |
| 22:29:02 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Nova net functional tests round 1 https://review.openstack.org/460356 | |
| 22:29:09 | jamielennox | mordred: ok, that's not in my current pre-coffee state | |
| 22:29:13 | mordred | nope | |
| 22:29:17 | mordred | not even close :) | |
| 22:31:47 | mordred | jamielennox: specific things of concern are that it doesn't describe user-visible behavior changes from that ksa is doing now, and that if we added support to ksa for the additional things described we could do so without introducing behavior changes (although might possibly introduce an additional api call somewhere that the user wouldn't notice) | |
| 23:41:14 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Nova net functional tests round 1 https://review.openstack.org/460356 | |
| 23:41:15 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Nova net functional tests round 2 https://review.openstack.org/460777 | |
| #openstack-sdks - 2017-04-28 | |||
| 11:57:41 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Nova net functional tests round 1 https://review.openstack.org/460356 | |
| 11:57:42 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Nova net functional tests round 2 https://review.openstack.org/460777 | |
| 14:40:12 | EmilienM | dtroyer, stevemar: hey ! https://review.openstack.org/#/c/459660/ broke TripleO CI | |
| 14:40:23 | EmilienM | (as you can see in the job results) | |
| 14:40:29 | EmilienM | https://bugs.launchpad.net/tripleo/+bug/1687032 | |
| 14:40:29 | openstack | Launchpad bug 1687032 in tripleo "Exception raised: 'ClientManager' object has no attribute 'auth'" [Critical,Triaged] | |
| 14:44:28 | dtroyer | EmilienM: I'm not familiar with tripleo jobs & logs, can you point me to the error? I haven't found it yet… | |
| 14:44:50 | EmilienM | dtroyer: hey, so the error is visible here: http://logs.openstack.org/15/359215/94/check-tripleo/gate-tripleo-ci-centos-7-ovb-nonha/38a2082/console.html#_2017-04-28_12_17_38_245239 | |
| 14:45:24 | EmilienM | dtroyer: but I need to enable --debug to the tripleoclient to know exactly where it failed in tripleo (heatclient? mistralclient? osc?) - I'm on it now | |
| 14:45:28 | dtroyer | that doesn't really tell me anything… what command is being executed? | |
| 14:45:30 | EmilienM | I just found useful to share this feedback | |
| 14:45:33 | dtroyer | ok | |
| 14:45:40 | EmilienM | yeah I don't have much clue now, i'm working on it | |
| 14:46:07 | dtroyer | sure, thanks. to he honest I had not even looked at this job yet, I'm still chasing the nova-net breakage from novaclient 8.0.0 | |
| 14:46:54 | EmilienM | dtroyer: i'm debugging with https://review.openstack.org/#/c/457037/ | |
| 14:47:00 | EmilienM | I'll post results here later | |
| 14:47:10 | dtroyer | thanks | |
| 14:47:15 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Nova net functional tests round 2 https://review.openstack.org/460777 | |
| 14:47:36 | EmilienM | dtroyer: i'm going to propose a revert just to see if it pass tripleo jobs, so I can confirm this is the patch. Don't merge it and ignore it please. | |
| 14:47:52 | dtroyer | ok | |
| 14:48:10 | openstackgerrit | Emilien Macchi proposed openstack/osc-lib master: DNM - Revert "Tell ClientManager when auth is required" https://review.openstack.org/461069 | |
| 17:41:14 | openstackgerrit | Merged openstack/keystoneauth master: Uncomment warning-is-error for doc building https://review.openstack.org/459673 | |
| 17:41:32 | openstackgerrit | Merged openstack/python-openstackclient master: Nova net functional tests round 1 https://review.openstack.org/460356 | |
| 17:42:13 | openstackgerrit | Merged openstack/python-openstackclient master: Nova net functional tests round 2 https://review.openstack.org/460777 | |
| 17:45:47 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Nova net functional tests round 3 https://review.openstack.org/461113 | |
| 17:47:53 | openstackgerrit | Monty Taylor proposed openstack/os-client-config master: Fix interactions with keystoneauth from newton https://review.openstack.org/461114 | |
| 17:53:19 | mordred | dtroyer: tiny little regression in our latest release. I'm 100% certain the only thing it's affecting is shade's non-voting legacy functional tips job which tests shade + tip of library branch against a stable/newton cloud | |
| 17:53:39 | mordred | dtroyer: it's such an absurd combination I was almost tempted to just work around it in shade's test definition | |
| 17:53:55 | dtroyer | if its a bug lets fix it | |
| 17:53:56 | mordred | but the actual fix in occ is super easy (and honestly less work than working around it) | |
| 18:08:38 | mordred | dtroyer: oh - I may not have put the right little carets in my earlier sentence... https://review.openstack.org/#/c/461114/ fixes the issue :) | |
| 18:13:28 | dtroyer | cool, thanks. will you need a 1.27.1 release for just that or can it wait a bit? | |
| 18:25:34 | mordred | dtroyer: I don't think it's super urgen - I've got a red non-voting job in shade - but I honestly don't think anyone else is going to hit this one | |
| 18:26:40 | dtroyer | ok, I'll hold off then in case something from bumping g-r shows up next week | |
| 19:18:03 | openstackgerrit | Merged openstack/python-openstackclient master: Fix Nova-net netowrk commands https://review.openstack.org/460679 | |
| 19:40:58 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Nova net functional tests round 3 https://review.openstack.org/461113 | |
| 19:43:21 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: DNM: Enable debug in functional tests https://review.openstack.org/458579 | |
| 20:07:15 | mordred | dtroyer: this time with tests working :) | |
| 20:07:41 | mordred | Shrews: ^^ so - legacy-functional-tips is definitely not doing what we think it's doing | |
| 20:07:59 | mordred | Shrews: but I do not yet have an answer as to wh | |
| 21:02:50 | openstackgerrit | Monty Taylor proposed openstack/service-types-authority master: Add explicit mention of versions https://review.openstack.org/460540 | |
| 21:02:51 | openstackgerrit | Monty Taylor proposed openstack/service-types-authority master: Add an attribute for aliases https://review.openstack.org/460539 | |
| 21:02:51 | openstackgerrit | Monty Taylor proposed openstack/service-types-authority master: Point keystone at overall API docs https://review.openstack.org/461152 | |
| 21:11:30 | openstackgerrit | Dean Troyer proposed openstack/python-openstackclient master: Fix volume qos spec list https://review.openstack.org/461153 | |
| 21:35:46 | openstackgerrit | Monty Taylor proposed openstack/os-client-config master: Fix interactions with keystoneauth from newton https://review.openstack.org/461114 | |
| 23:12:36 | mordred | dtroyer: ^^ yay passes tests this time. (the red test for shade legacy libs nv is expected - tracking that down separately | |
| #openstack-sdks - 2017-04-29 | |||
| 03:25:55 | openstackgerrit | Merged openstack/python-openstackclient master: Fix volume qos spec list https://review.openstack.org/461153 | |
| 03:44:49 | openstackgerrit | Merged openstack/python-openstackclient master: Nova net functional tests round 3 https://review.openstack.org/461113 | |
| 08:19:41 | openstackgerrit | Akihiro Motoki proposed openstack/python-openstacksdk master: Network tag support https://review.openstack.org/461194 | |
| 08:26:51 | openstackgerrit | Akihiro Motoki proposed openstack/python-openstackclient master: Network tag support https://review.openstack.org/461195 | |