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

Use lsst::log in pipe_base and pipe_tasks

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: pipe_base, pipe_tasks
    • Labels:
      None

      Description

      Per RFC-203, switch Task's logs from using pex.logging to lsst.log in pipe_base and pipe_tasks (stage 2)
      This implements the main idea of RFC-203 and migrate the logging framework used in the task framework.

        Attachments

          Issue Links

            Activity

            Hide
            rowen Russell Owen added a comment -

            For the record, here is the table of old levels vs new levels:

                      pex_logging       log
            TRACE     -trace level     5000
            DEBUG     -10             10000
            INFO        0             20000
            WARN[ING]  10             30000
            ERROR     N/A             40000
            FATAL      20             50000
            

            Show
            rowen Russell Owen added a comment - For the record, here is the table of old levels vs new levels: pex_logging log TRACE -trace level 5000 DEBUG -10 10000 INFO 0 20000 WARN[ING] 10 30000 ERROR N/A 40000 FATAL 20 50000
            Hide
            hchiang2 Hsin-Fang Chiang added a comment -

            Many thanks to the reviewers Russell Owen, Ian Sullivan, and Vishal Kasliwal [X] for reviewing the 11 packages! I think I have addressed all concerns brought up during the review. I'm going though the changes a few more times, running CI, and will run some command line tasks locally as extra manual tests. After that I'll merge the changes. Afterwards I will post on c.l.o. to announce the changes.

            Show
            hchiang2 Hsin-Fang Chiang added a comment - Many thanks to the reviewers Russell Owen , Ian Sullivan , and Vishal Kasliwal [X] for reviewing the 11 packages! I think I have addressed all concerns brought up during the review. I'm going though the changes a few more times, running CI, and will run some command line tasks locally as extra manual tests. After that I'll merge the changes. Afterwards I will post on c.l.o. to announce the changes.
            Show
            hchiang2 Hsin-Fang Chiang added a comment - Passed Jenkins, with lsst_ci : https://ci.lsst.codes/job/stack-os-matrix/label=centos-7,python=py2/15244//console https://ci.lsst.codes/job/stack-os-matrix/label=centos-6,python=py2/15244//console
            Show
            hchiang2 Hsin-Fang Chiang added a comment - It seems that ci_hsc is not included in lsst_ci , so here are Jenkins for ci_hsc : https://ci.lsst.codes/job/stack-os-matrix/label=centos-7,python=py2/15250//console https://ci.lsst.codes/job/stack-os-matrix/label=centos-6,python=py2/15250//console
            Hide
            hchiang2 Hsin-Fang Chiang added a comment -
            Show
            hchiang2 Hsin-Fang Chiang added a comment - Merged to master. Changes announced: https://community.lsst.org/t/logging-framework-transition/1114

              People

              • Assignee:
                hchiang2 Hsin-Fang Chiang
                Reporter:
                hchiang2 Hsin-Fang Chiang
                Reviewers:
                Russell Owen
                Watchers:
                Hsin-Fang Chiang, Paul Price, Russell Owen
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel