K8s log and stringify error for k8s cleanup
parent
a0eda4ad1d
commit
026f8f20a4
|
|
@ -2849,7 +2849,6 @@ var __importDefault = (this && this.__importDefault) || function (mod) {
|
|||
};
|
||||
Object.defineProperty(exports, "__esModule", ({ value: true }));
|
||||
const k8s = __importStar(__nccwpck_require__(89679));
|
||||
const __1 = __nccwpck_require__(41359);
|
||||
const core = __importStar(__nccwpck_require__(42186));
|
||||
const kubernetes_storage_1 = __importDefault(__nccwpck_require__(49043));
|
||||
const kubernetes_task_runner_1 = __importDefault(__nccwpck_require__(70801));
|
||||
|
|
@ -2971,7 +2970,7 @@ class Kubernetes {
|
|||
yield new Promise((promise) => setTimeout(promise, 5000));
|
||||
}
|
||||
catch (error) {
|
||||
if (error.response.body.message.endsWith(`not found`)) {
|
||||
if (error.response.body.reason === `not found`) {
|
||||
return;
|
||||
}
|
||||
cloud_runner_logger_1.default.log('Failed to cleanup, error:');
|
||||
|
|
@ -2994,7 +2993,11 @@ class Kubernetes {
|
|||
catch (_a) { }
|
||||
});
|
||||
}
|
||||
cleanup(buildGuid, buildParameters,
|
||||
cleanup(
|
||||
// eslint-disable-next-line no-unused-vars
|
||||
buildGuid,
|
||||
// eslint-disable-next-line no-unused-vars
|
||||
buildParameters,
|
||||
// eslint-disable-next-line no-unused-vars
|
||||
branchName,
|
||||
// eslint-disable-next-line no-unused-vars
|
||||
|
|
@ -3005,12 +3008,11 @@ class Kubernetes {
|
|||
yield this.kubeClient.deleteNamespacedPersistentVolumeClaim(this.pvcName, this.namespace);
|
||||
}
|
||||
catch (error) {
|
||||
if (error.response.body.message.endsWith(`not found`)) {
|
||||
if (error.response.body.reason === `not found`) {
|
||||
return;
|
||||
}
|
||||
cloud_runner_logger_1.default.log(`Cleanup failed ${JSON.stringify(error, undefined, 4)}`);
|
||||
}
|
||||
yield __1.Output.setBuildVersion(buildParameters.buildVersion);
|
||||
});
|
||||
}
|
||||
static findPodFromJob(kubeClient, jobName, namespace) {
|
||||
|
|
|
|||
File diff suppressed because one or more lines are too long
|
|
@ -1,5 +1,5 @@
|
|||
import * as k8s from '@kubernetes/client-node';
|
||||
import { BuildParameters, Output } from '../../..';
|
||||
import { BuildParameters } from '../../..';
|
||||
import * as core from '@actions/core';
|
||||
import { ProviderInterface } from '../provider-interface';
|
||||
import CloudRunnerSecret from '../../services/cloud-runner-secret';
|
||||
|
|
@ -166,7 +166,7 @@ class Kubernetes implements ProviderInterface {
|
|||
await this.kubeClient.deleteNamespacedSecret(this.secretName, this.namespace);
|
||||
await new Promise((promise) => setTimeout(promise, 5000));
|
||||
} catch (error: any) {
|
||||
if (error.response.body.message.endsWith(`not found`)) {
|
||||
if (error.response.body.reason === `not found`) {
|
||||
return;
|
||||
}
|
||||
CloudRunnerLogger.log('Failed to cleanup, error:');
|
||||
|
|
@ -192,7 +192,9 @@ class Kubernetes implements ProviderInterface {
|
|||
}
|
||||
|
||||
async cleanup(
|
||||
// eslint-disable-next-line no-unused-vars
|
||||
buildGuid: string,
|
||||
// eslint-disable-next-line no-unused-vars
|
||||
buildParameters: BuildParameters,
|
||||
// eslint-disable-next-line no-unused-vars
|
||||
branchName: string,
|
||||
|
|
@ -204,12 +206,11 @@ class Kubernetes implements ProviderInterface {
|
|||
try {
|
||||
await this.kubeClient.deleteNamespacedPersistentVolumeClaim(this.pvcName, this.namespace);
|
||||
} catch (error: any) {
|
||||
if (error.response.body.message.endsWith(`not found`)) {
|
||||
if (error.response.body.reason === `not found`) {
|
||||
return;
|
||||
}
|
||||
CloudRunnerLogger.log(`Cleanup failed ${JSON.stringify(error, undefined, 4)}`);
|
||||
}
|
||||
await Output.setBuildVersion(buildParameters.buildVersion);
|
||||
}
|
||||
|
||||
static async findPodFromJob(kubeClient: CoreV1Api, jobName: string, namespace: string) {
|
||||
|
|
|
|||
Loading…
Reference in New Issue