Skip to content

Conversation

@tarunb12
Copy link
Contributor

@tarunb12 tarunb12 commented Nov 22, 2025

Issue

aws/aws-cdk-rfcs#789

Reason for this change

This change adds a new alpha module for EC2 Image Builder L2 Constructs (@aws-cdk/aws-imagebuilder-alpha), as outlined in aws/aws-cdk-rfcs#789. This PR specifically implements the ImagePipeline construct.

Description of changes

This change implements the ImagePipeline construct, which is a higher-level construct of CfnImagePipeline.

Example

const imagePipeline = new imagebuilder.ImagePipeline(this, 'ImagePipeline-AMI', {
  imagePipelineName: 'test-image-pipeline',
  description: 'this is an image pipeline description.',
  recipe: imageRecipe,
  infrastructureConfiguration,
  distributionConfiguration: amiDistributionConfiguration,
  enabled: true,
  executionRole,
  schedule: {
    expression: events.Schedule.expression('cron(0 7 ? * mon *)'),
    timezone: cdk.TimeZone.PST8PDT,
    startCondition: imagebuilder.ScheduleStartCondition.EXPRESSION_MATCH_AND_DEPENDENCY_UPDATES_AVAILABLE,
    autoDisableFailureCount: 5,
  },
  workflows: [{ workflow: imagebuilder.AwsManagedWorkflow.buildImage(this, 'BuildImage') }],
  imageLogGroup,
  imagePipelineLogGroup,
  enhancedImageMetadataEnabled: true,
  imageTestsEnabled: false,
  imageScanningEnabled: false,
});

Describe any new or updated permissions being added

N/A - new L2 construct in alpha module

Description of how you validated changes

Validated with unit tests and integration tests. Manually verified generated CFN templates as well.

Checklist


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license

@aws-cdk-automation aws-cdk-automation requested a review from a team November 22, 2025 09:15
@github-actions github-actions bot added valued-contributor [Pilot] contributed between 6-12 PRs to the CDK p2 labels Nov 22, 2025
@aws-cdk-automation aws-cdk-automation added the pr/needs-further-review PR requires additional review from our team specialists due to the scope or complexity of changes. label Nov 22, 2025
@tarunb12 tarunb12 marked this pull request as ready for review November 22, 2025 09:44
@kumsmrit kumsmrit self-assigned this Nov 24, 2025
@tarunb12 tarunb12 force-pushed the imagebuilder-image-pipeline branch from e0e20dd to eb8afb7 Compare November 24, 2025 18:25
@mergify mergify bot dismissed kumsmrit’s stale review November 24, 2025 18:26

Pull request has been modified.

@tarunb12 tarunb12 force-pushed the imagebuilder-image-pipeline branch 3 times, most recently from cdecf33 to 6087d42 Compare November 25, 2025 04:03
@mergify mergify bot dismissed kumsmrit’s stale review November 25, 2025 08:23

Pull request has been modified.

@tarunb12 tarunb12 force-pushed the imagebuilder-image-pipeline branch from 6ed8b8f to f0a2266 Compare November 25, 2025 12:05
@mergify mergify bot dismissed kumsmrit’s stale review November 25, 2025 12:05

Pull request has been modified.

@mergify mergify bot dismissed kumsmrit’s stale review November 25, 2025 12:57

Pull request has been modified.

@tarunb12 tarunb12 force-pushed the imagebuilder-image-pipeline branch from b2cd86b to a890ee4 Compare November 25, 2025 13:01
@kumsmrit kumsmrit added the pr/request-cli-integ-tests Request CLI integ tests to be run. You will need to review the code and approve the deployment. label Nov 25, 2025
@tarunb12 tarunb12 force-pushed the imagebuilder-image-pipeline branch from a890ee4 to 6e987ca Compare November 25, 2025 16:42
@mergify mergify bot dismissed kumsmrit’s stale review November 25, 2025 16:42

Pull request has been modified.

@tarunb12 tarunb12 force-pushed the imagebuilder-image-pipeline branch from 6e987ca to ca4e570 Compare November 25, 2025 17:13
@tarunb12 tarunb12 force-pushed the imagebuilder-image-pipeline branch from ca4e570 to 4ccb839 Compare November 25, 2025 17:15
}),
];

const hasProps = props !== undefined;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we extract these conditional permissions to separate helpers? We can follow up with this in separate PR as well

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can address in follow up PRs

@kumsmrit kumsmrit added pr/request-cli-integ-tests Request CLI integ tests to be run. You will need to review the code and approve the deployment. and removed pr/request-cli-integ-tests Request CLI integ tests to be run. You will need to review the code and approve the deployment. labels Nov 25, 2025
@kumsmrit kumsmrit added pr/request-cli-integ-tests Request CLI integ tests to be run. You will need to review the code and approve the deployment. and removed pr/request-cli-integ-tests Request CLI integ tests to be run. You will need to review the code and approve the deployment. labels Nov 25, 2025
@aws-cdk-automation
Copy link
Collaborator

➡️ PR build request submitted to test-main-pipeline ⬅️

A maintainer must now check the pipeline and add the pr-linter/cli-integ-tested label once the pipeline succeeds.

@mergify
Copy link
Contributor

mergify bot commented Nov 26, 2025

Merge Queue Status Beta

✅ The pull request has been merged

This pull request spent 31 minutes 7 seconds in the queue, including 30 minutes 58 seconds waiting for CI.
The checks were run in-place.

Required conditions to merge

@mergify
Copy link
Contributor

mergify bot commented Nov 26, 2025

Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

@mergify
Copy link
Contributor

mergify bot commented Nov 26, 2025

Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork).

@mergify mergify bot merged commit d8c324a into aws:main Nov 26, 2025
17 of 18 checks passed
@github-actions
Copy link
Contributor

Comments on closed issues and PRs are hard for our team to see.
If you need help, please open a new issue that references this one.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Nov 26, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

p2 pr/needs-further-review PR requires additional review from our team specialists due to the scope or complexity of changes. pr/request-cli-integ-tests Request CLI integ tests to be run. You will need to review the code and approve the deployment. valued-contributor [Pilot] contributed between 6-12 PRs to the CDK

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants