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

docs: add guide to install new CSI drivers #214

Merged
merged 1 commit into from
Aug 12, 2024
Merged

Conversation

sprt
Copy link
Collaborator

@sprt sprt commented Aug 6, 2024

This adds a public guide on how to install and test the new storage CSI drivers for AKS confidential pods.

Merge Checklist
  • Followed patch format from upstream recommendation: https://github.com/kata-containers/community/blob/main/CONTRIBUTING.md#patch-format
    • Included a single commit in a given PR - at least unless there are related commits and each makes sense as a change on its own.
  • Aware about the PR to be merged using "create a merge commit" rather than "squash and merge" (or similar)
  • genPolicy only: Ensured the tool still builds on Windows
  • The upstream/missing label (or upstream/not-needed) has been set on the PR.
Summary

This adds a public guide on how to install and test the new storage CSI drivers for AKS confidential pods.

Test Methodology

Ran through this myself just now and was able to repro all instructions. However would appreciate it if someone else could test the az confcom katapolicygen calls, as I'm building genpolicy from scratch (because I'm using Ubuntu and running into that OpenSSL linking error).

@sprt sprt added the upstream/not-needed PRs that will not be upstreamed (e.g. internal) label Aug 6, 2024
@sprt sprt force-pushed the sprt/csi-instructions branch from 12c930f to bf90c9e Compare August 6, 2024 20:19
@sprt sprt marked this pull request as ready for review August 6, 2024 20:23
@sprt sprt requested review from a team as code owners August 6, 2024 20:23
@sprt sprt requested a review from arc9693 August 6, 2024 20:45
Copy link

@arc9693 arc9693 left a comment

Choose a reason for hiding this comment

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

LGTM

This adds a public guide on how to install and test the new storage CSI
drivers for AKS confidential pods.

Signed-off-by: Aurélien Bombo <[email protected]>
@sprt sprt force-pushed the sprt/csi-instructions branch from bf90c9e to 78e4962 Compare August 12, 2024 20:55
@sprt sprt merged commit 4c89a18 into msft-main Aug 12, 2024
42 of 53 checks passed
sprt added a commit that referenced this pull request Feb 10, 2025
docs: add guide to install new CSI drivers
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
upstream/not-needed PRs that will not be upstreamed (e.g. internal)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants