[dev/build] skip reporting unreliable metrics

[spalger]

RE #87529

The metrics for the distributable file count and size are inaccurate enough that most PRs show a change in these values even when they shouldn't, and when they should the change in size can't be trusted.

[elasticmachine]

Pinging @elastic/kibana-operations (Team:Operations)

[tylersmalley]

Do you know why these values aren't accurate? Should we continue reporting them and remove them from the comment?

[spalger]

I don't know, but I know that both the size and file count fluctuate even for the same commit. I suspect it's the node_modules or something but I plan to investigate them in a follow up PR so I'd like to have them in the comment for that PR rather than disable the metric server-side.

[spalger]

I have also seen similar fluctuations going back at least a month, so I don't think there's an urgent need to remove these metrics from PRs, another reason we probably don't need to disable them server-side

[spalger]

@elasticmachine merge upstream

[kibanamachine]

💚 Build Succeeded

• continuous-integration/kibana-ci/pull-request
• Commit: 8924333

Metrics [docs]

Distributable file count

id	before	after	diff
default	48020	-	-48020
oss	27710	-	-27710

History

• 💔 Build #97372 failed a6bc2b2
• 💔 Build #97368 failed ed0d5c5

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream