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

Output jointcal metrics via a metrics logger

    Details

    • Story Points:
      5
    • 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

            Activity

            Hide
            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!
            Hide
            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
            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
            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
            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.

              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:

                  Summary Panel