From 74c5f22c6ffb23154f2e0412ba43371f3c02f3b7 Mon Sep 17 00:00:00 2001
From: Feilong Wang <flwang@catalyst.net.nz>
Date: Fri, 1 Mar 2019 11:36:31 +1300
Subject: [PATCH] Support health_status on client side

Task: 29762
Story: 2002742

Depends-on: If702584fabe1402257b45db281561a5f5b83b972
Change-Id: Iffce9a2cc422bbce70152be3a58d2573e80b104b
---
 magnumclient/osc/v1/clusters.py                 | 5 ++++-
 magnumclient/tests/osc/unit/v1/fakes.py         | 2 ++
 magnumclient/tests/osc/unit/v1/test_clusters.py | 6 ++++--
 3 files changed, 10 insertions(+), 3 deletions(-)

diff --git a/magnumclient/osc/v1/clusters.py b/magnumclient/osc/v1/clusters.py
index d2829fa2..fa2bdcbc 100644
--- a/magnumclient/osc/v1/clusters.py
+++ b/magnumclient/osc/v1/clusters.py
@@ -24,6 +24,7 @@ from osc_lib import utils
 
 CLUSTER_ATTRIBUTES = [
     'status',
+    'health_status',
     'cluster_template_id',
     'node_addresses',
     'uuid',
@@ -45,6 +46,7 @@ CLUSTER_ATTRIBUTES = [
     'name',
     'master_flavor_id',
     'flavor_id',
+    'health_status_reason',
 ]
 
 
@@ -197,7 +199,8 @@ class ListCluster(command.Lister):
 
         mag_client = self.app.client_manager.container_infra
         columns = [
-            'uuid', 'name', 'keypair', 'node_count', 'master_count', 'status']
+            'uuid', 'name', 'keypair', 'node_count', 'master_count', 'status',
+            'health_status']
         clusters = mag_client.clusters.list(limit=parsed_args.limit,
                                             sort_key=parsed_args.sort_key,
                                             sort_dir=parsed_args.sort_dir)
diff --git a/magnumclient/tests/osc/unit/v1/fakes.py b/magnumclient/tests/osc/unit/v1/fakes.py
index df11d411..593e0e53 100644
--- a/magnumclient/tests/osc/unit/v1/fakes.py
+++ b/magnumclient/tests/osc/unit/v1/fakes.py
@@ -205,6 +205,7 @@ class FakeCluster(object):
         # set default attributes.
         cluster_info = {
             'status': 'CREATE_IN_PROGRESS',
+            'health_status': 'HEALTHY',
             'cluster_template_id': 'fake-ct',
             'node_addresses': [],
             'uuid': '3a369884-b6ba-484f-a206-919b4b718aff',
@@ -227,6 +228,7 @@ class FakeCluster(object):
             'master_flavor_id': None,
             'flavor_id': 'm1.medium',
             'project_id': None,
+            'health_status_reason': {'api': 'ok'}
         }
 
         # Overwrite default attributes.
diff --git a/magnumclient/tests/osc/unit/v1/test_clusters.py b/magnumclient/tests/osc/unit/v1/test_clusters.py
index 4783c138..fcb895c3 100644
--- a/magnumclient/tests/osc/unit/v1/test_clusters.py
+++ b/magnumclient/tests/osc/unit/v1/test_clusters.py
@@ -181,7 +181,8 @@ class TestClusterList(TestCluster):
         'keypair',
         'node_count',
         'master_count',
-        'status'
+        'status',
+        'health_status'
     ]
 
     datalist = (
@@ -191,7 +192,8 @@ class TestClusterList(TestCluster):
             _cluster.keypair,
             _cluster.node_count,
             _cluster.master_count,
-            _cluster.status
+            _cluster.status,
+            _cluster.health_status,
         ),
     )