-
Notifications
You must be signed in to change notification settings - Fork 554
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: fixed broken links #4958
doc: fixed broken links #4958
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great, thanks!
Thanks for the PR.
there was doc folder restructure PR #4907, @yati1998 can you please check for any other broken links that might need updating beyond those listed here? |
Requested by iPraveenParihar in ceph#4958 Signed-off-by: 尤理衡 (Li-Heng Yu) <007seadog@gmail.com>
Pull request has been modified.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@seadog007 Can you please squash your commits into one? |
The deploy link in the README is broken. Fixed more broken links requested by iPraveenParihar in ceph#4958 Signed-off-by: 尤理衡 (Li-Heng Yu) <007seadog@gmail.com>
The deploy link in the README is broken. Fixed more broken links requested by iPraveenParihar in ceph#4958 Signed-off-by: 尤理衡 (Li-Heng Yu) <007seadog@gmail.com>
Pull request has been modified.
The deploy link in the README is broken. Fixed more broken links requested by iPraveenParihar in ceph#4958 Signed-off-by: 尤理衡 (Li-Heng Yu) <007seadog@gmail.com>
The commit has been squash into one |
@Mergifyio queue |
🛑 The pull request has been removed from the queue
|
@Mergifyio rebase |
The deploy link in the README is broken. Fixed more broken links requested by iPraveenParihar in ceph#4958 Signed-off-by: 尤理衡 (Li-Heng Yu) <007seadog@gmail.com>
✅ Branch has been successfully rebased |
@Mergifyio queue |
✅ The pull request has been merged automaticallyThe pull request has been merged automatically at dc4ca20 |
/test ci/centos/upgrade-tests-cephfs |
/test ci/centos/k8s-e2e-external-storage/1.31 |
/test ci/centos/upgrade-tests-rbd |
/test ci/centos/mini-e2e-helm/k8s-1.31 |
/test ci/centos/mini-e2e/k8s-1.31 |
/test ci/centos/k8s-e2e-external-storage/1.30 |
/test ci/centos/k8s-e2e-external-storage/1.29 |
/test ci/centos/mini-e2e-helm/k8s-1.30 |
/test ci/centos/mini-e2e-helm/k8s-1.29 |
/test ci/centos/mini-e2e/k8s-1.30 |
/test ci/centos/mini-e2e/k8s-1.29 |
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 a6f1af4
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/<job-name>
: retest the<job-name>
after unrelatedfailure (please report the failure too!)