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

        David Liu created issue -
        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
        David Liu made changes -
        Field Original Value New Value
        Attachment BIGTOP-489.patch.txt [ 12519822 ]
        Hide
        David Liu added a comment -

        Please review the patch attached.

        Show
        David Liu added a comment - Please review the patch attached.
        David Liu made changes -
        Status Open [ 1 ] Patch Available [ 10002 ]
        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)"
        Roman Shaposhnik made changes -
        Fix Version/s 0.4.0 [ 12318889 ]
        Fix Version/s 0.3.0 [ 12317841 ]
        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?
        Gavin made changes -
        Workflow no-reopen-closed, patch-avail [ 12659485 ] patch-available, re-open possible [ 12665704 ]
        Roman Shaposhnik made changes -
        Fix Version/s 0.5.0 [ 12321865 ]
        Fix Version/s 0.4.0 [ 12318889 ]
        Roman Shaposhnik made changes -
        Fix Version/s 0.6.0 [ 12323895 ]
        Fix Version/s 0.5.0 [ 12321865 ]
        Hide
        Roman Shaposhnik added a comment -

        DUP of BIGTOP-895

        Show
        Roman Shaposhnik added a comment - DUP of BIGTOP-895
        Roman Shaposhnik made changes -
        Status Patch Available [ 10002 ] Resolved [ 5 ]
        Resolution Duplicate [ 3 ]
        Roman Shaposhnik made changes -
        Status Resolved [ 5 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Patch Available Patch Available
        1h 39m 1 David Liu 24/Mar/12 21:24
        Patch Available Patch Available Resolved Resolved
        396d 21h 33m 1 Roman Shaposhnik 25/Apr/13 18:58
        Resolved Resolved Closed Closed
        57d 4h 51m 1 Roman Shaposhnik 21/Jun/13 23:49

          People

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

            Dates

            • Created:
              Updated:
              Resolved:

              Development