Skip to content
This repository has been archived by the owner on Sep 26, 2019. It is now read-only.

[PAN-2009] - Fix cluster clean start after stop in Acceptance tests #1546

Merged
merged 7 commits into from
Jun 11, 2019
Merged

[PAN-2009] - Fix cluster clean start after stop in Acceptance tests #1546

merged 7 commits into from
Jun 11, 2019

Conversation

usmansaleem
Copy link
Contributor

@usmansaleem usmansaleem commented Jun 11, 2019

PR description

Allow a cluster to be restarted after a stop.

  • Clear bootnodes on cluster start
  • Cleaner logic to pass bootnodes list only to non-boot nodes in cluster startNode
  • Do not shutdown pantheonNodeRunner from stop method
  • Additional call to stop method from close method in Cluster
  • Fix getBootnodes to setBootnodes in NodeConfiguration
  • Initialise PluginContextImpl once per node in ThreadPantheonNodeRunner
    (allows clean start after a close)
  • Stop plugins in stopNode and shutdown methods of ThreadPantheonNodeRunner
  • Add new acceptance test for a cluster to clean start after stop

usmansaleem and others added 3 commits June 11, 2019 14:05
 - Clear bootnodes on cluster start
 - Pass bootnodes list only to non-boot nodes in cluster startNode
 - Do not shutdown pantheonNodeRunner from stop method
 - Call stop from cluster close method
 - Fix getBootnodes to setBootnodes in NodeConfiguration
 - Init outputProcessorExecutor in startNode in ProcessPantheonNodeRunner
    (allows clean start after a close or shutdown)
 - Init PluginContextImpl once per node in ThreadPantheonNodeRunner
    (allows clean start after a close or shutdown)
 - Add new acceptance test for a cluster to clean start after stop
 - Allow PantheonPluginContextImpl to clean start plugins after stop

Signed-off-by: Usman Saleem <[email protected]>
@usmansaleem usmansaleem marked this pull request as ready for review June 11, 2019 04:38
Copy link
Contributor

@macfarla macfarla left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@ajsutton ajsutton left a comment

Choose a reason for hiding this comment

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

LGTM.

@usmansaleem usmansaleem merged commit 115ffab into PegaSysEng:master Jun 11, 2019
@usmansaleem usmansaleem deleted the bug/fix_acceptance_test_cluster_restart branch June 11, 2019 06:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants