openstack/python-openstackclient
Code Issues Proposed changes

Merge "image: Add support for additional image import methods"

Browse Source
This commit is contained in:
Zuul 2023-01-25 18:04:00 +00:00 committed by Gerrit Code Review
commit f1da522cc3
2 changed files with 20 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
openstackclient/image/v2/image.py
View File

@ -1805,10 +1805,10 @@ class ImportImage(command.ShowOne):
image_client.import_image( image_client.import_image(
image, image,
method=parsed_args.import_method, method=parsed_args.import_method,
# uri=parsed_args.uri, uri=parsed_args.uri,
# remote_region=parsed_args.remote_region, remote_region=parsed_args.remote_region,
# remote_image=parsed_args.remote_image, remote_image=parsed_args.remote_image,
# remote_service_interface=parsed_args.remote_service_interface, remote_service_interface=parsed_args.remote_service_interface,
stores=parsed_args.stores, stores=parsed_args.stores,
all_stores=parsed_args.all_stores, all_stores=parsed_args.all_stores,
all_stores_must_succeed=not parsed_args.allow_failure, all_stores_must_succeed=not parsed_args.allow_failure,

20
openstackclient/tests/unit/image/v2/test_image.py
View File

@ -1859,6 +1859,10 @@ class TestImageImport(TestImage):
self.client.import_image.assert_called_once_with( self.client.import_image.assert_called_once_with(
self.image, self.image,
method='glance-direct', method='glance-direct',
uri=None,
remote_region=None,
remote_image=None,
remote_service_interface=None,
stores=None, stores=None,
all_stores=None, all_stores=None,
all_stores_must_succeed=False, all_stores_must_succeed=False,
@ -1883,7 +1887,10 @@ class TestImageImport(TestImage):
self.client.import_image.assert_called_once_with( self.client.import_image.assert_called_once_with(
self.image, self.image,
method='web-download', method='web-download',
# uri='https://example.com/', uri='https://example.com/',
remote_region=None,
remote_image=None,
remote_service_interface=None,
stores=None, stores=None,
all_stores=None, all_stores=None,
all_stores_must_succeed=False, all_stores_must_succeed=False,
@ -1981,6 +1988,10 @@ class TestImageImport(TestImage):
self.client.import_image.assert_called_once_with( self.client.import_image.assert_called_once_with(
self.image, self.image,
method='copy-image', method='copy-image',
uri=None,
remote_region=None,
remote_image=None,
remote_service_interface=None,
stores=['fast'], stores=['fast'],
all_stores=None, all_stores=None,
all_stores_must_succeed=False, all_stores_must_succeed=False,
@ -2008,9 +2019,10 @@ class TestImageImport(TestImage):
self.client.import_image.assert_called_once_with( self.client.import_image.assert_called_once_with(
self.image, self.image,
method='glance-download', method='glance-download',
# remote_region='eu/dublin', uri=None,
# remote_image='remote-image-id', remote_region='eu/dublin',
# remote_service_interface='private', remote_image='remote-image-id',
remote_service_interface='private',
stores=None, stores=None,
all_stores=None, all_stores=None,
all_stores_must_succeed=False, all_stores_must_succeed=False,