You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@ratis.apache.org by GitBox <gi...@apache.org> on 2021/01/19 11:48:09 UTC

[GitHub] [incubator-ratis] cchenax opened a new pull request #396: Support skip no permission storagedir with multiple volumes

cchenax opened a new pull request #396:
URL: https://github.com/apache/incubator-ratis/pull/396


   ## What changes were proposed in this pull request?
   
   Support skip no permission storagedir with multiple volumes
   
   ## What is the link to the Apache JIRA
   https://issues.apache.org/jira/browse/RATIS-1289
   
   ## How was this patch tested?
   
   I started the Ozone server which the ratis was replaced by my local changed ratis, when the one of volumes was only wrote by root authority,this volume would be skipped
   


----------------------------------------------------------------
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

[GitHub] [incubator-ratis] runzhiwang commented on pull request #396: RATIS-1289. Support skip the storagedir with multiple volumes which cause by bad disk

Posted by GitBox <gi...@apache.org>.
runzhiwang commented on pull request #396:
URL: https://github.com/apache/incubator-ratis/pull/396#issuecomment-763392809


   @amaliujia cc, this mainly fixed for bad disk, not permission, but the exception is same.


----------------------------------------------------------------
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

[GitHub] [incubator-ratis] runzhiwang closed pull request #396: RATIS-1289. Support skip the storagedir with multiple volumes which cause by bad disk

Posted by GitBox <gi...@apache.org>.
runzhiwang closed pull request #396:
URL: https://github.com/apache/incubator-ratis/pull/396


   


----------------------------------------------------------------
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

[GitHub] [incubator-ratis] runzhiwang merged pull request #396: RATIS-1289. Support skip the storagedir with multiple volumes which cause by bad disk

Posted by GitBox <gi...@apache.org>.
runzhiwang merged pull request #396:
URL: https://github.com/apache/incubator-ratis/pull/396


   


----------------------------------------------------------------
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

[GitHub] [incubator-ratis] amaliujia commented on pull request #396: RATIS-1289. Support skip the storagedir with multiple volumes which cause by bad disk

Posted by GitBox <gi...@apache.org>.
amaliujia commented on pull request #396:
URL: https://github.com/apache/incubator-ratis/pull/396#issuecomment-763398345


   +1 for bad disk it is ok.


----------------------------------------------------------------
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

[GitHub] [incubator-ratis] runzhiwang commented on pull request #396: RATIS-1289. Support skip the storagedir with multiple volumes which cause by bad disk

Posted by GitBox <gi...@apache.org>.
runzhiwang commented on pull request #396:
URL: https://github.com/apache/incubator-ratis/pull/396#issuecomment-763518965


   @cchenax Thanks the patch. @amaliujia Thanks for review. I have merged it.


----------------------------------------------------------------
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

[GitHub] [incubator-ratis] amaliujia commented on pull request #396: Support skip no permission storagedir with multiple volumes

Posted by GitBox <gi...@apache.org>.
amaliujia commented on pull request #396:
URL: https://github.com/apache/incubator-ratis/pull/396#issuecomment-763185165


   I think such permission issue shouldn't be handled by code. It is better to fix permission outside of Ratis.


----------------------------------------------------------------
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