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

Improvements to logging in xrootd

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: xrootd
    • Labels:
      None
    • Story Points:
      6
    • Sprint:
      DB_W16_02, DB_X16_05, DB_F16_6, DB_F16_7, DB_F16_8, DB_F16_9, DB_F16_10
    • Team:
      Data Access and Database

      Description

      Improve logging in xrootd to make it more compatible with qserv logging.

        Attachments

          Issue Links

            Activity

            Hide
            abh Andy Hanushevsky added a comment -

            I have pushed all the relevant changes to include trace and debugging messages in all forwarded output. That means these messages will also appear in the log4cxx log. This requires a sync of eh xrdssi (xrdssi-LSST) branch with the one that is used for eups.

            Show
            abh Andy Hanushevsky added a comment - I have pushed all the relevant changes to include trace and debugging messages in all forwarded output. That means these messages will also appear in the log4cxx log. This requires a sync of eh xrdssi (xrdssi-LSST) branch with the one that is used for eups.
            Hide
            salnikov Andy Salnikov added a comment -

            Looks mostly OK, few comments are on PR. One thing that I noticed is that messages that come from xrootd create empty lines in log file, it looks like they have trailing newline in the message, we should get rid of those.

            Show
            salnikov Andy Salnikov added a comment - Looks mostly OK, few comments are on PR. One thing that I noticed is that messages that come from xrootd create empty lines in log file, it looks like they have trailing newline in the message, we should get rid of those.
            Hide
            abh Andy Hanushevsky added a comment -

            I pushed changes as per review. We still need to consolidate the qserv logging function. The function will become a static method in a class whose name will correspond to the directory we decide this should go to.

            Show
            abh Andy Hanushevsky added a comment - I pushed changes as per review. We still need to consolidate the qserv logging function. The function will become a static method in a class whose name will correspond to the directory we decide this should go to.
            Hide
            abh Andy Hanushevsky added a comment -

            The updates have been merged to master. Since config files have changed, the new config files need to be reflected in the production setup and the cluster restarted.

            Show
            abh Andy Hanushevsky added a comment - The updates have been merged to master. Since config files have changed, the new config files need to be reflected in the production setup and the cluster restarted.

              People

              • Assignee:
                abh Andy Hanushevsky
                Reporter:
                fritzm Fritz Mueller
                Reviewers:
                Andy Salnikov
                Watchers:
                Andy Hanushevsky, Andy Salnikov, Jacek Becla
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel