Skip to content

[feature] Add input grype-version#228

Merged
willmurphyscode merged 29 commits intoanchore:mainfrom
ViacheslavKudinov:feature/add-version-input
Nov 20, 2023
Merged

[feature] Add input grype-version#228
willmurphyscode merged 29 commits intoanchore:mainfrom
ViacheslavKudinov:feature/add-version-input

Conversation

@ViacheslavKudinov
Copy link
Copy Markdown
Contributor

@ViacheslavKudinov ViacheslavKudinov commented May 30, 2023 •
edited
Loading

  • Added grype-version input to have ability to set specific version of Grype.
  • Removed debug input related things (input doesn't exist on the current version of JS code but adds extra annotation on Action tab:

"Unexpected input(s) 'debug', valid inputs are ['image', 'path', 'sbom', 'fail-build', 'output-format', 'severity-cutoff', 'only-fixed', 'add-cpes-if-none', 'by-cve']")

@ViacheslavKudinov ViacheslavKudinov marked this pull request as ready for review May 30, 2023 12:41
@ViacheslavKudinov ViacheslavKudinov changed the title Added input grype-version Add input grype-version Jun 1, 2023
@ViacheslavKudinov
Copy link
Copy Markdown
Contributor Author

ViacheslavKudinov commented Jun 1, 2023

Could be test of input has to be moved to some "demo" workflow, not sure about this moment.
Let me know if i have to move this part.

@ViacheslavKudinov
Copy link
Copy Markdown
Contributor Author

ViacheslavKudinov commented Jun 11, 2023 •
edited
Loading

Hi @kzantow
Could you, please, take a look?

@ViacheslavKudinov ViacheslavKudinov changed the title Add input grype-version [feature] Add input grype-version Jul 16, 2023
@ViacheslavKudinov
Copy link
Copy Markdown
Contributor Author

ViacheslavKudinov commented Nov 14, 2023

Hello,
any news on approving/rejecting/etc this one?

ViacheslavKudinov and others added 15 commits November 16, 2023 17:45
@ViacheslavKudinov
Added input grype-version
7d64c51 
Signed-off-by: Viacheslav Kudinov <viacheslav@kudinov.tech>
Signed-off-by: Viacheslav Kudinov <viacheslavkudinov@VK-MacBook-Pro-14-M1.local>
@ViacheslavKudinov
Added test of specific version
66ee7a4 
Signed-off-by: Viacheslav Kudinov <viacheslav@kudinov.tech>
Signed-off-by: Viacheslav Kudinov <viacheslavkudinov@VK-MacBook-Pro-14-M1.local>
@ViacheslavKudinov
Added specific version
1d3ba17 
Signed-off-by: Viacheslav Kudinov <viacheslav@kudinov.tech>
Signed-off-by: Viacheslav Kudinov <viacheslavkudinov@VK-MacBook-Pro-14-M1.local>
@ViacheslavKudinov
Added flag to no fail
b9cf4dc 
Signed-off-by: Viacheslav Kudinov <viacheslav@kudinov.tech>
Signed-off-by: Viacheslav Kudinov <viacheslavkudinov@VK-MacBook-Pro-14-M1.local>
@ViacheslavKudinov
Updated outdated Actions
cf68309 
Signed-off-by: Viacheslav Kudinov <viacheslav@kudinov.tech>
Signed-off-by: Viacheslav Kudinov <viacheslavkudinov@VK-MacBook-Pro-14-M1.local>
@ViacheslavKudinov
Removed debug input
ec07169 
Signed-off-by: Viacheslav Kudinov <viacheslav@kudinov.tech>
Signed-off-by: Viacheslav Kudinov <viacheslavkudinov@VK-MacBook-Pro-14-M1.local>
@ViacheslavKudinov
Removed debug from tests
3ae97a2 
Signed-off-by: Viacheslav Kudinov <viacheslav@kudinov.tech>
Signed-off-by: Viacheslav Kudinov <viacheslavkudinov@VK-MacBook-Pro-14-M1.local>
@ViacheslavKudinov
Removed debug from tests
3ceab09 
Signed-off-by: Viacheslav Kudinov <viacheslav@kudinov.tech>
Signed-off-by: Viacheslav Kudinov <viacheslavkudinov@VK-MacBook-Pro-14-M1.local>
@ViacheslavKudinov
Typo fix
9aba60f 
Signed-off-by: Viacheslav Kudinov <viacheslav@kudinov.tech>
Signed-off-by: Viacheslav Kudinov <viacheslavkudinov@VK-MacBook-Pro-14-M1.local>
@anchore-actions-token-generator @willmurphyscode
chore(deps): update Grype to v0.63.0 (#225)
4c71323 
Signed-off-by: GitHub <noreply@github.com>
Co-authored-by: willmurphyscode <willmurphyscode@users.noreply.github.com>
Signed-off-by: Viacheslav Kudinov <viacheslavkudinov@VK-MacBook-Pro-14-M1.local>
@wagoodman
add oss community board auto-add workflow (#231)
0c95533 
Signed-off-by: Viacheslav Kudinov <viacheslavkudinov@VK-MacBook-Pro-14-M1.local>
@ViacheslavKudinov
Merge witn main
7a57916 
Signed-off-by: Viacheslav Kudinov <viacheslav@kudinov.tech>
@ViacheslavKudinov
Update demo.yml with checkout@v4
bd43ad0 
Signed-off-by: Viacheslav Kudinov <viacheslav@kudinov.tech>
@ViacheslavKudinov
Update sarifdemo.yml with checkout@v4
1ea3736 
Signed-off-by: Viacheslav Kudinov <viacheslav@kudinov.tech>
@ViacheslavKudinov
Update test.yml with checkout@v4
f0a9f17 
Signed-off-by: Viacheslav Kudinov <viacheslav@kudinov.tech>
@ViacheslavKudinov ViacheslavKudinov marked this pull request as draft November 16, 2023 16:51
@ViacheslavKudinov
Merge witn main
ff3684e 
Signed-off-by: Viacheslav Kudinov <viacheslav@kudinov.tech>
@ViacheslavKudinov
Fix of default for grype_version
6bd6329 
Signed-off-by: Viacheslav Kudinov <viacheslav@kudinov.tech>
@ViacheslavKudinov
Update semver from 6.3.0 to 6.3.1
5e27c53 
Signed-off-by: Viacheslav Kudinov <viacheslav@kudinov.tech>
@ViacheslavKudinov
Revert update semver from 6.3.0 to 6.3.1
7d4da52 
Signed-off-by: Viacheslav Kudinov <viacheslav@kudinov.tech>
@ViacheslavKudinov
Audid fix
ce342b1 
Signed-off-by: Viacheslav Kudinov <viacheslav@kudinov.tech>
@ViacheslavKudinov
Dist
141e484 
Signed-off-by: Viacheslav Kudinov <viacheslav@kudinov.tech>
@ViacheslavKudinov
Set the Grype version to download v0.73.1
4151b23 
Signed-off-by: Viacheslav Kudinov <viacheslav@kudinov.tech>
@ViacheslavKudinov
Updated tests
8a55b4d 
Signed-off-by: Viacheslav Kudinov <viacheslav@kudinov.tech>
@ViacheslavKudinov
Merge branch 'main' into feature/add-version-input
9061c99
@ViacheslavKudinov
Rebuild
e7b99c5 
Signed-off-by: Viacheslav Kudinov <viacheslav@kudinov.tech>
@ViacheslavKudinov
Set timeout 50000
29a303c 
Signed-off-by: Viacheslav Kudinov <viacheslav@kudinov.tech>
@ViacheslavKudinov ViacheslavKudinov marked this pull request as ready for review November 16, 2023 19:29
@ViacheslavKudinov
Copy link
Copy Markdown
Contributor Author

ViacheslavKudinov commented Nov 17, 2023

Hi @willmurphyscode
is there any reaction from the "main" contributors?

@ViacheslavKudinov
Copy link
Copy Markdown
Contributor Author

ViacheslavKudinov commented Nov 17, 2023

or @wagoodman ?

@willmurphyscode
Copy link
Copy Markdown
Contributor

willmurphyscode commented Nov 20, 2023

Hi @ViacheslavKudinov,

Thanks much for the contribution, and sorry for the delay. I'm trying to get a release out today that will update to latest grype, and then have a couple additional chores to do in the repo. After that I'll get the changes from main into this branch and then try to get it reviewed and merged.

@willmurphyscode willmurphyscode self-assigned this Nov 20, 2023
@ViacheslavKudinov
Copy link
Copy Markdown
Contributor Author

ViacheslavKudinov commented Nov 20, 2023

Hi @willmurphyscode
thank you for the answer.
When new release done, i can fix conflicts to get it resolved before review.

@ViacheslavKudinov
Copy link
Copy Markdown
Contributor Author

ViacheslavKudinov commented Nov 20, 2023 •
edited
Loading

@willmurphyscode related question.
Why did you decide to remove jest snapshot testing when version was updated?
I could update it and get new snapshot with v0.73.1. Could be easy to get same for v0.73.3.

@willmurphyscode
Copy link
Copy Markdown
Contributor

willmurphyscode commented Nov 20, 2023

@ViacheslavKudinov good question! We removed snapshot testing for a few reasons:

  1. This repo's job is to invoke grype, not test the output of grype. Snapshot tests on the output of grype's output belong in the grype repo.
  2. Most changes to this repo are just bumping grype, which usually changes the output at least a bit, so we have to update snapshots on almost every change; having snapshot tests that need to change every time we bump grype just adds friction to keeping the repo up to date.

@willmurphyscode
Copy link
Copy Markdown
Contributor

willmurphyscode commented Nov 20, 2023

@ViacheslavKudinov thanks for offering to get your changes rebased on the new release!

I also plan to upgrade an pin some actions today. If you wouldn't mind waiting until #250 is in, that would probably save you rebasing / merging main again.

willmurphyscode
willmurphyscode reviewed Nov 20, 2023
View reviewed changes
Comment thread tests/sarif_output.test.js Outdated
@ViacheslavKudinov
Copy link
Copy Markdown
Contributor Author

ViacheslavKudinov commented Nov 20, 2023

I will merge GH Actions changes into my branch as well.

@ViacheslavKudinov ViacheslavKudinov marked this pull request as draft November 20, 2023 16:15
@ViacheslavKudinov ViacheslavKudinov marked this pull request as ready for review November 20, 2023 16:42
@willmurphyscode willmurphyscode merged commit 69f27a5 into anchore:main Nov 20, 2023
@willmurphyscode
Copy link
Copy Markdown
Contributor

willmurphyscode commented Nov 20, 2023

Thanks @ViacheslavKudinov! We'll get this out in the next release.

@ViacheslavKudinov
Copy link
Copy Markdown
Contributor Author

ViacheslavKudinov commented Nov 20, 2023

Thank you @willmurphyscode

@popey popey mentioned this pull request Jun 29, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@willmurphyscode willmurphyscode willmurphyscode approved these changes

Assignees

@willmurphyscode willmurphyscode

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ViacheslavKudinov @willmurphyscode @wagoodman