GitHub user miguelaferreira opened a pull request:
https://github.com/apache/cloudstack/pull/722
Fixing syntax error introduced in PR #717
Commit 0062ff2672e257f2a4290e054e23ef4333a34983 in PR #717 introduced a new
parameter in two constructors. However, there is a test class
(CitrixRequestWrapperTest) that uses these constructors and the respective
calls needed to be updated with the new parameters.
Testing:
- ran full maven build: `mvn clean install -P developer,systemvm`
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/miguelaferreira/cloudstack fixing-pr717
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/cloudstack/pull/722.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #722
----
commit 2a6c7f3ad2d045ffebba4744238359a5c6dc29bb
Author: Rohit Yadav <[email protected]>
Date: 2015-08-19T10:01:37Z
CLOUDSTACK-8748: VM UUID accessible in CreateVMSnapshotCommand and
RevertToVMSnapshotCommand
This patch makes it possible to expose VM UUID to subsystems, this can be
useful for implementing VM Snapshots for KVM in future.
Signed-off-by: Rohit Yadav <[email protected]>
commit 0bb7051b647caf95c989f1b1d3efe788262aab60
Author: Miguel Ferreira <[email protected]>
Date: 2015-08-20T07:56:32Z
Formatting class CitrixRequestWrapperTest
commit 551c55cef728f3add69b030b44fdcd1e892b42a0
Author: Miguel Ferreira <[email protected]>
Date: 2015-08-20T07:55:55Z
Add UUIDs to constructors (fixing syntax error from 2a6c7f3)
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
