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
Simplify datastore construction #317
Conversation
30ec2dd
to
a92942b
Compare
1def5da
to
1783cec
Compare
7ab53fe
to
3bbcb1c
Compare
pkg/cmd/serve/datastore_options.go
Outdated
@@ -0,0 +1,135 @@ | |||
// Code generated by github.com/ecordell/optgen. DO NOT EDIT. |
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.
Can we name this file with a regex that we can add to .gitattributes?
pkg/cmd/serve/datastore.go
Outdated
@@ -0,0 +1,156 @@ | |||
package serve |
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.
There will be places other than serve that needs the datastore to be initialized. This might be best in pkg/cmd top-level
occasionally the number of iterations picked will take longer than the timeout for the test suite. capping it means that we sometimes will not run the full iterations to be convincing, but the effect should be amortized over PR runs. The cap (and timeout) can be removed for any change that we suspect may have implications for the test.
Simpler to build datastores from CLI args