Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-2430 Make qserv server-side log messages more standard
  3. DM-2780

Document and test how to log PID via lsst/log

    XMLWordPrintable

    Details

    • Type: Technical task
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: log
    • Labels:
      None
    • Story Points:
      2
    • Sprint:
      DB_S15_05, DB_S15_06
    • Team:
      Data Access and Database

      Attachments

        Issue Links

        relates to

        Technical task - Created by JIRA Agile - do not edit or delete. Issue type for a technical task. DM-2751 Allow lsst/log library to log PID on the C++ side

        • Won't Fix

          Activity

          Ascending order - Click to sort in descending order
          Hide
          Permalink
          jammes Fabrice Jammes added a comment -

          Code is in Qserv and log repos.

          Show
          jammes Fabrice Jammes added a comment - Code is in Qserv and log repos.
          Hide
          Permalink
          salnikov Andy Salnikov added a comment -

          I don't think any of the changes on this ticket are needed. We should discuss this at the meeting tomorrow.

          Show
          salnikov Andy Salnikov added a comment - I don't think any of the changes on this ticket are needed. We should discuss this at the meeting tomorrow.
          Hide
          Permalink
          jammes Fabrice Jammes added a comment - - edited

          Hi @Andy,

          Ok, I've removed the API code, but keep:

          • unit tests related to log PID (also for child process),
          • other minor improvment to manage lines number in expected log messages.
          • documentation

          Would you agree to review this one ticket please?

          Show
          jammes Fabrice Jammes added a comment - - edited Hi @Andy, Ok, I've removed the API code, but keep: unit tests related to log PID (also for child process), other minor improvment to manage lines number in expected log messages. documentation Would you agree to review this one ticket please?
          Hide
          Permalink
          salnikov Andy Salnikov added a comment -

          Fabrice, it looks mostly OK. I do not think we need any changes in qserv but log looks fine, with one minor comment.

          Show
          salnikov Andy Salnikov added a comment - Fabrice, it looks mostly OK. I do not think we need any changes in qserv but log looks fine, with one minor comment.

            People

            Assignee:
            jammes Fabrice Jammes
            Reporter:
            jammes Fabrice Jammes
            Reviewers:
            Andy Salnikov
            Watchers:
            Andy Salnikov, Fabrice Jammes, Jacek Becla
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: