diff --git a/src/tasks/utils/points.cr b/src/tasks/utils/points.cr index 14d44f200..8ad37c9d8 100644 --- a/src/tasks/utils/points.cr +++ b/src/tasks/utils/points.cr @@ -389,7 +389,7 @@ module CNFManager end_time = Time.utc task_runtime = (end_time - start_time).milliseconds - Log.for(task).info { "task_runtime=#{task_runtime}; start_time=#{start_time}; end_time:#{end_time}" } + Log.for("#{task}").info { "task_runtime=#{task_runtime}; start_time=#{start_time}; end_time:#{end_time}" } # The task result info has to be appeneded to an array of YAML::Any # So encode it into YAML and parse it back again to assign it. diff --git a/src/tasks/workload/compatibility.cr b/src/tasks/workload/compatibility.cr index ce19e890c..145c869d0 100644 --- a/src/tasks/workload/compatibility.cr +++ b/src/tasks/workload/compatibility.cr @@ -426,13 +426,13 @@ end desc "Will the CNF install using helm with helm_deploy?" task "helm_deploy" do |_, args| + task_start_time = Time.utc testsuite_task = "helm_deploy" Log.for(testsuite_task).info { "Running #{testsuite_task}" } Log.for(testsuite_task).info { "helm_deploy args: #{args.inspect}" } if check_verbose(args) if check_cnf_config(args) || CNFManager.destination_cnfs_exist? CNFManager::Task.task_runner(args) do |args, config| - task_start_time = Time.utc Log.for(testsuite_task).info { "Starting test" } emoji_helm_deploy="⎈🚀" diff --git a/src/tasks/workload/configuration.cr b/src/tasks/workload/configuration.cr index 6e8515c43..95d99cb7b 100644 --- a/src/tasks/workload/configuration.cr +++ b/src/tasks/workload/configuration.cr @@ -358,7 +358,7 @@ task "hardcoded_ip_addresses_in_k8s_runtime_configuration" do |_, args| upsert_failed_task(testsuite_task, "✖️ 🏆 FAILED: Hard-coded IP addresses found in the runtime K8s configuration", task_start_time) end rescue - upsert_skipped_task(testsuite_task, "⏭️ 🏆 SKIPPED: unknown exception", task_start_time) + upsert_skipped_task(testsuite_task, "⏭️ 🏆 SKIPPED: unknown exception", Time.utc) ensure KubectlClient::Delete.command("namespace hardcoded-ip-test --force --grace-period 0") end @@ -595,7 +595,7 @@ task "immutable_configmap" do |_, args| # now we change then apply again template = ImmutableConfigMapTemplate.new("doesnt_matter_again").to_s - Log.for(testsuite_task)debug { "test immutable_configmap change template: #{template}" } + Log.for(testsuite_task).debug { "test immutable_configmap change template: #{template}" } File.write(test_config_map_filename, template) immutable_configmap_supported = true diff --git a/src/tasks/workload/observability.cr b/src/tasks/workload/observability.cr index ebf01ccd3..3924058ab 100644 --- a/src/tasks/workload/observability.cr +++ b/src/tasks/workload/observability.cr @@ -252,16 +252,17 @@ end desc "Does the CNF install use tracing?" task "tracing" do |_, args| - Log.for("verbose").info { "tracing" } if check_verbose(args) - Log.info { "tracing args: #{args.inspect}" } + testsuite_task = "tracing" + Log.for(testsuite_task).info { "Running test" } + Log.for(testsuite_task).info { "tracing args: #{args.inspect}" } + next if args.named["offline"]? - emoji_tracing_deploy="⎈🚀" + emoji_tracing_deploy="⎈🚀" if check_cnf_config(args) || CNFManager.destination_cnfs_exist? CNFManager::Task.task_runner(args) do |args, config| task_start_time = Time.utc - testsuite_task = "tracing" - Log.for(testsuite_task).info { "Starting test" } + Log.for(testsuite_task).info { "Starting test for CNF" } match = JaegerManager.match() Log.info { "jaeger match: #{match}" }