ceph / ceph-csi

CSI driver for Ceph
Apache License 2.0
1.29k stars 549 forks source link

doc: fixed broken links #4958

Closed seadog007 closed 1 week ago

seadog007 commented 2 weeks ago

Describe what this PR does

Fixed broken links for documents in 4 files.

Is there anything that requires special attention

Do you have any questions? No

Is the change backward compatible? No

Are there concerns around backward compatibility? No

Provide any external context for the change, if any. N/A

Related issues

Broken when #4907 merged See also https://github.com/ceph/ceph-csi/commit/a6f1af49c649f90047f466423785ed006190e247

Future concerns

N/A

Checklist:


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/`: retest the `` after unrelated failure (please report the failure too!)
iPraveenParihar commented 1 week ago

Thanks for the PR. Can you please fix the broken links at places as mentioned below?

  1. https://github.com/ceph/ceph-csi/blob/925ea1970c4cde5aa76dfb875a778edcc211d50a/charts/ceph-csi-cephfs/README.md?plain=1#L177

  2. https://github.com/ceph/ceph-csi/blob/925ea1970c4cde5aa76dfb875a778edcc211d50a/charts/ceph-csi-rbd/README.md?plain=1#L190

  3. https://github.com/ceph/ceph-csi/blob/925ea1970c4cde5aa76dfb875a778edcc211d50a/docs/resource-cleanup.md?plain=1#L30-L31

  4. https://github.com/ceph/ceph-csi/blob/925ea1970c4cde5aa76dfb875a778edcc211d50a/docs/resource-cleanup.md?plain=1#L59

there was doc folder restructure PR https://github.com/ceph/ceph-csi/pull/4907, @yati1998 can you please check for any other broken links that might need updating beyond those listed here?

iPraveenParihar commented 1 week ago

@seadog007 Can you please squash your commits into one?

seadog007 commented 1 week ago

The commit has been squash into one

Madhu-1 commented 1 week ago

@mergifyio queue

mergify[bot] commented 1 week ago

queue

🛑 The pull request has been removed from the queue default

The pull request can't be updated. You can take a look at `Queue: Embarked in merge queue` check runs for more details. In case of a failure due to a flaky test, you should first retrigger the CI. Then, re-embark the pull request into the merge queue by posting the comment `@mergifyio refresh` on the pull request.
Madhu-1 commented 1 week ago

@mergifyio rebase

mergify[bot] commented 1 week ago

rebase

✅ Branch has been successfully rebased

Madhu-1 commented 1 week ago

@Mergifyio queue

mergify[bot] commented 1 week ago

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at *dc4ca2015e073f09514588120d5165eab7b1020f*
ceph-csi-bot commented 1 week ago

/test ci/centos/upgrade-tests-cephfs

ceph-csi-bot commented 1 week ago

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

ceph-csi-bot commented 1 week ago

/test ci/centos/upgrade-tests-rbd

ceph-csi-bot commented 1 week ago

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

ceph-csi-bot commented 1 week ago

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

ceph-csi-bot commented 1 week ago

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

ceph-csi-bot commented 1 week ago

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

ceph-csi-bot commented 1 week ago

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

ceph-csi-bot commented 1 week ago

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

ceph-csi-bot commented 1 week ago

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

ceph-csi-bot commented 1 week ago

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