New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
GEODE-4893: rework reloading the cluster configuration on existing va… #1656
Conversation
…nilla servers. * adding acceptance test to verify existing server is not shut down * instead of closing/recreate the cache instance, reload the cluster configuration xml/properties/jars.
568a021
to
20e8de7
Compare
"import cluster-configuration --zip-file-name=" + clusterConfigZipPath, "sleep --time=1", | ||
"list members").execute(gfsh); | ||
|
||
list_members.getOutputText() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suspect these are meant to be assertions?
|
||
list_members.getOutputText() | ||
.contains("Successfully applied the imported cluster configuration on server"); | ||
list_members.getOutputText().contains("serverNotShutDown |"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This will match on the start server
command since the script contains many commands. Split the start-up, the configuration import, and the list members
into separate gfsh scripts.
After speaking with Jinmei, I went ahead and made the test corrections on her behalf. |
Precheckin green, excepting a failure due to GEODE-4980. |
…nilla servers.
Need review on reloading the cluster configuration. Though initial set of tests show everything is in order. Not sure if all the elements in the cache.xml gets properly applied.