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

[LIBCLOUD-555] Added support for openstack compute availability zone #295

Closed

Conversation

syndicut
Copy link
Contributor

@syndicut syndicut commented May 15, 2014

syndicut added a commit to syndicut/salt that referenced this issue May 15, 2014
@@ -1318,6 +1318,9 @@ def _create_args_to_params(self, node, **kwargs):
{}))
}

if 'availability_zone' in kwargs:
server_params['availability_zone'] = kwargs['availability_zone']
Copy link
Member

@Kami Kami May 16, 2014

Choose a reason for hiding this comment

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

This is an extension param so please prefix it with ex_:

 if 'ex_availability_zone' in kwargs:
    server_params['availability_zone'] = kwargs['ex_availability_zone']

Copy link
Member

@Kami Kami May 16, 2014

Choose a reason for hiding this comment

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

Copy link
Contributor Author

@syndicut syndicut May 16, 2014

Choose a reason for hiding this comment

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

Done!

@asfgit asfgit closed this in 68c12de May 16, 2014
@Kami
Copy link
Member

Kami commented May 16, 2014

Merged into trunk.

Thanks.

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

2 participants