chore(deps): bump the go_modules group across 3 directories with 3 updates#2331
chore(deps): bump the go_modules group across 3 directories with 3 updates#2331dependabot[bot] wants to merge 1 commit intomainfrom
Conversation
…dates Bumps the go_modules group with 2 updates in the /packages/api directory: [github.com/gohugoio/hugo](https://github.com/gohugoio/hugo) and [go.opentelemetry.io/otel/exporters/otlp/otlplog/otlploghttp](https://github.com/open-telemetry/opentelemetry-go). Bumps the go_modules group with 1 update in the /packages/orchestrator directory: [github.com/go-jose/go-jose/v4](https://github.com/go-jose/go-jose). Bumps the go_modules group with 2 updates in the /packages/shared directory: [go.opentelemetry.io/otel/exporters/otlp/otlplog/otlploghttp](https://github.com/open-telemetry/opentelemetry-go) and [github.com/go-jose/go-jose/v4](https://github.com/go-jose/go-jose). Updates `github.com/gohugoio/hugo` from 0.139.4 to 0.159.2 - [Release notes](https://github.com/gohugoio/hugo/releases) - [Commits](gohugoio/hugo@v0.139.4...v0.159.2) Updates `go.opentelemetry.io/otel/exporters/otlp/otlplog/otlploghttp` from 0.15.0 to 0.19.0 - [Release notes](https://github.com/open-telemetry/opentelemetry-go/releases) - [Changelog](https://github.com/open-telemetry/opentelemetry-go/blob/main/CHANGELOG.md) - [Commits](open-telemetry/opentelemetry-go@v0.15.0...v0.19.0) Updates `github.com/go-jose/go-jose/v4` from 4.1.3 to 4.1.4 - [Release notes](https://github.com/go-jose/go-jose/releases) - [Commits](go-jose/go-jose@v4.1.3...v4.1.4) Updates `go.opentelemetry.io/otel/exporters/otlp/otlplog/otlploghttp` from 0.15.0 to 0.19.0 - [Release notes](https://github.com/open-telemetry/opentelemetry-go/releases) - [Changelog](https://github.com/open-telemetry/opentelemetry-go/blob/main/CHANGELOG.md) - [Commits](open-telemetry/opentelemetry-go@v0.15.0...v0.19.0) Updates `github.com/go-jose/go-jose/v4` from 4.1.3 to 4.1.4 - [Release notes](https://github.com/go-jose/go-jose/releases) - [Commits](go-jose/go-jose@v4.1.3...v4.1.4) --- updated-dependencies: - dependency-name: github.com/gohugoio/hugo dependency-version: 0.159.2 dependency-type: indirect dependency-group: go_modules - dependency-name: go.opentelemetry.io/otel/exporters/otlp/otlplog/otlploghttp dependency-version: 0.19.0 dependency-type: indirect dependency-group: go_modules - dependency-name: github.com/go-jose/go-jose/v4 dependency-version: 4.1.4 dependency-type: indirect dependency-group: go_modules - dependency-name: go.opentelemetry.io/otel/exporters/otlp/otlplog/otlploghttp dependency-version: 0.19.0 dependency-type: indirect dependency-group: go_modules - dependency-name: github.com/go-jose/go-jose/v4 dependency-version: 4.1.4 dependency-type: indirect dependency-group: go_modules ... Signed-off-by: dependabot[bot] <support@github.com>
| github.com/gin-contrib/sse v1.1.1 // indirect | ||
| github.com/go-faster/city v1.0.1 // indirect | ||
| github.com/go-faster/errors v0.7.1 // indirect | ||
| github.com/go-jose/go-jose/v4 v4.1.3 // indirect | ||
| github.com/go-jose/go-jose/v4 v4.1.4 // indirect | ||
| github.com/go-logr/logr v1.4.3 // indirect | ||
| github.com/go-logr/stdr v1.2.2 // indirect | ||
| github.com/go-ole/go-ole v1.3.0 // indirect |
There was a problem hiding this comment.
🟡 packages/orchestrator/go.mod was not updated with go mod tidy after otel/log and otel/sdk/log were bumped to v0.19.0 in packages/shared — orchestrator still records v0.15.0 for both indirect deps and v1.79.3 for grpc (shared moved to v1.80.0). Go MVS resolves the correct higher versions at build time via the workspace and replace directive, so compiled binaries are correct, but the stale go.sum entries (missing v0.19.0 checksums) can cause build failures when building orchestrator outside workspace mode.
Extended reasoning...
What the bug is: This PR bumps go.opentelemetry.io/otel/log and go.opentelemetry.io/otel/sdk/log from v0.15.0 to v0.19.0 in packages/shared and packages/api, and bumps google.golang.org/grpc from v1.79.3 to v1.80.0 in both. However, packages/orchestrator/go.mod was not updated. It still declares otel/log v0.15.0 and otel/sdk/log v0.15.0 as indirect dependencies (visible in the unmodified portion of the diff), and grpc v1.79.3 as a direct dependency. This indicates go mod tidy was not run in packages/orchestrator after the shared/api changes.
The specific code path: packages/orchestrator/go.mod lines 157-163 (otel/log and otel/sdk/log at v0.15.0, grpc at v1.79.3). packages/shared/go.mod was updated to require otel/log v0.19.0 and otel/sdk/log v0.19.0 as direct deps, and grpc v1.80.0. orchestrator uses a local replace directive pointing to ../shared.
Why existing code does not prevent it: Go MVS (Minimum Version Selection) silently resolves to the higher versions at build time by aggregating requirements across the dependency graph. The workspace (go.work) further helps aggregate sum files. So the compiled binary uses the correct versions, masking the inconsistency during normal workspace builds.
Addressing the refutations: The refuters correctly note that MVS guarantees correct runtime behavior. For the grpc direct dependency specifically, they are right that go mod tidy does not bump a direct dependency just because a transitive dep requires a higher version — each module records its own minimum. However, the otel indirect deps are a different matter: go mod tidy WOULD update those. Furthermore, the deeper issue is that orchestrator/go.sum only has checksums for otel/log v0.15.0 and otel/sdk/log v0.15.0, not v0.19.0. When MVS resolves to v0.19.0 (because shared requires it), but go.sum lacks the v0.19.0 hash, building orchestrator in isolation outside workspace mode (cd packages/orchestrator && go build ./...) would fail with a missing go.sum entry error under Go 1.16+ default -mod=readonly behavior.
Impact: Any CI job or developer that builds orchestrator in isolation rather than via the workspace will encounter a build failure. The stale lower bounds also mislead developers about what versions are actually in use.
How to fix: Run go mod tidy in packages/orchestrator. This updates the indirect dep lower bounds to v0.19.0 and adds the required checksums to go.sum. For grpc (direct dep), manually update to v1.80.0 if orchestrator needs features from that version, or leave at v1.79.3 knowing MVS will resolve higher.
Step-by-step proof: (1) shared/go.mod bumped: otel/log v0.15.0 -> v0.19.0 as direct dep. (2) orchestrator/go.mod unchanged: otel/log v0.15.0 // indirect. (3) orchestrator depends on shared via replace directive, so at build time Go MVS selects v0.19.0. (4) orchestrator/go.sum has only hashes for otel/log v0.15.0 — no v0.19.0 entries. (5) Outside workspace mode: go build in packages/orchestrator fails with missing go.sum entry for go.opentelemetry.io/otel/log@v0.19.0. (6) Fix: run go mod tidy in packages/orchestrator.
|
This pull request was built based on a group rule. Closing it will not ignore any of these versions in future pull requests. To ignore these dependencies, configure ignore rules in dependabot.yml |
Bumps the go_modules group with 2 updates in the /packages/api directory: github.com/gohugoio/hugo and go.opentelemetry.io/otel/exporters/otlp/otlplog/otlploghttp.
Bumps the go_modules group with 1 update in the /packages/orchestrator directory: github.com/go-jose/go-jose/v4.
Bumps the go_modules group with 2 updates in the /packages/shared directory: go.opentelemetry.io/otel/exporters/otlp/otlplog/otlploghttp and github.com/go-jose/go-jose/v4.
Updates
github.com/gohugoio/hugofrom 0.139.4 to 0.159.2Release notes
Sourced from github.com/gohugoio/hugo's releases.
... (truncated)
Commits
5f4646areleaser: Bump versions for release of 0.159.2479fe6cFix potential content XSS by escaping dangerous URLs in links and images81a5cdcreleaser: Add standard withdeploy release assetsdf520e3resources/page: Fix shared reader in Source.ValueAsOpenReadSeekCloserb55d452testing: Simplify line ending handling in testsea7eac6readme: Update Go version to 1.25.0458ebddreleaser: Prepare repository for 0.160.0-DEV86c7d3areleaser: Bump versions for release of 0.159.142289d7minifiers: Keep x-bind and blank namespace in SVG minification0c013c2Adjust depreceated syntax in testsUpdates
go.opentelemetry.io/otel/exporters/otlp/otlplog/otlploghttpfrom 0.15.0 to 0.19.0Release notes
Sourced from go.opentelemetry.io/otel/exporters/otlp/otlplog/otlploghttp's releases.
... (truncated)
Changelog
Sourced from go.opentelemetry.io/otel/exporters/otlp/otlplog/otlploghttp's changelog.
... (truncated)
Commits
2b4fa96Release v0.19.0 (#1710)4beb704sdk/trace: removing ApplyConfig and Config (#1693)1d42be1Rename WithDefaultSampler TracerProvider option to WithSampler and update doc...860d5d8Add flag to determine whether SpanContext is remote (#1701)0fe65e6Comply with OpenTelemetry attributes specification (#1703)8888435Bump google.golang.org/api from 0.40.0 to 0.41.0 in /exporters/trace/jaeger (...345f264breaking(zipkin): removes servicName from zipkin exporter. (#1697)62cbf0fPopulate Jaeger's Span.Process from Resource (#1673)28eaaa9Add a test to prove the Tracer is safe for concurrent calls (#1665)8b1be11Rename resource pkg label vars and methods (#1692)Updates
github.com/go-jose/go-jose/v4from 4.1.3 to 4.1.4Release notes
Sourced from github.com/go-jose/go-jose/v4's releases.
Commits
0e59876Merge commit from forkddffdbcBump actions/checkout from 5 to 6 (#213)Updates
go.opentelemetry.io/otel/exporters/otlp/otlplog/otlploghttpfrom 0.15.0 to 0.19.0Release notes
Sourced from go.opentelemetry.io/otel/exporters/otlp/otlplog/otlploghttp's releases.
... (truncated)
Changelog
Sourced from go.opentelemetry.io/otel/exporters/otlp/otlplog/otlploghttp's changelog.
... (truncated)
Commits
2b4fa96Release v0.19.0 (#1710)4beb704sdk/trace: removing ApplyConfig and Config (#1693)1d42be1Rename WithDefaultSampler TracerProvider option to WithSampler and update doc...860d5d8Add flag to determine whether SpanContext is remote (#1701)0fe65e6Comply with OpenTelemetry attributes specification (#1703)8888435Bump google.golang.org/api from 0.40.0 to 0.41.0 in /exporters/trace/jaeger (...345f264breaking(zipkin): removes servicName from zipkin exporter. (#1697)62cbf0fPopulate Jaeger's Span.Process from Resource (#1673)28eaaa9Add a test to prove the Tracer is safe for concurrent calls (#1665)8b1be11Rename resource pkg label vars and methods (#1692)Updates
github.com/go-jose/go-jose/v4from 4.1.3 to 4.1.4Release notes
Sourced from github.com/go-jose/go-jose/v4's releases.
Commits
0e59876Merge commit from forkddffdbcBump actions/checkout from 5 to 6 (#213)Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore <dependency name> major versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself)@dependabot ignore <dependency name> minor versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself)@dependabot ignore <dependency name>will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself)@dependabot unignore <dependency name>will remove all of the ignore conditions of the specified dependency@dependabot unignore <dependency name> <ignore condition>will remove the ignore condition of the specified dependency and ignore conditionsYou can disable automated security fix PRs for this repo from the Security Alerts page.