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

Add image id filter in EC2 list images #77

Closed
wants to merge 1 commit into from
Closed

Add image id filter in EC2 list images #77

wants to merge 1 commit into from

Conversation

cpsaltis
Copy link

@cpsaltis cpsaltis commented Nov 8, 2012

This is very similar to what is being already done with ex_node_ids in list_nodes.

It helps you avoid the really large image list Amazon sends when you know exactly the ids you are looking for.

This works similarly to ex_node_ids in list_nodes of the same driver.
@cpsaltis
Copy link
Author

cpsaltis commented Dec 3, 2012

Haven't checked the contribution process before doing this, sorry. Would you like me to close this and do it again the proper way?

@techhat
Copy link
Contributor

techhat commented Feb 6, 2013

Has there been any traction on this? I could really use this.

@robromano
Copy link

I could also really use it. The commit from cpsaltis looked okay, can we merge it in?

@thatch45
Copy link

thatch45 commented Feb 6, 2013

Wow, this would be awesome, some of the checks I need to do in my code make aws a lot slower and this would speed it up a great deal!

@Kami
Copy link
Member

Kami commented Feb 7, 2013

As noted in LIBCLOUD-294 this patch has been merged to trunk. Thanks!

Sorry for the delay, but you've used an unofficial channel for contributing (official channel for Apache projects is JIRA). Until recently we didn't receive a notification when someone opened a pull request on Github so we've missed it.

P.S. Can you please close the pull request now? (we don't have access)

@cpsaltis
Copy link
Author

cpsaltis commented Feb 7, 2013

Thanks

@cpsaltis cpsaltis closed this Feb 7, 2013
@robromano
Copy link

Thanks everyone!

Kami pushed a commit to Kami/libcloud that referenced this pull request Oct 28, 2021
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.

None yet

5 participants