Skip to content

Commit 7ceff0e

Browse files
Jenkinsopenstack-gerrit
authored andcommitted
Merge "Fix server image create"
2 parents 870e7dd + da5e31d commit 7ceff0e

2 files changed

Lines changed: 5 additions & 9 deletions

File tree

‎openstackclient/compute/v2/server.py‎

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -393,7 +393,7 @@ def get_parser(self, prog_name):
393393
parser = super(CreateServerImage, self).get_parser(prog_name)
394394
parser.add_argument(
395395
'server',
396-
metavar='<server',
396+
metavar='<server>',
397397
help='Server (name or ID)',
398398
)
399399
parser.add_argument(
@@ -421,15 +421,15 @@ def take_action(self, parsed_args):
421421
else:
422422
name = server.name
423423

424-
image = compute_client.servers.create_image(
424+
image_id = compute_client.servers.create_image(
425425
server,
426426
name,
427427
)
428428

429429
if parsed_args.wait:
430430
if utils.wait_for_status(
431431
image_client.images.get,
432-
image,
432+
image_id,
433433
callback=_show_progress,
434434
):
435435
sys.stdout.write('\n')
@@ -443,7 +443,7 @@ def take_action(self, parsed_args):
443443

444444
image = utils.find_resource(
445445
image_client.images,
446-
image.id,
446+
image_id,
447447
)
448448

449449
info = {}

‎openstackclient/tests/compute/v2/test_server.py‎

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -80,11 +80,7 @@ def setUp(self):
8080
loaded=True,
8181
)
8282

83-
self.servers_mock.create_image.return_value = fakes.FakeResource(
84-
None,
85-
copy.deepcopy(image_fakes.IMAGE),
86-
loaded=True,
87-
)
83+
self.servers_mock.create_image.return_value = image_fakes.image_id
8884

8985
self.images_mock.get.return_value = fakes.FakeResource(
9086
None,

0 commit comments

Comments
 (0)