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

Upgrade ipython installed in Docker images #2021

Merged
merged 2 commits into from
Mar 1, 2023
Merged

Conversation

bandish-shah
Copy link
Contributor

@bandish-shah bandish-shah commented Mar 1, 2023

What does this PR do?

Follow on to PR #2007, previous PR upgraded the ipython package in Composer's setup.py. However the mosaicml/pytorch* images are failing because the version already installed in the image needs to be upgraded as well.

This should resolve CVE-2023-24816.

What issue(s) does this change relate to?

CO-1844

Before submitting

  • Have you read the contributor guidelines?
  • Is this change a documentation change or typo fix? If so, skip the rest of this checklist.
  • Was this change discussed/approved in a GitHub issue first? It is much more likely to be merged if so.
  • Did you update any related docs and document your change?
  • Did you update any related tests and add any new tests related to your change? (see testing)
  • Did you run the tests locally to make sure they pass?
  • Did you run pre-commit on your change? (see the pre-commit section of prerequisites)

@bandish-shah bandish-shah requested a review from a team as a code owner March 1, 2023 17:47
@bandish-shah bandish-shah enabled auto-merge (squash) March 1, 2023 17:48
@bandish-shah bandish-shah merged commit 50812e2 into dev Mar 1, 2023
@bandish-shah bandish-shah deleted the bandish/upgrade_ipython branch March 1, 2023 18:02
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