[aws-for-fluent-bit] Support terminationGracePeriodSeconds, fix kinesis_stream parameter in configmap

[JeongPope]

Issue

I have confirmed that there is no related Github issue or PR for this.

Description of changes

1. Fix incorrect parameters in the output plugin for kinesis_stream.

• FluentBit - Output plugin - Amazon Kinesis Data Streams

2. Support for the terminationGracePeriodSeconds option is provided for FluentBit pods. FluentBit supports a property grace in the SERVICE configuration for Graceful Shutdown.

• FluentBit - Configuration File - Service

Checklist

• Added/modified documentation as required (such as the README.md for modified charts)
• Incremented the chart version in Chart.yaml for the modified chart(s)
• Manually tested. Describe what testing was done in the testing section below
• Make sure the title of the PR is a good description that can go into the release notes

Testing

values-test.yaml (based on values.yaml)

# Added
terminationGracePeriodSeconds: 120

# Update
kinesis_streams:
  enabled: true
  time_key_format: test_time_key_format
  external_id: test_external_id
  auto_retry_requests: test_auto_retry_requests

Command

$ cd ./stable/aws-for-fluent-bit
$ helm template . -f values-test.yaml -s templates/daemonset.yaml -s templates/configmap.yaml

Result

# Source: aws-for-fluent-bit/templates/daemonset.yaml
apiVersion: apps/v1
kind: DaemonSet
metadata:
  name: release-name-aws-for-fluent-bit
# ... skip ...
spec:
# ... skip ...
  template:
# ... skip ...
    spec:
      serviceAccountName: release-name-aws-for-fluent-bit
      dnsPolicy: ClusterFirst
      terminationGracePeriodSeconds: 120
---
# Source: aws-for-fluent-bit/templates/configmap.yaml
apiVersion: v1
kind: ConfigMap
# ... skip ...
    [OUTPUT]
        Name                kinesis_streams
        Match               *
        region              us-east-1
        stream              my-kinesis-stream-name
        time_key_format     test_time_key_format
        external_id         test_external_id
        auto_retry_requests test_auto_retry_requests

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

[JeongPope]

Hello, @PettitWesley.
Thank you for your review. After resolving a conflict(a7d5b87), the approval was removed. Could you please review it again?

Thank you.

[PettitWesley]

can you please add support for time_key_format as well

[JeongPope]

It seems that time_key_format is already supported. If I am mistaken, please let me know.

• https://github.com/aws/eks-charts/blob/master/stable/aws-for-fluent-bit/templates/configmap.yaml#L260-L262