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

[WIP]feat(home): measure api #222

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

masaimu
Copy link
Collaborator

@masaimu masaimu commented Mar 13, 2023

Which issue does this PR close?

Closes #221

What changes are included in this PR?

  • All resource objects that need to be measured implement the io.holoinsight.server.common.service.Measurable interface.
  • FacadeTemplateImpl#manage verifies the permission owned by each token, including tenant permissions, resource access permissions, and access mode permissions.
  • FacadeTemplateImpl#manage measures access amount.
  • MeasureStatistician performs the statistics and outputs the measurement results periodically.

Are there any user-facing changes?

Non2.

How does this change test

UT:

E2E:

@masaimu masaimu marked this pull request as draft March 13, 2023 06:35
@masaimu masaimu self-assigned this Apr 4, 2023
@masaimu masaimu added the enhancement New feature or request label Apr 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Measure
1 participant