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

Tomaz Muraus edited comment on LIBCLOUD-333 at 5/30/13 3:09 AM:
----------------------------------------------------------------

Comments from LIBCLOUD-332 also apply here.

As far as the kwargs go, please directly access the dictionary value 
(kwargs[key]) instead of using pop. pop removes an item from a dictionary or 
returns a default value if key doesn't exist and a default value is provided.

Because of the way Python handlers kwargs, mutating kwargs dictionary won't 
have any bad side affects, but accessing dictionary in a normal way is 
preferred if there is no need to mutate the dictionary you operate on.
                
      was (Author: kami):
    Comments from LIBCLOUD-323 also apply here.

As far as the kwargs go, please directly access the dictionary value 
(kwargs[key]) instead of using pop. pop removes an item from a dictionary or 
returns a default value if key doesn't exist and a default value is provided.

Because of the way Python handlers kwargs, mutating kwargs dictionary won't 
have any bad side affects, but accessing dictionary in a normal way is 
preferred if there is no need to mutate the dictionary you operate on.
                  
> CloudStack driver does not deal with SSH KeyPairs
> -------------------------------------------------
>
>                 Key: LIBCLOUD-333
>                 URL: https://issues.apache.org/jira/browse/LIBCLOUD-333
>             Project: Libcloud
>          Issue Type: Bug
>          Components: Compute
>    Affects Versions: 0.12.3
>         Environment: trunk
>            Reporter: sebastien goasguen
>
> there are no extension functions for managing ssh key pairs.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to