foreman_acd 0.0.6 → 0.6.0

data/app/controllers/foreman_acd/app_definitions_controller.rb

@@ -6,33 +6,42 @@ module ForemanAcd
     include Foreman::Controller::AutoCompleteSearch
     include ::ForemanAcd::Concerns::AppDefinitionParameters
 
-    before_action :find_resource, :only => [:edit, :update, :destroy]
+    before_action :find_resource, :only => [:edit, :update, :destroy, :export]
+    before_action :read_hostgroups, :only => [:edit, :new, :import]
+    before_action :read_ansible_playbooks, :only => [:edit, :new]
+    before_action :handle_file_upload, :only => [:create]
 
     def index
       @app_definitions = resource_base.search_for(params[:search], :order => params[:order]).paginate(:page => params[:page])
     end
 
+    def read_ansible_playbooks
+      # Only use ansible playbooks for which the user pressed import group vars once.
+      @ansible_playbooks = AnsiblePlaybook.where.not(:vars => nil).map { |elem| { elem.id => elem.name } }.reduce({}) { |h, v| h.merge v }
+    end
+
     def read_hostgroups
       @hostgroups = Hostgroup.all.map { |elem| { elem.id => elem.name } }.reduce({}) { |h, v| h.merge v }
     end
 
     def new
-      read_hostgroups
       @app_definition = AppDefinition.new
     end
 
     def create
       @app_definition = AppDefinition.new(app_definition_params)
-      if @app_definition.save
-        process_success
-      else
-        process_error
+      begin
+        if @app_definition.save!
+          process_success
+        else
+          process_error
+        end
+      rescue StandardError, ValidationError => e
+        redirect_to new_app_definition_path, :flash => { :error => _(e.message) }
       end
     end
 
-    def edit
-      read_hostgroups
-    end
+    def edit; end
 
     def update
       if @app_definition.update(app_definition_params)
@@ -49,5 +58,33 @@ module ForemanAcd
         process_error
       end
     end
+
+    def action_permission
+      case params[:action]
+      when 'export'
+        :export
+      when 'import'
+        :import
+      else
+        super
+      end
+    end
+
+    def import
+      @app_definition = AppDefinition.new
+    end
+
+    def export
+      filename = "#{@app_definition.name}.yaml"
+      data = JSON.parse(@app_definition.services).to_yaml
+      send_data data, :type => 'text/yaml', :disposition => 'attachment', :filename => filename
+    end
+
+    private
+
+    def handle_file_upload
+      return unless params[:foreman_acd_app_definition] && (raw_file = params[:foreman_acd_app_definition][:app_definition_file])
+      params[:foreman_acd_app_definition][:services] = YAML.load_file(raw_file.tempfile).to_json
+    end
   end
 end

data/app/controllers/foreman_acd/app_instances_controller.rb

@@ -6,45 +6,53 @@ module ForemanAcd
     include Foreman::Controller::AutoCompleteSearch
     include ::ForemanAcd::Concerns::AppInstanceParameters
 
-    before_action :find_resource, :only => [:edit, :update, :destroy, :deploy]
+    before_action :find_resource, :only => [:edit, :update, :destroy_with_hosts, :deploy, :report]
+    before_action :read_applications, :only => [:new, :edit]
+    before_action :find_taxonomy
+    helper_method :collect_hosts_data
 
     def index
-      @app_instances = resource_base.search_for(params[:search], :order => params[:order]).paginate(:page => params[:page])
-    end
-
-    def read_applications
-      @applications = ForemanAcd::AppDefinition.all.map { |elem| { elem.id => elem.name } }.reduce({}) { |h, v| h.merge v }
+      @app_instances = resource_base.where(:organization => @organization).
+                       where(:location => @location).
+                       search_for(params[:search], :order => params[:order]).paginate(:page => params[:page])
     end
 
     def new
       @app_instance = AppInstance.new
-      read_applications
+      @app_instance.organization = @organization
+      @app_instance.location = @location
     end
 
     def create
       @app_instance = AppInstance.new(app_instance_params)
-      if @app_instance.save
-        process_success
-      else
-        process_error
+      begin
+        if @app_instance.save!
+          app_instance_has_foreman_hosts
+          process_success
+        else
+          process_error
+        end
+      rescue StandardError, ValidationError => e
+        redirect_to new_app_instance_path, :flash => { :error => _(e.message) }
       end
     end
 
-    def edit
-      read_applications
-    end
+    def edit; end
 
     def update
       if @app_instance.update(app_instance_params)
+        app_instance_has_foreman_hosts
         process_success
       else
         process_error
       end
     end
 
-    def destroy
+    def destroy_with_hosts
+      @app_instance = AppInstance.find(params[:id])
+      @app_instance.clean_hosts_by_id(params[:foreman_host_ids]) if params[:foreman_host_ids]
       if @app_instance.destroy
-        process_success
+        redirect_to app_instances_path, :flash => { :success => _('Successfully deleted %s') % @app_instance }
       else
         process_error
       end
@@ -54,105 +62,103 @@ module ForemanAcd
       case params[:action]
       when 'deploy'
         :deploy
+      when 'report'
+        :report
+      when 'destroy_with_hosts'
+        :destroy
       else
         super
       end
     end
 
     def deploy
-      params = host_attributes(set_host_params)
+      @app_instance.clean_all_hosts if params[:delete_hosts]
 
-      # Print to log for debugging purposes
-      logger.info("Host creation parameters:\n#{params}\n")
-
-      @host = Host.new(params)
-      apply_compute_profile(@host)
-      @host.suggest_default_pxe_loader
-      @host.save
-      success _('Successfully initiated host creation')
+      logger.info('Run async foreman task to deploy hosts')
+      async_task = ForemanTasks.async_task(::Actions::ForemanAcd::DeployAllHosts, @app_instance)
+      @app_instance.update!(:last_deploy_task => async_task)
+      process_success(:success_msg => _('Started task to deploy hosts for %s') % @app_instance)
     rescue StandardError => e
-      logger.error("Failed to initiate host creation: #{e.backtrace.join($INPUT_RECORD_SEPARATOR)}")
-    ensure
-      redirect_to app_instances_path
+      error_msg = "Error happend while deploying hosts of #{@app_instance}: #{e.message}"
+      logger.error("#{error_msg} - #{e.class}\n#{e.backtrace.join($INPUT_RECORD_SEPARATOR)}")
+      process_error :error_msg => error_msg
     end
 
-    private
-
-    # Copied from foreman/app/controllers/api/v2/hosts_controller.rb
-    def apply_compute_profile(host)
-      host.apply_compute_profile(InterfaceMerge.new(:merge_compute_attributes => true))
-      host.apply_compute_profile(ComputeAttributeMerge.new)
+    def report
+      @report_hosts = collect_host_report_data
+      logger.debug("app instance host details: #{@report_hosts.inspect}")
     end
 
-    # Copied from foreman/app/controllers/api/v2/hosts_controller.rb
-    def host_attributes(params, host = nil)
-      return {} if params.nil?
-
-      params = params.deep_clone
-      if params[:interfaces_attributes]
-        # handle both hash and array styles of nested attributes
-        params[:interfaces_attributes] = params[:interfaces_attributes].values if params[:interfaces_attributes].is_a?(Hash) || params[:interfaces_attributes].is_a?(ActionController::Parameters)
-        # map interface types
-        params[:interfaces_attributes] = params[:interfaces_attributes].map do |nic_attr|
-          interface_attributes(nic_attr, :allow_nil_type => host.nil?)
+    def app_instance_has_foreman_hosts
+      hosts = JSON.parse(@app_instance.hosts)
+      hosts.each do |h|
+        if @app_instance.foreman_hosts.where(:hostname => h['hostname']).exists?
+          @app_instance.foreman_hosts.where(:hostname => h['hostname']).
+            update(:service => h['service'], :description => h['description'],
+                   :foremanParameters => JSON.dump(h['foremanParameters']), :ansibleParameters => JSON.dump(h['ansibleParameters']))
+        else
+          @app_instance.foreman_hosts.create(:hostname => h['hostname'], :service => h['service'], :description => h['description'],
+                                             :foremanParameters => JSON.dump(h['foremanParameters']), :ansibleParameters => JSON.dump(h['ansibleParameters']))
         end
       end
-      params = host.apply_inherited_attributes(params) if host
-      params
-    end
 
-    # Copied from foreman/app/controllers/api/v2/hosts_controller.rb
-    def interface_attributes(params, allow_nil_type: false)
-      params[:type] = InterfaceTypeMapper.map(params[:type]) if params.key?(:type) || allow_nil_type
-      params
+      # Delete record if json hosts are deleted
+      deleted_json_hosts = @app_instance.foreman_hosts.pluck('hostname') - hosts.pluck('hostname')
+      @app_instance.foreman_hosts.where(:hostname => deleted_json_hosts).destroy_all if deleted_json_hosts
     end
 
-    def hardcoded_params
-      result = {}
-      result['managed'] = true
-      result['enabled'] = true
-      result['build'] = true
-      result['compute_attributes'] = { 'start' => '1' }
-      result['host_parameters_attributes'] = []
-      result
+    def collect_hosts_data
+      hosts_data = []
+      @app_instance.foreman_hosts.each do |h|
+        hosts_data << {
+          :id => h.id,
+          :hostname => h.hostname,
+          :service => h.service,
+          :description => h.description,
+          :foremanParameters => JSON.parse(h.foremanParameters),
+          :ansibleParameters => JSON.parse(h.ansibleParameters)
+        }
+      end
+      hosts_data
     end
 
-    def set_host_params
-      result = hardcoded_params
-      result['hostgroup_id'] = @app_instance.app_definition.hostgroup_id
-      JSON.parse(@app_instance.parameters).each do |param|
-        case param['type']
-
-        when 'computeprofile'
-          result['compute_profile_id'] = param['value']
-
-        when 'domain'
-          result['domain_id'] = param['value']
-
-        when 'hostname'
-          result['name'] = param['value']
-
-        when 'hostparam'
-          result['host_parameters_attributes'].push(:name => param['name'], :value => param['value'])
-
-        when 'ip'
-          result['ip'] = param['value']
-
-        when 'lifecycleenv'
-          result['content_facet_attributes'] = { 'lifecycle_environment_id' => param['value'] }
+    private
 
-        when 'ptable'
-          result['ptable_id'] = param['value']
+    def find_taxonomy
+      @organization = Organization.current
+      redirect_to '/select_organization?toState=' + request.path and return unless @organization
 
-        when 'puppetenv'
-          result['environment_id'] = param['value']
+      @location = Location.current
+      redirect_to root_path, :error => 'Select a location to show App Instances' and return unless @location
+    end
 
-        when 'password'
-          result['root_pass'] = param['value']
+    def read_applications
+      @applications = AppDefinition.all.map { |elem| { elem.id => elem.name } }.reduce({}) { |h, v| h.merge v }
+    end
 
+    def collect_host_report_data
+      report_data = []
+
+      @app_instance.foreman_hosts.each do |foreman_host|
+        a_host = {
+          :id => nil,
+          :name => foreman_host.hostname,
+          :build => nil,
+          :hostUrl => nil,
+          :progress_report => foreman_host.last_progress_report.empty? ? [] : JSON.parse(foreman_host.last_progress_report)
+        }
+
+        if foreman_host.host.present?
+          a_host.update({
+                          :id => foreman_host.host.id,
+                          :build => foreman_host.host.build,
+                          :hostUrl => host_path(foreman_host.host),
+                          :powerStatusUrl => power_api_host_path(foreman_host.host)
+                        })
         end
+        report_data << a_host
       end
-      result
+      report_data
     end
   end
 end

data/app/controllers/foreman_acd/concerns/ansible_playbook_parameters.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+module ForemanAcd
+  module Concerns
+    # Parameters for AnsiblePlaybooks
+    module AnsiblePlaybookParameters
+      extend ActiveSupport::Concern
+
+      class_methods do
+        def ansible_playbook_params_filter
+          Foreman::ParameterFilter.new(::ForemanAcd::AnsiblePlaybook).tap do |filter|
+            filter.permit(:name, :description, :scm_type, :path, :playfile, :location_ids => [], :organization_ids => [])
+          end
+        end
+      end
+
+      def ansible_playbook_params
+        param_name = parameter_filter_context.api? ? 'ansible_playbook' : 'foreman_acd_ansible_playbook'
+        self.class.ansible_playbook_params_filter.filter_params(params, parameter_filter_context, param_name)
+      end
+    end
+  end
+end

data/app/controllers/foreman_acd/concerns/app_definition_parameters.rb

@@ -9,7 +9,7 @@ module ForemanAcd
       class_methods do
         def app_definition_params_filter
           Foreman::ParameterFilter.new(::ForemanAcd::AppDefinition).tap do |filter|
-            filter.permit(:name, :description, :hostgroup_id, :parameters)
+            filter.permit(:name, :description, :acd_ansible_playbook_id, :services, :ansible_vars_all, :location_ids => [], :organization_ids => [])
           end
         end
       end

data/app/controllers/foreman_acd/concerns/app_instance_parameters.rb

@@ -9,7 +9,7 @@ module ForemanAcd
       class_methods do
         def app_instance_params_filter
           Foreman::ParameterFilter.new(::ForemanAcd::AppInstance).tap do |filter|
-            filter.permit(:name, :app_definition_id, :description, :parameters)
+            filter.permit(:name, :app_definition_id, :description, :hosts, :ansible_vars_all, :organization_id, :location_id)
           end
         end
       end

data/app/controllers/foreman_acd/remote_execution_controller.rb

@@ -0,0 +1,46 @@
+# frozen_string_literal: true
+
+module ForemanAcd
+  # Class to run remote execution jobs
+  class RemoteExecutionController < JobInvocationsController
+    def new
+      jobs = init_configuration
+      @composer = jobs.first
+    end
+
+    def create
+      customize_first = params[:customize] || false
+      jobs = init_configuration
+
+      if jobs.count == 1 && customize_first == false
+        @composer = jobs.first
+        @composer.trigger!
+        redirect_to job_invocation_path(@composer.job_invocation)
+      elsif customize_first == false
+        jobs.each(&:trigger)
+        redirect_to job_invocations_path
+      else
+        # redirect to the job itself if we want to customize the job
+        @composer = jobs.first
+        render :action => 'new'
+      end
+    end
+
+    # to overcome the isolated namespace engine difficulties with paths
+    helper Rails.application.routes.url_helpers
+    def _routes
+      Rails.application.routes
+    end
+
+    private
+
+    def init_configuration
+      app_instance = ForemanAcd::AppInstance.find_by(:id => params[:id])
+      app_configurator = ForemanAcd::AppConfigurator.new(app_instance)
+
+      jobs = app_configurator.configure
+      logger.debug("Creating #{jobs.count} job(s) to configure the app #{app_instance}. Customize first: #{params[:customize]}")
+      jobs
+    end
+  end
+end

data/app/controllers/ui_acd_controller.rb

@@ -6,16 +6,28 @@ class UiAcdController < ::Api::V2::BaseController
     @app_data = {}
     app_definition = ForemanAcd::AppDefinition.find(params[:id])
     @app_data['app_definition'] = app_definition
-    @app_data['fdata'] = collect_fdata(app_definition.hostgroup_id)
   end
 
-  def fdata
-    @fdata = collect_fdata(params['id'])
+  def foreman_data
+    @foreman_data = collect_foreman_data(params['id'])
+  end
+
+  def ansible_data
+    @ansible_data = collect_ansible_data(params['id'])
+  end
+
+  def action_permission
+    case params[:action]
+    when 'app', 'foreman_data', 'ansible_data'
+      :view
+    else
+      super
+    end
   end
 
   private
 
-  def collect_fdata(hostgroup_id)
+  def collect_foreman_data(hostgroup_id)
     hg = Hostgroup.find(hostgroup_id)
     fdata = OpenStruct.new(
       :environments => Environment.all,
@@ -27,4 +39,8 @@ class UiAcdController < ::Api::V2::BaseController
     )
     fdata
   end
+
+  def collect_ansible_data(playbook_id)
+    ForemanAcd::AnsiblePlaybook.find(playbook_id).as_unified_structobj
+  end
 end