[GitHub] [hadoop-ozone] elek commented on pull request #863: HDDS-3461. Add acceptance test to smoketest CSI service startup

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

elek commented on pull request #863:
URL: https://github.com/apache/hadoop-ozone/pull/863#issuecomment-618885558


   >  I'd prefer adding csi container to the base ozone environment instead of duplication, but we can discuss that later.
   
   Yeah, I don't know what is the best approach. It seems to be an optional component. Earlier we had a conversation about creating `ozone-minimal` and `ozone-full` containers, but since that, we already added **recon** and *s3g** to the normal cluster as required components.
   
   I think **recon** should be required, s3g is more like a very frequently used optional component. csi seems to be more optional, that's the reason why I created a separated cluster, but I am open to merge it with anything else. (In theory I agree to provide less cluster) 
   
   I will merge it as is to have this safety line as soon as possible, but open to move in any direction.
   
   Thanks the review @adoroszlai 


----------------------------------------------------------------
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: ozone-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: ozone-issues-help@hadoop.apache.org