[GitHub] [ozone] smengcl commented on pull request #2713: HDDS-5824. `ozone sh volume list` should print valid JSON array

[GitBox <gi...@apache.org>]

smengcl commented on pull request #2713:
URL: https://github.com/apache/ozone/pull/2713#issuecomment-935761493


   > Thanks @smengcl for the improvement.
   > 
   > Acceptance test failure is due to the output change (it already uses `jq` to verify output).
   > 
   > https://github.com/apache/ozone/blob/20e8429a86c4cf7bf0d960f3197e936f64be667c/hadoop-ozone/dist/src/main/smoketest/basic/ozone-shell-lib.robot#L37
   > 
   > Should we mention this change in release notes (it might break scripts as our tests)?
   
   Done. Good point. I should add a release note on the jira.
   
   Also I feel like I should do this to `bucket list` and `key list` as well, if that's not the case already.


-- 
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: issues-unsubscribe@ozone.apache.org

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



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