-
Notifications
You must be signed in to change notification settings - Fork 183
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
[WIP] beaker usually runs all tests on each provided os node #129
[WIP] beaker usually runs all tests on each provided os node #129
Conversation
control-repo has another usecase: run the test for each role onto a subset of os nodes we disable beaker task and generate new ones based on available acceptance tests for all roles to test we use multitask instead of task make papply puppet 5 aware
- add .fixtures.yml to build sandbox - add spec_helper and spec_helper_acceptance - add default nodeset (docker centos7) - add first profile test
- beaker installs puppet agent which will installenvironment.conf.rpmnew - added to gitignore - on beaker hosts no git is installed. check for git cli in config_version script - preserve beaker docker hosts
Let me know when is ready to be merged |
@alvagante
|
The problem is with hiera eyaml or with the missing keys? So, it's not possible to make beaker work with eyaml without keys (and without encrypted entries), I'd use a hiera.yaml dedicated for Beaker, with the burden to keep it aligned to the main one. |
I decided to go for the most simple approach: just generate a keypair. I know that in this case we can never test encrypted value lookup properly. |
Probably the configuration has to adapt to non ipv6 environments as probably the intances run by beaker. But that's another problem, I'd say, feel free to skip it (commenting or changing the default mail_class) or quickly fix the issue in the profile's template |
lint cleanup unicode cleanup
Tests are now running directly on psick root folder.
|
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.
Why moving profile spec tests in the main control repo dir?
I would place here only node/roles tests (http://rspec-puppet.com/documentation/hosts/)
I'd keep the spec structure in the main dir, add in spec/ only nodes spec for all the roles we want to test (just checking for compilation on different OS of most or all the roles would be great). |
+1 for adding hosts unit and acceptance tests only to psick root and keeping the other unit tests inside the site modules. |
control-repo has another usecase:
run the test for each role onto a subset of os nodes
we disable beaker task and generate new ones based on available
acceptance tests
for all roles to test we use multitask instead of task
make papply puppet 5 aware