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

[6.16.z] Fix pulp tasks purge test case #17198

Conversation

Satellite-QE
Copy link
Collaborator

Cherrypick of PR: #17190

Problem Statement

The test_purge_pulp_tasks fails with

AssertionError: Pulp tasks were unexpectedly purged
assert 25 > 25

because pulp task list command uses pagination, so it returns only 25 tasks event when there are more tasks to be listed.

Solution

The command offers --limit option for pagination control. Let's use it.

PRT test Cases example

trigger: test-robottelo
pytest: tests/foreman/cli/test_repositories.py -k test_purge_pulp_tasks

@Satellite-QE Satellite-QE requested a review from a team as a code owner December 17, 2024 19:59
@Satellite-QE Satellite-QE added 6.16.z Introduced in or relating directly to Satellite 6.16 Auto_Cherry_Picked Automatically cherrypicked PR using GHA No-CherryPick PR doesnt need CherryPick to previous branches labels Dec 17, 2024
@Satellite-QE
Copy link
Collaborator Author

trigger: test-robottelo
pytest: tests/foreman/cli/test_repositories.py -k test_purge_pulp_tasks

@Satellite-QE Satellite-QE added the AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing label Dec 17, 2024
@Satellite-QE
Copy link
Collaborator Author

PRT Result

Build Number: 9663
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/cli/test_repositories.py -k test_purge_pulp_tasks --external-logging
Test Result : ========== 1 passed, 6 deselected, 17 warnings in 1403.38s (0:23:23) ===========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Dec 17, 2024
@Satellite-QE Satellite-QE merged commit 7d66d7a into 6.16.z Dec 17, 2024
17 of 18 checks passed
@Satellite-QE Satellite-QE deleted the cherry-pick-6.16.z-8ef5d531319db0386539f2ce1768e76d7e6f407d branch December 17, 2024 20:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.16.z Introduced in or relating directly to Satellite 6.16 Auto_Cherry_Picked Automatically cherrypicked PR using GHA AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing No-CherryPick PR doesnt need CherryPick to previous branches PRT-Passed Indicates that latest PRT run is passed for the PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants