-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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 Collection] [schema] static_telemetry
#77902
[Usage Collection] [schema] static_telemetry
#77902
Conversation
Pinging @elastic/kibana-telemetry (Team:KibanaTelemetry) |
@elasticmachine merge upstream |
@elasticmachine merge upstream |
@elasticmachine merge upstream |
💚 Build SucceededBuild metricsdistributable file count
History
To update your PR or re-run it, just comment with: |
@@ -60,10 +61,12 @@ export function isFileReadable(path: string): boolean { | |||
* @param configPath The config file path. | |||
* @returns The unmodified JSON object if the file exists and is a valid YAML file. | |||
*/ | |||
export async function readTelemetryFile(path: string): Promise<object | undefined> { |
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.
Using configPath rather than path now matches the function description, thanks for making them consistent. We could have gone either way here, either updating the param as is the case here or changing the description.
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.
Thank you for noticing. I decided on the configPath
side because path
matches the 'path'
library (and I tend to avoid it as a variable when possible).
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
Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com> Co-authored-by: Elastic Machine <elasticmachine@users.noreply.github.com>
Pinging @elastic/kibana-core (Team:Core) |
Summary
Add
schema
definition to the collectorstatic_telemetry
.Related to #70180.
For maintainers