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

fs.copyFileSync hangs for file created within same layer in overlay filesystem #1536

Closed
ericsciple opened this issue Dec 6, 2021 · 7 comments · Fixed by #1651
Closed

fs.copyFileSync hangs for file created within same layer in overlay filesystem #1536

ericsciple opened this issue Dec 6, 2021 · 7 comments · Fixed by #1651
Assignees
Labels
bug Something isn't working

Comments

@ericsciple
Copy link
Collaborator

ericsciple commented Dec 6, 2021

Since upgrading node 12, we are seeing fs.copyFileSync hang during some container scenarios.

We should revert the node 12 update and continue the investigation. It's worth noting node 16 has the same issue.

Related issues:

@ericsciple ericsciple added the bug Something isn't working label Dec 6, 2021
@ericsciple ericsciple self-assigned this Dec 6, 2021
@crazy-max
Copy link

Thanks @ericsciple!

@ericsciple
Copy link
Collaborator Author

Two immediate action items:

  • Revert to older version of node 12
  • Create simplest possible repro we can provide for nodejs

@ericsciple
Copy link
Collaborator Author

fyi - the new runner finished rolling out to all accounts. We reverted back to the previous version of node12.

@ericsciple
Copy link
Collaborator Author

ericsciple commented Dec 8, 2021

OK here's the simplest repro that @ethomson and I could come up with:

Key details about the repro:

  • Reproduces on AKS and EKS (not minikube and not docker)
  • Uses copy_file_range to perform the copy
  • Copy a file from non-overlay file system to overlay file system, then from overlay file system to another location on the overlay file system (always copies 0 bytes)

@ericsciple
Copy link
Collaborator Author

Opened an AKS issue here

@ethomson
Copy link
Contributor

The most recent EKS AMIs (I'm using 1.21.5-20220123) are running kernel 5.4.172-90.336.amzn2 which has reverted this overlayfs behavior change.

In other words, fs.fileCopySync() now works properly on EKS.

@thboop thboop assigned thboop and unassigned ericsciple Jan 27, 2022
@thboop
Copy link
Collaborator

thboop commented Feb 2, 2022

@ericsciple and I went through this and it appears to be fixed in AKS as well, and it is not reproducible on ubuntu-latest, so we should be good to re-update node12 and close this issue

thboop added a commit that referenced this issue Feb 17, 2022
#1651)

* Revert "revert node12 version due to fs.copyFileSync hang #1536 (#1537)"

bef164a

* check hashs before tests because tests rely on right values + update hashes

* fix tests

* use hc trace
fhammerl pushed a commit that referenced this issue Feb 24, 2022
#1651)

* Revert "revert node12 version due to fs.copyFileSync hang #1536 (#1537)"

bef164a

* check hashs before tests because tests rely on right values + update hashes

* fix tests

* use hc trace
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants