Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

HDDS-11810. Secure acceptance test on arm64 fails with LoginException: Checksum failed #7498

Merged
merged 1 commit into from
Nov 29, 2024

Conversation

adoroszlai
Copy link
Contributor

What changes were proposed in this pull request?

Kerberos-enabled acceptance tests in Ozone use the ozone-testkrb5 Docker image as KDC. Keytab files are exported from the KDC at development time, running update-keytabs.sh, then added to Git. Ozone services access these via volume mounted in each container.

Multi-arch Docker images present a problem: keytab generation uses a specific platform variant of the image (depending on the architecture of the computer update-keytabs.sh is run on). Keytabs work only with that variant. Therefore Keytabs pre-generated with the amd64 image do not work with arm64 one:

failure to login: for principal: scm/[email protected] from keytab /etc/security/keytabs/scm.keytab javax.security.auth.login.LoginException: Checksum failed

This PR proposes to defer keytab export to the time when tests start the Docker Compose cluster. This ensures both use the same variant of the ozone-testkrb5 image. Pre-generated keytabs are no longer needed, removed from the repo.

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

How was this patch tested?

Tested on amd64 locally and in CI:
https://github.com/adoroszlai/ozone/actions/runs/12058615470

@ChenSammi tested it on arm64.

@adoroszlai adoroszlai self-assigned this Nov 28, 2024
Copy link
Contributor

@ChenSammi ChenSammi left a comment

Choose a reason for hiding this comment

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

Thanks @adoroszlai for the quick fix.

@adoroszlai adoroszlai merged commit ebcdc6a into apache:master Nov 29, 2024
26 checks passed
@adoroszlai adoroszlai deleted the HDDS-11810 branch November 29, 2024 04:42
@adoroszlai
Copy link
Contributor Author

Thanks @ChenSammi for reporting this and for the review.

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.

2 participants