Skip to content

Upgrade venafi-connection-lib to v0.7.0 - #849

Open
wallrj-cyberark wants to merge 1 commit into
masterfrom
venafi-connection-lib-v0.7.0
Open

wallrj-cyberark wants to merge 1 commit into
masterfrom
venafi-connection-lib-v0.7.0

Conversation

@wallrj-cyberark

Copy link
Copy Markdown
Contributor

This PR replaces the venafi-connection-lib pseudo-version from #847 with the v0.7.0 release, which contains the NGTS workspace ID change. We want a tagged version before we release v1.13.0.

Only the venafi-connection-lib line changes in go.mod. No other dependency moves, and make generate produces no changes, so the VenafiConnection CRD in both charts stays the same.

The release has six commits after the pinned commit (62e2d37):

  • Stricter NGTS zone lookup: an ambiguous template name now fails, and the zone may be a template ID. That code is only used to request certificates, which the agent does not do.
  • Go 1.27.2, golang.org/x/net v0.60.0 and release tooling fixes.
How I tested it
  • make verify passes.
  • make test-unit passes: 466 tests, 4 skipped. This includes the VenafiConnection NGTS case that checks workspace_id reaches the token request.
  • make verify-govulncheck: "Your code is affected by 0 vulnerabilities".

[with Claude]

- Replace the pseudo-version pinned for the NGTS workspace ID with the
  first release that contains it.
- The release also adds stricter NGTS zone lookup for certificate
  requests, which the agent does not make, and moves the library to
  Go 1.27.2. The VenafiConnection CRD is unchanged.

Co-Authored-By: Claude <noreply@anthropic.com>
Signed-off-by: Richard Wall <richard.wall@cyberark.com>
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.

1 participant