summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | | | | 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-125-8/+8
|\ \ \ \ \ \ \
| * | | | | | | Trivial: Rename ListTransferRequests to ListTransferRequestTang Chen2016-09-125-8/+8
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-1111-1/+450
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-119-8/+241
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-116-2/+159
|\ \ \ \
| * | | | Add "--limit" and "--marker" options to "volume list" commandHuanxuan Ao2016-09-116-2/+159
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | Merge "move all functional tests to tests module"Jenkins2016-09-0982-95/+138
|\ \ \
| * | | move all functional tests to tests moduleSteve Martinelli2016-09-0982-95/+138
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | Merge "standardize release note page ordering"Jenkins2016-09-091-1/+1
|\ \ \ \ | |/ / / |/| | |
| * | | standardize release note page orderingDoug Hellmann2016-09-071-1/+1
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | In order to support automatically updating the release notes when we create stable branches, we want the pages to be in a standard order. This patch updates the order to be reverse chronological, so the most recent notes appear at the top. Change-Id: Ib364dcc8eb31275a31c83b68d7914263b183e393 Signed-off-by: Doug Hellmann <doug@doughellmann.com>
* | | move unit tests to new "unit" test moduleSteve Martinelli2016-09-08126-206/+492
| | | | | | | | | | | | | | | | | | | | | | | | 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-086-4/+185
|\ \ \ \
| * | | | Add Subnet service-types to subnetsBrian Haley2016-08-196-4/+185
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
|\ \ \ \ \
| * | | | | Clean up FakeClient classes in volume fakesHuanxuan Ao2016-08-236-78/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Put FakeTransferClient and FakeServiceClient classes into FakeVolumeClient class, I think we need not to create FakeClient classes for every objects, we can use only FakeVolumeClient for all volume objects. This can reduce repetition, unified codes and make codes look cleaner and easier. Change-Id: I2729ef0e80166f4d49ccd9a48b653e2f215e3bfe
* | | | | | Merge "better functional test collection"Jenkins2016-09-061-6/+29
|\ \ \ \ \ \
| * | | | | | better functional test collectionSteve Martinelli2016-09-051-6/+29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | straight up copied from glanceclient [1] [1] https://github.com/openstack/python-glanceclient/blob/master/glanceclient/tests/functional/hooks/post_test_hook.sh Change-Id: I9fad6d5c86831a2b872f3a61d4c7fa7383fc1266
* | | | | | | Merge "Use assertIn(A, B) instead of assertTrue(A in B )"Jenkins2016-09-062-8/+8
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | |
| * | | | | | Use assertIn(A, B) instead of assertTrue(A in B )Bin Zhou2016-09-022-8/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Developers should use assertIn(A, B) instead of assertTrue(A in B ). TrivialFix Change-Id: I0d7c542272315590536c427a8865e49660361e55
* | | | | | | Merge "Add "volume service set" command"Jenkins2016-09-067-0/+442
|\ \ \ \ \ \ \ | |_|_|_|_|/ / |/| | | | | |
| * | | | | | Add "volume service set" commandHuanxuan Ao2016-08-177-0/+442
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add "volume service set" command in volume v1 and v2 (v1 is the same as v2) to disable or enable volume service. Change-Id: Ibb2db7e93b24cb2e0d2a7c28b6fd8bcc851b8d2f Closes-Bug: #1613597
* | | | | | | Update reno for stable/newtonDoug Hellmann2016-09-022-0/+7
| |/ / / / / |/| | | | | | | | | | | | | | | | | Change-Id: Ice2d2eb4cc922e911c1f92020e8f5a298383f7f3
* | | | | | Updated from global requirementsOpenStack Proposal Bot2016-09-011-1/+1
| |_|/ / / |/| | | | | | | | | | | | | | Change-Id: I610346bca6a1a8ff9e0ab20fec664d3aff58c44c
* | | | | Merge "Rearrange integration tests"Jenkins2016-09-015-195/+411
|\ \ \ \ \
| * | | | | Rearrange integration testsDean Troyer2016-08-315-195/+411
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Move the integration tests into their final home... * Create tests/integ to hold the integration tests * Split tests/test_shell_integ.py into tests/integ/base.py and tests/integ/cli/test_shell.py * Rename TestXXXXInteg classes to TestIntegXXXX * Adds tests/integ/cli/test_project.py for some simple project argument tests Change-Id: I2cdd340d1d446d61784eae35dd5aa09d40d5899d
* | | | | | Updated from global requirementsOpenStack Proposal Bot2016-08-311-1/+1
|/ / / / / | | | | | | | | | | | | | | | Change-Id: I87af564cffaf44669e1efe4ca45c75a6307f1286
* | | | | Updated from global requirements3.2.0OpenStack Proposal Bot2016-08-301-1/+1
| |_|_|/ |/| | | | | | | | | | | Change-Id: I2eedd06457425f48353e416b4d5145ce1967d57c
* | | | Provide fallback prompt function for current osc-libDean Troyer2016-08-292-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Leaving the pw_func uninitialize in osc-lib turned out to be a bad idea as the test to prompt in setup_auth() doesn't check for a callback of None. Also, release note Change-Id: I8f875fa8a942d02a040238359ee22c603a4e5956
* | | | Merge "Fix auth prompt brokenness"Jenkins2016-08-294-3/+102
|\ \ \ \
| * | | | Fix auth prompt brokennessDean Troyer2016-08-294-3/+102
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We start by fixing this in the already-present OSC_Config class so OSC can move forward. This change needs to get ported down into os-client-config in the near future, maybe even soon enough to make the client library freeze this week. * Add the pw-func argument to the OSC_Config (or OpenStackConfig) __init__() * When looping through the auth options from the KSA plugin look for any that have a prompt defined and do not have a value already, so ask for one. Closes-bug: #1617384 Change-Id: Ic86d56b8a6844516292fb74513712b486fec4442
* | | | | Merge "Updated from global requirements"Jenkins2016-08-291-1/+1
|\ \ \ \ \
| * | | | | Updated from global requirementsOpenStack Proposal Bot2016-08-291-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I90afe332e959ba8bbfb1f04cc84454d30a4cf4a8
View Lorry Controller Status