Skip to content

[UTR Logging] New Integration Test that compares LogEntries from UTR vs Google cloud exporter#2234

Draft
XuechunHou wants to merge 56 commits intomasterfrom
compare-log-entries
Draft

[UTR Logging] New Integration Test that compares LogEntries from UTR vs Google cloud exporter#2234
XuechunHou wants to merge 56 commits intomasterfrom
compare-log-entries

Conversation

@XuechunHou
Copy link
Contributor

@XuechunHou XuechunHou commented Mar 12, 2026

Description

This PR includes an integration test that compares LogEntries from the telemetry endpoint, VS those from google cloud exporter, given the same input log.

Related issue

b/490113021

How has this been tested?

Checklist:

  • Unit tests
    • Unit tests do not apply.
    • Unit tests have been added/modified and passed for this PR.
  • Integration tests
    • Integration tests do not apply.
    • Integration tests have been added/modified and passed for this PR.
  • Documentation
    • This PR introduces no user visible changes.
    • This PR introduces user visible changes and the corresponding documentation change has been made.
  • Minor version bump
    • This PR introduces no new features.
    • This PR introduces new features, and there is a separate PR to bump the minor version since the last release already.
    • This PR bumps the version.

1. otlp_exporter + otlp_logging, the generated otel.yaml only has one
   logging pipeline with otlp receiver and otlp exporter.
2. otlp_exporter + otel_logging, the generated otel.yaml has all the
   logging pipeline using otlp exporter.
3. otlp_exporter + otlp_logging + otel_logging, the generated otel.yaml
   has all the logging pipelines using otlp exporter.
4. otlp_logging + otel_logging, the generated otel.yaml has all the
   logging pipelines using the google cloud exporter.
@XuechunHou XuechunHou changed the title added comparison test [UTR Logging] New Integration that compares LogEntries from UTR vs Google cloud exporter Mar 12, 2026
@XuechunHou XuechunHou self-assigned this Mar 12, 2026
@XuechunHou XuechunHou changed the title [UTR Logging] New Integration that compares LogEntries from UTR vs Google cloud exporter [UTR Logging] New Integration Test that compares LogEntries from UTR vs Google cloud exporter Mar 12, 2026
Base automatically changed from utr-logging to master March 12, 2026 18:55
return plogotlp.NewGRPCClient(conn), nil
}

func entryToMap(entry *loggingpb.LogEntry) map[string]any {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why are you manually extracting the LogEntry proto here? That's going to interact poorly with int64 fields because you'll get them as the wrong type.

@XuechunHou XuechunHou force-pushed the compare-log-entries branch from 4208e77 to 323da2f Compare March 13, 2026 21:46
@XuechunHou XuechunHou force-pushed the compare-log-entries branch from 323da2f to 4d79afd Compare March 13, 2026 21:50
@XuechunHou XuechunHou force-pushed the compare-log-entries branch from d5e1335 to 2de3c8b Compare March 16, 2026 19:14
@XuechunHou XuechunHou added the kokoro:force-run Forces kokoro to run integration tests on a CL label Mar 16, 2026
@stackdriver-instrumentation-release stackdriver-instrumentation-release removed the kokoro:force-run Forces kokoro to run integration tests on a CL label Mar 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants