summaryrefslogtreecommitdiff
path: root/openstackclient
Commit message (Collapse)AuthorAgeFilesLines
* 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-183-1/+24
|\ \
| * | Propagate AttributeErrors when lazily loading pluginsTim Burke2016-04-143-1/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | State i18() changes and help messages improvedTang Chen2016-04-188-200/+229
| |/ |/| | | | | | | | | | | | | Add _() to wrap help message in network commands. And also some improvement for help message. Change-Id: Ib3e498f5976ba98c44fd4eec2d1623263b3db53e Partial-bug: 1570924
* | 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-162-3/+101
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-152-7/+34
|\ \ \
| * | | Append existing information during port setreedip2016-04-142-7/+34
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-152-8/+39
|\ \ \ \ | |_|/ / |/| | |
| * | | Add external network options to osc network setreedip2016-04-152-8/+39
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-142-0/+56
|\ \ \ \ | |_|_|/ |/| | |
| * | | Add project options to security group rule createRichard Theis2016-04-112-0/+56
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-142-14/+98
|\ \ \ \ | |/ / /
| * | | Add network options to security group rule createRichard Theis2016-04-112-14/+98
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 "use correct manager for volume snapshots"Jenkins2016-04-141-1/+1
|\ \ \ \ \
| * | | | | use correct manager for volume snapshotsSteve Martinelli2016-04-081-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | the snapshot manager is volume_snapshots, not snapshots. Closes-Bug: #1567895 Change-Id: I12c0238fc04507c97e92089fda3bec816171a973
* | | | | | Trivial: Fix incorrect comment textTang Chen2016-04-143-3/+3
| |_|_|/ / |/| | | | | | | | | | | | | | Change-Id: Ia6ec15f11a535a8de5769569d75e81094caed171
* | | | | Merge "Doc: Unify repeatable option comments"Jenkins2016-04-1311-26/+26
|\ \ \ \ \
| * | | | | Doc: Unify repeatable option commentsTang Chen2016-04-1211-26/+26
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | | Merge "Append existing information during subnet set"Jenkins2016-04-132-0/+26
|\ \ \ \ \
| * | | | | Append existing information during subnet setreedip2016-04-082-0/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 "Fix SSL/TLS verification for network commands"Jenkins2016-04-131-1/+2
|\ \ \ \ \ \
| * | | | | | Fix SSL/TLS verification for network commandsRichard Theis2016-04-081-1/+2
| | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The network commands ignored the --insecure and --os-cacert options and OS_CACERT environment variable which prevented them from properly completing SSL/TLS verification. This resulted in the network commands failing with "An SSL error occurred." Change-Id: I15167631ef58335e1476c16b828b079e3b0f13c1 Closes-Bug: #1560157
* | | | | | Merge "Add --address-scope option "subnet pool create/set""Jenkins2016-04-133-8/+160
|\ \ \ \ \ \
| * | | | | | Add --address-scope option "subnet pool create/set"Tang Chen2016-04-093-8/+160
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-123-6/+116
|\ \ \ \ \ \ \
| * | | | | | | Add support for setting volume-type-accessSheel Rana2016-04-113-6/+116
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-122-3/+3
| |_|_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | | | | | Merge "Move keys() methods in each resource class to FakeResource"Jenkins2016-04-114-166/+63
|\ \ \ \ \ \ \ | |_|_|_|_|/ / |/| | | | | |
| * | | | | | Move keys() methods in each resource class to FakeResourceTang Chen2016-04-114-166/+63
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | FakeXXX classes in network don't need to fake any method, except keys(). But keys() can be put in FakeResource since it just returns all attributes in _info. This patch moves removes all unnecessary fake methods code, moves keys() method to FakeResource. This patch also finds out some missing attributes in network and router tests. Change-Id: I799822c8715b9ac4f95b98f8350f196757d79d3e
* | | | | | | Merge "Doc: Fix network command documentation issues"Jenkins2016-04-111-9/+8
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | |
| * | | | | | Doc: Fix network command documentation issuesRichard Theis2016-04-081-9/+8
| | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix the following network command documentation issues: - Fix format of subnet pool positional arguments - Update port set options to match help - Fix network command documentation for Network v2 versus Compute v2 options and commands - Fix subnet command documentation errors and formatting problems in help text Change-Id: I808c2a70ca62eafc9e42d6873539cdd142ffe50c Related-Bug: #1558677
* | | | | | Merge "Refactor security group rule list to use SDK"Jenkins2016-04-114-332/+301
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | |
| * | | | | Refactor security group rule list to use SDKRichard Theis2016-03-284-332/+301
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Refactored the 'os security group rule list' command to use the SDK when neutron is enabled, but continue to use the nova client when nova network is enabled. In addition, a release note was added to document the features and fixes resulting from this refactor. Change-Id: I24d04b720102ed1c60776e1ca67d4ca20e31b663 Partial-Bug: #1519512 Implements: blueprint neutron-client
* | | | | | Merge "Add Subnet add/remove support to router"Jenkins2016-04-092-0/+133
|\ \ \ \ \ \
| * | | | | | Add Subnet add/remove support to routerreedip2016-03-222-0/+133
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The following patch adds the support for "router add subnet" and "router remove subnet" to the OSC as a part of migration of Neutron's CLI commands. Partial-Bug: #1546849 Implements: blueprint neutron-client-advanced-router Change-Id: Ia3770c41026194bdb1543d4e67446f81936d44d1
* | | | | | | Merge "Add "router remove port" to osc"Jenkins2016-04-092-0/+64
|\ \ \ \ \ \ \ | |/ / / / / /
| * | | | | | Add "router remove port" to oscreedip2016-03-212-1/+65
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch proposes the migration of port deletion of Neutron's "router-interface-delete" to OSC's "router remove port". Change-Id: Ifceabce080aacac9f6410c809fcc097760e0c5ee Partial-Bug: #1546849 Implements: blueprint neutron-client-advanced-router
* | | | | | | Fix pep8 fail that crept inDean Troyer2016-04-081-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Not sure how, but this crept in via https://review.openstack.org/#/c/281691/ Change-Id: I919276ee5204277dbc5e96f4de4b98381142423a
* | | | | | | Merge "Add "router add port" to osc"Jenkins2016-04-082-1/+66
|\ \ \ \ \ \ \ | |/ / / / / / | | | / / / / | |_|/ / / / |/| | | | |
| * | | | | Add "router add port" to oscreedip2016-03-212-1/+66
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch proposes the migration of port addition of Neutron's "router-interface-add" to OSC's "router add port". Change-Id: I3a8a76d384caa32975f8e77f50f3cf3c3be13786 Closes-Bug: #1546849 Depends-On: I9783bc4ccceae3d361dce52d51483ef2187920a9 Implements: blueprint neutron-client-advanced-router
* | | | | | Merge "TrivialFix: Rename provider segment option"Jenkins2016-04-082-4/+4
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | |
| * | | | | TrivialFix: Rename provider segment optionreedip2016-04-082-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As per [1], provider-segmentation-id is changed to provider-segment as segments may soon have a "name" support as well. [1]: https://review.openstack.org/#/c/294422/12/doc/source/command-objects/network.rst Change-Id: I74d4366301b41fa181faccbc12cff9530df60353
* | | | | | Merge "Make snapshot and backup name optional"Jenkins2016-04-076-4/+48
|\ \ \ \ \ \ | |_|_|_|_|/ |/| | | | |
View Lorry Controller Status