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

Port PublicCloud run_ltp to kirk #18920

Merged
merged 1 commit into from
May 10, 2024
Merged

Port PublicCloud run_ltp to kirk #18920

merged 1 commit into from
May 10, 2024

Conversation

mimi1vx
Copy link
Member

@mimi1vx mimi1vx commented Mar 20, 2024 •
edited
Loading

ricardobranco777
ricardobranco777 reviewed Mar 20, 2024
View reviewed changes
Copy link
Contributor

@ricardobranco777 ricardobranco777 left a comment

Choose a reason for hiding this comment

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

Can we ignore this failure in terraform destroy?

http://quasar.suse.cz/tests/3205#step/run_ltp/346

@mimi1vx
Copy link
Member Author

mimi1vx commented Mar 20, 2024

Can we ignore this failure in terraform destroy?

http://quasar.suse.cz/tests/3205#step/run_ltp/346

Yeah, not related to test itself and Pavel is now fixing it

@mimi1vx mimi1vx force-pushed the kirk branch 2 times, most recently from 6e77f1c to 30c3174 Compare March 20, 2024 14:07
@m-dati
Copy link
Contributor

m-dati commented Mar 20, 2024 •
edited
Loading

We'd need to have also more VRs,as on syscall and cve too, and tweaking the cloned job with a more complex LTP_COMMAND_EXCLUDE expression, if it works as expected.

@mimi1vx mimi1vx changed the title Port PublicCloud run_ltp to kirk WIP: Port PublicCloud run_ltp to kirk Mar 20, 2024
asmorodskyi
asmorodskyi approved these changes Mar 21, 2024
View reviewed changes
Copy link
Member

@asmorodskyi asmorodskyi left a comment

Choose a reason for hiding this comment

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

LGTM

@mimi1vx mimi1vx force-pushed the kirk branch 3 times, most recently from 09ae468 to c237b08 Compare May 6, 2024 13:22
@mimi1vx mimi1vx force-pushed the kirk branch 5 times, most recently from a6d342c to aeacdf0 Compare May 6, 2024 15:16
@mimi1vx mimi1vx force-pushed the kirk branch 8 times, most recently from 8fa261b to 7570fc5 Compare May 7, 2024 09:18
@mimi1vx mimi1vx changed the title WIP: Port PublicCloud run_ltp to kirk Port PublicCloud run_ltp to kirk May 7, 2024
@mimi1vx mimi1vx changed the title Port PublicCloud run_ltp to kirk WIP: Port PublicCloud run_ltp to kirk May 9, 2024
@mimi1vx mimi1vx force-pushed the kirk branch 2 times, most recently from a67d09a to 59dadd9 Compare May 9, 2024 14:23
@mimi1vx mimi1vx changed the title WIP: Port PublicCloud run_ltp to kirk Port PublicCloud run_ltp to kirk May 9, 2024
asmorodskyi
asmorodskyi approved these changes May 9, 2024
View reviewed changes
Copy link
Member

@asmorodskyi asmorodskyi left a comment

Choose a reason for hiding this comment

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

LGTM

@mimi1vx mimi1vx merged commit 3755d0e into os-autoinst:master May 10, 2024
20 checks passed
@mimi1vx mimi1vx deleted the kirk branch May 10, 2024 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ricardobranco777 ricardobranco777 ricardobranco777 approved these changes

@m-dati m-dati m-dati left review comments

@asmorodskyi asmorodskyi asmorodskyi approved these changes

@pdostal pdostal Awaiting requested review from pdostal pdostal is a code owner

@grisu48 grisu48 Awaiting requested review from grisu48 grisu48 is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@mimi1vx @m-dati @ricardobranco777 @asmorodskyi