Merge pull request #1501 from mike-tutkowski/cs-9354

Fixing an issue in Marvin around creating a template from a snapshotThis fixes 
the following ticket: https://issues.apache.org/jira/browse/CLOUDSTACK-9354

The problem was that Marvin was requiring you to pass in the "ispublic" 
parameter when creating a template from a snapshot.

As the ticket notes, this issue was introduced by the following commit: 
https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a=commit;h=bbe0fc4be9527d51820b067a602886003991db4d

The solution I've provided is simply to check if the "ispublic" property is in 
the dictionary before referencing it.

* pr/1501:
  CLOUDSTACK-9354 - Fixing an issue in Marvin around creating a template from a 
snapshot (if “is public” is not provided, there was a problem)

Signed-off-by: Will Stevens <[email protected]>


Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/7cd2284b
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/7cd2284b
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/7cd2284b

Branch: refs/heads/master
Commit: 7cd2284bf554baac4d66783fdf1f59daee32b377
Parents: f530a4c ba9a61a
Author: Will Stevens <[email protected]>
Authored: Thu Apr 21 16:36:50 2016 -0400
Committer: Will Stevens <[email protected]>
Committed: Thu Apr 21 16:36:50 2016 -0400

----------------------------------------------------------------------
 tools/marvin/marvin/lib/base.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


Reply via email to