-
Notifications
You must be signed in to change notification settings - Fork 458
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
Remove DataStore pluggability #1707
Remove DataStore pluggability #1707
Conversation
See spiffe#1650 for background. As part of the fix, extends the PluginInfo interface to include information about whether or not the plugin is built in (to facilitate the newly added check). Fixes spiffe#1650 Signed-off-by: Andrew Harding <andrew.harding@hpe.com>
e35f02e
to
4245f0c
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.
Woo-hoo!
We may point in https://github.com/spiffe/spire/blob/master/doc/plugin_server_datastore_sql.md that it is the only DataStore supported.
pkg/server/catalog/catalog.go
Outdated
p.DataStore = datastore_telemetry.WithMetrics(p.DataStore, config.Metrics) | ||
// The DataStore interface is no longer pluggable (see #1650). | ||
if !p.DataStore.BuiltIn() { | ||
return nil, fmt.Errorf("pluggability for the DataStore is deprecated; only the built-in SQL plugin is supported") |
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.
no format needed
return nil, fmt.Errorf("pluggability for the DataStore is deprecated; only the built-in SQL plugin is supported") | |
return nil, errors.New("pluggability for the DataStore is deprecated; only the built-in SQL plugin is supported") |
Signed-off-by: Andrew Harding <andrew.harding@hpe.com>
c3d23f7
to
1a8e13e
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.
🚀
See #1650 for background. As part of the fix, extends the PluginInfo interface to include information about whether or not the plugin is built in (to facilitate the newly added check).
Fixes #1650 .