Thursday, 2019-01-17

openstackgerritDuc Truong proposed openstack/senlin master: Miscellaneous fixes  https://review.openstack.org/63131700:11
*** Qiming has quit IRC00:59
*** Qiming has joined #senlin01:00
openstackgerritDuc Truong proposed openstack/senlin master: Do not set parent action status if stop node failed  https://review.openstack.org/63136501:33
*** sapd1_x has joined #senlin03:10
*** sapd1_x has quit IRC04:08
*** sapd1_x has joined #senlin05:12
*** sapd1_x has quit IRC11:42
*** irclogbot_3 has quit IRC12:47
*** irclogbot_3 has joined #senlin12:58
*** sapd1_x has joined #senlin13:21
*** irclogbot_3 has quit IRC13:35
*** irclogbot_3 has joined #senlin13:48
*** mordred has joined #senlin15:09
mordredeandersson, dtruong: the location header - if I do DELETE /clusters/{cluster_id} - am I right in understanding that the Loication header is going to have http://example.com/v1/clusters/{cluster_id}/{action_id}?15:10
mordred(like, there is an additional id it's returning)15:11
mordredif so - if one hits taht url - what is returned? the api docs don't indicate the content - is it just there until the action is done and then a 404 once it is?15:30
mordredremote:   https://review.openstack.org/631362 Add class to override clustering delete responses15:31
mordredI updated dtruong's patch with a nother approach - but I think that's incomplete without knowing what one is supposed to do with that location url15:32
mordredonce I know that - I think we should update clustering.wait_for_delete to do the right thing15:32
*** sapd1_x has quit IRC15:33
*** jmlowe has quit IRC15:47
*** sapd1_x has joined #senlin16:24
*** sapd1_x has quit IRC17:08
*** jmlowe has joined #senlin19:22
*** phuoc_ has joined #senlin20:11
*** phuoc has quit IRC20:14
*** irclogbot_3 has quit IRC20:41
*** irclogbot_3 has joined #senlin20:51
*** openstackgerrit has quit IRC20:56
*** jmlowe has quit IRC21:19
*** openstackgerrit has joined #senlin21:37
*** ChanServ sets mode: +v openstackgerrit21:37
openstackgerritJude Cross proposed openstack/senlin master: Add action_purge to senlin-manage  https://review.openstack.org/63165221:37
*** jmlowe has joined #senlin21:41
*** sapd1_ has joined #senlin22:57
*** sapd1 has quit IRC23:01
dtruongmordred: the location header for a cluster delete would be http://example.com/v1/actions/{action_id}23:03
dtruongsee http://paste.openstack.org/show/742902/ for a full capture of the response23:03
dtruongthe senlin client gets the location from the response header and strips out the action_id and prints it out to the user23:04
dtruonghttps://github.com/openstack/python-senlinclient/blob/master/senlinclient/v1/cluster.py#L36323:06
dtruongHitting that url returned in the location header will give you the action details: https://developer.openstack.org/api-ref/clustering/?expanded=show-action-details-detail#show-action-details23:10
dtruongThe clustering.wait_for_delete should query the location url and check the status returned by it.23:35
dtruongit should wait for the status to become either SUCCEEDED or FAILED23:36
dtruongThat's what we also expect users of gophercloud to do when calling cluster delete:23:37
dtruonghttps://github.com/gophercloud/gophercloud/blob/master/acceptance/openstack/clustering/v1/clustering.go#L29223:37
dtruonghttps://github.com/gophercloud/gophercloud/blob/master/acceptance/openstack/clustering/v1/clustering.go#L39923:37

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!