Bug #27719
closedAdding a 2nd disk type of storage_pod/datastore_cluster fails to create vm
Description
Error:
Failed to create a compute test (VMware) instance tammy-arnzen.toledo.satellite.lab.eng.rdu2.redhat.com: wrong number of arguments (given 2, expected 1)
Traceback:
```ruby
2019-08-27T16:40:18 [W|app|09a39f98] Failed to create a compute test (VMware) instance tammy-arnzen.toledo.satellite.lab.eng.rdu2.redhat.com: wrong number of arguments (given 2, expected 1)
ArgumentError: wrong number of arguments (given 2, expected 1)
/opt/theforeman/tfm/root/usr/share/gems/gems/fog-vsphere-3.2.0/lib/fog/vsphere/requests/compute/create_vm.rb:75:in `group_disks_by_storage_pod'
/opt/theforeman/tfm/root/usr/share/gems/gems/fog-vsphere-3.2.0/lib/fog/vsphere/requests/compute/create_vm.rb:55:in `create_vm_on_storage_pod'
/opt/theforeman/tfm/root/usr/share/gems/gems/fog-vsphere-3.2.0/lib/fog/vsphere/requests/compute/create_vm.rb:36:in `create_vm'
/opt/theforeman/tfm/root/usr/share/gems/gems/fog-vsphere-3.2.0/lib/fog/vsphere/models/compute/server.rb:302:in `save'
/usr/share/foreman/app/models/compute_resources/foreman/model/vmware.rb:484:in `create_vm'
/usr/share/foreman/app/models/concerns/orchestration/compute.rb:91:in `setCompute'
```
Files
Updated by Chris Roberts about 5 years ago
Broke in Katello 3.13/Foreman 1.23 as well
Updated by Chris Roberts about 5 years ago
- Status changed from Ready For Testing to Resolved
- Target version changed from 1.24.0 to 1.23.1
- Triaged changed from No to Yes
- Bugzilla link set to 1746175
Updated by The Foreman Bot about 5 years ago
- Pull request https://github.com/theforeman/foreman/pull/7004 added
Updated by Ondřej Ezr about 5 years ago
- Status changed from Resolved to Ready For Testing
- Pull request deleted (
https://github.com/fog/fog-vsphere/pull/234)
I've released new version. Otherwise we wouldn't have fixed the upstream.
Updated by Tomer Brisker about 5 years ago
- Found in Releases 1.23.0 added
- Found in Releases deleted (
1.24.0)
Updated by Ondřej Ezr almost 5 years ago
- Status changed from Ready For Testing to Closed
Applied in changeset 6c6e15dafb71a1afb67eca4511a1541fda7b9378.
Updated by The Foreman Bot almost 5 years ago
- Pull request https://github.com/theforeman/foreman/pull/7213 added