Skip to content
This repository was archived by the owner on May 16, 2023. It is now read-only.

[logstash] disable privileged container in psp#1000

Merged
jmlrt merged 3 commits intoelastic:masterfrom
jmlrt:logstash-psp-privileged-false
Jan 5, 2021
Merged

[logstash] disable privileged container in psp#1000
jmlrt merged 3 commits intoelastic:masterfrom
jmlrt:logstash-psp-privileged-false

Conversation

@jmlrt
Copy link
Copy Markdown
Member

@jmlrt jmlrt commented Dec 23, 2020

This commit disable privilged container when enabling PodSecurityPolicy
as Logstash container doesn't use any host device.

This commit disable privilged container when enabling PodSecurityPolicy
as Logstash container doesn't use any host device.
@jmlrt jmlrt added enhancement New feature or request logstash labels Dec 23, 2020
@jmlrt jmlrt requested a review from a team December 23, 2020 17:48
@Conky5
Copy link
Copy Markdown
Contributor

Conky5 commented Jan 4, 2021

@jmlrt
Copy link
Copy Markdown
Member Author

jmlrt commented Jan 5, 2021

This looks like it might be a legitimate test failure devops-ci.elastic.co/job/elastic+helm-charts+pull-request+template-testing/1123/CHART=logstash,label=docker&&virtual/console

Fixed @Conky5 please re-review

Copy link
Copy Markdown
Contributor

@Conky5 Conky5 left a comment

Choose a reason for hiding this comment

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

LGTM

@jmlrt jmlrt merged commit dda9b8e into elastic:master Jan 5, 2021
@jmlrt jmlrt deleted the logstash-psp-privileged-false branch January 5, 2021 15:47
@jmlrt jmlrt mentioned this pull request Feb 8, 2021
This was referenced Feb 12, 2021
@jmlrt jmlrt mentioned this pull request Feb 18, 2021
This was referenced Mar 15, 2021
@jmlrt jmlrt mentioned this pull request May 25, 2021
@jmlrt jmlrt mentioned this pull request Mar 8, 2022
@jmlrt jmlrt mentioned this pull request Apr 21, 2022
This was referenced Sep 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants