Merge pull request #101 from billfitzgerald0120/log_cleanup1

Change errors in log to error type.
ManageIQ · Apr 26, 2017 · 4bf5377 · 4bf5377
2 parents cd7d6db + 008b70d
commit 4bf5377
Show file tree

Hide file tree

Showing 5 changed files with 5 additions and 5 deletions.
diff --git a/...Tower/Service/Provisioning/StateMachines/Provision.class/__methods__/check_provisioned.rb b/...Tower/Service/Provisioning/StateMachines/Provision.class/__methods__/check_provisioned.rb
@@ -36,7 +36,7 @@ def check_provisioned(task, service)
 
                     unless @handle.root['ae_result'] == 'retry'
                       @handle.log("info", "AnsibleTower job finished. Status: #{@handle.root['ae_result']}, reason: #{@handle.root['ae_reason']}")
-                      @handle.log("info", "Please examine job console output for more details") if @handle.root['ae_result'] == 'error'
+                      @handle.log('error', 'Please examine job console output for more details') if @handle.root['ae_result'] == 'error'
 
                       job.refresh_ems
                       task.miq_request.user_message = @handle.root['ae_reason'].truncate(255) unless @handle.root['ae_reason'].blank?

diff --git a/...Tower/Service/Provisioning/StateMachines/Provision.class/__methods__/check_provisioned.rb b/...Tower/Service/Provisioning/StateMachines/Provision.class/__methods__/check_provisioned.rb
@@ -29,7 +29,7 @@ def check_provisioned(task, service)
 
     unless @handle.root['ae_result'] == 'retry'
       @handle.log("info", "AnsibleTower job finished. Status: #{@handle.root['ae_result']}, reason: #{@handle.root['ae_reason']}")
-      @handle.log("info", "Please examine job console output for more details") if @handle.root['ae_result'] == 'error'
+      @handle.log('error', 'Please examine job console output for more details') if @handle.root['ae_result'] == 'error'
 
       job.refresh_ems
       task.miq_request.user_message = @handle.root['ae_reason'].truncate(255) unless @handle.root['ae_reason'].blank?

diff --git a/...eployment/ContainerProvider/System/StateMachine/Deployment.class/__methods__/provision.rb b/...eployment/ContainerProvider/System/StateMachine/Deployment.class/__methods__/provision.rb
@@ -10,7 +10,7 @@ def check_state(finished, fail)
     $evm.root['ae_result'] = "ok"
   elsif fail
     $evm.root['ae_result'] = "error"
-    $evm.log(:info, "*********  Provision failed  ************")
+    $evm.log(:error, '*********  Provision failed  ************')
   else
     $evm.log(:info, "*********  Provision isnt finished re-trying in 1 min  ************")
     $evm.root['ae_result']         = 'retry'

diff --git a/...mate/ManageIQ/Service/Generic/StateMachines/GenericLifecycle.class/__methods__/execute.rb b/...mate/ManageIQ/Service/Generic/StateMachines/GenericLifecycle.class/__methods__/execute.rb
@@ -22,7 +22,7 @@ def main
                 rescue => err
                   @handle.root['ae_result'] = 'error'
                   @handle.root['ae_reason'] = err.message
-                  @handle.log("info", "Error in execute: #{err.message}")
+                  @handle.log('error', "Error in execute: #{err.message}")
                   update_task(err.message)
                 end
               end

diff --git a/...e/ManageIQ/Service/Generic/StateMachines/GenericLifecycle.class/__methods__/preprocess.rb b/...e/ManageIQ/Service/Generic/StateMachines/GenericLifecycle.class/__methods__/preprocess.rb
@@ -24,7 +24,7 @@ def main
                 rescue => err
                   @handle.root['ae_result'] = 'error'
                   @handle.root['ae_reason'] = err.message
-                  @handle.log(:info, "Error in preprocess: #{err.message}")
+                  @handle.log(:error, "Error in preprocess: #{err.message}")
                   update_task(err.message)
                 end
               end