Skip to content

Remove goreleaser Pro key from Atlas CLI Kubernetes plugin#188

Open
blva wants to merge 6 commits intomainfrom
CLOUDP-401819-remove-goreleaser-pro-key
Open

Remove goreleaser Pro key from Atlas CLI Kubernetes plugin#188
blva wants to merge 6 commits intomainfrom
CLOUDP-401819-remove-goreleaser-pro-key

Conversation

@blva
Copy link
Copy Markdown
Collaborator

@blva blva commented May 1, 2026

Summary

The project uses goreleaser Pro (via a license key managed in Evergreen) but uses no Pro-only features. This change switches to the community edition.

  • Replace the goreleaser_pro_tar_gz Pro binary download with a pinned community release (v2.15.4) in the Evergreen install goreleaser function.
  • Remove GORELEASER_KEY export from build/package/package.sh.
  • Remove goreleaser_key from include_expansions_in_env in the Evergreen package function.
  • Update the goreleaser config schema reference from schema-pro.json to schema.json.

Closes CLOUDP-401819.

Test plan

  • Run a patch build against the test-trace Evergreen task (test_mode: true, runs --snapshot) to confirm goreleaser installs and packages successfully without the Pro key.
  • Verify grep -r GORELEASER_KEY . and grep -r goreleaser_pro . return no results.

The project uses no Pro-only features, so there is no reason to carry
the Pro license key. Replace the Pro binary download with a pinned
community release, remove the GORELEASER_KEY export from the packaging
script, and update the goreleaser schema reference accordingly.

CLOUDP-401819
@blva blva requested a review from a team as a code owner May 1, 2026 18:01
@blva blva marked this pull request as draft May 1, 2026 18:02
Copy link
Copy Markdown
Collaborator

@josvazg josvazg left a comment

Choose a reason for hiding this comment

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

LGTM, waiting for CI to agree

Great to see we can go back to the community version here!

blva and others added 2 commits May 5, 2026 14:09
rhel90-small cannot reach GitHub's release CDN directly, so a hardcoded
GitHub URL fails at setup. Keep the same expansion variable pattern as
the original goreleaser_pro_tar_gz, renamed to goreleaser_tar_gz.

A project admin needs to add goreleaser_tar_gz in Evergreen project
settings pointing to the community binary at the same internal host.

CLOUDP-401819
@blva blva marked this pull request as ready for review May 6, 2026 08:19
@blva
Copy link
Copy Markdown
Collaborator Author

blva commented May 6, 2026

triggering a patch before merging

go install github.com/google/go-licenses@latest resolves to v1.6.0,
which fails on stdlib packages with Go 1.21+. The parent atlas-cli
project already uses the go tool directive pattern (v2.0.1) which
handles this correctly.

Add go-licenses/v2 as a tool directive in go.mod and switch
generate-notices.sh to use go tool go-licenses.
@josvazg
Copy link
Copy Markdown
Collaborator

josvazg commented May 6, 2026

@blva Good to merge?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants