Skip to content
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

feat(service): Add support for host plugin AssumeRole #49

Merged
merged 2 commits into from
Sep 24, 2024

Conversation

hugoghx
Copy link
Contributor

@hugoghx hugoghx commented Sep 12, 2024

This commit also introduces a new dry run validation step when creating or updating a host catalog. Additionally, it introduces new tests related to using AssumeRole credentials, as well as more error and success state cases.

service/host/plugin.go now closely follows service/storage/plugin.go. This is by design as to bring both "halves" of the AWS plugin into parity with each other.

@hugoghx hugoghx self-assigned this Sep 12, 2024
@hugoghx
Copy link
Contributor Author

hugoghx commented Sep 12, 2024

TODO: Update Boundary SDK dep when that gets merged into the Community edition

plugin/service/host/plugin.go Show resolved Hide resolved
plugin/service/host/plugin.go Show resolved Hide resolved
Copy link
Contributor

@louisruch louisruch left a comment

Choose a reason for hiding this comment

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

Some minor comments, just a ping to @ddebko to give this a review as well please

plugin/service/host/plugin.go Outdated Show resolved Hide resolved
plugin/service/host/plugin.go Outdated Show resolved Hide resolved
Copy link
Contributor

@ddebko ddebko left a comment

Choose a reason for hiding this comment

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

I want to take a second look at the unit tests with fresh eyes. So I will review the PR again tomorrow.

plugin/service/host/plugin.go Outdated Show resolved Hide resolved
plugin/service/host/plugin.go Show resolved Hide resolved
plugin/service/host/plugin.go Show resolved Hide resolved
plugin/service/host/plugin.go Show resolved Hide resolved
@hugoghx hugoghx requested review from ddebko and louisruch September 17, 2024 14:30
Copy link
Contributor

@ddebko ddebko left a comment

Choose a reason for hiding this comment

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

This looks great, thanks for all the hard work.

This commit also introduces a new dry run validation step when creating
or updating a host catalog. Additionally, it introduces new tests
related to using AssumeRole credentials, as well as more error and
success state cases.

service/host/plugin.go now closely follows service/storage/plugin.go.
This is by design as to bring both "halves" of the AWS plugin into
parity with each other.
@hugoghx hugoghx force-pushed the hugo-host-assumerole branch from 247146f to 5f96422 Compare September 23, 2024 17:07
@hugoghx hugoghx merged commit 5f96422 into main Sep 24, 2024
2 checks passed
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.

4 participants