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

cephFS: fix fetchIP to support more formats #4321

Merged
merged 1 commit into from
Dec 15, 2023
Merged

cephFS: fix fetchIP to support more formats #4321

merged 1 commit into from
Dec 15, 2023

Conversation

riya-singhal31
Copy link
Contributor

@riya-singhal31 riya-singhal31 commented Dec 15, 2023

the output of ceph tell mds.0 client ls returns inst to be prefixed by v1 or similar.
So adding the support for same in FetchIP, so that it can fetch correct IP from the inst of that format.

example - clientInfo: "client.24152 v1:100.64.0.7:0/3658550259"
expectedIP: "100.64.0.7"

@mergify mergify bot added the bug Something isn't working label Dec 15, 2023
@riya-singhal31 riya-singhal31 marked this pull request as ready for review December 15, 2023 14:58
@riya-singhal31
Copy link
Contributor Author

Thanks for the early review @nixpanic , @yati1998

@riya-singhal31
Copy link
Contributor Author

@Mergifyio queue

Copy link
Contributor

mergify bot commented Dec 15, 2023

queue

✅ The pull request has been merged automatically

The pull request has been merged automatically at bee77f7

Signed-off-by: Riya Singhal <rsinghal@redhat.com>
@mergify mergify bot added the ok-to-test Label to trigger E2E tests label Dec 15, 2023
@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-cephfs

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

/test ci/centos/upgrade-tests-rbd

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot
Copy link
Collaborator

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

@ceph-csi-bot ceph-csi-bot removed the ok-to-test Label to trigger E2E tests label Dec 15, 2023
@mergify mergify bot merged commit bee77f7 into ceph:devel Dec 15, 2023
34 checks passed
@riya-singhal31 riya-singhal31 added the backport-to-release-v3.10 Label to backport from devel to release-v3.10 branch label Dec 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-to-release-v3.10 Label to backport from devel to release-v3.10 branch bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants