-
Notifications
You must be signed in to change notification settings - Fork 49
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
Add default fallback log group and stream names #111
Conversation
We can update the go version. We should first update it here and check the build: https://github.com/aws/aws-for-fluent-bit/blob/master/Dockerfile.plugins#L1 And then update the build.yml for all the plugins: https://github.com/aws/amazon-cloudwatch-logs-for-fluent-bit/blob/mainline/.github/workflows/build.yml#L19 |
very strange....
I'm not sure what was causing the above error, but |
cloudwatch/cloudwatch.go
Outdated
|
||
if len(e.group) > maxGroupStreamLength { | ||
e.group = output.defaultLogGroupName | ||
logrus.Errorf("[cloudwatch %d] parsing log_group_name template '%s': %v", |
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.
may be add- "will use default group name" ?
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.
LGTM, one nit on an error string.
I am going to wait to merge this till after our planned release this Thursday for arm64 support, just to be safe.
We will need to update the other plugins to go 1.15, I'll add a note in our oncall log for that.
Issue #, if available: #99
Description of changes:
This adds two new input variables to the plugin with defaulted static values. These variables represent fallback group and stream names that get used any time a
$(variable)
in one of the templated inputs fails to parse.The build failed with these errors:
The
%w
verb requires go 1.13 or newer. Requires this: aws/aws-for-fluent-bit#87By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.