Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-9574

Log all measurement exceptions at debug level

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: meas_base
    • Labels:
      None
    • Story Points:
      4
    • Sprint:
      DRP S17-4, DRP S17-5
    • Team:
      Data Release Production

      Description

      The measurement framework should probably log all exceptions at debug level, even the ones thrown via MeasurementError that will result in a specific flag being set. This is a bit tricky because it probably needs to be done in per-Plugin loggers that are children of the measurement task's logger, so logging can be configured differently for different plugins. We should probably have the measurement framework create those loggers and pass them to Plugin constructors.

        Attachments

          Issue Links

            Activity

            Hide
            swinbank John Swinbank added a comment -

            Is it necessary that Jim Bosch review this work? I suspect we can find somebody else who is less overloaded.

            Show
            swinbank John Swinbank added a comment - Is it necessary that Jim Bosch review this work? I suspect we can find somebody else who is less overloaded.
            Hide
            pgee Perry Gee added a comment -

            This is a follow along to DM-430, which involved changing just a couple of modules. However, I won't create a pull request until DM-430 is done.

            Show
            pgee Perry Gee added a comment - This is a follow along to DM-430 , which involved changing just a couple of modules. However, I won't create a pull request until DM-430 is done.
            Hide
            pgee Perry Gee added a comment -

            mainpage.dox for 430/9574 added to this ticket.

            See tickets/DM-9574

            Show
            pgee Perry Gee added a comment - mainpage.dox for 430/9574 added to this ticket. See tickets/ DM-9574
            Hide
            rearmstr Bob Armstrong added a comment -

            Good work. Just two minor comments I made on github.

            Show
            rearmstr Bob Armstrong added a comment - Good work. Just two minor comments I made on github.
            Hide
            pgee Perry Gee added a comment -

            Changes made to mainpage.dox as requested.

            Show
            pgee Perry Gee added a comment - Changes made to mainpage.dox as requested.

              People

              • Assignee:
                pgee Perry Gee
                Reporter:
                jbosch Jim Bosch
                Reviewers:
                Bob Armstrong
                Watchers:
                Bob Armstrong, Jim Bosch, John Swinbank, Nate Lust, Perry Gee
              • Votes:
                0 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: