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

Output jointcal metrics via a metrics logger

    XMLWordPrintable

    Details

    • Story Points:
      5
    • Epic Link:
    • Sprint:
      Alert Production S17 - 2, Alert Production S17 - 3, Alert Production S17 - 4, Alert Production S17 - 5, Alert Production S17 - 6, Alert Production F17 - 9, Alert Production F17 - 10, Alert Production F17 - 11
    • Team:
      Alert Production

      Description

      Until we have butler metrics persistence system, we can use a dedicated logger to output each product's metrics. Since jointcal is the testbed for the new metrics system, we'll use it as an example for how to produce those logs.

        Attachments

          Issue Links

          is blocked by

          RFC - An issue whose primary purpose is to advertise an intent and allow stakeholders to comment. RFC-345 Add verify and verify_metrics to lsst_distrib

          • Implemented
          relates to

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-12253 Port validate_drp to lsst.verify

          • Done

            Activity

            Ascending order - Click to sort in descending order
            17 older comments
            Hide
            Permalink
            Parejkoj John Parejko added a comment - - edited

            Simon Krughoff I've redone the name changes (effectively rebase to master, but really just redo work on master). On slack you said you could convert this to use verify.Job, so I'll leave that to you.

            Let me know if you want help dealing with the existing unittests of measurements.

            Show
            Parejkoj John Parejko added a comment - - edited Simon Krughoff I've redone the name changes (effectively rebase to master, but really just redo work on master). On slack you said you could convert this to use verify.Job , so I'll leave that to you. Let me know if you want help dealing with the existing unittests of measurements.
            Hide
            Permalink
            krughoff Simon Krughoff added a comment -

            As mentioned on the PR for jointcal, I believe I'm done. Reassigning to John Parejko for final signoff.

            Show
            krughoff Simon Krughoff added a comment - As mentioned on the PR for jointcal , I believe I'm done. Reassigning to John Parejko for final signoff.
            Hide
            Permalink
            krughoff Simon Krughoff added a comment -

            I have merged the verify version of this since it was pretty much orthogonal.

            Show
            krughoff Simon Krughoff added a comment - I have merged the verify version of this since it was pretty much orthogonal.
            Hide
            Permalink
            Parejkoj John Parejko added a comment -

            I've rebased jointcal to master, squashed all the changes, and cleaned it up a bit. I won't push any new jointcal changes until this gets merged, to avoid further conflicts. I also fixed Simon Krughoff's truncated commit message in obs_base, rebased it to master, made the PR for it.

            New Jenkins run here: https://ci.lsst.codes/job/stack-os-matrix/27092/

            Simon Krughoff can you please give the PRs a quick once-over now that I've rebased them, to make sure I didn't fowl anything up? Once you give the word and if the above jenkins run passes, either of us should be ok to merge the remaining 3 PRs.

            Show
            Parejkoj John Parejko added a comment - I've rebased jointcal to master, squashed all the changes, and cleaned it up a bit. I won't push any new jointcal changes until this gets merged, to avoid further conflicts. I also fixed Simon Krughoff 's truncated commit message in obs_base, rebased it to master, made the PR for it. New Jenkins run here: https://ci.lsst.codes/job/stack-os-matrix/27092/ Simon Krughoff can you please give the PRs a quick once-over now that I've rebased them, to make sure I didn't fowl anything up? Once you give the word and if the above jenkins run passes, either of us should be ok to merge the remaining 3 PRs.
            Hide
            Permalink
            Parejkoj John Parejko added a comment -

            Happier jenkins: https://ci.lsst.codes/job/stack-os-matrix/27095/

            Simon Krughoff cleaned up the one comment on obs_base.

            Merged and done!

            Show
            Parejkoj John Parejko added a comment - Happier jenkins: https://ci.lsst.codes/job/stack-os-matrix/27095/ Simon Krughoff cleaned up the one comment on obs_base. Merged and done!

              People

              Assignee:
              krughoff Simon Krughoff
              Reporter:
              Parejkoj John Parejko
              Reviewers:
              John Parejko
              Watchers:
              Angelo Fausti, Frossie Economou, John Parejko, John Swinbank, Jonathan Sick, Michael Wood-Vasey, Simon Krughoff, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              8 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.