summaryrefslogtreecommitdiff
path: root/openstackclient
Commit message (Collapse)AuthorAgeFilesLines
* Add 'description' option to os subnet (pool) create/set cmdCao Xuan Hoang2016-09-205-0/+138
| | | | | | | | | This patch adds '--description' option to the commands. Change-Id: Ifc2828670c3c48a87a0493d98686a5babf9b2ae7 Closes-Bug: #1614458 Closes-Bug: #1614823 Partially-Implements: blueprint network-commands-options
* Merge "Doc, help and message updates for port unset"Jenkins2016-09-171-33/+24
|\
| * Doc, help and message updates for port unsetRichard Theis2016-09-141-33/+24
| | | | | | | | | | | | | | | | | | Update port unset based on review comments in [1]. [1] https://review.openstack.org/#/c/306236/ Change-Id: Icae6e3869516f2d53f90ad46efc03f897c00b0e5 Implements: blueprint network-property-unset
* | Correct login name deduction in SshServerCedric Brandily2016-09-151-1/+1
| | | | | | | | | | | | | | | | | | Currently SshServer ("server ssh" command) raises an AttributeError if no explicit login is provided because it uses an attribute which no more exists. Change-Id: I86f68230037d51efb41aca62e07e058733ecd67a Closes-Bug: #1624085
* | Merge "Support mark volume as bootable in volume set"Jenkins2016-09-146-1/+111
|\ \
| * | Support mark volume as bootable in volume setqtang2016-09-146-1/+111
| | | | | | | | | | | | | | | | | | | | | | | | Add --bootable | --non-bootable option in volume set to mark or unmark volume as bootable. Change-Id: Ifa6c2dd1642202f55b6d50e3b8614d3513d488f6 Closes-Bug:#1535704
* | | Merge "Support listing specified server's ports"Jenkins2016-09-142-5/+38
|\ \ \
| * | | Support listing specified server's portsRui Chen2016-09-142-5/+38
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | Add new option "--server" for "port list" command to list all of the ports that are attached on the specified server. Change-Id: I8b5550ea5068405b163711303465b704b5207410 Closes-Bug: #1614385
* | | Merge "Multi REST API calls error handling of "volume unset" command"Jenkins2016-09-142-4/+45
|\ \ \
| * | | Multi REST API calls error handling of "volume unset" commandHuanxuan Ao2016-09-112-4/+45
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Support multi REST API calls error handling for "volume unset" command follow the rule in doc/source/command-errors.rst. Also add a unit test for testing the error handling Change-Id: I2de7a7bd5a7a5e39817ed5cf6952abf4afba75e4
* | | | Merge "Do not show "os-volume-type-access:is_public" property of volume type"Jenkins2016-09-142-3/+5
|\ \ \ \
| * | | | Do not show "os-volume-type-access:is_public" property of volume typeHuanxuan Ao2016-09-072-3/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "os-volume-type-access:is_public" property is the same as "is_public" property in volume type object. So stop showing "os-volume-type-access:is_public" property and leave "is_public" property only. Change-Id: Ic78a9ee69b0ab356edff18cdb4c46fc24b495d2b Closes-Bug: #1620922
* | | | | Replace 'MagicMock' with 'Mock'qtang2016-09-1429-63/+63
| |_|/ / |/| | | | | | | | | | | | | | | Change-Id: I7aeceede6bd3cb88cf04f398454f9758dbee20f1 Closes-Bug: #1475722
* | | | Support fetching network project default quotaRui Chen2016-09-133-2/+27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Neutron server and openstacksdk had supported to fetch network project default quota, this patch add the CLI support in openstackclient. Change-Id: If0ef74c268c41a866c62156da0603a40ae4e6e31 Closes-Bug: #1204956 Depends-On: I6a4e2a146351dd1e7d652442511f1ef2c279da42
* | | | Merge "Display hypervisor information without uptime"Jenkins2016-09-122-12/+87
|\ \ \ \
| * | | | Display hypervisor information without uptimeHironori Shiina2016-09-112-12/+87
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some virt drivers such as ironic virt driver doesn't implement a method to get host uptime. For such drivers, hypervisor show command displays no information although these drivers provides other host information. This patch fixes the command to display hypervisor information in case where a virt driver doesn't provide host uptime by ignoring a HTTPNotImplemented exception. Change-Id: I7bcca5862cd9c05aadaf6192cb80aa651cd77cad Closes-Bug: 1612065
* | | | | Merge "Fix regular expression for uptime in hypervisor show"Jenkins2016-09-122-2/+11
|\ \ \ \ \ | |/ / / /
| * | | | Fix regular expression for uptime in hypervisor showHironori Shiina2016-09-112-2/+11
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Hypervisor show command has a few bugs as follows. - It doesn't trim an extra whitespace in the head of uptime information. - It doesn't display uptime information when the number of user is 1. This patch fixes the regular expression to match uptime information. Change-Id: Ic2f7fd9a9274466717084a0886f95f78e98a9007 Closes-Bug: 1611809
* | | | Merge "Set quota "per_volume_gigabytes", "backup_gigabytes" and "backups""Jenkins2016-09-122-1/+33
|\ \ \ \
| * | | | Set quota "per_volume_gigabytes", "backup_gigabytes" and "backups"Rui Chen2016-09-122-1/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | "per_volume_gigabytes", "backup_gigabytes" and "backups" items can be shown in "openstack quota show" command, but can't be updated by "openstack quota set". This patch fix the issue. Change-Id: I47db5a69d4e4ef6e140f2735257c83e1fb052760 Closes-Bug: #1609767
* | | | | Merge "Add functional tests for unset multiple volume type props"Jenkins2016-09-122-0/+30
|\ \ \ \ \
| * | | | | Add functional tests for unset multiple volume type propsRui Chen2016-09-122-0/+30
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | CinderClient bug/1596511 cause that OSC can't unset multiple properties of volume type, obviously no functional test case cover the part of logic in OSC, that make the issue raising, the patch add functional test cases for this scenario. Change-Id: I42cf9ac8cc72ccc2f1208926d8faf7b80ee2d288 Partial-Bug: #1596511 Depends-On: I60378a32cdc52aacdf869d69b246dec7eb6cdb77
* | | | | Merge "Trivial: Rename ListTransferRequests to ListTransferRequest"Jenkins2016-09-124-6/+6
|\ \ \ \ \
| * | | | | Trivial: Rename ListTransferRequests to ListTransferRequestTang Chen2016-09-124-6/+6
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | The class name should use plural. Change-Id: I2de2489ff9aa60c2d1bf12743cbd41f2091739ca
* | | | | Merge "Unit tests of quota command refactor"Jenkins2016-09-122-44/+112
|\ \ \ \ \ | | |/ / / | |/| | |
| * | | | Unit tests of quota command refactorRui Chen2016-09-122-44/+112
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Do refactor for quota command related unit tests, remove useless code, add new tests to cover '--volume-type' and '--class' option in "quota set" command and add the volume quota fake object. Change-Id: Iaf214740e98db1bfb4c739e810bac1c5ba6e9625
* | | | Merge "remove duplicate unit test"Jenkins2016-09-121-286/+0
|\ \ \ \
| * | | | remove duplicate unit testSteve Martinelli2016-09-101-286/+0
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | looks like this test snuck in during the refactor: $ diff openstackclient/tests/volume/v1/test_service.py \ openstackclient/tests/unit/volume/v1/test_service.py 17c17 < from openstackclient.tests.volume.v1 import fakes as service_fakes --- > from openstackclient.tests.unit.volume.v1 import fakes as service_fakes Change-Id: I769b2d39f28cfaf65e4027c785b0ddbbefa26aea
* | | | Merge "Add unit test for volume type in volume v1"Jenkins2016-09-122-0/+421
|\ \ \ \ | |_|/ / |/| | |
| * | | Add unit test for volume type in volume v1Huanxuan Ao2016-09-112-0/+421
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add unit test for volume type commands (create/delete/show/list/set/unset) in volume v1 Change-Id: I6ff1f1c7482bd0b4bfec5b4a1496807b722fa047
* | | | Implement "volume transfer request delete" commandHuanxuan Ao2016-09-118-0/+430
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add "volume transfer request delete" command in volume v1 and v2. Also add the unit tests, docs, release note and functional tests Change-Id: Ic3d375bc8df3312fac53c1800d75f48376b8c91c Implements: bp cinder-command-support Co-Authored-By: Sheel Rana <ranasheel2000@gmail.com>
* | | | Implement "volume transfer request create" commandHuanxuan Ao2016-09-116-7/+213
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | Add "volume transfer request create" command in volume v1 and v2. Also add the unit tests, docs, release note and functional tests Change-Id: If362df1acf214efdf6ba129cd917d33eb54e1030 Implements: bp cinder-command-support Co-Authored-By: Sheel Rana <ranasheel2000@gmail.com>
* | | Merge "Error handling of multi REST API calls for "snapshot set" command"Jenkins2016-09-113-10/+94
|\ \ \
| * | | Error handling of multi REST API calls for "snapshot set" commandHuanxuan Ao2016-09-113-11/+100
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | Support multi REST API calls error handling for "snapshot set" command follow the rule in doc/source/command-errors.rst. Also add a unit test for testing the error handling Change-Id: I0c6214271bc54a25b051c0a62438c3344c8b51d7
* | | Merge "Add "--limit" and "--marker" options to "volume list" command"Jenkins2016-09-114-2/+140
|\ \ \
| * | | Add "--limit" and "--marker" options to "volume list" commandHuanxuan Ao2016-09-114-2/+140
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | Add ``--limit`` option to ``volume list`` command in volume v1, add ``--limit`` and ``--marker`` options to ``volume list`` command in volume v2. Change-Id: I327a252aa83ed84166da99cf6aa80334e0e6dd44 Partial-Bug: #1612484
* | | Merge "Support error handling for delete commands in volume v1"Jenkins2016-09-117-33/+411
|\ \ \
| * | | Support error handling for delete commands in volume v1Huanxuan Ao2016-09-117-33/+411
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | Some delete commands in volume v1 support multi delete but do not support error handling, this patch fixes them, and this patch also refactor (or add new) unit tests for some delete commands in volume v1. Change-Id: Ia8177698f8733cfe75ea0ff00eee8fdc0820f62e
* | | Merge "Add importing file to import warnings"Jenkins2016-09-117-7/+28
|\ \ \ | |/ / |/| |
| * | Add importing file to import warningsDean Troyer2016-08-307-7/+28
| | | | | | | | | | | | | | | | | | | | | | | | Add the Python file doing the import to the warnings emitted by the modules moved to osc-lib. Users will at least have a hint as to which package is out-of-date. Change-Id: I633b440c30b2b15cfde7a9013e30dfa39ab200bc
* | | unwedge the gateSteve Martinelli2016-09-091-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | since changing the infra job to call the new bash script location, the functional tests now have a permission error: py.error.EACCES: [Permission denied]: mkdir('/opt/stack/new/python-openstackclient/.tox',) Change-Id: Ibe7057c99feac952d80156fb996d0193e1b0e497
* | | move all functional tests to tests moduleSteve Martinelli2016-09-0979-0/+4653
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | functional tests should be grouped with other tests (unit and integration tests). as part of this commit the "common" module was renamed to just "base", this was done for simplicity. the post_test_hook.sh file was also copied to the functional module since it should live there. a separate change to the infra repo will be made to call the new location, once that is merged we can remove the old one (a new change will also be posted for that) Needed-By: I49d54f009021d65c1ae49faf6b3f0a7acdadd7b3 Change-Id: Ie8c334f6223373b8e06df8bd8466500d2a2c8ede
* | | move unit tests to new "unit" test moduleSteve Martinelli2016-09-08125-205/+491
| | | | | | | | | | | | | | | | | | | | | | | | this will better isolate the unit tests from the functional tests. unfortunately, the "integration" tests had to be lumped into the "unit" tests since we need the separation in testr.conf Change-Id: Ifd12198c1f90e4e3c951c73bfa1884ab300d8ded
* | | Merge "format token expires time to prevent json loading datetime data"Jenkins2016-09-086-9/+13
|\ \ \
| * | | format token expires time to prevent json loading datetime dataSteve Martinelli2016-09-056-9/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When output to shell, the token issue time is fine; however when selecting the json formatter (via the --format json) option, an exception is raised when formatting the dataetime data. Rather than pass in the datetime data, we should format the data with the ISO 8601 formatting. Closes-Bug: 1619937 Change-Id: Iffebb2d5413fabfd283dfa94fc560fc37270f9dd
* | | | Merge "Add Subnet service-types to subnets"Jenkins2016-09-083-4/+141
|\ \ \ \
| * | | | Add Subnet service-types to subnetsBrian Haley2016-08-193-4/+141
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add '--service-type' to subnet arguments to support Subnet service-types. Change-Id: I215d83e4d4cf53e03fa35041c5e41a328641b3a9 Partially-implements: blueprint service-subnets
* | | | | Merge "Defer auth prompting until it is actually needed"Jenkins2016-09-083-3/+81
|\ \ \ \ \ | |_|_|_|/ |/| | | |
| * | | | Defer auth prompting until it is actually neededDean Troyer2016-09-083-3/+81
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Auth option prompting happens waaaay to early in the default os-client-config flow, we need to defer it until adter the commands have been parsed. This is why ClientManager.setup_auth() exists, as it is not called until the first attempt to connect to a server occurs. Commands that do not require authentication never hit this. Also, required options were not being enforced. By doing this we handle when no authentication info is present, we fail on missing auth-url rather than attempt to prompt for a password (default auth is password). Closes-Bug: 1619274 Change-Id: Ia4eae350e6904c9eb2c8507d9b3429fe52418726
* | | | Merge "Clean up FakeClient classes in volume fakes"Jenkins2016-09-066-78/+10
|\ \ \ \
View Lorry Controller Status