pr feedback

cloud-runner-develop
Frostebite 2025-12-06 05:13:54 +00:00
parent 192cb2e14e
commit 46e3ba8ba2
4 changed files with 17 additions and 5 deletions

10
dist/index.js vendored
View File

@ -5610,7 +5610,12 @@ class RemoteClient {
// WIP - need to give the pod permissions to create config map
await remote_client_logger_1.RemoteClientLogger.handleLogManagementPostJob();
// Ensure success marker is present in logs for tests
cloud_runner_logger_1.default.log(`Activation successful`);
// Log to both CloudRunnerLogger and stdout to ensure it's captured
const successMessage = `Activation successful`;
cloud_runner_logger_1.default.log(successMessage);
// Also output directly to stdout to ensure it's captured by log streaming
process.stdout.write(`${successMessage}\n`);
console.log(successMessage);
return new Promise((result) => result(``));
}
static async runCustomHookFiles(hookLifecycle) {
@ -7455,7 +7460,8 @@ echo "CACHE_KEY=$CACHE_KEY"`;
if [ ! -f "/data/cache/$CACHE_KEY/build/build-$BUILD_GUID.tar" ] && [ ! -f "/data/cache/$CACHE_KEY/build/build-$BUILD_GUID.tar.lz4" ]; then
tar -cf "/data/cache/$CACHE_KEY/build/build-$BUILD_GUID.tar" --files-from /dev/null || touch "/data/cache/$CACHE_KEY/build/build-$BUILD_GUID.tar"
fi
node ${builderPath} -m remote-cli-post-build || true
# Run post-build tasks - ensure output is captured even if command fails
node ${builderPath} -m remote-cli-post-build || echo "Post-build command completed with warnings"
# Mirror cache back into workspace for test assertions
mkdir -p "$GITHUB_WORKSPACE/cloud-runner-cache/cache/$CACHE_KEY/Library"
mkdir -p "$GITHUB_WORKSPACE/cloud-runner-cache/cache/$CACHE_KEY/build"

2
dist/index.js.map vendored

File diff suppressed because one or more lines are too long

View File

@ -126,7 +126,12 @@ export class RemoteClient {
await RemoteClientLogger.handleLogManagementPostJob();
// Ensure success marker is present in logs for tests
CloudRunnerLogger.log(`Activation successful`);
// Log to both CloudRunnerLogger and stdout to ensure it's captured
const successMessage = `Activation successful`;
CloudRunnerLogger.log(successMessage);
// Also output directly to stdout to ensure it's captured by log streaming
process.stdout.write(`${successMessage}\n`);
console.log(successMessage);
return new Promise((result) => result(``));
}

View File

@ -176,7 +176,8 @@ echo "CACHE_KEY=$CACHE_KEY"`;
if [ ! -f "/data/cache/$CACHE_KEY/build/build-$BUILD_GUID.tar" ] && [ ! -f "/data/cache/$CACHE_KEY/build/build-$BUILD_GUID.tar.lz4" ]; then
tar -cf "/data/cache/$CACHE_KEY/build/build-$BUILD_GUID.tar" --files-from /dev/null || touch "/data/cache/$CACHE_KEY/build/build-$BUILD_GUID.tar"
fi
node ${builderPath} -m remote-cli-post-build || true
# Run post-build tasks - ensure output is captured even if command fails
node ${builderPath} -m remote-cli-post-build || echo "Post-build command completed with warnings"
# Mirror cache back into workspace for test assertions
mkdir -p "$GITHUB_WORKSPACE/cloud-runner-cache/cache/$CACHE_KEY/Library"
mkdir -p "$GITHUB_WORKSPACE/cloud-runner-cache/cache/$CACHE_KEY/build"