[v9.5.x] Plugins: Skip instrumenting plugin build info for core and bundled plugins (#66113)

Plugins: Skip instrumenting plugin build info for core and bundled plugins (#66105)

* only instrument plugin build for non core/bundled plugins

* fix import

(cherry picked from commit 7808e74260)

Co-authored-by: Will Browne <wbrowne@users.noreply.github.com>
This commit is contained in:
Grot (@grafanabot)
2023-04-06 14:11:25 +03:00
committed by GitHub
co-authored by Will Browne
parent 734fb8a3e5
commit a013ee7be4
+4 -2
View File
@@ -177,8 +177,6 @@ func (l *Loader) loadPlugins(ctx context.Context, src plugins.PluginSource, foun
if err != nil {
return nil, err
}
metrics.SetPluginBuildInformation(p.ID, string(p.Type), p.Info.Version, string(p.Signature))
if errDeclareRoles := l.roleRegistry.DeclarePluginRoles(ctx, p.ID, p.Name, p.Roles); errDeclareRoles != nil {
l.log.Warn("Declare plugin roles failed.", "pluginID", p.ID, "err", errDeclareRoles)
}
@@ -188,6 +186,10 @@ func (l *Loader) loadPlugins(ctx context.Context, src plugins.PluginSource, foun
if err := l.load(ctx, p); err != nil {
l.log.Error("Could not start plugin", "pluginId", p.ID, "err", err)
}
if !p.IsCorePlugin() && !p.IsBundledPlugin() {
metrics.SetPluginBuildInformation(p.ID, string(p.Type), p.Info.Version, string(p.Signature))
}
}
return verifiedPlugins, nil