Skip to content

Conversation

@LZD-PratyushBhatt
Copy link
Contributor

What changes were proposed in this pull request?

scmha/scm-leader-transfer.robot is not run in any test suite currently.
Aim is to execute this test for atleast one of the suite.

What is the link to the Apache JIRA

HDDS-9453

How was this patch tested?

CI

@LZD-PratyushBhatt LZD-PratyushBhatt marked this pull request as draft October 14, 2023 11:43
@kerneltime kerneltime requested a review from adoroszlai October 16, 2023 16:34
@kerneltime
Copy link
Contributor

cc @tanvipenumudy @swamirishi can you please take a look.

@LZD-PratyushBhatt LZD-PratyushBhatt marked this pull request as ready for review October 17, 2023 12:20
@tanvipenumudy
Copy link
Contributor

scm-leader-transfer.robot was part of ozonesecure-ha/test.sh until the change: HDDS-8689, PR #4943 (looks like it has been moved to the cert-rotation suite).

@LZD-PratyushBhatt
Copy link
Contributor Author

scm-leader-transfer.robot was part of ozonesecure-ha/test.sh until the change: HDDS-8689, PR #4943 (looks like it has been moved to the cert-rotation suite).

Hi @tanvipenumudy , I dont see it being executed as part of https://github.com/apache/ozone/actions/runs/6538885863/job/17774323240?pr=5439 CI, this is for cert-rotation acceptance tests.
This commit disabled the root-ca-rotation tests hence disabling this test too.
So currently its not being called anywhere.

@adoroszlai adoroszlai merged commit 5b3abce into apache:master Oct 18, 2023
@adoroszlai
Copy link
Contributor

Thanks @LZD-PratyushBhatt for the patch, @tanvipenumudy for the review.

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants