Skip to content

VEX: add rh vex beta feed testing and parsing mitigations - #1871

Merged
crozzy merged 3 commits into
quay:mainfrom
crozzy:vex-beta-testing
Aug 19, 2026
Merged

VEX: add rh vex beta feed testing and parsing mitigations#1871
crozzy merged 3 commits into
quay:mainfrom
crozzy:vex-beta-testing

Conversation

@crozzy

@crozzy crozzy commented May 11, 2026

Copy link
Copy Markdown
Contributor

In order to ingest the new Red Hat VEX feed we need to update a couple of things:

  • Reconstruct the "self" URL as it doesn't exist in the document anymore.
  • Ignore source (src) entries in the fixed and known_not_affected arrays, the system is always assuming binaries here and we're always matching to binary RPMs from the layer filesystem

@crozzy
crozzy force-pushed the vex-beta-testing branch 7 times, most recently from a1a972f to d8bef0e Compare May 14, 2026 23:08
@crozzy
crozzy force-pushed the vex-beta-testing branch 3 times, most recently from 2e08e80 to 55ff5bc Compare May 26, 2026 21:53
@crozzy
crozzy force-pushed the vex-beta-testing branch from 55ff5bc to 27b08b4 Compare July 1, 2026 14:40
@crozzy
crozzy force-pushed the vex-beta-testing branch 2 times, most recently from 15b156e to cc79a27 Compare August 10, 2026 21:09
@crozzy crozzy changed the title test: add rh vex beta testing VEX: add rh vex beta feed testing and parsing mitigations Aug 10, 2026
@crozzy
crozzy force-pushed the vex-beta-testing branch 3 times, most recently from 61b93c3 to 44cb203 Compare August 10, 2026 22:29
@crozzy
crozzy marked this pull request as ready for review August 10, 2026 22:30
@crozzy
crozzy requested review from a team as code owners August 10, 2026 22:30
@crozzy
crozzy requested a review from hdonnay August 10, 2026 22:30
@crozzy
crozzy force-pushed the vex-beta-testing branch 2 times, most recently from 61ae779 to 2d3c93e Compare August 11, 2026 17:42
@crozzy

crozzy commented Aug 11, 2026

Copy link
Copy Markdown
Contributor Author

I moved the VEX integration tests to test/periodic because of latency, I think that is technically a better home but I'm going to prioritize cleaning up the periodic failures so we actually get good signal

Comment thread test/periodic/doc.go
@crozzy
crozzy force-pushed the vex-beta-testing branch 2 times, most recently from debb1bc to b5fca96 Compare August 19, 2026 16:05
Comment thread rhel/vex/updater.go Outdated
@crozzy
crozzy requested a review from hdonnay August 19, 2026 16:09
Comment thread rhel/vex/updater.go Outdated
hdonnay
hdonnay previously approved these changes Aug 19, 2026
@crozzy
crozzy force-pushed the vex-beta-testing branch 3 times, most recently from 6e7acda to a00e372 Compare August 19, 2026 18:30
crozzy added 3 commits August 19, 2026 11:37
Add WithBaseURL() ParserOption and use it to create a self URL when the
VEX document doesn't contain csaf.CSAF.Document.References['self'].

Signed-off-by: crozzy <joseph.crosland@gmail.com>
Ignore src/nosrc when building vulnerabilities so they do not
overwrite empty archs from binary entries that share an arch-agnostic
status key.

Signed-off-by: crozzy <joseph.crosland@gmail.com>
The new Red Hat VEX feed is generally available and these tests help
verify some specific regressions are fixed.

Signed-off-by: crozzy <joseph.crosland@gmail.com>
@crozzy
crozzy requested a review from hdonnay August 19, 2026 18:51
@crozzy

crozzy commented Aug 19, 2026

Copy link
Copy Markdown
Contributor Author

@hdonnay sorry, turns out the rebase was a little more involved due to me circling the same files for ~ a month so I think I need another review

@crozzy

crozzy commented Aug 19, 2026

Copy link
Copy Markdown
Contributor Author

/fast-forward

@github-actions

Copy link
Copy Markdown

Triggered from #1871 (comment) by @​crozzy.

Trying to fast forward main (6b23c3f) to vex-beta-testing (592dc74).

Target branch (main):

commit 6b23c3f53e46ab997e974eb54cc8a1d863f45de2 (HEAD -> main, origin/main)
Author: Hank Donnay <hdonnay@redhat.com>
Date:   Tue Aug 11 12:16:39 2026 -0500

    postgres: add XXH64 vulnerability hash for comparison
    
    Signed-off-by: Hank Donnay <hdonnay@redhat.com>
    Change-Id: I3f9b6a1675678dca3e669dced0007a8a6a6a6964

Pull request (vex-beta-testing):

commit 592dc742dc85bf6a60b4a01db976d9fc514891d4 (pull_request/vex-beta-testing)
Author: Joseph Crosland <jcroslan@redhat.com>
Date:   Wed Apr 29 14:14:48 2026 -0700

    test: add red hat vex beta testing
    
    The new Red Hat VEX feed is generally available and these tests help
    verify some specific regressions are fixed.
    
    Signed-off-by: crozzy <joseph.crosland@gmail.com>

Fast forwarding main (6b23c3f) to vex-beta-testing (592dc74).

$ git push origin 592dc742dc85bf6a60b4a01db976d9fc514891d4:main
To https://github.com/quay/claircore.git
 ! [remote rejected]   592dc742dc85bf6a60b4a01db976d9fc514891d4 -> main (refusing to allow a GitHub App to create or update workflow `.github/workflows/updater-check.yml` without `workflows` permission)
error: failed to push some refs to 'https://github.com/quay/claircore.git'

@crozzy
crozzy merged commit aede4fd into quay:main Aug 19, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants