Skip to content

Restore debian7 images needed for beats 7.17#621

Merged
mauri870 merged 3 commits intoelastic:mainfrom
mauri870:restore-debian7
Jun 3, 2025
Merged

Restore debian7 images needed for beats 7.17#621
mauri870 merged 3 commits intoelastic:mainfrom
mauri870:restore-debian7

Conversation

@mauri870
Copy link
Member

@mauri870 mauri870 commented May 28, 2025

Beats 7.17 has no further planned releases, but we must remain ready for potential emergency patches.

Given this policy, we also aim to keep up with newer Go versions. PR #510 removed debian7 and debian8 images, causing failures when we upgraded 7.17 to go1.24.3. We cannot migrate to debian 9-11 due to glibc 2.13 constraints.

That said, restore the debian7 images to fix our 7.17 builds.

Relates elastic/beats#43984.

Beats 7.17 has no further planned releases, but we must remain ready for
potential emergency patches.

Given this policy, we also aim to keep up with newer Go versions. PR elastic#510
removed debian7 and debian8 images, causing failures when we upgraded 7.17
to go1.24.3. We cannot migrate to debian 9-11 due to glibc 2.13 constraints.

That said, restore the debian7 images to fix our 7.17 builds.
@mauri870 mauri870 requested a review from v1v May 28, 2025 15:22
@mauri870 mauri870 self-assigned this May 28, 2025
@mauri870 mauri870 requested review from a team as code owners May 28, 2025 15:22
@v1v
Copy link
Member

v1v commented May 28, 2025

That said, restore the debian7 images to fix our 7.17 builds.

We can use 1.23, since it's a supported line and it already contains the support for Debian7, see #611

I prefer we don't bump 1.24 in beats@7.17 as long as 1.23 is a supported golang version. #509 was the agreement and some further details can be found at #510

@cmacknz
Copy link
Member

cmacknz commented May 28, 2025

Yes I remember that discussion now, but had forgotten it until we had put some work into getting the automated PR to bump 7.17 to Go 1.24.

Go 1.23 will go EOL in August I think, so we can safely keep 7.17 on Go 1.23.x until then without issue.

I don't think there is a burning reason to go to 1.24 in 7.17 so we can keep the original agreement to stay on 1.23 there.

@cmacknz cmacknz closed this May 28, 2025
@cmacknz
Copy link
Member

cmacknz commented May 30, 2025

OK, we need to keep 7.17 releasable until January 2026 which means it will will live past the EOL date of Go 1.23. We need this change but we don't need it urgently.

@mauri870
Copy link
Member Author

/test

@v1v
Copy link
Member

v1v commented May 30, 2025

unrelated co failures, see #615

I think we need to exclude running those other pipelines for changes in the main pipeline, so I'll raise a PR to avoid that scenario.

@elasticmachine
Copy link

💚 Build Succeeded

History

cc @mauri870

@mauri870 mauri870 enabled auto-merge (squash) June 2, 2025 20:56
@mauri870
Copy link
Member Author

mauri870 commented Jun 3, 2025

@pkoutsovasilis @andrzej-stencel Could I get a review from the Control Plane team? Thanks!

@mauri870 mauri870 merged commit 9897b55 into elastic:main Jun 3, 2025
6 checks passed
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.

5 participants