From 0022884dc21367c5965bb865c534b3b8cc382b98 Mon Sep 17 00:00:00 2001 From: Tobias Henkel Date: Fri, 31 Jan 2020 11:33:07 +0100 Subject: [PATCH] Deprecate -d switch for running in foreground Until the -f switch was introduced the only way of running in foreground was the -d switch which also implied debug logging. We now have the -f switch so we can decouple that in future. Thus depreciate the -d switch for the sake of running in forground. Change-Id: Ic66ba1a28b8e2837309df49fc0ff28d2495fc229 --- releasenotes/notes/foreground-c210261012f433fd.yaml | 6 ++++++ zuul/cmd/__init__.py | 6 +++++- 2 files changed, 11 insertions(+), 1 deletion(-) create mode 100644 releasenotes/notes/foreground-c210261012f433fd.yaml diff --git a/releasenotes/notes/foreground-c210261012f433fd.yaml b/releasenotes/notes/foreground-c210261012f433fd.yaml new file mode 100644 index 0000000000..755f8d56d1 --- /dev/null +++ b/releasenotes/notes/foreground-c210261012f433fd.yaml @@ -0,0 +1,6 @@ +--- +deprecations: + - | + Running the zuul processes with the -d switch to enforce running in + foreground is now deprecated. Switch to use -f instead for this purpose. + The -d switch will change to just enable debug logging in the future. diff --git a/zuul/cmd/__init__.py b/zuul/cmd/__init__.py index 9e5c6d40b1..77e5458ee1 100755 --- a/zuul/cmd/__init__.py +++ b/zuul/cmd/__init__.py @@ -174,7 +174,11 @@ class ZuulDaemonApp(ZuulApp, metaclass=abc.ABCMeta): def createParser(self): parser = super(ZuulDaemonApp, self).createParser() parser.add_argument('-d', dest='debug', action='store_true', - help='run in foreground with debug log') + help='run in foreground with debug log. Note ' + 'that in future this will be changed to only ' + 'request debug logging. If you want to keep ' + 'running the process in the foreground ' + 'migrate/add the -f switch.') parser.add_argument('-f', dest='foreground', action='store_true', help='run in foreground with info log') return parser