cloudstack-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From r...@apache.org
Subject [7/7] git commit: updated refs/heads/master to 535ab51
Date Tue, 27 Oct 2015 18:27:41 GMT
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 <github@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 <github@remi.nl>
Authored: Tue Oct 27 19:26:44 2015 +0100
Committer: Remi Bergsma <github@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(-)
----------------------------------------------------------------------



Mime
View raw message