compute: Reorder building of columns for 'server list'

This has no impact on the end result, but it should make fixing issues
introduced by API microversion 2.69 a little easier.

Conflicts:
    openstackclient/compute/v2/server.py

NOTE(melwitt): The conflict is because change
I36f292fb70c98f6e558f58be55d533d979c47ca7 (Switch image to use SDK) is
not in Train.

Change-Id: I7d70eac8aa1a6197ed05a49f071e6899ec219c03
Signed-off-by: Stephen Finucane <sfinucan@redhat.com>
(cherry picked from commit 4c3de28e83babb0672950320a20492dc61803b4a)
(cherry picked from commit f4adbcef7b1b18d9454109a020e5e2c6ad78b5f1)
(cherry picked from commit cb621f22d4547c20f5722058be8d99de936a48c6)
(cherry picked from commit 3c280727e7da7661e4a31266b11f5eb0893fdb0f)
(cherry picked from commit 03859c6c68bcbfa229d40237a28a7877d7286249)

1 files changed, 99 insertions, 77 deletions

diff --git a/openstackclient/compute/v2/server.py b/openstackclient/compute/v2/server.py
index d3bd1c72..d93322c8 100644
--- a/openstackclient/compute/v2/server.py
+++ b/openstackclient/compute/v2/server.py
@@ -1308,15 +1308,19 @@ class ListServer(command.Lister):
         # flavor name is given, map it to ID.
         flavor_id = None
         if parsed_args.flavor:
-            flavor_id = utils.find_resource(compute_client.flavors,
-                                            parsed_args.flavor).id
+            flavor_id = utils.find_resource(
+                compute_client.flavors,
+                parsed_args.flavor,
+            ).id
 
         # Nova only supports list servers searching by image ID. So if a
         # image name is given, map it to ID.
         image_id = None
         if parsed_args.image:
-            image_id = utils.find_resource(image_client.images,
-                                           parsed_args.image).id
+            image_id = utils.find_resource(
+                image_client.images,
+                parsed_args.image,
+            ).id
 
         search_opts = {
             'reservation_id': parsed_args.reservation_id,
@@ -1366,76 +1370,78 @@ class ListServer(command.Lister):
             try:
                 timeutils.parse_isotime(search_opts['changes-since'])
             except ValueError:
+                msg = _('Invalid changes-since value: %s')
                 raise exceptions.CommandError(
-                    _('Invalid changes-since value: %s') %
-                    search_opts['changes-since']
+                    msg % search_opts['changes-since']
                 )
 
+        columns = (
+            'id',
+            'name',
+            'status',
+        )
+        column_headers = (
+            'ID',
+            'Name',
+            'Status',
+        )
+
         if parsed_args.long:
-            columns = (
-                'ID',
-                'Name',
-                'Status',
+            columns += (
                 'OS-EXT-STS:task_state',
                 'OS-EXT-STS:power_state',
-                'Networks',
-                'Image Name',
-                'Image ID',
-                'Flavor Name',
-                'Flavor ID',
-                'OS-EXT-AZ:availability_zone',
-                'OS-EXT-SRV-ATTR:host',
-                'Metadata',
             )
-            column_headers = (
-                'ID',
-                'Name',
-                'Status',
+            column_headers += (
                 'Task State',
                 'Power State',
-                'Networks',
+            )
+
+        columns += ('networks',)
+        column_headers += ('Networks',)
+
+        if parsed_args.long:
+            columns += (
+                'image_name',
+                'image_id',
+            )
+            column_headers += (
                 'Image Name',
                 'Image ID',
+            )
+        else:
+            if parsed_args.no_name_lookup:
+                columns += ('image_id',)
+            else:
+                columns += ('image_name',)
+            column_headers += ('Image',)
+
+        if parsed_args.long:
+            columns += (
+                'flavor_name',
+                'flavor_id',
+            )
+            column_headers += (
                 'Flavor Name',
                 'Flavor ID',
-                'Availability Zone',
-                'Host',
-                'Properties',
             )
-            mixed_case_fields = [
-                'OS-EXT-STS:task_state',
-                'OS-EXT-STS:power_state',
-                'OS-EXT-AZ:availability_zone',
-                'OS-EXT-SRV-ATTR:host',
-            ]
         else:
             if parsed_args.no_name_lookup:
-                columns = (
-                    'ID',
-                    'Name',
-                    'Status',
-                    'Networks',
-                    'Image ID',
-                    'Flavor ID',
-                )
+                columns += ('flavor_id',)
             else:
-                columns = (
-                    'ID',
-                    'Name',
-                    'Status',
-                    'Networks',
-                    'Image Name',
-                    'Flavor Name',
-                )
-            column_headers = (
-                'ID',
-                'Name',
-                'Status',
-                'Networks',
-                'Image',
-                'Flavor',
+                columns += ('flavor_name',)
+            column_headers += ('Flavor',)
+
+        if parsed_args.long:
+            columns += (
+                'OS-EXT-AZ:availability_zone',
+                'OS-EXT-SRV-ATTR:host',
+                'metadata',
+            )
+            column_headers += (
+                'Availability Zone',
+                'Host',
+                'Properties',
             )
-            mixed_case_fields = []
 
         marker_id = None
 
@@ -1447,25 +1453,29 @@ class ListServer(command.Lister):
             if parsed_args.deleted:
                 marker_id = parsed_args.marker
             else:
-                marker_id = utils.find_resource(compute_client.servers,
-                                                parsed_args.marker).id
+                marker_id = utils.find_resource(
+                    compute_client.servers,
+                    parsed_args.marker,
+                ).id
 
-        data = compute_client.servers.list(search_opts=search_opts,
-                                           marker=marker_id,
-                                           limit=parsed_args.limit)
+        data = compute_client.servers.list(
+            search_opts=search_opts,
+            marker=marker_id,
+            limit=parsed_args.limit)
 
         images = {}
         flavors = {}
         if data and not parsed_args.no_name_lookup:
-            # Create a dict that maps image_id to image object.
-            # Needed so that we can display the "Image Name" column.
-            # "Image Name" is not crucial, so we swallow any exceptions.
-            # The 'image' attribute can be an empty string if the server was
-            # booted from a volume.
+            # create a dict that maps image_id to image object, which is used
+            # to display the "Image Name" column. Note that 'image.id' can be
+            # empty for BFV instances and 'image' can be missing entirely if
+            # there are infra failures
             if parsed_args.name_lookup_one_by_one or image_id:
-                for i_id in set(filter(lambda x: x is not None,
-                                       (s.image.get('id') for s in data
-                                        if s.image))):
+                for i_id in set(
+                    s.image['id'] for s in data
+                    if s.image and s.image.get('id')
+                ):
+                    # "Image Name" is not crucial, so we swallow any exceptions
                     try:
                         images[i_id] = image_client.images.get(i_id)
                     except Exception:
@@ -1478,12 +1488,17 @@ class ListServer(command.Lister):
                 except Exception:
                     pass
 
-            # Create a dict that maps flavor_id to flavor object.
-            # Needed so that we can display the "Flavor Name" column.
-            # "Flavor Name" is not crucial, so we swallow any exceptions.
+            # create a dict that maps flavor_id to flavor object, which is used
+            # to display the "Flavor Name" column. Note that 'flavor.id' is not
+            # present on microversion 2.47 or later and 'flavor' won't be
+            # present if there are infra failures
             if parsed_args.name_lookup_one_by_one or flavor_id:
-                for f_id in set(filter(lambda x: x is not None,
-                                       (s.flavor.get('id') for s in data))):
+                for f_id in set(
+                    s.flavor['id'] for s in data
+                    if s.flavor and s.flavor.get('id')
+                ):
+                    # "Flavor Name" is not crucial, so we swallow any
+                    # exceptions
                     try:
                         flavors[f_id] = compute_client.flavors.get(f_id)
                     except Exception:
@@ -1507,6 +1522,7 @@ class ListServer(command.Lister):
                 # processing of the image and flavor informations.
                 if not hasattr(s, 'image') or not hasattr(s, 'flavor'):
                     continue
+
             if 'id' in s.image:
                 image = images.get(s.image['id'])
                 if image:
@@ -1515,6 +1531,7 @@ class ListServer(command.Lister):
             else:
                 s.image_name = ''
                 s.image_id = ''
+
             if 'id' in s.flavor:
                 flavor = flavors.get(s.flavor['id'])
                 if flavor:
@@ -1531,12 +1548,17 @@ class ListServer(command.Lister):
         table = (column_headers,
                  (utils.get_item_properties(
                      s, columns,
-                     mixed_case_fields=mixed_case_fields,
+                     mixed_case_fields=(
+                         'OS-EXT-STS:task_state',
+                         'OS-EXT-STS:power_state',
+                         'OS-EXT-AZ:availability_zone',
+                         'OS-EXT-SRV-ATTR:host',
+                     ),
                      formatters={
                          'OS-EXT-STS:power_state':
                              _format_servers_list_power_state,
-                         'Networks': _format_servers_list_networks,
-                         'Metadata': utils.format_dict,
+                         'networks': _format_servers_list_networks,
+                         'metadata': utils.format_dict,
                      },
                  ) for s in data))
         return table