Skip to content

Merging to release-5.13: [TT-17154] Bump opentelemetry to post-v0.0.25 doc-comment fixes (#8240)#8249

Merged
mativm02 merged 3 commits into
release-5.13from
merge/release-5.13/TT-17154
May 28, 2026
Merged

Merging to release-5.13: [TT-17154] Bump opentelemetry to post-v0.0.25 doc-comment fixes (#8240)#8249
mativm02 merged 3 commits into
release-5.13from
merge/release-5.13/TT-17154

Conversation

@mativm02

@mativm02 mativm02 commented May 26, 2026

Copy link
Copy Markdown
Contributor

Summary

Cherry-pick of #8240 onto release-5.13.

Bumps github.com/TykTechnologies/opentelemetry from v0.0.25 to v0.0.26-0.20260521112223-3b0d024e6090, pulling in doc-comment fixes (TykTechnologies/opentelemetry#69). Documentation-only upstream change — no behavior, API, or default changes.

Test plan

  • CI green on this branch

Ticket Details

TT-17154
Status In Code Review
Summary Gateway API + config 5.13.0

Generated at: 2026-05-28 11:48:52

## Summary

- Bumps `github.com/TykTechnologies/opentelemetry` from `v0.0.25` to
`v0.0.26-0.20260521112223-3b0d024e6090`, pulling in
[opentelemetry#69](TykTechnologies/opentelemetry#69)
(merge commit `3b0d024e`).
- That PR is a documentation-only change: it clarifies the config struct
comments that drive user-facing OpenTelemetry docs generation
(`ExporterConfig.Exporter`, `ResourceName`, `OpenTelemetry.Enabled`,
`TLS.Enable`, `TLS.CAFile`). No behavior, API, or default changes.
- Pinned via Go pseudo-version because no new tag was cut after
`v0.0.25` for this commit.

## Motivation

PR review feedback on the gateway-side OpenTelemetry config docs noted
unclear/inconsistent wording and grammar issues. Since the docs are
generated from the upstream library's struct comments, the fixes live in
`TykTechnologies/opentelemetry`. This PR brings those fixes into the
gateway.

## Test plan

- [x] `go build ./...` — OTel-touching packages (`./internal/otel/...`,
`./internal/graphql/...`, `./gateway/...`) compile clean. The only build
noise (`test/goplugins` "main undeclared", linker duplicate `-ldl`
warning) is pre-existing on master and unrelated.
- [x] `go vet ./internal/otel/...` — passes.
- [x] `go mod tidy` — clean, no unrelated module churn.
- [ ] CI green on this branch.

## Notes for reviewers

- Since the upstream change is comment-only, the diff here is minimal (2
lines in `go.mod`, 4 in `go.sum`).
- Once a `v0.0.26` tag is cut on the opentelemetry repo, a follow-up can
replace the pseudo-version with the clean tag — but functionally
identical.
<!---TykTechnologies/jira-linter starts here-->

### Ticket Details

<details>
<summary>
<a href="https://tyktech.atlassian.net/browse/TT-17154" title="TT-17154"
target="_blank">TT-17154</a>
</summary>

|         |    |
|---------|----|
| Status  | In Code Review |
| Summary | Gateway API + config 5.13.0 |

Generated at: 2026-05-21 11:53:06

</details>

<!---TykTechnologies/jira-linter ends here-->

(cherry picked from commit 8a9a180)
@mativm02
mativm02 enabled auto-merge (squash) May 26, 2026 12:02
@probelabs

probelabs Bot commented May 26, 2026

Copy link
Copy Markdown
Contributor

This PR updates the github.com/TykTechnologies/opentelemetry dependency from v0.0.25 to v0.0.26-0.20260521112223-3b0d024e6090. This is a cherry-pick of PR #8240 onto the release-5.13 branch. The upstream change in the dependency is limited to documentation comment fixes, resulting in no changes to the gateway's behavior, API, or default configurations.

Files Changed Analysis

The changes are confined to Go module files:

  • go.mod: The version of github.com/TykTechnologies/opentelemetry is updated.
  • go.sum: The checksums for the dependency are updated to match the new version.

These changes are consistent with a routine dependency update.

Architecture & Impact Assessment

  • What this PR accomplishes: It incorporates documentation improvements from an upstream dependency into the release-5.13 branch, ensuring code clarity for developers who might inspect the dependency's source code.
  • Key technical changes introduced: The only change is the modification of version constraints in the go.mod and go.sum files. There are no code changes in this repository.
  • Affected system components: This change has no runtime impact. While the opentelemetry package is used for distributed tracing within the gateway, this update only affects code comments within that package. The gateway's tracing functionality remains unchanged.

Scope Discovery & Context Expansion

  • The scope is limited to dependency management. The change is self-contained within the go.mod and go.sum files.
  • Since the upstream update is purely for documentation, there are no functional changes to how traces are generated, propagated, or exported. The risk of regression is negligible.
Metadata
  • Review Effort: 1 / 5
  • Primary Label: chore

Powered by Visor from Probelabs

Last updated: 2026-05-28T11:49:47.498Z | Triggered by: pr_updated | Commit: 3d404be

💡 TIP: You can chat with Visor using /visor ask <your question>

@probelabs

probelabs Bot commented May 26, 2026

Copy link
Copy Markdown
Contributor

✅ Security Check Passed

No security issues found – changes LGTM.

✅ Architecture Check Passed

No architecture issues found – changes LGTM.

✅ Security Check Passed

No security issues found – changes LGTM.

\n\n

✅ Architecture Check Passed

No architecture issues found – changes LGTM.

\n\n

✅ Performance Check Passed

No performance issues found – changes LGTM.


Powered by Visor from Probelabs

Last updated: 2026-05-28T11:49:34.897Z | Triggered by: pr_updated | Commit: 3d404be

💡 TIP: You can chat with Visor using /visor ask <your question>

@mativm02 mativm02 added the deps-reviewed Dependency changes reviewed and approved for CI execution label May 26, 2026
@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@mativm02
mativm02 merged commit fa8b7c4 into release-5.13 May 28, 2026
83 of 86 checks passed
@mativm02
mativm02 deleted the merge/release-5.13/TT-17154 branch May 28, 2026 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deps-reviewed Dependency changes reviewed and approved for CI execution

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant