summaryrefslogtreecommitdiff
path: root/openstackclient/compute
Commit message (Collapse)AuthorAgeFilesLines
* Refactor security group delete to use SDKRichard Theis2016-02-021-22/+0
| | | | | | | | | | | | | | | | | | | | | | | Refactored the 'os security group delete' command to use the SDK when neutron is enabled, but continue to use the nova client when nova network is enabled. This patch set introduces a new NetworkAndComputeCommand class to be used for commands that must support neutron and nova network. The new class allows both the parser and actions to be unique. The current DeleteSecurityGroup class is now a subclass of this new class and has moved under the network v2 commands. This patch set also introduces a new FakeSecurityGroup class for testing security groups. And finally, this patch set updates the command documentation for security group and security group rule to indicate that Network v2 is also used. Change-Id: Ic21376b86b40cc6d97f360f3760ba5beed154537 Partial-Bug: #1519511 Related-to: blueprint neutron-client
* log take_action parameters in a single placeAkihiro Motoki2016-02-0215-261/+43
| | | | | | | | | | | | 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 "Refactor "os availability zone list""Jenkins2016-01-221-102/+0
|\
| * Refactor "os availability zone list"Richard Theis2016-01-141-102/+0
| | | | | | | | | | | | | | | | | | | | | | | | Refactor the "os availability zone list" command to make it a common command instead of a compute-only command. Since availability zones are common to compute, volume and network (new), this refactoring allows availability zone support to be added for volume and network. In addition to the refactor, unit and functional tests were added. Change-Id: I63e9d41d229b21cd38e5a083493042c096d65e05 Partial-Bug: #1532945
* | 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
* | Set up logger of each command by metaclassAkihiro Motoki2016-01-201-22/+6
| | | | | | | | | | | | | | compute.v2.flavor is changed in this commit as an initial example. Partial-Bug: #1532294 Change-Id: I262af6ade0ae03fbe1cd2ad198faf4ebb4ecf7ce
* | 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>
* | | Merge "Refactor network endpoint enablement checking"Jenkins2016-01-141-8/+2
|\ \ \ | |_|/ |/| |
| * | 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
* | | Further improve output for "os security group show"Richard Theis2016-01-061-1/+19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Improve the security group rules output when running the "os security group show" command. Empty and duplicate information for each security group rule is now removed. This will ensure that the rules remain readable when direction and ethertype information is returned as part of the transition to neutron networking. Change-Id: Ib49c27a9d7f4d5d38ceb2b0d785ddf94d88b2d89 Partial-Bug: #1519511 Related-To: blueprint neutron-client
* | | Merge "Add support to list all security group rules"Jenkins2016-01-041-11/+21
|\ \ \ | |/ / |/| |
| * | Add support to list all security group rulesRichard Theis2015-12-171-11/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Both nova and neutron allow security group rules to be listed without specifying the owning security group. This patch set makes the group argument on 'os security group rule list' optional. Behavior is unchanged when the argument is specified. When the argument is not specified then all accessible security group rules will be listed. The listing will include the owning security group for each rule. Change-Id: I6914baecf70a65354e1e82dad92c6afbd32b4973 Related-Bug: #1519512
* | | Docstring should say 'default' if option is defaultRushi Agrawal2015-12-311-1/+1
| | | | | | | | | | | | Change-Id: I59d83e7a0a590fa537f509f7f19940640f555679
* | | Improve output for "os security group show"Richard Theis2015-12-231-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Improve the security group rules output when running the "os security group show" command. Each security group rule is now displayed on a separate line. Current output example: $ openstack security group show default +-------------+------------------------- ... ---+ | Field | Value ... | +-------------+------------------------- ... ---+ | description | Default security group ... | | id | 048a5fc3-3be1-407d-ae47-9... | | name | default ... | | project_id | 3b96bb2020c1459da76963f9e... | | rules | [u"id='5d812367-9829-4340...t"] | +-------------+------------------------- ... ---+ New output example: +-------------+------------------------- ... ---+ | Field | Value ... | +-------------+------------------------- ... ---+ | description | Default security group ... | | id | 048a5fc3-3be1-407d-ae47-9... | | name | default ... | | project_id | 3b96bb2020c1459da76963f9e... | | rules | id='5d812367-9829-4340-95...lt' | | | id='ee451d1c-ade3-4975-8e...lt' | +-------------+------------------------- ... ---+ Change-Id: I1386075310896c58a2b776e2bbec3603bd00eff1 Partial-Bug: #1519511 Related-To: blueprint neutron-client
* | | Merge "Remove list output from "compute service set""Jenkins2015-12-181-11/+2
|\ \ \
| * | | Remove list output from "compute service set"xiexs2015-12-011-11/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Replace the super class from lister.Lister to command.Command, as no need to display the status for the "set" command. Change-Id: Ibaf3c1e349633223fca19937bbd9060d4f9ecbda Closes-Bug: #1517804
* | | | Merge "Trivial: Remove useless return from files in compute."Jenkins2015-12-1610-15/+0
|\ \ \ \ | |_|/ / |/| | |
| * | | Trivial: Remove useless return from files in compute.Tang Chen2015-12-1710-15/+0
| | |/ | |/| | | | | | | Change-Id: I9dc6749256fcd53d292d7f658912c032e9ce9df5
* | | Remote security group name not displayed for ruleRichard Theis2015-12-141-0/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The 'security group rule list' command was updated to display the remote security group name for a security group rule. This was done via a new 'Remote Security Group' column. The output of the 'security group rule create' and 'security group show' commands was also updated to include 'remote_security_group' information instead of the raw 'group' information returned from the API layer. Change-Id: I5f9600338c8331966d2c658109a24b502c538106 Closes-Bug: #1520003
* | | Merge "Add source security group support to create rule"Jenkins2015-12-111-2/+10
|\ \ \ | |/ / |/| |
| * | Add source security group support to create ruleRichard Theis2015-12-071-2/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The 'security group rule create' command was updated to support a source security group. Now either a source IP address block or source security group can be specified when creating a rule. The default remains the same. Change-Id: If57de2871810caddeeaee96482eb34146968e173 Closes-Bug: #1522969
* | | 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
* | | Merge "Support "server list" searching by both image name and ID."Jenkins2015-12-031-2/+9
|\ \ \
| * | | 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
* | | | Merge "Support "server list" searching by both flavor name and ID."Jenkins2015-12-031-2/+9
|\ \ \ \ | |/ / / | | / / | |/ / |/| |
| * | 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
* | Switch to ksa SessionDean Troyer2015-12-021-2/+2
|/ | | | | | | | * Change session imports to keystoneauth1 * Change keystoneclient.exception imports to keystoneauth1 * Change exceptions raised from internal API from keystoneclient to openstack.common Change-Id: I046d89f561d6fe04baae53726f9749d2e7fe2056
* 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
* Refactor: Order of security group class namesRichard Theis2015-11-251-123/+123
| | | | | | | | | Per comment in [1], refactor the security group class names to be in alphabetical order. [1] https://review.openstack.org/#/c/249223 Change-Id: If28a153cdab57c0659ff5c78b276766d4043467f
* 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
* | Merge "Doc: Update and add IP address"Jenkins2015-11-243-16/+16
|\ \
| * | Doc: Update and add IP addressRichard Theis2015-11-243-16/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix the 'ip floating' command documentation file name. Add missing command list documentation for the 'ip fixed' and 'ip floating pool' commands. In addition, update the command description and argument help to fix minor issues and use consistent terminology. And finally, cleanup the Network command object formatting. Change-Id: I0168891bc674c9f0d4df4062f7e93f497dc8fe6f
* | | Merge "Doc: Add security group and security group rule"Jenkins2015-11-241-10/+10
|\ \ \ | |/ / |/| |
| * | Doc: Add security group and security group ruleRichard Theis2015-11-241-10/+10
| |/ | | | | | | | | | | | | | | | | Add missing command list documentation for the 'security group' and 'security group rule' commands. In addition, update the command description and argument help to fix minor issues and use consistent terminology. Change-Id: I9f4a3fbac5637289f19511874e16391d3fe27132
* | 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
* Remove deprecated 'project usage list' commandDean Troyer2015-11-181-17/+0
| | | | | | | | | | Remove the 'project usage list' command that was deprecated in version 1.0.2 in review Ie08d4f88d71a660fca1862405351109cd0aa86b6. Note that the removed command class is a good example of how to wrap a command and show a deprecation message. Change-Id: I6c750730963615895f5d9953487d2d5a905885a8
* 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
|\ \ | |/
View Lorry Controller Status