Revert "fix: Allow concurrent validation_test steps"

This reverts commit c25c111. This was a minor speedup and we don't need it since we can easily cancel validation test steps if necessary.
saymedia · Aug 18, 2023 · fb9f50c · fb9f50c
1 parent c25c111
commit fb9f50c
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 0 deletions.
diff --git a/pipeline.go b/pipeline.go
@@ -132,6 +132,7 @@ func (p *Pipeline) Lower(context *Context) ([]interface{}, error) {
 							EnvironmentName:    envName,
 							QueueName:          "validation_test",
 							EmojiName:          "curly_loop",
+							PreventConcurrency: true,
 						}
 						loweredSteps, err := lowerSteps(
 							p.ValidationTest, context, stepContext,
@@ -156,6 +157,7 @@ func (p *Pipeline) Lower(context *Context) ([]interface{}, error) {
 					EnvironmentName:    envName,
 					QueueName:          "validation_test",
 					EmojiName:          "curly_loop",
+					PreventConcurrency: true,
 				}
 
 				// Manual deploys run sequentially, so that they can

diff --git a/testdata/basic_master.out.yaml b/testdata/basic_master.out.yaml
@@ -59,6 +59,9 @@ steps:
     environment: dev
     queue: validation_test
   command: integrationtest
+  concurrency: 1
+  concurrency_group: dev/myrepo
+  concurrency_method: eager
   env:
     JOBSWORTH_CAUTIOUS: "0"
     JOBSWORTH_CODE_VERSION: ""
@@ -102,6 +105,9 @@ steps:
     environment: prod
     queue: validation_test
   command: integrationtest
+  concurrency: 1
+  concurrency_group: prod/myrepo
+  concurrency_method: eager
   env:
     JOBSWORTH_CAUTIOUS: "0"
     JOBSWORTH_CODE_VERSION: ""