elek opened a new pull request #1149:
URL: https://github.com/apache/hadoop-ozone/pull/1149


   ## What changes were proposed in this pull request?
   
   _emphasized text_om.serviceId is required on case of OM.HA in all the client 
parameters even if there is only one om.serviceId and it can be chosen.
   
   My goal is:
   
    1. Provide better usability
    2. Simplify the documentation task ;-)
   
   With using the om.serviceId from the config if 
   
    1. config is available
    2. om ha is configured 
    3. only one service is configured
   
   It also makes easier to run the same tests with/without HA
   
   ## What is the link to the Apache JIRA
   
   https://issues.apache.org/jira/browse/HDDS-3878?filter=12349091
   
   ## How was this patch tested?
   
   1. Got full green build on my fork.
   2. ozone-ha acceptance tests
   
   ozone-ha acceptance tests are turned off long time ago (it included some 
life-cycle test to start/stop services. In this patch I simplified the `ha` 
cluster and added simple smoketest.
   
   Later we can restore the lifecycle tests (start/stop) but I would prefer to 
use a generic approach for all the clusters.


----------------------------------------------------------------
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:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: ozone-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: ozone-issues-h...@hadoop.apache.org

Reply via email to