appwrite icon indicating copy to clipboard operation
appwrite copied to clipboard

New usage stats

Open lohanidamodar opened this issue 3 years ago • 1 comments

What does this PR do?

Adds new usage stats

From influxdb

  • inbound

  • outbound

  • functions.functionId.builds

  • functions.functionId.build

  • functions.functionId.failures -> functions.functionId.executions.failures

  • functions.functionId.builds.failures

  • functions.functionId.compute <total function compute time -> build time + execution time>

  • ⚠️ The function stats test seems inconsistent because of the varying execution time, and stats collection time

Note: apart from inbound and outbound metrics, other metrics can also be calculated from Project DB without influxDB, however might miss some function that are created, executed and deleted between current 15 minute duration

From Project DB

  • functions.executionTime
  • functions.buildTime
  • functions.compute <total functions compute time { buildTime + executionTime }>

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work.)

Related PRs and Issues

(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)

Have you read the Contributing Guidelines on issues?

YES

lohanidamodar avatar Jun 29 '22 07:06 lohanidamodar

Style-wise, LGTM. I do worry about making the tests take more time considering how long they already take, but I'm not sure how to best solve that problem.

stnguyen90 avatar Aug 04 '22 19:08 stnguyen90

Can you update the base branch for this ? @lohanidamodar Is it still feat-audits label ?

christyjacob4 avatar Aug 17 '22 07:08 christyjacob4