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

LIBCLOUD-750 MCP2 deploy support #587

Closed
wants to merge 10 commits into
base: trunk
from

Conversation

Projects
None yet
3 participants
@tonybaloney
Contributor

tonybaloney commented Sep 23, 2015

  • Updated the create_node methods to support the new API, and there every datacenter.
  • Updated all of the _to_x methods so that they don't lookup the locations for each iteration, significantly quickening the time to make queries for large accounts
  • Updated the unit tests to reflect more production data
  • The deploy returns the node id now
  • Added support for filtering networks, image, network domain and vlan by location

mingsheng36 and others added some commits Sep 17, 2015

@asfgit asfgit closed this in 09ec78d Sep 26, 2015

asfgit pushed a commit that referenced this pull request Sep 26, 2015

LIBCLOUD-750 fixed bug where is_started must be lowercase otherwise l…
…ive API throws error. Updated tests which weren't previously providing values. Added extra parameters to the list methods to filter by location. Optimised the way each of the calls gets the location by fetching it once for a collection instead of each iteration. Fixed the ID lookup of VLAN and Network Domain.

Closes #587

asfgit pushed a commit that referenced this pull request Sep 26, 2015

asfgit pushed a commit that referenced this pull request Sep 26, 2015

@Kami

This comment has been minimized.

Show comment
Hide comment
@Kami

Kami Sep 26, 2015

Member

Looks good to me. Merged into trunk, thanks.

Member

Kami commented Sep 26, 2015

Looks good to me. Merged into trunk, thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment