Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

aws-codebuild: CodeBuild new environment types missing from CDK #32728

Closed
2 tasks
awszhen opened this issue Jan 3, 2025 · 1 comment · Fixed by #32729
Closed
2 tasks

aws-codebuild: CodeBuild new environment types missing from CDK #32728

awszhen opened this issue Jan 3, 2025 · 1 comment · Fixed by #32729
Labels
@aws-cdk/aws-codebuild Related to AWS CodeBuild feature-request A feature should be added or improved. needs-triage This issue or PR still needs to be triaged.

Comments

@awszhen
Copy link
Contributor

awszhen commented Jan 3, 2025

Describe the feature

Recently, CodeBuild released a new set of environment types which can be found here: LINUX_EC2, ARM_EC2 and WINDOWS_EC2. The CodeBuild CDK code should be updated to reflect these new environment types.

Use Case

Users can not create fleets with new environment types using CDK.

Proposed Solution

Add new environment types LINUX_EC2, ARM_EC2 and WINDOWS_EC2 into file.

Other Information

No response

Acknowledgements

  • I may be able to implement this feature request
  • This feature might incur a breaking change

CDK version used

Any

Environment details (OS name and version, etc.)

Any

@awszhen awszhen added feature-request A feature should be added or improved. needs-triage This issue or PR still needs to be triaged. labels Jan 3, 2025
@github-actions github-actions bot added the @aws-cdk/aws-codebuild Related to AWS CodeBuild label Jan 3, 2025
@mergify mergify bot closed this as completed in #32729 Jan 4, 2025
mergify bot pushed a commit that referenced this issue Jan 4, 2025
### Issue # (if applicable)

Closes #32728.

### Reason for this change

Recently, CodeBuild released a new set of environment types which can be found [here](https://docs.aws.amazon.com/codebuild/latest/APIReference/API_CreateFleet.html#CodeBuild-CreateFleet-request-environmentType): `LINUX_EC2`, `ARM_EC2` and `WINDOWS_EC2`. The CDK needs to be extended to support these new environment types.

### Description of changes

Added `LINUX_EC2`, `ARM_EC2` and `WINDOWS_EC2` values to [EnvironmentType](https://github.com/aws/aws-cdk/blob/main/packages/aws-cdk-lib/aws-codebuild/lib/environment-type.ts).

### Description of how you validated changes

Tests has been added. I ensured that the CDK was able to build, and that the output CloudFormation stack had the correct resources.

### 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*
Copy link

github-actions bot commented Jan 4, 2025

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 Jan 4, 2025
iankhou pushed a commit that referenced this issue Jan 13, 2025
### Issue # (if applicable)

Closes #32728.

### Reason for this change

Recently, CodeBuild released a new set of environment types which can be found [here](https://docs.aws.amazon.com/codebuild/latest/APIReference/API_CreateFleet.html#CodeBuild-CreateFleet-request-environmentType): `LINUX_EC2`, `ARM_EC2` and `WINDOWS_EC2`. The CDK needs to be extended to support these new environment types.

### Description of changes

Added `LINUX_EC2`, `ARM_EC2` and `WINDOWS_EC2` values to [EnvironmentType](https://github.com/aws/aws-cdk/blob/main/packages/aws-cdk-lib/aws-codebuild/lib/environment-type.ts).

### Description of how you validated changes

Tests has been added. I ensured that the CDK was able to build, and that the output CloudFormation stack had the correct resources.

### 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*
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
@aws-cdk/aws-codebuild Related to AWS CodeBuild feature-request A feature should be added or improved. needs-triage This issue or PR still needs to be triaged.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant