Skip to content

chore(deps): update test-tooling (9.6) - #8212

Open
elastic-renovate-prod[bot] wants to merge 1 commit into
9.6from
renovate/9.6-golang/test-tooling
Open

elastic-renovate-prod[bot] wants to merge 1 commit into
9.6from
renovate/9.6-golang/test-tooling

Conversation

@elastic-renovate-prod

Copy link
Copy Markdown
Contributor

This PR contains the following updates:

Package Type Update Change
github.com/Shopify/toxiproxy/v2 require minor v2.12.0 → v2.13.0
github.com/testcontainers/testcontainers-go require minor v0.44.0 → v0.45.0
github.com/testcontainers/testcontainers-go/modules/toxiproxy require minor v0.44.0 → v0.45.0

Release Notes

testcontainers/testcontainers-go (github.com/testcontainers/testcontainers-go)

v0.45.0

Compare Source

What's Changed
⚠️ Breaking Changes

Client SDK dependencies have moved into separate examples modules. For CockroachDB, ConnectionConfig is now available through cockroachdbexamples.NewContainer(c) in modules/cockroachdb/examples. The parent’s ConnectionString and MustConnectionString now return PostgreSQL URLs instead of registered pgx handles, and importing the parent no longer registers the pgx SQL driver. Use the examples adapter to retain the previous SDK behavior, or configure the driver yourself.

For Dex, AddClient, RemoveClient, AddUser, and RemoveUser have moved from *dex.Container to dexexamples.NewAdminClient(c) in modules/dex/examples. Static configuration through WithClient and WithUser is unchanged.

For Cosmos DB, ContainerPolicy and NewContainerPolicy have moved from modules/azure/cosmosdb to modules/azure/examples/cosmosdb. Update the adapter import; its constructor still accepts the parent container. Run, Container, and ConnectionString remain unchanged in the parent module.

🔒 Security
🚀 Features
🐛 Bug Fixes
📖 Documentation
🧹 Housekeeping
📦 Dependency updates

Configuration

📅 Schedule: (UTC)

  • Branch creation
    • "on monday"
  • Automerge
    • At any time (no schedule defined)

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Mend Renovate CLI.

@elastic-renovate-prod elastic-renovate-prod Bot added automation backport-skip Skip notification from the automated backport with mergify labels Nov 9, 2026
@elastic-renovate-prod
elastic-renovate-prod Bot requested a review from a team as a code owner November 9, 2026 01:37
@elastic-renovate-prod
elastic-renovate-prod Bot requested a balanced review from Copilot and removed request for a team November 9, 2026 01:37
@elastic-renovate-prod

Copy link
Copy Markdown
Contributor Author

ℹ️ Artifact update notice

File name: testing/go.mod

In order to perform the update(s) described in the table above, Renovate ran the go get command, which resulted in the following additional change(s):

  • 5 additional dependencies were updated
  • The go directive was updated for compatibility reasons

Details:

Package Change
go 1.26.10 -> 1.27.0
github.com/Azure/go-ansiterm v0.0.0-20250102033503-faa5f7b0171c -> v0.0.0-20261006220739-8c912ac31dd4
github.com/docker/go-connections v0.7.0 -> v0.8.2
github.com/magiconair/properties v1.8.10 -> v1.18.12
github.com/moby/go-archive v0.3.0 -> v0.3.3
github.com/moby/moby/client v0.5.0 -> v0.6.2

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

The testing module requires Go 1.27 while E2E jobs remain pinned to Go 1.26.10, forcing automatic toolchain downloads or causing failures when toolchain switching is disabled.

1 open issue
What changed in this PR

Updates Fleet Server’s test-tooling dependencies on the 9.6 branch, including container orchestration and network fault-injection libraries.

Changes:

  • Upgrades Toxiproxy to v2.13.0 and Testcontainers to v0.45.0.
  • Refreshes transitive dependencies and module checksums.
  • Raises the testing module’s minimum Go version from 1.26.10 to 1.27.0.
File Description
testing/​go.sum Updates checksums for upgraded dependencies and removes obsolete entries.
testing/​go.mod Updates test-tooling requirements and the minimum Go version.

Review effort: Balanced

Comment thread testing/go.mod
module github.com/elastic/fleet-server/testing

go 1.26.10
go 1.27.0
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

Dependency Update Analysis

Summary: This PR updates Go test-only dependencies in testing/go.mod; overall risk is low-to-moderate and compatible with current repository usage.

github.com/testcontainers/testcontainers-go + modules/toxiproxy v0.44.0 → v0.45.0

Ecosystem: Go

Check Result
Breaking changes ✅ None found for this repo usage
Testable in PR ✅ Yes (Buildkite PR pipeline runs unit/integration/e2e)
Changelog highlights (v0.44.0 → v0.45.0)
  • Upstream release includes breaking changes for module-specific client adapters (CockroachDB/Dex/examples paths).
  • Security-related note: dependency/security upgrades (including OTel log SDK bump) are included.
  • No CVE/GHSA specifically tied to APIs used by this repository’s current testcontainers-go usage was identified in the release body.
Usage in this repository
  • testing/e2e/agent_container_test.go imports and uses testcontainers container lifecycle APIs.
  • testing/e2e/stand_alone_container_test.go uses testcontainers.GenericContainer and container endpoint/health checks.
  • testing/e2e/scaffold/scaffold.go uses testcontainers-go/modules/toxiproxy.
Compatibility assessment
  • The reported upstream breaking changes target modules not used here (CockroachDB/Dex client adapters).
  • Current usage is container orchestration + toxiproxy test scaffolding, which remains compatible.
  • Impact is limited to test/e2e paths under testing/ (no production runtime dependency surface).

github.com/Shopify/toxiproxy/v2 v2.12.0 → v2.13.0

Ecosystem: Go

Check Result
Breaking changes ✅ None found for this repo usage
Testable in PR ✅ Yes (Buildkite PR pipeline covers e2e test paths using toxiproxy)
Changelog highlights (v2.12.0 → v2.13.0)
  • Added optional bearer-token auth for Toxiproxy HTTP API (TOXIPROXY_AUTH_TOKEN) and warning on unauthenticated non-loopback API.
  • Fixed hangs/panics in toxic handling (remove toxic, bandwidth toxic rate edge cases, slicer toxic validation).
  • No CVE/GHSA IDs listed in the module changelog section for 2.13.0.
Usage in this repository
  • testing/e2e/stand_alone_container_test.go and testing/e2e/stand_alone_test.go use toxiproxy/client to create/remove toxics and validate Fleet Server degradation/recovery behavior.
Compatibility assessment
  • Repository usage (creating/removing toxics, proxy lookups) is unchanged by the noted updates.
  • New auth option is additive; existing unauthenticated local/integration usage remains valid.
  • Fixes reduce flake/hang risk in failure-injection tests.

Go toolchain directive in testing/go.mod 1.26.10 → 1.27.0

Ecosystem: Go

Check Result
Breaking changes ✅ None found in current CI mode; requires Go 1.27 toolchain for testing/ module
Testable in PR ✅ Yes
Changelog highlights (1.26.10 → 1.27.0)
  • This is a Go toolchain requirement bump for the nested testing module (not an application runtime library bump).
Usage in this repository
  • mage e2e paths execute go test with cmd.Dir = "testing" (magefile.go), so this directive applies to test execution.
Compatibility assessment
  • With GOTOOLCHAIN=auto, go resolves and runs successfully for testing/.
  • Operational impact is limited to contributor/CI environments that force GOTOOLCHAIN=local and stay on Go 1.26.x.

Labels Applied

  • oblt-aw/ai/merge-ready — risk low-to-moderate, no repo-used breaking API/input changes identified, and impact is confined to test tooling.

What is this? | From workflow: Observability Agentic Workflow — Pull Request

Give us feedback! React with 🚀 if perfect, 👍 if helpful, 👎 if not.

@github-actions

github-actions Bot commented Nov 9, 2026

Copy link
Copy Markdown
Contributor

Automerge skipped

Automerge skipped this pull request because the go-dependencies collection is not enabled for this repository.

To allow automerge for this kind of update, enable the matching collection under Automerge on the Control Plane Dashboard (the oblt-aw/dashboard issue). See Automerge dependency collections for what each collection covers.

See Opt in or opt out for how to enable or disable Automerge collections on the Control Plane Dashboard.

Collections currently enabled for automerge: github-actions, pre-commit, vm-images

Changed files considered:

  • testing/go.mod
  • testing/go.sum

@github-actions

github-actions Bot commented Nov 9, 2026

Copy link
Copy Markdown
Contributor

TL;DR

Buildkite failed in Run check-ci because mage check:ci detected uncommitted auto-fix changes in testing/e2e/scaffold/scaffold.go (reported as needs update + git update-index --refresh failure). The PR should include the auto-generated/go-fix edits and rerun CI.

Remediation

  • On branch renovate/9.6-golang/test-tooling, run mage check:ci (or at minimum mage check:fix), commit any resulting file changes (notably testing/e2e/scaffold/scaffold.go), and push.
  • Validate with mage check:ci locally before re-running Buildkite.
Investigation details

Root Cause

check-ci failed on repository cleanliness, not a test assertion. The log shows a generated diff in:

  • testing/e2e/scaffold/scaffold.go around StartToxiproxy (@@ -691,12 +691,10 and @@ -704,12 +702,10)

After showing that diff, CI reports:

  • testing/e2e/scaffold/scaffold.go: needs update
  • Error: git update-index failure: running "git update-index --refresh" failed with exit code 1

Given this PR updates test-tooling dependencies (testing/go.mod, testing/go.sum), the dependency bump likely changed go fix/format output and the corresponding source update was not included in the PR commit.

Evidence

  • Build: https://buildkite.com/elastic/fleet-server/builds/17939
  • Job/step: Run check-ci (.buildkite/scripts/check_ci.sh)
  • Key log excerpt:
    • testing/e2e/scaffold/scaffold.go: needs update
    • Error: git update-index failure: running "git update-index --refresh" failed with exit code 1

Verification

  • Reviewed pre-fetched failure summary and full failing step log from /tmp/gh-aw/buildkite-logs/fleet-server-white_check_mark-run-check-ci.txt.
  • Fetched failing commit a643b7b6205a058d858ce80a07a20ae3789f0edd and confirmed PR diff contains only dependency-file changes (testing/go.mod, testing/go.sum).
  • Did not rerun full Buildkite pipeline locally.

Follow-up

  • I could not reliably inspect prior low-integrity PR comments in this environment, so if this diagnosis matches an earlier detective note, please treat this as confirmation rather than a new root cause.

What is this? | From workflow: Observability Agentic Workflow — Status

Give us feedback! React with 🚀 if perfect, 👍 if helpful, 👎 if not.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automation backport-skip Skip notification from the automated backport with mergify dependency oblt-aw/ai/merge-ready skip-changelog Team:Elastic-Agent-Control-Plane Label for the Agent Control Plane team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant