Skip to content

[hotfix-0.1348] Skip issue update if artefact is filtered out#864

Merged
8R0WNI3 merged 1 commit into
open-component-model:hotfix-0.1348from
8R0WNI3:hotfix-0.1348
Jun 18, 2026
Merged

[hotfix-0.1348] Skip issue update if artefact is filtered out#864
8R0WNI3 merged 1 commit into
open-component-model:hotfix-0.1348from
8R0WNI3:hotfix-0.1348

Conversation

@8R0WNI3

@8R0WNI3 8R0WNI3 commented Jun 18, 2026

Copy link
Copy Markdown
Member

What this PR does / why we need it:
It is possible to configure artefact filters per finding type to shortcut scans. In the past, filtered out artefacts were still processed in the issue replicator to ensure once openend issues are still going to be closed if the artefact has been filtered out only at a later point in time. However, this required multiple database queries as well as expensive GitHub API calls (bulk issue fetching) for a very minor use case.
Therefore, early-skip artefacts which are filtered out and leave issue closing to a (tbd) follow-up process.

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:

Release note:

The issue replicator now early-skips artefacts which are filtered out for a specific finding type, thus potentially leaving issues open for artefacts which were once included by the filter configuration but then excluded at a later point in time

It is possible to configure artefact filters per finding type to
shortcut scans. In the past, filtered out artefacts were still processed
in the issue replicator to ensure once openend issues are still going to
be closed if the artefact has been filtered out only at a later point in
time. However, this required multiple database queries as well as
expensive GitHub API calls (bulk issue fetching) for a very minor use
case.
Therefore, early-skip artefacts which are filtered out and leave issue
closing to a (tbd) follow-up process.

Signed-off-by: Jonas Brand (8R0WNI3) <j.brand@sap.com>
@8R0WNI3 8R0WNI3 self-assigned this Jun 18, 2026
@8R0WNI3 8R0WNI3 requested a review from a team as a code owner June 18, 2026 05:22
@8R0WNI3 8R0WNI3 added kind/task small task, normally part of feature or epic area/cicd Continuous Integration and Continuous Deployment related labels Jun 18, 2026
@coderabbitai

coderabbitai Bot commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: b10624ab-5805-4a13-94f9-8a273f776d62

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@8R0WNI3 8R0WNI3 merged commit 2308b82 into open-component-model:hotfix-0.1348 Jun 18, 2026
4 checks passed
@8R0WNI3 8R0WNI3 deleted the hotfix-0.1348 branch June 18, 2026 05:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/cicd Continuous Integration and Continuous Deployment related kind/task small task, normally part of feature or epic

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant