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: added note about monitor ips update during failover #4501

Open
wants to merge 1 commit into
base: devel
Choose a base branch
from

Conversation

kmadac
Copy link

@kmadac kmadac commented Mar 15, 2024

Docs update: IP addresses in csi-config-map needs to be changed after failover to secondary site. Without this change, applications can't use mirrored rbd images.

Is there anything that requires special attention

Only documentation update.

Related issues

Fixes: #4493

Future concerns

No future concerns

Checklist:

  • Commit Message Formatting: Commit titles and messages follow
    guidelines in the developer
    guide
    .
  • Reviewed the developer guide on Submitting a Pull
    Request
  • Pending release
    notes

    updated with breaking and/or notable changes for the next major release.
  • Documentation has been updated, if necessary.
  • Unit tests have been added, if necessary.
  • Integration tests have been added, if necessary.

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!)

IP addresses in csi-config-map needs to be changed after failover
to secondary site. Without this change, applications can't use
mirrored rbd images.

Signed-off-by: Kamil Madac <kamil.madac@gmail.com>
@mergify mergify bot added 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 labels Mar 15, 2024
@nixpanic nixpanic requested review from Madhu-1 and Rakshith-R March 15, 2024 13:52
@Rakshith-R
Copy link
Contributor

hey @kmadac

The document talks about having a new cluster-mapping.json to provide cephcsi with the required information to map IDs from one cluster to another.

The config.json section does not need to be modified in either cluster.
The configmap's config.json in each k8s cluster can have different config.json entries (mon ips, cluster IDs etc).

Just adding cluster-mapping.json info to each of these k8s cluster's cephcsi configmap is sufficient. The existing cephcsi configmap's config.json does not need to be modified.

Copy link

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in two weeks if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale label Apr 17, 2024
@kmadac
Copy link
Author

kmadac commented Apr 18, 2024

Sorry @Rakshith-R , I'm not sure I understand your comment. In the issue #4493 we found out that in order to successfully use volumes on secondary site during the failover period, it is necessary to set monitor ip addresses of primary cluster in config.json to secondary monitor ip addresses on secondary k8s cluster. Otherwise we see FailedMount errors in pods which tries to use those volumes.

Check please this comment in the issue: #4493 (comment)

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 stale
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cluster-mapping during failover not applied
2 participants