Skip to content

nautilus: test/cls_rbd: removed mirror peer pool test cases#30948

Merged
dillaman merged 1 commit intoceph:nautilusfrom
dillaman:wip-42333-nautilus
Nov 7, 2019
Merged

nautilus: test/cls_rbd: removed mirror peer pool test cases#30948
dillaman merged 1 commit intoceph:nautilusfrom
dillaman:wip-42333-nautilus

Conversation

@dillaman
Copy link

The mirror peer pool id has never been used and has been dropped
from the Octopus release. This will fix the breakage in the test
cases where the pool id was tested.

Fixes: https://tracker.ceph.com/issues/42333
Signed-off-by: Jason Dillaman dillaman@redhat.com

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 crimson perf
  • jenkins test signed
  • jenkins test make check
  • jenkins test make check arm64
  • jenkins test submodules
  • jenkins test dashboard
  • jenkins test dashboard backend
  • jenkins test docs
  • jenkins render docs

The mirror peer pool id has never been used and has been dropped
from the Octopus release. This will fix the breakage in the test
cases where the pool id was tested.

Fixes: https://tracker.ceph.com/issues/42333
Signed-off-by: Jason Dillaman <dillaman@redhat.com>
@dillaman dillaman added this to the nautilus milestone Oct 16, 2019
Copy link
Contributor

@trociny trociny left a comment

Choose a reason for hiding this comment

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

LGTM

@yuriw
Copy link
Contributor

yuriw commented Oct 30, 2019

@liewegas
Copy link
Member

liewegas commented Nov 4, 2019

@yuriw is this one ready to merge? until it merges we're seeing persistent failures in the master rados run.

@djgalloway
Copy link
Contributor

jenkins test make check

@djgalloway
Copy link
Contributor

Yuri is seeing a segfault in his wip branch he added this commit to. Jason thought it could be a memory issue on the builder but I'm skeptical so I'm re-running make check.

@dillaman
Copy link
Author

dillaman commented Nov 7, 2019

@dillaman
Copy link
Author

dillaman commented Nov 7, 2019

@dillaman dillaman merged commit 2e04e7b into ceph:nautilus Nov 7, 2019
@dillaman dillaman deleted the wip-42333-nautilus branch November 7, 2019 12:51
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.

6 participants