summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorOpenShift Merge Robot <openshift-merge-robot@users.noreply.github.com>2017-12-23 23:23:34 -0800
committerGitHub <noreply@github.com>2017-12-23 23:23:34 -0800
commit88b74bd543d2ac54b10475daef5a824411e98e55 (patch)
treeb6096e71d388deceaf00b0f084c047f080c239f8
parent9784668b6ea9aa0ba28d5ab0fe81cd7d366f69b6 (diff)
parentfd882160aa2ac03b6f3dc84647b227c65db50635 (diff)
downloadopenshift-88b74bd543d2ac54b10475daef5a824411e98e55.tar.gz
openshift-88b74bd543d2ac54b10475daef5a824411e98e55.tar.bz2
openshift-88b74bd543d2ac54b10475daef5a824411e98e55.tar.xz
openshift-88b74bd543d2ac54b10475daef5a824411e98e55.zip
Merge pull request #6497 from kwoodson/remove_docker_creds
Automatic merge from submit-queue. Adding no_log to registry_auth. We were seeing credentials show up in our jenkins logs. We'd prefer if they were not logged. I'm not sure the repercussions but these are sensitive and probably should be `no_log`.
-rw-r--r--roles/container_runtime/tasks/registry_auth.yml2
1 files changed, 2 insertions, 0 deletions
diff --git a/roles/container_runtime/tasks/registry_auth.yml b/roles/container_runtime/tasks/registry_auth.yml
index 2c7bc5711..4f1abd59a 100644
--- a/roles/container_runtime/tasks/registry_auth.yml
+++ b/roles/container_runtime/tasks/registry_auth.yml
@@ -15,6 +15,7 @@
- not openshift_docker_alternative_creds | bool
- oreg_auth_user is defined
- (not docker_cli_auth_credentials_stat.stat.exists or oreg_auth_credentials_replace) | bool
+ no_log: True
# docker_creds is a custom module from lib_utils
# 'docker login' requires a docker.service running on the local host, this is an
@@ -30,3 +31,4 @@
- openshift_docker_alternative_creds | bool
- oreg_auth_user is defined
- (not docker_cli_auth_credentials_stat.stat.exists or oreg_auth_credentials_replace) | bool
+ no_log: True