You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ozone.apache.org by GitBox <gi...@apache.org> on 2021/03/12 18:00:49 UTC
[GitHub] [ozone] elek opened a new pull request #2036: HDDS-4904. Remove mention of CSI support
elek opened a new pull request #2036:
URL: https://github.com/apache/ozone/pull/2036
## What changes were proposed in this pull request?
Thanks for @arp7 for the suggestion to document better the current state of Ozone CSI driver and the guarantees what it provides.
Patch includes some additional warning and changing the title image with adding alpha tag to the CSI/K8s icon.
Please note that there is an ongoind discussion about this approach on the Jira and on the mailing list. I created the PR only to show my proposal, do not merge it before any consensus.
## What is the link to the Apache JIRA
https://issues.apache.org/jira/browse/HDDS-4904
## How was this patch tested?
`hugo server` in `hadoop-hdds/docs`
----------------------------------------------------------------
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
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org
[GitHub] [ozone] elek commented on pull request #2036: HDDS-4904. Remove mention of CSI support
Posted by GitBox <gi...@apache.org>.
elek commented on pull request #2036:
URL: https://github.com/apache/ozone/pull/2036#issuecomment-809128718
Merging it. Thanks the review @arp7 and @dineshchitlangia
--
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
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org
[GitHub] [ozone] arp7 commented on pull request #2036: HDDS-4904. Remove mention of CSI support
Posted by GitBox <gi...@apache.org>.
arp7 commented on pull request #2036:
URL: https://github.com/apache/ozone/pull/2036#issuecomment-797669405
I am +1 on this. I will hold off as you requested, please feel free to commit though. Thank you!
----------------------------------------------------------------
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
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org
[GitHub] [ozone] elek merged pull request #2036: HDDS-4904. Remove mention of CSI support
Posted by GitBox <gi...@apache.org>.
elek merged pull request #2036:
URL: https://github.com/apache/ozone/pull/2036
--
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
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org
[GitHub] [ozone] dineshchitlangia commented on a change in pull request #2036: HDDS-4904. Remove mention of CSI support
Posted by GitBox <gi...@apache.org>.
dineshchitlangia commented on a change in pull request #2036:
URL: https://github.com/apache/ozone/pull/2036#discussion_r593832721
##########
File path: hadoop-hdds/docs/content/interface/CSI.md
##########
@@ -24,6 +24,11 @@ summary: Ozone supports Container Storage Interface(CSI) protocol. You can use O
limitations under the License.
-->
+<div class="alert alert-warning" role="alert">
+
+Ozone CSI support still in alpha phase and buckets can be mounted only via 3rd party S3 compatible Fuse implementation (like Goofys). Fuse over S3 can provide only limited performance compared to a native Fuse file systems. Long-term Ozone may support a custom solution to mount buckets which provides better user experience (with fuse or NFS or any other solution). Until that CSI is recommended to use only if you can live with this limitation and your use case is tested carefully.
+</div>
Review comment:
```suggestion
Ozone CSI support is still in alpha phase and buckets can be mounted only via 3rd party S3 compatible Fuse implementation (like Goofys). Fuse over S3 can provide only limited performance compared to a native Fuse file system. Long-term Ozone may support a custom solution to mount buckets which provides better user experience (with fuse or NFS or any other solution). Until that CSI is recommended to use only if you can live with this limitation and your use case is tested carefully.
</div>
```
----------------------------------------------------------------
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
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org
[GitHub] [ozone] elek commented on pull request #2036: HDDS-4904. Remove mention of CSI support
Posted by GitBox <gi...@apache.org>.
elek commented on pull request #2036:
URL: https://github.com/apache/ozone/pull/2036#issuecomment-804946168
@dineshchitlangia NIT is fixed. I assume we are good to go, are we? (note I also added 436fc0c)
--
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
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscribe@ozone.apache.org
For additional commands, e-mail: issues-help@ozone.apache.org