| *** bobh has joined #openstack-heat-translator | 00:41 | |
| *** jruano has quit IRC | 00:43 | |
| *** sridhar_ram has quit IRC | 01:46 | |
| *** jruano has joined #openstack-heat-translator | 01:47 | |
| *** bobh has quit IRC | 02:00 | |
| *** bobh has joined #openstack-heat-translator | 02:10 | |
| *** topol has joined #openstack-heat-translator | 02:51 | |
| *** topol has quit IRC | 03:10 | |
| *** spzala has quit IRC | 03:34 | |
| *** bobh has quit IRC | 03:54 | |
| *** jruano has quit IRC | 03:54 | |
| *** topol has joined #openstack-heat-translator | 03:56 | |
| *** topol has quit IRC | 04:12 | |
| *** topol has joined #openstack-heat-translator | 04:13 | |
| *** topol has quit IRC | 05:07 | |
| openstackgerrit | Vahid Hashemian proposed openstack/heat-translator: Fix translation issue with interface inputs and functions https://review.openstack.org/206295 | 05:20 |
|---|---|---|
| *** topol has joined #openstack-heat-translator | 05:31 | |
| *** topol has quit IRC | 05:35 | |
| *** vahidh has quit IRC | 07:29 | |
| *** openstackgerrit has quit IRC | 07:46 | |
| *** openstackgerrit has joined #openstack-heat-translator | 07:46 | |
| *** jruano has joined #openstack-heat-translator | 08:43 | |
| *** jruano has quit IRC | 09:14 | |
| *** topol has joined #openstack-heat-translator | 10:15 | |
| *** topol has quit IRC | 10:19 | |
| *** vahidh has joined #openstack-heat-translator | 10:49 | |
| *** vahidh has quit IRC | 10:54 | |
| *** bobh has joined #openstack-heat-translator | 12:21 | |
| *** bobh has quit IRC | 12:25 | |
| *** spzala has joined #openstack-heat-translator | 12:31 | |
| *** topol has joined #openstack-heat-translator | 12:36 | |
| *** topol has quit IRC | 12:41 | |
| *** vahidh has joined #openstack-heat-translator | 13:32 | |
| *** vahidh has quit IRC | 13:37 | |
| *** jruano has joined #openstack-heat-translator | 13:57 | |
| *** topol has joined #openstack-heat-translator | 14:03 | |
| *** bobh has joined #openstack-heat-translator | 14:26 | |
| *** sridhar_ram has joined #openstack-heat-translator | 14:36 | |
| *** vahidh has joined #openstack-heat-translator | 14:50 | |
| spzala | Hi all | 15:00 |
| vahidh | o/ | 15:00 |
| spzala | vahidh: o/ | 15:00 |
| spzala | who is around? srinivas: meena: ? | 15:01 |
| vahidh | they don't seem to be online | 15:01 |
| *** srinivas has joined #openstack-heat-translator | 15:02 | |
| srinivas | Hi Sahdev & Vahidh | 15:03 |
| spzala | vahidh: true but not any more :-) | 15:03 |
| spzala | Hey Srinivas | 15:03 |
| vahidh | Hi Srinivas | 15:03 |
| spzala | I haven't get chance to request to get into openstack meeting due to some other priorities but will do soon.. next week will give more time to me to work on that | 15:04 |
| vahidh | Sounds good. Thanks for the effort :) | 15:04 |
| spzala | but related good news is my patch on logging this channel is approved and merged so logging is in place now | 15:04 |
| srinivas | Hi Vahid. thanks for cyclic import reference.. very well explained | 15:04 |
| spzala | :-) np | 15:05 |
| spzala | Good Vahid. | 15:05 |
| vahidh | spzala: perfect. great news | 15:05 |
| vahidh | srinivas: no problem. glad it helped | 15:05 |
| srinivas | good news | 15:05 |
| spzala | vahidh: srinivas: yep :-) .. if you share the cyclic ref. email with me that would be good..help me with reviewing that patch. | 15:06 |
| spzala | http://eavesdrop.openstack.org/irclogs/%23openstack-heat-translator/ just fyi | 15:06 |
| spzala | Alright, so going with what I have as an agenda | 15:06 |
| topol | Hi spzala, quick suggestion | 15:07 |
| vahidh | spzala: you can find the link in the reviews of the patch. I added it in my comment there | 15:07 |
| spzala | topol: Hi, sure | 15:07 |
| topol | So when you start a meeting can you type in everyones name that typically attends. That way we all get pinged when you start | 15:07 |
| topol | just copy paste it from somewhere | 15:07 |
| spzala | vahidh: Cool, I will get it from there .. thought that was in regard to Srinivas's email on imports | 15:07 |
| spzala | topol: sure good idea | 15:08 |
| spzala | topol: will do ;) | 15:08 |
| topol | thanks. Im easily distracted | 15:08 |
| spzala | topol: yeah understood :) | 15:08 |
| spzala | Alright, so going with agenda I have (once we are in meeting channel it will be on-line but for now bear with me) | 15:09 |
| spzala | Pypi packages for parser and translator - since we have reached a good milestone i.e. supporting parsing for CSAR (and CSAR support for translation which may require some tests in progress) | 15:10 |
| spzala | and there is a need from user to have it, I have created pypi for parser yesterday | 15:10 |
| spzala | https://pypi.python.org/pypi/tosca-parser/0.2.0 | 15:10 |
| vahidh | perfect. thanks Sahdev. | 15:11 |
| spzala | For translator - it looks good overall based on Vahid and my testing. And Vahid is adding tests for CSAR in translator. Once that's done we shooting to have pypi for translator by end of week this week | 15:12 |
| spzala | vahidh: thanks and thanks for heavy lifting on CSAR | 15:12 |
| topol | spzala, did you have tagging issue? did you get that fixed? | 15:12 |
| topol | spazala I saw some email | 15:12 |
| spzala | topol: actually, it's not an issue at all | 15:13 |
| srinivas | sahdev & vahid, sounds good | 15:13 |
| topol | spzala oh goodie! | 15:13 |
| spzala | topol: yup like if you see here https://pypi.python.org/pypi/python-heatclient | 15:13 |
| vahidh | spzala: no problem, it was fun :) | 15:13 |
| spzala | you notice a similar duplication look like message but I figure that, it's due to README and it's common | 15:14 |
| topol | Good job vahidh! you are the ceasar of CSAR | 15:14 |
| spzala | LOL agree with topol | 15:14 |
| vahidh | topol: :) thanks | 15:14 |
| spzala | vahidh: :) glad to know it | 15:14 |
| spzala | srinivas: thanks | 15:14 |
| spzala | well going on with round table before we have open discussion. srinivas: good suggestion on a_file flag | 15:15 |
| srinivas | sahdev & vahid, i am thinking like it is better to have a small after reaching some milestone | 15:15 |
| srinivas | demo | 15:15 |
| spzala | srinivas: vahid: I have replied that email and it makes sense to me... will wait on Vahid's thought on it | 15:16 |
| vahidh | srinivas: I agree with your suggestion on a_file | 15:16 |
| vahidh | the thing is when I created that patch it was all under heat-translator | 15:16 |
| vahidh | so a_file was internal | 15:16 |
| spzala | srinivas: having demo right away speically on a quick release is difficult, but I meant to send and email announcement but I wanted to check on possible tagging issue before that | 15:16 |
| srinivas | demo after implementing a big feature is a good idea. So everyone understands the usecase. What do all say? | 15:17 |
| vahidh | when we broke it into two patches I missed the fact that a_file is now exposed | 15:17 |
| spzala | srinivas: tagging issue as topol mentioned which I resolved just 10 mins before meeting | 15:17 |
| srinivas | not immediately.. planned demos . like sprint demo | 15:17 |
| spzala | vahidh: yes internal and in CLI it's not needed to be provided by user but programmatically it does require (e.g. as we have in test methods) so good to handle it and remove that need in future | 15:18 |
| spzala | srinivas: yes that make sense | 15:18 |
| spzala | srinivas: we will be trying to have something ready around summit time | 15:18 |
| srinivas | ok sahdev. thanks | 15:19 |
| vahidh | spzala: yes I agree, but again this will impact heat-translator since it makes call to ToscaTemplate, so will have to plan carefully | 15:19 |
| spzala | srinivas: yrw | 15:19 |
| spzala | vahidh: that's true, we will handle it so that no backward compatibility is broken | 15:20 |
| vahidh | spzala: yes, we need to take that into consideration | 15:20 |
| spzala | vahidh: I am not suggesting any changes in 0.2.0 but in future. | 15:20 |
| vahidh | spzala: yes, understood :) | 15:20 |
| spzala | vahidh: yup and this should be easy to handle in parser side :) | 15:21 |
| vahidh | spzala: yes, I'm thinking just need to move the code that detects the format from heat-translator to tosca-parser | 15:21 |
| srinivas | Hi All, What is the issue? | 15:22 |
| spzala | vahidh: sure. | 15:22 |
| vahidh | srinivas: we are discussing how to handle the a_file issue that you brought up | 15:22 |
| srinivas | i could not able to follow | 15:22 |
| srinivas | ok. | 15:22 |
| spzala | srinivas: it's handling URL in toscatemplate class | 15:22 |
| spzala | I think we are good on that. Good suggestion srinivas | 15:23 |
| spzala | moving on | 15:23 |
| srinivas | yrw | 15:23 |
| spzala | srinivas: how's going overall? anything from you? policy? group? imports? | 15:23 |
| srinivas | imports.. i already sent patch for review. | 15:23 |
| spzala | srinivas: I know all under progress and for imports I guess new patch coming soon | 15:23 |
| spzala | srinivas: nice | 15:24 |
| srinivas | I will try for next patch with cyclic imports by tomorrow | 15:24 |
| spzala | srinivas: yeah, that's what I meant by new patch .. cool | 15:24 |
| spzala | srinivas: other than that patch looks good | 15:24 |
| srinivas | btw.. next 2 weeks i might be busy.. I am applying US BV next week.. might be travelling on oct11 to San jose | 15:24 |
| spzala | srinivas: sure, np vahidh is in valley not sure if you guys can meet f2f but good if can :-) | 15:25 |
| vahidh | srinivas: nice, business trip? | 15:25 |
| srinivas | yes. | 15:25 |
| srinivas | please provide your contact numbers.. If possible, we will meet | 15:25 |
| vahidh | yup, we can try to set up something to meet face to face if time allows | 15:26 |
| srinivas | will be there in san jose for 1 week | 15:26 |
| vahidh | just email me when you're here and we can take it from there | 15:26 |
| spzala | srinivas: I guess meena and others will continue contribution? or they traveling too? just wondering? | 15:26 |
| srinivas | no.. they will continue.. | 15:27 |
| srinivas | sahdev, I too will continue in next week.. whenever i find some free time | 15:27 |
| srinivas | ;) | 15:27 |
| spzala | srinivas: I am in North Carolina (NC) just in case you have any personal trip on this side of US :) | 15:27 |
| spzala | srinivas: :-) COOL | 15:27 |
| spzala | vahid: and I are trying to finish translator side changes with new parser pypi in next two days. anything else from you vahid: ? | 15:28 |
| vahidh | spzala: not really, you covered it well. I just add that I can now start submitting murano patches that were waiting on pypi release | 15:29 |
| spzala | vahidh: coo, yup, once we have pypi for translator this week getting back on Murano sounds like a plan | 15:30 |
| vahidh | spzala: yes :) | 15:31 |
| spzala | :-) | 15:31 |
| spzala | open discussion? any thing anyone wants to talk? | 15:31 |
| spzala | with our small crowd 30mins meeting time seems perfect, but I will keep it for an hour just in case. ;) | 15:31 |
| srinivas | Vahid, can you please brief me on Murano. May be some presentation on next week.. including CSAR.. I don't have idea on these | 15:31 |
| vahidh | srinivas: murano is another opestack project mainly run by Mirantis. it adds an application catalog to openstack | 15:32 |
| vahidh | it supports HOT and MuranoPL as the two formats for specifying applications that you add to the catalog | 15:33 |
| vahidh | we want to add TOSCA to its supported formats | 15:33 |
| vahidh | so you can for example take the wordpress or elk CSAR templates and import them into murano application catalog for deployment | 15:34 |
| srinivas | Vahidh.. can you plan small demo on this next week.. may be 10mins should be good enough | 15:34 |
| vahidh | srinivas: not sure, I don't have all the pieces ready yet | 15:35 |
| srinivas | ok. whenever we have vahidh.. | 15:35 |
| *** sridhar_ram has quit IRC | 15:35 | |
| vahidh | sure, definitely | 15:35 |
| spzala | vahidh: sounds good. | 15:36 |
| spzala | if nothing else, then we call this a meeting. | 15:37 |
| vahidh | srinivas: this is the spec for that work: https://review.openstack.org/#/c/194422/ | 15:37 |
| srinivas | ok vahidh. i will go through it. | 15:37 |
| srinivas | Sahdev, expecting demo on CSAR also :-) | 15:38 |
| spzala | :-) sure | 15:38 |
| spzala | Vahid and I have a techtalk submitted for summit, if it goes fine we will have it as part of it but regardless we will create..good to have it. | 15:39 |
| spzala | aite, I guess we call this a meeting. The channel is open so we can continue informal discussion as we need. Thanks again all!! | 15:41 |
| vahidh | thanks, good meeting | 15:41 |
| spzala | yup :-) | 15:41 |
| srinivas | I am leaving the meeting. bye to Sahdev, Vahidh & Topol | 15:43 |
| vahidh | bye srinivas | 15:43 |
| *** srinivas has quit IRC | 15:44 | |
| spzala | srinivas: good night! | 15:44 |
| *** jruano has quit IRC | 15:53 | |
| *** sridhar_ram has joined #openstack-heat-translator | 15:55 | |
| openstackgerrit | Sahdev Zala proposed openstack/heat-translator: Update README https://review.openstack.org/230005 | 16:26 |
| *** zaneb has quit IRC | 16:38 | |
| *** bobh has quit IRC | 17:00 | |
| *** jruano has joined #openstack-heat-translator | 17:03 | |
| *** bobh has joined #openstack-heat-translator | 17:15 | |
| openstackgerrit | Vahid Hashemian proposed openstack/heat-translator: Fix translation issue with interface inputs and functions https://review.openstack.org/206295 | 17:21 |
| *** topol has quit IRC | 17:47 | |
| *** jruano has quit IRC | 18:00 | |
| *** topol has joined #openstack-heat-translator | 18:16 | |
| *** topol has quit IRC | 18:20 | |
| *** sridhar_ram has quit IRC | 18:30 | |
| *** openstackgerrit has quit IRC | 18:31 | |
| *** openstackgerrit has joined #openstack-heat-translator | 18:31 | |
| *** topol has joined #openstack-heat-translator | 18:32 | |
| *** topol has quit IRC | 18:38 | |
| openstackgerrit | Merged openstack/heat-translator: Fix translation issue with interface inputs and functions https://review.openstack.org/206295 | 19:08 |
| *** zaneb has joined #openstack-heat-translator | 19:18 | |
| *** topol has joined #openstack-heat-translator | 19:34 | |
| openstackgerrit | Sahdev Zala proposed openstack/heat-translator: Update README https://review.openstack.org/230005 | 19:46 |
| *** topol has quit IRC | 20:00 | |
| openstackgerrit | Vahid Hashemian proposed openstack/heat-translator: Handle URLs for input templates and imports https://review.openstack.org/213392 | 20:02 |
| openstackgerrit | Merged openstack/heat-translator: Update trove classifier https://review.openstack.org/229604 | 20:48 |
| *** sridhar_ram has joined #openstack-heat-translator | 20:55 | |
| openstackgerrit | Vahid Hashemian proposed openstack/heat-translator: Handle URLs for input templates and imports https://review.openstack.org/213392 | 21:16 |
| *** bobh has quit IRC | 21:44 | |
| *** spzala has quit IRC | 21:54 | |
| *** spzala has joined #openstack-heat-translator | 21:56 | |
| spzala | vahidh: there? | 21:59 |
| spzala | vahidh_: there? :-) | 22:00 |
| vahidh | Hi spzala | 22:00 |
| spzala | Hey, my sametime is not connecting so just wanted ping that if need to chat anything I am here :-) | 22:00 |
| spzala | I am reviewing your updated patch | 22:01 |
| vahidh | np :) I just submitted a follow-on patch for the URL input to address your comments | 22:01 |
| vahidh | thanks. just saw your second line :) | 22:02 |
| spzala | :-) yrw | 22:04 |
| *** vahidh_ has quit IRC | 22:06 | |
| spzala | vahidh: it looks great, the only thing I see is for this file translator/tests/data/tosca_single_instance_wordpress_with_local_abspath_import.yaml - there is no unit test ? | 22:08 |
| spzala | I know we have a similar file and this one only has abs_path vs relative path - but in order to have all templates tested good to test this one | 22:09 |
| vahidh | there is one: test_hot_translate_template_by_url_with_local_abspath_import | 22:10 |
| vahidh | that uses an uploaded version of that file | 22:10 |
| vahidh | will have to update the link to github once the patch merges | 22:10 |
| spzala | Ok, cool..thx. So that particular test | 22:13 |
| spzala | exception | 22:14 |
| spzala | still has toscalib :-) | 22:14 |
| spzala | is it not getting tested? the exception? | 22:14 |
| *** topol has joined #openstack-heat-translator | 22:14 | |
| spzala | vahidh: ^^ :) | 22:16 |
| spzala | vahidh: line 379 ..no rush when you get chance | 22:17 |
| vahidh | sorry, the reason toscalib didn't matter there was the import path was given as an absolute path, and for URLs absolute reference doesn't make sense | 22:17 |
| vahidh | so it fails before even trying the import | 22:17 |
| vahidh | let me check | 22:17 |
| vahidh | do you mean: | 22:18 |
| vahidh | raise Exception( | 22:19 |
| vahidh | 'The unit test that was expected to fail did not fail.') | 22:19 |
| vahidh | ? | 22:19 |
| spzala | ok, right but then the assert really doesnt matter ? | 22:19 |
| vahidh | aah! now I see what you mean | 22:19 |
| spzala | no I was wondering, | 22:19 |
| vahidh | shoot. sorry :) | 22:19 |
| *** topol has quit IRC | 22:19 | |
| spzala | ok :-) | 22:19 |
| spzala | :-) np at all | 22:19 |
| vahidh | will have to resubmit | 22:20 |
| spzala | OK, sure np.. also before you resubmit | 22:20 |
| spzala | one more thing | 22:20 |
| vahidh | sure | 22:21 |
| spzala | translator/tests/data/tosca_single_instance_wordpress_with_local_abspath_import.yaml what I was saying about test here is - we seems doesn't have test that is testing the output of this template as HOT ? | 22:21 |
| vahidh | yes, I kind of thought that would be redundant because we are just playing around with imports (relative/absolute file paths and urls) | 22:22 |
| vahidh | if you want I can add one quickly | 22:22 |
| spzala | something we have for other templates usually | 22:22 |
| spzala | Yup, that's exactly what I was saying earlier | 22:22 |
| vahidh | I'll add it | 22:23 |
| spzala | so I totally understand, so the only thing is import statement but still we want to make sure that all of our templates are working out of box and we have covered in test | 22:23 |
| vahidh | makes sense. sure, should be quick adding it | 22:24 |
| spzala | yup, I would have done same as you did but like you said it's just an additional hot file | 22:24 |
| spzala | :-) | 22:24 |
| spzala | OK, thanks | 22:24 |
| vahidh | :) | 22:24 |
| vahidh | ooh, I know why I didn't add that one | 22:30 |
| vahidh | it's because I cannot know what the absolute path to the import wordpress.yaml will be | 22:31 |
| vahidh | it would be different depending on where it's running | 22:31 |
| vahidh | makes sense spzala? | 22:32 |
| spzala | vahidh: hmmm... so won't it always be /translator/tests/data/custom_types/wordpress.yaml if you use that particular template? | 22:33 |
| spzala | to me, that's why you have provided abs path? | 22:33 |
| vahidh | no, on my machine it's /home/stack/workspace/heat-translator/translator/tests/data/custom_types/wordpress.yaml | 22:33 |
| spzala | ahh true, well in that case it's still a relative path to the project and not an abs path. | 22:34 |
| vahidh | if I use that path on my machine it works, but I don't know what it would look like on jenkins server | 22:34 |
| spzala | simialr to custom_types/wordpress.yaml | 22:34 |
| vahidh | well, I didn't know what to use there. the fact that it starts with a '/' would make it an absolute path | 22:35 |
| spzala | true, not only jenkins but also anyone's local machine once project was downloaded | 22:35 |
| vahidh | true | 22:35 |
| vahidh | but the feature is there | 22:35 |
| vahidh | for anyone who wants to try it locally | 22:35 |
| spzala | OK, well then in the description of template can you please add something like "this template is a test template and uses an absolute path to the project" something like that if that makes sense and you think we convey the right purpose to the user? | 22:38 |
| spzala | or let's not add it .. I am thinking | 22:38 |
| spzala | if we can't test in Jenkin's evn. I feel it difficult to add | 22:38 |
| vahidh | if we add it and someone tests it against the hot_Wordpress it would fail since now the description is different | 22:39 |
| vahidh | but since we are not checking the output here we can add it, jenkins won't complain :) | 22:39 |
| spzala | vahidh: lol | 22:40 |
| vahidh | add it then? | 22:40 |
| spzala | well, no let's not do that .. i would rather change description and create a new hot_* yaml and compare it | 22:40 |
| spzala | what do you think? | 22:40 |
| spzala | or what if we don't change the description | 22:41 |
| spzala | but still add test to compare against hot_Wordpress | 22:41 |
| vahidh | sure, I think the name of the file gives some sort of a clue | 22:41 |
| vahidh | we can't add test, remember the absolute path thing would fail on jenkins | 22:41 |
| spzala | sure so if create unit test and test against existing hot_Wordpress that will work right? in that case, I am fine keeping it as it is and just add that test method | 22:42 |
| spzala | I think that covers the testing well | 22:42 |
| vahidh | the problem is we can't have a unit test for this one | 22:42 |
| vahidh | I can make it working on my machine, but when you download it there is no guarantee it would work on yours | 22:43 |
| spzala | ok, gotcha.. you said that I thought you were saying second time that it will work | 22:43 |
| vahidh | unless you have the file in exactly the same OS path as I do | 22:43 |
| spzala | well, then let's not add it at all | 22:43 |
| vahidh | nope, sorry if I confused you | 22:44 |
| spzala | that doesn't cause any problem | 22:44 |
| spzala | oh no.. no problem at all | 22:44 |
| vahidh | ok, then no change with respect to that file (just removing toscalib references) | 22:44 |
| spzala | as it is, it's just a hanging template for sample purpose I guess? | 22:44 |
| vahidh | yes, plus we have a unit test for it that would fail | 22:45 |
| vahidh | none that would succeed | 22:45 |
| spzala | OK, even we have unit test that would succeed (as failure) taht's fine but I don't see in your patch where you are using "tosca_single_instance_wordpress_with_local_abspath_import.yaml" file anywhere? | 22:46 |
| spzala | unless I am not searching well :( | 22:46 |
| vahidh | yes, because I'm using the URL for it (a box url that would change to github url later on) | 22:47 |
| spzala | the test method you said earlier doesn't have this file | 22:47 |
| vahidh | if you go to that URL inside the test you'll get the same file | 22:47 |
| spzala | I understood but where I am confused is: | 22:47 |
| spzala | ok so you are not referencing this file staright in test but the URL you have used is pointing to this file? | 22:48 |
| vahidh | yes | 22:48 |
| spzala | ahhh | 22:48 |
| spzala | sorry i didn't get that part earlier | 22:48 |
| vahidh | because I wanted to cover the case where the template is in a URL and it's referencing an import via absolute path, and we need to disallow it | 22:49 |
| spzala | OK, gotcha.. well, in this case you can sure modify the description right? | 22:49 |
| vahidh | yes | 22:49 |
| vahidh | I'll add a line there to explain the absolute path | 22:50 |
| spzala | Perfect, then I would modify description that this is a test file used in URL cover this particular test you mentioned. | 22:50 |
| spzala | Great.. my fear is someone will use it on CLI or something and complain that it failed | 22:50 |
| vahidh | yes, that could happen :) | 22:51 |
| spzala | :-) yup, and having description will cover us there I believe | 22:51 |
| spzala | so we are good with it then | 22:51 |
| vahidh | yes, I'll submit shortly | 22:51 |
| spzala | OK sounds great..thanks vahidh, | 22:54 |
| vahidh | np spzala | 22:54 |
| vahidh | should I change the path /translator/tests/data/custom_types/wordpress.yaml to something like /tmp/wordress.yaml? | 22:55 |
| vahidh | or it's good as is? | 22:55 |
| spzala | vahidh: yes that's a good idea | 22:55 |
| spzala | let's have a path which can also be self readable in abs path sense | 22:56 |
| vahidh | sure, thanks | 22:56 |
| spzala | np, thank you :-) | 22:56 |
| vahidh | np :) | 22:56 |
| openstackgerrit | Vahid Hashemian proposed openstack/heat-translator: Handle URLs for input templates and imports https://review.openstack.org/213392 | 22:58 |
| spzala | vahidh: thanks for the quick update looks Great ..only small thing like 379 | 23:07 |
| spzala | vahidh: shouldn't that change to /tmp/wordpress.yaml now? | 23:08 |
| spzala | s/line 379/like 379 | 23:08 |
| vahidh | yes :) | 23:09 |
| vahidh | not sure why it passed on my local machine | 23:09 |
| vahidh | good catch as always | 23:09 |
| spzala | :-) sure | 23:09 |
| spzala | thanks.. it's minor.. I thought it passed because again due to some URL work you are going to do later? as you mentioned in commit. | 23:10 |
| vahidh | aah, I should have updated the uploaded version too! | 23:10 |
| spzala | :-) OK, thanks vahidh!!! | 23:11 |
| vahidh | np :) thanks for catching it | 23:12 |
| spzala | :-) np, that was easy.. your quick updates aren't :-) | 23:12 |
| openstackgerrit | Vahid Hashemian proposed openstack/heat-translator: Handle URLs for input templates and imports https://review.openstack.org/213392 | 23:12 |
| vahidh | no they are easy too :) | 23:13 |
| vahidh | I hope this time we got them all | 23:13 |
| spzala | :-) nah but well. It's great this time. | 23:14 |
| spzala | :-) | 23:14 |
| vahidh | :) | 23:14 |
| openstackgerrit | Merged openstack/heat-translator: Handle URLs for input templates and imports https://review.openstack.org/213392 | 23:28 |
| openstackgerrit | Vahid Hashemian proposed openstack/heat-translator: Add translation of CSAR archives https://review.openstack.org/230206 | 23:42 |
| *** bobh has joined #openstack-heat-translator | 23:59 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!