PC enablement tools

Merge ~oem-solutions-engineers/pc-enablement/+git/bughamster:PCSWE-795 into ~oem-solutions-engineers/pc-enablement/+git/bughamster:master

  1. Git
  2. lp:~oem-solutions-engineers/pc-enablement/+git/bughamster
  3. PCSWE-795
  4. Merge into master
Proposed by Cyrus Lien
Status: Merged
Merged at revision: 92f49c409ac135565ce5efac223ba9e848d83d6f
Proposed branch: ~oem-solutions-engineers/pc-enablement/+git/bughamster:PCSWE-795
Merge into: ~oem-solutions-engineers/pc-enablement/+git/bughamster:master
Diff against target: 135 lines (+76/-6)
1 file modified
BugHamster/BugHamster.py (+76/-6)
Reviewer Review Type Date Requested Status
Andy Chi (community) Approve
Atlas Yu Approve
jeremyszu Pending
Review via email: mp+435403@code.staging.launchpad.net

Commit message

PCSWE-795: Won't file CVE bug if USN within 7 days.

Description of the change

Test cases:
1. CVE failed test, all USN exceed 7 days.
   Summission.jason: submission_cve-fail-all-exceed-test2.tar.xz.
   Result: https://bugs.launchpad.net/somerville/+bug/2002333/comments/1

2. CVE pass test, all USN within 7 days.
   Submission.json: submission_cve-all-within-buffer-X101.tzr.xz
   Result: https://bugs.launchpad.net/somerville/+bug/2002333/comments/6

3. CVE fail test, 3 of 4 USN exced 7 days.
   Submission.json: submission_cve-combo-tests-X100.tar.xz
   Result: https://bugs.launchpad.net/somerville/+bug/2002333/comments/8

To post a comment you must log in.
Revision history for this message
Atlas Yu (pseudoc) :
review: Approve
Revision history for this message
Atlas Yu (pseudoc) wrote :

Minutes after my approval, I realized that it may not only be USN(Ubuntu security notice), but LSN(Linux security notice) as well.

review: Needs Fixing
Revision history for this message
Atlas Yu (pseudoc) wrote :

You may find the intermediate "report.xml" under the path like:
/var/tmp/checkbox-ng/sessions/checkbox-run-2023-01-10T01.31.28.session/session-share/oval/

The cvescan test is dumping the first element of the cell under column "Reference ID" within the main table.

USN and LSN are like a debian meta package to some of the degree, which points to all involved CVEs.

Revision history for this message
Cyrus Lien (cyruslien) wrote :

@Atlas,

Updated, please review again.

Test result with LSN.
https://bugs.launchpad.net/somerville/+bug/2002333/comments/13

Revision history for this message
Atlas Yu (pseudoc) wrote :

LGTM

review: Approve
Revision history for this message
Andy Chi (andch) wrote :

LGTM.

review: Approve

There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches