Skip to content

[autobackport: sssd-2-10] ci: run long jobs only if Accepted label is not set#8195

Merged
pbrezina merged 4 commits intoSSSD:sssd-2-10from
sssd-bot:SSSD-sssd-backport-pr8182-to-sssd-2-10
Nov 14, 2025
Merged

[autobackport: sssd-2-10] ci: run long jobs only if Accepted label is not set#8195
pbrezina merged 4 commits intoSSSD:sssd-2-10from
sssd-bot:SSSD-sssd-backport-pr8182-to-sssd-2-10

Conversation

@sssd-bot
Copy link
Contributor

This is an automatic backport of PR#8182 ci: run long jobs only if Accepted label is not set to branch sssd-2-10, created by @pbrezina.

Caution

@pbrezina The patches did not apply cleanly. It is necessary to resolve conflicts before merging this pull request. Commits that introduced conflict are marked with CONFLICT!.

You can push changes to this pull request

git remote add sssd-bot git@github.com:sssd-bot/sssd.git
git fetch sssd-bot refs/heads/SSSD-sssd-backport-pr8182-to-sssd-2-10
git checkout SSSD-sssd-backport-pr8182-to-sssd-2-10
git push sssd-bot SSSD-sssd-backport-pr8182-to-sssd-2-10

Original commits

Backported commits

  • a0129e2 - CONFLICT! ci: remove final result job
  • 2ab455d - ci: remove result job from analyze-target
  • 6e3657a - ci: remove result job from static-code-analysis
  • 9a19961 - CONFLICT! ci: run long jobs only if Accepted label is not set

Original Pull Request Body

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.

@alexey-tikhonov
Copy link
Member

@pbrezina, could you please resolve conflicts?

@pbrezina pbrezina force-pushed the SSSD-sssd-backport-pr8182-to-sssd-2-10 branch from 9a19961 to a7b28e3 Compare November 14, 2025 11:52
@pbrezina
Copy link
Member

Done.

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 <atikhono@redhat.com>
Reviewed-by: Justin Stephenson <jstephen@redhat.com>
(cherry picked from commit e557ac7)
There is only one job now so it does not make sense.

Reviewed-by: Alexey Tikhonov <atikhono@redhat.com>
Reviewed-by: Justin Stephenson <jstephen@redhat.com>
(cherry picked from commit af225a0)
We do no not use it in the branch protection rules, so it is not needed.

Reviewed-by: Alexey Tikhonov <atikhono@redhat.com>
Reviewed-by: Justin Stephenson <jstephen@redhat.com>
(cherry picked from commit f100cb6)
To avoid wasting resources on our new merge workflow.

Reviewed-by: Alexey Tikhonov <atikhono@redhat.com>
Reviewed-by: Justin Stephenson <jstephen@redhat.com>
(cherry picked from commit 6f4e1f9)
@sssd-bot
Copy link
Contributor Author

The pull request was accepted by @pbrezina with the following PR CI status:


🟢 CodeQL (success)
🟢 rpm-build:centos-stream-10-x86_64:upstream (success)
🟢 Analyze (target) / All tests are successful (success)
🟢 Analyze (target) / cppcheck (success)
🟢 Build / make-distcheck (success)
🟢 ci / prepare (success)
🟡 ci / system (centos-10) (in_progress)
🟢 Static code analysis / codeql (success)
🟢 Static code analysis / pre-commit (success)
🟢 Static code analysis / python-system-tests (success)


There are unsuccessful or unfinished checks. Make sure that the failures are not related to this pull request before merging.

@sssd-bot sssd-bot force-pushed the SSSD-sssd-backport-pr8182-to-sssd-2-10 branch from a7b28e3 to 0e728ac Compare November 14, 2025 12:42
@pbrezina pbrezina marked this pull request as ready for review November 14, 2025 12:45
@pbrezina pbrezina merged commit b46acb8 into SSSD:sssd-2-10 Nov 14, 2025
9 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants