8000 ForgeOps-4394 secret manager v2 by lee-baines · Pull Request #272 · ForgeRock/secret-agent · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

ForgeOps-4394 secret manager v2 #272

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

Conversation

lee-baines
Copy link
Contributor

Cherry-pick @dnitsch's earlier PR to update secret-agent to work with secret manager v2 but create env var for AWS region so the secretmanager client can read the configured value in the secretagentconfiguration

* fix: securit update
update go 1.21
update AWS SDK v2 for secrets manager

* fix: update docs

* fix: run fmt locally

* fix: update SHAs

* fix: set buildvcs false

* fix: revert

* fix: remove cancel

* fix: update go directives

* fix: remove deprecated io/ioutil

* fix: set buildvcs to false

* fix: add GOFLAGS to the docker context

* fix: add more unit tests

* fix: re-run createcluster workflow...

* fix: update crypto pkg
@lee-baines lee-baines force-pushed the FORGEOPS-4394-secret-manager-v2 branch from a983758 to b716e5a Compare May 28, 2024 12:17
Copy link
Collaborator
@gandru gandru left a comment

Choose a reason for hiding this comment

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

LGTM

@lee-baines lee-baines self-assigned this May 28, 2024
@lee-baines lee-baines merged commit 5d52371 into ForgeRock:master May 28, 2024
6 checks passed
@lee-baines lee-baines deleted the FORGEOPS-4394-secret-manager-v2 branch May 28, 2024 16:09
@pearj
Copy link
Contributor
pearj commented May 30, 2024

I just noticed this commit also commits the kind binary. Was that on purpose?

@lee-baines
Copy link
Contributor Author

Good spot @pearj , I'll get this removed

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

Successfully merging this pull request may close these issues.

5 participants
0