You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@knox.apache.org by GitBox <gi...@apache.org> on 2020/04/30 19:42:35 UTC

[GitHub] [knox] pzampino opened a new pull request #333: KNOX-2368 - CM Cluster Configuration Monitor Does Not Support Rolling…

pzampino opened a new pull request #333:
URL: https://github.com/apache/knox/pull/333


   … Restart Events
   
   ## What changes were proposed in this pull request?
   
   Added cluster config monitor support for rolling restarts of individual services and complete clusters. This also fixes a previously-introduced defect whereby the monitor would recognize relevant events whose status is STARTED, which would result in false-positive inclusion of events that would trigger configuration analysis. All the relevant commands have status value of SUCCEEDED once they've successfully completed. Since the "Rolling Restart" option is disabled in the CM cluster against which I've tested, the implementation and associated test of that behavior is entirely based on log content that exposed the need for this PR.
   
   ## How was this patch tested?
   
   Refactored and augmented PollingConfigurationAnalyzerTest, and manually tested all the supported scenarios (except rolling CLUSTER restart).


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org