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

Add multi-cv enable helper and fixture #15433

Merged
merged 2 commits into from
Jul 25, 2024

Conversation

vsedmik
Copy link
Contributor

@vsedmik vsedmik commented Jun 17, 2024

Problem Statement

There is a "Multi-CV" feature partially implemented into 6.16 as a tech preview and we need to deliver related coverage for the implemented bits. For this reason we need to enable the feature in a non-standard way until it's released.

Solution

This PR just adds a helper to enable related setting (to be used with any Satellite) and a session-scoped fixture which would provide Satellite with that setting enabled and set.

@vsedmik vsedmik added the No-CherryPick PR doesnt need CherryPick to previous branches label Jun 17, 2024
@vsedmik vsedmik self-assigned this Jun 17, 2024
@vsedmik vsedmik marked this pull request as ready for review July 23, 2024 13:48
@vsedmik vsedmik requested a review from a team as a code owner July 23, 2024 13:48
Copy link
Contributor

@sambible sambible left a comment

Choose a reason for hiding this comment

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

Useful fixtures until we get proper multicv support. ACK

@vsedmik
Copy link
Contributor Author

vsedmik commented Jul 23, 2024

Tested locally with stream snap, looks to work well

image

@Gauravtalreja1 Gauravtalreja1 merged commit 37825a5 into SatelliteQE:master Jul 25, 2024
8 checks passed
jyejare pushed a commit to jyejare/robottelo that referenced this pull request Oct 19, 2024
* Add multi-cv enable helper and fixture

* Add docstring to the enable_multicv_setting helper
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
No-CherryPick PR doesnt need CherryPick to previous branches
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants