Hadoop Common
  1. Hadoop Common
  2. HADOOP-6970

SecurityAuth.audit should be generated under /build

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.22.0, 0.23.0
    • Fix Version/s: 0.22.0
    • Component/s: build
    • Labels:
      None
    • Hadoop Flags:
      Reviewed

      Description

      SecurityAuth.audit is generated under currently root project directory whenever I run anything, and is not being cleaned up by the clean target. It should be created under build directory instead.

        Issue Links

          Activity

          Hide
          Boris Shkolnik added a comment -

          Actually SecurityAuth.audit is a log file and should be generated under logs directory.
          Current problem is with the configuration, which by default creates the file in the current directory.

          Attached patch should fix that. If one is running test - no SecurityAuth.audit will be created at all. But if you start namenode - then the file will be created in the logs directory.

          Show
          Boris Shkolnik added a comment - Actually SecurityAuth.audit is a log file and should be generated under logs directory. Current problem is with the configuration, which by default creates the file in the current directory. Attached patch should fix that. If one is running test - no SecurityAuth.audit will be created at all. But if you start namenode - then the file will be created in the logs directory.
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12471126/SecurityAudit-Trunk.patch
          against trunk revision 1071084.

          +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 (version 1.3.9) 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.

          +1 system test framework. The patch passed system test framework compile.

          Test results: https://hudson.apache.org/hudson/job/PreCommit-HADOOP-Build/238//testReport/
          Findbugs warnings: https://hudson.apache.org/hudson/job/PreCommit-HADOOP-Build/238//artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html
          Console output: https://hudson.apache.org/hudson/job/PreCommit-HADOOP-Build/238//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/12471126/SecurityAudit-Trunk.patch against trunk revision 1071084. +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 (version 1.3.9) 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. +1 system test framework. The patch passed system test framework compile. Test results: https://hudson.apache.org/hudson/job/PreCommit-HADOOP-Build/238//testReport/ Findbugs warnings: https://hudson.apache.org/hudson/job/PreCommit-HADOOP-Build/238//artifact/trunk/build/test/findbugs/newPatchFindbugsWarnings.html Console output: https://hudson.apache.org/hudson/job/PreCommit-HADOOP-Build/238//console This message is automatically generated.
          Hide
          Konstantin Shvachko added a comment -

          +1 Worked for me.
          It looks like the same patch applies cleanly to 0.22. Should not be a problem committing.
          We probably need similar jiras for hdfs and mapreduce.

          Show
          Konstantin Shvachko added a comment - +1 Worked for me. It looks like the same patch applies cleanly to 0.22. Should not be a problem committing. We probably need similar jiras for hdfs and mapreduce.
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Common-trunk-Commit #507 (See https://hudson.apache.org/hudson/job/Hadoop-Common-trunk-Commit/507/)
          HADOOP-6970. SecurityAuth.audit should be generated under /build.

          Show
          Hudson added a comment - Integrated in Hadoop-Common-trunk-Commit #507 (See https://hudson.apache.org/hudson/job/Hadoop-Common-trunk-Commit/507/ ) HADOOP-6970 . SecurityAuth.audit should be generated under /build.
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Common-trunk-Commit #508 (See https://hudson.apache.org/hudson/job/Hadoop-Common-trunk-Commit/508/)
          HADOOP-6970. Reordering commit message in CHANGES.txt

          Show
          Hudson added a comment - Integrated in Hadoop-Common-trunk-Commit #508 (See https://hudson.apache.org/hudson/job/Hadoop-Common-trunk-Commit/508/ ) HADOOP-6970 . Reordering commit message in CHANGES.txt
          Hide
          Boris Shkolnik added a comment -

          Ok, I will commit it to trunk and 0.22.
          I don't think this is needed for hdfs or mapreduce because they don't have log4j conf file.

          Show
          Boris Shkolnik added a comment - Ok, I will commit it to trunk and 0.22. I don't think this is needed for hdfs or mapreduce because they don't have log4j conf file.
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Common-trunk #609 (See https://hudson.apache.org/hudson/job/Hadoop-Common-trunk/609/)
          HADOOP-6970. Reordering commit message in CHANGES.txt
          HADOOP-6970. SecurityAuth.audit should be generated under /build.

          Show
          Hudson added a comment - Integrated in Hadoop-Common-trunk #609 (See https://hudson.apache.org/hudson/job/Hadoop-Common-trunk/609/ ) HADOOP-6970 . Reordering commit message in CHANGES.txt HADOOP-6970 . SecurityAuth.audit should be generated under /build.
          Hide
          Hadoop QA added a comment -

          -1 overall. Here are the results of testing the latest attachment
          http://issues.apache.org/jira/secure/attachment/12471126/SecurityAudit-Trunk.patch
          against trunk revision 1071364.

          +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://hudson.apache.org/hudson/job/PreCommit-HADOOP-Build/258//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/12471126/SecurityAudit-Trunk.patch against trunk revision 1071364. +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://hudson.apache.org/hudson/job/PreCommit-HADOOP-Build/258//console This message is automatically generated.
          Hide
          Hudson added a comment -

          Integrated in Hadoop-Common-22-branch #29 (See https://hudson.apache.org/hudson/job/Hadoop-Common-22-branch/29/)

          Show
          Hudson added a comment - Integrated in Hadoop-Common-22-branch #29 (See https://hudson.apache.org/hudson/job/Hadoop-Common-22-branch/29/ )

            People

            • Assignee:
              Boris Shkolnik
              Reporter:
              Konstantin Shvachko
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development