You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by sw...@apache.org on 2016/04/21 22:41:07 UTC

[07/10] git commit: updated refs/heads/master to bebaea3

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 <wi...@gmail.com>


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 <wi...@gmail.com>
Authored: Thu Apr 21 16:36:50 2016 -0400
Committer: Will Stevens <wi...@gmail.com>
Committed: Thu Apr 21 16:36:50 2016 -0400

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