From 32710db7c1ae34f884c73a7d3b3c1cbc2e368eca Mon Sep 17 00:00:00 2001
From: Rodolfo Carvalho <rhcarvalho@gmail.com>
Date: Fri, 28 Jul 2017 13:42:22 +0200
Subject: Revert "Add health checks to upgrade playbook"

---
 .../test/disk_availability_test.py                 | 69 ++--------------------
 1 file changed, 5 insertions(+), 64 deletions(-)

(limited to 'roles/openshift_health_checker/test')

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):
-- 
cgit v1.2.3