Skip to content

Allow filtering livetest runs via PYTEST_MARK#1419

Merged
ggainey merged 1 commit into
pulp:mainfrom
gerrod3:livetest-pytest-mark-filter
Jun 8, 2026
Merged

Allow filtering livetest runs via PYTEST_MARK#1419
ggainey merged 1 commit into
pulp:mainfrom
gerrod3:livetest-pytest-mark-filter

Conversation

@gerrod3

@gerrod3 gerrod3 commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add a PYTEST_MARK make variable to the livetest target, defaulting to live
  • Plugin CI can now run only the live tests for a single component, e.g. make livetest PYTEST_MARK="live and pulp_container"

Test plan

  • make livetest still runs the full live suite (default unchanged)
  • make livetest PYTEST_MARK="live and pulp_container" runs only container live tests
  • Update plugin CI script.sh to use the filtered invocation once this is released

Made with Cursor

Plugin CI only needs live tests for a single component, but livetest
currently runs the entire live suite. Add a PYTEST_MARK make variable
(defaulting to live) so callers can run e.g. make livetest
PYTEST_MARK="live and pulp_container".
@gerrod3

gerrod3 commented Jun 8, 2026

Copy link
Copy Markdown
Contributor Author

The AI also suggested we could add this rule:

If you want shorter commands for common cases:

.PHONY: livetest-%
livetest-%:
	uv run $(MAKE) _livetest PYTEST_MARK="live and pulp_$*"

Then:

make livetest-container   # → live and pulp_container
make livetest-rpm         # → live and pulp_rpm

@ggainey ggainey merged commit 1633919 into pulp:main Jun 8, 2026
17 checks passed
@gerrod3 gerrod3 deleted the livetest-pytest-mark-filter branch June 8, 2026 18:57
@patchback

patchback Bot commented Jun 8, 2026

Copy link
Copy Markdown

Backport to 0.40: 💚 backport PR created

✅ Backport PR branch: patchback/backports/0.40/1633919ff9a63320fee3999787d73eb0a171c5af/pr-1419

Backported as #1420

🤖 @patchback
I'm built with octomachinery and
my source is open — https://github.com/sanitizers/patchback-github-app.

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