Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2017-04-27
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 openstack Launchpad bug 1687032 in tripleo "Exception raised: 'ClientManager' object has no attribute 'auth'" [Critical,Triaged]
14:40:29 EmilienM https://bugs.launchpad.net/tripleo/+bug/1687032
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: Point keystone at overall API docs https://review.openstack.org/461152
21:02:51 openstackgerrit Monty Taylor proposed openstack/service-types-authority master: Add an attribute for aliases https://review.openstack.org/460539
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
10:28:30 openstackgerrit Akihiro Motoki proposed openstack/python-openstackclient master: Network tag support https://review.openstack.org/461195
11:20:03 openstackgerrit Merged openstack/service-types-authority master: Add load-balancer service type https://review.openstack.org/434999
14:05:19 openstackgerrit Monty Taylor proposed openstack/service-types-authority master: Add validation and publication infrastructure https://review.openstack.org/461209
14:08:21 openstackgerrit Monty Taylor proposed openstack/service-types-authority master: Point keystone at overall API docs https://review.openstack.org/461152
14:08:22 openstackgerrit Monty Taylor proposed openstack/service-types-authority master: Add explicit mention of versions https://review.openstack.org/460540
14:08:22 openstackgerrit Monty Taylor proposed openstack/service-types-authority master: Add validation and publication infrastructure https://review.openstack.org/461209
14:08:23 openstackgerrit Monty Taylor proposed openstack/service-types-authority master: Add an attribute for aliases https://review.openstack.org/460539
14:40:07 openstackgerrit Akihiro Motoki proposed openstack/python-openstacksdk master: DNM: test functional test using OS_CLOUD=devstack-admin https://review.openstack.org/461211
15:42:15 openstackgerrit Akihiro Motoki proposed openstack/python-openstacksdk master: functional tests: minor cleanup https://review.openstack.org/461211
15:45:04 openstackgerrit Monty Taylor proposed openstack/service-types-authority master: Add the rest of the services we know about https://review.openstack.org/461221
15:45:04 openstackgerrit Monty Taylor proposed openstack/service-types-authority master: Add validation and publication infrastructure https://review.openstack.org/461209
15:52:28 openstackgerrit Monty Taylor proposed openstack/service-types-authority master: Add the rest of the services we know about https://review.openstack.org/461221
16:05:25 openstackgerrit Monty Taylor proposed openstack/service-types-authority master: Add the rest of the services we know about https://review.openstack.org/461221
16:28:23 openstackgerrit Akihiro Motoki proposed openstack/python-openstackclient master: Network L3 Router Commands for OSC https://review.openstack.org/385729
16:29:18 openstackgerrit Akihiro Motoki proposed openstack/python-openstacksdk master: Use constraints environment for testing https://review.openstack.org/461225
17:00:50 openstackgerrit Akihiro Motoki proposed openstack/python-openstacksdk master: Network tag support https://review.openstack.org/461194
17:09:13 openstackgerrit Akihiro Motoki proposed openstack/python-openstacksdk master: Network tag support https://review.openstack.org/461194
17:17:24 openstackgerrit Akihiro Motoki proposed openstack/python-openstacksdk master: Network tag support https://review.openstack.org/461194
17:33:06 openstackgerrit Akihiro Motoki proposed openstack/python-openstacksdk master: Fix document warnings https://review.openstack.org/461228
17:35:01 openstackgerrit Akihiro Motoki proposed openstack/python-openstacksdk master: Network tag support https://review.openstack.org/461194
18:47:16 EmilienM dtroyer: I was afk yesterday afternoon - I have more logs now: http://logs.openstack.org/37/457037/4/check-tripleo/gate-tripleo-ci-centos-7-ovb-nonha/5bf6daf/console.html#_2017-04-28_17_01_45_634230
18:47:20 EmilienM it's in tripleoclient, i'm investigating
#openstack-sdks - 2017-05-01
09:23:58 openstackgerrit Robert van Leeuwen proposed openstack/python-openstackclient master: Fixes osc port set allowed-address for ports without previous set allowed address https://review.openstack.org/461354
09:58:09 leeuwenrjj Hi, I just hit a bug and created a fix but I have a short question on the unit test since that did not catch the issue
09:59:05 leeuwenrjj The unit test for the network/v2/fakes.py for the port creates empty dictionaries e.g. for allowed_address_pairs
10:00:10 leeuwenrjj Because those empty values are not created in the real world some of the unit tests will work but in the real world they will break. See my change: https://review.openstack.org/#/c/461354/
10:00:51 leeuwenrjj So my question: should those unit tests be modified to not include empty dictionaries?
11:32:31 openstackgerrit Robert van Leeuwen proposed openstack/python-openstackclient master: Fixes osc port set allowed-address for ports without previous set allowed address https://review.openstack.org/461364
12:08:01 EmilienM dtroyer: hey, just fyi about the issue we had last week: https://review.openstack.org/#/c/461336

Earlier   Later