Send tools telemetry to init status report#1497
Conversation
a9545a6 to
f24d78c
Compare
init status report
a217825 to
4fa58ed
Compare
|
Looks like a bit of an inscrutable error, Also, oddly enough the error fails in the |
|
I think this could be as boring as just needing to merge in main and recompile the Action — some of the compiled files seem to be missing at the moment. |
…n/codeql-action into tools-telemetry-report
|
I have an issue with |
8ca90f9 to
919c531
Compare
henrymercer
left a comment
There was a problem hiding this comment.
I like how you've used this additional telemetry information to improve the tests for setting up the CodeQL tools! Here's a couple of initial comments:
|
I think I've addressed all comments and PR is ready for review again! |
Co-authored-by: Henry Mercer <henry.mercer@me.com>
This PR sends the following fields to the
initstatus report on success and failure:tools_sourcetools_sourcewasDOWNLOAD,tools_download_duration_msandtools_feature_flags_validNote that previously we only sent additional fields on
initsuccess, but because we would like to support the tools fields on failure as well, the PR changes the status report behavior so that theinitreport is sent on failure too, with as many fields populated as possible.Merge / deployment checklist