Skip to content

Backport of Set privileged to false unless on OpenShift without CNI into release/1.0.x#2761

Merged
hc-github-team-consul-core merged 2 commits intorelease/1.0.xfrom
backport/privileged-false/obviously-master-husky
Aug 11, 2023
Merged

Backport of Set privileged to false unless on OpenShift without CNI into release/1.0.x#2761
hc-github-team-consul-core merged 2 commits intorelease/1.0.xfrom
backport/privileged-false/obviously-master-husky

Conversation

@hc-github-team-consul-core
Copy link
Copy Markdown
Collaborator

Backport

This PR is auto-generated from #2755 to be assessed for backporting due to the inclusion of the label backport/1.0.x.

The below text is copied from the body of the original PR.


Changes proposed in this PR:

  • Sets privileged=false for regular transparent proxy connect injects
  • Also false when CNI is used
  • True for OpenShift with transparent proxy

How I've tested this PR:

  • unit tests and accpetance, manual tests on OpenShift

How I expect reviewers to test this PR:

👀

Checklist:


Overview of commits

@hc-github-team-consul-core hc-github-team-consul-core merged commit a5af8c5 into release/1.0.x Aug 11, 2023
@hc-github-team-consul-core hc-github-team-consul-core deleted the backport/privileged-false/obviously-master-husky branch August 11, 2023 18:51
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