Check log_groups existence and find log_stream with pagenation

fluent-plugins-nursery / fluent-plugin-cloudwatch-logs

CloudWatch Logs Plugin for Fluentd

MIT License

201 stars 141 forks source link

Closed cosmo0920 closed 4 years ago

cosmo0920 commented 4 years ago

This should be needed because default response contains only 50 log_group names.

Closes #45 Fixes #48

Related to #49

Signed-off-by: Hiroshi Hatake hatake@clear-code.com

cosmo0920 commented 4 years ago

@pecigonzalo @owen Could you check this patch?

pecigonzalo commented 4 years ago

@cosmo0920 unfortunately Im no longer working at that company, so I cant verify, but here is the fix we applied, https://github.com/coyainsurance/fluent-plugin-cloudwatch-logs/commit/b87380c612a1ad00b255ce05a7659f82423c83ad im not entirely sure why we did not PR that back here.

cosmo0920 commented 4 years ago

No problem. Thank you for the response. It seems that your patch is almost same as mine.