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

fix(aws_cloudwatch_logs sink): Fix healthcheck to allow for log group creation at runtime #7601

Merged
merged 6 commits into from
May 28, 2021
Merged

fix(aws_cloudwatch_logs sink): Fix healthcheck to allow for log group creation at runtime #7601

merged 6 commits into from
May 28, 2021

Conversation

spencergilbert
Copy link
Contributor

Signed-off-by: Spencer Gilbert [email protected]

Closes #7592

@spencergilbert spencergilbert requested a review from a team May 26, 2021 14:56
@spencergilbert spencergilbert changed the title fix(aws_cloudwatch_logs source): Fix healthcheck to allow for log group creation at runtime fix(aws_cloudwatch_logs sink): Fix healthcheck to allow for log group creation at runtime May 26, 2021
@spencergilbert spencergilbert added the sink: aws_cloudwatch_logs Anything `aws_cloudwatch_logs` sink related label May 27, 2021
Copy link
Member

@jszwedko jszwedko left a comment

Choose a reason for hiding this comment

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

Nice! I left a couple of suggestions for the info messages to provide more context since I don't think they will otherwise have much.

src/sinks/aws_cloudwatch_logs/mod.rs Outdated Show resolved Hide resolved
src/sinks/aws_cloudwatch_logs/mod.rs Outdated Show resolved Hide resolved
Signed-off-by: Spencer Gilbert <[email protected]>
@spencergilbert
Copy link
Contributor Author

Nice! I left a couple of suggestions for the info messages to provide more context since I don't think they will otherwise have much.

Doh, sorta figured it'd be built out like component_name::healthcheck message - updated.

@spencergilbert spencergilbert merged commit 498de2e into vectordotdev:master May 28, 2021
@spencergilbert spencergilbert deleted the cloudwatch-healthcheck branch May 28, 2021 21:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
sink: aws_cloudwatch_logs Anything `aws_cloudwatch_logs` sink related
Projects
None yet
Development

Successfully merging this pull request may close these issues.

(sink) aws_cloudwatch_logs: healthcheck errors if log group doesn't exist
2 participants