trinodb / grafana-trino

The Trino datasource allows to query and visualize Trino data from within Grafana.
Apache License 2.0
35 stars 10 forks source link

Bump github.com/grafana/grafana-plugin-sdk-go from 0.161.0 to 0.162.0 #210

Closed dependabot[bot] closed 1 year ago

dependabot[bot] commented 1 year ago

Bumps github.com/grafana/grafana-plugin-sdk-go from 0.161.0 to 0.162.0.

Release notes

Sourced from github.com/grafana/grafana-plugin-sdk-go's releases.

v0.162.0

What's Changed

Full Changelog: https://github.com/grafana/grafana-plugin-sdk-go/compare/v0.161.0...v0.162.0

Breaking changes

Both the Instance Manager and Instance Provider interfaces have been updated to require a context.Context as part of their APIs. This affects all plugins which perform manual instance management via the Instance Manager API. Adding context as a parameter to instance management faciliates propagation of contextual information, which is useful particularly for instance caching.

For example:

package plugin

type Plugin struct { im instancemgmt.InstanceManager }

func (p *Plugin) QueryData(ctx context.Context, req *backend.QueryDataRequest) (*backend.QueryDataResponse, error) { i, err := p.im.Get(ctx, req.PluginContext) // ctx is now required if err != nil { return nil, err } // .. }

Recommended fix

Automatic instance management

Automatic instance management for data sources was added to the SDK in version 0.97.0, which removes the need for plugin developers to use the Instance Manager directly. Support for app plugins was added in v0.140.0.

To use auto instance management, please refer to the relevant SDK documentation:

The following demonstrates usage of automatic instance management:

package main

func main() { err := datasource.Manage("grafana-test-datasource", plugin.New(), datasource.ManageOpts{}) if err != nil { os.Exit(1) } </tr></table>

... (truncated)

Commits
  • 2b4a9fb Use tenant ID from incoming gRPC meta for instance caching (#676)
  • e600743 Makes sure go build manifest file is generated with POSIX separators (#687)
  • See full diff in compare view


Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR: - `@dependabot rebase` will rebase this PR - `@dependabot recreate` will recreate this PR, overwriting any edits that have been made to it - `@dependabot merge` will merge this PR after your CI passes on it - `@dependabot squash and merge` will squash and merge this PR after your CI passes on it - `@dependabot cancel merge` will cancel a previously requested merge and block automerging - `@dependabot reopen` will reopen this PR if it is closed - `@dependabot close` will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually - `@dependabot ignore this major version` will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this minor version` will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself) - `@dependabot ignore this dependency` will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
cla-bot[bot] commented 1 year ago

Thank you for your pull request and welcome to the Trino community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. Continue to work with us on the review and improvements in this PR, and submit the signed CLA to cla@trino.io. Photos, scans, or digitally-signed PDF files are all suitable. Processing may take a few days. The CLA needs to be on file before we merge your changes. For more information, see https://github.com/trinodb/cla

dependabot[bot] commented 1 year ago

Superseded by #214.