Skip to content

project release: CVEs from packaging commits, follow aggregates; CHANGELOG-only release pushes skip populate - #198

Merged
rjd15372 merged 2 commits into
mainfrom
changelog-cve-scan
Oct 28, 2026
Merged

rjd15372 merged 2 commits into
mainfrom
changelog-cve-scan

Conversation

@rjd15372

Copy link
Copy Markdown
Collaborator

The 17.11-3 / 18.6-3 cuts (#197) listed python3-dns as "Not scanned" although 6068c66 bumped it for CVE-2026-114605. Two gaps:

  1. Staging carries python3-dns as an _aggregate of ppg:common:deps, so _find_pkg_service found no _service: no upstream URL for the CVE scanner and no release-note link on the Changed line. It now follows a local _aggregate to the source package's _service (also for subproject keys).
  2. The scanner only read upstream release notes. It now also reads this repo's commit messages since the previous release tag that touch the package's source dir, and collects CVE-… / GHSA-… ids the upstream line does not already list:
    - python3-dns: 2.8.0→2.9.0 fixes CVE-2026-114605 (packaging commit 6068c662)

Also, obs-release-pr: a push to a release PR that changes only release CHANGELOG.md files (hand-editing the notes) no longer repopulates. resolve compares the push (event.before...head) and reports kind=changelog; populate and QA are skipped and the existing pre-release stands. Opening the PR, a dispatch, any other file in the push, or a failed compare still repopulates.

Testing:

  • New tests for the aggregate follow, commit-advisory harvest (dedupe vs upstream, GHSA), and changelog_only; 748 passed, pyright 0 errors.
  • Real-tree check: _build_changed_packages + scan_release_cves for python3-dns against ppg/18.6-2 → upstream resolved to rthalley/dnspython, line above produced.

After merge: push the hand-edited CHANGELOG fix to #197 (should skip populate).

The release cut listed python3-dns as "Not scanned" although 6068c66
bumped it for CVE-2026-114605: staging carries it as an _aggregate of
ppg:common:deps, so no _service (and no upstream url) was found, and the
CVE named in our own commit was never looked at.

- _find_pkg_service follows a local _aggregate to the source package's
  _service (scanner input and the Changed-line release-note link).
- The CVE scan also reads this repo's commit messages since the previous
  release tag for the package's source dir, collecting CVE and GHSA ids
  the upstream line does not already list.
- obs-release-pr: a push that changes only release CHANGELOG.md files
  keeps the populated pre-release (kind=changelog, populate/QA skipped).

Signed-off-by: Ricardo Dias <ricardo.dias@percona.com>
0456f03 landed unformatted, so the Python Tests black check fails on
every PR.

Signed-off-by: Ricardo Dias <ricardo.dias@percona.com>
@rjd15372
rjd15372 merged commit 2f8098b into main Oct 28, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant