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

doc: adding empty storageClassName in static pvc #4110

Merged
merged 1 commit into from
Sep 7, 2023

Conversation

Rakshith-R
Copy link
Contributor

we need to add storageClassName: "" in cephfs static pvc similar what was done for rbd static pvc in #4010.

Kubernetes links that explain why the change is required
https://kubernetes.io/docs/concepts/storage/persistent-volumes/#class-1


Show available bot commands

These commands are normally not required, but in case of issues, leave any of
the following bot commands in an otherwise empty comment in this PR:

  • /retest ci/centos/<job-name>: retest the <job-name> after unrelated
    failure (please report the failure too!)

we need to add `storageClassName: ""` in cephfs static pvc
similar what was done for rbd static pvc in ceph#4010.

Signed-off-by: Rakshith R <[email protected]>
@Rakshith-R Rakshith-R requested review from a team September 7, 2023 05:16
@Rakshith-R Rakshith-R added ci/skip/e2e skip running e2e CI jobs ci/skip/multi-arch-build skip building on multiple architectures labels Sep 7, 2023
@mergify mergify bot added the component/docs Issues and PRs related to documentation label Sep 7, 2023
@Madhu-1 Madhu-1 requested a review from a team September 7, 2023 06:37
@riya-singhal31
Copy link
Contributor

@Mergifyio queue

@mergify
Copy link
Contributor

mergify bot commented Sep 7, 2023

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at 4eb2dbd

@mergify mergify bot added the ok-to-test Label to trigger E2E tests label Sep 7, 2023
@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/k8s-e2e-external-storage/1.27

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-cephfs

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/k8s-e2e-external-storage/1.28

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e-helm/k8s-1.27

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-rbd

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e-helm/k8s-1.28

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/k8s-e2e-external-storage/1.26

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e/k8s-1.27

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e/k8s-1.28

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e-helm/k8s-1.26

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/mini-e2e/k8s-1.26

@ceph-csi-bot ceph-csi-bot removed the ok-to-test Label to trigger E2E tests label Sep 7, 2023
@mergify mergify bot merged commit 4eb2dbd into ceph:devel Sep 7, 2023
33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci/skip/e2e skip running e2e CI jobs ci/skip/multi-arch-build skip building on multiple architectures component/docs Issues and PRs related to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants