Skip to content

Merge Patchlevel categories of ODG findings #483

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

heldkat
Copy link
Collaborator

@heldkat heldkat commented Jun 26, 2025

What this PR does / why we need it:

Due to recent updates to the processing times of outdated OS, a differentiation between the more-than-one-patchlevel-behind and one-patchlevel-behind categories became obsolete. Here, we made the necessary adaptations to merge these two groups.

Which issue(s) this PR fixes:
Fixes #

Special notes for your reviewer:

Release note:


@heldkat heldkat requested a review from a team as a code owner June 26, 2025 12:20
@heldkat heldkat changed the title Merge and ODG findings Merge Patchlevel categories of ODG findings Jun 26, 2025
Copy link
Collaborator

@ccwienk ccwienk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

generally: lgtm
however, I wonder whether we will need to migrate existing data

/cc @zkdev | @8R0WNI3 | @TuanAnh17N

@8R0WNI3
Copy link
Member

8R0WNI3 commented Jun 27, 2025

I wonder whether we will need to migrate existing data

Yes, in case we decide to replace existing categorisations, we have to adjust existing DB-entries with those old categorisations to use the new one instead.

Co-authored-by: Tuan Anh Nguyen (TuanAnh17N) <tuan-anh_nguyen@gmx.de>
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.

3 participants