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

Upgraded to pip 9.0.1 #154

Merged
merged 1 commit into from
Nov 15, 2016
Merged

Upgraded to pip 9.0.1 #154

merged 1 commit into from
Nov 15, 2016

Conversation

michael-k
Copy link
Contributor

No description provided.

@mattrobenolt
Copy link
Contributor

fwiw, just weighing in my opinion, this change does bring backwards incompatibility, and we definitely don't have any way of tagging to indicate a version of pip. So a python:2.7.12 pulled today is going to get 8.1.2, then pulling same image after this is merged is going to bring in 9.0.0 which may or may not break user applications.

Granted, I don't think that should hold us back from doing the upgrade, just something to keep in mind. I dealt with this when we upgraded to 8.0. :) See #83 for discussion.

Definitely need to figure out why tests are failing though.

@tianon
Copy link
Member

tianon commented Nov 4, 2016

Indeed, I'm wary of 9.0 given our experience with 8.0 (especially breaking our tests, which exposed a bug in pip itself). 😰

@mattrobenolt
Copy link
Contributor

I'll have some time to look at this over the weekend and see if something obvious is standing out why tests would fail for this. 😭

@tianon
Copy link
Member

tianon commented Nov 4, 2016

I think the tests might actually be failing due to docker-library/official-images#2284 (docker-library/buildpack-deps#49, #151).

@tianon
Copy link
Member

tianon commented Nov 4, 2016

Which doesn't really make sense, given that buildpack-deps was updated in docker-library/official-images#2292 😞

@mattrobenolt
Copy link
Contributor

mattrobenolt commented Nov 4, 2016

😨 🙏

@michael-k
Copy link
Contributor Author

All test that fail, fail on master too. I'll guess it's best to fix the tests on master and then rebase.

@michael-k
Copy link
Contributor Author

Updated the commit to pip 9.0.1 and upgraded pip for the 3.6 image too (missed that before).

The tests pass now, I guess due to:

libgdbm-dev is not needed since docker-library/buildpack-deps#49, but it looks like there was a hiccup in the signing process and so the new image is not showing up on the Docker Hub.

See #156 (comment)

If upgrading pip is so critical, maybe there should be images for the latest major version of pip and the one before.

@ye ye mentioned this pull request Nov 11, 2016
@michael-k
Copy link
Contributor Author

rebased

@tianon
Copy link
Member

tianon commented Nov 15, 2016

If upgrading pip is so critical, maybe there should be images for the latest major version of pip and the one before.

I wouldn't say it's necessarily critical, it's just caused problems in the past. I think we've got some pretty decent test coverage though, so I feel OK about updating this. 👍

@yosifkit yosifkit merged commit 1414e3e into docker-library:master Nov 15, 2016
@michael-k michael-k deleted the pip9.0.0 branch November 15, 2016 22:12
tianon added a commit to infosiftr/stackbrew that referenced this pull request Nov 17, 2016
- `cassandra`: 3.0.10
- `docker`: 1.13.0-rc1 (currently failing the `override-cmd` test; see moby/moby#28329), `docker daemon` -> `dockerd` (docker-library/docker#31)
- `drupal`: 8.2.3
- `java`: alpine 7.121.2.6.8-r0, 8.111.14-r0
- `openjdk`: alpine 7.121.2.6.8-r0, 8.111.14-r0
- `python`: 3.6.0b3 (docker-library/python#157), pip 9.0.1 (docker-library/python#154)
- `ruby`: 2.3.2, 2.2.6
- `tomcat`: 6.0.48, 7.0.73, 8.0.39, 8.5.8, 9.0.0.M13
- `wordpress`: add `WORDPRESS_*_FILE` support for passing parameters via files especially for [Docker secrets](moby/moby#27794) (docker-library/wordpress#186)
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.

4 participants