Hadoop Common
  1. Hadoop Common
  2. HADOOP-6518

Kerberos login in UGI should honor KRB5CCNAME

        Details

        • Type: Improvement Improvement
        • Status: Closed
        • Priority: Major Major
        • Resolution: Fixed
        • Affects Version/s: None
        • Fix Version/s: 0.21.0
        • Component/s: security
        • Labels:
          None

          Description

          UGI should honor the environment variable KRB5CCNAME as the location of the ticket cache.

          1. Text File
            h-6518.patch
            0.8 kB
            Owen O'Malley
          2. Text File
            HADOOP-6518-0_20.1.patch
            4 kB
            Jitendra Nath Pandey

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            Owen O'Malley added a comment -

            Here is the patch. It isn't possible to do meaniful unit tests of the Kerberos stuff.

            Show
            Owen O'Malley added a comment - Here is the patch. It isn't possible to do meaniful unit tests of the Kerberos stuff.
            Hide
            Permalink
            Owen O'Malley added a comment -

            I've tested this by hand and it works as expected.

            Show
            Owen O'Malley added a comment - I've tested this by hand and it works as expected.
            Hide
            Permalink
            Hadoop QA added a comment -

            -1 overall. Here are the results of testing the latest attachment
            http://issues.apache.org/jira/secure/attachment/12431682/h-6518.patch
            against trunk revision 904339.

            +1 @author. The patch does not contain any @author tags.

            -1 tests included. The patch doesn't appear to include any new or modified tests.
            Please justify why no new tests are needed for this patch.
            Also please list what manual steps were performed to verify this patch.

            +1 javadoc. The javadoc tool did not generate any warning messages.

            +1 javac. The applied patch does not increase the total number of javac compiler warnings.

            +1 findbugs. The patch does not introduce any new Findbugs warnings.

            +1 release audit. The applied patch does not increase the total number of release audit warnings.

            +1 core tests. The patch passed core unit tests.

            +1 contrib tests. The patch passed contrib unit tests.

            Test results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-h4.grid.sp2.yahoo.net/300/testReport/
            Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-h4.grid.sp2.yahoo.net/300/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
            Checkstyle results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-h4.grid.sp2.yahoo.net/300/artifact/trunk/build/test/checkstyle-errors.html
            Console output: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-h4.grid.sp2.yahoo.net/300/console

            This message is automatically generated.

            Show
            Hadoop QA added a comment - -1 overall. Here are the results of testing the latest attachment http://issues.apache.org/jira/secure/attachment/12431682/h-6518.patch against trunk revision 904339. +1 @author. The patch does not contain any @author tags. -1 tests included. The patch doesn't appear to include any new or modified tests. Please justify why no new tests are needed for this patch. Also please list what manual steps were performed to verify this patch. +1 javadoc. The javadoc tool did not generate any warning messages. +1 javac. The applied patch does not increase the total number of javac compiler warnings. +1 findbugs. The patch does not introduce any new Findbugs warnings. +1 release audit. The applied patch does not increase the total number of release audit warnings. +1 core tests. The patch passed core unit tests. +1 contrib tests. The patch passed contrib unit tests. Test results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-h4.grid.sp2.yahoo.net/300/testReport/ Findbugs warnings: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-h4.grid.sp2.yahoo.net/300/artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html Checkstyle results: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-h4.grid.sp2.yahoo.net/300/artifact/trunk/build/test/checkstyle-errors.html Console output: http://hudson.zones.apache.org/hudson/job/Hadoop-Patch-h4.grid.sp2.yahoo.net/300/console This message is automatically generated.
            Hide
            Permalink
            Kan Zhang added a comment -

            +1, patch looks good.

            Show
            Kan Zhang added a comment - +1, patch looks good.
            Hide
            Permalink
            Devaraj Das added a comment -

            I just committed this. Thanks, Owen!

            Show
            Devaraj Das added a comment - I just committed this. Thanks, Owen!
            Hide
            Permalink
            Hudson added a comment -

            Integrated in Hadoop-Common-trunk-Commit #153 (See http://hudson.zones.apache.org/hudson/job/Hadoop-Common-trunk-Commit/153/)
            . Makes the UGI honor the env var KRB5CCNAME. Contributed by Owen O'Malley.

            Show
            Hudson added a comment - Integrated in Hadoop-Common-trunk-Commit #153 (See http://hudson.zones.apache.org/hudson/job/Hadoop-Common-trunk-Commit/153/ ) . Makes the UGI honor the env var KRB5CCNAME. Contributed by Owen O'Malley.
            Hide
            Permalink
            Hudson added a comment -

            Integrated in Hadoop-Common-trunk #238 (See http://hudson.zones.apache.org/hudson/job/Hadoop-Common-trunk/238/)
            . Makes the UGI honor the env var KRB5CCNAME. Contributed by Owen O'Malley.

            Show
            Hudson added a comment - Integrated in Hadoop-Common-trunk #238 (See http://hudson.zones.apache.org/hudson/job/Hadoop-Common-trunk/238/ ) . Makes the UGI honor the env var KRB5CCNAME. Contributed by Owen O'Malley.
            Hide
            Permalink
            Jitendra Nath Pandey added a comment -

            Combined patch for HADOOP-6517 and HADOOP-6518 for 20 branch.

            Show
            Jitendra Nath Pandey added a comment - Combined patch for HADOOP-6517 and HADOOP-6518 for 20 branch.

              People

              • Assignee:
                Owen O'Malley
                Reporter:
                Owen O'Malley
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development