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
Add option to disable Windows logging for Windows clusters #1133
Add option to disable Windows logging for Windows clusters #1133
Conversation
packages/rancher-logging/generated-changes/patch/values.yaml.patch
Outdated
Show resolved
Hide resolved
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.
Based on conversation with @vincent99 offline, I think we should not introduce a brand new field but rather comment out global.cattle.windows.enabled
in the values.yaml for this chart and introduce a slight change to the way we check whether it is enabled to allow it to be unset in the values.yaml.
See my comment in this review.
packages/rancher-logging/generated-changes/overlay/templates/loggings/root/logging.yaml
Outdated
Show resolved
Hide resolved
519ece2
to
bde0e1d
Compare
Entirely agree with @aiyengar2's idea from the offline discussion. PR will now reflect that. |
bde0e1d
to
6b449a5
Compare
6b449a5
to
67dec49
Compare
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.
LGTM
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.
lgtm
93798b2
67dec49
to
93798b2
Compare
Add option to disable Windows logging for Windows clusters
Add option to disable Windows logging for Windows clusters
Merge pull request #1133 from nickgerace/dev-v2.5-source-windows Add option to disable Windows logging for Windows clusters
Merge pull request #1133 from nickgerace/dev-v2.5-source-windows Add option to disable Windows logging for Windows clusters
Relevant issues: