Details

    • Type: Story
    • Status: Done
    • Priority: Major
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: pipe_base, pipe_tasks
    • Labels:
      None
    • Templates:
    • Story Points:
      8
    • Team:
      Process Middleware

      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.

        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 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 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:

                Development