Added podGC on revert-chaos (#2849)

Signed-off-by: Amit Kumar Das <amit@chaosnative.com>
This commit is contained in:
Amit Kumar Das 2021-06-03 17:45:21 +05:30 committed by GitHub
parent a12d5ce067
commit c77e5d24fe
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 13 additions and 2 deletions

View File

@ -123,6 +123,9 @@ const WorkflowTable = forwardRef(
// Else if Revert Chaos is set to true and it is not already set in the manifest // Else if Revert Chaos is set to true and it is not already set in the manifest
// For Workflows // For Workflows
if (revertChaos && parsedYAML.kind === 'Workflow') { if (revertChaos && parsedYAML.kind === 'Workflow') {
parsedYAML.spec.podGC = {
strategy: 'OnWorkflowCompletion',
};
parsedYAML.spec.templates[0].steps.push([ parsedYAML.spec.templates[0].steps.push([
{ {
name: 'revert-chaos', name: 'revert-chaos',
@ -153,6 +156,9 @@ const WorkflowTable = forwardRef(
// Else if Revert Chaos is set to True and it is not already set in the manifest // Else if Revert Chaos is set to True and it is not already set in the manifest
// For Cron Workflow // For Cron Workflow
else if (revertChaos && parsedYAML.kind === 'CronWorkflow') { else if (revertChaos && parsedYAML.kind === 'CronWorkflow') {
parsedYAML.spec.workflowSpec.podGC = {
strategy: 'OnWorkflowCompletion',
};
parsedYAML.spec.workflowSpec.templates[0].steps.push([ parsedYAML.spec.workflowSpec.templates[0].steps.push([
{ {
name: 'revert-chaos', name: 'revert-chaos',

View File

@ -534,8 +534,11 @@ const TuneWorkflow = forwardRef((_, ref) => {
parsedManifest.kind === 'Workflow' && parsedManifest.kind === 'Workflow' &&
parsedManifest.spec.templates[0].steps[ parsedManifest.spec.templates[0].steps[
parsedManifest.spec.templates[0].steps.length - 1 parsedManifest.spec.templates[0].steps.length - 1
][0].name === 'revert-chaos' ][0].name === 'revert-chaos' &&
parsedManifest.spec.podGC
) { ) {
delete parsedManifest.spec.podGC;
parsedManifest.spec.templates[0].steps.pop(); // Remove the last step -> Revert Chaos parsedManifest.spec.templates[0].steps.pop(); // Remove the last step -> Revert Chaos
parsedManifest.spec.templates.pop(); // Remove the last template -> Revert Chaos Template parsedManifest.spec.templates.pop(); // Remove the last template -> Revert Chaos Template
@ -547,8 +550,10 @@ const TuneWorkflow = forwardRef((_, ref) => {
parsedManifest.kind === 'CronWorkflow' && parsedManifest.kind === 'CronWorkflow' &&
parsedManifest.spec.workflowSpec.templates[0].steps[ parsedManifest.spec.workflowSpec.templates[0].steps[
parsedManifest.spec.workflowSpec.templates[0].steps.length - 1 parsedManifest.spec.workflowSpec.templates[0].steps.length - 1
][0].name === 'revert-chaos' ][0].name === 'revert-chaos' &&
parsedManifest.spec.workflowSpec.podGC
) { ) {
delete parsedManifest.workflowSpec.spec.podGC;
parsedManifest.spec.workflowSpec.templates[0].steps.pop(); // Remove the last step -> Revert Chaos parsedManifest.spec.workflowSpec.templates[0].steps.pop(); // Remove the last step -> Revert Chaos
parsedManifest.spec.workflowSpec.templates.pop(); // Remove the last template -> Revert Chaos Template parsedManifest.spec.workflowSpec.templates.pop(); // Remove the last template -> Revert Chaos Template