ChenSammi commented on code in PR #5404:
URL: https://github.com/apache/ozone/pull/5404#discussion_r1375685819
##########
hadoop-ozone/ozone-manager/src/main/java/org/apache/hadoop/ozone/om/OzoneManager.java:
##########
@@ -647,6 +647,11 @@ private OzoneManager(OzoneConfiguration conf,
StartupOption startupOption)
omStorage, omInfo, "",
scmInfo == null ? null : scmInfo.getScmId(),
this::saveNewCertId, this::terminateOM);
+ if (recoverCertificate) {
+ certClient.initWithRecovery();
Review Comment:
@fapifta , I think the public key and certificate are the public part of the
security material. So the auto recovery of public key and certificate are just
fine. This recoverCertificate is introduced just for test. Otherwise, all the
secure integration tests which require OM startup and running, need to prepare
an normal SCM and start it. The introduce of recoverCertificate flag is to
simply these integration tests. I don't aware that it will affect the private
materials. Can you explain it a bit more?
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
