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

poll for task intended to fail #888

Merged
merged 1 commit into from
Feb 9, 2023

Conversation

pondrejk
Copy link
Contributor

@pondrejk pondrejk commented Jan 31, 2023

Description of changes

_poll_task did check if the task succeeded, which caused a problem for negative tests where we want wait on the task to finish but we expect failure. This pr adds a parameter to make this behavior configurable, the default is kept as it was, though no need to update existing callers.

Functional demonstration

Will provide a robottelo pr using the parameter.
[update] see SatelliteQE/robottelo#10591

@pondrejk pondrejk added CherryPick PR needs CherryPick to previous branches 6.13.z labels Jan 31, 2023
@pondrejk pondrejk requested review from a team January 31, 2023 13:35
@pondrejk pondrejk self-assigned this Jan 31, 2023
nailgun/entities.py Outdated Show resolved Hide resolved
@pondrejk
Copy link
Contributor Author

pondrejk commented Feb 6, 2023

@adamruzicka @Griffin-Sullivan comments addressed, mind revisiting? Thanks!

nailgun/entities.py Outdated Show resolved Hide resolved
@pondrejk
Copy link
Contributor Author

pondrejk commented Feb 9, 2023

@Gauravtalreja1 @Griffin-Sullivan mind revisiting?

@jyejare jyejare merged commit 905eb11 into SatelliteQE:master Feb 9, 2023
github-actions bot pushed a commit that referenced this pull request Feb 9, 2023
Gauravtalreja1 pushed a commit that referenced this pull request Feb 9, 2023
(cherry picked from commit 905eb11)

Co-authored-by: Peter Ondrejka <pondrejk@users.noreply.github.com>
pondrejk added a commit to pondrejk/nailgun that referenced this pull request May 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.13.z CherryPick PR needs CherryPick to previous branches
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants