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

Add KOTS security context #2492

Merged
merged 5 commits into from
Jul 12, 2024
Merged

Add KOTS security context #2492

merged 5 commits into from
Jul 12, 2024

Conversation

paigecalvert
Copy link
Contributor

@paigecalvert paigecalvert commented Jul 11, 2024

@paigecalvert paigecalvert requested a review from a team as a code owner July 11, 2024 16:41
@replicated-ci replicated-ci added type::docs Improvements or additions to documentation type::feature labels Jul 11, 2024
Copy link

netlify bot commented Jul 11, 2024

Deploy Preview for replicated-docs-upgrade ready!

Name Link
🔨 Latest commit c17b38a
🔍 Latest deploy log https://app.netlify.com/sites/replicated-docs-upgrade/deploys/669025a16b17ea00083aa484
😎 Deploy Preview https://deploy-preview-2492--replicated-docs-upgrade.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Jul 11, 2024

Deploy Preview for replicated-docs ready!

Name Link
🔨 Latest commit c17b38a
🔍 Latest deploy log https://app.netlify.com/sites/replicated-docs/deploys/669025a14992660008e69499
😎 Deploy Preview https://deploy-preview-2492--replicated-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@@ -14,7 +16,11 @@
<li>Ensures the container's root filesystem is mounted as read-only</li>
<li>Removes all Linux capabilities from the container</li>
</ul>
<p>The following shows the <code>securityContext</code> for KOTS Pods and containers when <code>--strict-security-context</code> is set:</p>
<StrictSecContextYaml/>
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

^ added the YAML to the strict-security-context flag description too

paigecalvert and others added 2 commits July 11, 2024 12:34
Co-authored-by: Alex Parker <7272359+ajp-io@users.noreply.github.com>
Co-authored-by: Alex Parker <7272359+ajp-io@users.noreply.github.com>
@ajp-io
Copy link
Member

ajp-io commented Jul 11, 2024

You could have Diamon or Salah take a peek too

@paigecalvert
Copy link
Contributor Author

You could have Diamon or Salah take a peek too

Will do; I'll get the cidr flag docs ready to so they can look at both at the same time

@ajp-io ajp-io merged commit a2b00b9 into main Jul 12, 2024
5 checks passed
@ajp-io ajp-io deleted the 108384 branch July 12, 2024 00:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type::docs Improvements or additions to documentation type::feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants