get-vpc-id-by-name

[fdavis]

we have a lot of accounts and vpcs so its helpful to be able to grab a vpc-id for other configs/placement

[tchia04]

How about adding a get-vpc-name-by-id like this?

aws ec2 describe-vpcs --filter "Name=vpc-id,Values=${1}" --query 'Vpcs[*].Tags[].Value' --output text

[fdavis]

Added, I like that it matches a partial vpc-id as well

[kyleknap]

Thanks for the PR! I like both aliases. I just had a feedback comment for both of the aliases.

[kyleknap]

I think you should be able to simplify your --query by just making it: --query 'Vpcs[].[VpcId] and should be the same result

[virgilwashere]

@kyleknap the alias as submitted with curly braces, seems correct.

$ aws ec2 describe-vpcs --filter "Name=tag:Name,Values=vpc_name" --query 'Vpcs[].[id:VpcId]' --output text

Bad value for --query Vpcs[].[id:VpcId]: Expecting: comma, got: colon: Parse error at column 10, token ":" (COLON), for expression:
"Vpcs[].[id:VpcId]"
           ^
$ aws ec2 describe-vpcs --filter "Name=tag:Name,Values=vpc_name" --query 'Vpcs[].{id:VpcId}' --output text
vpc-01234567890123456

However, @fdavis you can also do this:

Suggested change

	aws ec2 describe-vpcs --filter "Name=tag:Name,Values=${1}" --query 'Vpcs[*].{id:VpcId}' --output text
	aws ec2 describe-vpcs --query "Vpcs[?Tags[?Key==\`Name\`]|[?Value==\`${1:?Need a VPC Name}\`]].VpcId" --output text

[kyleknap]

This looks like we are including values of all tags. I think we should only be including the Name tag in what is returned.

[bgiorgini]

I would also recommend to include only the tag Name. +1 to include these 2 in the aliases.

[virgilwashere]

Suggested change

	aws ec2 describe-vpcs --filter "Name=vpc-id,Values=${1}" --query 'Vpcs[*].Tags[].Value' --output text
	aws ec2 describe-vpcs --vpc-ids "${1:?Need a VPCId}" --query 'Vpcs[].Tags[?Key==`Name`].Value' --output text