foreman_acd 0.6.0 → 0.9.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 809d34cc7063d66a26a1c0c913cd95287a6329cce55a75c1684046bb47b1afe1
-  data.tar.gz: d32aa387545b3fac503996354baae6704c30118a923d95706668895cad6440f5
+  metadata.gz: 2a738c9828728db02d8d06460befc527ffbf7c3662d0bd0ba4d252ffe416be7e
+  data.tar.gz: 30c1d33e7525179ac752d949116fd99bc9fefb27e11c1a6f69567803ef5a0e25
 SHA512:
-  metadata.gz: 996b0c911e71e045b232b3d6b27776eaa9c7e6c52f23f78d29c911f899d4a60024990fa8866e2f401e2d91f1c2a0c06a7264f16ffc2e4bd0b97c0e794a0a7c63
-  data.tar.gz: 445dc99ec810e7fbbdf33d2c1e69b3a60942c58938212105033e472981bee85e626ce51d7e2f36359bbfb9148dca240fff134d183481d1d8312a3e9f933f4815
+  metadata.gz: 8311857ec1c4a78e95e4f6a401548fa975b876fba04839a0641d84c4dd138d20858717df830d391651bb517ce07e423fc4faa2b4abc42e25a8fe332233e7460c
+  data.tar.gz: 455afe8abd543214663b4a621d92893a8e63e28a44e412b1de65695c32e712432906324b73b09c64ee94a6091bc4470f8b8640eb3f1507c3a21d804526ee45a8

data/README.md

@@ -1,5 +1,3 @@
-[![Build Status master](https://travis-ci.org/ATIX-AG/foreman_acd.svg?branch=master)](https://travis-ci.org/ATIX-AG/foreman_acd)
-
 # Foreman Application Centric Deployment
 
 A Foreman plugin providing application centric deployment and a self-service portal.
@@ -55,6 +53,10 @@ This plugin aims to setup all six hosts and to deploy the application.
 
 :warning: This plugin is still in development.
 
+## Documentation
+
+See [Application Centric Deployment Guide](https://docs.theforeman.org/nightly/Application_Centric_Deployment/index-foreman-el.html)
+
 ## Installation
 
 See the [installation](https://theforeman.org/plugins/#2.Installation) chapter of the Foreman plugins documentation on how to install Foreman plugins.
@@ -88,7 +90,7 @@ You need to refresh the smart proxy features in *Infrastructure > Smart Proxies
 ### Ansible Playbook
 
 * Copy (or checkout a git repository) an Ansible playbook.
-Store it in `/etc/foreman/plugins/foreman_acd/ansible-playbooks/` so that SELinux is able to read it.
+Store it in `/var/lib/foreman/foreman_acd/ansible-playbooks/` so that SELinux is able to read it.
 * Add a new Ansible Playbook via *Applications > Ansible Playbooks*.
 * Specify the path to the Ansible playbook and name of the playbook file. (e.g. `site.yml`).
 * Save it and press *Import group variables* for this newly created Ansible playbook.
@@ -117,8 +119,6 @@ All Foreman parameters require a value.
 
 ## TODO
 
-* Add `git` support for the Ansible playbooks.
-* Provide application templates which contains application definition and the required Ansible-playbook.
 * Add Saltstack support to configure the application.
 * Extend the Foreman parameter and value validation.
 

data/app/controllers/foreman_acd/ansible_playbooks_controller.rb

@@ -7,6 +7,7 @@ module ForemanAcd
     include ::ForemanAcd::Concerns::AnsiblePlaybookParameters
 
     before_action :find_resource, :only => [:edit, :update, :destroy, :import_vars]
+    after_action :delete_synced_repo, :only => [:new, :edit, :create, :update, :destroy, :index]
 
     def index
       @ansible_playbooks = resource_base.search_for(params[:search], :order => params[:order]).paginate(:page => params[:page])
@@ -18,6 +19,11 @@ module ForemanAcd
 
     def create
       @ansible_playbook = AnsiblePlaybook.new(ansible_playbook_params)
+      if session[:git_path]
+        @ansible_playbook.update(:path => ansible_playbook_full_path(ansible_playbook_rename(@ansible_playbook[:name])))
+        FileUtils.mv(session[:git_path], @ansible_playbook[:path])
+        session[:git_path] = nil
+      end
       if @ansible_playbook.save
         process_success :success_msg => _("Successfully created %s. You need to press the \"Import groups\" button
                                            before this ansible playbook can be used in App Definitions!") % @ansible_playbook
@@ -29,6 +35,20 @@ module ForemanAcd
     def edit; end
 
     def update
+      # Move synced repo to new path if ansible_playbook name is changed
+      if !session[:git_path].nil? && ansible_playbook_params[:name] != @ansible_playbook[:name]
+        FileUtils.mv(@ansible_playbook[:path], ansible_playbook_full_path(ansible_playbook_rename(ansible_playbook_params[:name])))
+        @ansible_playbook.update(:path => ansible_playbook_full_path(ansible_playbook_rename(ansible_playbook_params[:name])))
+        session[:git_path] = nil
+
+      # Remove old version and copy new version of synced repository
+      elsif !session[:git_path].nil? && ansible_playbook_params[:name] == @ansible_playbook.name
+        remove_ansible_dir(@ansible_playbook[:path]) if @ansible_playbook.path
+        @ansible_playbook.update(:path => ansible_playbook_full_path(ansible_playbook_rename(@ansible_playbook[:name])))
+        FileUtils.mv(session[:git_path], @ansible_playbook[:path])
+        session[:git_path] = nil
+      end
+
       if @ansible_playbook.update(ansible_playbook_params)
         process_success
       else
@@ -44,10 +64,62 @@ module ForemanAcd
       end
     end
 
+    def sync_git_repo
+      @ansible_playbook = AnsiblePlaybook.new
+      sync_params = params[:ansible_playbook]
+      dir = Dir.mktmpdir
+
+      begin
+        git = Git.init(dir)
+
+        if ForemanAcd.proxy_setting.present?
+          git.config('http.proxy', ForemanAcd.proxy_setting)
+          logger.info("HTTP Proxy used: #{git.config['http.proxy']}")
+        end
+
+        if sync_params[:git_commit].empty?
+          if ForemanAcd.proxy_setting.present?
+            err_msg = _('Please set the Git Branch/Tag/Commit. This setting is necessary if a HTTP proxy is used!')
+            raise StandardError.new err_msg
+          else
+            commit = Git.ls_remote(sync_params[:git_url])['head'][:sha]
+          end
+        else
+          commit = sync_params[:git_commit]
+        end
+
+        git.add_remote('origin', sync_params[:git_url])
+        git.fetch
+        git.checkout(commit)
+
+        session[:git_path] = git.dir.path
+      rescue StandardError => e
+        logger.error("Failed to sync git repository: #{e}")
+        render :json => { :status => 'error', :message => e }, :status => :internal_server_error
+      end
+    end
+
+    # Remove abandoned synced git repositories
+    def delete_synced_repo
+      names = []
+      AnsiblePlaybook.all.each do |ansible_playbook|
+        names.push(ansible_playbook_rename(ansible_playbook.name))
+      end
+      names.push('.', '..')
+      return unless Dir.exist?(ForemanAcd.ansible_playbook_path)
+      Dir.foreach(ForemanAcd.ansible_playbook_path) do |dirname|
+        next if names.include? dirname
+        remove_ansible_dir(ansible_playbook_full_path(dirname))
+        logger.info("Successfully removed #{dirname}")
+      end
+    end
+
     def action_permission
       case params[:action]
       when 'import_vars'
         :import_vars
+      when 'sync_git_repo'
+        :sync_git_repo
       when 'grab'
         :grab
       else
@@ -120,5 +192,23 @@ module ForemanAcd
         process_error :error_msg => _(errors.join(' ')), :redirect => ansible_playbooks_path
       end
     end
+
+    private
+
+    def ansible_playbook_rename(name)
+      name.split(/\W+/).join('_')
+    end
+
+    def remove_ansible_dir(dirpath)
+      unless dirpath.start_with? ForemanAcd.ansible_playbook_path
+        logger.error("Sorry, the directory #{dirpath} is not within #{ForemanAcd.ansible_playbook_path} and will therefore not be cleaned up!")
+        return false
+      end
+      FileUtils.rm_rf(dirpath) if Dir.exist?(dirpath)
+    end
+
+    def ansible_playbook_full_path(dirname)
+      File.join(ForemanAcd.ansible_playbook_path, dirname)
+    end
   end
 end

data/app/controllers/foreman_acd/app_definitions_controller.rb

@@ -9,7 +9,7 @@ module ForemanAcd
     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]
+    before_action :assign_ansible_playbook, :only => [:create]
 
     def index
       @app_definitions = resource_base.search_for(params[:search], :order => params[:order]).paginate(:page => params[:page])
@@ -37,6 +37,9 @@ module ForemanAcd
           process_error
         end
       rescue StandardError, ValidationError => e
+        if params[:foreman_acd_app_definition_import].present?
+          AnsiblePlaybook.find(params[:foreman_acd_app_definition][:acd_ansible_playbook_id]).delete if params[:foreman_acd_app_definition][:acd_ansible_playbook_id].present?
+        end
         redirect_to new_app_definition_path, :flash => { :error => _(e.message) }
       end
     end
@@ -75,16 +78,110 @@ module ForemanAcd
     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
+      dir_path = "#{Dir.mktmpdir}/#{@app_definition.name}.tar"
+      `tar -cvf "#{dir_path}" "#{@app_definition.ansible_playbook.path}" #{export_app_template_data.path}`
+      logger.info("Successfully created application template tar file for #{@app_definition.name}")
+      send_file dir_path
+    rescue StandardError => e
+      logger.info("Export of #{@app_definition.name} failed with the error: #{e}")
+      redirect_to app_definitions_path, :flash => { :error => _(e.message) }
+    end
+
+    def handle_file_upload
+      return unless params[:app_definition_file] && (raw_directory = params[:app_definition_file])
+      begin
+        dir = Dir.mktmpdir
+        untar_import_directory(raw_directory, dir)
+        ansible_file = Dir.glob("#{dir}/tmp/*.yaml")
+        data = JSON.parse(YAML.load_file(ansible_file[0]).to_json) if ansible_file
+        ansible_playbook_import = data.find { |d| d if d['ansible_playbook'] }
+
+        session[:ansible_playbook_params] = { :dir => dir, :ansible_playbook => ansible_playbook_import['ansible_playbook'] }
+        render :json => { :ansible_services => create_ansible_services(data, ansible_playbook_import) }, :status => :ok
+      rescue StandardError => e
+        render :json => { :status => 'error', :message => e }, :status => :internal_server_error
+      end
     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
+    def export_app_template_data
+      file = Tempfile.open([@app_definition.name, '.yaml'])
+      data = JSON.parse(@app_definition.services).append(:ansible_playbook => @app_definition.ansible_playbook.attributes.except('id', 'created_at', 'updated_at').as_json).to_yaml
+      file.write(data)
+      file.close
+      logger.info("Successfully created yaml file for app_template data: #{file.path}")
+      file
+    rescue StandardError => e
+      logger.info("Creation of app template data failed: #{e}")
+      redirect_to app_definitions_path, :flash => { :error => _(e.message) }
+    end
+
+    def rename_path_name(dir_path)
+      return dir_path unless Dir.exist?(dir_path)
+      ind = 1
+      loop do
+        path = dir_path
+        path += ind.to_s
+        return path unless Dir.exist?(path)
+        ind += 1
+      end
+    end
+
+    def create_ansible_playbook(dir, ansible_playbook, dir_path)
+      FileUtils.cp_r "#{dir}/#{ansible_playbook['path']}/.", dir_path
+      ansible_playbook['path'] = dir_path
+      ansible_playbook['name'] = File.basename(dir_path)
+      new_playbook = AnsiblePlaybook.create(ansible_playbook)
+      new_playbook
+    rescue StandardError => e
+      logger.info("Error while creating AnsiblePlaybook: #{e}")
+    end
+
+    def create_ansible_services(data, ansible_playbook_import)
+      ansible_services = []
+      session[:data_services] = data - [ansible_playbook_import]
+      session[:data_services].each do |d|
+        ansible_services.append({ :id => d['id'], :value => d['name'] })
+      end
+      ansible_services
+    end
+
+    def untar_import_directory(directory, dir)
+      `tar -xvf  #{directory.path} -C #{dir}`
+    rescue StandardError => e
+      logger.info("Failed to untar imported directory: #{e}")
+    end
+
+    def assign_ansible_playbook
+      return unless params[:foreman_acd_app_definition_import]
+      dir_path = "#{ForemanAcd.ansible_playbook_path}/#{session[:ansible_playbook_params][:ansible_playbook]['name'].split(/\W+/).join('_')}"
+
+      # Append path and name of ansible with n + 1 if ansible_playbook with same name or name[n] exists
+      ansible_playbook = create_ansible_playbook(session[:ansible_playbook_params][:dir], session[:ansible_playbook_params][:ansible_playbook], rename_path_name(dir_path))
+      params[:foreman_acd_app_definition][:acd_ansible_playbook_id] = ansible_playbook.id
+
+      begin
+        services = JSON.parse(params[:foreman_acd_app_definition_import][:services])
+        flag = 0
+        session[:data_services].each do |d|
+          hostgroup = services.find { |service| service['name'] == d['name'] }['hostgroup']
+          if hostgroup == ''
+            flag += 1
+            break
+          else
+            d['hostgroup'] = hostgroup
+          end
+        end
+        params[:foreman_acd_app_definition][:services] = session[:data_services].to_json
+      rescue StandardError => e
+        ansible_playbook&.destroy
+        redirect_to({ :action => 'import' }, :error => _("Hostgroups are not configured properly: #{e}"))
+      else
+        ansible_playbook&.destroy
+        redirect_to({ :action => 'import' }, :error => _('Some services are not assigned Hostgroups')) if flag.positive?
+      end
+      session[:data_services] = nil
     end
   end
 end

data/app/controllers/foreman_acd/app_instances_controller.rb

@@ -3,8 +3,9 @@
 module ForemanAcd
   # Application Instance Controller
   class AppInstancesController < ::ForemanAcd::ApplicationController
-    include Foreman::Controller::AutoCompleteSearch
+    include ::Foreman::Controller::AutoCompleteSearch
     include ::ForemanAcd::Concerns::AppInstanceParameters
+    include ::ForemanAcd::Concerns::AppInstanceMixins
 
     before_action :find_resource, :only => [:edit, :update, :destroy_with_hosts, :deploy, :report]
     before_action :read_applications, :only => [:new, :edit]
@@ -72,33 +73,58 @@ module ForemanAcd
     end
 
     def deploy
+      value = false
+      @app_instance.update!({ :last_deploy_task_id => nil,
+                              :initial_configure_task_id => nil })
+      @app_instance.foreman_hosts.each { |f| f.update!(:last_progress_report => nil) }
       @app_instance.clean_all_hosts if params[:delete_hosts]
-
+      value = safe_deploy? if params[:safe_deploy]
+      session.delete(:remember_hosts)
       logger.info('Run async foreman task to deploy hosts')
-      async_task = ForemanTasks.async_task(::Actions::ForemanAcd::DeployAllHosts, @app_instance)
+      async_task = ForemanTasks.async_task(::Actions::ForemanAcd::DeployAllHosts, @app_instance, value)
       @app_instance.update!(:last_deploy_task => async_task)
-      process_success(:success_msg => _('Started task to deploy hosts for %s') % @app_instance)
+
+      redirect_to report_app_instance_path, :success => _('Started task to deploy hosts for %s') % @app_instance
     rescue StandardError => e
       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
 
+    def safe_deploy?
+      return false if session[:remember_hosts].empty?
+      session[:remember_hosts]
+    end
+
     def report
-      @report_hosts = collect_host_report_data
+      @report_hosts = collect_host_report_data(@app_instance)
       logger.debug("app instance host details: #{@report_hosts.inspect}")
     end
 
     def app_instance_has_foreman_hosts
       hosts = JSON.parse(@app_instance.hosts)
+      session[:remember_hosts] = []
       hosts.each do |h|
         if @app_instance.foreman_hosts.where(:hostname => h['hostname']).exists?
+          old_host = @app_instance.foreman_hosts.find_by(:hostname => h['hostname'])
+
           @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']))
+
+          updated_host = @app_instance.foreman_hosts.find_by(:hostname => h['hostname'])
+
+          # Store hosts if updated for safe deploy
+          session[:remember_hosts] << updated_host.id if updated_host.updated_at != old_host.updated_at
         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']))
+          @app_instance.foreman_hosts.create(:hostname => h['hostname'],
+                                             :service => h['service'],
+                                             :description => h['description'],
+                                             :is_existing_host => h['isExistingHost'],
+                                             :foremanParameters => JSON.dump(h['foremanParameters']),
+                                             :ansibleParameters => JSON.dump(h['ansibleParameters']))
+          # Store new hosts for safe deploy
+          session[:remember_hosts] << @app_instance.foreman_hosts.find_by(:hostname => h['hostname']).id
         end
       end
 
@@ -115,6 +141,7 @@ module ForemanAcd
           :hostname => h.hostname,
           :service => h.service,
           :description => h.description,
+          :isExistingHost => h.is_existing_host,
           :foremanParameters => JSON.parse(h.foremanParameters),
           :ansibleParameters => JSON.parse(h.ansibleParameters)
         }
@@ -135,30 +162,5 @@ module ForemanAcd
     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
-      report_data
-    end
   end
 end

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

@@ -9,7 +9,7 @@ module ForemanAcd
       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 => [])
+            filter.permit(:name, :description, :scm_type, :path, :git_commit, :git_url, :playfile, :location_ids => [], :organization_ids => [])
           end
         end
       end

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

@@ -0,0 +1,36 @@
+# frozen_string_literal: true
+
+module ForemanAcd
+  module Concerns
+    # Shared code for AppInstance API and UI controller
+    module AppInstanceMixins
+      extend ActiveSupport::Concern
+
+      def collect_host_report_data(app_instance)
+        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),
+                            :isExistingHost => foreman_host.is_existing_host,
+                            :powerStatusUrl => power_api_host_path(foreman_host.host)
+                          })
+          end
+          report_data << OpenStruct.new(a_host)
+        end
+        report_data
+      end
+    end
+  end
+end

data/app/controllers/foreman_acd/remote_execution_controller.rb

@@ -4,25 +4,40 @@ module ForemanAcd
   # Class to run remote execution jobs
   class RemoteExecutionController < JobInvocationsController
     def new
-      jobs = init_configuration
-      @composer = jobs.first
+      set_app_instance
+      result, job = init_configuration
+
+      if result.success == true
+        @composer = job
+      else
+        redirect_to(app_instances_path, :error => _("Coult not create remote execution job to configure the app '%{app_instance}': %{msg}") % {
+          :app_instance => @app_instance, :msg => result.error
+        })
+      end
     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'
+      begin
+        set_app_instance
+        result, job = init_configuration
+
+        unless result.success == true
+          return redirect_to(app_instances_path, :error => _("Coult not create remote execution job to configure the app '%{app_instance}': %{msg}") % {
+            :app_instance => @app_instance, :msg => result.error
+          })
+        end
+
+        @composer = job
+        if customize_first == false
+          @composer.trigger!
+          redirect_to job_invocation_path(@composer.job_invocation)
+        else
+          # redirect to the job itself if we want to customize the job
+          render :action => 'new'
+        end
+      rescue StandardError => e
+        redirect_to app_instances_path, :error => _("#{job}, #{e}")
       end
     end
 
@@ -34,13 +49,14 @@ module ForemanAcd
 
     private
 
-    def init_configuration
-      app_instance = ForemanAcd::AppInstance.find_by(:id => params[:id])
-      app_configurator = ForemanAcd::AppConfigurator.new(app_instance)
+    def set_app_instance
+      @app_instance = ForemanAcd::AppInstance.find_by(:id => params[:id])
+    end
 
-      jobs = app_configurator.configure
-      logger.debug("Creating #{jobs.count} job(s) to configure the app #{app_instance}. Customize first: #{params[:customize]}")
-      jobs
+    def init_configuration
+      logger.debug("Creating job to configure the app #{@app_instance}. Customize first: #{params[:customize]}")
+      app_configurator = ForemanAcd::AppConfigurator.new(@app_instance)
+      app_configurator.configure
     end
   end
 end

data/app/controllers/ui_acd_controller.rb

@@ -2,6 +2,8 @@
 
 # Controller to create JSON data to be used in react app
 class UiAcdController < ::Api::V2::BaseController
+  include ::ForemanAcd::Concerns::AppInstanceMixins
+
   def app
     @app_data = {}
     app_definition = ForemanAcd::AppDefinition.find(params[:id])
@@ -16,9 +18,17 @@ class UiAcdController < ::Api::V2::BaseController
     @ansible_data = collect_ansible_data(params['id'])
   end
 
+  def report_data
+    @report_data = collect_report_data(params['id'])
+  end
+
+  def validate_hostname
+    @host_validation = hostname_duplicate?(params['appDefId'].to_i, params['serviceId'].to_i, params['hostname'])
+  end
+
   def action_permission
     case params[:action]
-    when 'app', 'foreman_data', 'ansible_data'
+    when 'app', 'foreman_data', 'ansible_data', 'validate_hostname'
       :view
     else
       super
@@ -30,17 +40,46 @@ class UiAcdController < ::Api::V2::BaseController
   def collect_foreman_data(hostgroup_id)
     hg = Hostgroup.find(hostgroup_id)
     fdata = OpenStruct.new(
-      :environments => Environment.all,
-      :lifecycle_environments => Katello::KTEnvironment.all,
       :domains => Domain.all,
       :computeprofiles => ComputeProfile.all,
       :hostgroup_id => hg.id,
       :ptables => hg&.operatingsystem&.ptables
     )
+
+    fdata[:environments] = Environment.all if defined?(ForemanPuppet)
+    fdata[:lifecycle_environments] = Katello::KTEnvironment.all if defined?(Katello)
+
     fdata
   end
 
   def collect_ansible_data(playbook_id)
     ForemanAcd::AnsiblePlaybook.find(playbook_id).as_unified_structobj
   end
+
+  def collect_report_data(app_instance_id)
+    app_instance = ForemanAcd::AppInstance.find(app_instance_id)
+
+    report_data = {
+      :hosts => collect_host_report_data(app_instance),
+      :deploymentState => app_instance.deployment_state.to_s,
+      :initialConfigureState => app_instance.initial_configure_state.to_s
+    }
+    report_data['initialConfigureJobUrl'] = job_invocation_path(app_instance.initial_configure_job) unless app_instance.initial_configure_job.nil?
+
+    OpenStruct.new(report_data)
+  end
+
+  def hostname_duplicate?(app_def_id, service_id, hostname)
+    app_definition = ForemanAcd::AppDefinition.find(app_def_id)
+    service_data = JSON.parse(app_definition.services).select { |k| k['id'] == service_id }.first
+    domain_name = Hostgroup.find(service_data['hostgroup']).domain.name
+    validation_hostname = "#{hostname}.#{domain_name}"
+
+    vdata = OpenStruct.new(
+      :hostname => hostname,
+      :fqdn => validation_hostname,
+      :result => Host.find_by(:name => validation_hostname).nil?
+    )
+    vdata
+  end
 end

data/app/lib/actions/foreman_acd/deploy_all_hosts.rb

@@ -4,27 +4,32 @@ module Actions
   module ForemanAcd
     # DeployAllHosts implements a Foreman Task EntryAction
     class DeployAllHosts < Actions::EntryAction
-      def plan(app_instance)
-        action_subject(app_instance)
+      def plan(app_instance, safe_deploy)
+        action_subject(app_instance, :safe_deploy => safe_deploy)
         plan_self(:id => app_instance.id)
       end
 
       def run
         output[:status] = 'IN PROGRESS'
         app_instance = ::ForemanAcd::AppInstance.find(input.fetch(:id))
+        safe_deploy = input.fetch(:safe_deploy)
 
-        # Goal: all or nothing
+        # Goal: all, safe_deploy or nothing
         begin
-          ::Foreman::Logging.logger('foreman_acd').info "Start to deploy all hosts of the app #{app_instance}"
+          if safe_deploy
+            ::Foreman::Logging.logger('foreman_acd').info "Start to safe deploy hosts of the app #{app_instance}"
+          else
+            ::Foreman::Logging.logger('foreman_acd').info "Start to deploy all hosts of the app #{app_instance}"
+          end
           app_deployer = ::ForemanAcd::AppDeployer.new(app_instance)
-          output[:data] = app_deployer.deploy
+          output[:data] = app_deployer.deploy(safe_deploy)
           output[:status] = 'SUCCESS'
         rescue StandardError => e
-          ::Foreman::Logging.logger('foreman_acd').error "Error while deploying hosts for application instance. Clean up all other hosts: #{e}"
+          ::Foreman::Logging.logger('foreman_acd').error "Error while deploying hosts for application instance '#{app_instance.name}'. Clean up all other hosts: #{e}"
           app_instance.clean_all_hosts
 
-          output[:error] = e.to_s
           output[:status] = 'FAILURE'
+          raise "Error while deploying hosts for application instance '#{app_instance.name}': (#{e.message})"
         end
       end