compute: Deprecate 'server create --file'

The parameter isn't actually deprecated, since we need to support older
API microversion, however, we now emit an error if someone attempts to
boot a server with the wrong microversion. This would happen server-side
anyway since this parameter was removed entirely in API microversion
2.57.

Change-Id: I73864ccbf5bf181fecf505ca168c1a35a8b0af3a
Signed-off-by: Stephen Finucane <sfinucan@redhat.com>

Author: stephenfin

openstackclient/compute/v2/server.py

@@ -989,8 +989,9 @@ def get_parser(self, prog_name):
             action='append',
             default=[],
             help=_(
-                'File to inject into image before boot '
+                'File(s) to inject into image before boot '
                 '(repeat option to set multiple files)'
+                '(supported by --os-compute-api-version 2.57 or below)'
             ),
         )
         parser.add_argument(
@@ -1201,6 +1202,15 @@ def _match_image(image_api, wanted_properties):
         flavor = utils.find_resource(
             compute_client.flavors, parsed_args.flavor)
 
+        if parsed_args.file:
+            if compute_client.api_version >= api_versions.APIVersion('2.57'):
+                msg = _(
+                    'Personality files are deprecated and are not supported '
+                    'for --os-compute-api-version greater than 2.56; use '
+                    'user data instead'
+                )
+                raise exceptions.CommandError(msg)
+
         files = {}
         for f in parsed_args.file:
             dst, src = f.split('=', 1)

releasenotes/notes/deprecated-server-create-file-option-80246b13bd3c1b43.yaml

@@ -0,0 +1,7 @@
+---
+upgrade:
+  - |
+    The ``server create`` command will now error out if the ``--file`` option
+    is specified alongside ``--os-compute-api-version`` of ``2.57`` or greater.
+    This reflects the removal of this feature from the compute service in this
+    microversion.