Skip to content

improvement(WorkArea): Add Test Execute button #608

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

Merged
merged 1 commit into from
Apr 4, 2025

Conversation

k0machi
Copy link
Collaborator

@k0machi k0machi commented Mar 17, 2025

This commits adds a new button to the test runs within argus, allowing
user to either start a build from scratch or rebuild a latest run
without having to open a run to do so. The from scratch implementation
will try to retrieve previous build parameters (In case previous build
wasn't submitted to argus) and if that's not possible, will offer user
to start a build without parameters to be able to retrieve the build on
subsequent call.

Fixes #542

@k0machi k0machi self-assigned this Mar 17, 2025
@k0machi k0machi requested review from fruch, soyacz and roydahan March 17, 2025 12:42
This commits adds a new button to the test runs within argus, allowing
user to either start a build from scratch or rebuild a latest run
without having to open a run to do so. The from scratch implementation
will try to retrieve previous build parameters (In case previous build
wasn't submitted to argus) and if that's not possible, will offer user
to start a build without parameters to be able to retrieve the build on
subsequent call.

Fixes scylladb#542
@k0machi k0machi force-pushed the job-execute-button branch from 9c3e69d to b61790d Compare April 2, 2025 03:33
@k0machi k0machi requested a review from soyacz April 2, 2025 03:33
@k0machi
Copy link
Collaborator Author

k0machi commented Apr 2, 2025

Added additional logic to handle executed jobs without parameters (first build as done by SCT by executing without parameters)

Copy link
Collaborator

@soyacz soyacz left a comment

Choose a reason for hiding this comment

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

LGTM

@k0machi k0machi merged commit 9d78379 into scylladb:master Apr 4, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Job execute button
2 participants