Skip to content

Conversation

@szetszwo
Copy link
Contributor

What changes were proposed in this pull request?

Currently SCM HA is supported only for fresh installations. SCM HA must be enabled when starting the Ozone service in the beginning. After a SCM is started in non-HA mode, it cannot be easily changed to HA mode. We should document this clearly.

What is the link to the Apache JIRA

https://issues.apache.org/jira/browse/HDDS-7254

How was this patch tested?

NA

@adoroszlai adoroszlai requested a review from errose28 October 17, 2022 11:28
@kerneltime kerneltime self-requested a review October 17, 2022 16:06
@ChenSammi
Copy link
Contributor

Hi @szetszwo , currently SCM ratis is by default enabled in master. So we still cannot support moving SCM from one instance to three instances with ratis enabled?

@szetszwo szetszwo requested review from errose28, kerneltime and nandakumar131 and removed request for errose28 and kerneltime October 18, 2022 03:36
@szetszwo
Copy link
Contributor Author

@ChenSammi , you are right. The problem is the CA and certificates, but not the Ratis server.

Copy link
Member

@kaijchen kaijchen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. SCM-HA.zh.md needs to be updated also (perhaps in another PR).

@errose28
Copy link
Contributor

I see some previous discussion here but I just want to clarify before I +1. This is saying we cannot go from Ratis 1 to Ratis 3 SCM due to current limitations with certificates? Is this something that is planning to be resolved in the future cert rotation/management changes that @fapifta is working on?

@kerneltime kerneltime merged commit 11f4686 into apache:master Oct 18, 2022
@szetszwo
Copy link
Contributor Author

@errose28 , when configuring with HA, the CA structure has two levels -- root CA and sub CA. Without HA, there is only one level of CA. Therefore, the old certificates will not work when changing from non-HA to HA. I am not sure if there is a plan for fixing it.

Note that the problem has nothing to do with Ratis.

@szetszwo
Copy link
Contributor Author

... SCM-HA.zh.md needs to be updated also (perhaps in another PR).

@kaijchen , yes, are you interested in updating it?

@szetszwo
Copy link
Contributor Author

@ChenSammi , @nandakumar131, @kaijchen, @kerneltime , @errose28 , thanks everyone for reviewing this!

@kaijchen
Copy link
Member

yes, are you interested in updating it?

I have created #3861 to update it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants