Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

Topology init #487

Closed
wants to merge 12 commits into from
Closed

Conversation

ravihansa3000
Copy link
Contributor

No description provided.

@@ -176,7 +188,7 @@ private void executeCoordinatorTasks() {
componentStartUpSynchronizer.addEventListener(new ComponentActivationEventListener() {
@Override
public void activated(Component component) {
if (component == Component.StratosManager) {
if (component == Component.CloudController) {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What could be the reason for this change?

@imesh
Copy link
Contributor

imesh commented Nov 9, 2015

Great work Akila! Few points:

  • Did we remove existing complete topology publishing logic? May be this can be removed now.
  • Did we update PCA to work with this model?

Thanks

@lahirugmg
Copy link
Contributor

Hi Imesh,

Do we plan to remove existing, periodically published complete topology?

Thanks.

@ravihansa3000
Copy link
Contributor Author

We cannot remove existing, periodical complete topology event publish task. Because there might situations where CC is not active (maintenance restarts, failures etc.) and if clients publish complete topology request event, it might get ignored.

@ravihansa3000
Copy link
Contributor Author

I've updated PCA to use this initialization mechanism. Also updated PCA live test to verify the functionality.

@lahirugmg
Copy link
Contributor

Thanks for the info Akila.

@imesh
Copy link
Contributor

imesh commented Nov 30, 2015

@akila: Did we merged this to master branch? There seems to be conflicts in this PR now.
BTW it might be better to add a short description about the changes you did.
Thanks

@ravihansa3000
Copy link
Contributor Author

Refer to mail thread on Stratos dev list with subject "[Dev] Publishing Complete Topology event on Demand" for more information on changes done.

Merged in 60b8011

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
3 participants