Always capture tar version info

This lets us validate how well tar.zstd rollout will go before 2.19.0.
This commit is contained in:
Henry Mercer
2024-08-23 16:26:36 +01:00
parent e621ea8332
commit 73da3732dc
3 changed files with 6 additions and 9 deletions
+3 -4
View File
@@ -720,13 +720,12 @@ export async function setupCodeQLBundle(
features: FeatureEnablement,
logger: Logger,
): Promise<SetupCodeQLResult> {
let tarVersion: tar.TarVersion | undefined = undefined;
let zstdError: unknown = undefined;
const availableResult = await tar.isZstdAvailable(logger);
if (!toolsInput && (await features.getValue(Feature.ZstdBundle))) {
try {
const availableResult = await tar.isZstdAvailable(logger);
tarVersion = availableResult.version;
if (availableResult.available) {
return await setupCodeQLBundleWithZstdOption(
toolsInput,
@@ -765,7 +764,7 @@ export async function setupCodeQLBundle(
result.toolsDownloadStatusReport = Object.assign(
{},
result.toolsDownloadStatusReport,
tarVersion ? { tarVersion } : {},
{ tarVersion: availableResult.version },
zstdError ? { zstdError: wrapError(zstdError).message } : {},
);
return result;