Skip to content

test: pass peer_uuid when invoking fs snapshot mirror peer_add#40146

Merged
batrick merged 2 commits intoceph:masterfrom
vshankar:wip-49822
Mar 23, 2021
Merged

test: pass peer_uuid when invoking fs snapshot mirror peer_add#40146
batrick merged 2 commits intoceph:masterfrom
vshankar:wip-49822

Conversation

@vshankar
Copy link
Contributor

Also, add a check in peer_add monitor interface baed on remote cluster spec and filesystem name to check an existing file system peer.

Checklist

  • References tracker ticket
  • Updates documentation if necessary
  • Includes tests for new functionality or reproducer for bug

Show available Jenkins commands
  • jenkins retest this please
  • jenkins test classic perf
  • jenkins test crimson perf
  • jenkins test signed
  • jenkins test make check
  • jenkins test make check arm64
  • jenkins test submodules
  • jenkins test dashboard
  • jenkins test api
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox

…ystem name

Signed-off-by: Venky Shankar <vshankar@redhat.com>
@vshankar vshankar added the cephfs Ceph File System label Mar 16, 2021
@vshankar vshankar requested a review from a team March 16, 2021 05:34
@vshankar vshankar changed the title test: pass perr_uuid when invoking fs snapshot mirror peer_add test: pass peer_uuid when invoking fs snapshot mirror peer_add Mar 16, 2021
Fixes: http://tracker.ceph.com/issues/49822
Signed-off-by: Venky Shankar <vshankar@redhat.com>
@vshankar
Copy link
Contributor Author

jenkins test make check

batrick added a commit to batrick/ceph that referenced this pull request Mar 18, 2021
* refs/pull/40146/head:
	test: pass peer uuid when adding cephfs mirror peers
	mon: check cephfs mirror peer based on remote cluster spec and file system name
@batrick
Copy link
Member

batrick commented Mar 23, 2021

https://pulpito.ceph.com/pdonnell-2021-03-18_13:46:31-fs-wip-pdonnell-testing-20210318.024145-distro-basic-smithi/

@batrick batrick merged commit 4924185 into ceph:master Mar 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants