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

feat(assets): exclude "cdk.out" from docker assets #14842

Closed

Conversation

AlexGue
Copy link

@AlexGue AlexGue commented May 23, 2021

This PR is intended to cover a small bug that creates an infinite loop generating folders in cdk.out if the project root path is specified as the build directory.

With this fix 'cdk.out' folder will be added as glob exclude pattern no matters what.

More details in issue #14841

Closes #14841


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

@gitpod-io
Copy link

gitpod-io bot commented May 23, 2021

eladb
eladb previously requested changes May 24, 2021
Copy link
Contributor

@eladb eladb left a comment

Choose a reason for hiding this comment

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

Please add unit test

packages/@aws-cdk/aws-ecr-assets/lib/image-asset.ts Outdated Show resolved Hide resolved
@mergify mergify bot dismissed eladb’s stale review May 27, 2021 00:28

Pull request has been modified.

@peterwoodworth peterwoodworth added the @aws-cdk/aws-ecr-assets Related to AWS CDK Docker Image Assets label Jun 3, 2021
@eladb
Copy link
Contributor

eladb commented Jun 30, 2021

Should this be done in a separate PR or in this one?

We can include it in this PR if you are inclined.

@eladb eladb self-requested a review July 1, 2021 11:32
@eladb
Copy link
Contributor

eladb commented Jul 4, 2021

@AlexGue are you interested in continuing to work on this PR or should we close for now?

@AlexGue
Copy link
Author

AlexGue commented Jul 6, 2021

@AlexGue are you interested in continuing to work on this PR or should we close for now?

Yes, hopefully I have some time during this week to solve it.

@mergify mergify bot dismissed eladb’s stale review July 11, 2021 22:08

Pull request has been modified.

@eladb
Copy link
Contributor

eladb commented Jul 20, 2021

Hey @AlexGue thanks for following up on this. I wanted to make sure you are aware that you can run npx cdk-integ --dry-run to update integ test snapshots.

Let me know if you need help to get this across the finish line.

@AlexGue
Copy link
Author

AlexGue commented Jul 20, 2021

Hey @AlexGue thanks for following up on this. I wanted to make sure you are aware that you can run npx cdk-integ --dry-run to update integ test snapshots.

Let me know if you need help to get this across the finish line.

Thanks for the comment, I was unaware. I tried to run the command to update the assets regarding aws-lambda-python package, but it seems to fail. (I am doing this directly on Gitpod):

image

Maybe Gitpod is not able to build docker images, or am I missing something?

@jogold
Copy link
Contributor

jogold commented Jul 20, 2021

@AlexGue not sure you can mount volumes in Gitpod... integrate this commit jogold@f3140f8 and you should be good.

@aws-cdk-automation
Copy link
Collaborator

AWS CodeBuild CI Report

  • CodeBuild project: AutoBuildProject89A8053A-LhjRyN9kxr8o
  • Commit ID: a19fdfd
  • Result: FAILED
  • Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

@eladb
Copy link
Contributor

eladb commented Aug 10, 2021

@AlexGue how are we doing here? Need help?

@AlexGue
Copy link
Author

AlexGue commented Aug 15, 2021

@eladb seems that @jogold have it finished on #16034
Thanks to both of you for the help with the PR.

@eladb
Copy link
Contributor

eladb commented Aug 15, 2021

Superseded by #16034

@eladb eladb closed this Aug 15, 2021
mergify bot pushed a commit that referenced this pull request Aug 15, 2021
This PR is intended to cover a small bug that creates an infinite loop generating folders in cdk.out if the project root path is specified as the build directory.

With this fix 'cdk.out' folder will be added as glob exclude pattern no matters what.

More details in issue #14841

Original PR from @AlexGue

Closes #14841
Closes #14842



----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
hollanddd pushed a commit to hollanddd/aws-cdk that referenced this pull request Aug 26, 2021
This PR is intended to cover a small bug that creates an infinite loop generating folders in cdk.out if the project root path is specified as the build directory.

With this fix 'cdk.out' folder will be added as glob exclude pattern no matters what.

More details in issue aws#14841

Original PR from @AlexGue

Closes aws#14841
Closes aws#14842



----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
TikiTDO pushed a commit to TikiTDO/aws-cdk that referenced this pull request Sep 6, 2021
This PR is intended to cover a small bug that creates an infinite loop generating folders in cdk.out if the project root path is specified as the build directory.

With this fix 'cdk.out' folder will be added as glob exclude pattern no matters what.

More details in issue aws#14841

Original PR from @AlexGue

Closes aws#14841
Closes aws#14842



----

*By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license*
david-doyle-as24 pushed a commit to david-doyle-as24/aws-cdk that referenced this pull request Sep 7, 2021
This PR is intended to cover a small bug that creates an infinite loop generating folders in cdk.out if the project root path is specified as the build directory.

With this fix 'cdk.out' folder will be added as glob exclude pattern no matters what.

More details in issue aws#14841

Original PR from @AlexGue

Closes aws#14841
Closes aws#14842



----

*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 join this conversation on GitHub. Already have an account? Sign in to comment
Labels
@aws-cdk/aws-ecr-assets Related to AWS CDK Docker Image Assets
Projects
None yet
Development

Successfully merging this pull request may close these issues.

(aws-ecr-assets): Infinite folder creation when cdk.out is not excluded
5 participants