feat(HUM-2061): optimize filter-already-released-advisory-rpms for speed and reliability#2233
Merged
johnbieren merged 1 commit intoJun 8, 2026
Conversation
scoheb
requested review from
arewm,
gbenhaim,
ralphbean and
twaugh
as code owners
Contributor
Qodo reviews are paused for this user.Troubleshooting steps vary by plan Learn more → On a Teams plan? Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center? |
scoheb
force-pushed
the
HUM-2061-filter-already-released-advisory-rpms
branch
from
3346fcc to
98f0c76
Compare
mmalina
reviewed
May 21, 2026
mmalina
reviewed
May 21, 2026
mmalina
reviewed
May 21, 2026
scoheb
force-pushed
the
HUM-2061-filter-already-released-advisory-rpms
branch
3 times, most recently
from
561382f to
662e337
Compare
mmalina
previously approved these changes
May 25, 2026
1 task
scoheb
force-pushed
the
HUM-2061-filter-already-released-advisory-rpms
branch
from
662e337 to
7989c6d
Compare
scoheb
force-pushed
the
HUM-2061-filter-already-released-advisory-rpms
branch
from
63ff635 to
c27f30e
Compare
Contributor
|
gitlint is failing on too long of a line. but also, I think you can squash the commits now. |
scoheb
force-pushed
the
HUM-2061-filter-already-released-advisory-rpms
branch
2 times, most recently
from
a7d37b4 to
66228f5
Compare
mmalina
previously approved these changes
May 28, 2026
scoheb
force-pushed
the
HUM-2061-filter-already-released-advisory-rpms
branch
from
66228f5 to
869cce1
Compare
davidmogar
previously approved these changes
May 29, 2026
johnbieren
previously approved these changes
May 29, 2026
scoheb
force-pushed
the
HUM-2061-filter-already-released-advisory-rpms
branch
from
869cce1 to
f976089
Compare
Member
Author
|
/retest |
scoheb
force-pushed
the
HUM-2061-filter-already-released-advisory-rpms
branch
4 times, most recently
from
9965877 to
785bb85
Compare
Contributor
PR Code Suggestions ✨Warning
Inline suggestions were posted as code suggestions. |
![qodo-code-review[bot]](https://avatars.githubusercontent.com/in/484649?s=60&v=4){kind=small}
Contributor
PR Code Suggestions ✨Warning
Inline suggestions were posted as code suggestions. |
Contributor
PR Code Suggestions ✨Warning
Inline suggestions were posted as code suggestions. |
Contributor
PR Code Suggestions ✨Warning
Inline suggestions were posted as code suggestions. |
Refactor managed and internal filter tasks for large advisory handling
with oras auth and credential setup. Fix arg-too-long errors using
file-based processing. Optimize purl-to-advisory mapping performance.
Treat unpublished advisory RPMs as unreleased. Restore advisory as
authoritative source of truth. Add direct task results for test
validation. Add internal pipeline and unit tests. Update local test
runner.
Unquote ORAS_OPTIONS to prevent empty arg: when ORAS_OPTIONS is empty,
"${ORAS_OPTIONS:-}" produces an empty string argument that oras
interprets as the reference, causing "invalid reference: missing
registry or repository". Add shellcheck disable directives for the
intentionally unquoted variable.
Signed-off-by: Scott Hebert <shebert@redhat.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Scott Hebert <scoheb@gmail.com>
scoheb
force-pushed
the
HUM-2061-filter-already-released-advisory-rpms
branch
from
785bb85 to
29617f0
Compare
Contributor
PR Code Suggestions ✨Warning
Inline suggestions were posted as code suggestions. |
mmalina
approved these changes
Jun 5, 2026
johnbieren
approved these changes
Jun 5, 2026
ach912
approved these changes
Jun 7, 2026
davidmogar
approved these changes
Jun 8, 2026
twaugh
approved these changes
Jun 8, 2026
Production Approval Record
Approved |
5 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
8 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Test plan
tasks/managed/filter-already-released-advisory-rpmstasks/internal/filter-already-released-advisory-rpms-taskJira
https://redhat.atlassian.net/browse/HUM-2061
Part of HUM-959 - PR 1 of 6
🤖 Generated with Claude Code