Posted	Nick	Remark
#openstack-sdks - 2017-11-28
10:35:45	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: WIP Fix py35 tox env https://review.openstack.org/523383
10:36:22	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: WIP Fix py35 tox env https://review.openstack.org/523383
15:15:51	dtroyer	mordred: just like making things idiot-proof only breeds smarter idiots, I'm sure we'll think of something :)
15:17:01	mordred	dtroyer: ++
15:23:40	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Document current_user_id in a release note https://review.openstack.org/522973
17:06:52	mordred	dtroyer: in osc4 - are you using or planning to use get_legacy_client from openstack.config to get the python-*client objects you need?
17:07:31	dtroyer	mordred: no, I've never used them and do not plan to. the client setup remains as-is until each API moved to use SDK directly
17:07:33	mordred	dtroyer: (asking because I wasn't sure if we should keep that method around in openstack.config and just keep it in the occ compat shim - or whether it was providing value to you in the osc layer)
17:07:47	mordred	dtroyer: cool. I think that says to me that we should remove it from the sdk codebase
17:07:55	dtroyer	that works for me
17:09:05	dtroyer	so, fwiw, I'm having a hell of a time sorting out the "Cloud defaults was not found" error in the current osc functioal-tips job… I have no idea where that comes from in the middle of neutron setup
17:09:19	dtroyer	and can't duplicate it locally
17:11:49	mordred	dtroyer: yah - I also can't duplicate it locally - I'm almost at the point of holding a devstack node and seeing if we can reproduce it by hand there
17:13:33	mordred	dtroyer: I've added an autohold to zuul - next time the job fails on an OSC patch zuul will not delete the node and we can investigate
17:14:17	mordred	dtroyer: I'd love to get that job working and stable enough to be voting - I think that would give us a ton of confidence to break things much more creatively
17:17:10	dtroyer	mordred: ++ This should be pushing me back to the collect-configs-from-runs work, maybe after lunch…
17:17:51	mordred	dtroyer: well - one of my goals for this week is getting the base job changes rolled out which will make collecting additional files much easier
17:18:22	dtroyer	cool…
17:28:50	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Fix py35 tox env https://review.openstack.org/523383
17:28:50	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Cleanup objects that we create on behalf of images https://review.openstack.org/522974
17:28:51	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Add method to cleanup autocreated image objects https://review.openstack.org/522975
17:28:51	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Complete move of baremetal machine tests https://review.openstack.org/522976
17:28:52	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Treat clouds.yaml with one cloud like envvars https://review.openstack.org/522977
17:28:52	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Protect against p_opt not having prompt attribute https://review.openstack.org/522978
17:28:53	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Update make_rest_client to work with version discovery https://review.openstack.org/522979
17:28:53	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Added nat_source flag for networks. https://review.openstack.org/522980
17:28:54	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Shift image tests from test_operator https://review.openstack.org/523258
17:28:54	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Switch baremetal nics/ports tests over https://review.openstack.org/523355
17:28:55	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Start using Connection in openstack.cloud https://review.openstack.org/523259
17:28:55	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Remove get_legacy_client https://review.openstack.org/523483
17:30:49	mordred	dtroyer, Shrews: ^^ ok, having fixed the tests in https://review.openstack.org/#/c/522973/ - that stack *should* all pass ... and should be just cherry-picks of already landed change up to https://review.openstack.org/523355
17:30:56	mordred	oh - poo - one minor issue in that one ...
17:31:57	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Switch baremetal nics/ports tests over https://review.openstack.org/523355
17:31:58	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Start using Connection in openstack.cloud https://review.openstack.org/523259
17:31:58	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Remove get_legacy_client https://review.openstack.org/523483
17:33:05	mordred	Shrews, dtroyer: https://review.openstack.org/523259 and https://review.openstack.org/523483 are actually interesting patches
17:35:53	mordred	briancurtin: ^^ you might find https://review.openstack.org/523259 interesting- it's the start of using the sdk Connection object in the openstack.cloud methods - it's a baby step - mostly want to get all of the REST calls switched over to using the SDK REST passthrough methods so we can verify discovery and whatnot is all working as expected - but I think the openstack.cloud methods want to use the
17:35:55	mordred	sdk object interface instead of the rest interface - but one step at a time
17:52:31	Shrews	mordred: ack. will look
21:35:55	openstackgerrit	Merged openstack/python-openstackclient master: Replace %r with %s on printing string variable https://review.openstack.org/519385
22:07:00	openstackgerrit	Julia Kreger proposed openstack-infra/shade master: De-client-ify baremetal node_set_provision_state https://review.openstack.org/499773
22:53:19	openstackgerrit	Merged openstack/python-openstackclient master: Fix file mode on network-topology.rst https://review.openstack.org/521878
22:57:47	SamYaple	mordred: good to go with the redirect
22:58:15	mordred	SamYaple: woot!
23:04:56	TheJulia	o/
23:17:12	openstackgerrit	Zane Bitter proposed openstack/python-openstackclient master: Set correct designate endpoint in docs https://review.openstack.org/523567
#openstack-sdks - 2017-11-29
00:03:43	openstackgerrit	Julia Kreger proposed openstack-infra/shade master: de-client-ify baremetal get_machine https://review.openstack.org/499774
00:03:45	openstackgerrit	Julia Kreger proposed openstack-infra/shade master: Catch attrbute error for other APIs https://review.openstack.org/499775
00:03:46	openstackgerrit	Julia Kreger proposed openstack-infra/shade master: De-clientify baremetal create/delete https://review.openstack.org/500119
00:03:48	openstackgerrit	Julia Kreger proposed openstack-infra/shade master: De-client-ify baremetal machine port list https://review.openstack.org/500120
00:03:49	openstackgerrit	Julia Kreger proposed openstack-infra/shade master: De-client-ify machine patch operations https://review.openstack.org/500121
01:14:24	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Fix py35 and pypy tox env https://review.openstack.org/523383
01:14:24	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Cleanup objects that we create on behalf of images https://review.openstack.org/522974
01:14:25	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Add method to cleanup autocreated image objects https://review.openstack.org/522975
01:14:25	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Complete move of baremetal machine tests https://review.openstack.org/522976
01:14:26	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Treat clouds.yaml with one cloud like envvars https://review.openstack.org/522977
01:14:26	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Protect against p_opt not having prompt attribute https://review.openstack.org/522978
01:14:27	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Update make_rest_client to work with version discovery https://review.openstack.org/522979
01:14:27	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Added nat_source flag for networks. https://review.openstack.org/522980
01:14:28	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Shift image tests from test_operator https://review.openstack.org/523258
01:14:28	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Switch baremetal nics/ports tests over https://review.openstack.org/523355
01:14:29	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Start using Connection in openstack.cloud https://review.openstack.org/523259
01:14:29	openstackgerrit	Monty Taylor proposed openstack/python-openstacksdk master: Remove get_legacy_client https://review.openstack.org/523483
01:14:50	rods	wow
01:16:15	SamYaple	one patch a day mordred. thats my review rate for your patches. youre being throttled
01:40:58	openstackgerrit	Hengqing Hu proposed openstack/python-openstacksdk master: Add pools attribute to load balancer heath monitor https://review.openstack.org/522962
02:03:36	TheJulia	SamYaple: What about for patches written by others? :)
02:04:35	SamYaple	TheJulia: dont subvert the spirit of my decleration with your hacker-think!
02:04:45	TheJulia	Muahahahahaha!
03:40:38	openstackgerrit	Jamie Lennox proposed openstack/keystoneauth master: Mark SAML loader properties as required https://review.openstack.org/523675
03:58:23	openstackgerrit	Jamie Lennox proposed openstack/keystoneauth master: Mark SAML loader properties as required https://review.openstack.org/523675
08:58:23	openstackgerrit	OpenStack Proposal Bot proposed openstack/keystoneauth master: Updated from global requirements https://review.openstack.org/523736
10:11:53	openstackgerrit	Édouard Thuleau proposed openstack/python-openstacksdk master: Set empty Tag list if Network Tag API extension not supported https://review.openstack.org/523834
12:41:37	frickler	mordred: how about doing a shade release fixing https://storyboard.openstack.org/#!/story/2001265 ?
13:38:58	openstackgerrit	David Shrewsbury proposed openstack-infra/shade master: WIP: os_router idempotency check https://review.openstack.org/523873
13:41:10	Shrews	mordred: wow, your changes are not happy moving through to gate
13:41:36	Shrews	frickler: i'm looking at that now
13:41:54	Shrews	523873 trying to confirm what's being seen
13:50:34	frickler	Shrews: the bug is triggered only when Neutron is configured with HA (or maybe DVR) and the router also uses that, so I'm not sure you will see it in gate easily
13:51:15	Shrews	frickler: yeah, just noticed that comment. trying to find the api for that
13:54:23	Shrews	frickler: not being familiar with that extension, was https://review.openstack.org/482153 totally the wrong thing to do then?
13:54:52	Shrews	mordred: i'm wondering what problem 482153 was solving and if it should just be reverted
13:56:57	Shrews	the commit message says "L3 HA ports do not correspond to a router interface" but obviously that's not correct
13:58:56	Shrews	frickler: do you know if that is something that can be enabled in devstack?
14:01:24	frickler	Shrews: iiuc 482153 solved the issue not to list the L3 HA ports that are more or less internal to the Neutron HA implementation
14:02:26	frickler	Shrews: and you can sure enable things in devstack, neutron is running various scenarios with DVR, not sure about HA
14:03:08	Shrews	oh! seems a fix is already committed
14:03:44	Shrews	ok, i thought we needed a fix AND release. but just a release is needed then
14:03:54	frickler	Shrews: ah, yes, I did a fix for that, sorry, I assumed that you had seen it. I thought you were trying to verify the fix
14:04:20	Shrews	frickler: my fault, i misunderstood. yeah, only mordred can do the release
14:05:36	frickler	Shrews: adding your check above and maybe a devstack scenario that runs Neutron in HA mode still seem useful to me
14:05:46	Shrews	would be nice if we could find out if devstack supported this though
14:06:26	Shrews	frickler: yeah. i don't know the devstack magic for that :(
14:07:14	frickler	Shrews: I'll put it on my list, hopefully sometime this week
14:15:24	cdent	sdague: how did your microversions talk go? was it at apistrat?
14:16:51	sdague	it went pretty well, it was at apistrat. It was the end of the last day, so I got a couple of questions on it, but not a lot of space for broader discussion
14:18:53	cdent	did anyone have an epiphany?
14:38:48	mordred	Shrews, frickler: releases are actually run by the release team now, so you can actually submit a patch requesting one without needing me ... but, I can do that this morning - I think we're overdue for a release anyway
14:42:20	Shrews	mordred: oh right. is that process described anywhere?
14:46:54	mordred	Shrews: it's pretty much making a patch to openstack/releases ... you'll find a file, deliverables/queens/shade.yaml - that has entries you can copy-pasta

Earlier   Later