ceph: add workaround for Parallel RBD PVC Creation hangs on new pools

[Rakshith-R]

This commit adds workaround for Parallel RBD PVC Creation hangs on
new pools in ceph-csi-troubleshooting.md.
Refer: #8696

Signed-off-by: Rakshith R rar@redhat.com

Description of your changes:

Which issue is resolved by this Pull Request:
Resolves #

Checklist:

• Commit Message Formatting: Commit titles and messages follow guidelines in the developer guide.
• Skip Tests for Docs: Add the flag for skipping the build if this is only a documentation change. See here for the flag.
• Skip Unrelated Tests: Add a flag to run tests for a specific storage provider. See test options.
• Reviewed the developer guide on Submitting a Pull Request
• Documentation has been updated, if necessary.
• Unit tests have been added, if necessary.
• Integration tests have been added, if necessary.
• Pending release notes updated with breaking and/or notable changes, if necessary.
• Upgrade from previous release is tested and upgrade user guide is updated, if necessary.
• Code generation (make codegen) has been run to update object specifications, if necessary.

[Madhu-1]

@idryomov PTAL

[Rakshith-R]

updated docs with WA which does not leave any stale resources as discussed here ceph/ceph-csi#2521 (comment). PTAL
thanks!

[travisn]

Do we currently have a CSI release with the fix? I'm still not clear in what version of CSI this is or will be fixed.

[Rakshith-R]

Do we currently have a CSI release with the fix?

@travisn
No, we need ceph pacific release with fix ceph/ceph#43113 to build a new cephcsi built on ceph pacific to resolve this issue.

I'm still not clear in what version of CSI this is or will be fixed.

This will be an issue in CSI v3.4.0 (and also v3.4.1 which will come out soon).

From ceph/ceph#43113 (comment) ,
We should be able to pick up the fix with next ceph pacific release in cephcsi v3.4.2 and make changes in the WA doc at that time.

[Madhu-1]

@travisn @Rakshith-R instead of documenting how about new cephcsi 3.4.1 with octopus as the base image ceph/ceph-csi#2521 (comment). If we do that we don't need to document any workaround.

[Rakshith-R]

@travisn @Rakshith-R instead of documenting how about new cephcsi 3.4.1 with octopus as the base image ceph/ceph-csi#2521 (comment). If we do that we don't need to document any workaround.

This is not an option since with ceph octopus deep_copy() does not work as expected, refer : ceph/ceph-csi#2521 (comment)

Or rook can call rbd pool init <pool_name> right after creation?
(IMO since it will be used for rbd images so no harm to initialize it too ?)

@idryomov @travisn Do you think its a more preferrable solution ?

cc @Madhu-1

[Madhu-1]

Sounds better instead of documentation 👍

[travisn]

So we can avoid this issue if Rook always calls rbd pool init <pool_name> immediately after pool creation? And we would just do this upon creation of CephBlockPool CRs?

[Rakshith-R]

So we can avoid this issue if Rook always calls rbd pool init <pool_name> immediately after pool creation? And we would just do this upon creation of CephBlockPool CRs?

Yes, it just needs to be called once. If it is done right after pool create/ setting property, it will avoid this issue.

#8696 (comment)

[travisn]

So we can avoid this issue if Rook always calls rbd pool init <pool_name> immediately after pool creation? And we would just do this upon creation of CephBlockPool CRs?

Yes, it just needs to be called once. If it is done right after pool create/ setting property, it will avoid this issue.

#8696 (comment)

@Rakshith-R Are you planning on this change in Rook, or somebody else should pick it up?

[mergify]

This pull request has merge conflicts that must be resolved before it can be merged. @Rakshith-R please rebase it. https://rook.io/docs/rook/latest/development-flow.html#updating-your-fork

[Rakshith-R]

Closing this pr, since #8923 is merged and should fix the issue.