summaryrefslogtreecommitdiff
path: root/openstackclient/tests
Commit message (Collapse)AuthorAgeFilesLines
* remove assert in favor an if/elseMichael McCune2016-05-021-12/+13
| | | | | | | | | | | | | | the assert usage in the NonNegativeAction has the potential to allow unexpected behavior when the python is byte-compiled with optimization turned on. Changes * remove assert in favor of if/else in NonNegativeAction class * add type specifier to parser arguments for non-negative actions * correct tests for new int based values Change-Id: I093e7440b8beff4f179e2c4ed81daff82704c40e Closes-Bug: #1576375
* Merge "Fixes BadRequest when no --pool-prefix given"Jenkins2016-05-021-1/+14
|\
| * Fixes BadRequest when no --pool-prefix givenInessa Vasilevskaya2016-04-271-1/+14
| | | | | | | | | | | | | | --pool-prefix is made required on subnetpool creation. Closes-bug: #1536479 Change-Id: I3d183e45e9b96bc08011c36f45ec2b7a9c01b627
* | Merge "Fix error in flavor set/unset command"Jenkins2016-04-281-4/+8
|\ \
| * | Fix error in flavor set/unset commandHuanxuan Ao2016-04-281-4/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In the "flavor set/unset" command,the "flavor" parameter can be a name but can not be a id of a flavor. I think we should find a flavor by using "utils.find_resource()" in these commands. Change-Id: I5836788f7ed18813f1ebde31bb808b7c3f932b80 Closes-Bug: #1575624
* | | Merge "Support for volume service list"Jenkins2016-04-284-0/+464
|\ \ \ | |/ / |/| |
| * | Support for volume service listSheel Rana2016-04-254-11/+475
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | OSC does not support to list volume services. This patch will provide support for adding volume service related support. Closes-bug:#1550999 Implements: bp cinder-command-support Change-Id: I50ac14aeb96c4b8ddbf7b33e519feea0d126f752
* | | Merge "Support quota show for current project"Jenkins2016-04-251-0/+11
|\ \ \ | |_|/ |/| |
| * | Support quota show for current projectRichard Theis2016-04-211-0/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The "os quota show" command "<project/class>" argument is now optional. If not specified, the user's current project is used. This allows non-admin users to show quotas for their current project. Change-Id: I602d4cc09c9d29ce84271eff78137f8810cb1a47 Closes-Bug: #1572733
* | | Merge "Add new share and default parms to subnet pool cmds"Jenkins2016-04-211-0/+86
|\ \ \ | |/ / |/| |
| * | Add new share and default parms to subnet pool cmdsBrad Behle2016-04-201-0/+86
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add the "share" and "default" parms to subnet pool create command. Add the "default" and "no-default" parms to subnet pool set command. Note that "share" can not be modified once subnet pool has been created, so do not add this to the set command. Change-Id: I1eecad69527a1cde7fb234669f4aff2be2db491e Partial-Bug: #1544591 Partial-Bug: #1544586
* | | Merge "Add "server group show" command"Jenkins2016-04-201-0/+23
|\ \ \
| * | | Add "server group show" commandRui Chen2016-04-201-0/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Support compute v2 "server group show" command in OSC. Implements: blueprint nova-server-group-support Closes-Bug: #1542171 Change-Id: I1a6d103151c704bda5b67cb9a49cc43c7d9f1d6a
* | | | Merge "Fix mutable default arguments in tests"Jenkins2016-04-205-56/+85
|\ \ \ \ | |/ / / |/| | |
| * | | Fix mutable default arguments in testsTang Chen2016-04-205-56/+85
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Python’s default arguments are evaluated only once when the function is defined, not each time the function is called. This means that if you use a mutable default argument (like list and dict) and mutate it, you will and have mutated that object for all future calls to the function as well. More details about this wrong usage here: http://docs.python-guide.org/en/latest/writing/gotchas/#mutable-default-arguments In unit tests, most FakeXXX classes' methods take mutable arguments with default values [] or {}. We should change them to None. Change-Id: Iea833b66aa1379829511ad5c6d4432b72f3488e2 Closed-bug: #1550320
* | | | Add "server group list" commandRui Chen2016-04-201-0/+68
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Support compute v2 "server group list" command in OSC. Implements: blueprint nova-server-group-support Partial-Bug: #1542171 Change-Id: I68b80e3c4458692472af671028cd1f939736bcb8
* | | | Add "server group delete" commandRui Chen2016-04-201-0/+84
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Support compute v2 "server group delete" command in OSC. Implements: blueprint nova-server-group-support Partial-Bug: #1542171 Related-Bug: #1563301 Change-Id: I7d792d669b147b24dc774844cfc9dbacd60d017b
* | | | Add "server group create" commandRui Chen2016-04-202-0/+142
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | Support compute v2 "server group create" command in OSC. Implements: blueprint nova-server-group-support Partial-Bug: #1542171 Change-Id: I96ffb07764d3adb715e048943cfee3b879c280f6
* | | Merge "Rename --profile to --os-profile"2.4.0Jenkins2016-04-191-1/+1
|\ \ \
| * | | Rename --profile to --os-profileDean Troyer2016-04-191-1/+1
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | * The --profile global option is deprecated but will be supported through at least April 2017. * Update man page Closes-bug: #1571812 Change-Id: I2e623411a56096b4cc352f4eedbf770632ae2cc3
* | | Merge "Add provider network options to osc network set"Jenkins2016-04-191-0/+9
|\ \ \
| * | | Add provider network options to osc network setreedip2016-04-191-0/+9
| |/ / | | | | | | | | | | | | | | | | | | | | | The following patch adds the provider network options to OSC "network set". Change-Id: I23b617077eda25d16164172a8e280082750eaf18 Partial-Bug: #1545537
* | | Merge "Add options to security group rule list"Jenkins2016-04-191-16/+83
|\ \ \ | |/ / |/| |
| * | Add options to security group rule listRichard Theis2016-04-181-16/+83
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add the following options to the 'os security group rule list' command: --long: Display direction and ethertype for Network v2 --all-projects: Display information from all projects for Compute v2 Change-Id: If8a1cbd7669cdfa6577d6d2f6fffd9e999a39a82 Partial-Bug: #1519512 Implements: blueprint neutron-client
* | | Merge "Remove methods argument from vloume/v2/fakes.py"Jenkins2016-04-181-9/+3
|\ \ \ | |/ / |/| |
| * | Remove methods argument from vloume/v2/fakes.pyTang Chen2016-04-181-9/+3
| | | | | | | | | | | | | | | | | | | | | methods argument in FakeAvailabilityZone class is not necessary. Remove it. Change-Id: Idf136bf90bd94e7045b0d471b8e03cd843693251
* | | Merge "Propagate AttributeErrors when lazily loading plugins"Jenkins2016-04-181-0/+8
|\ \ \ | |/ / |/| |
| * | Propagate AttributeErrors when lazily loading pluginsTim Burke2016-04-141-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, if an AttributeError was raised in a plugin's make_client method, the plugin simply wouldn't be an attribute of the ClientManager, producing tracebacks like Traceback (most recent call last): File ".../openstackclient/shell.py", line 118, in run ret_val = super(OpenStackShell, self).run(argv) ... File ".../openstackclient/object/v1/container.py", line 150, in take_action data = self.app.client_manager.object_store.container_list( File ".../openstackclient/common/clientmanager.py", line 66, in __getattr__ raise AttributeError(name) AttributeError: object_store This made writing minimal third-party auth plugins difficult, as it obliterated the original AttributeError. Now, AttributeErrors that are raised during plugin initialization will be re-raised as PluginAttributeErrors, and the original traceback will be preserved. This gives much more useful information to plugin developers, as in Traceback (most recent call last): File ".../openstackclient/shell.py", line 118, in run ret_val = super(OpenStackShell, self).run(argv) ... File ".../openstackclient/object/v1/container.py", line 150, in take_action data = self.app.client_manager.object_store.container_list( File ".../openstackclient/common/clientmanager.py", line 57, in __get__ err_val, err_tb) File ".../openstackclient/common/clientmanager.py", line 51, in __get__ self._handle = self.factory(instance) File ".../openstackclient/object/client.py", line 35, in make_client interface=instance._interface, File ".../openstackclient/common/clientmanager.py", line 258, in get_endpoint_for_service_type endpoint = self.auth_ref.service_catalog.url_for( PluginAttributeError: 'NoneType' object has no attribute 'url_for' Change-Id: I0eee7eba6eccc6d471a699a381185c4e76da10bd
* | | Merge "Remove methods argument from FakeHypervisorStats"Jenkins2016-04-161-2/+1
|\ \ \
| * | | Remove methods argument from FakeHypervisorStatsTang Chen2016-04-151-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | methods argument is not necessary in FakeHypervisorStats class. Remove it. Change-Id: I288f71b412beb3e583e957a9fd99210bc6f3f543
* | | | Add support for removing volume-type-accessSheel Rana2016-04-161-0/+58
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | OSC does not support to remove volume type access to project. This feature will provide support to remove volume type access from project. Closes-Bug:#1554890 Implements: bp cinder-command-support Change-Id: I029a4292da05f028e8937962cb845ec6e00b0279
* | | | Merge "Fix wrong attribute name and add functional test for --snapshot"Jenkins2016-04-151-0/+43
|\ \ \ \
| * | | | Fix wrong attribute name and add functional test for --snapshotJirayut Nimsaeng2016-04-151-0/+43
| |/ / / | | | | | | | | | | | | | | | | Change-Id: I91f2091ef06a55bcf5373d1beeea2dd81e9f1334 Closes-Bug: #1567895
* | | | Merge "Append existing information during port set"Jenkins2016-04-151-1/+20
|\ \ \ \
| * | | | Append existing information during port setreedip2016-04-141-1/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Existing --fixed-ip and --binding-profile information is currently overwritten when a user executes 'port set', but actually that data should be appended. This patch fixes the issue. Closes-Bug: #1564453 Change-Id: I62500c10ccbbc68167f24e9d4fa49e85345d82c4
* | | | | Merge "Add external network options to osc network set"Jenkins2016-04-151-0/+9
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Add external network options to osc network setreedip2016-04-151-0/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The following patch adds the options "--external" & "--internal" and the suboptions to "external": "--default" & "--no-default", to "osc network set" CLI to provide the user an option to set a network as an external network or remove the setting. Change-Id: I3a7f2cb249bc8101cbb01322d7732e913237d6cd Partial-Bug: #1545537
* | | | | Merge "Add project options to security group rule create"Jenkins2016-04-141-0/+41
|\ \ \ \ \ | |_|_|/ / |/| | | |
| * | | | Add project options to security group rule createRichard Theis2016-04-111-0/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add the --project and --project-domain options to the 'os security group rule create' command. These options are for Network v2 only. Change-Id: Ie3e136be076f0f2c22fbe7048d1d6eaebf5aa655 Partial-Bug: #1519512 Implements: blueprint neutron-client
* | | | | Merge "Add network options to security group rule create"Jenkins2016-04-141-2/+51
|\ \ \ \ \ | |/ / / /
| * | | | Add network options to security group rule createRichard Theis2016-04-111-2/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add the following network options to the "os security group rule" command: (1) --ingress and --egress (2) --ethertype These options enable egress and IPv6 security group rules for Network v2. Change-Id: Ie30b5e95f94e0c087b0ce81e518de72d2dda25ad Partial-Bug: #1519512 Implements: blueprint neutron-client
* | | | | Merge "Remove fake methods code from compute network"Jenkins2016-04-141-77/+15
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Remove fake methods code from compute networkTang Chen2016-04-121-77/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Network objects don't have any method needs to fake. keys() method is only used by _get_columns() helper to obtain all attributes of an object. But in compute network implementation, attributes are obtained from obj._info directly, which is a dictionary itself. So there is no need to fake this method. Change-Id: Ie6a46ef6a3042641e55a7002573ef501db7b60e1
* | | | | Merge "Append existing information during subnet set"Jenkins2016-04-131-0/+20
|\ \ \ \ \
| * | | | | Append existing information during subnet setreedip2016-04-081-0/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Existing values of --dns-nameserver, --allocation-pool and --houst-routes is currently overwritten when a user executes 'port set', but actually that data should be appended. This patch fixes the issue. Closes-Bug: #1564447 Change-Id: I3dba9afa68d869abb3960b55a6880401a10eebf7
* | | | | | Merge "Add --address-scope option "subnet pool create/set""Jenkins2016-04-132-6/+132
|\ \ \ \ \ \
| * | | | | | Add --address-scope option "subnet pool create/set"Tang Chen2016-04-092-6/+132
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds --address-scope option to "subnet pool create/set" commands, and --no-address-scope option to "subnet pool set" command to clear the address scope setting. Change-Id: Ie2c370a50b52574fa6ec268083ad013b7544361e Partial-Bug: #1544586 Partial-Bug: #1544591
* | | | | | | Merge "Add support for setting volume-type-access"Jenkins2016-04-122-0/+68
|\ \ \ \ \ \ \
| * | | | | | | Add support for setting volume-type-accessSheel Rana2016-04-112-0/+68
| | |_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | OSC does not support to set volume type access to project. This patch will provide support for adding volume type access to existing project. Closes-Bug:#1554889 Implements: bp cinder-command-support Change-Id: Ie36e202bdde7de36eb263a476eb66699d82f7565
* | | | | | | Fix prefixes output for subnet pool listRichard Theis2016-04-121-2/+2
| |_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fixed "os subnet pool list" command to properly disply the list of subnet pool prefixes in the "Prefixes" column. This fix is consistent with the "os subnet pool create" and "os subnet pool show" command output. Change-Id: I431d85c3b7f5bf8a327500decf3a15063fc5b120 Closes-Bug: #1569480