Skip to content

HDDS-15044. Exercise ozone debug kerberos commands#10085

Merged
Gargi-jais11 merged 4 commits into
apache:masterfrom
navinko:HDDS-15044
May 4, 2026
Merged

HDDS-15044. Exercise ozone debug kerberos commands#10085
Gargi-jais11 merged 4 commits into
apache:masterfrom
navinko:HDDS-15044

Conversation

@navinko
Copy link
Copy Markdown
Contributor

@navinko navinko commented Apr 17, 2026

What changes were proposed in this pull request?

Added robot test fro ozone debug kerberos subcommands

Please describe your PR in detail:

Added robot test for :
ozone debug kerberos diagnose
- checked 'should contain' for o/p and considered few probe only .
ozone debug kerberos translate-principal
- checked 'should contain' + return code for invalid principal scenario.

What is the link to the Apache JIRA

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

How was this patch tested?

ran test-debug-tools.sh from ozonesecure-ha locally
and collected xml generated robot-004.xml
Successful CI: https://github.com/navinko/ozone/actions/runs/24730108195/job/72344081175

Copy link
Copy Markdown
Contributor

@sreejasahithi sreejasahithi left a comment

Choose a reason for hiding this comment

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

Thanks @navinko for working on this.

Comment thread hadoop-ozone/dist/src/main/smoketest/debug/ozone-debug-kerberos.robot Outdated
Copy link
Copy Markdown
Contributor

@Gargi-jais11 Gargi-jais11 left a comment

Choose a reason for hiding this comment

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

Thanks @navinko for adding robot test for the command.
Please find the inline comment.

Comment thread hadoop-ozone/dist/src/main/smoketest/debug/ozone-debug-kerberos.robot Outdated
Copy link
Copy Markdown
Contributor

@Gargi-jais11 Gargi-jais11 left a comment

Choose a reason for hiding this comment

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

LGTM!

Copy link
Copy Markdown
Contributor

@sreejasahithi sreejasahithi left a comment

Choose a reason for hiding this comment

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

Thanks @navinko for updating the patch
LGTM

@sarvekshayr sarvekshayr self-requested a review April 22, 2026 05:12
@adoroszlai adoroszlai added test tools Tools that helps with debugging labels Apr 22, 2026
@navinko
Copy link
Copy Markdown
Contributor Author

navinko commented May 1, 2026

Thanks @Gargi-jais11 @sreejasahithi @adoroszlai for reviewing .

@Gargi-jais11 Gargi-jais11 merged commit 1bf9a3b into apache:master May 4, 2026
33 checks passed
@Gargi-jais11
Copy link
Copy Markdown
Contributor

Thanks @navinko for working on this and @sreejasahithi for the review.

@navinko
Copy link
Copy Markdown
Contributor Author

navinko commented May 4, 2026

Thanks @Gargi-jais11 for merging this PR

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

Labels

test tools Tools that helps with debugging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants