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

chore: update cilium-operator security context #2755

Merged
merged 2 commits into from
May 30, 2024

Conversation

jshr-w
Copy link
Contributor

@jshr-w jshr-w commented May 29, 2024

Update Cilium 1.13 and 1.14 templates to include security context for reduced permissions.

Reason for Change:

Best practice.

Issue Fixed:

Requirements:

Notes:

@jshr-w jshr-w requested a review from camrynl May 29, 2024 17:59
@jshr-w jshr-w self-assigned this May 29, 2024
@jshr-w jshr-w requested a review from a team as a code owner May 29, 2024 17:59
@camrynl
Copy link
Contributor

camrynl commented May 29, 2024

since this change is for all cilium versions, can you add this to the 1.14 charts as well?

@jshr-w
Copy link
Contributor Author

jshr-w commented May 29, 2024

/azp run Azure Container Networking PR

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@jshr-w jshr-w enabled auto-merge May 29, 2024 22:23
@jshr-w jshr-w added this pull request to the merge queue May 29, 2024
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks May 30, 2024
@jpayne3506 jpayne3506 added this pull request to the merge queue May 30, 2024
Merged via the queue into master with commit 6dabf70 May 30, 2024
12 checks passed
@jpayne3506 jpayne3506 deleted the shjayaraman/cilium11313update branch May 30, 2024 20:09
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.

None yet

3 participants