Show "Forced Down" compute service status with --long

Currently, the unified client does not have the ability to show the
"Forced Down" field of a GET /os-services response in microversion 2.11
even though the legacy client can.

This adds a "Forced Down" column to the 'openstack compute service list
--long' command output when microversion 2.11 is used.

Story: 2009115
Task: 43011

Change-Id: I10bc2fedbf0e867a990227962b2b6e60f5681f69

Author: melwitt

openstackclient/compute/v2/service.py

@@ -103,6 +103,10 @@ def take_action(self, parsed_args):
                 "Updated At",
                 "Disabled Reason"
             )
+            has_forced_down = (
+                compute_client.api_version >= api_versions.APIVersion('2.11'))
+            if has_forced_down:
+                columns += ("Forced Down",)
         else:
             columns = (
                 "ID",

openstackclient/tests/unit/compute/v2/fakes.py

@@ -722,6 +722,8 @@ def create_one_service(attrs=None):
             'state': 'state-' + uuid.uuid4().hex,
             'updated_at': 'time-' + uuid.uuid4().hex,
             'disabled_reason': 'earthquake',
+            # Introduced in API microversion 2.11
+            'forced_down': False,
         }
 
         # Overwrite default attributes.

openstackclient/tests/unit/compute/v2/test_service.py

@@ -190,6 +190,38 @@ def test_service_list_with_long_option(self):
         self.assertEqual(self.columns_long, columns)
         self.assertEqual(self.data_long, list(data))
 
+    def test_service_list_with_long_option_2_11(self):
+        arglist = [
+            '--host', self.service.host,
+            '--service', self.service.binary,
+            '--long'
+        ]
+        verifylist = [
+            ('host', self.service.host),
+            ('service', self.service.binary),
+            ('long', True)
+        ]
+        parsed_args = self.check_parser(self.cmd, arglist, verifylist)
+        self.app.client_manager.compute.api_version = api_versions.APIVersion(
+            '2.11')
+
+        # In base command class Lister in cliff, abstract method take_action()
+        # returns a tuple containing the column names and an iterable
+        # containing the data to be listed.
+        columns, data = self.cmd.take_action(parsed_args)
+
+        self.service_mock.list.assert_called_with(
+            self.service.host,
+            self.service.binary,
+        )
+
+        # In 2.11 there is also a forced_down column.
+        columns_long = self.columns_long + ('Forced Down',)
+        data_long = [self.data_long[0] + (self.service.forced_down,)]
+
+        self.assertEqual(columns_long, columns)
+        self.assertEqual(data_long, list(data))
+
 
 class TestServiceSet(TestService):
 

releasenotes/notes/compute-service-list-forced-down-2b16d1cb44f71a08.yaml

@@ -0,0 +1,5 @@
+---
+features:
+  - |
+    Add column ``Forced Down`` to the output of ``compute service list
+    --long``. Only available starting with ``--os-compute-api-version 2.11``.