Skip to content

Conversation

@adoroszlai
Copy link
Contributor

@adoroszlai adoroszlai commented Jul 15, 2021

What changes were proposed in this pull request?

Fix filename of datanode HTTP keytab in ozonesecure env, which caused:

[main] ERROR ozone.HddsDatanodeService: HttpServer failed to start.
...
Caused by: javax.servlet.ServletException: Keytab does not exist: /etc/security/keytabs/datanode.keytab

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

How was this patch tested?

Started ozonesecure cluster, no error related to HTTP server:

[main] INFO http.BaseHttpServer: Starting Web-server for hddsDatanode at: http://0.0.0.0:9882
[main] INFO http.BaseHttpServer: Hadoop Security Enabled: true Ozone Security Enabled: true Ozone HTTP Security Enabled: true 
[main] INFO http.BaseHttpServer: HttpAuthType: hdds.datanode.http.auth.type = kerberos
...
[main] INFO http.BaseHttpServer: HTTP server of hddsDatanode listening at http://0.0.0.0:9882

Regular CI:
https://github.com/adoroszlai/hadoop-ozone/actions/runs/1031181668

@adoroszlai adoroszlai self-assigned this Jul 15, 2021
Copy link
Contributor

@mukul1987 mukul1987 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
Contributor

@xiaoyuyao xiaoyuyao left a comment

Choose a reason for hiding this comment

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

LGTM, +1.

@xiaoyuyao xiaoyuyao merged commit 76384ca into apache:master Jul 15, 2021
@adoroszlai adoroszlai deleted the HDDS-5442 branch July 15, 2021 14:45
@adoroszlai
Copy link
Contributor Author

Thanks @mukul1987 for the review and @xiaoyuyao for merging this.

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.

3 participants