Merge pull request #130 from bheuvel/issue127/basic_provision_problem

Timothy van Zadelhoff · Timothy van Zadelhoff · commit 39eaf599b404 · 2016-02-02T13:56:23.000+01:00
Fog 1.32.0 / ACS requires comma separated string for lists
diff --git a/lib/vagrant-cloudstack/action/run_instance.rb b/lib/vagrant-cloudstack/action/run_instance.rb
@@ -129,7 +129,7 @@ def call(env)
             }
 
             options['network_ids'] = @network.id unless @network.id.nil?
-            options['security_group_ids'] = security_group_ids unless security_group_ids.nil?
+            options['security_group_ids'] = security_group_ids.join(',') unless security_group_ids.nil?
             options['project_id'] = project_id unless project_id.nil?
             options['key_name']   = keypair unless keypair.nil?
             options['name']       = hostname unless hostname.nil?

Original file line number	Diff line number	Diff line change
`@@ -129,7 +129,7 @@ def call(env)`
`129`	`129`	`}`
`130`	`130`
`131`	`131`	`options['network_ids'] = @network.id unless @network.id.nil?`
`132`		`- options['security_group_ids'] = security_group_ids unless security_group_ids.nil?`
	`132`	`+ options['security_group_ids'] = security_group_ids.join(',') unless security_group_ids.nil?`
`133`	`133`	`options['project_id'] = project_id unless project_id.nil?`
`134`	`134`	`options['key_name'] = keypair unless keypair.nil?`
`135`	`135`	`options['name'] = hostname unless hostname.nil?`