From eca1fcd65f22ead444ec39fe20f48c83f8d7c1cf Mon Sep 17 00:00:00 2001
From: David Caro <me@dcaro.es>
Date: Wed, 2 Jun 2021 16:33:53 +0200
Subject: [PATCH] Include hosts in aggregate list --long

This makes it easier to get the total list of aggregates and the hosts
belonging to each of them (specially for scripting purposes).

Change-Id: I94833c15075ae655bc11e7c0fc47c0abad5846fc
Signed-off-by: David Caro <me@dcaro.es>
---
 openstackclient/compute/v2/aggregate.py                 | 2 ++
 openstackclient/tests/unit/compute/v2/test_aggregate.py | 2 ++
 2 files changed, 4 insertions(+)

diff --git a/openstackclient/compute/v2/aggregate.py b/openstackclient/compute/v2/aggregate.py
index e39eb2d272..37522a788a 100644
--- a/openstackclient/compute/v2/aggregate.py
+++ b/openstackclient/compute/v2/aggregate.py
@@ -193,12 +193,14 @@ class ListAggregate(command.Lister):
                 "Name",
                 "Availability Zone",
                 "Properties",
+                "Hosts",
             )
             columns = (
                 "ID",
                 "Name",
                 "Availability Zone",
                 "Metadata",
+                "Hosts",
             )
         else:
             column_headers = columns = (
diff --git a/openstackclient/tests/unit/compute/v2/test_aggregate.py b/openstackclient/tests/unit/compute/v2/test_aggregate.py
index 8563f98811..92ff607f94 100644
--- a/openstackclient/tests/unit/compute/v2/test_aggregate.py
+++ b/openstackclient/tests/unit/compute/v2/test_aggregate.py
@@ -234,6 +234,7 @@ class TestAggregateList(TestAggregate):
         "Name",
         "Availability Zone",
         "Properties",
+        "Hosts",
     )
 
     list_data = ((
@@ -251,6 +252,7 @@ class TestAggregateList(TestAggregate):
             for key, value in TestAggregate.fake_ag.metadata.items()
             if key != 'availability_zone'
         }),
+        format_columns.ListColumn(TestAggregate.fake_ag.hosts),
     ), )
 
     def setUp(self):