ci: run long jobs only if Accepted label is not set#8182
Conversation
|
Note Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported. |
pbrezina
added
backport-to-sssd-2-8
backport-to-sssd-2-9
backport-to-sssd-2-10
backport-to-sssd-2-9-4
justin-stephenson
left a comment
justin-stephenson
left a comment
There was a problem hiding this comment.
Ack, thank you.
alexey-tikhonov
removed
backport-to-sssd-2-8
backport-to-sssd-2-9
backport-to-sssd-2-9-4
|
@pbrezina, I know for sure it will not apply cleanly to branches older than 2-11 (and no idea about 2-8 at all). I kept only 2-10 because imo it's easier to resolve conflict in 2-10 backport pr once and then to add labels 2-9-4 and 2-9 to this backport pr than to resolve conflict in multiple backport prs. |
The intention behind this job was to use it in a branch protection rule to require it to be green. However, the intg tests still fails quite often and even rawhide is red every now end then so this is unusable. Reviewed-by: Alexey Tikhonov <[email protected]> Reviewed-by: Justin Stephenson <[email protected]>
There is only one job now so it does not make sense. Reviewed-by: Alexey Tikhonov <[email protected]> Reviewed-by: Justin Stephenson <[email protected]>
We do no not use it in the branch protection rules, so it is not needed. Reviewed-by: Alexey Tikhonov <[email protected]> Reviewed-by: Justin Stephenson <[email protected]>
To avoid wasting resources on our new merge workflow. Reviewed-by: Alexey Tikhonov <[email protected]> Reviewed-by: Justin Stephenson <[email protected]>
|
The pull request was accepted by @alexey-tikhonov with the following PR CI status: 🟢 CodeQL (success) There are unsuccessful or unfinished checks. Make sure that the failures are not related to this pull request before merging. |
4 participants
Avoid running long running and unnecessary jobs when the pull request is Accepted so it can be quickly merged and backport automation has free resources to run.