Do not continue on error and error handling improvements
parent
aa2579095b
commit
a2d1e8ac4f
|
|
@ -18,7 +18,6 @@ jobs:
|
|||
k8sBuilds:
|
||||
name: K8s build for ${{ matrix.targetPlatform }} on version ${{ matrix.unityVersion }}
|
||||
runs-on: ubuntu-latest
|
||||
continue-on-error: true
|
||||
strategy:
|
||||
fail-fast: false
|
||||
matrix:
|
||||
|
|
|
|||
|
|
@ -1079,18 +1079,18 @@ class Kubernetes {
|
|||
core.info(chunk.toString());
|
||||
next();
|
||||
};
|
||||
try {
|
||||
yield new Promise((resolve) => new client_node_1.Log(this.kubeConfig).log(this.namespace, this.podName, this.containerName, stream, resolve, {
|
||||
const logOptions = {
|
||||
follow: true,
|
||||
pretty: true,
|
||||
previous: true,
|
||||
}));
|
||||
core.info('end of log stream');
|
||||
};
|
||||
try {
|
||||
yield new Promise((resolve) => new client_node_1.Log(this.kubeConfig).log(this.namespace, this.podName, this.containerName, stream, resolve, logOptions));
|
||||
}
|
||||
catch (error) {
|
||||
core.error(JSON.stringify(error, undefined, 4));
|
||||
throw error;
|
||||
}
|
||||
core.info('end of log stream');
|
||||
});
|
||||
}
|
||||
cleanup() {
|
||||
|
|
|
|||
File diff suppressed because one or more lines are too long
|
|
@ -366,19 +366,19 @@ class Kubernetes implements RemoteBuilderProviderInterface {
|
|||
core.info(chunk.toString());
|
||||
next();
|
||||
};
|
||||
try {
|
||||
await new Promise((resolve) =>
|
||||
new Log(this.kubeConfig).log(this.namespace, this.podName, this.containerName, stream, resolve, {
|
||||
const logOptions = {
|
||||
follow: true,
|
||||
pretty: true,
|
||||
previous: true,
|
||||
}),
|
||||
};
|
||||
try {
|
||||
await new Promise((resolve) =>
|
||||
new Log(this.kubeConfig).log(this.namespace, this.podName, this.containerName, stream, resolve, logOptions),
|
||||
);
|
||||
core.info('end of log stream');
|
||||
} catch (error) {
|
||||
core.error(JSON.stringify(error, undefined, 4));
|
||||
throw error;
|
||||
}
|
||||
core.info('end of log stream');
|
||||
}
|
||||
|
||||
async cleanup() {
|
||||
|
|
|
|||
Loading…
Reference in New Issue