Skip to content
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

2.32.0 release #733

Merged
merged 6 commits into from
Oct 19, 2023
Merged

2.32.0 release #733

merged 6 commits into from
Oct 19, 2023

Conversation

matthewfala
Copy link
Contributor

@matthewfala matthewfala commented Sep 26, 2023

2.32.0 Release

This release includes:

  • Fluent Bit 1.9.10
  • Amazon CloudWatch Logs for Fluent Bit 1.9.4
  • Amazon Kinesis Streams for Fluent Bit 1.10.2
  • Amazon Kinesis Firehose for Fluent Bit 1.7.2

Compared to 2.31.12.20230911 this release adds:

  • Feature - Amazon Managed Prometheus support added via sigv4 authentication on prometheus_remote_write aws-for-fluent-bit:256
  • Enhancement - Allow multiple cloudwatch_log output workers by removing sequence tokens from API requests aws-for-fluent-bit:526
  • Bug - Fix multiline input behavior when multiple streams are parsed (stderr, stdout) together. Multiline logs are no longer terminated when streams are switched between fluent-bit:7469

@matthewfala
Copy link
Contributor Author

Please see the following cherry-pick diffs:
Cloudwatch Sequence Token Deprecation: matthewfala/fluent-bit#58
Prometheus: matthewfala/fluent-bit#59
Multiline flush fix: matthewfala/fluent-bit#60

@matthewfala matthewfala force-pushed the 2.32.0-release branch 2 times, most recently from b369bd9 to e82ef87 Compare September 26, 2023 22:18
@matthewfala
Copy link
Contributor Author

Also updated the cherry-picking system in the docker build file to end build on a single failure.

@matthewfala
Copy link
Contributor Author

@matthewfala
Copy link
Contributor Author

Updated

Comment on lines -86 to -94
# TCP input: user friendly message for skipping logs https://github.com/fluent/fluent-bit/pull/6061
https://github.com/fluent/fluent-bit.git 1.9 c5d57ccf4f85af77e2a2bbb1121df95887e2962f

# ElasticSearch: fix buffer bulk over run https://github.com/fluent/fluent-bit/pull/5770
https://github.com/fluent/fluent-bit.git 1.9 d3a0697c3c4a876250cc6c2c4ff44cd77908ea3c

# windows: fix time zone offset https://github.com/fluent/fluent-bit/pull/6368
https://github.com/fluent/fluent-bit.git 1.9 cae9a8f9cb6eab7449f14d914f9ba1d8d4e93cf0

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@matthewfala why are these deleted?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah, thanks

@matthewfala
Copy link
Contributor Author

matthewfala commented Sep 28, 2023

Update wording to
Compared to 2.31.12.20230911 this release adds:

- Feature - Customers can send metrics to Amazon Managed Prometheus via added sigv4 authentication on prometheus_remote_write https://github.com/aws/aws-for-fluent-bit/issues/256
- Enhancement - Customers can route logs to CloudWatch Logs at higher throughputs by increasing number of output workers as cloudwatch output plugin removed sequence tokens from API requests https://github.com/aws/aws-for-fluent-bit/issues/526
- Fix - Fix multiline input behavior when multiple streams are parsed (stderr, stdout) together. Multiline logs are no longer terminated when streams are switched between https://github.com/fluent/fluent-bit/pull/7469
  • Update public doc - Request for doc change (not-blocking)
  • Update public doc - AWS FireLens Example (9/28)
    • Add link to release notes
  • Update public docs once

@matthewfala
Copy link
Contributor Author

matthewfala commented Sep 29, 2023

Update the examples: aws-samples/amazon-ecs-firelens-examples#125
Update cloudwatch_logs guidance issue: #340

@matthewfala matthewfala force-pushed the 2.32.0-release branch 2 times, most recently from 133a592 to 44ac0a9 Compare September 29, 2023 03:25
CHANGELOG.md Outdated

Compared to `2.31.12.20230911` this release adds:
* Feature - Amazon Managed Prometheus support added via sigv4 authentication on `prometheus_remote_write` [aws-for-fluent-bit:256](https://github.com/aws/aws-for-fluent-bit/issues/256)
* Enhancement - Allow multiple `cloudwatch_log` output workers by removing sequence tokens from API requests [aws-for-fluent-bit:526](https://github.com/aws/aws-for-fluent-bit/issues/526)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's link to the new firelens example somewhere in the release notes.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the final link is predictable

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure! Link added.

@@ -28,7 +28,7 @@ release: build build-init
debug: main-debug init-debug

.PHONY: build
build: linux-plugins
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

build should still build linux-plugins?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Build dockerfile/image does not depend on the linux-plugins image. Actually only the release and main-release depends on the "linux plugins"

It was slowing down my fluent-bit build tests.

CHANGELOG.md Outdated
* Amazon Kinesis Streams for Fluent Bit 1.10.2
* Amazon Kinesis Firehose for Fluent Bit 1.7.2

Compared to `2.31.12.20230911` this release adds:
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These notes are not updated. Resolve before merge

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also the release note should be Compared to 2.31.12, the date stamped re-builds for linux only don't count as a full release.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sure! Updated.

@matthewfala matthewfala force-pushed the 2.32.0-release branch 2 times, most recently from 8911319 to 1d379bd Compare September 29, 2023 20:43
* Amazon Kinesis Firehose for Fluent Bit 1.7.2

Compared to `2.31.12` this release adds:
* Feature - Customers can send metrics to Amazon Managed Prometheus via added sigv4 authentication on `prometheus_remote_write`. Refer to [amazon-ecs-firelens-examples](https://github.com/aws-samples/amazon-ecs-firelens-examples/blob/mainline/examples/fluent-bit/amazon-managed-service-for-prometheus/README.md) for information on how to export metrics to AMP on ECS [aws-for-fluent-bit:256](https://github.com/aws/aws-for-fluent-bit/issues/256)
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added link to prometheus example

* Amazon Kinesis Streams for Fluent Bit 1.10.2
* Amazon Kinesis Firehose for Fluent Bit 1.7.2

Compared to `2.31.12` this release adds:
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Updated version comparison to 2.31.12

@matthewfala
Copy link
Contributor Author

Consider adding the following patches:

@matthewfala
Copy link
Contributor Author

Throttle fix is a backport of: https://github.com/fluent/fluent-bit/pull/6500/files

@matthewfala
Copy link
Contributor Author

@matthewfala matthewfala force-pushed the 2.32.0-release branch 2 times, most recently from 492bf18 to 250a64e Compare October 5, 2023 23:34
* Fix - Fix multiline input behavior when multiple streams are parsed (stderr, stdout) together. Multiline logs are no longer terminated when streams are switched between [fluent-bit:7469](https://github.com/fluent/fluent-bit/pull/7469)
* Fix - Fix networking edgecase causing data loss and OOM related issues via net timeout event injection resolution [fluent-bit:7728](https://github.com/fluent/fluent-bit/pull/7728/files)
* Fix - Allow reduced filter_throttle log output with `print_status` `false` configuration option fix [fluent-bit:7469](https://github.com/fluent/fluent-bit/pull/6500/files)

Copy link
Contributor

@PettitWesley PettitWesley Oct 16, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@matthewfala we should add on3e more things from here, when I tried to stage 2.32: https://github.com/aws/aws-for-fluent-bit/pull/658/files

  1. AWS Profile support

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Documenting our decision:

  1. Will further review this feature item in the future in the 2.33.0 release. Requires further review and testing. link
  2. Proceed with release

@matthewfala matthewfala merged commit 00496b6 into aws:mainline Oct 19, 2023
@matthewfala
Copy link
Contributor Author

For documentation: Test the network fix 2.32.0-premature-connection-destruction change via:

diff --git a/src/flb_network.c b/src/flb_network.c
index f8cc7068d..d63d7ab94 100644
--- a/src/flb_network.c
+++ b/src/flb_network.c
@@ -441,7 +441,7 @@ static int net_connect_async(int fd,
     ret = mk_event_add(u_conn->evl,
                        fd,
                        FLB_ENGINE_EV_THREAD,
-                       MK_EVENT_WRITE, &u_conn->event);
+                       MK_EVENT_READ, &u_conn->event);
     u_conn->event.priority = FLB_ENGINE_PRIORITY_CONNECT;
     if (ret == -1) {
         /*

This will cause the network events to artificially timeout, causing the injection timeout code which is modified to be triggered.

@matthewfala
Copy link
Contributor Author

Add update to the error log to include api caller: https://github.com/fluent/fluent-bit/pull/8122/files

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants