Skip to content
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

WIP: Reuse panels across dashboards #3825

Closed

Conversation

mtanda
Copy link
Collaborator

@mtanda mtanda commented Jan 22, 2016

I have so many composite graph for performance monitoring.
I want to reuse the panel setting.
#1803

This PR will fix the issue.

@torkelo How do you think about this approach?

@akrus
Copy link

akrus commented May 11, 2016

Any progress here?

@mtanda
Copy link
Collaborator Author

mtanda commented May 11, 2016

nothing from me..

@bergquist
Copy link
Contributor

I think reusable panels should be a different entity rather then symlinking to other dashboards. But I'm not sure.

@mtanda
Copy link
Collaborator Author

mtanda commented Jun 26, 2016

@bergquist
Honestly, I don't need to reuse panel itself. I want to reuse panel definition (e.g. query expression, style, etc...)
So, I want the feature like "templated panel".
I want to define "templated panel" and embed the panel to several dashboard.

"Copy and paste panels" doesn't fit my use case.
If same panel is copied to several dashboard, and if want to update all panel definition, it is very hard.

I tried to reuse the panel definitions by defining panel in file based dashboard, the panel definition contains template variables, and load the panel definition by panel id.

The good point of loading panel directly is, user can edit the definition by the same panel editor, and check the result quickly.
The bad point is, we should handle "refereed panel is missing" error.

Maybe #4940 is similar feature what I want.
And I think the feature takes better approach, if the feature support updating all panel definitions when updating original one.

@mtanda
Copy link
Collaborator Author

mtanda commented Jun 26, 2016

After the read issue again, I notice I misunderstand what the feature is.
But, I think save panel definition to template is good idea, so I agree with the point "should be a different entity rather then symlinking to other dashboards".

@mtanda
Copy link
Collaborator Author

mtanda commented Jul 2, 2016

This approach is not good. So close this. (If I have a time, try to make PR by another approach.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/external This PR is from external contributor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants