Posted	Nick	Remark
#openstack-sdks - 2018-04-06
18:13:59	mordred	_alastor_: I'm having a similar but not *quite* the same issue myself, with the same symptoms - works in a virtualenv, has issues non-venv
18:14:11	johnsom	Ok, yeah, ping me with it. I can use that as a test case too. However it will likely be later today before I have something. I pivoted to another project.
18:14:12	mordred	_alastor_: I don't have answers yet, but clearly this is a big problem
18:14:42	mordred	johnsom: let's just circle back to it early next week maybe - I got up super early today and am almost done for the day
18:14:59	johnsom	Sounds good. Have a good weekend
18:15:49	mordred	you too!
18:16:06	_alastor_	mordred: Ok, I'm both glad and sad it's not a unique issue :)
18:16:45	mordred	_alastor_: me too - I really would have preferred it if your machine was just weird :)
18:21:54	gtmanfred	mordred: when you have time, i would like to talk about this https://storyboard.openstack.org/#!/story/2001794 We have just released our new cloud driver for salt-cloud that uses shade, but are running into this race condition with one of our customers
18:50:33	ssurana	Hey Guys! need some more reviews on https://review.openstack.org/#/c/558461/ thanks!
18:53:09	mordred	gtmanfred: yes - I'd love to talk about that - I'm not 100% here at the moment, so maybe I can catch you on Monday
18:53:23	gtmanfred	That would work perfectly!
18:53:37	mordred	gtmanfred: fwiw - there are two mechanisms shade uses for doing auto floating ips - one of them is safe for multi-process use, the other is not
18:53:59	gtmanfred	cool, i might need to just switch to the safe one, because right now I am only using create_server
18:54:03	gtmanfred	with wait=True
18:54:46	mordred	gtmanfred: gotcha. so - lemme dig up the part that does the safe mechanism
18:54:46	gtmanfred	yeah, we have updated the customer that we are going to work with upstream to get this fixed, so we are clear for a bit on my end.
18:54:51	gtmanfred	:+1:
18:55:02	mordred	sweet
18:56:20	gtmanfred	Thanks!
18:56:34	mordred	gtmanfred: ok. the code isn't friendly to link to - the tl;dr is - rather than doing create floating ip + attach floating ip - if you find the neutron port that is assocated with the target fixed ip on the server, you can pass that port to the create floating ip call - and if you do, neutron will create-and-attach the floating ip -so it'll never be in a state where another process can find it and try
18:56:36	mordred	to attach it
18:57:48	mordred	gtmanfred: I'll see if I can't write up a little something about it that would be more readable than attempting to trade the shade logic itself (which is a bit crazier than one would like)
18:59:52	gtmanfred	that would be awesome! thanks! take your time, I can wait till monday if you need
19:00:29	gtmanfred	also, here is one other question
19:01:02	gtmanfred	the only other problem is if you use reuse=True, you could just run into a race condition where multiple servers grab the same available ip, and try to use it
#openstack-sdks - 2018-04-08
01:37:37	openstackgerrit	Doug Hellmann proposed openstack/openstackclient master: add lower-constraints job https://review.openstack.org/556037
03:50:34	openstackgerrit	chenyb4 proposed openstack/openstacksdk master: Fix wait for futures append 'result' error https://review.openstack.org/559523
11:37:59	openstackgerrit	Dongcan Ye proposed openstack/python-openstackclient master: Remove deprecated ip floating commands https://review.openstack.org/546924
12:05:14	openstackgerrit	Dongcan Ye proposed openstack/python-openstackclient master: Network: Add tag support for floating ip https://review.openstack.org/547316
15:14:59	openstackgerrit	Dongcan Ye proposed openstack/python-openstackclient master: Network: Add tag support for floating ip https://review.openstack.org/547316
#openstack-sdks - 2018-04-09
09:16:30	openstackgerrit	Merged openstack/os-client-config master: Add osc tips jobs back https://review.openstack.org/526706
13:13:44	openstackgerrit	Jens Harbott (frickler) proposed openstack/python-openstackclient master: Add help for nova interface-list to decoder https://review.openstack.org/559734
13:52:45	openstackgerrit	Jens Harbott (frickler) proposed openstack/python-openstackclient master: Add bgp commands to neutron decoder https://review.openstack.org/559742
14:21:12	openstackgerrit	Doug Hellmann proposed openstack/keystoneauth master: add lower-constraints job https://review.openstack.org/555625
14:24:46	openstackgerrit	Doug Hellmann proposed openstack/cliff master: add lower-constraints job https://review.openstack.org/555405
15:01:48	openstackgerrit	Doug Hellmann proposed openstack/os-client-config master: add lower-constraints job https://review.openstack.org/556042
16:40:20	openstackgerrit	Duc Truong proposed openstack/openstacksdk master: Add function examples test for clustering https://review.openstack.org/555762
17:26:45	openstackgerrit	Jens Harbott (frickler) proposed openstack/python-openstackclient master: Add bgp commands to neutron decoder https://review.openstack.org/559742
17:31:27	openstackgerrit	Doug Hellmann proposed openstack/keystoneauth master: add lower-constraints job https://review.openstack.org/555625
20:11:47	openstackgerrit	Merged openstack/python-openstackclient master: Add help for nova interface-list to decoder https://review.openstack.org/559734
#openstack-sdks - 2018-04-10
14:48:43	openstackgerrit	Sean McGinnis proposed openstack/python-openstackclient master: Remove invalid 'unlock-volume' migration arg https://review.openstack.org/558910
18:29:40	openstackgerrit	Doug Hellmann proposed openstack/cliff master: exclude cmd2 0.8.3 and update to 0.8.4 https://review.openstack.org/560121
18:52:34	eandersson	mordred, would this patch work for pike? https://review.openstack.org/#/c/558710/
19:35:57	openstackgerrit	Sean McGinnis proposed openstack/python-openstackclient master: Remove invalid 'unlock-volume' migration arg https://review.openstack.org/558910
19:35:58	openstackgerrit	Sean McGinnis proposed openstack/python-openstackclient master: Clean up W503 and E402 pep8 errors https://review.openstack.org/560132
21:04:06	openstackgerrit	Sean McGinnis proposed openstack/python-openstackclient master: Remove invalid 'unlock-volume' migration arg https://review.openstack.org/558910
21:07:55	openstackgerrit	Sean McGinnis proposed openstack/python-openstackclient master: Clean up W503 and E402 pep8 errors https://review.openstack.org/560132
21:30:02	openstackgerrit	Sean McGinnis proposed openstack/python-openstackclient master: Remove invalid 'unlock-volume' migration arg https://review.openstack.org/558910
22:27:32	openstackgerrit	Merged openstack/openstacksdk master: Fix resource not exist the resource.status error https://review.openstack.org/558757
22:38:46	openstackgerrit	Merged openstack/os-client-config master: add lower-constraints job https://review.openstack.org/556042
#openstack-sdks - 2018-04-11
04:03:56	openstackgerrit	Merged openstack/openstacksdk master: Strip the version prefix from the next link for pagination https://review.openstack.org/558461
08:53:44	frickler	amotoki: mordred: dtroyer_zz: these three would seem to close the last gap in DNS support between neutron and OSC https://review.openstack.org/516701 https://review.openstack.org/558820 https://review.openstack.org/558824
09:24:34	jpich	dtroyer_zz: Hi! I submitted a couple of release patches for osc stable/ocata as well, if you would have time to give them a look over at some point I would be grateful :) Thank you
11:38:45	openstackgerrit	Jacky Hu proposed openstack/openstacksdk master: Allow cascade deletion of load balancer https://review.openstack.org/553404
12:25:45	dtroyer_zz	jpich: I'll have a look this afternoon, thanks!
13:01:46	jpich	dtroyer_zz: Thank you!
13:05:43	pooja_jadhav	mordred: Hi
13:17:39	openstackgerrit	Jens Harbott (frickler) proposed openstack/openstacksdk master: WIP: Add functional tests for Neutron DNS extension https://review.openstack.org/559068
13:40:42	pooja_jadhav	hi all, any body knows about split logging?
13:41:10	pooja_jadhav	I need some help
15:22:36	frickler	mugsie: do you happen to know why there is a broken functional recordset test in the sdk? might that be due to the ovo changes? http://logs.openstack.org/68/559068/4/check/openstacksdk-functional-devstack-tips/796096e/testr_results.html.gz
15:22:52	mugsie	I freaking hope not
15:22:56	mugsie	let me look
15:27:17	frickler	mugsie: might also be imported from shade and never tested before. in that patch I enable designate for the functional test for the first time in an attempt to work around an issue with neutron integration
15:27:46	mugsie	shade was pretty well tested against designate afaik
15:27:58	frickler	mugsie: just a bit disappointed that I got the original issue resolved and now things are still red because of this failure
15:28:04	mugsie	for sure
15:28:30	frickler	anyway, I'm off to enjoy the remainder of the beautiful weather here, will continue tomorrow
15:28:51	mugsie	frickler: enjoy :)
15:32:29	mugsie	frickler: I think I found it - let me see if I can get a fix together
15:45:32	openstackgerrit	Graham Hayes proposed openstack/openstacksdk master: Fix DNS Recordset CRUD to use IDs https://review.openstack.org/560477
15:45:43	mugsie	frickler: ^ should fix it
17:00:49	frickler	mugsie: good job, I had only looked at the broken test so far and not yet noticed that the implemented function indeed doesn't live up to what it promises. added comments
17:01:44	mugsie	frickler: https://review.openstack.org/#/c/560545/1 was added to test the fix as well :)
17:03:12	frickler	mugsie: ah, I didn't see that, marvelous
17:16:13	mugsie	gah - that is failing unit tests anyway
#openstack-sdks - 2018-04-12
06:44:57	frickler	mugsie: seems the unit test will need added mocking for the new calls you inserted, let me know if you want help with that
07:02:00	mhen	Hello!
07:03:17	mhen	This channel is also related to the OpenStack client, correct?
07:08:28	openstackgerrit	Jens Harbott (frickler) proposed openstack/openstacksdk master: WIP: Add functional tests for Neutron DNS extension https://review.openstack.org/559068
07:11:51	frickler	mhen: yes, feel free to ask your real question. we should also consider making the channel topic more inviting
07:12:22	mhen	okay
07:20:34	mhen	Is there no IRC meeting for the client anymore? I found a schedule and agendas online but the most recent seems to be mid 2017.
07:24:04	mhen	background is: I'm trying to implement image signing functionality into the OpenStack client (which currently has to be done manually using 'openssl' commands). I'd like to discuss if this would be worth a contribution and if so, where it would be best to put related crypto code (openstack-sdk or cursive library?).
07:29:58	frickler	mhen: I haven't seen an IRC meeting and the schedule on eavesdrop doesn't seem to list one. probably outdated information in the wiki?
07:30:59	frickler	mhen: regarding the other question I have no idea, maybe dtroyer_zz or mordred have some idea. but that topic may also better be suited for a discussion on the dev-ml
07:37:04	mhen	For reference, this is the meeting schedule I'm referring to: http://eavesdrop.openstack.org/#OpenStackClient_Team_Meeting
07:39:51	mhen	frickler, you referring to this I guess? http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
07:41:45	openstackgerrit	Jens Harbott (frickler) proposed openstack/openstacksdk master: Fix DNS Recordset CRUD to use IDs https://review.openstack.org/560477
07:42:19	frickler	mugsie: fixed a small glitch in your patch first ^^
07:44:41	frickler	mhen: you are correct about th mailing list
07:45:46	frickler	mhen: and no idea why I missed the meeting in the schedule. probably should get dropped there if it doesn't happen anyway
07:46:12	mhen	frickler, okay. Thanks for your guidance! I will try there :)
08:57:50	openstackgerrit	Jens Harbott (frickler) proposed openstack/openstacksdk master: Add designate to the services deployed for functional tests https://review.openstack.org/560831
09:01:18	frickler	mugsie: FYI ^^ waiting to see what others think about that now
09:17:02	openstackgerrit	Graham Hayes proposed openstack/openstacksdk master: Fix DNS Recordset CRUD to use IDs https://review.openstack.org/560477
09:19:02	openstackgerrit	Graham Hayes proposed openstack/openstacksdk master: DNM: Fix DNS Recordset CRUD to use IDs https://review.openstack.org/560545
09:19:43	mugsie	frickler: cool. ^^ should fix the unit tests, and I reup'd the testing patch on top as well
13:11:57	openstackgerrit	Nick Jones proposed openstack/os-client-config master: Remove DataCentred from list of vendors https://review.openstack.org/560894

Earlier   Later