Skip to content

Update CSI provider installation on OpenShift#13763

Merged
kalafut merged 1 commit intomainfrom
joatmon08-openshift-csi-docs
Jan 26, 2022
Merged

Update CSI provider installation on OpenShift#13763
kalafut merged 1 commit intomainfrom
joatmon08-openshift-csi-docs

Conversation

@joatmon08
Copy link
Copy Markdown
Contributor

@joatmon08 joatmon08 commented Jan 24, 2022

Include recommendation to use Vault agent injector on OpenShift instead of CSI due to production security constraints. Additional instructions included for testing and development clusters.

Relates to hashicorp/vault-csi-provider#113.

@joatmon08 joatmon08 marked this pull request as ready for review January 24, 2022 18:08
Comment thread website/content/docs/platform/k8s/helm/openshift.mdx Outdated
Comment thread website/content/docs/platform/k8s/csi/installation.mdx
Comment thread website/content/docs/platform/k8s/helm/openshift.mdx
Comment thread website/content/docs/platform/k8s/helm/openshift.mdx Outdated
Copy link
Copy Markdown
Contributor

@taoism4504 taoism4504 left a comment

Choose a reason for hiding this comment

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

Couple of minor nits; text overall looks good.

Copy link
Copy Markdown
Contributor

@tomhjp tomhjp left a comment

Choose a reason for hiding this comment

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

Thanks for adding this!

Comment thread website/content/docs/platform/k8s/helm/openshift.mdx Outdated
Comment thread website/content/docs/platform/k8s/csi/installation.mdx Outdated
@hashicorp-cla
Copy link
Copy Markdown

hashicorp-cla commented Jan 25, 2022

CLA assistant check
All committers have signed the CLA.

Include recommendation to use Vault agent injector on OpenShift
instead of CSI due to production security constraints.
Additional instructions included for testing and development
clusters.
@kalafut
Copy link
Copy Markdown
Contributor

kalafut commented Jan 26, 2022

@joatmon08 Do you want me to merge this?

@joatmon08
Copy link
Copy Markdown
Contributor Author

@kalafut Yes please! I've finished with the updates.

@kalafut kalafut merged commit 2d14aa7 into main Jan 26, 2022
@kalafut kalafut deleted the joatmon08-openshift-csi-docs branch January 26, 2022 15:44
apurvagandhi pushed a commit to fidelity-contributions/hashicorp-vault that referenced this pull request Apr 9, 2026
* ci: add more backup self-hosted runner types (hashicorp#13763)

Signed-off-by: Ryan Cragun <me@ryan.ec>
Co-authored-by: Ryan Cragun <me@ryan.ec>
MochaCaffe pushed a commit to kosmos-education/vault that referenced this pull request Apr 15, 2026
…x.x+ent into ce/release/2.x.x

* ci: add more backup self-hosted runner types (hashicorp#13763) (hashicorp#13784)

Signed-off-by: Ryan Cragun <me@ryan.ec>
Co-authored-by: Ryan Cragun <me@ryan.ec>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants