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

Fixed the loadbalancer method for cloudstack incl tests #434

Closed
wants to merge 1 commit into
base: trunk
from

Conversation

Projects
None yet
3 participants
@jeroendekorte
Contributor

jeroendekorte commented Jan 16, 2015

Load balancer:
Updated the loadbalancer for Cloudstack since this was not working.

Compute:
Updated the list_images to check if a image size is returned or not. Some templates/images don't have a size.

@Kami

This comment has been minimized.

Show comment
Hide comment
@Kami

Kami Jan 17, 2015

Member

/cc @runseb

Member

Kami commented Jan 17, 2015

/cc @runseb

@sebgoa

This comment has been minimized.

Show comment
Hide comment
@sebgoa

sebgoa Jan 17, 2015

Member

yep, saw it. looks good to me. I can merge it on wednesday..

Member

sebgoa commented Jan 17, 2015

yep, saw it. looks good to me. I can merge it on wednesday..

@sebgoa

This comment has been minimized.

Show comment
Hide comment
@sebgoa

sebgoa Jan 21, 2015

Member

LGTM, I applied it. Forgot to close the pull request. @jeroendekorte thanks, can you close the PR.

Member

sebgoa commented Jan 21, 2015

LGTM, I applied it. Forgot to close the pull request. @jeroendekorte thanks, can you close the PR.

@sebgoa

This comment has been minimized.

Show comment
Hide comment
@sebgoa

sebgoa Feb 27, 2015

Member

@jeroendekorte can you close this. I applied your patch but cannot close the pr on github.

Member

sebgoa commented Feb 27, 2015

@jeroendekorte can you close this. I applied your patch but cannot close the pr on github.

@jeroendekorte

This comment has been minimized.

Show comment
Hide comment
@jeroendekorte

jeroendekorte Feb 27, 2015

Contributor

Done, thanks guys!

Contributor

jeroendekorte commented Feb 27, 2015

Done, thanks guys!

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