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
chore: #21804 update che server config to use .spec.properties.server.properties for cheLogLevel and customCheProperties.CHE_LOGS_APPENDERS_IMPL, as CHE_WORKSPACE_DEFAULT__MEMORY__LIMIT__MB is obsolete #2494
chore: #21804 update che server config to use .spec.properties.server.properties for cheLogLevel and customCheProperties.CHE_LOGS_APPENDERS_IMPL, as CHE_WORKSPACE_DEFAULT__MEMORY__LIMIT__MB is obsolete #2494
Conversation
🎊 Navigate the preview: https://6373db9f143ca0008d591c9b--eclipse-che-docs-pr.netlify.app 🎊 |
hmm. so that didn't work... I was able to change
to
and get less console logging in the devspaces (che-server) pod
to my CheCluster yaml ... kept being removed by the editor. So... not sure how to apply this env var. |
extraProperties: | ||
pass:[CHE_WORKSPACE_DEFAULT__MEMORY__LIMIT__MB]: "2048" | ||
logLevel: INFO | ||
customCheProperties: |
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.
I dont know why we need these changes since there is no customCheProperties anymore.
It is all about extraProperties
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.
spec:
components:
cheServer:
extraProperties:
CHE_LOGS_APPENDERS_IMPL: json
This is the correct setup, basically, the docs should say that this way one can add any env var to the che server pod
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.
ok, changed. thanks! I assume that's the correct yaml ?
….properties for cheLogLevel and customCheProperties.CHE_LOGS_APPENDERS_IMPL, as CHE_WORKSPACE_DEFAULT__MEMORY__LIMIT__MB is obsolete Signed-off-by: Nick Boldt <nboldt@redhat.com> update sample - still need to verify this works Signed-off-by: Nick Boldt <nboldt@redhat.com> Update con_understanding-che-server-advanced-configuration.adoc
cc52dce
to
aa9e053
Compare
spec: | ||
components: | ||
cheServer: | ||
extraProperties: | ||
pass:[CHE_WORKSPACE_DEFAULT__MEMORY__LIMIT__MB]: "2048" | ||
CHE_LOGS_APPENDERS_IMPL: json | ||
---- |
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.
sounds good
….properties for cheLogLevel and customCheProperties.CHE_LOGS_APPENDERS_IMPL, as CHE_WORKSPACE_DEFAULT__MEMORY__LIMIT__MB is obsolete (#2494)
What does this PR do?
chore: #21804 update che server config to use .spec.properties.server.properties for cheLogLevel and customCheProperties.CHE_LOGS_APPENDERS_IMPL, as CHE_WORKSPACE_DEFAULT__MEMORY__LIMIT__MB is obsolete
Signed-off-by: Nick Boldt nboldt@redhat.com
fixes eclipse-che/che#21804
Screenshot/screencast of this PR
N/A
What issues does this PR fix or reference?
eclipse-che/che#21804
How to test this PR?
N/A
PR Checklist
As the author of this Pull Request I made sure that:
Reviewers
Reviewers, please comment how you tested the PR when approving it.