appsmith icon indicating copy to clipboard operation
appsmith copied to clipboard

JGitInternalException: Exception caught during execution of reset command. Cannot lock /data/git-storage/628e37967901344...

Open sentry[bot] opened this issue 3 years ago • 7 comments

Sentry Issue: APPSMITH-1SC

LockFailedException: Cannot lock /data/git-storage/628e37967901344ba8d296e3/629f4d18360bee1cd628fe46/suzuya-studio-rental/.git/index. Ensure that no other process has an open file handle on the lock file /data/git-storage/628e37967901344ba8d296e3/629f4d18360bee1cd628fe46/suzuya-studio-rental/.git/index.lock, then you may delete the lock file and retry.
    at org.eclipse.jgit.dircache.DirCache.lock(DirCache.java:221)
    at org.eclipse.jgit.dircache.DirCache.lock(DirCache.java:292)
    at org.eclipse.jgit.dircache.DirCache.lock(DirCache.java:258)
    at org.eclipse.jgit.lib.Repository.lockDirCache(Repository.java:1282)
    at org.eclipse.jgit.api.ResetCommand.checkoutIndex(ResetCommand.java:407)
...
(12 additional frame(s) were not displayed)

JGitInternalException: Exception caught during execution of reset command. Cannot lock /data/git-storage/628e37967901344ba8d296e3/629f4d18360bee1cd628fe46/suzuya-studio-rental/.git/index. Ensure that no other process has an open file handle on the lock file /data/git-storage/628e37967901344ba8d296e3/629f4d18360bee1cd628fe46/suzuya-studio-rental/.git/index.lock, then you may delete the lock file and retry.
    at org.eclipse.jgit.api.ResetCommand.call(ResetCommand.java:209)

sentry[bot] avatar Jun 07 '22 14:06 sentry[bot]

@AnaghHegde @vuiets we're suddenly seeing a lot of these sentry issues. Can we fix on priority? What has changed here?

Nikhil-Nandagopal avatar Jul 27 '22 06:07 Nikhil-Nandagopal

@Nikhil-Nandagopal The sudden spike in the sentry was due to efs. We noticed a sudden spike in efs throughput yesterday and this caused the users to not be able to use the git feature. Hence the sentry had so many cannot lock file(as java process could not access the file system) errors in sentry. PR is done, just waiting for Arpits final review and then its good to merge to release.

AnaghHegde avatar Jul 27 '22 09:07 AnaghHegde

@AnaghHegde could we close this if it's merged?

vuiets avatar Aug 19 '22 05:08 vuiets

Yeah can we please fix this? I see this very often and I can even reproduce this on prod

Nikhil-Nandagopal avatar Aug 19 '22 05:08 Nikhil-Nandagopal

@AnaghHegde to fix one flaky test for this, on a PR raised for this.

hiteshjoshi avatar Sep 30 '22 07:09 hiteshjoshi

#12174

hiteshjoshi avatar Sep 30 '22 07:09 hiteshjoshi

This will be fixed after the new export/import infra changes we working on.

hiteshjoshi avatar Dec 09 '22 06:12 hiteshjoshi