-
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] Add config for billInstancesAfter #12257
Conversation
@@ -243,6 +243,7 @@ EOF`); | |||
private configureUsage(slice: string) { | |||
exec(`yq w -i ${this.options.installerConfigPath} experimental.webapp.usage.enabled true`, { slice: slice }) | |||
exec(`yq w -i ${this.options.installerConfigPath} experimental.webapp.usage.schedule 1m`, { slice: slice }) | |||
exec(`yq w -i ${this.options.installerConfigPath} experimental.webapp.usage.billInstancesAfter "2022-08-11T08:05:32.499Z"`, { slice: slice }) |
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.
How was this value determined? Is this value still appropriate given current contexts?
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.
This is the context for the value.
@@ -4,6 +4,7 @@ | |||
"default": 0.1666666667, | |||
"gitpodio-internal-xl": 0.3333333333 | |||
}, | |||
"billInstancesAfter":"2022-08-11T08:05:32.499Z", |
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.
[nit] Space after colon
"billInstancesAfter":"2022-08-11T08:05:32.499Z", | |
"billInstancesAfter": "2022-08-11T08:05:32.499Z", |
Description
This sets the preview environment config for
billInstancesAfter
to be as mentioned here.For staging and prod will be PRs against the Ops repo.
Related Issue(s)
Relates #12223
How to test
kubectl get pods
kubectl logs -f [USAGE-POD]
billInstancesAfter
with the specified value.Release Notes
Documentation
Werft options: