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

get_dashboards() returns empty dashboard_state and image_hash #290

Open
tshepitko opened this issue Jan 27, 2020 · 5 comments
Open

get_dashboards() returns empty dashboard_state and image_hash #290

tshepitko opened this issue Jan 27, 2020 · 5 comments

Comments

@tshepitko
Copy link

tshepitko commented Jan 27, 2020

get_dashboards() method returns list of dashboards where each dashboard has empty parameters which are mandatory for create_dashboard() call:

  • dashboard_state
@wamsiv
Copy link
Contributor

wamsiv commented Jan 27, 2020

dashboard_state is intentionally left empty to reduce payload size(especially when returning a list of large amount of dashboards). You can grab the dashboard_id of the interesting dashboard from the list and make a call to get_dashboard() to get full details.

@tshepitko
Copy link
Author

Definitely I can use get_dashboard() method, but single call get_dashboards() will work faster, because:

  • one TCP connection instead of N consequent TCP connections
  • with TLS enabled only single TLS handshake occurs
  • TCP slow start (1 vs N connections)

How many dashboards cause issues with single call get_dashboards() where dashboard_state is exposed?

@wamsiv
Copy link
Contributor

wamsiv commented Jan 29, 2020

What is the use case here? As I understand if you are trying to fetch dashboard details only to create a new dashboard then you may use duplicate_dashboard() call which takes dashboard_id as an argument and do the same thing for you internally.

@wamsiv
Copy link
Contributor

wamsiv commented Jan 29, 2020

We can certainly look into sending dashboard_state and other missing details over the wire but have not yet received a use case where user may have to create/duplicate several dashboards from the existing ones.

@tshepitko
Copy link
Author

My use case is to make backup of all currently available dashboards which will be stored in files and exported to external storage

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants