summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Fix one remaining E125 error and remove it from ignore list"2.4.0Jenkins2015-04-092-7/+5
|\
| * Fix one remaining E125 error and remove it from ignore listAkihiro Motoki2015-04-092-7/+5
| | | | | | | | | | | | | | | | | | | | E125 continuation line does not distinguish itself from next logical line Also update the comment of H405 since multi line docstring is used frequently in unit test documentation and it seems better to hold it in the ignore list not temporarily. Change-Id: Ic24e6cebe0aa531e5a4556d3e23c752358bf3696
* | Add Neutron subnetpool APIzengfagao2015-04-096-1/+269
|/ | | | | | | | | | | | | Add Neutron subnetpool listing, creating, updating, showing and deleting Client API. neutron subnetpool-list neutron subnetpool-create neutron subnetpool-update neutron subnetpool-show neutron subnetpool-delete Change-Id: I19a6782d33609609f43353df8293864e60ead817 Partially-Implements: blueprint subnet-allocation
* Revert "Remove unused AlreadyAttachedClient"Akihiro Motoki2015-04-091-0/+6
| | | | | | | | | | | | The removal of AlreadyAtatchedClient exception from neutronclient suddenly causes the failure of neutronclient icehouse job. It is better to wait the removal of the exception until Icehouse EOL (which will come soon). This reverts commit 30b198edec988d7c969c1edd4a2754bd9c9c79f4. Closes-Bug: #1441969 Change-Id: Ifa02a4ca39bc9c0fe9b65d5b0d66c588885cb12c
* Merge "Fix E265 block comment should start with '# '"Jenkins2015-04-084-11/+6
|\
| * Fix E265 block comment should start with '# 'Akihiro Motoki2015-03-234-11/+6
| | | | | | | | | | | | All E265 errors are now fixed and remove it from ignore list. Change-Id: I09ada4cc0f33cae5da4cbc29a4e843e3ce14e111
* | Merge "Remove author tag"Jenkins2015-04-0848-90/+1
|\ \ | |/
| * Remove author tagAkihiro Motoki2015-03-2348-90/+1
| | | | | | | | | | | | | | We have a hacking rule H105 (Don't use author tags) now and the same policy has been applied to other neutron repos. Change-Id: I1a1a35bbfaad6bbe229b9b12cf6cfb405b6ed516
* | Merge "Update hacking to 0.10"Jenkins2015-04-084-6/+5
|\ \ | |/
| * Update hacking to 0.10Akihiro Motoki2015-03-235-7/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | Release notes: http://git.openstack.org/cgit/openstack-dev/hacking/tag/?id=0.10.0 * Fix W292 (no newline at end of file) * Fix H238 (old style class declaration, use new style) * Skip H105 (Don't use author tags) temporarily. * Remove H307 from ignore list since it is removed from Hacking rule. * Remove H302 (import only modules) since there is no violation now. Change-Id: Ic5967652819c9c322de327d504882a0e3029d462
* | Merge "Make secgroup rules more readable in security-group-show"Jenkins2015-04-081-0/+1
|\ \
| * | Make secgroup rules more readable in security-group-showAkihiro Motoki2015-03-251-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously security_group_rules in security-group-show is displayed in JSON dump in a format of one line per rule, but each line is too long and hard to understand rules. After this commit each rule is displayed as JSON dump with indentation and we can easily read security group rules. Partial-Bug: #1153766 Change-Id: I0a0d4afad340aee0346c73cf8881900375978858
* | | Merge "Support fwaasrouterinsertion extension"Jenkins2015-04-082-2/+71
|\ \ \
| * | | Support fwaasrouterinsertion extensionCedric Brandily2015-04-022-2/+71
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | fwaasrouterinsertion extension allows to specify which routers implement a firewall (on create/update). This changes adds its support by allowing to: * set routers with --router option in firewall-create/update commands, * unset routers with --no-routers option in firewall-update command. Change-Id: I654c1ddd4140a60b8a09237f7142ad211e951d92 Closes-Bug: #1435264
* | | Merge "Allow passing None for subnetpool"Jenkins2015-04-071-2/+5
|\ \ \
| * | | Allow passing None for subnetpoolCarl Baldwin2015-04-061-2/+5
| | | | | | | | | | | | | | | | | | | | Change-Id: I90be335bbe466381ba40423cfef345ca380676b5 Partially-Implements: blueprint subnet-allocation
* | | | Merge "Add Neutron subnet-create with subnetpool"Jenkins2015-04-072-8/+67
|\ \ \ \ | |/ / /
| * | | Add Neutron subnet-create with subnetpoolzengfagao2015-04-062-8/+67
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add neutron subnet-create with subnetpool feature. With subnetpool, user can either have prefixlen or cidr. The cidr for subnet is optional parameter with subnetpool. Change-Id: I795b233985f9ab7cb59d7318a5d7a1e9c4431a1c Partially-Implements: blueprint subnet-allocation
* | | | Adding VLAN Transparency support to neutronclientPritesh Kothari2015-04-042-0/+21
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adding VLAN Transparency support to neutronclient, it allows to create new vlan transparent networks. It adds support for displaying the network vlan transparency support via net-list. DocImpact Depends-on: Ie87087a70b83dab589419aa5c17ce7ccafd64cbd Implements: blueprint nfv-vlan-trunks Change-Id: I24c2eecd34123933f98b596654baaf76ed1453e8
* | | | 'neutron port-create' missing help info for --binding:vnic-typeVikram Hosakote2015-04-022-0/+70
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | --binding:vnic-type can be specified for 'neutron port-create' to create SR-IOV vNIC ports, but the help info is missing for --binding:vnic-type. This patch fixes the above bug. Below is the output. $ neutron port-create --help | grep -i vnic [--vnic-type <direct | macvtap | normal>] --vnic-type <direct | macvtap | normal> VNIC type for this port. DocImpact Change-Id: Ib31c345d84df8861a058fdd221bb7bc9fc33b6c9 Closes-bug: #1432393
* | | | Merge "Prefer argparse mutual exclusion"Jenkins2015-03-302-13/+7
|\ \ \ \
| * | | | Prefer argparse mutual exclusionCedric Brandily2015-03-292-13/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change replaces homemade option mutual exclusion check by argparse one (ArgumentParser.add_mutually_exclusive_group). Change-Id: Ib626aba4b63db4dd635a4ef000c2dd5f19dd4446
* | | | | Merge "Add HA router state to l3-agent-list-hosting-router"Jenkins2015-03-281-1/+5
|\ \ \ \ \ | |_|_|_|/ |/| | | |
| * | | | Add HA router state to l3-agent-list-hosting-routerAssaf Muller2015-03-271-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Depends-On: Ie0f53b7565d53ff791b0cdcca20be2b4415b49cc Change-Id: I888fb62f9b9013f655e02a463063db549654b00e Partially-Implements: blueprint report-ha-router-master
* | | | | Merge "Updated from global requirements"Jenkins2015-03-232-8/+8
|\ \ \ \ \ | |_|_|_|/ |/| | | |
| * | | | Updated from global requirementsOpenStack Proposal Bot2015-03-212-8/+8
| | |_|/ | |/| | | | | | | | | | Change-Id: I5df27bf88d70b29da62932a0c25c896849d6171d
* | | | Merge "security-group-rule-list: show all info of rules briefly"Jenkins2015-03-223-75/+200
|\ \ \ \
| * | | | security-group-rule-list: show all info of rules brieflyAkihiro Motoki2015-03-153-75/+200
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously security-group-rule-list does not display full information of rules by default (e.g., port ranges, ether types) and users need to use security-group-rule-show to check details. It is not convenient. This commit introduces some aggregated columns ("protocol/port" and "remote") to show infomration briefly and as a result full attributes of rules will be displayed. Closes-Bug: #1182629 Change-Id: I047bf9a1ccba5b023d66f22ef5256f7786196113
* | | | | Merge "Show rules in handy format in security-group-list"Jenkins2015-03-223-2/+146
|\ \ \ \ \ | |/ / / / | | | | / | |_|_|/ |/| | |
| * | | Show rules in handy format in security-group-listAkihiro Motoki2015-03-153-2/+146
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously security group rules in security-group-list was just JSON dump of security group rules if security_group_rules field is requested to display and it leads to long lines. This commit defines a formater for sg rules for ListSecurityGroup. It changes the default columns to show "security_group_rules". I believe it is useful for users because we don't need to run neutron security-group-rule-list to know rules. Closes-Bug: #1153766 Change-Id: I5e1b93bea6ab1121f85dc19d2e75fffd065cd627
* | | | Merge "Fix failures when calling list operations using Python binding"Jenkins2015-03-214-17/+21
|\ \ \ \ | |_|/ / |/| | |
| * | | Fix failures when calling list operations using Python bindingAnand Shanmugam2015-03-214-17/+21
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This bug is caused because of the regression caused by I1b719bed406b83c5f2deac06e127798a91f51ad7 . The original bug was raised because check_max_uri was not working after the introduction of sessionclient. The change created a regression in which the python bindings of the neutron client was not usable and causing traces. The fix is to revert the change id and add a new check_max_uri to the sessionclient. Now uri length will be checked in sessionclient and httpclient as well. please see bug for further info. Closes-Bug: #1431449 Change-Id: Ief2352a90bb75a76e8c671d51beb0fb7a53a22f9
* | | Merge "Add commands from extensions to available commands"Jenkins2015-03-182-2/+12
|\ \ \
| * | | Add commands from extensions to available commandsmathieu-rohon2015-03-132-2/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Any project can extend available commands in neutronclient thanks to change : I5b2fe530c90b5ce1243fc10341d6d434a1ecea7a However, commands submitted by external modules are not correctly added to available commands in the client. Change-Id: I53783f18e7811deee9ae7e1f48fc429afcba8936 Closes-bug: #1430825
* | | | Merge "Updates pool session persistence options"Jenkins2015-03-182-17/+9
|\ \ \ \
| * | | | Updates pool session persistence optionsptoohill12015-03-172-17/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Updates pool options to take kv args for type and cookie_name Updates tests Change-Id: I780948430db295af01b7297ee828f57d2a91c6d4 Closes-Bug: #1430541
* | | | | Merge "Added client calls for the lbaas v2 agent scheduler"Jenkins2015-03-184-0/+96
|\ \ \ \ \
| * | | | | Added client calls for the lbaas v2 agent schedulerBrandon Logan2015-02-254-0/+96
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The lbaas v2 agent requires separate REST calls from the v1 agent so it is necessary to also add these commands to the client. Change-Id: I2fa88e64094f74e198925f2fec397bf6f2f941d5 Depends-On: Ifc09e922e9afbdfa31d2edf022c28d56446a9403 Partially-implements: blueprint lbaas-refactor-haproxy-namespace-driver-to-new-driver-interface
* | | | | | Merge "Updating lbaas cli for TLS"Jenkins2015-03-182-5/+21
|\ \ \ \ \ \
| * | | | | | Updating lbaas cli for TLSptoohill12015-03-122-5/+21
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adding additional params for TLS support in the LBaaS cli Updates tests to verify TLS params Partially-Implements: lbaas-ssl-termination Depends-On: I4c5c494370479176af6f80623ae3a0c6af30ef14 Change-Id: Ie0ae991e11b491143ef63adc5d6f0b879477dd54
* | | | | | Merge "Change Creates to Create in help text"Jenkins2015-03-172-3/+3
|\ \ \ \ \ \ | |/ / / / / |/| | | | |
| * | | | | Change Creates to Create in help textDanny Choi2014-11-262-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In the help text for cisco-credential-create and cisco-network-profile-create, change the word Creates to Create to be consistent with others. Change-Id: I7b22816738e8b7a7e629ad78108b2e588282fbc0 Closes-bug: #1395352
* | | | | | Merge "Remove unused AlreadyAttachedClient"Jenkins2015-03-121-6/+0
|\ \ \ \ \ \
| * | | | | | Remove unused AlreadyAttachedClientDoug Fish2015-03-111-6/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This exception is no longer referenced in Horizon. It should be removed as noted in the TODO being removed. Change-Id: I9970ca375d19476c12891475d0a8c0c559e0507f
* | | | | | | Copy functional tests from tempest cliKyle Mestery2015-03-121-4/+156
| |_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This copies in all the tests from the tempest file tempest/cli/simple_read_only/network/test_neutron.py in preparation for removing that file. Change-Id: Icaaaa094c1637f5d0e5296491d349a282fadc2cc Closes-Bug: #1429289 Depends-On: I00e56cab162941bb9d9bd84d0fffd66e9698bb4c
* | | | | | Merge "exec permission to port_test_hook.sh"Jenkins2015-03-111-0/+0
|\ \ \ \ \ \ | |/ / / / / |/| | | | |
| * | | | | exec permission to port_test_hook.shAkihiro Motoki2015-03-111-0/+0
| | |_|/ / | |/| | | | | | | | | | | | | Change-Id: I516b27a7dc59a2e2fd7fb8128aa35c43a8c0b5b8
* | | | | Merge "Reinstate Max URI length checking to V2_0 Client"Jenkins2015-03-115-16/+28
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Reinstate Max URI length checking to V2_0 ClientMark Rawlings2015-03-095-16/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A previous commit 799e288f48e5d99731dedbfb94808a8cbe01c05c has broken the ability to recognise long URIs (>8192) and raise an exception used to prevent the excessive URI being sent to the server. This impacts the commands net-list and security-group-rule-list. The previous edit removed one of the duplicate _check_uri_length functions. This edit 'swaps' the removal and updates the unit test stubbing accordingly. The capability to split excessive URIs into managable chunks remained in the code, but no longer executed because this exception was not raised. It should be noted that as a side effect of recognising the long URI, data is returned with the exception that indicates 'how excessive' the URI length was. This allows the URI to be broken into chunks that will be supported. Restore the capability to recognise that an excessive URI has been provided and to raise the expected exception with related data. Closes-Bug: #1422736 Change-Id: I1b719bed406b83c5f2deac06e127798a91f51ad7
* | | | | Add post_test_hook for functional testsKyle Mestery2015-03-091-0/+50
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add post_test_hook to get functional job working for python-neutronclient. Once the project config side of this lands this script can be self gating. Change-Id: I21e691155f69b83940aa859d77e16c4289d9f78c Partial-Bug: #1429289