Friday, 2017-01-06

*** bobh has joined #openstack-heat-translator00:29
*** spzala has joined #openstack-heat-translator00:44
*** zaneb has quit IRC01:13
*** aloga has quit IRC01:13
*** zaneb has joined #openstack-heat-translator01:14
*** aloga has joined #openstack-heat-translator01:14
*** tbh has joined #openstack-heat-translator02:44
*** bobh has quit IRC02:56
*** bobh has joined #openstack-heat-translator03:00
*** bobh has quit IRC03:01
*** bobh has joined #openstack-heat-translator03:01
*** tbh has quit IRC03:11
*** spzala has quit IRC03:13
*** bobh has quit IRC03:50
*** spzala has joined #openstack-heat-translator03:52
*** bobh has joined #openstack-heat-translator03:57
*** spzala has quit IRC04:07
*** bobh has quit IRC04:32
*** spzala has joined #openstack-heat-translator06:08
*** spzala has quit IRC06:12
*** tarun has joined #openstack-heat-translator06:44
tarunHi, i am creating a heat template where i boot a new instance and attach floating ip to it. Now i want this floating ip to be the output but i am facing issues with the syntax as most of the examples on net tells how to get the private ip of instance and not floating ip attached. Can anyone help how to get floating ip as output of heat template06:48
*** openstackgerrit has quit IRC08:31
*** trozet has quit IRC08:31
*** zaneb has quit IRC08:31
*** topol_ has quit IRC08:31
*** aloga has quit IRC08:31
*** aloga has joined #openstack-heat-translator08:37
*** zaneb has joined #openstack-heat-translator08:37
*** openstackgerrit has joined #openstack-heat-translator08:37
*** topol_ has joined #openstack-heat-translator08:37
*** trozet has joined #openstack-heat-translator08:37
*** spzala has joined #openstack-heat-translator09:09
*** spzala has quit IRC09:13
*** bobh has joined #openstack-heat-translator13:03
*** spzala has joined #openstack-heat-translator13:10
*** spzala has quit IRC13:14
*** bobh has quit IRC13:19
*** spzala has joined #openstack-heat-translator13:54
*** bobh has joined #openstack-heat-translator14:04
openstackgerritMerged openstack/heat-translator: Provide support for legacy translate api  https://review.openstack.org/41720414:06
*** bobh has quit IRC14:11
*** topol_ is now known as topol14:44
openstackgerritMathieu Velten proposed openstack/heat-translator: Handle the writing of side files in a generic way in the stdout case  https://review.openstack.org/41740014:46
*** tbh has joined #openstack-heat-translator14:47
openstackgerritMathieu Velten proposed openstack/heat-translator: Handle the writing of side files in a generic way in the stdout case  https://review.openstack.org/41740014:47
*** bobh has joined #openstack-heat-translator15:08
openstackgerritMathieu Velten proposed openstack/heat-translator: Handle the writing of side files in a generic way in the stdout case  https://review.openstack.org/41740015:12
*** bobh has quit IRC15:12
openstackgerritMerged openstack/heat-translator: Handle the writing of side files in a generic way in the stdout case  https://review.openstack.org/41740015:26
*** tarun has quit IRC15:54
openstackgerritSahdev Zala proposed openstack/heat-translator: Do no create Alarm when triggers not part of policies  https://review.openstack.org/41724015:54
openstackgerritSahdev Zala proposed openstack/heat-translator: Do no create Alarm when triggers not part of policies  https://review.openstack.org/41724015:56
*** bobh has joined #openstack-heat-translator16:07
spzalatbh: ping17:31
spzalatbh: https://review.openstack.org/#/c/417240/ Kanagaraj has +1ed it but asked for tacker template test, I tried creating one with exact tacker way (outside tacker).. no rush but when you get time please take a look17:32
tbhspzala, I didn't get his comment17:54
spzalatbh: in the patch17:55
spzala(not in the bug :))17:55
tbhspzala, yes17:56
spzalatbh: or you can wait till he review this again.. it's a simple patch I think17:56
spzalatbh: oh you mean you didn't understand it ?17:56
tbhspzala, we need to test the sample he provided in bug page with this patch?17:56
spzalatbh: true and I think that's what is in the patch17:56
spzalatbh: lemme double check17:57
spzalatbh: at one point I removed few things from original templ to test it non-tacker env17:57
tbhspzala, UT passed?17:58
spzalatbh: https://review.openstack.org/#/c/417240/3/translator/tests/data/nfv/test_tosca_nfv_autoscaling.yaml17:58
spzalatbh: yes17:58
tbhspzala, got it, we have tacker_defs.yaml in data folder now17:59
spzalatbh: the patch is not unnecessary creating alarm and seems like he was good with changes but wanted to add this template as you mentioned17:59
spzalatbh: yes, so his template and the two type definitions that's needed for it are all in the tests/data/nfv18:00
tbhspzala, as jenkins gave +1 we are good to go. But do you think is it ok to test tacker functionality from heat-translator side?18:00
spzalatbh: yep +1 is good to go to me too18:00
tbhspzala, I will check this manually , give me a moment18:01
spzalatbh: yes, this actually is testing heat-translator only18:01
tbhspzala, cool then18:01
spzalatbh: but as you noticed in a similar fashion to tacker, only diff is tacker programmatically/implicitly adds import and we in HT as this test template shows using import explicitely18:02
spzalatbh: that's my understanding :-)18:02
tbhspzala, yup that's right. makes sense18:02
spzalatbh: cool :) thanks!!18:03
spzalatbh: I am heading for lunch already 1pm .. but will be back in 1 hr or around18:04
tbhspzala, mean while I will test it from my side18:04
tbhspzala, and paste my results here18:04
spzalatbh: OK sounds great!!18:05
spzalatbh: bye for now18:05
*** spzala has quit IRC18:05
*** spzala has joined #openstack-heat-translator18:06
*** spzala has quit IRC18:10
tbhHi Sahdev, tested manually, worked as expected18:15
*** tbh has quit IRC18:38
*** tbh has joined #openstack-heat-translator19:34
*** spzala has joined #openstack-heat-translator19:42
spzalatbh: back now.. thanks for the review!19:43
tbhspzala, np, I tested with tox again with latest code from my side19:44
spzalatbh: awesome! thanks, qq did u test tosca-parser as well?19:44
tbhspzala, I commented tosca-parser and ht in requirements.txt19:45
spzalatbh: I m sure you tested HT with latest tosca-parser but wondering if there is need to just run tox with latest tosca-parser19:45
tbhand installed HT and TP manually and ran tox19:45
spzalatbh: OK, so that should use the latest parser and HT19:45
spzalatbh: Cool19:46
tbh(by sourcing for each pyenv)19:46
spzalatbh: sure so you did latest of both :-)19:46
tbhspzala, yup19:46
spzalatbh: nice19:46
spzalatbh: I can take much needed nap now on my desk after indian buffet :-)19:47
tbhspzala, haha19:47
spzalatbh: somebody did wish be 'Nappy new new' (with a typo)19:47
tbh:-)19:47
spzalatbh: :-)19:48
*** tbh has quit IRC20:26
*** spzala has quit IRC20:37
*** spzala has joined #openstack-heat-translator20:45
*** spzala has quit IRC20:49
*** spzala has joined #openstack-heat-translator21:04
*** spzala has quit IRC21:05
*** spzala has joined #openstack-heat-translator21:05
openstackgerritMerged openstack/heat-translator: Do no create Alarm when triggers not part of policies  https://review.openstack.org/41724021:48
*** spzala has quit IRC21:56
*** bobh has quit IRC22:35
*** spzala has joined #openstack-heat-translator23:41
*** bobh has joined #openstack-heat-translator23:45

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