Skip to content

Try to find 0 error bug and change auto update to PR #5

Merged
abhigyanghosh30 merged 11 commits intomainfrom
deploy
Aug 21, 2025
Merged

Try to find 0 error bug and change auto update to PR #5
abhigyanghosh30 merged 11 commits intomainfrom
deploy

Conversation

@abhigyanghosh30
Copy link
Contributor

@abhigyanghosh30 abhigyanghosh30 commented Aug 4, 2025

The fields on the USN files changed and this PR updates the code to fetch from the correct fields
It also removes the auto updates to create a PR with the changes

QA steps:
Run python main.py locally
The files should have non zero value (ideally count should be higher than the previous counts)

@abhigyanghosh30 abhigyanghosh30 marked this pull request as draft August 4, 2025 22:34
@abhigyanghosh30 abhigyanghosh30 marked this pull request as ready for review August 5, 2025 09:55
@abhigyanghosh30 abhigyanghosh30 requested a review from Copilot August 5, 2025 09:57

This comment was marked as outdated.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@abhigyanghosh30
Copy link
Contributor Author

@terameta even though the workflow fails, the PR is made with the correct variables

@abhigyanghosh30 abhigyanghosh30 requested a review from Copilot August 6, 2025 13:23
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR fixes a bug where USN data extraction was yielding zero results due to incorrect field mapping, and changes the automation workflow from auto-commits to creating pull requests for manual review.

  • Updates USN data parsing to extract from upstream field instead of related field and handle "UBUNTU-" prefixed identifiers
  • Adds debug logging to track the number of CVE entries and packages processed
  • Replaces automatic commit workflow with pull request creation for better change visibility

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
main.py Updates USN data field mapping and adds debug print statements
.github/workflows/schedule.yaml Replaces auto-commit with PR creation workflow

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@abhigyanghosh30
Copy link
Contributor Author

@terameta the latest workflow ran without errors. Can we merge this?
Ref action: https://github.com/canonical/pro-cve-aggregator/actions/runs/16910276736

@abhigyanghosh30 abhigyanghosh30 merged commit 9ecc84d into main Aug 21, 2025
2 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.

3 participants