summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Implement "address scope show" command"Jenkins2016-05-114-0/+94
|\
| * Implement "address scope show" commandHuanxuan Ao2016-05-114-0/+94
| | | | | | | | | | | | | | | | This patch add a command that supports showing address scope details Change-Id: Ic0b41c1cab8c618904c7a6046d7493db5b74b430 Partial-Bug: #1566269
* | Merge "Implement "address scope list" command"Jenkins2016-05-115-0/+98
|\ \ | |/
| * Implement "address scope list" commandHuanxuan Ao2016-05-115-0/+98
| | | | | | | | | | | | | | | | This patch add a command that supports listing address scopes Change-Id: Id14757011560cacf28011ba51841a8e23b824f33 Partial-Bug: #1566269
* | Merge "Implement "address scope delete" command"Jenkins2016-05-114-0/+67
|\ \ | |/
| * Implement "address scope delete" commandHuanxuan Ao2016-05-114-0/+67
| | | | | | | | | | | | | | | | This patch add a command that supports deleting a address scope Change-Id: Ie028058c759b9511d105a530d7e89b841865e7d6 Partial-Bug: #1566269
* | Merge "Implement "address scope create" command"Jenkins2016-05-115-0/+311
|\ \ | |/
| * Implement "address scope create" commandHuanxuan Ao2016-05-115-0/+311
| | | | | | | | | | | | | | | | | | This patch supports creating a new address scope, with --ip-version,--project,--project-domain and --share or --no-share options. Change-Id: I37c73391a41ac239dd72d55dbc0adbebd7701f4a Partial-Bug: #1566269
* | Merge "Documentation updates"Jenkins2016-05-101-2/+2
|\ \
| * | Documentation updatesDean Troyer2016-04-301-2/+2
| | | | | | | | | | | | | | | | | | * Update link to meeting information Change-Id: I8dc6044b1faaa411e3b8d3b93fb401a480e82348
* | | Updated from global requirementsOpenStack Proposal Bot2016-05-101-1/+1
| | | | | | | | | | | | Change-Id: Id56a0313f630dd812218b9b7dfa812f185958a5d
* | | Merge "Ignore domain related config when using with keystone v2"Jenkins2016-05-091-3/+3
|\ \ \
| * | | Ignore domain related config when using with keystone v2Hieu LE2016-05-091-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, "/usr/bin/openstack --insecure token issue" fails when OS_AUTH_URL and OS_IDENTITY_API_VERSION indicate keystone v2 if OS_PROJECT_DOMAIN_NAME or OS_USER_DOMAIN_NAME are set. This patchset ignore domain related configs if using with keystone v2 and print warning for each ignored config. Change-Id: I8afbda787df7855c3f8e868b0f07cbf3b9cd97fd Closes-bug: #1447704
* | | | Merge "Map server power state num to meanful string"Jenkins2016-05-093-1/+12
|\ \ \ \ | |/ / / |/| | |
| * | | Map server power state num to meanful stringTang Chen2016-05-073-1/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In _prep_server_detail(), power_state is not formatted by _format_servers_list_power_state(). So when executing "server show" or "server create", the power state is represented by number. This patch map the numbers to meanful strings. This patch also adds power_state attribute to FakeServer, and improves unit tests for this attribute. Change-Id: I2ec674327de4e5133b8712ba6bb53fa5ce55e3f4
* | | | Merge "Fix functional test failures"Jenkins2016-05-081-9/+11
|\ \ \ \
| * | | | Fix functional test failuresRichard Theis2016-05-071-9/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There have been a lot of functional test failures for compute and network. I believe they are timing related in that the test_server is sometimes booting a server with a flavor, image and/or network created by another functional test. Such resources are then deleted by the owning functional tests causing various types of failures Change-Id: Ib82edd487da46610374fb0eb5796cb98f2d3aa2c
* | | | | Merge "Trivial: Remove unuseful comments for assertRaise() checking"Jenkins2016-05-075-16/+0
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Trivial: Remove unuseful comments for assertRaise() checkingTang Chen2016-05-065-16/+0
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | "Missing required args should bail here" is not understandable and not necessary. The code is obvious enough. And some of the comments are misused because of code copy. So remove them. Change-Id: I031395f2c882386c7a708db5cf4eee75393dc639
* | | | Updated from global requirementsOpenStack Proposal Bot2016-05-061-1/+1
| | | | | | | | | | | | | | | | Change-Id: Ic26729d1ce6e367d1b1cff38f6802cfed0d509ec
* | | | Merge "Add describe of overwrite options behavior into devref"Jenkins2016-05-061-7/+26
|\ \ \ \
| * | | | Add describe of overwrite options behavior into devrefRui Chen2016-05-031-7/+26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update the devref to add the describe and code example about overwrite options behavior. Change-Id: I65e9a3a30acf8d427906096bde24fa8b4c3ac3f7 Implements: blueprint allow-overwrite-set-options
* | | | | Merge "Spec to Implement IP Availability"Jenkins2016-05-061-0/+61
|\ \ \ \ \
| * | | | | Spec to Implement IP AvailabilityAnkur Gupta2016-05-021-0/+61
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Specification to lay out new commands to be added to OSC. New feature has been implemented and released for Neutron and python-neutronclient to display IP usages. Commands will display the IP usage across all networks, or detailed IP usage specifics for a given network Partially Implements Blueprint: neutron-ip-capacity Change-Id: I1c25de1bd924d92a96c20d60fa24d9e966b92dcd
* | | | | | Merge "Make "flavor show" command to show a private flavor properly"Jenkins2016-05-063-21/+37
|\ \ \ \ \ \ | |_|_|/ / / |/| | | | |
| * | | | | Make "flavor show" command to show a private flavor properlyHuanxuan Ao2016-05-023-21/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The "flavor show" command could not show a private flavor by flavor name becauce it could not find a private flavor by flavor name. In "until.find_resource(parsed_args.flavor)", If parsed_args.falvor is a name of a flavor, "flavors.find(name=parsed_args.flavor)"will be called to find a flavor.But the default value of "is_public" is "Ture" in "flavors.find()" so that we can only find public flavors.If we want to find all flaovrs by flavor name,we should add "is_public=None" in "flavors.find()". So I tried to change "until.find_resource(parsed_args.flavor)" to "until.find_resource(parsed_args.flavor, is_public=None)", but then I could not find any flavor by flavor id because "is_public" is an unexpected argument of "flavors.get()" in "until.find_resource()". In this case,I think "until.find_resource()" can not find a private flavor properly,and we should combine "manager.get(flavor.id)" and "manager.find(name=flavor.name, is_public=None)" by ourselve to find a flavor. Also,this bug affects other flavor commands like "flavor set/unset/delete",so I fix them in this patch too. Change-Id: I4a4ed7b0a2f522ee04d1c3270afcda7064285c39 Closes-Bug: #1575478
* | | | | | Merge "Ignore domain related config when using with keystone v2"Jenkins2016-05-061-0/+12
|\ \ \ \ \ \
| * | | | | | Ignore domain related config when using with keystone v2Hieu LE2016-05-061-0/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, "/usr/bin/openstack --insecure token issue" fails when OS_AUTH_URL and OS_IDENTITY_API_VERSION indicate keystone v2 if OS_PROJECT_DOMAIN_NAME or OS_USER_DOMAIN_NAME are set. This patchset ignore domain related configs if using with keystone v2. Change-Id: If7eea2ed1a4877c60d055ed0114a5e5f31e282a0 Closes-bug: #1447704
* | | | | | | Remove unnecessary type conversions in network unit testsTang Chen2016-05-065-46/+52
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In some tests, when comparing the results data with the expected ones, many unnecessary type conversions are used. So remove them to clean up. Change-Id: I560ca78a3ab5e7b99087bfe1667de500f92c68de Partial-bug: #1550633
* | | | | | bump timeout to prevent gate failuresSteve Martinelli2016-05-051-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | attempt to fix the transient gate failures by increasing the timeout Change-Id: I837652013f94b0d1ed3f4b40fe14ce5a47c687b6
* | | | | | Merge "Add a unit test for "flavor create" command"Jenkins2016-05-052-2/+162
|\ \ \ \ \ \
| * | | | | | Add a unit test for "flavor create" commandHuanxuan Ao2016-05-042-2/+162
| | |_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There was not a unit test for "flavor create" command in the "test_flavor.py".So I add the unit test. Change-Id: Ib1e821ea524eb33c0ba73643164228c7b83253b4
* | | | | | Merge "Add functional tests for commands of floating ip"Jenkins2016-05-051-0/+58
|\ \ \ \ \ \
| * | | | | | Add functional tests for commands of floating ipHuanxuan Ao2016-04-281-0/+58
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch add functinal tests for commands of floating ip Change-Id: I7f29578d0e14884f21183bfb82228d2fe7b7a029
* | | | | | | Added "name" parameter to the help message.sharat.sharma2016-05-041-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In OS_IDENTITY_API_VERSION=3, the "openstack help service delete" allows the user to remove a service based on the name of the service. So, this patch includes name as positional argument. Change-Id: Iae7cb0a82af3cdd4d88e0ed2eb651abf9af30fd9 Closes-Bug: #1566909
* | | | | | | Merge "Fix router set --route option"Jenkins2016-05-033-11/+39
|\ \ \ \ \ \ \
| * | | | | | | Fix router set --route optionRichard Theis2016-04-184-13/+43
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix the "--route" option on the "os router set" command. The option did not properly format the new routes to set which resulted in a "HttpException: Bad Request" error. In addition, the output for routes was fixed to improve readability and to align with the "--route" option on the "os router set" command. Change-Id: I9c514153ec201e2feae32be6dd281771e3298b9c Closes-Bug: #1564460
* | | | | | | | remove assert in favor an if/elseMichael McCune2016-05-023-15/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-024-9/+28
|\ \ \ \ \ \ \ \
| * | | | | | | | Fixes BadRequest when no --pool-prefix givenInessa Vasilevskaya2016-04-274-9/+28
| | |/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | --pool-prefix is made required on subnetpool creation. Closes-bug: #1536479 Change-Id: I3d183e45e9b96bc08011c36f45ec2b7a9c01b627
* | | | | | | | Merge "add a bandit environment to tox"Jenkins2016-05-022-2/+32
|\ \ \ \ \ \ \ \ | |_|_|_|/ / / / |/| | | | | | |
| * | | | | | | add a bandit environment to toxMichael McCune2016-04-282-2/+32
| | |_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change is being proposed as part of the OpenStack Security Project working session at the Austin 2016 summit. It adds support for running the bandit[1] security linting tool against the python-openstackclient codebase. This change adds a targetted environment for bandit and also adds bandit as part of the pep8 job. The bandit configuration has been tailored to exclude tests that are currently producing warning against the codebase. These issues will be followed up with bug reports and patches. [1]: https://wiki.openstack.org/wiki/Security/Projects/Bandit Depends-On: Iccd81c17e84df03d249c1012277dad9cb68c5845 Change-Id: I691829c1224557d1d239c9f665ac539d0f13c4d3
* | | | | | | Replace tempest-lib with tempest.libDaniel Gonzalez2016-04-2917-22/+22
| |_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | tempest-lib is deprecated, so replace it with tempest.lib. Co-Authored-By: Sheel Rana <ranasheel2000@gmail.com> Change-Id: I0495eba110bb7581623fbcf49dc63a27e9cb6d64 Closes-Bug: #1553047
* | | | | | Merge "Doc: Add network resource descriptions"Jenkins2016-04-288-0/+32
|\ \ \ \ \ \
| * | | | | | Doc: Add network resource descriptionsRichard Theis2016-04-258-0/+32
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | Add descriptions to the network resource command documentation. Change-Id: I547ffb48f8950311a5ee65d6b535846f2aca0efc
* | | | | | Merge "Fix error in flavor set/unset command"Jenkins2016-04-283-6/+17
|\ \ \ \ \ \
| * | | | | | Fix error in flavor set/unset commandHuanxuan Ao2016-04-283-6/+17
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-2810-0/+649
|\ \ \ \ \ \ | |/ / / / / |/| | | | |
| * | | | | Support for volume service listSheel Rana2016-04-2510-11/+660
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-255-17/+46
|\ \ \ \ \ | |_|/ / / |/| | | |
View Lorry Controller Status