Skip to content

[UBP] Add LastUsageReconciliationTime RPC #12612

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

Closed
wants to merge 3 commits into from

Conversation

andrew-farries
Copy link
Contributor

@andrew-farries andrew-farries commented Sep 2, 2022

Description

Add a LastUsageReconcilationTime RPC to the usage service. The RPC returns the last time of the most recent usage reconciliation.

The intention is to make this information visible on the usage view in the UI in order to give an indication of the 'freshness' of the data on that page.

Related Issue(s)

Part of #12064

How to test

  1. Port forward to the usage service gRPC server: kubectl port-forward svc/usage 9001:9001
  2. Use evans -r -p 9001 to connect to it.
  3. Invoke the LastUsageReconcilationTime RPC on the UsageService.

Release Notes

NONE

Documentation

Werft options:

  • /werft with-preview

Andrew Farries added 3 commits September 1, 2022 13:57
@andrew-farries andrew-farries requested a review from a team September 2, 2022 10:50
@github-actions github-actions bot added the team: webapp Issue belongs to the WebApp team label Sep 2, 2022
@werft-gitpod-dev-com
Copy link

started the job as gitpod-build-af-return-report-generation-time.8 because the annotations in the pull request description changed
(with .werft/ from main)

@Songyuwasd
Copy link

Songyuwasd commented Sep 2, 2022

求助

@andrew-farries
Copy link
Contributor Author

Closing after internal discussion in favour of #12624.

@andrew-farries andrew-farries deleted the af/return-report-generation-time branch September 2, 2022 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note-none size/XXL team: webapp Issue belongs to the WebApp team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants