Skip to content
This repository has been archived by the owner on Mar 4, 2024. It is now read-only.

ec2/ec2test: handle "tag:" filter for volumes #56

Merged
merged 1 commit into from Jul 22, 2015

Conversation

axw
Copy link

@axw axw commented Jul 22, 2015

Handle the "tag:key=value" type filter for
volumes. This is required for Juju to be able
to list volumes belonging to an environment.


filter := ec2.NewFilter()
filter.Add("tag:key", "value")
resp, err := s.ec2.Volumes(nil, filter)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

maybe also a check that the volume fails to match the filter before the tags are added

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@wallyworld
Copy link
Contributor

LGTM

Handle the "tag:key=value" type filter for
volumes. This is required for Juju to be able
to list volumes belonging to an environment.
@axw axw force-pushed the ec2test-volumes-filter-tag branch from b4e3d1d to 9728e8c Compare July 22, 2015 05:04
axw added a commit that referenced this pull request Jul 22, 2015
ec2/ec2test: handle "tag:" filter for volumes
@axw axw merged commit 6b2122e into go-amz:v3 Jul 22, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants