Merge pull request #42 from lsst/tickets/DM-31384

DM-31384: Add --log-label to bps command
lsst · Aug 12, 2021 · 4eaa668 · 4eaa668
2 parents a0c8eb1 + a8a74ff
commit 4eaa668
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 2 deletions.
diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md
@@ -0,0 +1,4 @@
+## Checklist
+
+- [ ] ran Jenkins
+- [ ] added a release note for user-visible changes to `doc/changes`
diff --git a/doc/changes/DM-31884.feature.rst b/doc/changes/DM-31884.feature.rst
@@ -0,0 +1 @@
+Add ``--log-label`` option to ``bps`` command to allow extra information to be injected into the log record.
diff --git a/python/lsst/ctrl/bps/cli/bps.py b/python/lsst/ctrl/bps/cli/bps.py
@@ -21,7 +21,13 @@
 import click
 
 from lsst.daf.butler.cli.butler import LoaderCLI
-from lsst.daf.butler.cli.opt import log_level_option, long_log_option, log_file_option, log_tty_option
+from lsst.daf.butler.cli.opt import (
+    log_level_option,
+    long_log_option,
+    log_file_option,
+    log_tty_option,
+    log_label_option,
+)
 from lsst.daf.butler.cli.utils import unwrap
 
 
@@ -48,7 +54,8 @@ class BpsCli(LoaderCLI):
 @long_log_option()
 @log_file_option()
 @log_tty_option()
-def cli(log_level, long_log, log_file, log_tty):
+@log_label_option()
+def cli(log_level, long_log, log_file, log_tty, log_label):
     pass