[GitHub] [beam] Ardagan commented on pull request #11555: [BEAM-8134] Grafana dashboards for Load Tests and IO IT Performance Tests

[GitBox <gi...@apache.org>]

Ardagan commented on pull request #11555:
URL: https://github.com/apache/beam/pull/11555#issuecomment-622141777


   Can be useful to add link instructions on how to access dashboards into PR description.
   [CWIKI](https://cwiki.apache.org/confluence/display/BEAM/Community+Metrics), section Grafana UI
   
   I took a brief look at dashboards, here are my ideas:
   
   - I'd put graphs with similar arguments side-by-side, make graphs narrower. Goal is to see general sense of current status. Current layout is too sparse and it takes way too much time to scroll.
   - Set default dashboard interval to last month. If you feel month is not enough, choose value you feel more relevant.
   - Some space can be recovered by removing time scale.
   - I think that similar metrics can be put into single graph, ex: TextIO 1Gb [GCS, HDFS].
   - I would add a short description of what tests are executed. We all can access code, but it would be much more handy. This can be done either as a text field at the top of dashboard, or as a description hint on each of graphs, or both.
   
   These are my initial thoughts.
   
   R: @aaltay 
   Ahmet, I thought you might be interested as well.
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org