[PATCH aeolus-cli] bug 786220 - Scope images to Pool Family

Scott Seago sseago at redhat.com
Fri Feb 17 05:46:58 UTC 2012


On 02/15/2012 05:04 PM, Matt Wagner wrote:
> On Wed, Feb 15, 2012 at 03:24:26PM -0500, Matt Wagner wrote:
>> On Wed, Feb 15, 2012 at 01:51:37PM -0500, Scott Seago wrote:
>>> diff --git a/lib/aeolus_cli/command/config_parser.rb b/lib/aeolus_cli/command/config_parser.rb
>>> index ef64349..7bc9515 100644
>>> --- a/lib/aeolus_cli/command/config_parser.rb
>>> +++ b/lib/aeolus_cli/command/config_parser.rb
>>> @@ -87,6 +87,9 @@ module Aeolus
>>>             opts.on('-i', '--images', 'Retrieve a list of images') do
>>>               @options[:subcommand] = :images
>>>             end
>>> +          opts.on('-E', '--environment ENVIRONMENT', 'Limit image list to environment') do |environment|
>>> +            @options[:environment] =  environment
>>> +          end
>>>             opts.on('-b', '--builds ID', 'Retrieve the builds of an image') do |id|
>>>               @options[:subcommand] = :builds
>>>               @options[:id] = id
>>> @@ -108,11 +111,15 @@ module Aeolus
>>>             opts.on('-a', '--accounts', 'Retrieve the values available for the --account parameter') do
>>>               @options[:subcommand] = :accounts
>>>             end
>>> +          opts.on('-c', '--environments', 'Retrieve the values available for the --environment parameter') do
>>> +            @options[:subcommand] = :environments
>>> +          end
>>>             opts.on( '-h', '--help', 'Get usage information for this command')
>>>
>>>             opts.separator ""
>>>             opts.separator "Examples:"
>>>             opts.separator "aeolus-image list --images                     # list available images"
>>> +          opts.separator "aeolus-image list --images --envionment $env   # list available images for an environment"
>> Nit (while I review the rest of the code and test), but ^^^^^^^^^^^^ is
>> missing an "r": "--envionment".
> Other than this nit, ACK to this patch
Fixed the nit -- I'll push tomorrow once my commit rights to aeolus-cli 
are sorted out.

Scott




More information about the aeolus-devel mailing list