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

dockerfile: update to bookworm base image #8916

Merged
merged 5 commits into from
Jun 6, 2024

Conversation

patrick-stephens
Copy link
Contributor

@patrick-stephens patrick-stephens commented Jun 5, 2024

Update to Debian bookworm as the base for the container images.
Increments to libssl3, libldap-2.5 and libffi8 as part of this. Also added libcap2 to prevent runtime linkage problems.


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

Testing

CI covers build and run sanity check.


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.

Signed-off-by: Patrick Stephens <[email protected]>
Signed-off-by: Patrick Stephens <[email protected]>
Signed-off-by: Patrick Stephens <[email protected]>
Signed-off-by: Patrick Stephens <[email protected]>
@patrick-stephens
Copy link
Contributor Author

patrick-stephens commented Jun 5, 2024

Integration test failures seem unrelated, current master fails too: https://github.com/fluent/fluent-bit/actions/runs/9394014042

@edsiper edsiper added this to the Fluent Bit v3.0.7 milestone Jun 5, 2024
@patrick-stephens
Copy link
Contributor Author

Windows downloads seem to have flaky/slow issues with Chocolatey, this is a known issue and unrelated.

@patrick-stephens patrick-stephens merged commit 2181632 into master Jun 6, 2024
21 of 27 checks passed
@patrick-stephens patrick-stephens deleted the update_to_bookworm branch June 6, 2024 10:09
@mwheeler-ep
Copy link

Just some feedback on this. I feel like this should have been part of a major release (or very least a minor release). We had some lua scripts break due to the libssl change that happened. I don't think many users who are extending the docker container are expecting the base operating system to change out on them in a patch release.

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

Successfully merging this pull request may close these issues.

4 participants