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

Upgrade fluent-bit to 2.0.9 #1078

Merged
merged 5 commits into from
Feb 21, 2023
Merged

Upgrade fluent-bit to 2.0.9 #1078

merged 5 commits into from
Feb 21, 2023

Conversation

jefferbrecht
Copy link
Member

@jefferbrecht jefferbrecht commented Feb 1, 2023

Description

Upgrade fluent-bit from 1.9.8 to 2.0.9. Notably, this upgrade includes fixes for the Windows 2012 lockup and Windows time zone parsing issues.

Related issue

  • b/254846111 (2.x upgrade)
  • b/240564518 (Windows 2012 lockup)
  • b/218888265 (Windows time zone parsing)

How has this been tested?

  • Unit tests pass
  • Letting PR tests run
  • build_and_test_all.sh passed all builds + tests (with some flakes/unrelated test failures; see upgrade ticket for details)

Checklist:

  • Unit tests
    • Unit tests do not apply.
    • Unit tests have been added/modified and passed for this PR.
  • Integration tests
    • Integration tests do not apply.
    • Integration tests have been added/modified and passed for this PR.
  • Documentation
    • This PR introduces no user visible changes. [1]
    • This PR introduces user visible changes and the corresponding documentation change has been made.
  • Minor version bump
    • This PR introduces no new features.
    • This PR introduces new features, and there is a separate PR to bump the minor version since the last release already.
    • This PR bumps the version.

[1] We document the time zone issue for Windows here. This shouldn't be needed once users upgrade to the new release of Ops Agent which includes this fix, but it may still be worth keeping around for a bit for users who haven't upgraded yet.

@jefferbrecht jefferbrecht force-pushed the jefferbrecht-flb2 branch 5 times, most recently from 04afaf4 to 884aacf Compare February 7, 2023 15:00
@jefferbrecht jefferbrecht changed the title draft don't merge Upgrade fluent-bit to 2.0.9 Feb 7, 2023
@jefferbrecht
Copy link
Member Author

rocky-linux-9 failure is a flake; re-ran locally which passed:

=== PAUSE TestFluentForwardLog/rocky-linux-9
=== CONT  TestFluentForwardLog/rocky-linux-9
    agents.go:680: Test logs: /tmp/4094947281/TestFluentForwardLog_rocky-linux-9
    agents.go:680: Instance Log: https://console.cloud.google.com/logs/viewer?resource=gce_instance%2Finstance_id%2F5398937227559420999&project=stackdriver-kubernetes-1337
--- PASS: TestFluentForwardLog (0.00s)
    --- PASS: TestFluentForwardLog/rocky-linux-9 (167.66s)
PASS
ok      command-line-arguments  168.656s

@jefferbrecht jefferbrecht marked this pull request as ready for review February 16, 2023 17:59
@jefferbrecht jefferbrecht requested review from a team, ridwanmsharif and sophieyfang and removed request for a team and ridwanmsharif February 16, 2023 18:02
@jefferbrecht jefferbrecht added the kokoro:force-run Forces kokoro to run integration tests on a CL label Feb 17, 2023
@stackdriver-instrumentation-release stackdriver-instrumentation-release removed the kokoro:force-run Forces kokoro to run integration tests on a CL label Feb 17, 2023
@jefferbrecht jefferbrecht added kokoro:force-run Forces kokoro to run integration tests on a CL and removed kokoro:force-run Forces kokoro to run integration tests on a CL labels Feb 17, 2023
@jefferbrecht jefferbrecht added the kokoro:force-run Forces kokoro to run integration tests on a CL label Feb 21, 2023
@stackdriver-instrumentation-release stackdriver-instrumentation-release removed the kokoro:force-run Forces kokoro to run integration tests on a CL label Feb 21, 2023
@jefferbrecht jefferbrecht force-pushed the jefferbrecht-flb2 branch 2 times, most recently from 46edbbe to e7a9aa2 Compare February 21, 2023 16:26
braydonk and others added 2 commits February 21, 2023 14:56
stretch, centos7, sles12 and sles15 don't install a
new enough CMake (>= 3.12) with their default
package managers. stretch has an appropriate
version available via backports, and we can install
CMake manually on the other platforms.

This includes an earlier series of commits rebased into the new generated Dockerfile system:
- bff0e5a
- 2038796
- 888e6ff
- c3bb8ca
@jefferbrecht jefferbrecht merged commit fcdfcba into master Feb 21, 2023
@jefferbrecht jefferbrecht deleted the jefferbrecht-flb2 branch February 21, 2023 21:23
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.

4 participants