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 Label PR check by using python venv for requests library #1973

Merged

Conversation

rooftopcellist
Copy link
Member

@rooftopcellist rooftopcellist commented Oct 16, 2024

SUMMARY

Fix Label PR check by using python venv for requests library

Failed CI workflow:

Fixes the following error:

Run pip install requests
error: externally-managed-environment

× This environment is externally managed
╰─> To install Python packages system-wide, try apt install
    python3-xyz, where xyz is the package you are trying to
    install.
    
    If you wish to install a non-Debian-packaged Python package,
    create a virtual environment using python3 -m venv path/to/venv.
    Then use path/to/venv/bin/python and path/to/venv/bin/pip. Make
    sure you have python3-full installed.
    
    If you wish to install a non-Debian packaged Python application,
    it may be easiest to use pipx install xyz, which will manage a
    virtual environment for you. Make sure you have pipx installed.
    
    See /usr/share/doc/python3.12/README.venv for more information.

note: If you believe this is a mistake, please contact your Python installation or OS distribution provider. You can override this, at the risk of breaking your Python installation or OS, by passing --break-system-packages.
hint: See PEP 668 for the detailed specification.
ISSUE TYPE
  • Bug, Docs Fix or other nominal change
ADDITIONAL INFORMATION

@rooftopcellist
Copy link
Member Author

Looks like it will keep failing until it is merged because it uses the checkout from devel. I created another test PR here so that I could hardcode the branch to be used when checking out awx-operator to run the Label PR check.

The test check worked:

@TheRealHaoLiu This gives us the confidence to bypass CI and merge this one.

@rooftopcellist rooftopcellist merged commit 8a5ec6e into ansible:devel Oct 16, 2024
6 of 8 checks passed
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