summaryrefslogtreecommitdiffstats
path: root/roles/openshift_health_checker/test
diff options
context:
space:
mode:
authorRodolfo Carvalho <rhcarvalho@gmail.com>2017-07-28 13:42:22 +0200
committerGitHub <noreply@github.com>2017-07-28 13:42:22 +0200
commit32710db7c1ae34f884c73a7d3b3c1cbc2e368eca (patch)
treeb988a53a46daee702f5a32049af01b57cedb8c3f /roles/openshift_health_checker/test
parentd0761aee7107dffa4803f39b2aaecb660d767752 (diff)
downloadopenshift-32710db7c1ae34f884c73a7d3b3c1cbc2e368eca.tar.gz
openshift-32710db7c1ae34f884c73a7d3b3c1cbc2e368eca.tar.bz2
openshift-32710db7c1ae34f884c73a7d3b3c1cbc2e368eca.tar.xz
openshift-32710db7c1ae34f884c73a7d3b3c1cbc2e368eca.zip
Revert "Add health checks to upgrade playbook"
Diffstat (limited to 'roles/openshift_health_checker/test')
-rw-r--r--roles/openshift_health_checker/test/disk_availability_test.py69
1 files changed, 5 insertions, 64 deletions
diff --git a/roles/openshift_health_checker/test/disk_availability_test.py b/roles/openshift_health_checker/test/disk_availability_test.py
index 5720eeacf..e98d02c58 100644
--- a/roles/openshift_health_checker/test/disk_availability_test.py
+++ b/roles/openshift_health_checker/test/disk_availability_test.py
@@ -97,9 +97,8 @@ def test_succeeds_with_recommended_disk_space(group_names, configured_min, ansib
assert not result.get('failed', False)
-@pytest.mark.parametrize('name,group_names,configured_min,ansible_mounts,extra_words', [
+@pytest.mark.parametrize('group_names,configured_min,ansible_mounts,extra_words', [
(
- 'test with no space available',
['masters'],
0,
[{
@@ -109,7 +108,6 @@ def test_succeeds_with_recommended_disk_space(group_names, configured_min, ansib
['0.0 GB'],
),
(
- 'test with a higher configured required value',
['masters'],
100, # set a higher threshold
[{
@@ -119,7 +117,6 @@ def test_succeeds_with_recommended_disk_space(group_names, configured_min, ansib
['100.0 GB'],
),
(
- 'test with 1GB available, but "0" GB space requirement',
['nodes'],
0,
[{
@@ -129,7 +126,6 @@ def test_succeeds_with_recommended_disk_space(group_names, configured_min, ansib
['1.0 GB'],
),
(
- 'test with no space available, but "0" GB space requirement',
['etcd'],
0,
[{
@@ -139,17 +135,16 @@ def test_succeeds_with_recommended_disk_space(group_names, configured_min, ansib
['0.0 GB'],
),
(
- 'test with enough space for a node, but not for a master',
['nodes', 'masters'],
0,
[{
'mount': '/',
+ # enough space for a node, not enough for a master
'size_available': 15 * 10**9 + 1,
}],
['15.0 GB'],
),
(
- 'test failure with enough space on "/", but not enough on "/var"',
['etcd'],
0,
[{
@@ -163,8 +158,8 @@ def test_succeeds_with_recommended_disk_space(group_names, configured_min, ansib
}],
['0.0 GB'],
),
-], ids=lambda argval: argval[0])
-def test_fails_with_insufficient_disk_space(name, group_names, configured_min, ansible_mounts, extra_words):
+])
+def test_fails_with_insufficient_disk_space(group_names, configured_min, ansible_mounts, extra_words):
task_vars = dict(
group_names=group_names,
openshift_check_min_host_disk_gb=configured_min,
@@ -175,61 +170,7 @@ def test_fails_with_insufficient_disk_space(name, group_names, configured_min, a
assert result['failed']
for word in 'below recommended'.split() + extra_words:
- assert word in result.get('msg', '')
-
-
-@pytest.mark.parametrize('name,group_names,context,ansible_mounts,failed,extra_words', [
- (
- 'test without enough space for master under "upgrade" context',
- ['nodes', 'masters'],
- "upgrade",
- [{
- 'mount': '/',
- 'size_available': 1 * 10**9 + 1,
- 'size_total': 21 * 10**9 + 1,
- }],
- True,
- ["1.0 GB"],
- ),
- (
- 'test with enough space for master under "upgrade" context',
- ['nodes', 'masters'],
- "upgrade",
- [{
- 'mount': '/',
- 'size_available': 10 * 10**9 + 1,
- 'size_total': 21 * 10**9 + 1,
- }],
- False,
- [],
- ),
- (
- 'test with not enough space for master, and non-upgrade context',
- ['nodes', 'masters'],
- "health",
- [{
- 'mount': '/',
- # not enough space for a master,
- # "health" context should not lower requirement
- 'size_available': 20 * 10**9 + 1,
- }],
- True,
- ["20.0 GB", "below minimum"],
- ),
-], ids=lambda argval: argval[0])
-def test_min_required_space_changes_with_upgrade_context(name, group_names, context, ansible_mounts, failed, extra_words):
- task_vars = dict(
- r_openshift_health_checker_playbook_context=context,
- group_names=group_names,
- ansible_mounts=ansible_mounts,
- )
-
- check = DiskAvailability(fake_execute_module, task_vars)
- result = check.run()
-
- assert result.get("failed", False) == failed
- for word in extra_words:
- assert word in result.get('msg', '')
+ assert word in result['msg']
def fake_execute_module(*args):