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

Ensure all loggers include lsst prefix

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: pipe_base
    • Labels:
      None
    • Story Points:
      8
    • Team:
      Architecture
    • Urgent?:
      No

      Description

      Ensure that all the loggers created for tasks include the lsst prefix.

        Attachments

          Issue Links

          is duplicated by

          Bug - A problem which impairs or prevents the functions of the product. DM-32143 Gen 3 ap_verify ingestion does not produce logs

          • Invalid

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-31802 Migrate standalone loggers to use lsst prefix

          • Invalid
          is triggered by

          RFC - An issue whose primary purpose is to advertise an intent and allow stakeholders to comment. RFC-801 Use lsst prefix in logger names

          • Implemented

            Activity

            Ascending order - Click to sort in descending order
            5 older comments
            Hide
            Permalink
            czw Christopher Waters added a comment -

            I've looked over cp_pipe and ip_isr and both look fine.

            Show
            czw Christopher Waters added a comment - I've looked over cp_pipe and ip_isr and both look fine.
            Hide
            Permalink
            ktl Kian-Tat Lim added a comment -

            pipe_base is OK, with one minor comment.

            Show
            ktl Kian-Tat Lim added a comment - pipe_base is OK, with one minor comment.
            Hide
            Permalink
            lskelvin Lee Kelvin added a comment -

            Just one minor comment re: string syntax in meas_deblender, but otherwise looks good to me - thanks Tim!

            Show
            lskelvin Lee Kelvin added a comment - Just one minor comment re: string syntax in meas_deblender , but otherwise looks good to me - thanks Tim!
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            Thank you everyone. I think that's all reviews in. I really do appreciate the speed.

            Show
            tjenness Tim Jenness added a comment - Thank you everyone. I think that's all reviews in. I really do appreciate the speed.
            Hide
            Permalink
            lauren Lauren MacArthur added a comment -

            You got an efficient set of reviewers indeed, and I will take the honour of officially hitting the button (with the requisite “as long as Jenkins is happy” clause)!

            Show
            lauren Lauren MacArthur added a comment - You got an efficient set of reviewers indeed, and I will take the honour of officially hitting the button (with the requisite “as long as Jenkins is happy” clause)!

              People

              Assignee:
              tjenness Tim Jenness
              Reporter:
              tjenness Tim Jenness
              Reviewers:
              Lauren MacArthur
              Watchers:
              Andrés Alejandro Plazas Malagón, Andy Salnikov, Christopher Waters, Ian Sullivan, John Parejko, Kian-Tat Lim, Lauren MacArthur, Lee Kelvin, Meredith Rawls, Nate Lust, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              11 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.