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(-)
----------------------------------------------------------------------