Skip to content

Conversation

@lucasts
Copy link
Contributor

@lucasts lucasts commented Aug 2, 2017

enable pip to upgrade packages which changed since last deploy.
also, will avoid cleaning up app's images manually

  • Warning *

This can lead to problems to apps careless about their packages versions.

@andrestc
Copy link
Member

andrestc commented Aug 2, 2017

LGTM but would like to have more opnions regarding the point mentioned in the warning. In my opnion we should favor users that follow the best practices (those would not have problems, if I understood correctly).

@lucasts lucasts force-pushed the feat/upgrade-packages branch from 294a981 to 37050eb Compare August 2, 2017 14:28
@lucasts
Copy link
Contributor Author

lucasts commented Aug 2, 2017

@andrestc agreed. Tsuru servers admins can choose to warn users about this change and allow them some time to check and update their requirements/setup if necessary.

@lucasts lucasts changed the title feat: upgrade python packages when deploying [python] feature: upgrade packages when deploying Aug 2, 2017
@ggarnier
Copy link
Member

ggarnier commented Aug 7, 2017

I agree. Correct if I'm wrong, but if someone run into problems with this, he can fix it by setting specific dependency versions in requirements.txt.

@lucasts
Copy link
Contributor Author

lucasts commented Aug 7, 2017

@ggarnier that's correct.

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