-
Notifications
You must be signed in to change notification settings - Fork 77
Cleanup CentOS 7 #1753
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
Cleanup CentOS 7 #1753
Conversation
44474ea to
95ef167
Compare
1f0f58b to
ad8ca47
Compare
igorpeshansky
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TL;DR: as-is, this PR will break our build infrastructure. Let's deal with centos8->rockylinux8 in a separate PR.
4e3f6cd to
afc5fbd
Compare
|
I've put all the CentOS 8 changes back where they were. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The debian-10->debian-11 rename should probably have been part of #1752, but I'll just cross-link them here.
6a5ebbe to
c073e73
Compare
igorpeshansky
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
remove additional debian-10 references in progress use correct GCS URL and Debian version Cleanup CentOS CentOS 7 is now EOL, and all references and configuration for it are being removed. I stumbled on some remaining reference to CentOS8, for which support has already been removed and there were some dangling references in our code. reinstate centos8 restore deleted centos8 config address review comments
c073e73 to
2c9e901
Compare
Description
CentOS 7 is now EOL, and all references and configuration for it are being removed.
Related issue
b/350698170
How has this been tested?
Integration tests in this PR.
Checklist: