Upgrade aws-for-fluent-bit chart to v0.2.0 (upstream version to 3.2.1)#1294
Upgrade aws-for-fluent-bit chart to v0.2.0 (upstream version to 3.2.1)#1294sapphirew merged 1 commit intoaws:masterfrom
Conversation
|
@PettitWesley can you please review this PR? Tested on an EKS cluster. All 6 output plugins (cloudWatchLogs, cloudWatch, kinesis_streams, kinesis, firehose, s3) verified working. Fresh install and upgrade from v0.1.35 both successful. See PR description for details |
|
Trying to tag more maintainers of this repo - @moshiaiz @dims @bryantbiggs @arditti @liubnu @BennettJames can any of you please review this PR? |
|
Hey, is it intended that only version bump up is needed? no other configuration change? |
Yes, most configurations are forward compatible. Some deprecate configurations are retained for ensuring backward compatibility. We are going to check with users if any additional configurations are needed in order to include them in one single release. Will hold on merging this PR until that's confirmed. |
stable/aws-for-fluent-bit/Chart.yaml
Outdated
| description: A Helm chart to deploy aws-for-fluent-bit project | ||
| version: 0.1.35 | ||
| appVersion: 2.32.2.20240516 | ||
| version: 0.1.36 |
There was a problem hiding this comment.
should we bump the minor version sinc the major version of aws-for-fluent-bit is changing? Like 0.2.0
There was a problem hiding this comment.
good call, changed version to 0.2.0
19ae048 to
f5699c4
Compare
|
Looks good. Please do a check through the fluent-bit change logs to make sure none of our config setup is broken by the transition from fluent-bit 1.9->4.2 Also, looks like there's a newer upstream version now: https://github.com/aws/aws-for-fluent-bit/releases/tag/v3.2.1 - so probably worth pulling that in before merging. Other than that, lgtm |
This upgrade updates the aws-for-fluent-bit Helm chart from v0.1.35 to v0.1.36, which includes AWS for Fluent Bit 3.2.0 (based on Fluent Bit 4.2.2). Changes: - Chart.yaml: version 0.1.35 -> 0.1.36, appVersion 2.32.2.20240516 -> 3.2.0 - values.yaml: image.tag 2.32.2.20240516 -> 3.2.0 - README.md: Updated version documentation and added upgrade notes EKS Cluster Testing Results: - Fresh install: Pods reached Running state, Fluent Bit 4.2.2 started successfully, CloudWatch Logs output plugin working, logs delivered to CloudWatch - Upgrade from v0.1.35: Helm upgrade completed without errors, pods restarted with new version, log collection continued without interruption - Verified backward compatibility: Same values.yaml structure, existing configs work Key version changes: - Fluent Bit: 1.9.10 -> 4.2.2 - Base image: Amazon Linux 2 -> Amazon Linux 2023 - Image tag format: date suffix removed (2.32.2.20240516 -> 3.2.0)
f5699c4 to
656a6fa
Compare
|
The followings are the breaking changes for Fluent Bit 1.9.x → 4.2.x, but none of them is used in this helm chart, so there is no configuration change required in this PR
Also bumped upstream version to https://github.com/aws/aws-for-fluent-bit/releases/tag/v3.2.1 |
Summary
This upgrade updates the aws-for-fluent-bit Helm chart from v0.1.35 to v0.2.0, which includes AWS for Fluent Bit 3.2.1 (based on Fluent Bit 4.2.2).
Tests
Test 1: CloudWatch Logs (C Plugin)
Helm Install Command
Pod Status
Fluent Bit Logs
CloudWatch Verification
{ "logStreams": [ { "logStreamName": "fluentbit-kube.var.log.containers.aws-for-fluent-bit-6r4jt_kube-system_aws-for-fluent-bit-...", "creationTime": 1769047291463, "firstEventTimestamp": 1769047291053, "lastEventTimestamp": 1769047291192 } ] }Test 2: CloudWatch (Go Plugin - Legacy)
Helm Upgrade Command
Fluent Bit Logs
CloudWatch Verification
{ "logStreams": [ { "logStreamName": "fluentbit-kube.var.log.containers.aws-for-fluent-bit-fsv4n_kube-system...", "creationTime": 1769047478848, "firstEventTimestamp": 1769047478591 } ] }Test 3: Kinesis Streams (C Plugin)
Helm Upgrade Command
Fluent Bit Logs
Kinesis Verification
Test 4: Kinesis (Go Plugin)
Helm Upgrade Command
Kinesis Verification
aws kinesis get-records --shard-iterator "$SHARD_ITERATOR" --region us-west-2 --limit 3{ "Records": [ { "SequenceNumber": "49671078933949352870379830273367104338065091517155901442", "ApproximateArrivalTimestamp": 1769060693.6, "Data": "eyJfcCI6IkYiLCJrdWJlcm5ldGVzIjp7ImFubm90YXRpb25zIjp7...", "PartitionKey": "6Ozsv5nt" } ], "MillisBehindLatest": 7000 }Test 5: Firehose (Go Plugin)
Prerequisites
Helm Upgrade Command
Fluent Bit Logs
Test 6: S3 (C Plugin)
Helm Upgrade Command
helm upgrade aws-for-fluent-bit stable/aws-for-fluent-bit \ --namespace kube-system \ --set serviceAccount.create=false \ --set serviceAccount.name=aws-for-fluent-bit \ --set firehose.enabled=false \ --set s3.enabled=true \ --set s3.region=us-west-2 \ --set s3.bucket=fluent-bit-test-803909145761 \ --set s3.s3KeyFormat='/s3-direct/$TAG/%Y-%m-%d/%H-%M-%S' \ --set s3.totalFileSize=15M \ --set s3.uploadChunkSize=6M \ --set s3.uploadTimeout=1mFluent Bit Logs
S3 Verification
Confirmed Fluent Bit Version
Upgrade test
Upgrade from v0.1.35: Helm upgrade completed without errors, pods restarted with new version, log collection continued without interruption
Verified backward compatibility: Same values.yaml structure, existing configs work
Key version changes:
Issue
Description of changes
Checklist
README.mdfor modified charts)versioninChart.yamlfor the modified chart(s)Testing
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.