Skip to content

Commit e85231c

Browse files
authored
feat(codepipeline): change default pipeline type to V2 (under feature flag) (#29096)
### Reason for this change We discussed about the default pipeline type, it was concluded that the new type (V2) should be the default. #28538 (comment) https://github.com/aws/aws-cdk/blob/main/packages/aws-cdk-lib/aws-codepipeline/lib/pipeline.ts#L492 ### Description of changes Change the default value using a feature flag so as not to affect existing processing. ### Description of how you validated changes Bot unit and integ tests ### Checklist - [x] My code adheres to the [CONTRIBUTING GUIDE](https://github.com/aws/aws-cdk/blob/main/CONTRIBUTING.md) and [DESIGN GUIDELINES](https://github.com/aws/aws-cdk/blob/main/docs/DESIGN_GUIDELINES.md) ---- *By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
1 parent a1fbd51 commit e85231c

File tree

63 files changed

+2284
-63
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

63 files changed

+2284
-63
lines changed

packages/@aws-cdk-testing/framework-integ/test/aws-appconfig/test/integ.configuration.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,11 @@ def handler(event, context):
3232
print('This is my dummy validator')
3333
`;
3434

35-
const app = new App();
35+
const app = new App({
36+
postCliContext: {
37+
'@aws-cdk/aws-codepipeline:defaultPipelineTypeToV2': false,
38+
},
39+
});
3640

3741
const stack = new Stack(app, 'aws-appconfig-configuration');
3842

packages/@aws-cdk-testing/framework-integ/test/aws-codepipeline-actions/test/cloudformation/integ.stacksets.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,11 @@ export class StackSetPipelineStack extends Stack {
7676
}
7777
}
7878

79-
const app = new App();
79+
const app = new App({
80+
postCliContext: {
81+
'@aws-cdk/aws-codepipeline:defaultPipelineTypeToV2': false,
82+
},
83+
});
8084
const stack = new StackSetPipelineStack(app, 'StackSetPipelineStack');
8185
new IntegTest(app, 'StackSetPipelineStackInteg', {
8286
testCases: [stack],

packages/@aws-cdk-testing/framework-integ/test/aws-codepipeline-actions/test/integ.cfn-template-from-repo.lit.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,11 @@ import * as codepipeline from 'aws-cdk-lib/aws-codepipeline';
33
import * as cdk from 'aws-cdk-lib';
44
import * as cpactions from 'aws-cdk-lib/aws-codepipeline-actions';
55

6-
const app = new cdk.App();
6+
const app = new cdk.App({
7+
postCliContext: {
8+
'@aws-cdk/aws-codepipeline:defaultPipelineTypeToV2': false,
9+
},
10+
});
711
const stack = new cdk.Stack(app, 'aws-cdk-codepipeline-cloudformation');
812

913
/// !show

packages/@aws-cdk-testing/framework-integ/test/aws-codepipeline-actions/test/integ.lambda-deployed-through-codepipeline.lit.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,11 @@ import * as cdk from 'aws-cdk-lib';
77
import * as codepipeline_actions from 'aws-cdk-lib/aws-codepipeline-actions';
88
import { STANDARD_NODEJS_RUNTIME } from '../../config';
99

10-
const app = new cdk.App();
10+
const app = new cdk.App({
11+
postCliContext: {
12+
'@aws-cdk/aws-codepipeline:defaultPipelineTypeToV2': false,
13+
},
14+
});
1115

1216
/// !show
1317
const lambdaStack = new cdk.Stack(app, 'LambdaStack');

packages/@aws-cdk-testing/framework-integ/test/aws-codepipeline-actions/test/integ.lambda-pipeline.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,11 @@ import * as cdk from 'aws-cdk-lib';
66
import * as cpactions from 'aws-cdk-lib/aws-codepipeline-actions';
77
import { STANDARD_NODEJS_RUNTIME } from '../../config';
88

9-
const app = new cdk.App();
9+
const app = new cdk.App({
10+
postCliContext: {
11+
'@aws-cdk/aws-codepipeline:defaultPipelineTypeToV2': false,
12+
},
13+
});
1014

1115
const stack = new cdk.Stack(app, 'aws-cdk-codepipeline-lambda');
1216

packages/@aws-cdk-testing/framework-integ/test/aws-codepipeline-actions/test/integ.pipeline-alexa-deploy.ts

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,11 @@ import * as s3 from 'aws-cdk-lib/aws-s3';
33
import { App, RemovalPolicy, SecretValue, Stack } from 'aws-cdk-lib';
44
import * as cpactions from 'aws-cdk-lib/aws-codepipeline-actions';
55

6-
const app = new App();
7-
6+
const app = new App({
7+
postCliContext: {
8+
'@aws-cdk/aws-codepipeline:defaultPipelineTypeToV2': false,
9+
},
10+
});
811
const stack = new Stack(app, 'aws-cdk-codepipeline-alexa-deploy');
912

1013
const bucket = new s3.Bucket(stack, 'PipelineBucket', {

packages/@aws-cdk-testing/framework-integ/test/aws-codepipeline-actions/test/integ.pipeline-cfn-cross-region.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,11 @@ import * as s3 from 'aws-cdk-lib/aws-s3';
33
import * as cdk from 'aws-cdk-lib';
44
import * as cpactions from 'aws-cdk-lib/aws-codepipeline-actions';
55

6-
const app = new cdk.App();
6+
const app = new cdk.App({
7+
postCliContext: {
8+
'@aws-cdk/aws-codepipeline:defaultPipelineTypeToV2': false,
9+
},
10+
});
711

812
const region = 'us-west-2'; // hardcode the region
913
const stack = new cdk.Stack(app, 'aws-cdk-codepipeline-cloudformation-cross-region', {

packages/@aws-cdk-testing/framework-integ/test/aws-codepipeline-actions/test/integ.pipeline-cfn-with-action-role.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,11 @@ import * as s3 from 'aws-cdk-lib/aws-s3';
44
import * as cdk from 'aws-cdk-lib';
55
import * as cpactions from 'aws-cdk-lib/aws-codepipeline-actions';
66

7-
const app = new cdk.App();
7+
const app = new cdk.App({
8+
postCliContext: {
9+
'@aws-cdk/aws-codepipeline:defaultPipelineTypeToV2': false,
10+
},
11+
});
812

913
const stack = new cdk.Stack(app, 'aws-cdk-codepipeline-cloudformation-cross-region-with-action-role', {});
1014

packages/@aws-cdk-testing/framework-integ/test/aws-codepipeline-actions/test/integ.pipeline-cfn.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,11 @@ import * as s3 from 'aws-cdk-lib/aws-s3';
44
import * as cdk from 'aws-cdk-lib';
55
import * as cpactions from 'aws-cdk-lib/aws-codepipeline-actions';
66

7-
const app = new cdk.App();
7+
const app = new cdk.App({
8+
postCliContext: {
9+
'@aws-cdk/aws-codepipeline:defaultPipelineTypeToV2': false,
10+
},
11+
});
812

913
const stack = new cdk.Stack(app, 'aws-cdk-codepipeline-cloudformation');
1014

packages/@aws-cdk-testing/framework-integ/test/aws-codepipeline-actions/test/integ.pipeline-code-build-batch.ts

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,11 @@ import * as s3 from 'aws-cdk-lib/aws-s3';
55
import * as cdk from 'aws-cdk-lib';
66
import * as cpactions from 'aws-cdk-lib/aws-codepipeline-actions';
77

8-
const app = new cdk.App();
8+
const app = new cdk.App({
9+
postCliContext: {
10+
'@aws-cdk/aws-codepipeline:defaultPipelineTypeToV2': false,
11+
},
12+
});
913

1014
const stack = new cdk.Stack(app, 'aws-cdk-codepipeline-codebuild-batch');
1115

0 commit comments

Comments
 (0)