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

cc_ansible: install_method: pip cannot find ansible-pull command path #6021

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

AliyevH
Copy link
Contributor

@AliyevH AliyevH commented Feb 13, 2025

Proposed Commit Message


1. site path ~/.local must be added to env PATH if self.run_user is defined.
2. bootstrap pip if required - must be run before install not after.
3. is_installed - pip list command must use --user if self.run_user is defined.
4. [sys.executable, "-c", "'import site; print(site.getuserbase())'"] - incorrect
5. [sys.executable, "-c", "import site; print(site.getuserbase())"] - correct. single quotes inside double quotes behaves incorrectly. and returns nothing.


Fixes GH-5720

Additional Context

Test Steps

Merge type

  • Squash merge using "Proposed Commit Message"
  • Rebase and merge unique commits. Requires commit messages per-commit each referencing the pull request number (#<PR_NUM>)

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.

1 participant