GitHub user ProjectMoon opened a pull request:
https://github.com/apache/cloudstack/pull/1111
Fix event UUIDS missing on event bus
The fixing of CLOUDSTACK-8816 introduced a regression that removed the
first class entities in the event bus description property. This is because
everything was changed to use the Class as a key... Everything but the
populateFirstClassEntities method in ActionEventUtils.
This commit tries to load the class key instead of the String key, which
re-enables the populateFirstClassEntities method.
Likely this was not caught because of the trace exception handling... maybe
some better logging/unit tests would be good for this PR.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/greenqloud/cloudstack pr-eventbus-entity-uuids
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/cloudstack/pull/1111.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 #1111
----
commit a528c396d718f0ea6eecbdda7c23c8d7f670d99c
Author: jeff <[email protected]>
Date: 2015-11-23T17:15:57Z
Fix event UUIDS missing on event bus
The fixing of CLOUDSTACK-8816 introduced a regression that removed the
first class entities in the event bus description property. This is
because everything was changed to use the Class as a key... Everything
but the populateFirstClassEntities method in ActionEventUtils.
----
---
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.
---
