[ 
https://issues.apache.org/jira/browse/ARIA-210?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16001338#comment-16001338
 ] 

ASF GitHub Bot commented on ARIA-210:
-------------------------------------

Github user ran-z commented on a diff in the pull request:

    https://github.com/apache/incubator-ariatosca/pull/128#discussion_r115325816
  
    --- Diff: aria/cli/service_template_utils.py ---
    @@ -53,7 +53,7 @@ def get(source, service_template_filename):
                 return _get_service_template_file_from_archive(source, 
service_template_filename)
    --- End diff --
    
    What about adding a short test or two for these scenarios? :)


> Relative paths may cause issues in `aria service-templates` commands
> --------------------------------------------------------------------
>
>                 Key: ARIA-210
>                 URL: https://issues.apache.org/jira/browse/ARIA-210
>             Project: AriaTosca
>          Issue Type: Bug
>            Reporter: Ran Ziv
>            Assignee: Avia Efrat
>
> The CLI commands {{aria service-templates store}} and {{aria 
> service-templates create-archive}} use the {{os.path.dirname}} method to get 
> the directory of the main service-template file.
> In the former ({{store}} command) this causes errors when the user is already 
> in the directory of the main service-template file and passes the file 
> directly (e.g. {{aria service-templates store service-template.yaml 
> my-service-template}})) - It'll fail when trying to copy the 
> service-template's resources like so:
> {{cannot copy tree '': not a directory}}
> In the latter ({{create-archive}}), any relative path usage will result in an 
> error claiming the file does not exist.



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

Reply via email to