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

crush: fix cli tests for new crush tunables #8107

Merged
merged 2 commits into from Mar 16, 2016
Merged

Conversation

liewegas
Copy link
Member

No description provided.

As of 000344e.

Signed-off-by: Sage Weil <sage@redhat.com>
Adjust accordingly to keep tests passing.

Signed-off-by: Sage Weil <sage@redhat.com>
@liewegas liewegas changed the title crush: fix cli tests for new crush tuanbles crush: fix cli tests for new crush tunables Mar 14, 2016
@liewegas liewegas added this to the jewel milestone Mar 14, 2016
@tchaikov
Copy link
Contributor

lgtm

@tchaikov
Copy link
Contributor

@liewegas sage, i am merging this changeset despite of the "needs-qa" label, because

  • they are changes for the tests
  • and the fixed tests passed in the jenkins run
  • the master is broken due to the failed tests

tchaikov added a commit that referenced this pull request Mar 16, 2016
crush: fix cli tests for new crush tunables

Reviewed-by: Kefu Chai <kchai@redhat.com>
@tchaikov tchaikov merged commit 167ddd6 into ceph:master Mar 16, 2016
@liewegas
Copy link
Member Author

liewegas commented Mar 16, 2016 via email

@liewegas liewegas deleted the wip-crush branch March 17, 2016 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants