*** altlogbot_2 has joined #senlin | 00:29 | |
*** altlogbot_2 has quit IRC | 00:37 | |
*** altlogbot_1 has joined #senlin | 01:07 | |
*** altlogbot_1 has quit IRC | 01:15 | |
*** altlogbot_1 has joined #senlin | 02:20 | |
*** irclogbot_1 has joined #senlin | 02:20 | |
*** altlogbot_1 has quit IRC | 02:25 | |
*** irclogbot_1 has quit IRC | 02:26 | |
*** irclogbot_0 has joined #senlin | 04:08 | |
*** irclogbot_0 has quit IRC | 04:12 | |
*** altlogbot_1 has joined #senlin | 04:14 | |
*** altlogbot_1 has quit IRC | 04:17 | |
*** altlogbot_3 has joined #senlin | 05:04 | |
*** irclogbot_2 has joined #senlin | 05:08 | |
*** altlogbot_3 has quit IRC | 05:21 | |
*** irclogbot_2 has quit IRC | 05:22 | |
*** altlogbot_2 has joined #senlin | 07:04 | |
*** altlogbot_2 has quit IRC | 07:08 | |
*** irclogbot_3 has joined #senlin | 07:41 | |
*** irclogbot_3 has quit IRC | 07:44 | |
*** irclogbot_3 has joined #senlin | 08:59 | |
*** irclogbot_3 has quit IRC | 09:02 | |
*** altlogbot_2 has joined #senlin | 09:27 | |
*** altlogbot_2 has quit IRC | 09:28 | |
*** altlogbot_2 has joined #senlin | 09:31 | |
*** altlogbot_2 has quit IRC | 09:34 | |
*** altlogbot_3 has joined #senlin | 09:46 | |
*** altlogbot_3 has quit IRC | 09:50 | |
*** irclogbot_3 has joined #senlin | 10:25 | |
*** irclogbot_3 has quit IRC | 10:26 | |
*** altlogbot_3 has joined #senlin | 13:12 | |
*** altlogbot_3 has quit IRC | 13:16 | |
*** irclogbot_2 has joined #senlin | 13:19 | |
*** irclogbot_2 has quit IRC | 13:22 | |
*** altlogbot_0 has joined #senlin | 15:10 | |
*** altlogbot_0 has quit IRC | 15:12 | |
*** rouk has joined #senlin | 17:24 | |
rouk | anyone using senlin on stein? it seems like heat's OS::Senlin::Cluster is busted for stein senlin... | 17:24 |
---|---|---|
rouk | trying to debug it, but tracing the code by hand isnt super exciting. | 17:24 |
dtruong | We are using Stein version of Senlin. But we don't use the Heat senlin resource. | 17:25 |
dtruong | Can you paste the heat stack file that you are trying to run in paste.openstack.org? | 17:26 |
dtruong | And also the error message that you are getting. | 17:26 |
rouk | http://paste.openstack.org/show/gPbo2v79Zn0HCMKxvREq/ | 17:27 |
rouk | for some reason the location isnt a string now? i guess? failing a split attribute. | 17:28 |
rouk | heat side of these bits hasnt changed in years, not sure what from the senlin side needs to be adjusged | 17:28 |
rouk | grabbing the code off the guy having issues. it used to run, though. | 17:30 |
*** mloza has joined #senlin | 17:31 | |
rouk | dtruong: http://paste.openstack.org/show/WsK23nc2HTN3pFIVzGRZ/ | 17:35 |
rouk | any ideas? | 17:36 |
dtruong | still looking | 17:46 |
openstackgerrit | Merged openstack/senlin master: Add Python 3 Train unit tests https://review.opendev.org/669505 | 17:47 |
openstackgerrit | Merged openstack/senlin master: Switch to the new canonical constraints URL on master https://review.opendev.org/666948 | 17:58 |
rouk | only seems to be a problem for OS::Senlin::Cluster, other stuff is working as intended | 17:58 |
*** irclogbot_3 has joined #senlin | 18:07 | |
*** irclogbot_3 has quit IRC | 18:08 | |
dtruong | rouk ok, i think this is similar to what we fixed here: https://review.opendev.org/#/c/634326/1/senlinclient/v1/cluster.py | 18:13 |
rouk | well that would for sure remove th issue | 18:14 |
dtruong | Would it be easy for you to change the heat code here https://github.com/openstack/heat/blob/master/heat/engine/resources/openstack/senlin/cluster.py#L221 | 18:14 |
dtruong | to get the id key in the dictionary as we did in that patch set and try out our heat stack again? | 18:15 |
rouk | any idea if youll be getting merged? | 18:15 |
dtruong | That patch set I merged is a change I made in the python senlinclient. | 18:15 |
dtruong | We can propose the same change to the heat resource if that fix works for you | 18:16 |
rouk | well i think it would solve it? why wasnt this found... | 18:16 |
dtruong | Heat might not have any integration test for the senlin resource in their gate testing | 18:17 |
dtruong | So they would not have notice this breaking change until someone used it | 18:17 |
dtruong | So you might be the first one to use the Heat senlin resource in the Stein | 18:18 |
rouk | isnt cluster like... core function of the openstack clustering service? | 18:20 |
rouk | lol | 18:20 |
dtruong | I think most people don't use heat to create senlin cluster. They just create the Senlin clusters thru Senlin directly. | 18:26 |
rouk | well, the patch wont apply 1:1, what would i need? | 18:26 |
rouk | since that client code is for delete ops, etc | 18:26 |
dtruong | can you change https://github.com/openstack/heat/blob/master/heat/engine/resources/openstack/senlin/cluster.py#L221 to the following: | 18:28 |
dtruong | action_id = cluster['id'] | 18:28 |
*** altlogbot_2 has joined #senlin | 19:04 | |
*** altlogbot_2 has quit IRC | 19:04 | |
rouk | dtruong: new error for you http://paste.openstack.org/show/A3kGRIhhq2hnPpA4N0vR/ | 19:05 |
rouk | is the cluster id not the action id? | 19:08 |
*** irclogbot_1 has joined #senlin | 19:21 | |
*** irclogbot_1 has quit IRC | 19:22 | |
rouk | the cluster id is never the stack id, wouldnt i need to pull the stack id? | 19:25 |
*** openstackgerrit has quit IRC | 19:49 | |
dtruong | rouk: the cluster id is not the same as action id | 19:57 |
dtruong | action id is a uuid returned when performing an action e.g. cluster create | 19:57 |
dtruong | the action id is used to track the progress of the action like cluster create | 19:57 |
*** mloza has quit IRC | 20:12 | |
*** mloza has joined #senlin | 20:15 | |
*** altlogbot_2 has joined #senlin | 20:28 | |
*** altlogbot_2 has quit IRC | 20:32 | |
*** altlogbot_1 has joined #senlin | 21:05 | |
*** altlogbot_1 has quit IRC | 21:08 | |
*** altlogbot_1 has joined #senlin | 21:19 | |
*** altlogbot_1 has quit IRC | 21:22 | |
*** irclogbot_1 has joined #senlin | 21:50 | |
*** irclogbot_1 has quit IRC | 21:52 | |
dtruong | rouk Ok, try this patch set: https://review.opendev.org/#/c/670202/ | 21:57 |
dtruong | That works for me | 21:57 |
*** altlogbot_0 has joined #senlin | 23:05 | |
*** altlogbot_0 has quit IRC | 23:08 | |
*** altlogbot_2 has joined #senlin | 23:14 | |
*** altlogbot_2 has quit IRC | 23:18 | |
*** altlogbot_2 has joined #senlin | 23:21 | |
*** altlogbot_2 has quit IRC | 23:24 | |
*** altlogbot_3 has joined #senlin | 23:27 | |
*** altlogbot_3 has quit IRC | 23:30 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!