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.

Sign up for GitHub

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

Jump to bottom

Add PVC skiplist for the LTS Driver #986

Closed
vlad-penkin opened this issue Apr 26, 2024 · 3 comments · Fixed by #1456
Closed

Add PVC skiplist for the LTS Driver #986

vlad-penkin opened this issue Apr 26, 2024 · 3 comments · Fixed by #1456
Assignees
@AshburnLee
Labels
ci enhancement New feature or request lts tests: ut validation
Milestone
8. Validation

Comments

@vlad-penkin
Copy link
Contributor

No description provided.

@vlad-penkin vlad-penkin added enhancement New feature or request tests: ut ci labels Apr 26, 2024
@vlad-penkin vlad-penkin added this to the 8. Validation milestone Apr 26, 2024
@AshburnLee AshburnLee self-assigned this May 9, 2024
@AshburnLee AshburnLee mentioned this issue May 20, 2024
Closed
@vlad-penkin vlad-penkin linked a pull request May 20, 2024 that will close this issue
[UT] add skiplist in pvc w/ lts for skipped cases #1156
Closed
@AshburnLee AshburnLee mentioned this issue May 28, 2024
Draft
@AshburnLee
Copy link
Contributor

My approach to getting skip list is different from Pavel‘s and Pavel has left comment about this, we'll discuss about it.

@AshburnLee
Copy link
Contributor

Automation script #1196 is not in the scope of this issue

@AshburnLee
Copy link
Contributor

Close this one as the fix is approved by one reviewer but still waiting the other reviewer's approval.

This was referenced Jun 24, 2024
Merged
Merged
pbchekin added a commit that referenced this issue Jun 24, 2024
@pbchekin
Skip list for LTS (#1456)
76d28ce 
To have green workflows that we run on each PR and push to the default
branch.

PR to enable a custom skip list in CI to follow.

Fixes #986.
pbchekin added a commit that referenced this issue Jun 24, 2024
@pbchekin
Set a default/custom skip list in CI (#1460)
67ef6f3 
Add a new workflow input `skip_list` to specify a skip list when
starting a workflow manually. If `skip_list` is not set, then workflow
checks if there is a skip list for a given `driver_version` (for example
`lts`) and selects it if it exists. Uses a default skip list in other
cases. In future, we will potentially need to add a GPU part to the skip
list name, for example `a770-lts`.

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

@AshburnLee AshburnLee

Labels
ci enhancement New feature or request lts tests: ut validation
Projects
None yet
Milestone
8. Validation
2 participants
@AshburnLee @vlad-penkin