-
Notifications
You must be signed in to change notification settings - Fork 12
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
Add tests for barclamp-ceph #28
base: master
Are you sure you want to change the base?
Conversation
|
@mmnelemane can you rebase instead of merge master here? |
I'm not sure if it it wouldn't make more sense to run the ceph testsuite rather than writing our own tests here.. (see testsetup which does that already) |
3adfcc3
to
360e92d
Compare
Rebased it to latest commit in master. |
360e92d
to
3b26a43
Compare
Feature: Tests Ceph barclamp deployment | ||
As an actor | ||
I want to perform various validations | ||
In order to verify the feature functionality |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This description is the generated default one, please adapt the wording.
00f2028
to
8b3039a
Compare
end | ||
|
||
desc "Run barclamp tests" | ||
task :barclamps do | ||
|
||
invoke_task "feature_barclamp:ceph" | ||
invoke_task "feature:barclamp:database" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't think the database line is correct...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@vuntz: Thanks for observation. actually, its the ceph line needing correction. just corrected the same.
2b56ae7
to
a891267
Compare
@mmnelemane can you please rebase? |
The tests for Ceph barclamp for basic deployment with all ceph elements and tested on a 3-node cluster.