summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | | Merge "Use osc_lib in server_image.py"Jenkins2016-06-202-5/+6
|\ \ \ \ \ \ \ \ | |_|_|_|_|/ / / |/| | | | | | |
| * | | | | | | Use osc_lib in server_image.pyTang Chen2016-06-202-5/+6
| | |_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | server_image.py is a newly created file. So I think we forgot to use osc_lib in it. Change-Id: Ieda13438662ea55b03f549108aac63c18b9af913
* | | | | | | Merge "Add doc for logger usage"Jenkins2016-06-202-0/+74
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | |
| * | | | | | Add doc for logger usageTang Chen2016-06-162-0/+74
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds a doc to record rules of usage of loggers in OSC. Change-Id: I27a09f9abb523393031560e2310bfdd1af0b8922 Implements: blueprint log-usage
* | | | | | Merge "Do not prompt for scope options with default scoped tokens"Jenkins2016-06-175-33/+49
|\ \ \ \ \ \
| * | | | | | Do not prompt for scope options with default scoped tokensDolph Mathews2016-06-175-33/+49
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This changes the scope validation to occur after a token has already been created. Previous flow: 1. Validate authentication options. 2. Validate authorization options if the command requires a scope. 3. Create a token (using authentication + authorization options) 4. Run command. This means that scope was being checked, even if a default scope was applied in step 3 by Keystone. New flow: 1. Validate authentication options. 2. Create token (using authentication + authorization options) 3 Validate authorization options if the command requires a scope and the token is not scoped. 4. Run command. Change-Id: Idae368a11249f425b14b891fc68b4176e2b3e981 Closes-Bug: 1592062
* | | | | | Merge "Updated from global requirements"Jenkins2016-06-171-1/+1
|\ \ \ \ \ \
| * | | | | | Updated from global requirementsOpenStack Proposal Bot2016-06-171-1/+1
| |/ / / / / | | | | | | | | | | | | | | | | | | Change-Id: I2d6aadd0509f5f311d7433ea457ecf5ad6e4e64e
* | | | | | Merge "Modify unit tests of compute agent delete"Jenkins2016-06-171-0/+10
|\ \ \ \ \ \ | |/ / / / / |/| | | | |
| * | | | | Modify unit tests of compute agent deletesunyajing2016-06-081-0/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | add no-input test to ``compute agent delete`` unit tests. Change-Id: Iee22b75c9a9431e57cb634dc28a5efa9b43b7369
* | | | | | Merge "Make set/unset command in identity and image pass normally when ↵Jenkins2016-06-177-40/+89
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | nothing specified"
| * | | | | | Make set/unset command in identity and image pass normally when nothing ↵sunyajing2016-06-177-40/+89
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | specified Also update its unit tests. Change-Id: I82b90658b0d4247cdc9a650f14aceda640a32059 Partial-bug: #1588588
* | | | | | | move release note to correct directorySteve Martinelli2016-06-161-0/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | not sure how this one release note was slotted into it's own directory, but let's move it with the others. Change-Id: I20eb36b1778b6fc9b5e550390962ec11dae38db4
* | | | | | | Merge "Support bulk deletion for delete commands in networkv2"Jenkins2016-06-168-31/+215
|\ \ \ \ \ \ \
| * | | | | | | Support bulk deletion for delete commands in networkv2Huanxuan Ao2016-06-168-31/+215
| | |_|/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch support bulk deletion for delete commands below: 1.subnet delete 2.subnet pool delete Up to now, all the delete commands in networkv2 support bulk deletion. Change-Id: I63f6d1d02bad1fcc26e72b7028b53958a68ce2dc Partially-Implements: blueprint multi-argument-network Partial-Bug: #1592906
* | | | | | | Merge "Add "--password-prompt" to user.rst"Jenkins2016-06-161-0/+1
|\ \ \ \ \ \ \
| * | | | | | | Add "--password-prompt" to user.rstsunyajing2016-06-161-0/+1
| |/ / / / / / | | | | | | | | | | | | | | | | | | | | | Change-Id: I2e86114698887a6343c248783c343f269bf4988d
* | | | | | | Merge "Set up 3.x release notes"Jenkins2016-06-164-7/+31
|\ \ \ \ \ \ \
| * | | | | | | Set up 3.x release notesDean Troyer2016-06-074-7/+31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Give the 2.x series its own release notes page, put the 3.x notes into the 'current' series page. This removes the OpenStack development cycle name from the release notes. We follow the intermediary release model, however the association with the common cycle release is something that we do not need to reflect in the doc structure. We want users to use the current version even with older clouds, not to think they need to use a 'Mitaka' OSC with a Mitaka cloud. Change-Id: Idbb6844824d50e5cd0fa64871b5aa625d4d237af
* | | | | | | | Merge "Move server image create command to its own resource file."Jenkins2016-06-167-224/+346
|\ \ \ \ \ \ \ \
| * | | | | | | | Move server image create command to its own resource file.Dean Troyer2016-06-037-224/+346
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: If37e82072bd7a32b81bfb1a8bb048f018dd5b04f
* | | | | | | | | Merge "Modify doc issues about property option of aggregate"Jenkins2016-06-161-4/+3
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Modify doc issues about property option of aggregateHuanxuan Ao2016-06-161-4/+3
| | |_|/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The formal of multi properties in most docs likes this: [--property <key=value> [...] ] [--property <key> [...] ] This patch modify the formal of multi properties in aggregate.rst. Change-Id: I557cd51696d2721c8955c4a79f1ab85de3a0b9ce
* | | | | | | | | Merge "Add functional test for "aggregate unset" command"Jenkins2016-06-161-0/+8
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Add functional test for "aggregate unset" commandHuanxuan Ao2016-06-161-0/+8
| |/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There was not functional test for "aggregate unset" command, this patch add the test. Change-Id: Icc8f51e863231e915b2a8cca59baaedd54a96de5
* | | | | | | | | Merge "Fix help msg of identity endpoint"Jenkins2016-06-163-11/+11
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Fix help msg of identity endpointsunyajing2016-06-163-11/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | <service> argument of `endpoint create` command doesn't mean `new endpoint service`, but an existent service that the new endpoint attached to. Change-Id: I846fdb501bdea14499f42288186f375a3b2b5951
* | | | | | | | | | Finish osc-lib transition for command.pyDean Troyer2016-06-152-3/+3
| |/ / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://review.openstack.org/#/c/328318/ was the transition of openstackclient/common/command.py to osc-lib but it was incomplete; https://review.openstack.org/#/c/329885/ started to fix it but was also incomplete. Third time is charm? Change-Id: Ib6c0266155c2155ae59067ce7b15bebca1e375e8
* | | | | | | | | Merge "Fix volume functional tests"Jenkins2016-06-158-13/+40
|\ \ \ \ \ \ \ \ \ | |_|_|_|_|_|_|_|/ |/| | | | | | | |
| * | | | | | | | Fix volume functional testsRichard Theis2016-06-158-13/+40
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The volume v2 functional tests occasionally fail with "unrecognized arguments: --private". This appears to occur when volume v1 and volume v2 functional tests are run at the same time when one or more of the tests do not set OS_VOLUME_API_VERSION appropriately. This patch also fixes a timing issue seen when running the tests locally due to a missing wait and moves the snapshot test to volume v2 since it was written for v2. Change-Id: I9c9863fe1aef81b0564920f36452c6f27211d5ba Closes-Bug: #1589793
* | | | | | | | | Merge "Setup deprecate msg for command.py"Jenkins2016-06-151-38/+8
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Setup deprecate msg for command.pyTang Chen2016-06-151-38/+8
| |/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This could be a miss when we use osc_lib first in OSC. command.py has been moved to osc_lib, and should reuse it and print a deprecate msg if the old file is used. Change-Id: Ibc35659f6f78b8f5e3c6026db2644a876ef7c549
* | | | | | | | | Merge "Support bulk deletion for commands that exist in both network and ↵Jenkins2016-06-1512-72/+459
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | compute."
| * | | | | | | | | Support bulk deletion for commands that exist in both network and compute.Huanxuan Ao2016-06-1512-72/+459
| |/ / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some delete commands in networkv2 are exist in both network and compute, They can use NetworkAndComputeDeleteclass to supprot bulk deletion and error handling and the codes are similar, so I change them all in this patch. The changed commands including: 1.floating ip delete 2.security group delete 3.security group rule delete Also, I update unit tests and docs for these commands in this patch. Change-Id: I6c94c3d10ba579ddd9b14d17673c821e3481fd8a Partially-Implements: blueprint multi-argument-network
* | | | | | | | | Merge "Support compute service force down/up"Jenkins2016-06-154-20/+165
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Support compute service force down/upRui Chen2016-06-144-20/+165
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Aims to evacuate servers from compute host as soon as possible, operators might set the compute service force down manually. Novaclient support the behavior, this patch support it in OSC. Change-Id: I22ff1c5d670c449771fdcb3f4f39cd82f428531a Closes-Bug: #1589348
* | | | | | | | | | Add default IP version and fix help messages for "ip availability list"Huanxuan Ao2016-06-154-5/+15
| |/ / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There was not a default IP version in "ip availability list" command, if we used this command without "--ip-version" option, the output was different from the outputs with the option "--ip-version 4" and "--ip-version 6" and it is not right. This patch add default IP version (default is 4) in ``ip availability list`` command and make this command work properly without ``--ip-version`` option. And also fix the help message. Change-Id: Idc08ab6eaf05946eb2ab59bfb3d4497a383d987d Closes-Bug: #1592761
* | | | | | | | | Merge "Add geneve provider network type"Jenkins2016-06-153-6/+11
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Add geneve provider network typeRichard Theis2016-06-103-6/+11
| | |_|_|_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add the "geneve" choice to the "os network create" command's "--provider-network-type" option. Change-Id: I7573232ec3594ec4acbfae43a8456b8c3fcd1a83 Implements: blueprint neutron-client
* | | | | | | | | Merge "Standardize logger usage in volume"Jenkins2016-06-154-30/+42
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Standardize logger usage in volumeTang Chen2016-06-144-30/+42
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | self.app.log is the logger in class OpenStackShell, which should be used to record logs that have nothing to do with any specific command. So, use the file logger instead. This patch also fixes some usage that doesn't follow rules in: http://docs.openstack.org/developer/oslo.i18n/guidelines.html 1. add variables to logger as an argument 2. do not wrap variables with str() Change-Id: I248861a38a4de0412a080046aa7a6f6473c3e082 Implements: blueprint log-usage
* | | | | | | | | | Merge "Fix foundation copyrights"Jenkins2016-06-147-7/+7
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Fix foundation copyrightsDean Troyer2016-06-147-7/+7
| |/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The OpenStack LLC copyrights date from before OpenStack was transferred to the foundation. These appear to be getting copy-pasted to new files so at least should reflect the correct entity. Change-Id: I02953d752cb24ead6aa4ad8bfe257a48317c9f13
* | | | | | | | | | Merge "Remove blank line in release notes"Jenkins2016-06-141-1/+0
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Remove blank line in release notesSamYaple2016-06-141-1/+0
| |/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The blank line was causing yamllint to fail in other projects. [1] Related-Id: I400131bf84a4cf48a83952efa7dfac8e4954b587 [1] http://logs.openstack.org/64/329564/1/check/gate-heat-templates-pep8/e538974/console.html#_2016-06-14_17_39_02_316 Change-Id: Icfd902b6c9408ccb9a710d10d52d65efe753b77f
* | | | | | | | | | Merge "Add release note for network endpoint type bugfix."Jenkins2016-06-141-0/+6
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Add release note for network endpoint type bugfix.Stuart McLaren2016-06-141-0/+6
| |/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Related-bug: 1592368 Change-Id: I9bd8c3490e72b32ba48dc9d884c101d5e50867f9
* | | | | | | | | | Merge "Fix errors in ``set/unset flavor`` unit tests"Jenkins2016-06-141-9/+20
|\ \ \ \ \ \ \ \ \ \
| * | | | | | | | | | Fix errors in ``set/unset flavor`` unit testsTang Chen2016-06-141-9/+20
| | |_|_|_|_|_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1. We should assert flavor.set_keys() is called correctly or not called in ``set flavor`` unit tests. 2. Commit c7e6973ff50ff84af9ad55f7bdaeeea83ae40f0b from me removed test_flavor_unset_no_project which used to test if --project was specified as '', and assert CommandError was raised, which is incorrect. So I removed it. But after looking into the code, I think we should not remove it, but specify nothing after --project, and assert ParserException was raised. So in this patch, we fix it as so. ('--project', '') --> ('--project') assert CommandError --> assert ParserException Change-Id: Ifd33c72d5b7581aaabffb09e9b5e38ecc67e18c0
* | | | | | | | | | Fix missing i18n supports in api/ and shell.pyHuanxuan Ao2016-06-144-37/+52
| |/ / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I28d79d7f44b27d2b600dedad2a3601180650ad83 Partial-bug: #1574965