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

fix: Fixing pip installation as a system package #7768

Merged
merged 7 commits into from
Nov 6, 2024

Conversation

KrishnanPrash
Copy link
Contributor

What does the PR do?

The PR is to address CI jobs that are failing due to tritonfrontend and tritonserver wheel installation PR [Link] with the following error message:

+ python3 -m pip install --upgrade pip
Requirement already satisfied: pip in /usr/lib/python3/dist-packages (24.0)
Collecting pip
  Downloading pip-24.3.1-py3-none-any.whl.metadata (3.7 kB)
Downloading pip-24.3.1-py3-none-any.whl (1.8 MB)
   ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 1.8/1.8 MB 45.0 MB/s eta 0:00:00
Installing collected packages: pip
  Attempting uninstall: pip
    Found existing installation: pip 24.0
ERROR: Cannot uninstall pip 24.0, RECORD file not found. Hint: The package was installed by debian.

From my understanding, the CI jobs were previously failing because the only relevant package installed was apt-get install python3-pip which is a system level package leading to the following error. This can be resolved by adding python3 -m pip install --upgrade pip or pip3 install --upgrade pip which upgrades pip to a user-level version that can be upgraded in the future without the same level of protections.

  • CI Pipeline ID: 20137857

@KrishnanPrash KrishnanPrash added PR: build Changes that affect the build system or external dependencies PR: fix A bug fix labels Nov 6, 2024
@KrishnanPrash KrishnanPrash merged commit cc3bb11 into main Nov 6, 2024
3 checks passed
@KrishnanPrash KrishnanPrash deleted the kprashanth-pip-fix branch November 6, 2024 23:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: build Changes that affect the build system or external dependencies PR: fix A bug fix
Development

Successfully merging this pull request may close these issues.

2 participants