diff options
author | James E. Blair <jeblair@hp.com> | 2015-02-11 18:09:20 -0800 |
---|---|---|
committer | James E. Blair <jeblair@hp.com> | 2015-02-25 09:54:04 -0800 |
commit | b0e8be618f0e73af454da3223eaaf8e2aec25686 (patch) | |
tree | 0e4da4e86bb2a928c0129be2b3204884745617dc | |
parent | 458970bb627fd9082dedb6ae574b6841abe24232 (diff) | |
download | zuul-b0e8be618f0e73af454da3223eaaf8e2aec25686.tar.gz |
Remove job parameters from status.json
They don't have anything particularly sensitive now, but they might
in the future, and they can be quite large (thus making status.json
large).
Since the zuul client command uses the same data, remove the
parameters column from its output.
Change-Id: I681c576a25012a6fb0733f468778576d6003c088
-rwxr-xr-x | tests/test_scheduler.py | 1 | ||||
-rw-r--r-- | zuul/cmd/client.py | 3 | ||||
-rw-r--r-- | zuul/model.py | 1 |
3 files changed, 0 insertions, 5 deletions
diff --git a/tests/test_scheduler.py b/tests/test_scheduler.py index 4c8c832f7..b44dba6c1 100755 --- a/tests/test_scheduler.py +++ b/tests/test_scheduler.py @@ -3072,7 +3072,6 @@ For CI problems and help debugging, contact ci@example.org""" self.assertEqual('project-merge', job['name']) self.assertEqual('gate', job['pipeline']) self.assertEqual(False, job['retry']) - self.assertEqual(13, len(job['parameters'])) self.assertEqual('https://server/job/project-merge/0/', job['url']) self.assertEqual(7, len(job['worker'])) diff --git a/zuul/cmd/client.py b/zuul/cmd/client.py index 766a4ef83..bc2c152d8 100644 --- a/zuul/cmd/client.py +++ b/zuul/cmd/client.py @@ -232,9 +232,6 @@ class Client(zuul.cmd.ZuulApp): 'number': { 'title': 'Number' }, - 'parameters': { - 'title': 'Parameters' - }, 'worker.name': { 'title': 'Worker' }, diff --git a/zuul/model.py b/zuul/model.py index 1786fd9bd..f530a91f9 100644 --- a/zuul/model.py +++ b/zuul/model.py @@ -780,7 +780,6 @@ class QueueItem(object): 'canceled': build.canceled if build else None, 'retry': build.retry if build else None, 'number': build.number if build else None, - 'parameters': build.parameters if build else None, 'worker': worker }) |