HBase
  1. HBase
  2. HBASE-7829

zookeeper kerberos conf keytab and principal parameters interchanged

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 0.94.6, 0.95.0
    • Fix Version/s: 0.94.6, 0.95.0
    • Component/s: None
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Issue Links

        Activity

        Hide
        Hadoop QA added a comment -

        -1 overall. Here are the results of testing the latest attachment
        http://issues.apache.org/jira/secure/attachment/12568924/HBASE-7829_94.patch
        against trunk revision .

        +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 patch. The patch command could not apply the patch.

        Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/4420//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/12568924/HBASE-7829_94.patch against trunk revision . +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 patch . The patch command could not apply the patch. Console output: https://builds.apache.org/job/PreCommit-HBASE-Build/4420//console This message is automatically generated.
        Hide
        Lars Hofhansl added a comment -

        Oops.
        How did this ever work correctly?

        Show
        Lars Hofhansl added a comment - Oops. How did this ever work correctly?
        Hide
        Lars Hofhansl added a comment -

        So how bad is this? Will this prevent secure operation? In that case I'll sink the current 0.94.5 RC.

        Show
        Lars Hofhansl added a comment - So how bad is this? Will this prevent secure operation? In that case I'll sink the current 0.94.5 RC.
        Hide
        Francis Liu added a comment -

        It's not that bad, since the previous method of enabling zookeeper with security still works. No one has probably tried this new method else we would've seen a jira earlier.

        Show
        Francis Liu added a comment - It's not that bad, since the previous method of enabling zookeeper with security still works. No one has probably tried this new method else we would've seen a jira earlier.
        Hide
        Francis Liu added a comment -

        94 patch will work on trunk as well.

        Show
        Francis Liu added a comment - 94 patch will work on trunk as well.
        Hide
        Matteo Bertozzi added a comment -

        yup, that's my fault.. looking at my configuration the values are swapped
        +1 on the patch (for 0.94 and trunk), thanks!

        Show
        Matteo Bertozzi added a comment - yup, that's my fault.. looking at my configuration the values are swapped +1 on the patch (for 0.94 and trunk), thanks!
        Hide
        Ted Yu added a comment -

        Integrated to 0.94 and trunk.

        Thanks for the patch, Francis.

        Thanks for the reviews, Lars and Matteo.

        Show
        Ted Yu added a comment - Integrated to 0.94 and trunk. Thanks for the patch, Francis. Thanks for the reviews, Lars and Matteo.
        Hide
        Hudson added a comment -

        Integrated in HBase-0.94 #838 (See https://builds.apache.org/job/HBase-0.94/838/)
        HBASE-7829 zookeeper kerberos conf keytab and principal parameters interchanged (Francis) (Revision 1445211)

        Result = FAILURE
        tedyu :
        Files :

        • /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKUtil.java
        Show
        Hudson added a comment - Integrated in HBase-0.94 #838 (See https://builds.apache.org/job/HBase-0.94/838/ ) HBASE-7829 zookeeper kerberos conf keytab and principal parameters interchanged (Francis) (Revision 1445211) Result = FAILURE tedyu : Files : /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKUtil.java
        Hide
        Andrew Purtell added a comment -

        Thanks. This points to we are generally not testing security. I raised the priority of HBASE-7781 to critical and hope to have some time for it soon.

        Show
        Andrew Purtell added a comment - Thanks. This points to we are generally not testing security. I raised the priority of HBASE-7781 to critical and hope to have some time for it soon.
        Hide
        Hudson added a comment -

        Integrated in HBase-TRUNK #3873 (See https://builds.apache.org/job/HBase-TRUNK/3873/)
        HBASE-7829 zookeeper kerberos conf keytab and principal parameters interchanged (Francis) (Revision 1445213)

        Result = FAILURE
        tedyu :
        Files :

        • /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKUtil.java
        Show
        Hudson added a comment - Integrated in HBase-TRUNK #3873 (See https://builds.apache.org/job/HBase-TRUNK/3873/ ) HBASE-7829 zookeeper kerberos conf keytab and principal parameters interchanged (Francis) (Revision 1445213) Result = FAILURE tedyu : Files : /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKUtil.java
        Hide
        Hudson added a comment -

        Integrated in HBase-0.94-security #110 (See https://builds.apache.org/job/HBase-0.94-security/110/)
        HBASE-7829 zookeeper kerberos conf keytab and principal parameters interchanged (Francis) (Revision 1445211)

        Result = SUCCESS
        tedyu :
        Files :

        • /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKUtil.java
        Show
        Hudson added a comment - Integrated in HBase-0.94-security #110 (See https://builds.apache.org/job/HBase-0.94-security/110/ ) HBASE-7829 zookeeper kerberos conf keytab and principal parameters interchanged (Francis) (Revision 1445211) Result = SUCCESS tedyu : Files : /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKUtil.java
        Hide
        Hudson added a comment -

        Integrated in HBase-TRUNK-on-Hadoop-2.0.0 #405 (See https://builds.apache.org/job/HBase-TRUNK-on-Hadoop-2.0.0/405/)
        HBASE-7829 zookeeper kerberos conf keytab and principal parameters interchanged (Francis) (Revision 1445213)

        Result = FAILURE
        tedyu :
        Files :

        • /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKUtil.java
        Show
        Hudson added a comment - Integrated in HBase-TRUNK-on-Hadoop-2.0.0 #405 (See https://builds.apache.org/job/HBase-TRUNK-on-Hadoop-2.0.0/405/ ) HBASE-7829 zookeeper kerberos conf keytab and principal parameters interchanged (Francis) (Revision 1445213) Result = FAILURE tedyu : Files : /hbase/trunk/hbase-server/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKUtil.java
        Hide
        Hudson added a comment -

        Integrated in HBase-0.94-security-on-Hadoop-23 #12 (See https://builds.apache.org/job/HBase-0.94-security-on-Hadoop-23/12/)
        HBASE-7829 zookeeper kerberos conf keytab and principal parameters interchanged (Francis) (Revision 1445211)

        Result = FAILURE
        tedyu :
        Files :

        • /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKUtil.java
        Show
        Hudson added a comment - Integrated in HBase-0.94-security-on-Hadoop-23 #12 (See https://builds.apache.org/job/HBase-0.94-security-on-Hadoop-23/12/ ) HBASE-7829 zookeeper kerberos conf keytab and principal parameters interchanged (Francis) (Revision 1445211) Result = FAILURE tedyu : Files : /hbase/branches/0.94/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKUtil.java

          People

          • Assignee:
            Francis Liu
            Reporter:
            Francis Liu
          • Votes:
            0 Vote for this issue
            Watchers:
            8 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development