edit vm

https://bugzilla.redhat.com/show_bug.cgi?id=1428250

update custom identifier

app/controllers/api/vms_controller.rb

@@ -8,6 +8,8 @@ class VmsController < BaseController
     include Subcollections::Software
     include Subcollections::Snapshots
 
+    VALID_EDIT_ATTRS = %w(custom_1 description children parent).freeze
+
     def start_resource(type, id = nil, _data = nil)
       raise BadRequestError, "Must specify an id for starting a #{type} resource" unless id
 
@@ -86,6 +88,18 @@ def shelve_offload_resource(type, id = nil, _data = nil)
       end
     end
 
+    def edit_resource(type, id, data)
+      attrs = validate_edit_data(data)
+      parent, children = build_parent_children(data)
+      resource_search(id, type, collection_class(type)).tap do |vm|
+        vm.update_attributes!(attrs)
+        vm.replace_children(children)
+        vm.set_parent(parent)
+      end
+    rescue => err
+      raise BadRequestError, "Cannot edit VM - #{err}"
+    end
+
     def delete_resource(type, id = nil, _data = nil)
       raise BadRequestError, "Must specify an id for deleting a #{type} resource" unless id
 
@@ -231,6 +245,30 @@ def request_console_resource(type, id = nil, data = nil)
 
     private
 
+    def validate_edit_data(data)
+      invalid_keys = data.keys - VALID_EDIT_ATTRS
+      raise BadRequestError, "Cannot edit values #{invalid_keys.join(', ')}" if invalid_keys.present?
+      data.dup.except('parent', 'children')
+    end
+
+    def build_parent_children(data)
+      children = if data.key?('children')
+                   data['children'].collect do |child|
+                     child_collection, child_id = parse_href(child['href'])
+                     resource_search(child_id, child_collection, collection_class(child_collection))
+                   end
+                 else
+                   []
+                 end
+
+      parent = if data.key?('parent')
+                 parent_collection, parent_id = parse_href(data.fetch('parent', 'href'))
+                 resource_search(parent_id, parent_collection, collection_class(parent_collection))
+               end
+
+      [parent, children]
+    end
+
     def vm_ident(vm)
       "VM id:#{vm.id} name:'#{vm.name}'"
     end

config/api.yml

@@ -2265,7 +2265,6 @@
         :disabled: true
       - :name: edit
         :identifier: vm_edit
-        :disabled: true
       - :name: add_lifecycle_event
         :identifier: vm_edit
       - :name: add_event
@@ -2313,7 +2312,6 @@
       :post:
       - :name: edit
         :identifier: vm_edit
-        :disabled: true
       - :name: add_lifecycle_event
         :identifier: vm_edit
       - :name: add_event

spec/requests/api/vms_spec.rb

@@ -28,6 +28,75 @@ def update_raw_power_state(state, *vms)
     vms.each { |vm| vm.update_attributes!(:raw_power_state => state) }
   end
 
+  context 'Vm edit' do
+    let(:new_vms) { FactoryGirl.create_list(:vm_openstack, 2) }
+
+    before do
+      vm.set_child(vm_openstack)
+      vm.set_parent(vm_openstack1)
+    end
+
+    it 'cannot edit a VM without an appropriate role' do
+      api_basic_authorize
+
+      run_post(vms_url(vm.id), :action => 'edit')
+
+      expect(response).to have_http_status(:forbidden)
+    end
+
+    it 'can edit a VM with an appropriate role' do
+      api_basic_authorize collection_action_identifier(:vms, :edit)
+      children = new_vms.collect do |vm|
+        { 'href' => vms_url(vm.id) }
+      end
+
+      run_post(vms_url(vm.id), :action      => 'edit',
+                               :description => 'bar',
+                               :children    => children,
+                               :custom_1    => 'foobar',
+                               :parent      => vms_url(vm_openstack2.id))
+
+      expected = {
+        'description' => 'bar'
+      }
+      expect(response).to have_http_status(:ok)
+      expect(response.parsed_body).to include(expected)
+      expect(vm.reload.children).to match_array(new_vms)
+      expect(vm.parent).to eq(vm_openstack2)
+      expect(vm.custom_1).to eq('foobar')
+    end
+
+    it 'only allows edit of custom_1, description, parent, and children' do
+      api_basic_authorize collection_action_identifier(:vms, :edit)
+
+      run_post(vms_url(vm.id), :action => 'edit', :name => 'foo', :autostart => true, :power_state => 'off')
+
+      expected = {
+        'error' => a_hash_including(
+          'kind'    => 'bad_request',
+          'message' => 'Cannot edit VM - Cannot edit values name, autostart, power_state'
+        )
+      }
+      expect(response).to have_http_status(:bad_request)
+      expect(response.parsed_body).to include(expected)
+    end
+
+    it 'can edit multiple vms' do
+      api_basic_authorize collection_action_identifier(:vms, :edit)
+
+      run_post(vms_url, :action => 'edit', :resources => [{ :id => vm.id, :description => 'foo' }, { :id => vm_openstack.id, :description => 'bar'}])
+
+      expected = {
+        'results' => [
+          a_hash_including('description' => 'foo'),
+          a_hash_including('description' => 'bar')
+        ]
+      }
+      expect(response).to have_http_status(:ok)
+      expect(response.parsed_body).to include(expected)
+    end
+  end
+
   context "Vm accounts subcollection" do
     let(:acct1) { FactoryGirl.create(:account, :vm_or_template_id => vm.id, :name => "John") }
     let(:acct2) { FactoryGirl.create(:account, :vm_or_template_id => vm.id, :name => "Jane") }