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

Add time stamps to the standard outputs to BatchCmdLineTask

    Details

    • Type: Improvement
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ctrl_pool
    • Labels:
      None

      Description

      Add time stamps to the logs by default in ctrl_pool jobs.
      In slurm jobs, this changes the log format in the JOBNAME.oJOBID log files.

        Attachments

          Issue Links

            Activity

            Hide
            hchiang2 Hsin-Fang Chiang added a comment -

            only a bandaid...

            Show
            hchiang2 Hsin-Fang Chiang added a comment - only a bandaid...
            Hide
            price Paul Price added a comment -

            Great, thanks for doing this! A minor comment is on the GitHub PR.
            Should we do the same in pipe_tasks?

            Show
            price Paul Price added a comment - Great, thanks for doing this! A minor comment is on the GitHub PR. Should we do the same in pipe_tasks?
            Hide
            hchiang2 Hsin-Fang Chiang added a comment -

            We could add time stamps to the pipe_base default too; it would be here. I'm not as certain whether there would be objections; I have a vague impression that some people wanted to keep the default logs clean and short. Users can get a longer format using --longlog in CmdLineTask, though.

            Show
            hchiang2 Hsin-Fang Chiang added a comment - We could add time stamps to the pipe_base default too; it would be here . I'm not as certain whether there would be objections; I have a vague impression that some people wanted to keep the default logs clean and short. Users can get a longer format using --longlog in CmdLineTask, though.
            Hide
            hchiang2 Hsin-Fang Chiang added a comment -

            Thanks for bringing this up and thanks for the review. Merged to master.

            I think I'm not gonna change the pipe_base default for now, as the optional --longlog can provide the time stamps relatively easily for the CmdLineTasks.

            Show
            hchiang2 Hsin-Fang Chiang added a comment - Thanks for bringing this up and thanks for the review. Merged to master. I think I'm not gonna change the pipe_base default for now, as the optional --longlog can provide the time stamps relatively easily for the CmdLineTasks.

              People

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

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel