Details

    • Sub-task
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 1.2.3
    • 1.2.3
    • None
    • None

    Description

      Changes to configurations need to also be applied to service checks.

      Attachments

        1. AMBARI-1703.patch
          6 kB
          Nate Cole

        Activity

          ncole@hortonworks.com Nate Cole added a comment -

          Uploaded patch.

          ncole@hortonworks.com Nate Cole added a comment - Uploaded patch.
          mahadev Mahadev Konar added a comment -

          +1 for the patch. One minor nit, can we write a unit test for this so that this does not break again?

          mahadev Mahadev Konar added a comment - +1 for the patch. One minor nit, can we write a unit test for this so that this does not break again?
          ncole@hortonworks.com Nate Cole added a comment -

          I'll see if we can somehow test.

          Thanks

          ncole@hortonworks.com Nate Cole added a comment - I'll see if we can somehow test. Thanks
          ncole@hortonworks.com Nate Cole added a comment -

          Ok, my bad - the test was already being run, just without configurations. Please review again. Thanks.

          ncole@hortonworks.com Nate Cole added a comment - Ok, my bad - the test was already being run, just without configurations. Please review again. Thanks.
          mahadev Mahadev Konar added a comment -

          +1 on the patch. Thanks a lot Nate!

          mahadev Mahadev Konar added a comment - +1 on the patch. Thanks a lot Nate!
          ncole@hortonworks.com Nate Cole added a comment -

          Committed revision 1461246.

          ncole@hortonworks.com Nate Cole added a comment - Committed revision 1461246.

          People

            ncole@hortonworks.com Nate Cole
            ncole@hortonworks.com Nate Cole
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: