On Wed, Nov 11, 2015 at 1:41 PM, Daniel Depaoli < daniel.depa...@create-net.org> wrote:
> Hi all. > I'm starting to resolve the todo at these line[1]. To solve this I think > to hardcoded the role in the file, for example: > > *$swift_proxies = get_nodes_hash_by_roles($network_metadata, > ['primary-swift-proxy', 'swift-proxy']) ? {* > * true => get_nodes_hash_by_roles($network_metadata, > ['primary-swift-proxy', 'swift-proxy']),* > * false => get_nodes_hash_by_roles($network_metadata, > ['primary-controller', 'controller']),* > *}* > > > Is this the right way or do you suggest something more clean? > Function *get_nodes_hash_by_roles returns *hash*. *Provided above switch shouldn't work. But this way is true. IMHO list of roles name should be constructed before. /sv
__________________________________________________________________________ OpenStack Development Mailing List (not for usage questions) Unsubscribe: openstack-dev-requ...@lists.openstack.org?subject:unsubscribe http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev