summaryrefslogtreecommitdiff
path: root/openstackclient/compute/v2/server.py
Commit message (Collapse)AuthorAgeFilesLines
* Add newline to strings in stdout/stderr.write()Lu lei2016-06-061-1/+1
| | | | | | Function stdout/stderr.write() can't break line automatically. Change-Id: I903c2d1cc1a669adb6be5aa4eb783d3b9943e685
* Merge "Fix output and error log in server.py"Jenkins2016-05-311-9/+17
|\
| * Fix output and error log in server.pyTang Chen2016-05-281-9/+17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch fixes 2 problems: 1. The '\n' should be in the end of a message, not the beginning. i.e.: The original code was: sys.stdout.write(_('\nError deleting server')) It will make the output look like this: [root@tangchen /]# openstack server delete aaa bbb Error deleting server Error deleting server[root@tangchen /]# We change it to: sys.stdout.write(_('Error deleting server\n')) Then the output will become: [root@tangchen /]# openstack server delete aaa bbb Error deleting server Error deleting server [root@tangchen /]# which is much better. 2. Record the error in log for those who didn't. Change-Id: I38b00c2321014757970183205f95f026e20a8090
* | Support to set server stateRui Chen2016-05-281-0/+9
|/ | | | | | | | | | When a server is unexpected state, OSC don't support reset the server to active or error state, that's supported by novaclient, and it's an important command for operators, the patch implement this function. Change-Id: I3e7800feb192832b0719ef9a353945beb6bfd509 Implements: blueprint server-reset-state
* Fix i18n support for help and error messages in computeTang Chen2016-05-241-14/+23
| | | | | Change-Id: Id6eebcb48d1b7b49b6636524506294edbc44a83f Partial-bug: #1574965
* remove #noqa from i18n importsSteve Martinelli2016-05-131-1/+1
| | | | | | hacking checks no longer fail on `import _` Change-Id: Idd60f0a0e71e5081691eacb39e5091ab08fcce6d
* Merge "Use find_resource() instead of get() in _prep_server_detail()"Jenkins2016-05-111-4/+1
|\
| * Use find_resource() instead of get() in _prep_server_detail()Tang Chen2016-05-091-4/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | There is such a comment in test_server.py: # Call .get() to retrieve all of the server information # as findall(name=blah) and REST /details are not the same # and do not return flavor and image information. This is an out of date comment. There is no function named findall() in OSC now. So use find_resource() instead of get(), and remove this comment. Change-Id: I8d79afc5f341fb5caf771d905506b7f1c7a41ae8
* | Map server power state num to meanful stringTang Chen2016-05-071-0/+5
|/ | | | | | | | | | | | | In _prep_server_detail(), power_state is not formatted by _format_servers_list_power_state(). So when executing "server show" or "server create", the power state is represented by number. This patch map the numbers to meanful strings. This patch also adds power_state attribute to FakeServer, and improves unit tests for this attribute. Change-Id: I2ec674327de4e5133b8712ba6bb53fa5ce55e3f4
* Doc: Unify repeatable option commentsTang Chen2016-04-121-4/+4
| | | | | | | | | | | | There are lots of "this option can be repeated" comments in the doc, which are not consistent to other similar docs. This patch changes them to the following format: "repeat option to do something" Change-Id: I54e01053091c428bf87bb36bb95f73a0b80ab6e7
* Fix "server unset" document issueRui Chen2016-03-231-1/+1
| | | | | | | Fix "--property" option describe issue in "server unset" document, and update the help message to keep consistent. Change-Id: I68022a187e83fad6320365400ad2a1b0c8cf9a61
* Trivial: Use 'SSH' rather than 'Ssh'Stephen Finucane2016-03-091-1/+1
| | | | | | This is reflected in the '--help' screen. Change-Id: Ic22a65ff6a56b069b37a0ea8365cce2b3f93621c
* Merge "" openstack server image create " doesn't print proper info"Jenkins2016-03-061-5/+19
|\
| * " openstack server image create " doesn't print proper infoMohan Muppidi2016-03-031-5/+19
| | | | | | | | | | | | | | | | | | | | | | After creating a snapshot of a running instance, a print out similar to server create is expected, but it prints out something like "_info" which is nothing related to created image. _prep_image_detail method is added to /compute/v2/server.py to enable the priting, while running the test properly. Change-Id: I4b06be959768bcdaafd9aa8df497490958bee649 Closes-Bug:1551586
* | Merge "[compute] Support restore server"Jenkins2016-03-041-0/+22
|\ \
| * | [compute] Support restore serverjichenjc2016-02-231-0/+22
| | | | | | | | | | | | | | | | | | Server in soft-delete state can be restored, add this command. Change-Id: Id9d7246f89ae65273505f36dcb664996534ae986
* | | Merge "Add test cases to test some commands with '--wait' and fix bug"Jenkins2016-03-041-2/+2
|\ \ \ | |_|/ |/| |
| * | Add test cases to test some commands with '--wait' and fix bugting.wang2016-03-021-2/+2
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | "server image create" "server rebuild" "server resize" Above 3 commands are not covered by unit test. So add some unit tests. Meanwhile, fix bug. Now that image name is an optional argument, we'd better record error messages with positional arguments instead of optional argument. So, record server name. Change-Id: I41bc025d4824dc46f63a3213d82e1528bacbbe12
* | Defaults are ignored with flake8Tom Cocozzello2016-02-231-1/+1
|/ | | | | | | | | | | | If “ignore” is not set under flake8 in the tox.ini file there there are defaults set to be ignored. The depended patch fixes many of the problems. Change-Id: Ieed2fe1c4654e201d3fe6d40ef93e247ee736f8b Doc: http://flake8.readthedocs.org/en/latest/config.html#default Depends-On: I935ab48e7c5bac5f88ecdb3a05f73fb44fc9f41d Closes-Bug: #1548910
* Add support for triggering an crash dumpxiexs2016-02-051-0/+28
| | | | | | | | | | | | | The triggering crash dump feature is supported by nova [1] and novaclient [2] now, it's time to introduce this feature into OSC correspondingly. [1]The change id is: I6ed777ff637254b4b79417008f9055dd19fc7405 [2]The change id is: If03b1864bbe7074c720b946fc2700bd5d07debc3 Change-Id: I5a411f283fdf0fc3c00380d069848a332c799cdd Closes-Bug: #1538372 Co-Authored-By: Tang Chen <chen.tang@easystack.cn>
* log take_action parameters in a single placeAkihiro Motoki2016-02-021-32/+0
| | | | | | | | | | | | 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
* log_method: get logger from decorated method if unspecifiedAkihiro Motoki2016-01-201-90/+31
| | | | | | | | | | | | | This commit makes 'log' optional. 'log' attribute of each command class does not exist when the class is defined because 'log' is now setup dynamically when a class is instantiated. Instead log_method looks for a logger from a decorating method. compute.v2.server is changed in this commit as an example. Change-Id: Ic4d128f8e027d3b8e6f884f31369e9085c0f0871 Partial-Bug: #1532294
* Merge "Make --image parameter optional in "server rebuild""Jenkins2016-01-151-5/+6
|\
| * Make --image parameter optional in "server rebuild"David Moreau Simard2015-12-111-5/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | The command will now default to the image currently in-use by the server, effectively making the --image parameter optional. This commit also adds basic tests for ServerRebuild since there wasn't any. Will add more full tests for it. Change-Id: I733fd3ad5a825f06563c72aa430122e1a0e3b3b0 Closes-bug: #1524406 Co-Authored-By: David Moreau Simard <dms@redhat.com> Co-Authored-By: Tang Chen <tangchen@cn.fujitsu.com>
* | Refactor network endpoint enablement checkingRichard Theis2016-01-041-8/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Move the network endpoint enablement checking from the 'server create' command to the common client manager. This allows future network commands to use either nova or neutron networking based on the cloud environment. This patch set also includes related unit test enhancements to the common client manager to trigger authentication on the tests. Change-Id: Ia37e81d4fb05a1e2fceb3e5d367bda769ab8e64b Related-Bug: #1519511 Related-to: blueprint neutron-client
* | Trivial: Remove useless return from files in compute.Tang Chen2015-12-171-2/+0
|/ | | | Change-Id: I9dc6749256fcd53d292d7f658912c032e9ce9df5
* SDK integration extensions and server create networksTerry Howe2015-12-091-13/+6
| | | | | | | Finish up the SDK integration with server create network and port find and extension list. Change-Id: I18dbada784d8aa92a45a937f251023ddf899c53e
* Merge "Trivial: Import network.common as network_common in server.py"Jenkins2015-12-051-9/+13
|\
| * Trivial: Import network.common as network_common in server.pyTang Chen2015-12-061-9/+13
| | | | | | | | | | | | | | | | In server.py, identity.common is imported as identity_common. But network.common is imported as common, which is confuseing. This patch imports network.common as network_common. Change-Id: I74295bc88b22de398ab64fe556aedaca2453d17d
* | Merge "Trivial: Improve doc for "server create" command"Jenkins2015-12-051-4/+4
|\ \ | |/ |/|
| * Trivial: Improve doc for "server create" commandTang Chen2015-12-041-4/+4
| | | | | | | | | | | | | | | | | | The following options of "server create" command support searching by both name and ID. So add this info into doc. --image, --volume, --flavor, --security. Change-Id: I93b167da58144e5de6c9996009b7ea2449fb4cd8
* | Support "server list" searching by both image name and ID.Tang Chen2015-12-011-2/+9
| | | | | | | | | | | | | | | | | | | | | | | | Nova API only supports list servers searching by image ID. In OSC, we can support both image name and ID by mapping the name to ID. This patch also fix the inconsistent doc in .py and .rst files. Closes-Bug: 1521492 Change-Id: I70613843f82d74732bd32a457cd4a31aba57825f
* | Support "server list" searching by both flavor name and ID.Tang Chen2015-12-011-2/+9
|/ | | | | | | | | | | | Nova API only supports list servers searching by flavor ID. In OSC, we can support both flavor name and ID by mapping the name to ID. This patch also fix the inconsistent doc in .py and .rst files. Partial-Bug: 1521492 Change-Id: I1d1a6aa91aef4e2846745babe8382481185fa96e
* Merge "Add --limit option to "server list" command."Jenkins2015-12-011-1/+12
|\
| * Add --limit option to "server list" command.Tang Chen2015-12-011-1/+12
| | | | | | | | | | | | This option will limit the total amount of items the command will list up. Change-Id: I46af0d479d795ebb6a74585d0f76629dd940b117
* | Merge "Add --marker option to "server list" command."Jenkins2015-11-301-1/+16
|\ \ | |/
| * Add --marker option to "server list" command.Tang Chen2015-12-011-1/+16
| | | | | | | | | | | | | | This option will give user a chance to display the server list from wherever they want. Change-Id: I92cca5e98cd473f1113a9106eb9d1f490694b1fe
* | Add "openstack server unshelve" into OSCxiexs2015-11-281-25/+50
|/ | | | | | | | The unshelve operation is not supported by OSC, and this patch tries to add it. Change-Id: Ic60a4616cb63ad21c1a3c8e02611da8bad3a8bd0 Implements: blueprint introduce-shelve-into-osc
* Merge "Add "openstack server shelve" into OSC"Jenkins2015-11-241-0/+25
|\
| * Add "openstack server shelve" into OSCxiexs2015-11-241-0/+25
| | | | | | | | | | | | | | | | Currently, the shelve operation is not supported by OSC. So, this patch attempts to add it into OSC. Change-Id: I92545300bef006a069338168d2de800e8a58af69 Implements: blueprint introduce-shelve-into-osc
* | Enable "openstack server resume" command to take multiple servers.Tang Chen2015-11-241-6/+8
| | | | | | | | | | | | | | | | | | Current "openstack server resume" command could only resume one server. Improve it to be able to handle more than one servers. Also improve the doc to reflect the new feature. Change-Id: I726eb86bfa3df3a9911f45770e6641264dbc1e0b Implements: blueprint cmd-with-multi-servers
* | Enable "openstack server suspend" command to take multiple servers.Tang Chen2015-11-241-6/+8
|/ | | | | | | | | Current "openstack server suspend" command could only suspend one server. Improve it to be able to handle more than one servers. Also improve the doc to reflect the new feature. Change-Id: Ic0417ee28f46c9198a35744c0180342e61966b26 Implements: blueprint cmd-with-multi-servers
* Merge "Enable "openstack server unlock" command to take multiple servers."Jenkins2015-11-171-6/+8
|\
| * Enable "openstack server unlock" command to take multiple servers.Tang Chen2015-11-131-6/+8
| | | | | | | | | | | | | | | | | | Current "openstack server unlock" command could only unlock one server. Improve it to be able to handle more than one servers. Also improve the doc to reflect the new feature. Change-Id: Ibf57b2021a504da950a491d63139a438087aed0b Implements: blueprint cmd-with-multi-servers
* | Merge "Enable "openstack server lock" command to take multiple servers."Jenkins2015-11-161-6/+8
|\ \ | |/
| * Enable "openstack server lock" command to take multiple servers.Tang Chen2015-11-131-6/+8
| | | | | | | | | | | | | | | | | | Current "openstack server lock" command could only lock one server. Improve it to be able to handle more than one servers. Also improve the doc to reflect the new feature. Change-Id: Ifcf103b1c32e6c547ac09f688b887b1c03f92b09 Implements: blueprint cmd-with-multi-servers
* | Merge "Enable "openstack server unpause" command to take multiple servers."Jenkins2015-11-161-6/+8
|\ \ | |/
| * Enable "openstack server unpause" command to take multiple servers.Tang Chen2015-11-131-6/+8
| | | | | | | | | | | | | | | | | | Current "openstack server unpause" command could only unpause one server. Improve it to be able to handle more than one servers. Also improve the doc to reflect the new feature. Change-Id: I069ebdd6dcd121f6e55c2bf40d42197f93830e0c Implements: blueprint cmd-with-multi-servers
* | Merge "Improve "server list" command to have the same output as "nova list"."Jenkins2015-11-131-2/+43
|\ \
| * | Improve "server list" command to have the same output as "nova list".Tang Chen2015-11-141-2/+43
| |/ | | | | | | | | | | | | | | | | | | | | "nova list" will also output "Task State" and "Power State" by default. This patch improves "server list" command to have the same columns, but not by default. These two columns will be output if --long is added. The power state is an int, so also adds a formatter helper function to translate it to human readable string, just as "Networks" does. Change-Id: I0530a910bec03835839a5ba7687c66d5643338f3