Posted	Nick	Remark
#openstack-sdks - 2018-04-06
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	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:46	mordred	gtmanfred: gotcha. so - lemme dig up the part that does the safe mechanism
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
13:47:22	mordred	mugsie, frickler: I'd be fine just adding designate to the normal functional test config for openstacksdk as long as doing so doesn't increase the memory load on the devstack such that stuff starts failing
13:49:02	mordred	mugsie, frickler: however, if adding it to the default doesn't work, the senlin folks have a patch up adding a senlin-enabled job, we could also add designate to that (and name it somethin gdifferent)

Earlier   Later