Skip to content

update cloudwatch logs client in message publisher #8280

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
May 12, 2023
Merged

update cloudwatch logs client in message publisher #8280

merged 2 commits into from
May 12, 2023

Conversation

pinzon
Copy link
Member

@pinzon pinzon commented May 8, 2023

This PR updates the Cloudwatch Logs client in the EventBridge message publisher

@coveralls
Copy link

coveralls commented May 8, 2023 •
edited
Loading

Coverage Status

Coverage: 82.169% (+0.008%) from 82.161% when pulling ee91648 on test_events-logs-iam into 42f62d3 on master.

@github-actions
Copy link

github-actions bot commented May 8, 2023 •
edited
Loading

LocalStack Community integration with Pro

2 034 tests   1 753 ✔️  1h 16m 25s ⏱️
       2 suites     281 💤
       2 files           0 ❌

Results for commit ee91648.

♻️ This comment has been updated with latest results.

@dfangl dfangl added the semver: patch Non-breaking changes which can be included in patch releases label May 10, 2023
@pinzon pinzon marked this pull request as ready for review May 10, 2023 13:29
@pinzon pinzon requested a review from dominikschubert May 10, 2023 13:33
@pinzon pinzon merged commit e1d4725 into master May 12, 2023
@pinzon pinzon deleted the test_events-logs-iam branch May 12, 2023 13:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dominikschubert dominikschubert Awaiting requested review from dominikschubert

Assignees
No one assigned
Labels
semver: patch Non-breaking changes which can be included in patch releases
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@pinzon @coveralls @dfangl