summaryrefslogtreecommitdiff
path: root/openstackclient
Commit message (Collapse)AuthorAgeFilesLines
* Create log configuration classTerryHowe2015-08-264-262/+189
| | | | | | | | | | | | Configuration of logging gets triggered twice. The first time it uses the CLI options when the application is started and second it uses the configuration file after that is read. The state of the logging needs to be saved from the first to the second time, so I created a class. Implements: blueprint logging-migration Change-Id: I7b8d1a3b6fd128e98cafd7c16009c7b694a52146
* Optimize log formattingTerryHowe2015-08-132-92/+60
| | | | | | | | | | | | There is no way to change the configuration variables we want printed in log messages, so format them in the constructor. This will save us from overridding the format method and a couple cpu cycles every log message. This change also moves the _LOG* variables into the formatter since they aren't really globally needed. Change-Id: I706e9db7da3daec20332f9d1533fe665f2739dea Implements: blueprint logging-migration
* Extract log level from configuration fileTerryHowe2015-08-132-10/+53
| | | | | | | | | | Extract log_level from configuration file if the level was not overridden by the command line option. The default command line option is 1 and there is no command line option to set the verbose_level to 1, so if it is 1, it has not be set. Change-Id: I1be04367c72f83c1181f92ca4c2c83165b66995c Implements: blueprint logging-migration
* Move options to log level out of shell.pyTerryHowe2015-08-133-12/+28
| | | | | | | | Move the conversion of command line options to log level out of shell.py. Change-Id: I86cb45a85cd63927aa1c87c1eed27542981df659 Implements: blueprint logging-migration
* Move set warnings filters to logging moduleTerryHowe2015-08-133-4/+20
| | | | | | | This is the first step in moving logging out of shell.py Change-Id: I3dcb4e17bb4687988ddf9b793ad1a308ef89b242 Implements: blueprint logging-migration
* Set up every time record log in fileDaisuke Fujita2015-08-084-15/+387
| | | | | | | | | This will allow users to record logs of all their commands into a predefined log file, in clouds.yaml. The log should have a format similar to that of oslo.log. Change-Id: I1b334bf429d575fc25809c9706fc0b11116be3f1 Implements: blueprint every-time-record-log-in-file
* Merge "Add configuration show command"Jenkins2015-07-244-0/+153
|\
| * Add configuration show commandTerryHowe2015-07-234-0/+153
| | | | | | | | | | | | | | | | | | | | | | | | Create a `configuration show` command that displays the current configuration of the CLI. Different configurations can be displayed using options such as --os-cloud. Passwords and tokens are redacted by default unless the --unmask option is specified. Closes-Bug: #1476729 Change-Id: I0792365d0c5fa526cd09c0ed88c6bb1e2cb813a7
* | Fix --os-auth-plugin in auth_with_unscoped_samlHugh Saunders2015-07-221-1/+1
| | | | | | | | | | | | | | | | The error message refers to --os-auth-plugin which is not a valid option. This patch changes that to --os-auth-type. Change-Id: I02ec0b7855131180bb8c674051930ebb51cb7303 Closes-Bug: #1477083
* | Merge "Make trustee/trustor/project searchable by ID"Jenkins2015-07-211-23/+8
|\ \
| * | Make trustee/trustor/project searchable by IDSteve Martinelli2015-07-201-23/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In the previous implementation, we were always including the domain argument, which caused a lookup by name for trustee/trustor and project. By excluding it when not necessary, we do a search by ID in find_resources. Change-Id: Id756aeab522b5dccb2dc6b31d137a28514b0fdf6 Closes-Bug: 1473298
* | | Merge "Add create and list for volume type v2"Jenkins2015-07-202-0/+226
|\ \ \
| * | | Add create and list for volume type v2chengkunye2015-07-202-0/+226
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Volume API v2 is missing create and list features. implements bp: volume-v2 Change-Id: I34a1ae440e9620b1c65546f4f43b369c8661250d
* | | | Merge "Format volume type properties when showing"Jenkins2015-07-203-2/+15
|\ \ \ \
| * | | | Format volume type properties when showingSteve Martinelli2015-07-193-2/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, the properties of a volume type are unformatted. Use the formatter to keep things consistent with the way properties are represented in OSC. Change-Id: I81c6bd3fdbc30568c269e501c740473b2a1ffb4e
* | | | | Merge "add image member commands for image API"Jenkins2015-07-203-0/+250
|\ \ \ \ \
| * | | | | add image member commands for image APIchengkunye2015-07-203-0/+250
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit adds the following commands: image project add image project remove Closes-Bug: 1402420 Change-Id: I07954e9fa43a3ad6078dd939ecedf9f038299e7b
* | | | | | Fix the way auth_type default value is overridenJavier Pena2015-07-201-3/+4
| |_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Commit 50f05448982b5fafd9d9a7783b639dd145090a0d to os-client-config removed the default values in the _defaults dict. This makes any call to cloud_config.set_default() before initializing the dict fail. The fix changes the way the auth_type default is overriden, by doing it when cloud_config.OpenStackConfig() is executed. Change-Id: If37d3ba303f01d4c77fd7c15a3cde9634534b64a Closes-bug: #1473921
* | | | | Rename type.py to volume_type.pySteve Martinelli2015-07-191-0/+0
| |/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | For Volume V1, we have a type.py file which should be renamed to volume_type.py (as it's named for V2). Change-Id: If860bbafe4a801d8b4fa06938eef20658c4fcc2c Closes-Bug: 1475958
* | | | Merge "Fix image save with API v2"Jenkins2015-07-191-1/+1
|\ \ \ \
| * | | | Fix image save with API v2David Moreau Simard2015-07-151-1/+1
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | Glanceclient v2 no longer expects the whole image object, just the image id. Change-Id: I8f34acfa50ca2d50eb7c9eb1dd5114c4621ad158 Closes-bug: #1475001
* | | | Merge "Removes trailing blank in trust show"Jenkins2015-07-192-4/+4
|\ \ \ \
| * | | | Removes trailing blank in trust showGilles Dubreuil2015-07-192-4/+4
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | Previously a blank character at the end of the roles value would remain Change-Id: I0961a5f9fb4b270a6055ee69898eadee315e416a Closes-Bug: 1474707
* | | | Merge "Add volume type show for volume v1"Jenkins2015-07-191-0/+24
|\ \ \ \
| * | | | Add volume type show for volume v1jiaxi2015-07-191-0/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | volume type show is missing for volume V1 API. Closes-Bug: #1475879 Change-Id: Ic8a0845ecec04146d536412463175f57ef6511ae
* | | | | Merge "--property should be required in `os unset` commands"Jenkins2015-07-193-0/+3
|\ \ \ \ \ | |_|_|_|/ |/| | | |
| * | | | --property should be required in `os unset` commandsjiaxi2015-07-193-0/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The three commands below openstack volume unset openstack snapshot unset openstack volume type unset Should have --property as a required argument, not optional. The reason is the command will not work without --property. Closes-Bug: #1475872 Change-Id: Ib9e29392472db38982cc2817af2dd5055f5a01ca
* | | | | More minor docs fixesTerryHowe2015-07-192-3/+3
|/ / / / | | | | | | | | | | | | Change-Id: Ia74b8e14bacb562d9bac29221f511acbab5296df
* | | | Making --property as required when openstack flavor unsetjiaxi2015-07-181-0/+1
|/ / / | | | | | | | | | | | | | | | | | | | | | When using openstack flavor unset, the --property is needed, so the --property should be a required parameter. Close-Bug: #1474237 Change-Id: Id6913e1b263eddeb5dd44c3edb957206b68f816c
* | | Merge "Remove unnecessary test extension mock"Jenkins2015-07-181-7/+0
|\ \ \
| * | | Remove unnecessary test extension mockTerryHowe2015-07-171-7/+0
| | | | | | | | | | | | | | | | | | | | | | | | The network extension test does not need this. Change-Id: I8dd2cad759a813d7df0476afa6c56f455ddff616
* | | | Merge "enhance tests for user v3"Jenkins2015-07-181-0/+33
|\ \ \ \
| * | | | enhance tests for user v3Guojian Shao2015-07-171-0/+33
| |/ / / | | | | | | | | | | | | | | | | Change-Id: Ib17ba0cd71068ba8d7e7665160daf3ae0735971b Related-Bug: #1475357
* | | | Support multiple volume qos deleteTerryHowe2015-07-174-92/+90
|/ / / | | | | | | | | | | | | | | | | | | Also make sure that we are using the singular qos_spec when we should and the plural qos_specs otherwise. Change-Id: If4bbeb6fc245d7d80dc3d7dccfe9f949e802653c
* | | Merge "Rename endpoint type to interface"Jenkins2015-07-1711-40/+36
|\ \ \
| * | | Rename endpoint type to interfaceTerryHowe2015-07-1511-40/+36
| | | | | | | | | | | | | | | | | | | | Change-Id: I4e21d09bc747e8210f4f79a1d6c4c7ccf2f25d1c Closes-Bug: #1454392
* | | | Merge "only return endpoints that have url"Jenkins2015-07-172-2/+44
|\ \ \ \
| * | | | only return endpoints that have urlGuojian Shao2015-07-162-2/+44
| | |/ / | |/| | | | | | | | | | | | | | Change-Id: I97a502252c0c377fce573e92b83c0122812f6f80 Closes-Bug: #1474656
* | | | add --project-domain option for user v3Guojian Shao2015-07-162-6/+70
| |_|/ |/| | | | | | | | | | | | | | | | | | | | | | | user v3 create/set only support --project option, we need --project-domain to prevent collisions between project names exist. Change-Id: I2d62e5b9bb6df4c5c5a9542514faf2e4365bb18b Closes-Bug: #1475357
* | | Fix the way we call find_resource when only using IDSteve Martinelli2015-07-161-6/+18
|/ / | | | | | | | | Change-Id: I6fb08edd5499767863e0e67f363bcd9fff3aea60 Closes-Bug: 1475127
* | Merge "Remove the --dhcp option to network list"Jenkins2015-07-142-85/+27
|\ \
| * | Remove the --dhcp option to network listTerryHowe2015-07-082-85/+27
| |/ | | | | | | | | | | | | | | | | | | The --dhcp option lists agents, not networks. This does not make a lot of sense. Another command should be created to list agents. BackwardsIncompatibleImpact Closes-Bug: #1472613 Change-Id: I5ecfe3fc046a07eb64a4dabd41dbd99de7c2215f
* | temporarily skip help testsSteve Martinelli2015-07-131-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | We're trying to change cliff for the better, by improving the UX of the help command. But cliff tests against the tip of OSC. Temporarily skip the tests that are failing and then update them once the fix for cliff has been merged. The patch in question: https://review.openstack.org/#/c/201258/ Change-Id: Ie1e7ddd66477ddf4a3bafedad12934e900ace48f
* | Merge "Fix interactive password prompt"Jenkins2015-07-131-1/+1
|\ \
| * | Fix interactive password promptTerryHowe2015-07-131-1/+1
| | | | | | | | | | | | | | | Change-Id: Ie0e7a9cd6016b5c646a111a76e8372e10602a25c Closes-Bug: #1473862
* | | Merge "Show which aggregate a hypervisor is member of"Jenkins2015-07-131-0/+23
|\ \ \ | |/ / |/| |
| * | Show which aggregate a hypervisor is member ofDavid Moreau Simard2015-07-021-0/+23
| | | | | | | | | | | | | | | | | | | | | | | | | | | This adds support for showing which host aggregates a hypervisor is member of, if any. It supports hypervisors with or without nova cells. Closes-bug: #1470875 Change-Id: I0cfe4f15fa8f8ba0be3295b79cd438998893114c
* | | Merge "Remove testing of cliff command line options"Jenkins2015-07-122-31/+4
|\ \ \
| * | | Remove testing of cliff command line optionsTerryHowe2015-07-082-31/+4
| | |/ | |/| | | | | | | | | | | | | These options are part of cliff, let cliff test them. Change-Id: I802c25ba80048607eef6909a21709dcda63231cc
* | | Merge "Fix address parsing for server ssh command"Jenkins2015-07-122-15/+91
|\ \ \
View Lorry Controller Status