Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.21.0
    • Component/s: security
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      Occasionally, MapReduce jobs need to launch other MapReduce jobs. With security enabled, the task needs to authenticate to the JobTracker as the user with a token.

      1. mr-1433.patch
        32 kB
        Owen O'Malley
      2. mr-1433.patch
        32 kB
        Owen O'Malley
      3. mr-1433.patch
        32 kB
        Owen O'Malley
      4. mr-1433.patch
        32 kB
        Owen O'Malley
      5. mr-1433.patch
        32 kB
        Owen O'Malley
      6. 1433.bp20.patch
        25 kB
        Devaraj Das
      7. 1433.bp20.patch
        25 kB
        Devaraj Das

        Activity

        Hide
        Owen O'Malley added a comment -

        A preliminary patch

        Show
        Owen O'Malley added a comment - A preliminary patch
        Hide
        Devaraj Das added a comment -

        Looks good. You need to fix JobTracker.renewDelegationToken to actually do the renewal (there is a copy-paste error there smile)

        Show
        Devaraj Das added a comment - Looks good. You need to fix JobTracker.renewDelegationToken to actually do the renewal (there is a copy-paste error there smile )
        Hide
        Devaraj Das added a comment -

        And, please define the config variables in mapred-default.xml

        Show
        Devaraj Das added a comment - And, please define the config variables in mapred-default.xml
        Hide
        Owen O'Malley added a comment -

        Updated with a few more fixes.

        Show
        Owen O'Malley added a comment - Updated with a few more fixes.
        Hide
        Owen O'Malley added a comment -

        This time attaching the right file. smile

        Show
        Owen O'Malley added a comment - This time attaching the right file. smile
        Hide
        Owen O'Malley added a comment -

        Bump the version number of ClientProtocol

        Show
        Owen O'Malley added a comment - Bump the version number of ClientProtocol
        Hide
        Owen O'Malley added a comment -

        Ok, this has an improved test and fixes a copy and paste bug.

        Show
        Owen O'Malley added a comment - Ok, this has an improved test and fixes a copy and paste bug.
        Hide
        Owen O'Malley added a comment -

        Adds license to the test case.

        Show
        Owen O'Malley added a comment - Adds license to the test case.
        Hide
        Owen O'Malley added a comment -

        Add some code that sets the service name on the received token.

        All tests pass and test-patch is clean.

        Show
        Owen O'Malley added a comment - Add some code that sets the service name on the received token. All tests pass and test-patch is clean.
        Hide
        Owen O'Malley added a comment -

        Updated with new code to normalize the hostname.

        Show
        Owen O'Malley added a comment - Updated with new code to normalize the hostname.
        Hide
        Devaraj Das added a comment -

        Please pass the right text to setService in getDelegationToken

        Show
        Devaraj Das added a comment - Please pass the right text to setService in getDelegationToken
        Hide
        Owen O'Malley added a comment -

        Ok, now the patch has the right fix in it.

        Show
        Owen O'Malley added a comment - Ok, now the patch has the right fix in it.
        Hide
        Devaraj Das added a comment -

        +1

        Show
        Devaraj Das added a comment - +1
        Hide
        Devaraj Das added a comment -

        Patch for Y20. Not for commit.

        Show
        Devaraj Das added a comment - Patch for Y20. Not for commit.
        Hide
        Owen O'Malley added a comment -

        It passes unit tests and test-patch.

        Show
        Owen O'Malley added a comment - It passes unit tests and test-patch.
        Hide
        Devaraj Das added a comment -

        More up-to-date version of the backported patch. Not for commit.

        Show
        Devaraj Das added a comment - More up-to-date version of the backported patch. Not for commit.
        Hide
        Hudson added a comment -

        Integrated in Hadoop-Mapreduce-trunk-Commit #233 (See http://hudson.zones.apache.org/hudson/job/Hadoop-Mapreduce-trunk-Commit/233/)

        Show
        Hudson added a comment - Integrated in Hadoop-Mapreduce-trunk-Commit #233 (See http://hudson.zones.apache.org/hudson/job/Hadoop-Mapreduce-trunk-Commit/233/ )
        Hide
        Hudson added a comment -

        Integrated in Hadoop-Mapreduce-trunk #232 (See http://hudson.zones.apache.org/hudson/job/Hadoop-Mapreduce-trunk/232/)

        Show
        Hudson added a comment - Integrated in Hadoop-Mapreduce-trunk #232 (See http://hudson.zones.apache.org/hudson/job/Hadoop-Mapreduce-trunk/232/ )

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development