[Impala-ASF-CR] KUDU-2385: Fix typo in KinitContext::DoRenewal()
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/9842 ) Change subject: KUDU-2385: Fix typo in KinitContext::DoRenewal() .. Patch Set 1: Verified+1 -- To view, visit http://gerrit.cloudera.org:8080/9842 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I3a22b8d41d15eb1982a3fd5b96575e28edaad31c Gerrit-Change-Number: 9842 Gerrit-PatchSet: 1 Gerrit-Owner: Sailesh MukilGerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Michael Ho Gerrit-Comment-Date: Thu, 29 Mar 2018 01:40:04 + Gerrit-HasComments: No
[Impala-ASF-CR] KUDU-2385: Fix typo in KinitContext::DoRenewal()
Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9842 ) Change subject: KUDU-2385: Fix typo in KinitContext::DoRenewal() .. KUDU-2385: Fix typo in KinitContext::DoRenewal() On platforms without krb5_get_init_creds_opt_set_out_ccache(), krb5_cc_store_cred() is called to insert the newly acquired credential into the ccache. However, there was a typo in the code which resulted in inserting the old credential into ccache. This change fixes the typo to make sure the new credential is inserted into ccache. Testing done: confirmed on SLES11 that the new credential is being inserted by checking the 'auth time' of the ticket in ccache. Impala uses a slightly different #ifdef which explicitly checks if krb5_get_init_creds_opt_set_out_ccache() is defined on the platform so this code path is actually used when running Impala on SLES11. Change-Id: I3a22b8d41d15eb1982a3fd5b96575e28edaad31c Reviewed-on: http://gerrit.cloudera.org:8080/9840 Reviewed-by: Todd LipconTested-by: Todd Lipcon Reviewed-on: http://gerrit.cloudera.org:8080/9842 Reviewed-by: Michael Ho Tested-by: Impala Public Jenkins --- M be/src/kudu/security/init.cc 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Michael Ho: Looks good to me, approved Impala Public Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/9842 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I3a22b8d41d15eb1982a3fd5b96575e28edaad31c Gerrit-Change-Number: 9842 Gerrit-PatchSet: 2 Gerrit-Owner: Sailesh Mukil Gerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Michael Ho
[Impala-ASF-CR] KUDU-2385: Fix typo in KinitContext::DoRenewal()
Impala Public Jenkins has posted comments on this change. ( http://gerrit.cloudera.org:8080/9842 ) Change subject: KUDU-2385: Fix typo in KinitContext::DoRenewal() .. Patch Set 1: Build started: https://jenkins.impala.io/job/gerrit-verify-dryrun/2200/ -- To view, visit http://gerrit.cloudera.org:8080/9842 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I3a22b8d41d15eb1982a3fd5b96575e28edaad31c Gerrit-Change-Number: 9842 Gerrit-PatchSet: 1 Gerrit-Owner: Sailesh MukilGerrit-Reviewer: Impala Public Jenkins Gerrit-Reviewer: Michael Ho Gerrit-Comment-Date: Wed, 28 Mar 2018 21:44:47 + Gerrit-HasComments: No
[Impala-ASF-CR] KUDU-2385: Fix typo in KinitContext::DoRenewal()
Michael Ho has posted comments on this change. ( http://gerrit.cloudera.org:8080/9842 ) Change subject: KUDU-2385: Fix typo in KinitContext::DoRenewal() .. Patch Set 1: Code-Review+2 -- To view, visit http://gerrit.cloudera.org:8080/9842 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I3a22b8d41d15eb1982a3fd5b96575e28edaad31c Gerrit-Change-Number: 9842 Gerrit-PatchSet: 1 Gerrit-Owner: Sailesh MukilGerrit-Reviewer: Michael Ho Gerrit-Comment-Date: Wed, 28 Mar 2018 21:44:21 + Gerrit-HasComments: No
[Impala-ASF-CR] KUDU-2385: Fix typo in KinitContext::DoRenewal()
Sailesh Mukil has removed Todd Lipcon from this change. ( http://gerrit.cloudera.org:8080/9842 ) Change subject: KUDU-2385: Fix typo in KinitContext::DoRenewal() .. Removed reviewer Todd Lipcon. -- To view, visit http://gerrit.cloudera.org:8080/9842 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: deleteReviewer Gerrit-Change-Id: I3a22b8d41d15eb1982a3fd5b96575e28edaad31c Gerrit-Change-Number: 9842 Gerrit-PatchSet: 1 Gerrit-Owner: Sailesh MukilGerrit-Reviewer: Michael Ho
[Impala-ASF-CR] KUDU-2385: Fix typo in KinitContext::DoRenewal()
Hello Todd Lipcon, I'd like you to do a code review. Please visit http://gerrit.cloudera.org:8080/9842 to review the following change. Change subject: KUDU-2385: Fix typo in KinitContext::DoRenewal() .. KUDU-2385: Fix typo in KinitContext::DoRenewal() On platforms without krb5_get_init_creds_opt_set_out_ccache(), krb5_cc_store_cred() is called to insert the newly acquired credential into the ccache. However, there was a typo in the code which resulted in inserting the old credential into ccache. This change fixes the typo to make sure the new credential is inserted into ccache. Testing done: confirmed on SLES11 that the new credential is being inserted by checking the 'auth time' of the ticket in ccache. Impala uses a slightly different #ifdef which explicitly checks if krb5_get_init_creds_opt_set_out_ccache() is defined on the platform so this code path is actually used when running Impala on SLES11. Change-Id: I3a22b8d41d15eb1982a3fd5b96575e28edaad31c Reviewed-on: http://gerrit.cloudera.org:8080/9840 Reviewed-by: Todd LipconTested-by: Todd Lipcon --- M be/src/kudu/security/init.cc 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://gerrit.cloudera.org:29418/Impala-ASF refs/changes/42/9842/1 -- To view, visit http://gerrit.cloudera.org:8080/9842 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I3a22b8d41d15eb1982a3fd5b96575e28edaad31c Gerrit-Change-Number: 9842 Gerrit-PatchSet: 1 Gerrit-Owner: Sailesh Mukil Gerrit-Reviewer: Michael Ho Gerrit-Reviewer: Todd Lipcon