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: remove ceph mgr issue reference and correct the csi configuration reference #3512

Merged
merged 2 commits into from
Nov 10, 2022

Conversation

humblec
Copy link
Collaborator

@humblec humblec commented Nov 10, 2022

  • remove a fixed issue reference from the cephfs guide
  • correct reference to the unresolved link

Signed-off-by: Humble Chirammal hchiramm@redhat.com

@humblec humblec added the ci/skip/e2e skip running e2e CI jobs label Nov 10, 2022
@humblec humblec requested a review from a team November 10, 2022 07:25
@Madhu-1 Madhu-1 added the ci/skip/multi-arch-build skip building on multiple architectures label Nov 10, 2022
client mount gid = 0
```

This is due to an [issue](http://tracker.ceph.com/issues/40927) in Ceph v14.2.2
Copy link
Collaborator

Choose a reason for hiding this comment

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

if we are removing this we also need to drop support for 14.2.2?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@Madhu-1 the issue fix is available since 14.2.2-x commit. there are 2 aspects to drop support of 14.2.2. First of all, 14.x is out of support from Ceph POV. so, I am planning to have a followup PR which update the README to mention the support is from 15.x of ceph which is listed as active release. in short drop 14.x itself.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

here it is #3513

@nixpanic nixpanic changed the title remove ceph mgr issue reference and correct the csi configuration reference doc: remove ceph mgr issue reference and correct the csi configuration reference Nov 10, 2022
@mergify mergify bot added the component/docs Issues and PRs related to documentation label Nov 10, 2022
@humblec
Copy link
Collaborator Author

humblec commented Nov 10, 2022

@Mergifyio rebase

this issue has been already fixed long back and available with
ceph backports. We no longer require this mention in the docs.

Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
this commit correct the csi configuration documentation in
rbd deployment doc.

Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
@mergify
Copy link
Contributor

mergify bot commented Nov 10, 2022

rebase

✅ Branch has been successfully rebased

@humblec humblec added the ok-to-test Label to trigger E2E tests label Nov 10, 2022
@github-actions
Copy link

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

@github-actions
Copy link

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

@github-actions
Copy link

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

@github-actions
Copy link

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

@github-actions
Copy link

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

@mergify mergify bot removed the ok-to-test Label to trigger E2E tests label Nov 10, 2022
@github-actions
Copy link

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

@github-actions
Copy link

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

@github-actions
Copy link

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

@github-actions
Copy link

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

@github-actions
Copy link

/test ci/centos/upgrade-tests-cephfs

@github-actions
Copy link

/test ci/centos/upgrade-tests-rbd

@mergify mergify bot added ok-to-test Label to trigger E2E tests and removed ok-to-test Label to trigger E2E tests labels Nov 10, 2022
@mergify mergify bot merged commit 24fb43d into ceph:devel Nov 10, 2022
@github-actions
Copy link

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

@github-actions
Copy link

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

@github-actions
Copy link

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

@github-actions
Copy link

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

@github-actions
Copy link

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

@github-actions
Copy link

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

@github-actions
Copy link

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

@github-actions
Copy link

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

@github-actions
Copy link

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

@github-actions
Copy link

/test ci/centos/upgrade-tests-cephfs

@github-actions
Copy link

/test ci/centos/upgrade-tests-rbd

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.

3 participants