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
Allow to partially re-schedule a product from a job #5233
Conversation
It is possible to set required parameters to falsy values when scheduling a product, e.g. `VERSION=0`. This changes allows re-triggering such a scheduled product without running into a validation error.
* Allow to re-trigger a scheduled product from the job details page of a job of this product * Re-schedule the job itself and all its chained children and parallel dependencies * Do *not* re-schedule chained parents * Refactor JavaScript code for re-scheduling so it can be used on the job details page as well * Improve error handling in JavaScript code for re-scheduling * State successful re-triggering via "The product has been re-triggered as …" with a link (instead of "… should appear when refreshing page."). * Related ticket: https://progress.opensuse.org/issues/124469
Codecov Report
@@ Coverage Diff @@
## master #5233 +/- ##
=======================================
Coverage 98.31% 98.31%
=======================================
Files 389 389
Lines 37133 37140 +7
=======================================
+ Hits 36506 36514 +8
+ Misses 627 626 -1
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me but we should give more people the opportunity to review over a bit longer time hence adding not-ready label
* Ask for confirmation before invoking the re-trigger * Related ticket: https://progress.opensuse.org/issues/124469
* Use a distinct icon for the reschedule button * Related ticket: https://progress.opensuse.org/issues/124469
Removing the not-ready label as we gave people enough time to review. There's still at least one more review required anyways. |
const id = jqXHR.responseJSON?.scheduled_product_id; | ||
const msg = | ||
typeof id === 'number' | ||
? `The product has been re-triggered as <a href="/admin/productlog?id=${id}">${id}</a>.` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This results in
unterminated quoted string literal at /usr/lib/perl5/vendor_perl/5.26.1/Mojolicious/Plugin/AssetPack/Pipe/JavaScript.pm line 22.
in https://build.opensuse.org/package/live_build_log/devel:openQA/openQA/openSUSE_Leap_15.4/x86_64
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I proposed #5264 to address this
(see commit message for details)
page of a job of this product
and parallel dependencies
on the job details page as well
re-triggered as …" with a link (instead of "… should appear
when refreshing page.").