Accumulo
  1. Accumulo
  2. ACCUMULO-1870

Functional tests don't configure Monitor log

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.4.5, 1.5.1
    • Fix Version/s: 1.4.5, 1.5.1
    • Component/s: test
    • Labels:

      Description

      In reviewing ACCUMULO-1658, we missed one line that has an extra level of "conf":

      test/system/auto/TestUtils.py line 50

      LOG_MONITOR = os.path.join(ACCUMULO_CONF_DIR, 'conf', 'monitor_logger.xml')
      

      I believe the net result is tests end up using the original monitor logger file.

        Issue Links

          Activity

          Hide
          Josh Elser added a comment -

          Applied to 1.4.5-SNAPSHOT and 1.5.1-SNAPSHOT, OBE in >=1.6.0-SNAPSHOT.

          Thanks for the patch, Ryan Fishel; sorry for the delay.

          Show
          Josh Elser added a comment - Applied to 1.4.5-SNAPSHOT and 1.5.1-SNAPSHOT, OBE in >=1.6.0-SNAPSHOT. Thanks for the patch, Ryan Fishel ; sorry for the delay.
          Hide
          ASF subversion and git services added a comment -

          Commit 1e96138bb544e02829474011df850d5f28f299e6 in branch refs/heads/master from ofishel
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=1e96138 ]

          ACCUMULO-1870 Functional tests don't configure Monitor log

          Signed-off-by: Josh Elser <elserj@apache.org>

          Show
          ASF subversion and git services added a comment - Commit 1e96138bb544e02829474011df850d5f28f299e6 in branch refs/heads/master from ofishel [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=1e96138 ] ACCUMULO-1870 Functional tests don't configure Monitor log Signed-off-by: Josh Elser <elserj@apache.org>
          Hide
          ASF subversion and git services added a comment -

          Commit 1e96138bb544e02829474011df850d5f28f299e6 in branch refs/heads/1.6.0-SNAPSHOT from ofishel
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=1e96138 ]

          ACCUMULO-1870 Functional tests don't configure Monitor log

          Signed-off-by: Josh Elser <elserj@apache.org>

          Show
          ASF subversion and git services added a comment - Commit 1e96138bb544e02829474011df850d5f28f299e6 in branch refs/heads/1.6.0-SNAPSHOT from ofishel [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=1e96138 ] ACCUMULO-1870 Functional tests don't configure Monitor log Signed-off-by: Josh Elser <elserj@apache.org>
          Hide
          ASF subversion and git services added a comment -

          Commit 1e96138bb544e02829474011df850d5f28f299e6 in branch refs/heads/1.5.1-SNAPSHOT from ofishel
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=1e96138 ]

          ACCUMULO-1870 Functional tests don't configure Monitor log

          Signed-off-by: Josh Elser <elserj@apache.org>

          Show
          ASF subversion and git services added a comment - Commit 1e96138bb544e02829474011df850d5f28f299e6 in branch refs/heads/1.5.1-SNAPSHOT from ofishel [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=1e96138 ] ACCUMULO-1870 Functional tests don't configure Monitor log Signed-off-by: Josh Elser <elserj@apache.org>
          Hide
          ASF subversion and git services added a comment -

          Commit 1e96138bb544e02829474011df850d5f28f299e6 in branch refs/heads/1.4.5-SNAPSHOT from ofishel
          [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=1e96138 ]

          ACCUMULO-1870 Functional tests don't configure Monitor log

          Signed-off-by: Josh Elser <elserj@apache.org>

          Show
          ASF subversion and git services added a comment - Commit 1e96138bb544e02829474011df850d5f28f299e6 in branch refs/heads/1.4.5-SNAPSHOT from ofishel [ https://git-wip-us.apache.org/repos/asf?p=accumulo.git;h=1e96138 ] ACCUMULO-1870 Functional tests don't configure Monitor log Signed-off-by: Josh Elser <elserj@apache.org>
          Hide
          Sean Busbey added a comment -

          Increasing the priority to Major, because this actually causes the functional tests to fail unless you create a $ACCUMULO_CONF_DIR/conf directory

          Show
          Sean Busbey added a comment - Increasing the priority to Major, because this actually causes the functional tests to fail unless you create a $ACCUMULO_CONF_DIR/conf directory
          Hide
          Sean Busbey added a comment -

          +1 (non-binding) looks good.

          Show
          Sean Busbey added a comment - +1 (non-binding) looks good.

            People

            • Assignee:
              Ryan Fishel
              Reporter:
              Sean Busbey
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development