Skip to content

Conversation

@kimago
Copy link
Member

@kimago kimago commented Dec 30, 2025

Thanks for opening a PR. Please make sure you've read and followed the Contributing guide, including signing the Contributor License Agreement.

Feature or Bug Description

Motivation

Allow go vet to pass in environments where docker test libraries were causing it to fail, by removing use of those libraries.

https://vmw-jira.broadcom.net/browse/TNZ-72735
(Backports to v1.1 work done here, plus one additional commit to provide a 5.7-compatible healthcheck.)

abg and others added 2 commits December 30, 2025 15:39
Simplifies the dependency chain for this integration test and avoids a
new go vet error observed when bumping to the latest docker libraries.

Now this test uses a thin wrapper around the docker cli and does not
require a Go docker library + a long tail of transitive dependencies.

[TNZ-72735](https://vmw-jira.broadcom.net/browse/TNZ-72735)
percona 5.7 docker images don't contain "mysqladmin" tool
used by 8.x healthchecks, so special-case an alternate.

[TNZ-72735](https://vmw-jira.broadcom.net/browse/TNZ-72735)

Authored-by: Kim Bassett <kim.bassett@broadcom.com>
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Dec 31, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: abg / name: Andrew Garner (2650c6a)
  • ✅ login: kimago / name: Kimbrough Bassett (9375a6e)

@kimago kimago force-pushed the chore/fix-go-bump-TNZ-72735 branch from cbac94e to 9375a6e Compare December 31, 2025 20:05
@kimago kimago marked this pull request as ready for review December 31, 2025 20:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants