Skip to content

Commit

Permalink
Fix typos for init log and timestamp
Browse files Browse the repository at this point in the history
  • Loading branch information
HashNuke committed Oct 26, 2023
1 parent c54cea6 commit aefb164
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 9 deletions.
2 changes: 1 addition & 1 deletion src/tasks/utils/points.cr
Original file line number Diff line number Diff line change
Expand Up @@ -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.
Expand Down
2 changes: 1 addition & 1 deletion src/tasks/workload/compatibility.cr
Original file line number Diff line number Diff line change
Expand Up @@ -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="⎈🚀"
Expand Down
4 changes: 2 additions & 2 deletions src/tasks/workload/configuration.cr
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down Expand Up @@ -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
Expand Down
11 changes: 6 additions & 5 deletions src/tasks/workload/observability.cr
Original file line number Diff line number Diff line change
Expand Up @@ -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}" }
Expand Down

0 comments on commit aefb164

Please sign in to comment.