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

(general): hundreds of nested cdk.out/asset directories in %temp% broke Windows #11730

Closed
beatyt opened this issue Nov 26, 2020 · 5 comments
Closed
Assignees
Labels
closed-for-staleness This issue was automatically closed because it hadn't received any attention in a while. guidance Question that needs advice or information. os/windows Related specifically to Windows behavior response-requested Waiting on additional info and feedback. Will move to "closing-soon" in 7 days.

Comments

@beatyt
Copy link

beatyt commented Nov 26, 2020

❓ General Issue

The Question

Original problem: Windows machine after a restart was hanging at "Please wait for the User Profile Service" [1]. I let it run over night and had no success.

After some investigating, I tried to delete my user %TEMP% directory. The only thing that failed to delete was a CDK.out asset directory. I used robocopy to delete the many nested directories. [2] [3] [4]

Resolution: After 30 minutes of running a command, all of the nested directories were deleted and I was able to restart my machine.

It should be noted I've also been using VSCode as my IDE. It could be contributing or causing the issue as well?

Environment

  • CDK CLI Version: 1.74.0 (build e86602f)
  • Module Version: n/a
  • Node.js Version: v12.13.0
  • OS: Windows 10 Version 1909 (OS Build 18363.959)
  • Language (Version): TypeScript 4.0.3

Other information

(#1) The original problem and error message.
image
(#2) Many nested cdk.out asset directories
image
(#3) Running the delete
image
(#4) The result of deletion - robocopy newTempFolder Temp /purge was the command I ran to delete everything in Temp
image

Thank you and please let me know if anything needs clarifying or more info. I can share the CDK project I was playing with if that would help.

@beatyt beatyt added guidance Question that needs advice or information. needs-triage This issue or PR still needs to be triaged. labels Nov 26, 2020
@SomayaB SomayaB added the os/windows Related specifically to Windows behavior label Nov 30, 2020
@NetaNir
Copy link
Contributor

NetaNir commented Nov 30, 2020

Hi @beatyt!

Sorry about that!

To help us debug this, can you share if this happen after upgrading your CDK version? If so, what version were you upgrading from?

Thanks!

@NGL321 NGL321 added investigating This issue is being investigated and/or work is in progress to resolve the issue. p0 and removed needs-triage This issue or PR still needs to be triaged. labels Nov 30, 2020
@NGL321
Copy link
Contributor

NGL321 commented Nov 30, 2020

Hey @beatyt,

Thanks for reporting this, and sorry you ran into it! It would be super helpful to get a little more information on this so we can better investigate and try to reproduce.

The following info would be great:

  • Minimum sample code for reproduction (what were you actually building)
  • As @NetaNir mentioned, if the code worked with a different version, which version upgrade caused the problem?

That should give us enough to start looking into what caused this 😄

😸 😷

@NGL321 NGL321 added response-requested Waiting on additional info and feedback. Will move to "closing-soon" in 7 days. and removed p0 labels Nov 30, 2020
@beatyt
Copy link
Author

beatyt commented Nov 30, 2020

I've just been playing with CDK off and on. Not really a previous version that I can say worked.

I have a mix of versions: https://github.com/beatyt/typescript-web-api-with-aws-cdk/blob/master/infrastructure/package.json#L34 so I should probably fix those and upgrade to latest.

@aws-cdk/core & aws-cdk is 1.67.0

Sample project: https://github.com/beatyt/typescript-web-api-with-aws-cdk/tree/master/infrastructure

The project is broken into an ./application and ./infrastructure directory, the CDK stuff is in ./infrastructure.

@github-actions github-actions bot removed the response-requested Waiting on additional info and feedback. Will move to "closing-soon" in 7 days. label Dec 1, 2020
@NGL321
Copy link
Contributor

NGL321 commented Dec 21, 2020

@beatyt,

After reviewing the CDK code it all looks kosher, I dont see a reason the code itself would cause this; especially since you don't seem to have any assets to deploy.

HOWEVER, I am surprised you are able to get the code to synth at all since we have a requirement that all packages in the project share a version (having different versions should cause synth-time exceptions). So with that said, my suggestion is to first update all the packages to the same version (latest or otherwise), and see if that fixes the problem. If not, I can keep digging.

😸 😷

@NGL321 NGL321 added response-requested Waiting on additional info and feedback. Will move to "closing-soon" in 7 days. and removed investigating This issue is being investigated and/or work is in progress to resolve the issue. labels Dec 21, 2020
@github-actions
Copy link

This issue has not received a response in a while. If you want to keep this issue open, please leave a comment below and auto-close will be canceled.

@github-actions github-actions bot added closing-soon This issue will automatically close in 4 days unless further comments are made. closed-for-staleness This issue was automatically closed because it hadn't received any attention in a while. and removed closing-soon This issue will automatically close in 4 days unless further comments are made. labels Dec 29, 2020
@github-actions github-actions bot closed this as completed Jan 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
closed-for-staleness This issue was automatically closed because it hadn't received any attention in a while. guidance Question that needs advice or information. os/windows Related specifically to Windows behavior response-requested Waiting on additional info and feedback. Will move to "closing-soon" in 7 days.
Projects
None yet
Development

No branches or pull requests

4 participants