Skip to content
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

feat(chart): Improved default security context #7279

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

stevehipwell
Copy link
Contributor

Fixes #N/A

Description
This PR improves the default security context posture and adds support for setting container values which should be user defined.

How was this change tested?
The Helm chart was templated with the new values.

Does this change impact docs?

  • Yes, PR includes docs updates
  • Yes, issue opened: #
  • No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@stevehipwell stevehipwell requested a review from a team as a code owner October 25, 2024 15:53
@stevehipwell stevehipwell requested a review from tzneal October 25, 2024 15:53
Copy link

netlify bot commented Oct 25, 2024

Deploy Preview for karpenter-docs-prod canceled.

Name Link
🔨 Latest commit be44880
🔍 Latest deploy log https://app.netlify.com/sites/karpenter-docs-prod/deploys/675abe5d00afcc0008ef515e

@stevehipwell
Copy link
Contributor Author

CC @jonathan-innis

@stevehipwell stevehipwell force-pushed the chart-security-context branch from e211a98 to 0e90ebf Compare November 4, 2024 15:42
@stevehipwell stevehipwell force-pushed the chart-security-context branch from 0e90ebf to dacfc30 Compare November 18, 2024 09:47
@stevehipwell
Copy link
Contributor Author

@tzneal could you take a look at this please?

Copy link
Contributor

github-actions bot commented Dec 2, 2024

This PR has been inactive for 14 days. StaleBot will close this stale PR after 14 more days of inactivity.

@stevehipwell
Copy link
Contributor Author

@jonathan-innis @tzneal could someone please take a look at this?

@stevehipwell stevehipwell force-pushed the chart-security-context branch from 5cc1c72 to be44880 Compare December 12, 2024 10:43
@stevehipwell
Copy link
Contributor Author

@jonathan-innis @tzneal could someone please take a look at this?

@saurav-agarwalla
Copy link
Contributor

I'll take a look at this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants