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

drop el-7, debian-buster, amazonlinux-2 #14990

Draft
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

Habbie
Copy link
Member

@Habbie Habbie commented Dec 20, 2024

Short description

el-7 and debian buster are EOL already. Amazonlinux-2 goes EOL in six months and we never shipped those packages.

To be done (separate PR perhaps): the el7 devtoolset check in misc-dailies should be migrated for el8.

This is entirely untested and needs a strong review because of all the "if el7" things I took out from various files in various syntaxes.

Checklist

I have:

  • read the CONTRIBUTING.md document
  • compiled this code
  • tested this code
  • included documentation (including possible behaviour changes)
  • documented the code
  • added or modified regression test(s)
  • added or modified unit test(s)
  • checked that this code was merged to master

@Habbie Habbie force-pushed the drop-el7-debianbuster branch from 50272e0 to 70a9a45 Compare December 20, 2024 13:12
@Habbie Habbie marked this pull request as draft December 20, 2024 13:14
@Habbie Habbie added this to the common-soon milestone Dec 20, 2024
@Habbie
Copy link
Member Author

Habbie commented Dec 20, 2024

Amazonlinux-2 goes EOL in six months

this claim was based on https://endoflife.date/amazon-linux which says 30 Jun 2025 (update submitted). However, https://aws.amazon.com/amazon-linux-2/faqs/ says 30 Jun 2026.

That said, I don't think Amazon Linux 2 is reason enough to hang on to all that legacy. I'm not even sure we have any users on it.

@coveralls
Copy link

coveralls commented Dec 20, 2024

Pull Request Test Coverage Report for Build 13178463027

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 288 unchanged lines in 16 files lost coverage.
  • Overall coverage decreased (-8.2%) to 64.722%

Files with Coverage Reduction New Missed Lines %
pdns/misc.cc 1 62.77%
pdns/pollmplexer.cc 1 83.66%
pdns/recursordist/test-syncres_cc2.cc 3 88.91%
pdns/remote_logger.cc 3 53.81%
pdns/iputils.cc 3 56.07%
pdns/rcpgenerator.cc 3 90.64%
pdns/dnsdistdist/dnsdist-lbpolicies.cc 4 70.51%
pdns/dnsdistdist/dnsdist.cc 4 68.73%
pdns/recursordist/pdns_recursor.cc 4 72.54%
pdns/recursordist/test-syncres_cc1.cc 5 89.8%
Totals Coverage Status
Change from base Build 13133792161: -8.2%
Covered Lines: 128273
Relevant Lines: 167090

💛 - Coveralls

@phonedph1
Copy link
Contributor

I cannot offer a strong review but a weak review it eye balls good to me. There are a few spec file comments that refer to EL7 still (see the systemd runtime dir bits) but thats minor.

@Habbie
Copy link
Member Author

Habbie commented Dec 20, 2024

There are a few spec file comments that refer to EL7 still (see the systemd runtime dir bits) but thats minor.

ah yes. I missed at least one comment there, thanks!

@@ -16,10 +16,8 @@ on:
type: string
# please remember to update build-packages.yml as well
default: >-
el-7
el-8
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should this list be in sync with the list in builder.yml?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

no, in fact these days perhaps it should explicitly not overlap with it at all I think. builder-dispatch should contain a good default set for shipping packages (even if we don't builder-dispatch for releases any more). builder is older than all the more specific ones, and was for daily testing of distros we -don't- autobuild for in any other way.

builder-dispatch should be in sync with builder-packages (or ideally the list would live in one of them, but that's tricky). builder (which we should maybe give a useful suffix to explain what it does) should then cover distros we do not, or do not yet, ship for

./install_quiche.sh; \
fi
RUN cd /pdns/builder-support/helpers/ && ./install_rust.sh; \
yum install -y git cmake clang; \
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ISTM all these commands want && chaining for resilience, but this is not a new problem

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

agreed. Would make sense as a separate commit if we're touching all this anyway.

@Habbie Habbie force-pushed the drop-el7-debianbuster branch from 3bcc285 to c149fdd Compare February 4, 2025 11:56
@Habbie Habbie changed the title drop el-7, debian-buster, amazonlinux-2, CircleCI drop el-7, debian-buster, amazonlinux-2 Feb 4, 2025
@Habbie
Copy link
Member Author

Habbie commented Feb 4, 2025

test builds:

@mind04
Copy link
Contributor

mind04 commented Feb 4, 2025

Maybe not for this pull, but without el7 is is also possible to replace yum (deprecated) by dnf

@Habbie
Copy link
Member Author

Habbie commented Feb 6, 2025

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.

6 participants