Skip to content

added timestamp to logging#2475

Closed
liferoad wants to merge 1 commit intoGoogleCloudPlatform:mainfrom
liferoad:add-timestamp-logging
Closed

added timestamp to logging#2475
liferoad wants to merge 1 commit intoGoogleCloudPlatform:mainfrom
liferoad:add-timestamp-logging

Conversation

@liferoad
Copy link
Copy Markdown
Contributor

The below timestamp is not helpful.

2025-06-19T15:28:32.8571119Z [pool-1-thread-6] INFO org.apache.beam.it.common.PipelineOperator - Condition was not met yet. Checking if job is finished.
2025-06-19T15:28:32.8573464Z [pool-1-thread-6] INFO org.apache.beam.it.common.PipelineOperator - Job 2025-06-19_08_14_13-1924383829825156948 is in state JOB_STATE_DRAINING
2025-06-19T15:28:32.8576530Z [pool-1-thread-6] INFO org.apache.beam.it.common.PipelineOperator - Job not finished and conditions not met. Will check again in 15 seconds (total wait: 256s of max 900s)
2025-06-19T15:28:32.8578104Z [pool-1-thread-6] INFO org.apache.beam.it.common.PipelineOperator - Condition was not met yet. Checking if job is finished.
2025-06-19T15:28:32.8580702Z [pool-1-thread-6] INFO org.apache.beam.it.common.PipelineOperator - Job 2025-06-19_08_14_13-1924383829825156948 is in state JOB_STATE_DRAINED
2025-06-19T15:28:32.8582712Z [pool-1-thread-6] INFO org.apache.beam.it.common.PipelineOperator - Detected that launch was finished, checking conditions once more.
2025-06-19T15:28:32.8584618Z [pool-1-thread-6] INFO org.apache.beam.it.common.PipelineOperator - Condition was not met yet. Checking if job is finished.
2025-06-19T15:28:32.8586221Z [pool-1-thread-6] INFO org.apache.beam.it.common.PipelineOperator - Launch was finished, stop checking.

Added the specific timestamp for structured-logging

@codecov
Copy link
Copy Markdown

codecov bot commented Jun 21, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 49.61%. Comparing base (43be033) to head (4e79975).
Report is 1 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #2475      +/-   ##
============================================
- Coverage     49.62%   49.61%   -0.01%     
+ Complexity     4808     4804       -4     
============================================
  Files           941      941              
  Lines         57661    57667       +6     
  Branches       6233     6233              
============================================
  Hits          28614    28614              
- Misses        27007    27011       +4     
- Partials       2040     2042       +2     
Components Coverage Δ
spanner-templates 69.93% <ø> (-0.02%) ⬇️
spanner-import-export 68.60% <ø> (-0.04%) ⬇️
spanner-live-forward-migration 78.77% <ø> (ø)
spanner-live-reverse-replication 77.36% <ø> (ø)
spanner-bulk-migration 87.89% <ø> (ø)
Files with missing lines Coverage Δ
...google/cloud/teleport/v2/logging/JsonAppender.java 100.00% <100.00%> (ø)

... and 4 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@liferoad
Copy link
Copy Markdown
Contributor Author

does not impact the IT logs.

@liferoad liferoad closed this Jun 22, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant