-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
[usage] Enable usage view with feature flag #13065
Conversation
started the job as gitpod-build-se-usage-view.38 because the annotations in the pull request description changed |
/werft run 👍 started the job as gitpod-build-se-usage-view.39 |
751d56c
to
2807a77
Compare
const featureFlags: FeatureFlagConfig = { | ||
workspace_classes: { defaultValue: true, setter: setShowWorkspaceClassesUI }, | ||
persistent_volume_claim: { defaultValue: true, setter: setShowPersistentVolumeClaimUI }, | ||
usage_view: { defaultValue: true, setter: setShowUsageView }, | ||
}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What's the reason for making these very explicit (rather than reflecting over the set of flags)? I feel this has the potential to confuse others who are trying to use feature flags - having to define them here.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah never mind, I missed that you just moved the definition (and the problem already exists)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, assuming we've resolved the Q if usage should be generally visible - thread
/hold for Q around consensus on this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
- team usage view behavior does not require the feature flag if UBP, but does if not.
- personal usage view requires feature flag
/unhold |
Description
Since we have discoupled "usage" from stripe now, we can allow all users to use the usage view in order to understand how they use Gitpod (and where their hours go). This is also helpful to get a forecast of how many credits I'd need next month in case I want to switch to usage-based pricing.
This change enabled the usage view for everyone (depending on a feature flag, which is currently disabled for everyone in prod).
Related Issue(s)
Fixes #
How to test
Login to the preview environment and see usage view on your individual account and all team accounts.
Release Notes
Documentation
Werft options: