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

Remove Zookeeper 3.3.6 #42

Merged
merged 2 commits into from
Aug 1, 2018
Merged

Remove Zookeeper 3.3.6 #42

merged 2 commits into from
Aug 1, 2018

Conversation

dustinschultz
Copy link
Contributor

@dustinschultz dustinschultz commented Jul 31, 2018

3.3.6 distro doesn't exist at https://www.apache.org/dist anymore, only at https://archive.apache.org/dist

@tianon
Copy link

tianon commented Jul 31, 2018

Is 3.3 still even supported? It seems like it was moved to archive.apache.org because it's EOL, in which case I think removing it here is probably more appropriate, right?

Removing it here (and in library/zookeeper) will remove it from the "Supported" section on the Hub readme (and will prevent us from spending cycles rebuilding it on the official build server), but the tag will still be available to users who want it. (See https://github.com/docker-library/official-images#library-definition-files for more detail on this.)

@dustinschultz
Copy link
Contributor Author

Good question ... I can't find anything that explicitly says it's EOL. It has only been moved to "old versions".

The zookeeper JIRA has a 3.3.7 fix version with some issues in it but I don't see any tags related to it and the issues in it are from 2012.

I'd be +1 for removing it.

@31z4
Copy link
Owner

31z4 commented Aug 1, 2018

3.3.6 hasn't been updated in two years. I think we're safe to remove.

@dustinschultz dustinschultz changed the title Update 3.3.6 Dockerfile to new location of 3.3.6 distribution (archive.apache.org) Remove Zookeeper 3.3.6 Aug 1, 2018
@dustinschultz
Copy link
Contributor Author

👍 - PR has been updated to remove 3.3.6

@31z4 31z4 merged commit 9cf14e7 into 31z4:master Aug 1, 2018
@31z4
Copy link
Owner

31z4 commented Aug 1, 2018

Thanks!

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.

3 participants