Earlier  
Posted Nick Remark
#openstack-sdks - 2018-01-18
17:33:22 edleafe dtantsur: ack
18:33:45 mordred dtantsur|afk: left a note - I think we have a bikeshed we need to have about naming
18:39:18 openstackgerrit Merged openstack/cliff master: remove -s alias for --sort-columns https://review.openstack.org/534771
18:53:48 openstackgerrit Merged openstack/python-openstackclient master: Fix indentation in authentication.rst https://review.openstack.org/534848
19:06:24 mordred dtroyer: if you have a sec, would you mind weighing in on https://review.openstack.org/#/c/533264 and https://review.openstack.org/#/c/533265 ?
19:20:22 dtantsur|afk mordred: oh, yeah, it's going to be fun ;) will respond tomorrow
19:34:34 johnsom Is anyone looking at the neutron breakage in openstacksdk 0.10.0 that is blocking the GR update? https://review.openstack.org/#/c/534345/
19:34:52 johnsom We, octavia, really need that release in queens...
19:50:10 mordred johnsom: I am now :)
19:50:45 johnsom mordred Ok, thanks. I just heard a report of this being broken in the field too. Let me know if I can help in some way.
19:50:50 mordred wait a second ... there's a new openstacksdk release?
19:51:01 openstackgerrit Merged openstack-infra/shade master: Updated from global requirements https://review.openstack.org/525368
19:51:04 johnsom Evidently
19:51:18 mordred oh joy. that's ... unfortunate
19:52:06 johnsom Well, I was hoping for one to get that sdk fix in for octavia. We really need that fix for the queens release.
19:53:01 mordred well, yes - I think a 0.10 release is a great idea - but there's a few more patches I'd wanted to get landed before we cut one this week
19:53:05 johnsom I figured you knew, but I guess not: https://review.openstack.org/#/c/533624/
19:53:40 johnsom I just saw it as I've been watching for it due to the octavia fix
19:54:30 mordred well - such is life - I'll look in to the neutron failure for you
19:54:39 johnsom Ok, thanks
19:55:25 johnsom The person I heard also saw it was just trying to create a floating IP via OSC and got a failure.
19:58:45 mordred http://logs.openstack.org/45/534345/2/check/neutron-grenade/9831f38/logs/grenade.sh.txt.gz#_2018-01-17_18_04_33_423
19:58:48 mordred seems to be the error
20:01:32 mordred I can reproduce locally
20:09:23 dtroyer mordred: +2 on both, ready to +W them?
20:09:33 mordred dtroyer: yup!
20:09:43 dtroyer done
20:10:20 mordred dtroyer: didn't we land a patch in osc to try importing new things from sdk if they're there?
20:11:36 dtroyer Hmmm… I actually don't remember for sure, but it wouldn't have been global, just getting Connection set up with OSC's sessiion or something
20:12:03 mordred yah
20:12:13 mordred well - I'll look in to that *next*
20:12:14 dtroyer the semantics are different enough in most of the client libs that it'll be a bit more than s// to change over
20:12:29 mordred oh - I mean in the network plugin where we make one for Neutron
20:12:55 dtroyer ah, yes, that is where we would have done that. If it hasn't moved to clientManager it should though
20:13:23 dtroyer I have unfortunately let most of OSC leave L1 cache in my brain the last few weeks
20:18:49 mordred dtroyer: I hear you
20:18:52 mordred there's only so much cache space
20:18:57 mordred johnsom: found bug - patch coming
20:20:16 mordred dtroyer: I think we may want to add the neutron-grenade job to the openstacksdk gate - neutron-grenade caught an issue that the other sdk/osc shared jobs didn't
20:26:07 dtroyer mordred: is that something that we'll need to think about for the other APIs, and if it'll take N jobs to do N APIs?
20:26:23 mordred dtroyer: no - this was just a bug in the backwards-compat code around Profile
20:27:02 mordred dtroyer: it's also possible that adding neutron-grenade doesn't really gain us anything systemically and we just got lucky/unlucky this time
20:27:38 johnsom Awesome you found the bug
20:28:13 mordred johnsom: yah - it's a silly likttle thing - but it's also easy enough to trigger, so making sure it's all gone
20:35:23 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Make sure we use config defaults in profile compat code https://review.openstack.org/535476
20:35:47 mordred johnsom, dtroyer: ^^ that should fix the break in https://review.openstack.org/#/c/534345/
20:42:30 mordred Shrews: ^^ you too, if you get a chance
20:43:50 johnsom Shouldn't there be a unit test update here?
20:52:17 mordred johnsom: yup. almost certainly
20:52:19 mordred :)
20:53:22 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Invert the attribute mapping https://review.openstack.org/533822
21:06:03 openstackgerrit Merged openstack/python-openstacksdk master: Rationalize logging helpers and docs https://review.openstack.org/532634
21:08:36 openstackgerrit Merged openstack/python-openstacksdk master: Add ability to register non-official services https://review.openstack.org/533263
21:10:45 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Make sure we use config defaults in profile compat code https://review.openstack.org/535476
21:11:01 mordred johnsom, Shrews: ^^ k. now with a test. I verified that the test breaks without the patch
21:11:07 mordred (and breaks in the way that osc was breaking)
21:42:21 johnsom Nice
21:58:18 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Make sure we use config defaults in profile compat code https://review.openstack.org/535476
22:55:00 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Use version definition from openstack.version https://review.openstack.org/535336
22:55:01 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Move openstack_cloud helper functions https://review.openstack.org/535515
23:14:55 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Invert the attribute mapping https://review.openstack.org/533822
#openstack-sdks - 2018-01-19
01:53:12 openstackgerrit Merged openstack/keystoneauth master: Updated from global requirements https://review.openstack.org/534936
01:53:13 openstackgerrit Merged openstack/keystoneauth master: Shift additional_user_agent in the stack https://review.openstack.org/456259
02:34:22 openstackgerrit Merged openstack/python-openstacksdk master: Remove meter service https://review.openstack.org/533264
02:54:52 openstackgerrit Merged openstack/python-openstacksdk master: Remove metric service https://review.openstack.org/533265
05:29:27 openstackgerrit Merged openstack/keystoneauth master: Use stestr in tox.ini https://review.openstack.org/534763
05:36:45 openstackgerrit Merged openstack/python-openstacksdk master: Add deprecation warnings to profile https://review.openstack.org/534837
07:15:40 openstackgerrit chenyb4 proposed openstack/python-openstacksdk master: Add clustering guides receiver file,examples receiver code https://review.openstack.org/535640
08:21:50 openstackgerrit Chen Hanxiao proposed openstack/python-openstackclient master: Add --image-with parameter in 'server create' https://review.openstack.org/535664
09:09:18 openstackgerrit Hengqing Hu proposed openstack/python-openstacksdk master: Handle resource deletion properly https://review.openstack.org/535679
10:19:15 openstackgerrit David Rabel proposed openstack/python-openstackclient master: Corrected spelling mistake https://review.openstack.org/535710
11:16:54 openstackgerrit Merged openstack/python-openstacksdk master: Remove message v1 support https://review.openstack.org/533322
11:38:31 openstackgerrit Yandy Sun proposed openstack/osc-lib master: Replace 'assertFalse(a in b)' with 'assertNotIn(a, b)' https://review.openstack.org/535733
12:50:00 openstackgerrit OpenStack Proposal Bot proposed openstack/python-openstackclient master: Updated from global requirements https://review.openstack.org/535769
15:05:18 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Invert the attribute mapping https://review.openstack.org/533822
16:08:39 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Fix typo in the external service loader code https://review.openstack.org/534452
16:08:39 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Fixed a few nits in the README https://review.openstack.org/534453
16:08:40 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Clean up a bit after the mapping inversion https://review.openstack.org/533823
16:08:40 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Update type conversion to handle subtypes more better https://review.openstack.org/533824
16:08:41 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Remove resource and proxy https://review.openstack.org/533825
16:08:41 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Rename resource2 and proxy2 to resource and proxy https://review.openstack.org/533826
16:08:42 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Remove deprecated wait_for methods from Proxy https://review.openstack.org/534832
16:40:23 openstackgerrit Merged openstack/python-openstacksdk master: Make sure we use config defaults in profile compat code https://review.openstack.org/535476
17:08:32 openstackgerrit Merged openstack/python-openstacksdk master: Add clustering guides receiver file,examples receiver code https://review.openstack.org/535640
17:20:47 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Rename resource2 and proxy2 to resource and proxy https://review.openstack.org/533826
17:20:47 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Remove deprecated wait_for methods from Proxy https://review.openstack.org/534832
17:20:48 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Add resource2/proxy2 wrappers with deprecations https://review.openstack.org/535860
17:32:07 openstackgerrit Merged openstack/python-openstacksdk master: Migrate object_store to resource2/proxy2 https://review.openstack.org/533323
17:33:43 mordred adriant: ^^
17:48:08 openstackgerrit Lance Bragstad proposed openstack/keystoneauth master: Add documentation and release note for app creds https://review.openstack.org/535867
18:15:31 openstackgerrit Merged openstack/python-openstackclient master: Corrected spelling mistake https://review.openstack.org/535710
18:37:51 openstackgerrit Merged openstack/keystoneauth master: Implement system scope https://review.openstack.org/529665
19:59:34 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Clean up a bit after the mapping inversion https://review.openstack.org/533823
20:02:14 mnaser mordred: is there any historic interesting story behind why the shade data model has "created_at" for most objects, but nova is "created" ?
20:03:02 mnaser (do we want to look into making both available, because it can make doing operations that involve looping over all objects and checking their created_at field easier)
20:03:26 mordred mnaser: yes - that's a bug and it should be created_at
20:03:48 mnaser i'll file a bug and hack on that, is that ok?
20:03:55 mordred mnaser: shade/_normalize.py _normalize_server ... we should update it to add created_at - and to not have created when strict=True
20:03:58 mordred mnaser: please do!
20:04:11 mnaser ok thanks for that hint, i was looking deep in openstackcloud.py

Earlier   Later