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>

1 files changed, 11 insertions, 1 deletions

diff --git a/openstackclient/compute/v2/server.py b/openstackclient/compute/v2/server.py
index 111c4a6b..48f5b7cf 100644
--- a/openstackclient/compute/v2/server.py
+++ b/openstackclient/compute/v2/server.py
@@ -989,8 +989,9 @@ class CreateServer(command.ShowOne):
             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 @@ class CreateServer(command.ShowOne):
         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)