Earlier  
Later
New Search
Posted Nick Remark
#openstack-sdks - 2018-12-18
02:25:45 kmalloc so i'm not touching it
02:26:10 kmalloc and cirros 3.4
02:26:27 kmalloc ah because they are fakes
02:26:34 kmalloc s/fakes/mocks
02:29:28 openstackgerrit Morgan Fainberg proposed openstack/openstacksdk master: Fix dogpile.cache 0.7.0 interaction https://review.openstack.org/625370
02:30:28 openstackgerrit Morgan Fainberg proposed openstack/openstacksdk master: Unpin dogpile.cache https://review.openstack.org/625759
02:32:03 openstackgerrit Morgan Fainberg proposed openstack/openstacksdk master: Unpin dogpile.cache https://review.openstack.org/625759
05:12:53 kmalloc mordred, ianw: i think i have most of the stuff ready. someone should follow up with it and get it through the door. It should be close if not good to go
05:13:03 kmalloc and with that... logging out, see you all next year
07:17:25 openstackgerrit Merged openstack/python-openstackclient master: Modify the help message for 'registered limit set' https://review.openstack.org/609985
08:38:35 openstackgerrit Merged openstack/openstacksdk master: Avoid dogpile.cache 0.7.0 https://review.openstack.org/624485
14:03:29 openstackgerrit Mário Santos proposed openstack/openstacksdk master: Adds kwargs support when creating a Neutron subnet https://review.openstack.org/625920
14:15:06 openstackgerrit Monty Taylor proposed openstack/openstacksdk master: Use munch for server security groups https://review.openstack.org/625923
15:49:42 openstackgerrit Merged openstack/openstacksdk master: Fix dogpile.cache 0.7.0 interaction https://review.openstack.org/625370
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

Earlier   Later