8000 Add root command by wagoodman · Pull Request #76 · anchore/grype-db · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add root command #76

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.

Sign up for GitHub

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

Jump to bottom
8000 Merged
merged 9 commits into from
Mar 15, 2023
Merged

Add root command #76

merged 9 commits into from
Mar 15, 2023

Conversation

wagoodman
Copy link
Contributor
@wagoodman wagoodman commented Mar 13, 2023 •
edited
Loading

In an effort to make grype-db easier to use I've made a few updates:

  • Adds a single root command to pull, build, and package the DB
  • Improves clarity of log statements
  • Add provider config generation option
  • Refactors the cache command to use disk state instead of config state (the provider config generation requires this change)
  • Remove checked-in grype-db application config (publishing uses config in publish directory now)
  • Decomposes some command options into smaller structs (due to cache commands using disk state and not config state)
  • Removes cache list-files command (was not being used as intended)

@wagoodman
add grype root command
6a84b71 
Signed-off-by: Alex Goodman <alex.goodman@anchore.com>
@wagoodman
add more logging around DB writers
67d0ab2 
Signed-off-by: Alex Goodman <alex.goodman@anchore.com>
@wagoodman
move grype-db config
32307e8 
Signed-off-by: Alex Goodman <alex.goodman@anchore.com>
@wagoodman
add generation options to vunnel configuration
78987d1 
Signed-off-by: Alex Goodman <alex.goodman@anchore.com>
@wagoodman
update docs with latests config changes
0d7fd54 
Signed-off-by: Alex Goodman <alex.goodman@anchore.com>
@wagoodman wagoodman marked this pull request as draft March 13, 2023 14:07
@wagoodman
fix oss build config
a37ddfb 
Signed-off-by: Alex Goodman <alex.goodman@anchore.com>
@wagoodman
cache commands should use disk state, not config state
1b89815 
Signed-off-by: Alex Goodman <alex.goodman@anchore.com>
@wagoodman wagoodman requested a review from a team March 15, 2023 13:35
@wagoodman
generate configs should honor vunnel executor
403bc06 
Signed-off-by: Alex Goodman <alex.goodman@anchore.com>
@wagoodman wagoodman marked this pull request as ready for review March 15, 2023 14:13
kzantow
kzantow approved these changes Mar 15, 2023
View reviewed changes
Copy link
Contributor
@kzantow kzantow left a comment

Choose a reason for hiding this comment

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

Just the nit about json:",inline"

@wagoodman
remove json inline struct tags + slim down grype-db config
8ae6a0e 
Signed-off-by: Alex Goodman <alex.goodman@anchore.com>
@wagoodman wagoodman merged commit 03bfb78 into main Mar 15, 2023
@wagoodman wagoodman deleted the last-day-cleanup branch March 15, 2023 19:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kzantow kzantow kzantow approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@wagoodman @kzantow
0