New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adding ability to set the serviceAccountConfig.limitSecretReferences setting in master-config.yaml #4516
Conversation
@sdodson @mtnbikenc, I tested this against 3.6 and this worked. This is in regards to the bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1442332 |
roles/openshift_master/README.md
Outdated
| openshift_master_console_url | UNDEF | | | ||
| openshift_master_public_api_url | UNDEF | | | ||
| openshift_master_public_console_url | UNDEF | | | ||
| openshift_master_saconfig_limitsecretrefereces | UNDEF | | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo, missing 'n', openshift_master_saconfig_limitsecretreferences
Also, it seems to me that this should reference the default value as being false
, since that is what happens in the code. Possibly also add the description that is in the inventory example.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mtnbikenc, Thanks, let me fix.
aos-ci-test |
[merge][severity:blocker] |
Evaluated for openshift ansible merge up to c140a54 |
[test]ing while waiting on the merge queue |
Evaluated for openshift ansible test up to c140a54 |
continuous-integration/openshift-jenkins/test FAILURE (https://ci.openshift.redhat.com/jenkins/job/test_pull_request_openshift_ansible/258/) (Base Commit: de93b27) (PR Branch Commit: c140a54) |
continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/merge_pull_request_openshift_ansible/621/) (Base Commit: 7db01fe) (PR Branch Commit: c140a54) (Extended Tests: blocker) |
No description provided.