-
Notifications
You must be signed in to change notification settings - Fork 11.6k
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: Restartable playlist & Playlist based on template variables #2841
Conversation
For screenshots please refer #773 |
Looks good, some unit tests would be good. Especially for playlistCtrl and its methods |
Thanks for the review. :) Sure, will add tests. This feature will be so useful to everyone. |
yes, its a very highly voted feature request! Not sure if it will make into 2.5 as that is very close to beta release. Maybe, no promises. |
I completely understand the priority of release as compared to merging this feature. Feel free to accept the PR at your own pace. Also, as this PR has conflicts with the master branch, let me know if you want me to merge the PR with the latest commit in master branch, I can do that. Thanks for this awesome piece of software. Grafana has been super useful to us. Keep up the good work. :) |
why does it have conflicts? |
Conflict is due to changes in "public/app/partials/playlist.html" file. #2831 fix changed this file and is merged into grafana master while I was working on this feature on my forked repo. I can merge the conflicts and push another commit to this PR. Do you want me to do that? |
…list Conflicts: public/app/partials/playlist.html
Added couple of unit tests, merged with Grafana's master branch. |
As now the 2.5v release date is moved ahead to Oct 15th, is there a chance that we merge this PR for #773, I am just curios? :) |
no, so much to do before then (docs, release article/blog, whats new in 2.5 docs) and need to finish cloudwatch data source supports |
not sure, would like to combine this and the persisted playlist feature and do a major overhaul of the playlist feature, not sure when there will be time for that |
Sure, let me know if there is any help needed. |
I understand you would like to have a major overhaul of the playlist feature, which is the best solution for playlist feature and I agree with you on that. But in my opinion, we can also work step by step, one step at a time. As the first step we can go ahead with this feature in the next release(2.5.1 or 2.6), so that playlist feature is much more richer than before. Then as the second step or an improvement, we can work on persisted playlists in (2.6 or 3.0) future releases, completing the rest of the work you want for playlist feature. Until then let users use this feature and make full use of it as this is one of the most asked features. Honestly, I do not see any harm in providing this feature to the users right now, it will only make their lives easy and make Grafana more powerful. :) |
I am starting to work on the persisted playlist in this same PR. |
Add flag option to display app version (Issue 3374)
…o utkarshcmu-hide-zero
…ni-moreno/grafana into toni-moreno-fix_refresh_variable_from_url
Closing this PR, will open another one with enhanced playlist features. |
Now, a dashboard can have a playlist based on selected template variables. A playlist can have some non-changing variables and some changing variables. The dashboard will display all the possible combinations of the changing variables depending on the playlist interval.
Please review the code, and suggest if anything needs modification.