-
Notifications
You must be signed in to change notification settings - Fork 85
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.
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
Louis/remove db coalescer #106
Conversation
a22ea1c
to
8881690
Compare
8881690
to
573ec7f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a comment nit and I think you should use the errgroup differently
internal/indexer/linux/coalescer.go
Outdated
@@ -23,18 +22,12 @@ type layerArtifacts struct { | |||
// for dpkg, os-release, and apt scanners | |||
type Coalescer struct { | |||
// a store to access scanartifacts |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
old comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK, but drone is complaining about mergeability.
Just dealt with merge conflicts. The new Digest type makes it tough to test arbitrary layer fixtures. |
8de852c
to
9cc28eb
Compare
9cc28eb
to
cd147dd
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Thanks for pulling the random digest generator out into the test package.
This PR removes the indexer's Store implementation from the Coalescer implementations. This is step one to moving internal indexer interfaces to a driver package.