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

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

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

    https://github.com/apache/incubator-ariatosca/pull/138#discussion_r119428083
  
    --- Diff: aria/orchestrator/workflows/events_logging.py ---
    @@ -35,20 +35,20 @@ def _get_task_name(task):
     
     @events.start_task_signal.connect
     def _start_task_handler(task, **kwargs):
    -    # If the task has not implementation this is an empty task.
    -    if task.implementation:
    +    # If the task has no function this is an empty task.
    +    if task.function:
             suffix = 'started...'
             logger = task.context.logger.info
         else:
    -        suffix = 'has no implementation'
    +        suffix = 'has no function'
    --- End diff --
    
    actually, in this specific instance, it should probably remain 
`implementation` - function is an implementation (sorry for the confusing term 
in this context :D) detail, but in practice what the message means is that the 
user didn't input (there i go with the terms again.. :D) any implementation for 
the operation (AKA empty operation)


> Support instrinsic functions in "dependencies" operation configuration
> ----------------------------------------------------------------------
>
>                 Key: ARIA-149
>                 URL: https://issues.apache.org/jira/browse/ARIA-149
>             Project: AriaTosca
>          Issue Type: Story
>            Reporter: Tal Liron
>            Assignee: Tal Liron
>
> There are a few issues here. First, the following YAML is totally broken;
> {code}
>               dependencies:
>                 - ssh.address > { get_attribute: virtual_ip.floating_ip }
> {code}
> The problem is that in YAML, due to the location of the ":", this gets parsed 
> as a dict where the key is {code}ssh.address > { get_attribute{code} and the 
> value is {code}virtual_ip.floating_ip }{code} which is not what we want at 
> all.
> The solution is to encase the whole value in quotes to enforce parsing as a 
> string:
> {code}
>               dependencies:
>                 - "ssh.address > { get_attribute: virtual_ip.floating_ip }"
> {code}
> Note that the one attractive solution is to to change this ">" format to a 
> real dict. So:
> {code}
>               dependencies:
>                 - { ssh.address: { get_attribute: virtual_ip.floating_ip } }
> {code}
> But this will break other TOSCA parsers that expect a string here, so no go. 
> :(



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

Reply via email to