Uploaded image for project: 'Hadoop Common'
  1. Hadoop Common
  2. HADOOP-6518

Kerberos login in UGI should honor KRB5CCNAME

        Details

        • Type: Improvement
        • Status: Closed
        • Priority: 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.omalley 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.omalley 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.omalley Owen O'Malley added a comment -

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

            Show
            owen.omalley Owen O'Malley added a comment - I've tested this by hand and it works as expected.
            Hide
            Permalink
            hadoopqa 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
            hadoopqa 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
            kzhang Kan Zhang added a comment -

            +1, patch looks good.

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

            I just committed this. Thanks, Owen!

            Show
            devaraj Devaraj Das added a comment - I just committed this. Thanks, Owen!
            Hide
            Permalink
            hudson 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 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 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 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
            jnp Jitendra Nath Pandey added a comment -

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

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

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development