feat(ubuntu): Expose Ubuntu fix status.

[skahn007gl]

Description

Supporting PR for TrivyDB PR407 as requested in comment by @knqyf263

Related PRs

• TrivyDB expose Ubuntu fix status

Remove this section if you don't have related PRs.

Checklist

• I've read the guidelines for contributing to this repository.
• I've followed the conventions in the PR title.
• I've added tests that prove my fix is effective or that my feature works.
• I've updated the documentation with the relevant information (if needed).

[skahn007gl]

Hey @knqyf263
Test's are passing with Status added.
I was a bit surprised I did not need to add the status field into the test data, but i introduced failures to observe the status field is being populated.

Please kick off a test run & review when you can.

Thanks for holding my hand through this while I learned how these tests worked.

[DmitriyLewen]

Hello @skahn007gl

Please run go mod edit -replace github.com/aquasecurity/trivy-db=github.com/skahn007gl/trivy-db@9bdfe07de664866c0d889b2d1f8d6ecc11b640ad (this is latest commit from aquasecurity/trivy-db#407) and go mod tidy + push changes of go.mod and go.sum files.
This is necessary to ensure that changes to Trivy-db don't break Trivy.

And resolve the conflict with the file integration/testdata/fixtures/db/vulnerability.yaml.

Regards, Dmitriy

[skahn007gl]

Apologies @chen-keinan, @simar7, @nikpivkin.
You are not required to review this PR, the request was made in error.

[skahn007gl]

Hello @DmitriyLewen
I have updated go.mod to target the latest commit for PR407
Fixed up the test error in PR407 to.

[DmitriyLewen]

@skahn007gl sorry for delay.

Resolve conflict with go.mod file, please.

[skahn007gl]

@skahn007gl sorry for delay.

Resolve conflict with go.mod file, please.

Hi @DmitriyLewen, sorry for the delay on my side.
Can you kick off a test run and re-review this when you have time.

[skahn007gl]

Hey @knqyf263 Thank you for kicking off a test run.
I will look into the failing test now :)

[skahn007gl]

Hi @knqyf263 Please kick off another test run.
The failing test was new to me as it was not included in mage test:integration, I have updated pr.md to include mage test:vm

[skahn007gl]

Hi @DmitriyLewen @knqyf263
All checks are now passing, can we proceed to Merging?

Can you please kick off a test run on aquasecurity/trivy-db#407

[skahn007gl]

@knqyf263 I have updated this PR to match removing ignored from trivy-db PR

[knqyf263]

You can point to aquasecurity/trivy-db@42b851c now.

[knqyf263]

We had a critical problem and reverted the commit in trivy-db. We need to rethink about implementation in trivy-db.

[skahn007gl]

We had a critical problem and reverted the commit in trivy-db. We need to rethink about implementation in trivy-db.

As it is a type error, we could convert trivy-db to use the string value instead.