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

Declare End-Of-Life for Ubuntu 12.04 LTS #1080

Merged
merged 4 commits into from
May 26, 2017
Merged

Declare End-Of-Life for Ubuntu 12.04 LTS #1080

merged 4 commits into from
May 26, 2017

Conversation

vutny
Copy link
Contributor

@vutny vutny commented May 25, 2017

What does this PR do?

It drops support for Ubuntu LTS 12.04 which reached EOL at April 28, 2017.
Also, installation on derivatives forked from 12.04 release is no longer possible.
The README has been updated accordingly.

Previous Behavior

Salt installation on Ubuntu 12.04 was possible.

New Behavior

The script notifies a user that Ubuntu 12.04 reached EOL and no longer supported.

vutny added 3 commits May 25, 2017 10:52
This drops support for Ubuntu releases less than 14, including all
derivatives based on those releases.
Copy link
Contributor

@rallytime rallytime left a comment

Choose a reason for hiding this comment

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

Very nice! Thanks for cleaning this up. I like moving the EOL checks up further in the script. I had some small comments on that function. I know that was largely copy/pasted from below, but we might as well fix it now. :)


fedora)
# Fedora lower than 18 are no longer supported
if [ "$DISTRO_MAJOR_VERSION" -lt 23 ]; then
Copy link
Contributor

Choose a reason for hiding this comment

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

I think 23 is EOL too, so let's make that 24 instead.

Also, can you make the comment line above match? (18 should be 24)

Copy link
Contributor

@rallytime rallytime May 25, 2017

Choose a reason for hiding this comment

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

This change would need an update to the README as well.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@rallytime I think it would be better if I make a change for Fedora in separate PR just to make things clear and separate concerns.

;;

centos)
# CentOS versions lower than 5 are no longer supported
Copy link
Contributor

Choose a reason for hiding this comment

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

Comment should be lower than 6 for clarity.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, I'll fix this and later comments to match real comparisons. Thanks for spotting that!

;;

red_hat*linux)
# Red Hat (Enterprise) Linux versions lower than 5 are no longer supported
Copy link
Contributor

Choose a reason for hiding this comment

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

Same thing here, this should read lower than 6.

;;

oracle*linux)
# Oracle Linux versions lower than 5 are no longer supported
Copy link
Contributor

Choose a reason for hiding this comment

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

Same thing here. There are a couple more below this comment that should be updated as well, but I won't add a comment for each one.

@vutny
Copy link
Contributor Author

vutny commented May 26, 2017

@rallytime I've fixed the comments in __check_end_of_life_versions() function. I will drop Fedora 23 support with another PR later. Thanks for review.

@rallytime
Copy link
Contributor

@vutny sounds good. thanks!

@rallytime rallytime merged commit ae7b954 into saltstack:develop May 26, 2017
@vutny vutny deleted the ubuntu12lts-eol branch May 29, 2017 07:13
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.

2 participants