| *** spzala has quit IRC | 00:15 | |
| *** openstackgerrit has quit IRC | 07:50 | |
| *** openstackstatus has quit IRC | 10:13 | |
| *** openstack has joined #openstack-heat-translator | 10:17 | |
| *** tbh is now known as tbh_afk | 11:40 | |
| *** topol has quit IRC | 11:55 | |
| *** topol_ has joined #openstack-heat-translator | 11:55 | |
| *** tbh_afk has quit IRC | 13:46 | |
| *** bobh has joined #openstack-heat-translator | 13:48 | |
| *** tbh_afk has joined #openstack-heat-translator | 13:55 | |
| *** spzala has joined #openstack-heat-translator | 14:09 | |
| *** spzala has quit IRC | 15:19 | |
| *** spzala has joined #openstack-heat-translator | 15:52 | |
| *** spzala has quit IRC | 15:53 | |
| *** spzala has joined #openstack-heat-translator | 15:53 | |
| spzala | Hi guys | 16:00 |
|---|---|---|
| spzala | bobh: topol_: tbh: tbh_: aloga: Happy New Year! | 16:00 |
| *** bobh has quit IRC | 16:01 | |
| spzala | sridhar_ram: good morning | 16:22 |
| *** tbh_afk is now known as tbh | 16:35 | |
| tbh | hi spzala | 16:35 |
| tbh | sorry for the late | 16:35 |
| spzala | tbh: no worries, I saw you 'afk' status :-) | 16:36 |
| spzala | tbh: seems like folks are still in new year mood :) | 16:36 |
| spzala | tbh: and there wasn't much to discuss | 16:36 |
| tbh | spzala, still the inertia is there | 16:36 |
| spzala | :-_ | 16:37 |
| spzala | :-) | 16:37 |
| spzala | The priority is release 0.7.0 and the main issue for now is to confirm from sridhar_ram that https://bugs.launchpad.net/heat-translator/+bug/1652928 is not a problem | 16:38 |
| openstack | Launchpad bug 1652928 in Heat Translator "Fails to translate the derived polices" [High,New] - Assigned to Sahdev Zala (spzala) | 16:38 |
| spzala | tbh: other than that it looks good so far | 16:38 |
| tbh | spzala, any response from kanagarajM | 16:39 |
| spzala | tbh: nope :( | 16:39 |
| tbh | spzala, just now he commented | 16:40 |
| tbh | in the bug page | 16:40 |
| spzala | tbh: ahhh nice let me check | 16:40 |
| spzala | tbh: cool, hopefully we will hear from him soon | 16:44 |
| spzala | tbh: anything else that you want to discuss? | 16:45 |
| spzala | tbh: If you can grab the latest HT and TP and try run tox with Tacker now when you get chance (today or tomorrow) that would be very helpful. | 16:45 |
| spzala | tbh: so that if anything comes up we can fix it before Monday | 16:46 |
| tbh | spzala, sure | 16:46 |
| spzala | tbh: I am also testing with tox and few manual tests | 16:46 |
| tbh | spzala, nothing much from my side | 16:46 |
| spzala | tbh: thanks Bharath! Just fyi https://bugs.launchpad.net/heat-translator/+bug/1641734 | 16:47 |
| openstack | Launchpad bug 1641734 in Heat Translator "get_operation_output translation is broken" [High,New] - Assigned to Sahdev Zala (spzala) | 16:47 |
| spzala | that's the bug I ran into while testing HT and TP for 0.7.0 few days back | 16:47 |
| spzala | I have taken care of it for now and will be fixing it right after TP 0.7.0 release | 16:47 |
| spzala | before HT release so that Tacker don't run into any long failing issue (it's just one test case) | 16:48 |
| spzala | just giving you heads up | 16:48 |
| tbh | spzala, oh you already worked on it | 16:48 |
| tbh | cool | 16:48 |
| spzala | tbh: sounds great ... thanks again and ttyl :-) I am gonna go for early lunch today :) | 16:49 |
| tbh | spzala, sure ttyl | 16:49 |
| *** spzala has quit IRC | 17:02 | |
| *** bobh has joined #openstack-heat-translator | 17:02 | |
| *** spzala has joined #openstack-heat-translator | 17:02 | |
| *** spzala_ has joined #openstack-heat-translator | 17:04 | |
| *** spzala has quit IRC | 17:07 | |
| *** bobh has quit IRC | 17:08 | |
| *** spzala_ has quit IRC | 17:08 | |
| *** bobh has joined #openstack-heat-translator | 18:03 | |
| *** spzala has joined #openstack-heat-translator | 18:04 | |
| *** bobh has quit IRC | 18:08 | |
| tbh | spzala, ping | 18:54 |
| spzala | tbh: Hi | 19:07 |
| tbh | spzala, currently tacker uses TOSCATranslator().translate() | 19:07 |
| tbh | spzala, this makes the unit tests fail | 19:08 |
| spzala | tbh: ah | 19:08 |
| tbh | spzala, once I convert from translate to output_to_yaml() | 19:08 |
| tbh | all tests are passing | 19:08 |
| spzala | tbh: that's cool | 19:09 |
| spzala | tbh: thinking but I think I would change the output_to_yaml to translate | 19:09 |
| spzala | tbh: on a side note, I am going to look for another issues kanagaraj mentioned in the bug in my non-tacker environment | 19:10 |
| spzala | tbh: I know you are busy but if you get chance please look at his latest comment in bug | 19:11 |
| spzala | tbh: and if you think of anything please let me know | 19:11 |
| tbh | spzala, changing to translate make life easy from tacker point of view :) | 19:11 |
| spzala | tbh: you can add fix too if you it | 19:11 |
| tbh | spzala, sure will look into that | 19:11 |
| tbh | spzala, currently leaving from office | 19:12 |
| spzala | tbh: :) sure and translate is a legacy name too | 19:12 |
| spzala | tbh: sure, no worries whenever and if you get chance | 19:12 |
| spzala | tbh: back to home? | 19:12 |
| tbh | spzala, yeah just starting now | 19:13 |
| spzala | tbh: OK cool | 19:13 |
| spzala | tbh: thanks!! and be safe. | 19:13 |
| tbh | spzala, sure, ttyl | 19:13 |
| spzala | tbh: tty later when get back on irc | 19:14 |
| *** tbh has quit IRC | 19:14 | |
| *** tbh has joined #openstack-heat-translator | 19:58 | |
| *** bobh has joined #openstack-heat-translator | 20:05 | |
| *** bobh has quit IRC | 20:09 | |
| tbh | spzala, ping | 20:11 |
| spzala | tbh: pong | 20:12 |
| spzala | tbh: that was quick :-) | 20:12 |
| tbh | spzala, just came :) | 20:12 |
| tbh | spzala, I just saw the comment | 20:13 |
| spzala | tbh: :) cool | 20:13 |
| tbh | spzala, one of the comment is not having output | 20:13 |
| spzala | tbh: oh the bug comment - sure the one on output didn't make sense to me | 20:13 |
| tbh | spzala, but outputs section is not there in tosca template | 20:13 |
| spzala | tbh: because ... yes | 20:13 |
| spzala | tbh: exactly | 20:13 |
| spzala | tbh: the other one seems valid do to me except odd thing is description has null something I was thinking to fix already | 20:14 |
| spzala | tbh: but though it looks valid, as he mentioned it shouldn't be produced in this translation at all | 20:15 |
| spzala | tbh: I guess as he said it's alarm wasn't mentioned in the template | 20:15 |
| tbh | spzala, ok, the other comment I am not much aware of whether to include or not ... or it is specific req from tacker | 20:15 |
| spzala | tbh: well what I understood is he said the tacker template doesn't have anything for Alarm so it shouldn't created in the translated template | 20:16 |
| spzala | tbh: seems more like valid to me if that's the case | 20:17 |
| tbh | spzala, but more of our testcases I can see we are not mentioning any alarm in tosca | 20:17 |
| tbh | spzala, like this https://github.com/openstack/heat-translator/blob/3c07e41264faccc08dca3e8b58ff745cd42a4191/translator/tests/data/autoscaling/tosca_autoscaling.yaml | 20:17 |
| spzala | tbh: true | 20:18 |
| tbh | spzala, but one more issue | 20:19 |
| spzala | tbh: if I recall, while translation we found that for triggers to fire we need alram | 20:20 |
| spzala | tbh: sure .. :) ? | 20:20 |
| tbh | spzala, if we add outputs section in the same template, I can see the outputs in the parent template but not in SP1_res.yaml | 20:20 |
| spzala | tbh: on alarm, I don't see trigger in his template so we shouldn't produce Alarm I guess that's what he says? | 20:21 |
| tbh | spzala, is that expected or do you think it will work? | 20:21 |
| tbh | spzala, yes | 20:21 |
| spzala | tbh: to me that looks like valid scenario, the use of 'outputs' is to provide user whatever the template author wants to display | 20:22 |
| tbh | spzala, ok | 20:22 |
| spzala | tbh: and the having output in parent can serve that purpose well so seems reasonable to me | 20:22 |
| spzala | tbh: could be wrong .. need to check more | 20:23 |
| tbh | spzala, then it's fine | 20:23 |
| tbh | :) | 20:23 |
| spzala | tbh: what did you use to add outputs? | 20:23 |
| spzala | tbh: :) | 20:23 |
| spzala | tbh: but I guess what I said makes sense in general | 20:23 |
| spzala | tbh: since I am not aware of Tacker nfv types attributes | 20:23 |
| tbh | spzala, http://paste.openstack.org/show/594050/ | 20:23 |
| spzala | tbh: nice | 20:24 |
| tbh | spzala, output http://paste.openstack.org/show/594051/ | 20:24 |
| spzala | tbh: cool | 20:25 |
| spzala | tbh: or outputs I am not sure why he expected it in the child template | 20:25 |
| spzala | *for | 20:25 |
| tbh | yes | 20:26 |
| spzala | tbh: let me reply him on that | 20:26 |
| spzala | tbh: I didn't looked into the second issue much ..thought I will fix the 'translate' issue you mentioned first | 20:27 |
| spzala | tbh: and going to update a patch on it in few mins | 20:27 |
| spzala | tbh: so that if we ask him to try any fix he can use translate as such | 20:27 |
| tbh | spzala, makes sense | 20:28 |
| spzala | tbh: :) thanks.. getting a call at work but will get back to you soon | 20:28 |
| tbh | sure | 20:29 |
| *** tbh has quit IRC | 20:31 | |
| *** tbh has joined #openstack-heat-translator | 20:34 | |
| spzala | tbh: back .. | 20:51 |
| tbh | :) | 20:52 |
| spzala | tbh: :) | 21:02 |
| *** tbh has quit IRC | 21:05 | |
| *** openstackgerrit has joined #openstack-heat-translator | 21:51 | |
| openstackgerrit | Sahdev Zala proposed openstack/heat-translator: Provide support for legacy translate api https://review.openstack.org/417204 | 21:51 |
| openstackgerrit | Sahdev Zala proposed openstack/heat-translator: Provide support for legacy translate api https://review.openstack.org/417204 | 22:15 |
| openstackgerrit | Sahdev Zala proposed openstack/heat-translator: Do no create Alarm when triggers not part of policies https://review.openstack.org/417240 | 23:36 |
| *** spzala has quit IRC | 23:46 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!