You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cloudstack.apache.org by re...@apache.org on 2015/10/27 19:27:41 UTC
[7/7] git commit: updated refs/heads/master to 535ab51
Merge pull request #849 from karuturi/CLOUDSTACK-8816-take2
Cloudstack-8816 some of the events do not have resource uuidsThe key objects in the context map are sometimes String and sometimes object. This causes missing uuids when an entity put in the context map with key entity.toString is queried with key entity
Testing:
manually tested by deploying a vm and checked that the created events in rabbitmq now has uuids.
events before and after the change are update at https://issues.apache.org/jira/browse/CLOUDSTACK-8816?focusedCommentId=14805239
unittests
```
$ mvn -pl :cloud-api test -Dtest=CallContextTest
-------------------------------------------------------
T E S T S
-------------------------------------------------------
Running org.apache.cloudstack.context.CallContextTest
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.152 sec - in org.apache.cloudstack.context.CallContextTest
Results :
Tests run: 1, Failures: 0, Errors: 0, Skipped: 0
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 11.445 s
[INFO] Finished at: 2015-09-18T14:58:53+05:30
[INFO] Final Memory: 55M/448M
[INFO] ------------------------------------------------------------------------
```
* pr/849:
CLOUDSTACK-8816 added missing events
CLOUDSTACK-8816: fixed missing resource uuid in delete network cmd
CLOUDSTACK-8816: fixed missing resource uuid in destroy vm event
Cloudstack-8816: Fixed missing resource uuid in delete snapshot events
CLOUDSTACK-8816: some of the events do not have resource uuids
CLOUDSTACK-8816: some of the events do not have resource uuids
Signed-off-by: Remi Bergsma <gi...@remi.nl>
Project: http://git-wip-us.apache.org/repos/asf/cloudstack/repo
Commit: http://git-wip-us.apache.org/repos/asf/cloudstack/commit/535ab51b
Tree: http://git-wip-us.apache.org/repos/asf/cloudstack/tree/535ab51b
Diff: http://git-wip-us.apache.org/repos/asf/cloudstack/diff/535ab51b
Branch: refs/heads/master
Commit: 535ab51b9ad829ed1bd25973b913d4f731440ad5
Parents: fb4e6ed 2c0af1f
Author: Remi Bergsma <gi...@remi.nl>
Authored: Tue Oct 27 19:26:44 2015 +0100
Committer: Remi Bergsma <gi...@remi.nl>
Committed: Tue Oct 27 19:26:44 2015 +0100
----------------------------------------------------------------------
.../cloudstack/api/ApiCommandJobType.java | 3 +-
.../command/user/network/DeleteNetworkCmd.java | 10 +++
.../apache/cloudstack/context/CallContext.java | 13 ++-
.../cloudstack/context/CallContextTest.java | 83 ++++++++++++++++++++
server/src/com/cloud/api/ApiDBUtils.java | 16 ++--
.../src/com/cloud/user/AccountManagerImpl.java | 6 ++
server/src/com/cloud/vm/UserVmManagerImpl.java | 1 +
7 files changed, 122 insertions(+), 10 deletions(-)
----------------------------------------------------------------------