Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Handle a list of strings from a dict in text output #516

Merged
merged 1 commit into from
Nov 27, 2013

Conversation

jamesls
Copy link
Member

@jamesls jamesls commented Nov 27, 2013

This situation comes up in commands such as
aws elb describe-load-balancers. The output
now looks as expected (note the "us-west-2b")

Before:

AVAILABILITYZONES   us-west-2a
HEALTHCHECK 10  30  HTTP:80/index.html  5   2
LISTENER    80  HTTP    80  HTTP
SOURCESECURITYGROUP amazon-elb-sg   amazon-elb
LOADBALANCERDESCRIPTIONS        a   b   c   d
us-west-2b

Now:

AVAILABILITYZONES   us-west-2a
HEALTHCHECK 10  30  HTTP:80/index.html  5   2
LISTENER    80  HTTP    80  HTTP
SOURCESECURITYGROUP amazon-elb-sg   amazon-elb
LOADBALANCERDESCRIPTIONS        a   b   c   d
AVAILABILITYZONES   us-west-2b

See: https://forums.aws.amazon.com/thread.jspa?threadID=140646

This situation comes up in commands such as
`aws ec2 describe-load-balancers`.  The output
now looks as expected:

Now:
AVAILABILITYZONES	us-west-2a
HEALTHCHECK	10	30	HTTP:80/index.html	5	2
LISTENER	80	HTTP	80	HTTP
SOURCESECURITYGROUP	amazon-elb-sg	amazon-elb
LOADBALANCERDESCRIPTIONS		a	b	c	d
AVAILABILITYZONES	us-west-2b

Before:

AVAILABILITYZONES	us-west-2a
HEALTHCHECK	10	30	HTTP:80/index.html	5	2
LISTENER	80	HTTP	80	HTTP
SOURCESECURITYGROUP	amazon-elb-sg	amazon-elb
LOADBALANCERDESCRIPTIONS		a	b	c	d
us-west-2b
@garnaat
Copy link
Contributor

garnaat commented Nov 27, 2013

LGTM

@jamesls jamesls merged commit 5ce0c9f into aws:develop Nov 27, 2013
@jamesls jamesls deleted the text-output-list-string branch June 23, 2014 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants