Skip to content
This repository has been archived by the owner on Nov 19, 2020. It is now read-only.

wip: try to make ds plugin #15

Merged
merged 31 commits into from
Sep 16, 2019
Merged

wip: try to make ds plugin #15

merged 31 commits into from
Sep 16, 2019

Conversation

kylebrandt
Copy link
Contributor

go mod vendor works now - FML.

Plugin is attempted to be run by Grafana if type is changed to datasource from app. Process dies because it trys to speak netrpc... maybe old version of grpc being vendored with more go mod pain?

@kylebrandt
Copy link
Contributor Author

Grafana poc branch will run this now after a build if the plugin.json is changed to datasource. But currently /api/tsdb/query expects a different model, so the request is rejected before it gets to the plugin since the len of Queries is 0.

But logs indicate the plugin is running, and not crashing or giving a 'netrpc' error anymore due to mismatch of grpc / plugin / protobuf libs.

* master:
  fix tests
  DataFrameHelper to MutableDataFrame
@kylebrandt kylebrandt mentioned this pull request Sep 11, 2019
@kylebrandt
Copy link
Contributor Author

Fixed the plugin.json files, no need to edit type anymore.

@ryantxu
Copy link
Member

ryantxu commented Sep 12, 2019

@kylebrandt -- I also updated the circle task and Makefile so it will be in sync (or close anyway!) with grafana/grafana#19029

@kylebrandt
Copy link
Contributor Author

Oops, last commit included mod files that have my paths in them, will fix when I have revisions to point go.mod at.

@kylebrandt kylebrandt marked this pull request as ready for review September 16, 2019 11:54
@kylebrandt kylebrandt merged commit c0e863d into master Sep 16, 2019
@kylebrandt kylebrandt deleted the dsPlug branch September 17, 2019 16:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants