-
Notifications
You must be signed in to change notification settings - Fork 75
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
Enable openstack integration tests #1322
Comments
@tserong This issue arises because of
In other words, the openstack orchestration is creating pools with a hard-coded number of PGs (128) which is inappropriate for the (small) cluster deployed by the CI. The qa scripting already has a "pool pre-creation" mechanism which calculates the "right" number of PGs per pool based on the number of pools and the number of OSDs in the cluster. I can easily add an option to the qa deployment script to pre-create these four pools. However, this alone is not sufficient to fix the issue because the openattic functest orchestration deletes these pools at the beginning of the test: https://github.com/SUSE/DeepSea/blob/SES5/srv/salt/ceph/functests/1node/openstack/default.sls#L3-L6 (It deletes the pools at the end of the test, too, but that has little or no bearing on this issue since the CI always runs the tests in a fresh cluster which is destroyed at the end of the test.) So, how do you suggest to address this issue? The only thing I can think of right now is to drop the "clean environment at start" bit. Combined with my tweak on the qa side, this would make the functest work in the CI, but it would mean that folks running the functest manually would have to take care to manually delete these pools, first, or risk running the tests on a "dirty" environment. |
Ugh. I wonder if we should optionally parameterize the number of PGs instead? That'd possibly help real deployments too (where the admin wanted to have control over the number of PGs). |
What does "parameterize" mean, though? Like have the SLS call a runner that reads a YAML file where the number of PGs could be optionally set? |
All the The only problem here is the functest is deleting the pool and re-creating it with the wrong number of PGs. |
Nope, just thinking to let the number of PGs be optionally passed in somehow as another command line param (the last thing I want to do is add more YAML :-)) But let's not do that either, I'll just get rid of the initial pool deletion. |
If we don't initially delete the openstack pools, it means QA infra can pre-create these pools with the right number of PGs for the test environment. Fixes: #1322 Signed-off-by: Tim Serong <tserong@suse.com>
Description of Issue/Question
There's an effort to enable the openstack tests in #1316. That's the result
We need to tweak something in the tests/env to make that work out of the box
The text was updated successfully, but these errors were encountered: