Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-01-18
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: Fixed a few nits in the README https://review.openstack.org/534453
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:40 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Update type conversion to handle subtypes more better https://review.openstack.org/533824
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:41 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Rename resource2 and proxy2 to resource and proxy https://review.openstack.org/533826
16:08:41 openstackgerrit Monty Taylor proposed openstack/python-openstacksdk master: Remove resource and proxy https://review.openstack.org/533825
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: Remove deprecated wait_for methods from Proxy https://review.openstack.org/534832
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: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
20:04:28 mordred mnaser: oh - there's an issue with the os_keystone_endpoint module that I noticed while working on a different patch
20:05:14 mordred mnaser: it has a parameter "interface" (for setting the interface) - but unfortunately there is also a base parameter "interface" that is there for setting the interface you want to use to connect to keystone
20:05:54 mnaser mordred: thats weird, i've actually been using it for a little while and it hasn't broken (and it was creating public/internal/admin)
20:06:12 mnaser i wonder if it was because i used a with_items: ['admin', 'internal', 'public'] and maybe that does something weird
20:07:07 mordred mnaser: hrm. maybe it does ... or maybe you're just lucky becuase you're using it in a place where you happen tobe able to also connect to th eadmin, internal and public endpoints for keystone
20:07:23 mordred mnaser: so the fact that the with_items is setting interface to different values doesn't break you
20:07:44 mnaser oh i see, yes, they're pretty much all the same exact value
20:08:02 mnaser (yet to find a cloud which runs actual 3 different interfaces but lets not get into that discussion now)
20:44:16 openstackgerrit Hongbin Lu proposed openstack/python-openstackclient master: Support filtering port with IP address substring https://review.openstack.org/532324
20:53:11 openstackgerrit Ihar Hrachyshka proposed openstack/python-openstacksdk master: DNM testing whether lib/neutron switch breaks this repo https://review.openstack.org/535943
20:59:42 openstackgerrit Ihar Hrachyshka proposed openstack/python-openstackclient master: DNM testing whether lib/neutron switch breaks this repo https://review.openstack.org/535953
22:18:56 openstackgerrit Lance Bragstad proposed openstack/keystoneauth master: Add documentation and release note for app creds https://review.openstack.org/535867
22:29:47 mnaser mordred: ehhh TIL nova uses "created" in the API (but if you wanted to filter by created time, you use created_at in the query)
22:30:21 mnaser so *technically* shade is giving the right info, do we still want to return created_at to maintain consistency across services?
22:45:46 openstackgerrit Merged openstack/keystoneauth master: Add support for application credentials https://review.openstack.org/534455
23:15:40 mordred mnaser: yah - we care more about internal consistency in shade than in being consistent with whatever the rest api field might be
#openstack-sdks - 2018-01-20
03:34:30 openstackgerrit Mohammed Naser proposed openstack-infra/shade master: Switch to providing created_at field for servers https://review.openstack.org/536003
04:29:29 openstackgerrit chenyb4 proposed openstack/python-openstacksdk master: Add clustering guides file, Examples code. https://review.openstack.org/536008
04:38:28 openstackgerrit chenyb4 proposed openstack/python-openstacksdk master: Replace clustering examples code cluster to clustering https://review.openstack.org/536009

Earlier   Later