search for: svc_vm_actions

...Instance variables + # * <tt>@vms</tt> VMs identified by +vm_ids+ + # === Required permissions + # permission is action-specific as determined by + # <tt>VmTask.action_privilege(@action)</tt> + # This method can be called to initiate an action on one or more vms + def svc_vm_actions(vm_ids, vm_action, action_args) + vm_ids = [vm_ids] unless vm_ids.is_a?(Array) + successful_vms = [] + failed_vms = {} + vm_ids.each do |vm_id| + begin + successful_vms << svc_vm_action(vm_id, vm_action, action_args) + rescue Exception => ex + failed_vms...

The following two patches should be applied in the order: * [PATCH server] Add svc_vm_actions method to VmService. * [PATCH server] Cloud UI layer to initiate actions on vms. The service layer patch should work fine on its own (and not break anything), but cloud patch relies on service being there.

This resend includes the rebase, but fixes a problem I just noticed with nil/empty checks.

This is a resend of the 2-patch series from a week or so ago, needed to be rebased due to some commits on next

...mServiceTest < ActiveSupport::TestCase # (should be the same message if one or more, so we have one test for # each of those cases) def test_success_message_from_single_vm - assert_equal("shutdown_vm successful.", + assert_equal("shutdown_vm submitted.", svc_vm_actions(vms(:production_mysqld_vm).id, 'shutdown_vm', nil)) end @@ -92,7 +92,7 @@ class VmServiceTest < ActiveSupport::TestCase # (should be the same message if one or more, so we have one test for # each of those cases) def test_success_message_for_multiple_vms - assert_equal(&...

Forwarding to the list for Jay or Scott to have a look. Ignazio, do you see any more details (traceback or something) in the rails log? -------------- next part -------------- An embedded message was scrubbed... From: ignazio.cassano at provincia.torino.it Subject: Re: [Ovirt-devel] [PATCH node-image] keep NIC and HBA firmwares Date: Wed, 16 Sep 2009 16:48:47 +0200 Size: 4031 URL:

...index 00cf982..8930a4b 100644 --- a/src/app/services/vm_resource_pool_service.rb +++ b/src/app/services/vm_resource_pool_service.rb @@ -73,7 +73,6 @@ module VmResourcePoolService # permission is action-specific as determined by # <tt>VmTask.action_privilege(@action)</tt> def svc_vm_actions(pool_id, vm_action, vm_ids) - # from before_filter @pool = VmResourcePool.find(pool_id) @parent = @pool.parent @action = vm_action diff --git a/src/app/services/vm_service.rb b/src/app/services/vm_service.rb index fb58b51..4c29bcf 100644 --- a/src/app/services/vm_service.rb +++ b/...

This patch creates the shell for much of what the cloud ui will become. It is focused on layout (lightly fedora themed), accessibility, and 'bookmarkability'. This layout uses no javascript (that will be used to enhance interactivity in a future patch). It also introduces the idea of using a simple table rather than the more conplex js/json we use in the admin side. Plan here is to