Skip to contentSkip to conversation

package/fluent-bit: bump fluent-bit to 5.1.2

PATCH In review Aaron Mestdagh 3 participants 5 messages Oct 22 – Oct 24

lore
Share & tools
lore
#0 lore raw

PATCH 4/4 package/fluent-bit: bump fluent-bit to 5.1.2

From: Aaron <[email protected]>

Upgrade from 4.2.0 to 5.1.2, bringing multi-worker network ingestion and improved OpenTelemetry processing.

The 5.x releases also add Google Cloud Storage output, Windows Event Tracing input, NVIDIA GPU metrics, and new metrics processors, alongside FIPS, TLS, and proxy improvements.

---
 ...001-ripser-initialize-temporary-entry.patch | 18 ++++++++++++++++++
 package/fluent-bit/fluent-bit.hash             |  2 +-
 package/fluent-bit/fluent-bit.mk               |  2 +-
 3 files changed, 20 insertions(+), 2 deletions(-)
 create mode 100644 package/fluent-bit/0001-ripser-initialize-temporary-entry.patch
package/fluent-bit/0001-ripser-initialize-temporary-entry.patch new file +18 -0
new file mode 100644
index 0000000000..c98bce88a9
@@ -0,0 +1,18 @@
+From: Buildroot package compatibility
+Subject: [PATCH] ripser: initialize temporary diameter entry
+Upstream: https://github.com/Ripser/ripser
+
+GCC 6 does not accept the implicit default construction of
+diameter_entry_t. The temporary is assigned before it is used, so initialize
+it with Ripser's existing invalid-index sentinel.
+
+Upstream-Status: Pending (https://github.com/fluent/fluent-bit/pull/12533)
+
+Signed-off-by: Aaron <[email protected]>
+diff --git a/lib/ripser-1.2.1/ripser.cpp b/lib/ripser-1.2.1/ripser.cpp
+--- a/lib/ripser-1.2.1/ripser.cpp
++++ b/lib/ripser-1.2.1/ripser.cpp
+@@ -185,0 +186,1 @@
++	diameter_entry_t() : std::pair<value_t, entry_t>() {}
+-- 
+2.43.0
package/fluent-bit/fluent-bit.hash +1 -1
index be02761ed5..7fb94d60b2 100644
@@ -1,3 +1,3 @@
 # Locally computed
-sha256  44fe0f52e89a63b213695748f99691d0a6247a4bd05065f1b517c798d9f89bcc  fluent-bit-4.2.0.tar.gz
+sha256  1971d86c7dc0f3e6b906890297635d6a3a84e5e7ad64d36521b74da202fda62f  fluent-bit-5.1.2.tar.gz
 sha256  0d542e0c8804e39aa7f37eb00da5a762149dc682d7829451287e11b938e94594  LICENSE
package/fluent-bit/fluent-bit.mk +1 -1
index a3a44b8dbe..b53da90706 100644
@@ -4,7 +4,7 @@
 #
 ################################################################################
 
-FLUENT_BIT_VERSION = 4.2.0
+FLUENT_BIT_VERSION = 5.1.2
 FLUENT_BIT_SITE = $(call github,fluent,fluent-bit,v$(FLUENT_BIT_VERSION))
 FLUENT_BIT_LICENSE = Apache-2.0
 FLUENT_BIT_LICENSE_FILES = LICENSE
Signature · 6 lines
2.43.0

_______________________________________________
buildroot mailing list
[email protected]
https://lists.buildroot.org/mailman/listinfo/buildroot
#1 lore raw

Hello,

Thanks for your patch. However, why is this PATCH 4/4 ? I haven't not seen your PATCH 1/4, 2/4 and 3/4. Is this a mistake or was there a technical issue (perhaps on our side)?

Quoted 1 line · Aaron Mestdagh

From: Aaron <[email protected]>

Could you please fix your Git setup so that your full name is first name + last name ?

Quoted 6 lines

Upgrade from 4.2.0 to 5.1.2, bringing multi-worker network ingestion and improved OpenTelemetry processing.

The 5.x releases also add Google Cloud Storage output, Windows Event Tracing input, NVIDIA GPU metrics, and new metrics processors, alongside FIPS, TLS, and proxy improvements.

Did you run this update through ./utils/test-pkg ?

Quoted 1 line

Your Git setup should also make sure this uses first name + last name.

Quoted 7 lines
package/fluent-bit/0001-ripser-initialize-temporary-entry.patch new file +1 -0
new file mode 100644
index 0000000000..c98bce88a9
@@ -0,0 +1,18 @@
+From: Buildroot package compatibility

package/fluent-bit/0001-ripser-initialize-temporary-entry.patch line 11

Quoted 10 lines
package/fluent-bit/0001-ripser-initialize-temporary-entry.patch new file +10 -0
+Subject: [PATCH] ripser: initialize temporary diameter entry
+Upstream: https://github.com/Ripser/ripser
+
+GCC 6 does not accept the implicit default construction of
+diameter_entry_t. The temporary is assigned before it is used, so initialize
+it with Ripser's existing invalid-index sentinel.
+
+Upstream-Status: Pending (https://github.com/fluent/fluent-bit/pull/12533)
+
+Signed-off-by: Aaron <[email protected]>

This patch looks very odd. Could you please generate your patch with
git format-patch? Or now that it has been merged, just take
https://github.com/fluent/fluent-bit/commit/56df8d278c927b2e7b81d28d7c08128fd7ded167.patch ?

Use an Upstream: tag pointing to the upstream commit, i.e
https://github.com/fluent/fluent-bit/commit/56df8d278c927b2e7b81d28d7c08128fd7ded167.

Could you send a v2 after addressing those comments?

Thanks a lot!

Thomas

Signature · 7 lines
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
[email protected]
https://lists.buildroot.org/mailman/listinfo/buildroot
#2 lore raw

Hi Thomas, Aaron,

Op do 22 okt 2026, 23:04 schreef Thomas Petazzoni < [email protected]>:

Quoted 6 lines

Hello,

Thanks for your patch. However, why is this PATCH 4/4 ? I haven't not seen your PATCH 1/4, 2/4 and 3/4. Is this a mistake or was there a technical issue (perhaps on our side)?

This patch continues on this: series
https://patchwork.buildroot.org/project/buildroot/list/?series=88
Is it possible to handle those too?

Quoted 54 lines · Aaron Mestdagh
Quoted 1 line

From: Aaron <[email protected]>

Could you please fix your Git setup so that your full name is first name + last name ?

Quoted 6 lines

Upgrade from 4.2.0 to 5.1.2, bringing multi-worker network ingestion and improved OpenTelemetry processing.

The 5.x releases also add Google Cloud Storage output, Windows Event Tracing input, NVIDIA GPU metrics, and new metrics processors, alongside FIPS, TLS, and proxy improvements.

Did you run this update through ./utils/test-pkg ?

Quoted 1 line

Your Git setup should also make sure this uses first name + last name.

Quoted 1 line

diff --git

a/package/fluent-bit/0001-ripser-initialize-temporary-entry.patch b/package/fluent-bit/0001-ripser-initialize-temporary-entry.patch

package/fluent-bit/0001-ripser-initialize-temporary-entry.patch line 6

Quoted 11 lines

new file mode 100644
index 0000000000..c98bce88a9

package/fluent-bit/0001-ripser-initialize-temporary-entry.patch new file +6 -0
@@ -0,0 +1,18 @@
+From: Buildroot package compatibility
+Subject: [PATCH] ripser: initialize temporary diameter entry
+Upstream: https://github.com/Ripser/ripser
+
+GCC 6 does not accept the implicit default construction of
+diameter_entry_t. The temporary is assigned before it is used, so

initialize

package/fluent-bit/0001-ripser-initialize-temporary-entry.patch

Quoted 3 lines
package/fluent-bit/0001-ripser-initialize-temporary-entry.patch new file +3 -0
+it with Ripser's existing invalid-index sentinel.
+
+Upstream-Status: Pending (

https://github.com/fluent/fluent-bit/pull/12533)

package/fluent-bit/0001-ripser-initialize-temporary-entry.patch line 11

Quoted 2 lines
package/fluent-bit/0001-ripser-initialize-temporary-entry.patch new file +2 -0
+
+Signed-off-by: Aaron <[email protected]>

This patch looks very odd. Could you please generate your patch with git format-patch? Or now that it has been merged, just take

https://github.com/fluent/fluent-bit/commit/56df8d278c927b2e7b81d28d7c08128fd7ded167.patch
?

Use an Upstream: tag pointing to the upstream commit, i.e

https://github.com/fluent/fluent-bit/commit/56df8d278c927b2e7b81d28d7c08128fd7ded167
.

Could you send a v2 after addressing those comments?

Quoted 7 lines

Thanks a lot!

Thomas
--
Thomas Petazzoni, co-owner and CEO, Bootlin Embedded Linux and Kernel engineering and training
https://bootlin.com

Kr,

Thomas Devoogdt

Quoted 2 lines

_______________________________________________
buildroot mailing list
[email protected]
https://lists.buildroot.org/mailman/listinfo/buildroot

#3 lore raw

PATCH v2 1/1 package/fluent-bit: bump fluent-bit to 5.1.2

From: Aaron Mestdagh <[email protected]>

Upgrade from 4.2.0 to 5.1.2, bringing multi-worker network ingestion and improved OpenTelemetry processing.

The 5.x releases also add Google Cloud Storage output, Windows Event Tracing input, NVIDIA GPU metrics, and new metrics processors, alongside FIPS, TLS, and proxy improvements.

---
v2:
- Fixed author name: Aaron --> Aaron Mestdagh
- Fixed patch index: [PATCH 4/4] package/fluent-bit: bump fluent-bit to 5.1.2 --> [PATCH v2] The old patch was on top of the "[v1,1/3] package/librdkafka: add new package" series.
https://patchwork.buildroot.org/project/buildroot/list/?series=88&state=%2A&archive=both
- Refreshed the merged 0001-lib-ripser-initialize-temporary-diameter-entry patch
- Used msgpack-c 7.0.2 & librdkafka 2.16.0 to test, all ./utils/test-pkg -p fluent-bit -a passed.

---
 ...-initialize-temporary-diameter-entry.patch | 31 +++++++++++++++++++
 package/fluent-bit/fluent-bit.hash            |  2 +-
 package/fluent-bit/fluent-bit.mk              |  2 +-
 3 files changed, 33 insertions(+), 2 deletions(-)
 create mode 100644 package/fluent-bit/0001-lib-ripser-initialize-temporary-diameter-entry.patch
package/fluent-bit/0001-lib-ripser-initialize-temporary-diameter-entry.patch new file +31 -0
new file mode 100644
index 00000000000..a7213b8de14
@@ -0,0 +1,31 @@
+From 83086bf5092d76c2fffbcd7f0cb671083bf0a3c3 Mon Sep 17 00:00:00 2001
+From: Aaron Mestdagh <[email protected]>
+Date: Thu, 8 Oct 2026 11:52:22 +0200
+Subject: [PATCH] lib: ripser: initialize temporary diameter entry
+
+GCC 6 does not accept implicit default construction of diameter_entry_t.
+The temporary is assigned before use, so initialize it with Ripser's existing
+invalid-index sentinel.
+
+Upstream: https://github.com/fluent/fluent-bit/pull/12533
+Signed-off-by: Aaron Mestdagh <[email protected]>
+Signed-off-by: Thomas Devoogdt <[email protected]>
+---
+ lib/ripser-1.2.1/ripser.cpp | 1 +
+ 1 file changed, 1 insertion(+)
+
+diff --git a/lib/ripser-1.2.1/ripser.cpp b/lib/ripser-1.2.1/ripser.cpp
+index 6bddf7271..b9ccc89d6 100644
+--- a/lib/ripser-1.2.1/ripser.cpp
++++ b/lib/ripser-1.2.1/ripser.cpp
+@@ -183,6 +183,7 @@ value_t get_diameter(const index_diameter_t& i) { return i.second; }
+ 
+ struct diameter_entry_t : std::pair<value_t, entry_t> {
+ 	using std::pair<value_t, entry_t>::pair;
++	diameter_entry_t() : std::pair<value_t, entry_t>() {}
+ 	diameter_entry_t(value_t _diameter, index_t _index, coefficient_t _coefficient)
+ 	    : diameter_entry_t(_diameter, make_entry(_index, _coefficient)) {}
+ 	diameter_entry_t(const diameter_index_t& _diameter_index, coefficient_t _coefficient)
+-- 
+2.43.0
+
package/fluent-bit/fluent-bit.hash +1 -1
index be02761ed53..7fb94d60b25 100644
@@ -1,3 +1,3 @@
 # Locally computed
-sha256  44fe0f52e89a63b213695748f99691d0a6247a4bd05065f1b517c798d9f89bcc  fluent-bit-4.2.0.tar.gz
+sha256  1971d86c7dc0f3e6b906890297635d6a3a84e5e7ad64d36521b74da202fda62f  fluent-bit-5.1.2.tar.gz
 sha256  0d542e0c8804e39aa7f37eb00da5a762149dc682d7829451287e11b938e94594  LICENSE
package/fluent-bit/fluent-bit.mk +1 -1
index a3a44b8dbe7..b53da90706f 100644
@@ -4,7 +4,7 @@
 #
 ################################################################################
 
-FLUENT_BIT_VERSION = 4.2.0
+FLUENT_BIT_VERSION = 5.1.2
 FLUENT_BIT_SITE = $(call github,fluent,fluent-bit,v$(FLUENT_BIT_VERSION))
 FLUENT_BIT_LICENSE = Apache-2.0
 FLUENT_BIT_LICENSE_FILES = LICENSE
Signature · 6 lines
2.43.0

_______________________________________________
buildroot mailing list
[email protected]
https://lists.buildroot.org/mailman/listinfo/buildroot
#4 lore raw
Quoted 11 lines · Thomas Devoogdt

From: Aaron Mestdagh <[email protected]>

Upgrade from 4.2.0 to 5.1.2, bringing multi-worker network ingestion and improved OpenTelemetry processing.

The 5.x releases also add Google Cloud Storage output, Windows Event Tracing input, NVIDIA GPU metrics, and new metrics processors, alongside FIPS, TLS, and proxy improvements.

I've updated the Upstream: tag to point to the commit merged upstream, and applied. Thanks a lot!

Best regards,

Thomas Petazzoni

Signature · 7 lines
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
[email protected]
https://lists.buildroot.org/mailman/listinfo/buildroot