config: fix compilation with FLB_HAVE_STATIC_CONF.

[bglimepoint]

This is a small change to fix a compilation error when using FLB_HAVE_STATIC_CONF.

---

Enter [N/A] in the box, if an item is not applicable to your change.

Testing
Before we can approve your change; please submit the following in a comment:

• [N/A] Example configuration file for the change
• [N/A] Debug log output from testing the change

• [N/A] Attached Valgrind output that shows no leaks or memory corruption was found

If this is a change to packaging of containers or native binaries then please confirm it works for all targets.

• [N/A] Run local packaging test showing all targets (including any new ones) build.
• [N/A] Set ok-package-test label to test for all targets (requires maintainer to do).

Documentation

• [N/A] Documentation required for this feature

Backporting

• Backport to latest stable release.

---

Fluent Bit is licensed under Apache 2.0, by submitting this pull request I understand that this code will be released under the terms of that license.

[bglimepoint]

We noticed this when trying to move to 2.1.x; but I'm not sure which branch to target for "Backport to latest stable release," (but might just be blind, so going to have a look :-) can't find a 2.1 branch? :-s )

[patrick-stephens]

2.0 branch is the intended backport target.

[bglimepoint]

2.0 branch is the intended backport target.

Thanks @patrick-stephens :-)

To confirm, the bug is not present in the 2.0 branch as the sighup config reloading feature hasn't been backported to 2.0, so have just left the backport for now - hopefully that's alright :-)

[bglimepoint]

Hello, we were just wondering if this is waiting for anything additional on our side? :-)

Not a stress if no one's had a chance to have look and it's just waiting for that, but wanted to check as it's a fairly simple compile error that it would be great to fix, so just wanted to check :-)

Thanks, and sorry for the noise :-)

[github-actions]

This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.

[github-actions]

This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days.

[bglimepoint]

@patrick-stephens sorry to pester; but is there anything we could do to help get this merged? It's still broken in master :-s

It would appear to be a relatively unused feature (I guess?), but we do find it useful, so it would be nice to fix up FLB_HAVE_STATIC_CONF - and it is just the one line change :-s

[patrick-stephens]

I'm not a codeowner so cannot review this section but I'll ping @edsiper and @leonardo-albertovich to see if they can look.

It would help to have some way to test it - this will also prevent it in the future. Any CI we can add to help?

[bglimepoint]

Ah, thanks for the note @patrick-stephens :-)

And for the extra pings :-)

I can have a look into getting a test for this; although it hadn't been immediately obvious when I'd looked previously - but I'll see if I can take another look soon ish

All it needs is a build with the FLB_STATIC_CONF arg set, e.g. cmake -DFLB_STATIC_CONF=./fluent-bit-config.yaml - I'll try and see if I can figure out how to get that into the CI jobs :-) (Although there might be a bit of a delay on my side)