Skip to content

EXP Use make to run cli tests#443

Closed
mdellweg wants to merge 1 commit intopulp:mainfrom
mdellweg:update-cli-tests
Closed

EXP Use make to run cli tests#443
mdellweg wants to merge 1 commit intopulp:mainfrom
mdellweg:update-cli-tests

Conversation

@mdellweg
Copy link
Copy Markdown
Member

If this works, we should make this via the plugin template

📜 Checklist

  • Commits are cleanly separated with meaningful messages (simple features and bug fixes should be squashed to one commit)
  • A changelog entry or entries has been added for any significant changes
  • Follows the Pulp policy on AI Usage
  • (For new features) - User documentation and test coverage has been added

See: Pull Request Walkthrough

Comment thread .github/workflows/scripts/script.sh Outdated
@mdellweg mdellweg force-pushed the update-cli-tests branch 2 times, most recently from 31cff81 to 1601128 Compare April 29, 2026 09:55
Comment thread .github/workflows/scripts/script.sh Outdated
pip install -r test_requirements.txt
pytest -v tests -m "pulp_gem"
else
make livetest
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe uv is using a new venv without Pulp’s certificate?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was using the certificates dangled into the installed certifi's cert store. Because someone in python/requests/??? decided that the certstore to use should be certifi by default and not the systems ca-certificates by default and back then we needed a fix around that soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants