From b0e8be618f0e73af454da3223eaaf8e2aec25686 Mon Sep 17 00:00:00 2001
From: "James E. Blair" <jeblair@hp.com>
Date: Wed, 11 Feb 2015 18:09:20 -0800
Subject: [PATCH] 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
---
 tests/test_scheduler.py | 1 -
 zuul/cmd/client.py      | 3 ---
 zuul/model.py           | 1 -
 3 files changed, 5 deletions(-)

diff --git a/tests/test_scheduler.py b/tests/test_scheduler.py
index 4c8c832f7c..b44dba6c15 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 766a4ef83b..bc2c152d80 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 1786fd9bd1..f530a91f96 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
             })