You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@ozone.apache.org by GitBox <gi...@apache.org> on 2020/06/20 02:46:53 UTC

[GitHub] [hadoop-ozone] xiaoyuyao opened a new pull request #1101: HDDS-3821. Disable Ozone SPNEGO should not fall back to hadoop.http.a…

xiaoyuyao opened a new pull request #1101:
URL: https://github.com/apache/hadoop-ozone/pull/1101


   …uthentication configuration.
   
   ## What changes were proposed in this pull request?
   
   When ozone http kerberos is disabled, ensure the authentication filter does not fallback to configuration from hadoop.http.authentication.*
   
   ## What is the link to the Apache JIRA
   
   https://issues.apache.org/jira/browse/HDDS-3821
   
   ## How was this patch tested?
   
   Acceptance test and Manual test when ozone http kerberos is disabled but  hadoop.http.authentication.* is enabled. 


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: ozone-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: ozone-issues-help@hadoop.apache.org


[GitHub] [hadoop-ozone] adoroszlai commented on a change in pull request #1101: HDDS-3821. Disable Ozone SPNEGO should not fall back to hadoop.http.a…

Posted by GitBox <gi...@apache.org>.
adoroszlai commented on a change in pull request #1101:
URL: https://github.com/apache/hadoop-ozone/pull/1101#discussion_r455019812



##########
File path: hadoop-ozone/integration-test/src/test/resources/log4j.properties
##########
@@ -19,3 +19,6 @@ log4j.appender.stdout.layout.ConversionPattern=%d{ISO8601} [%t] %-5p %c{2} (%F:%
 
 log4j.logger.org.apache.hadoop.security.ShellBasedUnixGroupsMapping=ERROR
 log4j.logger.org.apache.hadoop.util.NativeCodeLoader=ERROR
+
+log4j.logger.org.apache.ratis.grpc.server.GrpcLogAppender=DEBUG
+log4j.logger.org.apache.ratis.server.impl.RaftServerImpl=DEBUG

Review comment:
       Hi @xiaoyuyao,
   
   Were these Ratis log levels intentionally set to DEBUG in the final commit, or is it a leftover that should have been removed?




----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: ozone-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: ozone-issues-help@hadoop.apache.org


[GitHub] [hadoop-ozone] vivekratnavel commented on pull request #1101: HDDS-3821. Disable Ozone SPNEGO should not fall back to hadoop.http.a…

Posted by GitBox <gi...@apache.org>.
vivekratnavel commented on pull request #1101:
URL: https://github.com/apache/hadoop-ozone/pull/1101#issuecomment-648993739


   @xiaoyuyao Thanks for working on this and @dineshchitlangia for reviewing it!


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: ozone-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: ozone-issues-help@hadoop.apache.org


[GitHub] [hadoop-ozone] vivekratnavel merged pull request #1101: HDDS-3821. Disable Ozone SPNEGO should not fall back to hadoop.http.a…

Posted by GitBox <gi...@apache.org>.
vivekratnavel merged pull request #1101:
URL: https://github.com/apache/hadoop-ozone/pull/1101


   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: ozone-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: ozone-issues-help@hadoop.apache.org


[GitHub] [hadoop-ozone] adoroszlai commented on pull request #1101: HDDS-3821. Disable Ozone SPNEGO should not fall back to hadoop.http.a…

Posted by GitBox <gi...@apache.org>.
adoroszlai commented on pull request #1101:
URL: https://github.com/apache/hadoop-ozone/pull/1101#issuecomment-658755342


   > This should be removed.
   
   Thanks for the confirmation.  Created HDDS-3967 for this.  Will submit PR soon.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: ozone-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: ozone-issues-help@hadoop.apache.org


[GitHub] [hadoop-ozone] xiaoyuyao commented on pull request #1101: HDDS-3821. Disable Ozone SPNEGO should not fall back to hadoop.http.a…

Posted by GitBox <gi...@apache.org>.
xiaoyuyao commented on pull request #1101:
URL: https://github.com/apache/hadoop-ozone/pull/1101#issuecomment-647050915


   org.apache.hadoop.ozone.client.rpc.Test2WayCommitInRatis seems to be flaky. Pass in the first commit but not the second one. 


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: ozone-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: ozone-issues-help@hadoop.apache.org


[GitHub] [hadoop-ozone] xiaoyuyao commented on pull request #1101: HDDS-3821. Disable Ozone SPNEGO should not fall back to hadoop.http.a…

Posted by GitBox <gi...@apache.org>.
xiaoyuyao commented on pull request #1101:
URL: https://github.com/apache/hadoop-ozone/pull/1101#issuecomment-658750232


   This should be removed. Thanks @adoroszlai
   
   On Wed, Jul 15, 2020 at 5:40 AM Doroszlai, Attila <no...@github.com>
   wrote:
   
   > *@adoroszlai* commented on this pull request.
   > ------------------------------
   >
   > In hadoop-ozone/integration-test/src/test/resources/log4j.properties
   > <https://github.com/apache/hadoop-ozone/pull/1101#discussion_r455019812>:
   >
   > > +log4j.logger.org.apache.ratis.grpc.server.GrpcLogAppender=DEBUG
   > +log4j.logger.org.apache.ratis.server.impl.RaftServerImpl=DEBUG
   >
   > Hi @xiaoyuyao <https://github.com/xiaoyuyao>,
   >
   > Were these Ratis log levels intentionally set to DEBUG in the final
   > commit, or is it a leftover that should have been removed?
   >
   > —
   > You are receiving this because you were mentioned.
   > Reply to this email directly, view it on GitHub
   > <https://github.com/apache/hadoop-ozone/pull/1101#pullrequestreview-448912144>,
   > or unsubscribe
   > <https://github.com/notifications/unsubscribe-auth/ABVWRUHCRB3AMJLBUVLX7H3R3WPU5ANCNFSM4ODFYWVA>
   > .
   >
   


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: ozone-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: ozone-issues-help@hadoop.apache.org