GitHub user karuturi opened a pull request:
https://github.com/apache/cloudstack/pull/849
Cloudstack-8816 some of the events do not have resource uuids
The 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]
------------------------------------------------------------------------
```
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/karuturi/cloudstack CLOUDSTACK-8816-take2
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/cloudstack/pull/849.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 #849
----
commit fbdcd04a1950b2374a25993aef324fe52527909c
Author: Rajani Karuturi <rajani.karut...@citrix.com>
Date: 2015-09-18T09:23:11Z
CLOUDSTACK-8816: some of the events do not have resource uuids
the key for an entity is sometimes an object a String with value
object.toString() due to serialization and deserialization of them.
Addressed this in the getter of CallContext to check for key.toString
if an object is not found with key.
commit 2b6a0b363c0518f9571dd1ff092784f3fde92a95
Author: Rajani Karuturi <rajani.karut...@citrix.com>
Date: 2015-09-18T09:26:02Z
CLOUDSTACK-8816: some of the events do not have resource uuids
uuid is missing in the first event of VM create as the entity is just
created and never put in the Context.
Added the entity uuid to context on successful creation.
----
---
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 infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---
