Skip to content

chore: update dependencies and CI for RoadRunner v3, fix the acceptance tests - #841

Merged
rustatian merged 9 commits into
masterfrom
dependabot/go_modules/go.opentelemetry.io/otel/sdk-1.47.0
Oct 8, 2026
Merged

rustatian merged 9 commits into
masterfrom
dependabot/go_modules/go.opentelemetry.io/otel/sdk-1.47.0

Conversation

@dependabot

@dependabot dependabot Bot commented on behalf of github Oct 6, 2026 •

Copy link
Copy Markdown
Contributor

Reason for This PR

Update the plugin, the tests and CI for RoadRunner v3 (the v6 plugin ecosystem), and fix the failing RR <-> PHP <-> Temporal acceptance tests.

Description of Changes

  • Update Go dependencies in the root and tests modules, the PHP test dependencies (temporal/sdk ^2.19), and the Temporal CLI dev server image (1.9.1, server 1.32.0).
  • Bump all GitHub Actions to their latest releases and fix the root package coverage path, which still pointed to the v5 module.
  • Build the acceptance RoadRunner from RoadRunner master with the local plugin. The Velox build used RoadRunner v2025.1.4, which imports roadrunner-temporal/v5, so it failed and the job silently tested the released rr.
  • Bump the PHP SDK submodule to fix the JsonProtobufTest TypeError that stopped the acceptance worker at boot.
  • Simplify the plugin internals without behavior changes: remove duplicated code, dead branches, the unused frame pool and unused protocol helpers.
  • Run the general LA tests with the LA worker (.rr-proto-la.yaml), like the TLS suite. Remove exact duplicate tests, merge the test server constructors and share the RPC test helpers.
  • Simplify CI: run all root module unit tests in one step (this adds the aggregatedpool and internal tests), let dependabot update tests/go.mod, untrack the generated TLS test certs and drop dead linter, codecov and compose config.
  • Replace github.com/pborman/uuid with github.com/google/uuid in the tests. It is already in the module graph and is the widely used implementation.

License Acceptance

By submitting this pull request, I confirm that my contribution is made under
the terms of the MIT license.

PR Checklist

  • All commits in this PR are signed (git commit -s).
  • The reason for this PR is clearly provided (issue no. or explanation).
  • The description of changes is clear and encompassing.
  • Any required documentation changes (code and docs) are included in this PR.
  • Any user-facing changes are mentioned in CHANGELOG.md.
  • All added/changed functionality is tested.

Bumps [go.opentelemetry.io/otel/sdk](https://github.com/open-telemetry/opentelemetry-go) from 1.46.0 to 1.47.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@v1.46.0...v1.47.0)

---
updated-dependencies:
- dependency-name: go.opentelemetry.io/otel/sdk
  dependency-version: 1.47.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot
dependabot Bot requested a review from rustatian as a code owner October 6, 2026 02:23
@dependabot dependabot Bot added dependencies Pull requests that update a dependency file go Pull requests that update Go code labels Oct 6, 2026
Signed-off-by: Valery Piashchynski <piashchynski.valery@gmail.com>
Signed-off-by: Valery Piashchynski <piashchynski.valery@gmail.com>
…l plugin

The Velox build used RoadRunner v2025.1.4, which imports roadrunner-temporal/v5,
so it failed against the v6 module and the job silently ran the released rr.
Bump the PHP SDK submodule to fix the JsonProtobufTest TypeError.

Signed-off-by: Valery Piashchynski <piashchynski.valery@gmail.com>
Remove duplicated code and dead branches without behavior changes: one
payload pool and one pool response reader in aggregatedpool, shared status
and replay helpers, the prometheus default sanitizer, and the unused frame
pool, codec guards and protocol helpers.

Signed-off-by: Valery Piashchynski <piashchynski.valery@gmail.com>
The general LA tests started worker.php, so they ran the same code as their
non-LA twins. Use .rr-proto-la.yaml like the TLS suite. Remove exact duplicate
tests, move shared RPC helpers to tests/helpers, merge the test server
constructors, and replace pborman/uuid with google/uuid.

Signed-off-by: Valery Piashchynski <piashchynski.valery@gmail.com>
Run all root module unit tests in one step (this adds the aggregatedpool and
internal tests), install the PHP test deps with composer-install, let
dependabot update tests/go.mod, and drop dead linter, codecov and compose
config. CI generates the TLS test certs on every run.

Signed-off-by: Valery Piashchynski <piashchynski.valery@gmail.com>
@rustatian rustatian changed the title chore(deps): bump go.opentelemetry.io/otel/sdk from 1.46.0 to 1.47.0 chore: update dependencies and CI for RoadRunner v3, fix the acceptance tests Oct 8, 2026
… container

openssl creates new private keys with mode 0600. The Temporal container runs
as another user, so it could not read the keys after the certs were untracked.

Signed-off-by: Valery Piashchynski <piashchynski.valery@gmail.com>
@rustatian
rustatian merged commit d7fad6e into master Oct 8, 2026
12 checks passed
@dependabot
dependabot Bot deleted the dependabot/go_modules/go.opentelemetry.io/otel/sdk-1.47.0 branch October 8, 2026 15:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file go Pull requests that update Go code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant