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

Fix faro sky object metric lookup name

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: faro
    • Labels:
      None
    • Story Points:
      2
    • Epic Link:
    • Team:
      Data Release Production
    • Urgent?:
      No

      Description

      The sky object metric values defined in verify_metrics start with a lowercase 's', whereas faro has been configured to look for an uppercase 's'. This ticket fixes that discrepancy.

        Attachments

          Issue Links

          relates to

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-34489 Add faro flux stat task into RC2/DC2 pipelines

          • Done

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            jcarlin Jeffrey Carlin added a comment -

            Looks good - the metrics should (hopefully) appear on the dashboard tomorrow evening.

            Show
            jcarlin Jeffrey Carlin added a comment - Looks good - the metrics should (hopefully) appear on the dashboard tomorrow evening.
            Hide
            Permalink
            lskelvin Lee Kelvin added a comment -

            Thanks for the review Jeffrey Carlin, and more importantly, thank you for the catch! I've kicked off a Jenkins run just to be safe. I didn't need to change the task name, but, to maintain consistency with the names in verify_metrics, I changed those too.

            Show
            lskelvin Lee Kelvin added a comment - Thanks for the review Jeffrey Carlin , and more importantly, thank you for the catch! I've kicked off a Jenkins run just to be safe. I didn't need to change the task name, but, to maintain consistency with the names in verify_metrics , I changed those too.
            Hide
            Permalink
            lskelvin Lee Kelvin added a comment - - edited

            Jenkins successfully informed me that I need to also update the task names in all of the pipeline files I've touched (thanks Jenkins). Additional PRs for both drp_pipe and rc2_subset also added. I'm not sure if you want to look at these again or not Jeffrey Carlin? A new Jenkins is currently running with the above PRs in place.

            faro PR: https://github.com/lsst/faro/pull/135/files
            drp_pipe PR: https://github.com/lsst/drp_pipe/pull/19/files
            rc2_subset PR: https://github.com/lsst-dm/rc2_subset/pull/13/files

            Show
            lskelvin Lee Kelvin added a comment - - edited Jenkins successfully informed me that I need to also update the task names in all of the pipeline files I've touched (thanks Jenkins). Additional PRs for both drp_pipe and rc2_subset also added. I'm not sure if you want to look at these again or not Jeffrey Carlin ? A new Jenkins is currently running with the above PRs in place. faro PR: https://github.com/lsst/faro/pull/135/files drp_pipe PR: https://github.com/lsst/drp_pipe/pull/19/files rc2_subset PR: https://github.com/lsst-dm/rc2_subset/pull/13/files
            Hide
            Permalink
            jcarlin Jeffrey Carlin added a comment -

            Oh, right - thanks, Jenkins!

            PRs approved - glad you ran Jenkins and caught that!

             

            Show
            jcarlin Jeffrey Carlin added a comment - Oh, right - thanks, Jenkins! PRs approved - glad you ran Jenkins and caught that!  
            Hide
            Permalink
            lskelvin Lee Kelvin added a comment -

            Thanks again Jeff. All seemed well with Jenkins; branch merged and deleted. Furthermore, I've set up two panels on the nightly RC2 dashboard to track these metrics, and all seems to be working as expected!

            Show
            lskelvin Lee Kelvin added a comment - Thanks again Jeff. All seemed well with Jenkins; branch merged and deleted. Furthermore, I've set up two panels on the nightly RC2 dashboard to track these metrics, and all seems to be working as expected!

              People

              Assignee:
              lskelvin Lee Kelvin
              Reporter:
              lskelvin Lee Kelvin
              Reviewers:
              Jeffrey Carlin
              Watchers:
              Jeffrey Carlin, Lee Kelvin
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.