Skip to content
This repository has been archived by the owner on Aug 9, 2022. It is now read-only.

Add overall frontend metrics for actions #287

Merged

Conversation

zhongnansu
Copy link
Member

@zhongnansu zhongnansu commented Jan 6, 2021

Issue #, if available:

Description of changes:

  • Add some overall metrics for "action"(API call)
 "request_total": 5, # total requests since the server is up
 "request_count": 0, # total requests from last minute
 "success_count": 0, # successgul requests from last minute
 "failed_request_count_system_error": 0, # from last minute
 "failed_request_count_user_error": 0 # from last minute
  • response format
{
 "report": {
   "create": {
     "count": 0,
     "system_error": 0,
     "user_error": 0,
     "total": 0
   },
   "create_from_definition": {
     "count": 0,
     "system_error": 0,
     "user_error": 0,
     "total": 0
   },
   "download": {
     "count": 0,
     "system_error": 0,
     "user_error": 0,
     "total": 1
   },
   "list": {
     "count": 0,
     "system_error": 0,
     "user_error": 0,
     "total": 2
   },
   "info": {
     "count": 0,
     "system_error": 0,
     "user_error": 0,
     "total": 0
   }
 },
 "report_definition": {
   "create": {
     "count": 0,
     "system_error": 0,
     "user_error": 0,
     "total": 0
   },
   "list": {
     "count": 0,
     "system_error": 0,
     "user_error": 0,
     "total": 2
   },
   "info": {
     "count": 0,
     "system_error": 0,
     "user_error": 0,
     "total": 0
   },
   "update": {
     "count": 0,
     "system_error": 0,
     "user_error": 0,
     "total": 0
   },
   "delete": {
     "count": 0,
     "system_error": 0,
     "user_error": 0,
     "total": 0
   }
 },
 "report_source": {
   "list": {
     "count": 0,
     "system_error": 0,
     "user_error": 0,
     "total": 0
   }
 },
 "dashboard": {
   "pdf": {
     "download": {
       "count": 0,
       "total": 0
     }
   },
   "png": {
     "download": {
       "count": 0,
       "total": 0
     }
   }
 },
 "visualization": {
   "pdf": {
     "download": {
       "count": 0,
       "total": 0
     }
   },
   "png": {
     "download": {
       "count": 0,
       "total": 0
     }
   }
 },
 "saved_search": {
   "csv": {
     "download": {
       "count": 0,
       "total": 1
     }
   }
 },
#### newly-added metrics ####
 "request_total": 5,
 "request_count": 0,
 "success_count": 0,
 "failed_request_count_system_error": 0,
 "failed_request_count_user_error": 0
}

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@zhongnansu zhongnansu changed the title Add extra frontend overall metrics for actions Add overall frontend metrics for actions Jan 6, 2021
@codecov
Copy link

codecov bot commented Jan 6, 2021

Codecov Report

Merging #287 (75aacab) into dev (45abee7) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##              dev     #287   +/-   ##
=======================================
  Coverage   74.94%   74.94%           
=======================================
  Files          32       32           
  Lines        1764     1764           
  Branches      342      342           
=======================================
  Hits         1322     1322           
  Misses        437      437           
  Partials        5        5           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 45abee7...75aacab. Read the comment docs.

@zhongnansu zhongnansu marked this pull request as ready for review January 6, 2021 22:06
@zhongnansu zhongnansu merged commit 89a5fcf into opendistro-for-elasticsearch:dev Jan 6, 2021
zhongnansu added a commit to zhongnansu/kibana-reports that referenced this pull request Jan 6, 2021
zhongnansu added a commit to zhongnansu/kibana-reports that referenced this pull request Jan 7, 2021
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

3 participants