Skip to content

Commit

Permalink
Merge pull request #2612 from github/angelapwen/report-linux-runner-r…
Browse files Browse the repository at this point in the history
…elease

Telemetry: report OS release for GitHub-hosted Linux runners
  • Loading branch information
angelapwen authored Nov 18, 2024
2 parents e3c67a0 + f9ada54 commit e782c3a
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 3 deletions.
4 changes: 3 additions & 1 deletion lib/status-report.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion lib/status-report.js.map

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

5 changes: 4 additions & 1 deletion src/status-report.ts
Original file line number Diff line number Diff line change
Expand Up @@ -10,6 +10,7 @@ import {
getWorkflowRunAttempt,
getActionVersion,
getRequiredInput,
isSelfHostedRunner,
} from "./actions-util";
import { getAnalysisKey, getApiClient } from "./api-client";
import { type Config } from "./config-utils";
Expand Down Expand Up @@ -340,7 +341,9 @@ export async function createStatusReportBase(
// Values other than X86, X64, ARM, or ARM64 are discarded server side
statusReport.runner_arch = process.env["RUNNER_ARCH"];
}
if (runnerOs === "Windows" || runnerOs === "macOS") {
if (!(runnerOs === "Linux" && isSelfHostedRunner())) {
// We do not report the release number for Linux self-hosted runners
// because the custom build suffix may be private customer information.
statusReport.runner_os_release = os.release();
}
if (codeQlCliVersion !== undefined) {
Expand Down

0 comments on commit e782c3a

Please sign in to comment.