Bigtop
  1. Bigtop
  2. BIGTOP-489

TestCLI with two test cases fails ID - 146 and 147 related to refreshServiceAcl

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Duplicate
    • Affects Version/s: 0.3.0
    • Fix Version/s: 0.6.0
    • Component/s: Tests
    • Labels:
      None

      Description

      12/03/23 23:47:57 INFO cli.TestCLI: Detailed results:
      12/03/23 23:47:57 INFO cli.TestCLI: ----------------------------------

      12/03/23 23:47:57 INFO cli.TestCLI: -------------------------------------------
      12/03/23 23:47:57 INFO cli.TestCLI: Test ID: [146]
      12/03/23 23:47:57 INFO cli.TestCLI: Test Description: [refreshServiceAcl: refreshing security authorization policy for namenode]
      12/03/23 23:47:57 INFO cli.TestCLI:
      12/03/23 23:47:57 INFO cli.TestCLI: Test Commands: [-fs hdfs://localhost:58873 -refreshServiceAcl ]
      12/03/23 23:47:57 INFO cli.TestCLI:
      12/03/23 23:47:57 INFO cli.TestCLI:
      12/03/23 23:47:57 INFO cli.TestCLI: Comparator: [ExactComparator]
      12/03/23 23:47:57 INFO cli.TestCLI: Comparision result: [fail]
      12/03/23 23:47:57 INFO cli.TestCLI: Expected output: []
      12/03/23 23:47:57 INFO cli.TestCLI: Actual output: [12/03/23 23:47:53 INFO authorize.ServiceAuthorizationManager: Authorization successfull for ubuntu for protocol=interface org.apache.hadoop.security.authorize.RefreshAuthorizationPolicyProtocol
      ]
      12/03/23 23:47:57 INFO cli.TestCLI:
      12/03/23 23:47:57 INFO cli.TestCLI: -------------------------------------------
      12/03/23 23:47:57 INFO cli.TestCLI: Test ID: [147]
      12/03/23 23:47:57 INFO cli.TestCLI: Test Description: [refreshServiceAcl: refreshing security authorization policy for jobtracker]
      12/03/23 23:47:57 INFO cli.TestCLI:
      12/03/23 23:47:57 INFO cli.TestCLI: Test Commands: [-jt localhost:44180 -refreshServiceAcl ]
      12/03/23 23:47:57 INFO cli.TestCLI:
      12/03/23 23:47:57 INFO cli.TestCLI:
      12/03/23 23:47:57 INFO cli.TestCLI: Comparator: [ExactComparator]
      12/03/23 23:47:57 INFO cli.TestCLI: Comparision result: [fail]
      12/03/23 23:47:57 INFO cli.TestCLI: Expected output: []
      12/03/23 23:47:57 INFO cli.TestCLI: Actual output: [12/03/23 23:47:53 INFO authorize.ServiceAuthorizationManager: Authorization successfull for ubuntu for protocol=interface org.apache.hadoop.security.authorize.RefreshAuthorizationPolicyProtocol
      ]
      12/03/23 23:47:57 INFO cli.TestCLI:
      12/03/23 23:47:57 INFO cli.TestCLI: Summary results:
      12/03/23 23:47:57 INFO cli.TestCLI: ----------------------------------

      12/03/23 23:47:57 INFO cli.TestCLI: Testing mode: test
      12/03/23 23:47:57 INFO cli.TestCLI:
      12/03/23 23:47:57 INFO cli.TestCLI: Overall result: — FAIL —
      12/03/23 23:47:57 INFO cli.TestCLI: # Tests pass: 152 (98%)
      12/03/23 23:47:57 INFO cli.TestCLI: # Tests fail: 2 (1%)
      12/03/23 23:47:57 INFO cli.TestCLI: # Validations done: 298 (each test may do multiple validations)
      12/03/23 23:47:57 INFO cli.TestCLI:
      12/03/23 23:47:57 INFO cli.TestCLI: Failing tests:
      12/03/23 23:47:57 INFO cli.TestCLI: --------------
      12/03/23 23:47:57 INFO cli.TestCLI: 146: refreshServiceAcl: refreshing security authorization policy for namenode
      12/03/23 23:47:57 INFO cli.TestCLI: 147: refreshServiceAcl: refreshing security authorization policy for jobtracker

        Activity

        Hide
        David Liu added a comment -

        Tests now passing with the updated expected results, and matching function:

        12/03/24 13:47:46 INFO cli.TestCLI: 146: refreshServiceAcl: refreshing security authorization policy for namenode
        12/03/24 13:47:46 INFO cli.TestCLI: 147: refreshServiceAcl: refreshing security authorization policy for jobtracker

        Show
        David Liu added a comment - Tests now passing with the updated expected results, and matching function: 12/03/24 13:47:46 INFO cli.TestCLI: 146: refreshServiceAcl: refreshing security authorization policy for namenode 12/03/24 13:47:46 INFO cli.TestCLI: 147: refreshServiceAcl: refreshing security authorization policy for jobtracker
        Hide
        David Liu added a comment -

        Please review the patch attached.

        Show
        David Liu added a comment - Please review the patch attached.
        Hide
        Roman Shaposhnik added a comment -

        David, thanks for the patch. It looks good. That said, I still have one question for you before we can commit it: am I correct in my understanding that if you try running the same HDFS command manually you're seeing the following message on your terminal:

        12/03/23 23:47:53 INFO authorize.ServiceAuthorizationManager: Authorization successfull for ubuntu for protocol=interface org.apache.hadoop.security.authorize.RefreshAuthorizationPolicyProtocol
        

        If that's the case it would be prudent to figure out whether we need to fix our log4j settings to NOT see this spurious INFO printed out.

        Can you, please, take a look?

        Show
        Roman Shaposhnik added a comment - David, thanks for the patch. It looks good. That said, I still have one question for you before we can commit it: am I correct in my understanding that if you try running the same HDFS command manually you're seeing the following message on your terminal: 12/03/23 23:47:53 INFO authorize.ServiceAuthorizationManager: Authorization successfull for ubuntu for protocol=interface org.apache.hadoop.security.authorize.RefreshAuthorizationPolicyProtocol If that's the case it would be prudent to figure out whether we need to fix our log4j settings to NOT see this spurious INFO printed out. Can you, please, take a look?
        Hide
        David Liu added a comment -

        Roman,

        Thanks for the feedback. Here is the output of the command run on command
        line.

        $ hadoop dfsadmin -Dhadoop.job.ugi=blah,blah -refreshServiceAcl
        refreshServiceAcl:
        $

        So the INFO message is part of the test print out, not coming from the
        command line.

        Thanks
        David

        On 3/28/12 10:15 AM, "Roman Shaposhnik (Commented) (JIRA)"

        Show
        David Liu added a comment - Roman, Thanks for the feedback. Here is the output of the command run on command line. $ hadoop dfsadmin -Dhadoop.job.ugi=blah,blah -refreshServiceAcl refreshServiceAcl: $ So the INFO message is part of the test print out, not coming from the command line. Thanks David On 3/28/12 10:15 AM, "Roman Shaposhnik (Commented) (JIRA)"
        Hide
        Roman Shaposhnik added a comment -

        Sorry, I guess I'm confused – if the output is not there when you run it manually how come we have to account for it in the tests?

        Show
        Roman Shaposhnik added a comment - Sorry, I guess I'm confused – if the output is not there when you run it manually how come we have to account for it in the tests?
        Hide
        Roman Shaposhnik added a comment -

        DUP of BIGTOP-895

        Show
        Roman Shaposhnik added a comment - DUP of BIGTOP-895

          People

          • Assignee:
            David Liu
            Reporter:
            David Liu
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development