Skip to content

Conversation

@rafaelwestphal
Copy link
Contributor

Description

The otlphttp exporter needs googleclientauth to be able to send metrics to that endpoint.

Related issue

b/435217364

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.

@rafaelwestphal rafaelwestphal added the kokoro:force-run Forces kokoro to run integration tests on a CL label Jul 31, 2025
@stackdriver-instrumentation-release stackdriver-instrumentation-release removed the kokoro:force-run Forces kokoro to run integration tests on a CL label Jul 31, 2025
@franciscovalentecastro
Copy link
Contributor

franciscovalentecastro commented Jul 31, 2025

The build issue in Build (Ubuntu 22.04 Jammy AArch64) is related to Ubuntu Jammy not having access to a IANA Time Zone database so setting TZ=America/Los_Angeles was not working in transformation tests 1. I've read the Openetelemetry time parsing docs 2 which points to how Otel looks for the time zone database.

This issues do seem to be caused by the new way of building the transformprocessor in GoogleCloudPlatform/opentelemetry-operations-collector#350.

I have found two solutions to the issue so far :

  • [Best] Add the import _ "time/tzdata" to the transformprocessor/factory.go 3 which will embed the timezone database in every otelopscol build (see 2).
  • [Simple, Not the best solution overall] Install tzdata package (contains timezone database) in Ubuntu Jammy tests. I've tried this in e12ded2 and it works.

Footnotes

  1. https://github.com/GoogleCloudPlatform/ops-agent/blob/master/transformation_test/transformation_test.go#L432

  2. https://github.com/open-telemetry/opentelemetry-collector-contrib/tree/main/pkg/ottl/ottlfuncs#time 2

  3. https://github.com/GoogleCloudPlatform/opentelemetry-operations-collector/blob/master/components/otelopscol/processor/transformprocessor/factory.go#L18

@franciscovalentecastro
Copy link
Contributor

franciscovalentecastro commented Aug 6, 2025

CC @rafaelwestphal

Update : I found a correct and simple solution for the failure in Build (Ubuntu 22.04 Jammy AArch64) (b/435664151). Some details :

Copy link
Contributor

@franciscovalentecastro franciscovalentecastro left a comment

Choose a reason for hiding this comment

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

Please remove the installation of tzdata in this PR. I will do a followup PR to add tzdata as an Ops Agent dependency.

build-essential cmake bison flex file libsystemd-dev tzdata \

@rafaelwestphal
Copy link
Contributor Author

Please remove the installation of tzdata in this PR. I will do a followup PR to add tzdata as an Ops Agent dependency.

build-essential cmake bison flex file libsystemd-dev tzdata \

Done

Copy link
Contributor

@franciscovalentecastro franciscovalentecastro left a comment

Choose a reason for hiding this comment

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

LGTM!

@rafaelwestphal rafaelwestphal merged commit 7ea0176 into master Aug 6, 2025
55 of 62 checks passed
@rafaelwestphal rafaelwestphal deleted the westphalrafael-googleclientauth branch August 6, 2025 17:02
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.

4 participants