summaryrefslogtreecommitdiff
path: root/openstackclient/image
Commit message (Collapse)AuthorAgeFilesLines
* osc-lib: shellDean Troyer2016-08-051-6/+6
| | | | | | | | | | | | | Convert to using ClientManager and OpenStackShell from osc-lib. * Change all internal uses of ClientManager private attributes that are now public in osc-lib's ClientManager. Leave back-compat copies in place in OSC's clientManager so we don't break plugins. * Put some work-arounds in place for changes in osc-lib that we need until a new release makes it through the g-r and u-c change process. * Add a test for Unicode decoding of argv in shell.main() to parallel the one in osc-lib. Change-Id: I85289740d4ca081f2aca8c9b40ec422ad25d302c
* Remove execute permission on a few filesSongmingYan2016-07-221-0/+0
| | | | | | | Some files have execute permission unnecessarily. Change them from 755 to 644. Change-Id: I471ebd1c3d123ad4a7376f7f5996f53f8c2d9b0b
* image list: Add Checksum columnMarc Abramowitz2016-07-112-0/+4
| | | | | | | | | The checksum could potentially be useful for spotting duplicated images or checking if images with the same name are identical or different. Closes-Bug: #1602073 Change-Id: Ia0c41970c846d550de14297e18bc738e847e5a3b
* Correct reraising of exceptionqinchunhua2016-07-071-4/+4
| | | | | | | | | When an exception was caught and rethrown, it should call 'raise' without any arguments because it shows the place where an exception occured initially instead of place where the exception re-raised. Change-Id: I5fb6dea5da7fb6e1e2b339a713c7d37f8c99e407
* Merge "Make set/unset commands in compute/image/common return normally when ↵Jenkins2016-06-221-4/+0
|\ | | | | | | nothing specified"
| * Make set/unset commands in compute/image/common return normally when nothing ↵Tang Chen2016-06-211-4/+0
| | | | | | | | | | | | | | | | | | | | specified After this patch, all set/unset commands will return normally when nothing specified. Change-Id: Id94d0329faa1a674006a9aae901f834b41917317 Close-bug: #1588588
* | Make the print info support i18nliyifeng2016-06-211-1/+2
|/ | | | | | | | When OSC failed to set a image's property, it will print the image name and status, which is an useful info to users. So translate this massage, and record it in log. Change-Id: Icdff4dab17dedcb40289700c4cd278e6e62eea25
* Standardize logger usageTang Chen2016-06-202-28/+27
| | | | | | | | | | | | | | | | | | | Use file logger for all command specific logs. This patch also fixes some usage that doesn't follow rules in: http://docs.openstack.org/developer/oslo.i18n/guidelines.html After this patch, all self.log and self.app.log will be standardized to LOG(). NOTE: In shell.py, we got the log in class OpenStackShell, which is also known as self.app.log in other classes. This logger is used to record non-command-specific logs. So we leave it as-is. Change-Id: I114f73ee6c7e84593d71e724bc1ad00d343c1896 Implements: blueprint log-usage
* Merge "Make set/unset command in identity and image pass normally when ↵Jenkins2016-06-171-13/+0
|\ | | | | | | nothing specified"
| * Make set/unset command in identity and image pass normally when nothing ↵sunyajing2016-06-171-13/+0
| | | | | | | | | | | | | | | | | | specified Also update its unit tests. Change-Id: I82b90658b0d4247cdc9a650f14aceda640a32059 Partial-bug: #1588588
* | Merge "Fix image delete multiple arguments error"Jenkins2016-06-141-5/+19
|\ \ | |/ |/|
| * Fix image delete multiple arguments errorsunyajing2016-06-131-5/+19
| | | | | | | | | | | | | | | | | | Fix image delete command, support processing multiple arguments delete error. Fix doc/source/command-errors.rst, make the msg format correct. Change-Id: Icbe347fe077bc148bf71ea8f7399b0e934b7cdf9 Partially-Implements: blueprint multi-argument-image
* | osc-lib: commandDean Troyer2016-06-132-2/+2
| | | | | | | | | | | | | | Leave command.py and test_command.py as a sanity check during the deprecation period. Change-Id: I24e1b755cbfbcbcaeb5273ec0c9706b82384fc85
* | osc-lib: parseractionsDean Troyer2016-06-132-3/+3
| | | | | | | | | | | | | | Leave parseractions.py and test_parseractions.py as a sanity check during the deprecation period. Change-Id: I1a7469b6d872284e0276502a1a287bc0b87f8f83
* | osc-lib: utilsDean Troyer2016-06-133-4/+6
| | | | | | | | | | | | | | | | Use osc-lib directly for utils. Leave openstackclient.common.utils for deprecation period. Change-Id: I5bd9579abc4e07f45219ccd0565626e6667472f7
* | osc-lib: exceptionsDean Troyer2016-06-131-1/+1
|/ | | | | | | | Use osc-lib directly for exceptions. Leave openstackclient.common.exceptions for deprecation period. Change-Id: Iea3e862302372e1b31ccd27f69db59b4953ca828
* fix image unsetzheng yin2016-06-061-1/+1
| | | | | | modify "target" to "tagret" Change-Id: I80674b23804b26430aed13b5e6c6dc2b240771cd
* Add "image unset" commandsunyajing2016-05-281-2/+87
| | | | | | | | This patch add a command that supports unsetting image tags and properties Change-Id: I6f2cf45a61ff89da6664f3a34ae49fdd85d8c986 Closes-Bug:#1582968
* Fix i18n support for help and log.warning in imageXiaoyang Zhang2016-05-193-141/+145
| | | | Change-Id: I6e2764aaf4b8c0efa78872646a7b3482ba044dc8
* Image API v2: make volume_type optionalNicolas Simonds2016-03-181-1/+4
| | | | | | | | It is perfectly valid for a Cinder volume to have a volume_type of "none", so don't make the Image API v2 require that one be set. Change-Id: I1a6da8d791fa0ae67cac46eec81bcbcb420729c3 Closes-Bug: #1559344
* Fix return value of "image set" commandTang Chen2016-02-291-1/+1
| | | | | | | "image set" command should return None. But in one path, it returns ({}, {}). This patch fixes this. Change-Id: I3847e661cb7e89863921a3f0a859d9b1a8077ede
* Fixed a bunch of spacingBrandon Palm2016-02-231-4/+4
| | | | | | | Nothing too complicated here. I fixed a bunch of spacing issues that I saw in OSC. Change-Id: I935ab48e7c5bac5f88ecdb3a05f73fb44fc9f41d
* Add --marker option to "image list" commandTang Chen2016-02-031-0/+11
| | | | | | | | Users could specify the last image (name or ID) of the previous page with --marker option to control the start image of the output. Change-Id: Idca0235ee83b1226b00c89cf3d38500fa898b7d0 Closes-Bug: #1540988
* Add limit option to "image list" commandxiexs2016-02-031-0/+8
| | | | | | | | This option is quite useful if there are too many images. Change-Id: If6a901c27c5da2d1f4412e8fa9ba3bed3b72fdd9 Co-Authored-By: Tang Chen <chen.tang@easystack.cn> Partial-Bug: #1540988
* Remove marker and loop from "image list" commandTang Chen2016-02-031-10/+1
| | | | | | | | | | | | Since --page-size has never worked, there is no paginate logic needs to be implemented in "image list" command. So remove the unnecessary loop. And also, the marker is not necessary because --marker option has not been implemented. Will add it back when implementing --marker option. Change-Id: I71fea1502f92f447a49697edb52e8e82f336772f Partial-Bug: #1540988
* log take_action parameters in a single placeAkihiro Motoki2016-02-022-66/+9
| | | | | | | | | | | | Previously each command logs take_action parameters explicitly by using @utils.log_method decorator or log.debug(). Some commands have no logging. This commit calls a logger in the base class and drops all logging definition from individual commands. Closes-Bug: #1532294 Change-Id: I43cd0290a4353c68c075bade9571c940733da1be
* Merge "Initialize activation status"Jenkins2016-01-211-0/+1
|\
| * Initialize activation statusTerry Howe2016-01-191-0/+1
| | | | | | | | | | | | | | activation_status throws an exception if the image set fails because it is not initialized. Change-Id: Iff6d5a8844eed954fb1c0bcea96118b99ac6bcf4
* | Change --owner to --project in image commandsDean Troyer2016-01-152-32/+117
|/ | | | | | | | | | * image create and image set now use --project to specify an alternate project to own the image * --owner is still silently accepted but deprecated, add warning messages * --project and --owner are mutually exclusive to prevent precedence issues Closes Bug: 1527833 Change-Id: Iccb1a1d9175ef9b5edcd79d294607db12641c1f0
* Merge "Trivial: Remove useless return from files in image and volume"Jenkins2015-12-281-2/+0
|\
| * Trivial: Remove useless return from files in image and volumeTang Chen2015-12-181-2/+0
| | | | | | Change-Id: I3526ecd202d0908d91305a066ad72d03cee794b5
* | Add owner validation for "openstack image create/set"xiexs2015-12-211-0/+18
| | | | | | | | | | | | | | | | Owner validation is necessary if a new image owner will be created/set. Change-Id: I621774e02866bfa98a31b613deff5d7b6a962737 Closes-Bug: #1517134
* | Add image re/deactivate commandsNiallBunting2015-12-161-2/+30
|/ | | | | | | | | | | | | | | | This change allows admins to deactivate and reactivate their images. Currently this has to be done with the REST api or the glanceclient. This change introduces `--deactivate` and `--activate` for the `image set` command. This requires glanceclient 1.2.0. Which got bumped here: https://review.openstack.org/#/c/257512/ Change-Id: I476c44a0343cdc92d58ddc93fb06470242de2345 Depends-On: I2c370c6bf6ff664d94d756cc76aaa983fbdb8869 Closes-Bug: 1516661
* Merge "Add --volume to Image `create`"Jenkins2015-12-041-5/+41
|\
| * Add --volume to Image `create`NiallBunting2015-11-201-5/+41
| | | | | | | | | | | | | | | | | | | | This was previously part of the command, so added it in for backwards compatibility. This adds the --volume command and the complimentary --force command allowing users to create images from volumes. It seems it may not be possible to add it to image set v2. Change-Id: Ica36e70989f75d80959af3227f66708758fae68d
* | Add status column for "openstack image list"xiexs2015-11-292-3/+8
| | | | | | | | | | | | | | | | | | Actually, the status column is useful for the user. So, it`s better to output this info by default (or, user had to specify the extra option, i.e. --long). Change-Id: Id2a9f86f0de5310f8f5ff9a46bf1b7411094b519 Closes-Bug: #1519181
* | Merge "Add --owner to `image create`"Jenkins2015-11-121-3/+7
|\ \
| * | Add --owner to `image create`NiallBunting2015-11-061-3/+7
| |/ | | | | | | | | | | | | This adds --owner to `image create`. This is backwards compatable with v1. Change-Id: I9e79cf880c91a1386419db729818d23dfe632179 Depends-On: I8d572a070bbb04dccdd051b8e0ad199c5754746e
* | Fix issue when displaying image_memberlin-hua-cheng2015-10-231-1/+1
|/ | | | | | | | image_member doesn't have a _info attribute, glanceclient returns warlock object instead of a Resource object. Change-Id: If6e7c4bd404454bd6cbe8c111879c1afa1380211 Closes-Bug: #1509054
* Merge "image set should not show the resource"Jenkins2015-10-082-11/+4
|\
| * image set should not show the resourceSteve Martinelli2015-10-062-11/+4
| | | | | | | | | | | | | | | | the rest of OSC set commands do not show the resource after it has been updated. unless the update fails then we report back a failure, otherwise the user should assume everything went fine. Change-Id: I2bd4188450c3853b4a1bc25f80fc9450cda32bdd
* | Merge "Add tags to `image set`"Jenkins2015-10-071-4/+15
|\ \ | |/
| * Add tags to `image set`NiallBunting2015-10-061-4/+15
| | | | | | | | | | | | | | | | | | | | | | | | This adds --tag to the v2 version of `image set`. This is another step to compatability between the osc image api. Added merge of tags into existing tags and handling duplicates, and tests for same. Co-Authored-By: Steve Martinelli <stevemar@ca.ibm.com> Change-Id: Ie800fcbf8bbc0978c54ace3278750a18023e8ce4
* | Merge "Clean up Image v2 image set command"Jenkins2015-10-061-90/+90
|\ \ | |/
| * Clean up Image v2 image set commandDean Troyer2015-10-061-90/+90
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make the Image v2 image set command meet at the intersection of the v1 image set command and the v2 image create command: * Add visibility to the deadopts list and remove the option * Put the options in the same order as v1 image set * Make the help text match * Add --properties * Move the additional options that do not appear in either v1 image set or v2 image create after --property as they are really pre-defined properties * Add tests for v2 image set to match v1 and then some * Put the SetImage class in v2/image.py in alphabetical order Change-Id: I102b914e8ad09a014f6fdd846c5766b6c2eaadb8
* | Merge "Image fix bug with --volume"Jenkins2015-10-061-12/+14
|\ \ | |/ |/|
| * Image fix bug with --volumeNiallBunting2015-09-181-12/+14
| | | | | | | | | | | | | | | | | | | | Currently after calling the cinderclient to create an image from a volume, it also then tries to create another image. This fails as the keyword volume is unexpected. This add checks so the image is not created in this case. Allowing --volume to not throw an error when it has worked. Change-Id: I67e650eb0b8c331d86515e3e326c39a5d6dad5e1 Closes-Bug: 1497221
* | Merge "Glance `image set` Resolve Fracturing"Jenkins2015-09-231-6/+58
|\ \
| * | Glance `image set` Resolve FracturingNiallBunting2015-09-231-6/+59
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently `image set` uses the new api, where other parts of osc the old api is used. This deprecates the v2 api in favour of the v1 to maintain the same commands across osc. However the functionality now remains there as people could now be using this functionality. This also adds the --unprotected argument, as in the previous version if --protected was not supplied it would just make the argument --unprotected without the users explicit consent. The patch also fixes the documentation for image set as it was outdated. Change-Id: I990d20332c80165102badef7ac94ddbeb7824950 Closes-Bug: 1498092
* | Merge "Use format_list instead of format_dict when listing images"Jenkins2015-09-211-1/+1
|\ \
View Lorry Controller Status