ChenSammi commented on pull request #934: URL: https://github.com/apache/hadoop-ozone/pull/934#issuecomment-635042923
> > > @ChenSammi Thank you for your review. > > > can you explain in what situation this command is useful? > > The situation is that i want to close all pipeline and new some pipeline with my specify replication factor and replication type, so that, i can test which factor is supported by Ozone and ratis. And this command is very useful for me, i can't imagine without this command, how many works will i do to close pipeline one by one. > > > This command is too powerful, how can we prevent user from mis-operation? > > In fact, this command is the combine operation of list pipelines and close them, without this command, we also can do the same operation by writing a bash script to achieve the same goal. > And, pipeline is already admin command, i think we can have a high level general works we should do is make the admin commands safe, such as verify the role of the user. The following JIRA to track this issue https://issues.apache.org/jira/browse/HDDS-3617 @maobaolong , thanks for the explaination. It it's only for the test purpose, I stronlgy suggest use a moderate way to achieve this. One is as @nandakumar131 suggested, allow close pipeline command to accept a list of pipeline IDs or a file of pipeline IDs list. Another is switch on/off the current background pipeline creation process for production/test. A mis-operation of close all pipeline action in produciton environement can lead serious impact, no matter it's an admin or not. ---------------------------------------------------------------- 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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
