echonesis opened a new pull request, #11652: URL: https://github.com/apache/ozone/pull/11652
## What changes were proposed in this pull request? **SecretKeyProtocol** duplicated the SCM server Kerberos metadata already defined by the role-specific PB interfaces used by Hadoop RPC. This PR: - removes the redundant **@KerberosInfo** annotation from the base **SecretKeyProtocol** interface; - keeps the RPC metadata in the PB interfaces and preserves the role-specific Java annotations required by service authorization; - adds regression tests verifying each role’s protocol name, protocol version, PB Kerberos metadata, and Java-side client principal resolution. This removes unused metadata while preserving the existing RPC identities, wire protocol, and service authorization behavior. ## What is the link to the Apache JIRA https://issues.apache.org/jira/browse/HDDS-16246 ## How was this patch tested? Local Tests ``` mvn -pl :hdds-server-framework -am test \ -Dtest=TestSecretKeyProtocolPB \ -Dsurefire.failIfNoSpecifiedTests=false \ -DskipShade -DskipRecon -DskipDocs ``` GitHub Actions CI: https://github.com/echonesis/ozone/actions/runs/39466788477 Generated-by: Codex (GPT-6) -- 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]
