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(remote/aws): support AWS Secrets Manager as remote component #718
base: main
Are you sure you want to change the base?
feat(remote/aws): support AWS Secrets Manager as remote component #718
Conversation
I think it would be nice if it could poll but set the default to not poll. Most of our pulling mechanisms support it so it would be odd if this one did not have that capability. |
Thanks Matt! I'll implement the poller and fix whatever makes the tests failed. Maybe in the long run, we can have a reusable implementation. But that's for another day, I suppose :D |
@@ -106,11 +100,11 @@ func (s *Component) Run(ctx context.Context) error { | |||
func (s *Component) Update(args component.Arguments) error { | |||
newArgs := args.(Arguments) | |||
|
|||
s3cfg, err := generateS3Config(newArgs) | |||
awsCfg, err := aws_common_config.GenerateAWSConfig(newArgs.Options.Client) | |||
if err != nil { | |||
return nil |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@rfratto pinging 'cause you authored this in the past. Do you know why this returns nil
in the present of non-nil error? 👀
docs/sources/reference/components/remote.aws.secrets_manager.md
Outdated
Show resolved
Hide resolved
docs/sources/reference/components/remote.aws.secrets_manager.md
Outdated
Show resolved
Hide resolved
docs/sources/reference/components/remote.aws.secrets_manager.md
Outdated
Show resolved
Hide resolved
docs/sources/reference/components/remote.aws.secrets_manager.md
Outdated
Show resolved
Hide resolved
docs/sources/reference/components/remote.aws.secrets_manager.md
Outdated
Show resolved
Hide resolved
docs/sources/reference/components/remote.aws.secrets_manager.md
Outdated
Show resolved
Hide resolved
Docs look ok. |
68ec808
to
e453b57
Compare
…tions used in remote AWS components Signed-off-by: hainenber <dotronghai96@gmail.com>
Signed-off-by: hainenber <dotronghai96@gmail.com>
Signed-off-by: hainenber <dotronghai96@gmail.com>
Signed-off-by: hainenber <dotronghai96@gmail.com>
…n test in Docker-present env Signed-off-by: hainenber <dotronghai96@gmail.com>
Signed-off-by: hainenber <dotronghai96@gmail.com>
…ial poller is configured Signed-off-by: hainenber <dotronghai96@gmail.com>
Co-authored-by: Clayton Cornell <131809008+clayton-cornell@users.noreply.github.com>
Signed-off-by: hainenber <dotronghai96@gmail.com>
Signed-off-by: hainenber <dotronghai96@gmail.com>
e453b57
to
20a34d0
Compare
PR Description
Which issue(s) this PR fixes
Closes #689
Notes to the Reviewer
component.Component
incomponenttest
in order to verify the component's health status. Is this A-OK?PR Checklist