Skip to content

Conversation

josvazg
Copy link
Collaborator

@josvazg josvazg commented Sep 23, 2025

Summary

Try to manage Go tools with go.mod rather than devbox.

Open Questions>

  • Will the license tool complain for tools licenses or ignore them as they are not linked in?
  • How easy or painful the change is in terms automation changes?
  • Does developer UX improve?

Tools moved:

  • golangci-lint (GPLv3)
  • act (MIT)
  • govulncheck (BSD 3 Clause)
  • ginkgo (MIT)
  • operator-sdk (Apache 2.0)
  • controller-gen (Apache 2.0)
  • setup-envtest (Apache 2.0)
  • yq (MIT)

Tools moved back into devbox:

  • go-licenses was being using with go run directly. It failed as a go tool because it broke some denpendencies, so it had to be a devbox dependency instead.

Removed unused:

  • gettext
  • gotools

Replaced:

  • wget replaced by curl.

Go programs left in devbox:

  • kubectl: It does not even work as a go tool.
  • jq is a C tool.
  • docker: Is a complex client-server setup.
  • cosign broke dependencies for mgechev/revive/formatter.
  • kubernetes-helm broke dependencies for kustomize/kyaml.
  • kubebuilder broke dependencies for controller-gen.
  • shellcheck is a Haskel program.
  • awscli2 is a Python tool.
  • docker-sbom does not work as a go tool.
  • openshift is a complex setup.
  • gci broke dependencies for kustomize.
  • gh broke dependencies for controller-gen.
  • kustomize broke e2e tests as a go tool.

Proof of Work

CI should work.

Checklist

  • Have you linked a jira ticket and/or is the ticket in the title?
  • Have you checked whether your jira ticket required DOCSP changes?
  • Have you signed our CLA?

@josvazg josvazg requested a review from a team as a code owner September 23, 2025 08:00
@josvazg josvazg marked this pull request as draft September 23, 2025 08:01
@josvazg josvazg force-pushed the CLOUDP-346302/go-managed-tools branch 3 times, most recently from 8857d8d to 9ee8303 Compare September 23, 2025 09:39
@josvazg josvazg force-pushed the CLOUDP-346302/go-managed-tools branch from 658faaa to 3eaf99f Compare September 23, 2025 13:05
@josvazg josvazg force-pushed the CLOUDP-346302/go-managed-tools branch from d494e60 to 3d323be Compare September 23, 2025 13:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cloud-tests Run expensive Cloud Tests: Integration & E2E hold Do not merge for now test/e2e2/*
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant