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

GitHub Actions failing due to pip error #2249

Closed
damithc opened this issue Oct 11, 2024 · 5 comments
Closed

GitHub Actions failing due to pip error #2249

damithc opened this issue Oct 11, 2024 · 5 comments

Comments

@damithc
Copy link
Collaborator

damithc commented Oct 11, 2024

The error:

Run pip install requests
error: externally-managed-environment
...

Example https://github.com/nus-cs2113-AY2425S1/ip-dashboard/actions/runs/11285488693/job/31388669590

@damithc
Copy link
Collaborator Author

damithc commented Oct 11, 2024

@gok99
Copy link
Contributor

gok99 commented Oct 11, 2024

It looks like the breaking CI is using Ubuntu 24.04 instead of 22.04, which has configurations to prevent apt and pip from intefering with one another. Seems like the updated latest ubuntu runner isn't fully deployed.

We can probably pin the runner to ubuntu-22.04 instead of ubuntu-latest for now.

@sikai00
Copy link
Member

sikai00 commented Oct 11, 2024

Beyond pinning to ubuntu-22.04 as a temporary fix, we can also look into adding a step to use venv to resolve the root issue which was introduced in PEP 668.

Can take reference from the multiple fixes that other projects have done, e.g., fcitx/mozc@e8d25a2#diff-1ee5360cc67f8ec7ed8a3ca75252c859e3acded65ab5e6d4688bcb8585b3e7efR29

@gok99
Copy link
Contributor

gok99 commented Oct 11, 2024

@damithc I've pinned publish-RepoSense to ubuntu-22.04 for the time being - could you try merging it?

@sikai00 This would be good to do, together with adding ubuntu-24.04 to our CI runners for the RepoSense repo. Could you create issues for these?

@damithc
Copy link
Collaborator Author

damithc commented Oct 11, 2024

@damithc I've pinned publish-RepoSense to ubuntu-22.04 for the time being - could you try merging it?

Yup, that worked. Thanks @gok99 for the quick fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Closed/Completed
Development

No branches or pull requests

3 participants