Skip to content

Commit 1705ced

Browse files
authored
Revert "Remove kubeconfig value from module invocation log (#826)" (#899)
This reverts commit 6efabd3. SUMMARY Fixes #870 A better solution is necessary to address #782. The current code makes getting manifests practically unusable. We need to revert this commit until a better solution is found. ISSUE TYPE Bugfix Pull Request COMPONENT NAME kubeconfig Reviewed-by: Bianca Henderson <[email protected]>
1 parent 6a0635a commit 1705ced

File tree

3 files changed

+4
-2
lines changed

3 files changed

+4
-2
lines changed
Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
---
2+
bugfixes:
3+
- kubeconfig option should return the full manifest output (https://github.com/ansible-collections/kubernetes.core/issues/870).

plugins/module_utils/args_common.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ def list_dict_str(value):
1818
)
1919

2020
AUTH_ARG_SPEC = {
21-
"kubeconfig": {"type": "raw", "no_log": True},
21+
"kubeconfig": {"type": "raw"},
2222
"context": {},
2323
"host": {},
2424
"api_key": {"no_log": True},

plugins/module_utils/helm_args_common.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@
1616
type="raw",
1717
aliases=["kubeconfig_path"],
1818
fallback=(env_fallback, ["K8S_AUTH_KUBECONFIG"]),
19-
no_log=True,
2019
),
2120
host=dict(type="str", fallback=(env_fallback, ["K8S_AUTH_HOST"])),
2221
ca_cert=dict(

0 commit comments

Comments
 (0)