Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-12-18
16:40:48 mriedem sdk question for a tempest change https://review.openstack.org/#/c/622316/3/tempest/config.py@347
16:40:57 mriedem about using the legacy keystone v2 *URL endpoint types
16:41:03 mriedem e.g. publicURL
16:52:56 mordred mriedem: looking
16:56:31 mordred mriedem: left comment - tl;dr I agree with you
16:59:16 mriedem THANKS
16:59:24 mriedem oops, slip o the caps
19:56:37 mordred slaweq: https://review.openstack.org/#/c/625920/ is neutron related - look ok to you?
20:22:24 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Use munch for server security groups https://review.openstack.org/625923
20:50:14 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Don't try to upload to images in saving or queued state https://review.openstack.org/624188
20:51:30 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Import code for building ironic-compatible configdrives https://review.openstack.org/624441
20:52:13 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Drop self.conn from base.TestCase https://review.openstack.org/625115
20:52:27 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Add network segment range resource https://review.openstack.org/625381
21:02:46 openstackgerrit Merged openstack/openstacksdk master: Change openstack-dev to openstack-discuss https://review.openstack.org/625247
21:10:39 slaweq mordred: yes, +W'ed already
21:10:45 mordred slaweq: sweet
21:47:02 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: More volume functional test fixes https://review.openstack.org/626020
22:50:08 openstackgerrit Merged openstack/openstacksdk master: Adds kwargs support when creating a Neutron subnet https://review.openstack.org/625920
#openstack-sdks - 2018-12-19
00:41:53 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Turn off unneeded devstack services https://review.openstack.org/626058
01:10:21 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: More volume functional test fixes https://review.openstack.org/626020
04:33:19 openstackgerrit Dean Troyer proposed openstack/python-openstackclient master: More volume functional test fixes https://review.openstack.org/626020
19:29:42 mordred Shrews: sorry for the delay - I'm sort of down a little bit of a rabbit hole poking at tests that aren't doing what we think they are
19:32:15 Shrews mordred: i don't have you on any timer
19:33:08 mordred Shrews: I ... I started pulling on some threads
19:33:29 Shrews spelunk down those rabbit holes if it entertains you :)
19:35:09 mordred Shrews: "entertain"
19:45:56 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Properly munch for resource sub-dicts https://review.openstack.org/625923
19:45:57 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Start using direct REST in normalize tests https://review.openstack.org/626379
19:45:57 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: WIP Compute location properly in server https://review.openstack.org/626380
19:46:19 mordred Shrews: the first two are solid, the third is a deeper rabbit hole that needs to be done but is not essential for this moment in time
19:46:40 Shrews looking
19:48:30 Shrews mordred: what is 'original_names'?
19:49:51 mordred Shrews: it's whether you want the dict from the sdk Resource to have the keys as they came from the remote host, or whether you want it to have its already normalized names
19:50:17 mordred Shrews: right now in the shade layer we're asking for original_names so that our normalize things, which were written to normalize the unnormalized names, will still work
19:50:23 mordred but that's so many layers or normalizing
19:51:44 mordred Shrews: I was figuring that a two-step approach to getting to just returning the Resource objects directly would be a patch to say original_names=False with the update to the normalize function - then a patch to update the Resource with any missing fields, remove the normalize function and just return the resource
19:52:08 mordred Shrews: although, in reality, it's likely more confusing to do that- and just doing a patch to return the resource directly is fine...
19:52:27 mordred Shrews: at least once the test_normalize file is actually doing all requests_mock so we're testing the outcome of calling the method
19:52:38 mordred since that shoudl then let us test that we're not breaking contract over the switch
19:54:57 Shrews mordred: we should probably make tobiash's new test depend on that when it is ready to validate it fixes the nodepool issue
19:55:14 Shrews (and doesn't break something _else_)
19:55:34 mordred ++
19:58:28 Shrews mordred: https://review.openstack.org/#/c/626379/1/openstack/tests/unit/base.py <-- you don't like 'p' in your project names???? :)
19:58:40 Shrews line 216
19:58:54 mordred Shrews: p's are for lusers
19:59:11 Shrews i thought they were for posers
20:19:36 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Properly munch for resource sub-dicts https://review.openstack.org/625923
20:19:37 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Start using direct REST in normalize tests https://review.openstack.org/626379
20:19:37 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: WIP Compute location properly in server https://review.openstack.org/626380
20:19:46 mordred Shrews: ok. I think that should take care of the issues
20:20:08 mordred tobiash: ^^
20:22:06 mordred Shrews: while I've got you here: https://review.openstack.org/#/c/626058/ is a copy of stuff clarkb did for nodepool
20:23:54 mordred slaweq: if you have a sec, another neutron patch for you: https://review.openstack.org/#/c/625381/3
20:25:53 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Drop self.conn from base.TestCase https://review.openstack.org/625115
20:37:18 slaweq mordred: looking
20:44:37 slaweq mordred: commented
20:45:12 slaweq mordred: basically this feature has only spec merged so far and I'm not sure if it's good time to merge it
20:58:24 mordred slaweq: cool- thanks! (This is why it's good to defer to subject matter experts :) )
21:01:19 slaweq mordred: yes, I agree :)
21:38:52 openstackgerrit Merged openstack/openstacksdk master: Import code for building ironic-compatible configdrives https://review.openstack.org/624441
21:53:55 openstackgerrit Merged openstack/openstacksdk master: Turn off unneeded devstack services https://review.openstack.org/626058
#openstack-sdks - 2018-12-20
00:33:51 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Start using direct REST in normalize tests https://review.openstack.org/626379
00:33:51 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Drop self.conn from base.TestCase https://review.openstack.org/625115
00:33:52 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: WIP Compute location properly in server https://review.openstack.org/626380
01:27:50 openstackgerrit Merged openstack/openstacksdk master: Properly munch for resource sub-dicts https://review.openstack.org/625923
02:07:19 openstackgerrit jacky06 proposed openstack/os-api-ref master: Remove support for py34 https://review.openstack.org/626300
06:10:02 openstackgerrit Dou Rui Yuan proposed openstack/python-openstackclient master: Remove testr.conf as it's been replaced by stestr https://review.openstack.org/626492
11:22:54 frickler shade-ansible job looks broken for a couple of days, did anyone look into that yet? http://zuul.openstack.org/builds?job_name=shade-ansible-functional-devstack
11:30:04 frickler mordred: I rebased https://review.openstack.org/526115 for you, seems that that would still be relevant to have as test coverage
12:53:31 frickler mordred: this pins to ansible==2.5.0, which seems to have a broken "user" module functionality on bionic, so this is fallout of the devstack bionic change. pinning to "ansible<2.6", which seems to have more of the intended effect anyway, solves the issue on my test machine. https://review.openstack.org/#/c/570680/1/tox.ini,unified
12:55:47 openstackgerrit Jens Harbott (frickler) proposed openstack-infra/shade master: Fix ansible stable pin in tox test https://review.openstack.org/626568
12:56:14 frickler mordred: Shrews: ^^ that should fix the test failures
13:16:42 openstackgerrit Rajat Dhasmana proposed openstack/python-openstackclient master: Fix: Restore output 'VolumeBackupsRestore' object is not iterable https://review.openstack.org/624860
13:22:21 mordred frickler: awesome! thanks
13:46:34 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Drop self.conn from base.TestCase https://review.openstack.org/625115
13:46:34 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: WIP Compute location properly in server https://review.openstack.org/626380
14:51:33 openstackgerrit Jens Harbott (frickler) proposed openstack/python-openstackclient master: Add osc repo to the base job definition https://review.openstack.org/626590
14:51:58 frickler mordred: ^ seems this is needed for the devstack change
14:56:43 mordred frickler: +A - I think that's a clear enough change
15:00:24 openstackgerrit Merged openstack/openstacksdk master: Start using direct REST in normalize tests https://review.openstack.org/626379
15:18:21 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Add possibility to override base_path for resource operations https://review.openstack.org/621153
15:21:55 openstackgerrit Artem Goncharov proposed openstack/openstacksdk master: Add possibility to override base_path for resource operations https://review.openstack.org/621153
15:45:09 openstackgerrit Merged openstack-infra/shade master: Fix ansible stable pin in tox test https://review.openstack.org/626568
15:48:08 Shrews mordred: remind me why our ansible tox test is pinned?
15:50:10 Shrews oh, that's shade
15:54:24 mordred Shrews: yeah- and it was to test master of shade against stable of ansible
16:00:40 edleafe he last API-SIG Office Hour of 2018 has begun
16:00:47 edleafe *The last
16:01:23 elmiko hello/
16:01:47 edleafe hola
16:02:01 elmiko did you see the email that Artem sent to the list?
16:02:08 elmiko i think he may join us today, not sure though
16:02:14 gtema Hi, I am here
16:02:20 elmiko hey!
16:02:21 elmiko welcome =)
16:02:25 gtema thanks
16:02:46 elmiko so, as i mentioned in the email thread, we don't have a standing "meeting" anymore just these office hours
16:02:47 edleafe Hi gtema
16:03:14 gtema edleafe: hi
16:03:15 elmiko that said, if you would like to represent the sdk side of the house within the api-sig then i am all for it =)

Earlier   Later