-
Notifications
You must be signed in to change notification settings - Fork 15k
Add documentation for the EnvFiles feature gate #51415
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
Conversation
👷 Deploy Preview for kubernetes-io-vnext-staging processing.
|
This is a work in progress /retitle [WIP] Add documentation for the EnvFiles feature gate |
7ee5e19
to
b47814f
Compare
content/en/docs/tasks/inject-data-application/define-environment-variable-via-file.md
Outdated
Show resolved
Hide resolved
content/en/examples/pods/inject/dapi-envars-file-container.yaml
Outdated
Show resolved
Hide resolved
✅ Pull request preview available for checkingBuilt without sensitive environment variables
To edit notification comments on pull requests, go to your Netlify project configuration. |
content/en/examples/pods/inject/dapi-envars-file-container.yaml
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.
/lgtm
LGTM label has been added. Git tree hash: 53fe740178c56d5094f482ee52c85c40790aee88
|
content/en/docs/reference/command-line-tools-reference/feature-gates/EnvFiles.md
Outdated
Show resolved
Hide resolved
content/en/docs/tasks/inject-data-application/define-environment-variable-via-file.md
Outdated
Show resolved
Hide resolved
content/en/docs/tasks/inject-data-application/define-environment-variable-via-file.md
Outdated
Show resolved
Hide resolved
content/en/docs/tasks/inject-data-application/define-environment-variable-via-file.md
Outdated
Show resolved
Hide resolved
content/en/docs/tasks/inject-data-application/define-environment-variable-via-file.md
Outdated
Show resolved
Hide resolved
content/en/docs/tasks/inject-data-application/define-environment-variable-via-file.md
Outdated
Show resolved
Hide resolved
While these environment variables can store sensitive information, | ||
please note that `emptyDir` volumes don't provide the same protection mechanisms as | ||
dedicated Secret objects. Therefore, exposing confidential environment variables | ||
to containers through this feature is not considered a security best practice. |
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 am not sure about this; the Secret API also has problems.
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.
Secrets generally offer higher security.
Ref: https://kubernetes.io/docs/concepts/configuration/secret/#information-security-for-secrets
content/en/docs/tasks/inject-data-application/define-environment-variable-via-file.md
Outdated
Show resolved
Hide resolved
content/en/docs/tasks/inject-data-application/define-environment-variable-via-file.md
Outdated
Show resolved
Hide resolved
content/en/examples/pods/inject/dapi-envars-file-container.yaml
Outdated
Show resolved
Hide resolved
Hello @HirazawaUi 👋! I'm reaching out from the Docs team. Just checking in as we approach Docs Freeze on Wednesday August 6, 2025 18:00 PDT. This documentation appears to still be under review. To meet the Docs Freeze, this PR must have a technical review as well as lgtm and approve labels applied, without any unaddressed comments or concerns from SIG Docs. Thank you! |
a301bc4
to
f83015e
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
LGTM label has been added. Git tree hash: c1017caec234d5de61203016c2f2bb635e0a2272
|
/assign @tengqm |
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: reylejano, SergeyKanzhelev The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Description
This PR adds documentation for KEP-3721.
KEP link: kubernetes/enhancements#3721
Issue
Closes: #