LZD-PratyushBhatt commented on PR #5312: URL: https://github.com/apache/ozone/pull/5312#issuecomment-1727623307
> Thanks @LZD-PratyushBhatt for the change. Current patch looks good . Another good to have test-case is when the configured OM service id's are greater than 1 and the command is run without a serviceId an exception is expected. It might be tricky to run this from a robot test but I believe we can override the parameter ozone.om.service.ids in the command to achieve this. Please check if this works. > > ```shell > ozone admin --set=ozone.om.service.ids=om1,om2 om roles > ``` Hi @sadanand48 , added the patch, please review. Thanks! -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
