| *** sridhar_ram1 has joined #openstack-heat-translator | 00:31 | |
| *** sridhar_ram has quit IRC | 00:33 | |
| *** sridhar_ram1 has quit IRC | 01:06 | |
| *** sridhar_ram has joined #openstack-heat-translator | 03:19 | |
| *** sridhar_ram has quit IRC | 03:22 | |
| *** bobh has joined #openstack-heat-translator | 03:47 | |
| *** bobh has quit IRC | 04:48 | |
| *** sridhar_ram has joined #openstack-heat-translator | 06:05 | |
| *** sridhar_ram1 has joined #openstack-heat-translator | 06:12 | |
| *** sridhar_ram has quit IRC | 06:13 | |
| *** sridhar_ram1 has quit IRC | 06:28 | |
| *** openstackgerrit has quit IRC | 10:02 | |
| *** openstackgerrit has joined #openstack-heat-translator | 10:03 | |
| *** spzala has joined #openstack-heat-translator | 15:39 | |
| openstackgerrit | Sahdev Zala proposed openstack/heat-translator: Log warning if Compute cap property not provided https://review.openstack.org/281917 | 15:49 |
|---|---|---|
| *** mvelten has joined #openstack-heat-translator | 15:53 | |
| spzala | Hi all | 16:00 |
| topol | o/ | 16:00 |
| spzala | topol: o/ mvelten: o/ | 16:00 |
| spzala | mvelten: you made it on time :) | 16:00 |
| mvelten | yes :D | 16:00 |
| spzala | :-) | 16:00 |
| mvelten | even 10mn in advance, wow | 16:00 |
| mvelten | hello all | 16:00 |
| spzala | lol | 16:00 |
| spzala | yeah, wow | 16:01 |
| spzala | bobh: there? | 16:01 |
| spzala | I had discussion with India folks in last two days offline so they may not join | 16:01 |
| spzala | let's see if bobh joins but meanwhile, we can start | 16:02 |
| spzala | mvelten: so get_param work, | 16:02 |
| spzala | I had a comment yesterday | 16:02 |
| spzala | did you see it? | 16:02 |
| mvelten | yes, let me remind it to myself :) | 16:02 |
| mvelten | you tested get_param on an actual Heat ? | 16:03 |
| spzala | : ) it all looks good, only worry is about Computer capability properties | 16:03 |
| spzala | yes I tested a small template, and it does gets value | 16:03 |
| spzala | but I didn't try the one with list | 16:03 |
| mvelten | ok | 16:04 |
| spzala | where we are setting get_param to list even for a single value, I believe it should work | 16:04 |
| spzala | but will need to test it | 16:04 |
| spzala | so as I was thinking more on handling it for OS and HOST, | 16:04 |
| spzala | I think it may not be too complicated | 16:04 |
| spzala | what we can do is, for each Compute node if any of OS and HOST properties has {get_input: x} then for that 'x' user input | 16:05 |
| spzala | we skip get_param and set the actual value | 16:05 |
| mvelten | I change that in my last patch, it should generate a non-list get_param wgen only one parameter is provided to not bloat the output too much | 16:06 |
| spzala | Awesome | 16:06 |
| spzala | and that's a typical case where we will only have a one param | 16:06 |
| spzala | thanks | 16:06 |
| spzala | does the approach to handle OS and HOST param looks good? | 16:07 |
| mvelten | ok fine for me ! for OS/HOST properties we perhaps need to remove it from the template parameters too | 16:07 |
| spzala | yes, we thought of it in the past, but I think it's OK not to worry about it | 16:08 |
| mvelten | hum this is tricky actually | 16:08 |
| spzala | as it doesn't hurt deployment | 16:08 |
| mvelten | ok fine for me, because on principle the parameter could be used somewhere else too | 16:08 |
| spzala | so we can leave those inputs in the template, deleting them can be done but why to add complexities (we can do it later with low priority) | 16:08 |
| spzala | yep but that's more like cleaning up TOSCA specific things | 16:09 |
| spzala | something we worry about later | 16:09 |
| spzala | once we have get_param patch of your merged, I believe that will really look good and simple | 16:10 |
| mvelten | did you have the time to look at my other patch "Map get_property to an actual property" ? | 16:10 |
| spzala | yes I did, you have added new templates per our discussion so that looks good.. let me quickly review it again today | 16:11 |
| spzala | and we can let it in | 16:11 |
| spzala | it looks really good not to see hard coded values any more | 16:11 |
| spzala | ohhh, yes one thing is I wanted to test it with Heat | 16:11 |
| spzala | like wordpress one | 16:11 |
| spzala | and then I will add my comments | 16:12 |
| spzala | sounds good? | 16:13 |
| spzala | Just heard from bobh that he is traveling to MWC in Europe | 16:14 |
| spzala | I guess it's in Spain | 16:14 |
| spzala | srinivas is also attending | 16:14 |
| spzala | mvelten: distracted? | 16:16 |
| mvelten | yes sorry :) give me 2mn | 16:17 |
| mvelten | ok finish reading :) fine let me know if something doesn't deploy | 16:17 |
| mvelten | I'll work on the get_param in the meantime to do what we said | 16:18 |
| spzala | :) no problem at all | 16:19 |
| spzala | OK, sounds like a plan | 16:19 |
| spzala | since MWC is in Spain, wondering are you going? | 16:20 |
| spzala | since Bob and Srinivas from our team is going to be there, good opportunity for you'll to meet if you are going | 16:20 |
| mvelten | nop unfortunatly, I should be in the OpenStack summit in Barcelona in 6 months | 16:21 |
| mvelten | if you can come it is a great city | 16:21 |
| spzala | OK. Yup, that would be great. I usually am going to all summits since last 5 summits so most likely will be there | 16:22 |
| spzala | but hey, how about Austin summit in two months? :) | 16:23 |
| spzala | I guess if our submission get enough vote you sure will be coming | 16:23 |
| mvelten | regarding get_param, do we settle on key_name as a specific param to inject ? or perhaps something like __key_name__ to avoid conflict to the maximum ? | 16:24 |
| spzala | so for key_name our approach is, something I talked to Bob who is working on it, - | 16:25 |
| spzala | we treat key_name as a special 'key' and if it's provided by user, we add a it under Compute | 16:26 |
| spzala | as get_param: {} | 16:26 |
| spzala | with your work, you can skip key_name part since Bob's updating that | 16:27 |
| spzala | sounds good? | 16:27 |
| mvelten | ok so we inject a get_param: keyname and add the keyname as a template parameter, with default set to a value if provided by the user ? | 16:28 |
| spzala | yup | 16:29 |
| spzala | to keep it simple, I would like to just add key_name in template with value provided by user at the translation time but | 16:30 |
| spzala | again that will not give opportunity to user to add key_name value at the time of deployment | 16:30 |
| spzala | but by adding 'input' dict with key_name it shouldn't add much complexity | 16:31 |
| spzala | mvelten: there? :-) | 16:39 |
| spzala | mvelten: I guess we can wrap up meeting if nothing else to discuss | 16:39 |
| mvelten | yes sorry :) all fine for me we can stop | 16:40 |
| spzala | mvelten: :) no problem at all | 16:41 |
| spzala | mvelten: :) OK, cool. Thanks! and have a nice evening. | 16:41 |
| spzala | mvelten: me stepping away for lunch | 16:41 |
| mvelten | nice day to you ! | 16:41 |
| spzala | thanks! | 16:41 |
| mvelten | enjoy | 16:42 |
| spzala | :) I will | 16:42 |
| *** spzala has quit IRC | 16:44 | |
| *** spzala has joined #openstack-heat-translator | 16:45 | |
| *** spzala_ has joined #openstack-heat-translator | 16:46 | |
| *** spzala has quit IRC | 16:50 | |
| *** spzala_ has quit IRC | 16:50 | |
| *** mvelten has left #openstack-heat-translator | 16:51 | |
| *** spzala has joined #openstack-heat-translator | 17:04 | |
| *** spzala has quit IRC | 17:08 | |
| *** tbh has joined #openstack-heat-translator | 17:13 | |
| *** sridhar_ram has joined #openstack-heat-translator | 17:13 | |
| *** spzala has joined #openstack-heat-translator | 17:19 | |
| *** bobh has joined #openstack-heat-translator | 17:21 | |
| *** spzala has quit IRC | 17:23 | |
| *** spzala has joined #openstack-heat-translator | 17:25 | |
| *** spzala has quit IRC | 17:29 | |
| *** spzala has joined #openstack-heat-translator | 17:31 | |
| *** spzala has quit IRC | 17:35 | |
| *** spzala has joined #openstack-heat-translator | 18:03 | |
| *** bobh has quit IRC | 18:25 | |
| *** spzala has quit IRC | 18:26 | |
| *** spzala has joined #openstack-heat-translator | 18:28 | |
| openstackgerrit | Sahdev Zala proposed openstack/heat-translator: Log warning if Compute cap property not provided https://review.openstack.org/281917 | 18:57 |
| *** spzala has quit IRC | 18:59 | |
| *** sridhar_ram1 has joined #openstack-heat-translator | 19:02 | |
| *** spzala has joined #openstack-heat-translator | 19:03 | |
| *** sridhar_ram has quit IRC | 19:04 | |
| *** spzala has quit IRC | 19:08 | |
| *** bobh has joined #openstack-heat-translator | 19:12 | |
| *** bobh has quit IRC | 19:26 | |
| *** bobh has joined #openstack-heat-translator | 19:27 | |
| *** spzala has joined #openstack-heat-translator | 19:28 | |
| *** spzala has quit IRC | 19:28 | |
| *** spzala has joined #openstack-heat-translator | 19:28 | |
| *** zaneb has quit IRC | 20:12 | |
| *** zaneb has joined #openstack-heat-translator | 20:12 | |
| *** bobh has quit IRC | 20:51 | |
| *** sridhar_ram1 has quit IRC | 21:03 | |
| *** sridhar_ram has joined #openstack-heat-translator | 21:04 | |
| *** sridhar_ram has quit IRC | 21:41 | |
| *** sridhar_ram has joined #openstack-heat-translator | 21:44 | |
| *** spzala has quit IRC | 22:08 | |
| *** sridhar_ram has quit IRC | 22:15 | |
| *** sridhar_ram has joined #openstack-heat-translator | 22:18 | |
| *** bobh has joined #openstack-heat-translator | 22:50 | |
| *** bobh has quit IRC | 23:13 | |
| *** bobh has joined #openstack-heat-translator | 23:22 | |
| *** bobh has quit IRC | 23:28 | |
| *** openstackgerrit has quit IRC | 23:47 | |
| *** openstackgerrit_ is now known as openstackgerrit | 23:47 | |
| *** openstackgerrit_ has joined #openstack-heat-translator | 23:48 | |
| *** openstackgerrit_ is now known as openstackgerrit | 23:48 | |
| *** openstackgerrit_ has joined #openstack-heat-translator | 23:49 | |
| *** openstackgerrit_ has quit IRC | 23:55 | |
| *** openstackgerrit_ has joined #openstack-heat-translator | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!