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/13 03:32:21 UTC

[GitHub] [ozone] bharatviswa504 commented on pull request #2000: HDDS-4915. [SCM HA Security] Integrate CertClient.

bharatviswa504 commented on pull request #2000:
URL: https://github.com/apache/ozone/pull/2000#issuecomment-797858821


   > @bharatviswa504 thanks for working on this. Can you separate the patch into smaller ones? For example
   > separate the bootstrap of the non-primary from this patch and focus only on the init of primary SCM.
   > Otherwise, the review will take longer time.
   
   @xiaoyuyao I understand it is a big change, sorry for the trouble. Bootstrap changes will be less than 100 LOC, but if it is still needed I can split it into a new Jira.  
   
   Actually, I have planned to implement CertClient and Integration in two parts, later when integrating found issues and closed the other one. Let me see if I can split again, but 2nd PR integration one will be dependent on this. Can we also have a call for a review?


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