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.

Sign up for GitHub

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

Jump to bottom

Revert "os/bluestore: Enable sharding by default" #34970

Merged
merged 1 commit into from
May 12, 2020
Merged

Revert "os/bluestore: Enable sharding by default" #34970

merged 1 commit into from
May 12, 2020

Conversation

sebastian-philipp
Copy link
Contributor

This is just a changeset to attempt to get the cephadm upgrade tests running again. Let's see if this is already enough to get the tests pass.

not intended to get merged.

Relates to https://tracker.ceph.com/issues/45335

This reverts commit aa1a116.

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 dashboard backend
  • jenkins test docs
  • jenkins render docs
  • jenkins test ceph-volume all
  • jenkins test ceph-volume tox

@sebastian-philipp sebastian-philipp added the wip-swagner-testing My Teuthology tests label May 8, 2020
@tchaikov
Copy link
Contributor

tchaikov commented May 9, 2020

the right fix of #34864 is still being tested.

aclamk
aclamk approved these changes May 11, 2020
View reviewed changes
Copy link
Contributor

@aclamk aclamk left a comment

Choose a reason for hiding this comment

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

APPROVE!

@sebastian-philipp
Copy link
Contributor Author

APPROVE!

@aclamk you'll then take care of re-enabling this, as soon as you're finished with #34864 ?

@aclamk
Copy link
Contributor

aclamk commented May 11, 2020

APPROVE!

@aclamk you'll then take care of re-enabling this, as soon as you're finished with #34864 ?

If this one merges, re-enabling will became part of #34864.

@sebastian-philipp
Revert "os/bluestore: Enable sharding by default"
619fcdd 
This reverts commit aa1a116.

Signed-off-by: Sebastian Wagner <sebastian.wagner@suse.com>
@sebastian-philipp
Copy link
Contributor Author

http://pulpito.ceph.com/swagner-2020-05-11_17:25:18-rados-wip-swagner-testing-2020-05-11-1648-distro-basic-smithi/

@sebastian-philipp sebastian-philipp merged commit cf51532 into ceph:master May 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aclamk aclamk aclamk approved these changes

Assignees
No one assigned
Labels
bluestore cephadm wip-mgfritch-testing wip-swagner2-testing wip-swagner-testing My Teuthology tests
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@sebastian-philipp @tchaikov @aclamk @mgfritch