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

Error 500 when creating new pull request #1434

Closed
2 of 7 tasks
tobias-- opened this issue Apr 3, 2017 · 6 comments
Closed
2 of 7 tasks

Error 500 when creating new pull request #1434

tobias-- opened this issue Apr 3, 2017 · 6 comments
Labels

Comments

@tobias--
Copy link

tobias-- commented Apr 3, 2017

  • Gitea version (or commit ref): 1.1.0
  • Git version:
  • Operating system: Windows 2008 R2
  • Database (use [x]):
    • PostgreSQL
    • MySQL
    • MSSQL
    • SQLite
  • Can you reproduce the bug at https://try.gitea.io:
    • Yes (provide example URL)
    • No
    • Not relevant
  • Log gist:
2017/04/03 15:01:21 [T] PullRequest[0].testPatch (patchPath): E:\gogs-repositories\XXX\XXX.git\pulls\34.patch
2017/04/03 15:01:21 [T] NewMessageFrom (htmlBody):
[E-Mail contents]
2017/04/03 15:01:21 [T] New e-mail sending request [XXXX@XXX]: Subject: [XXX] XXXXX (#34), issue comment
2017/04/03 15:01:22 [T] PushToBaseRepo[7]: pushing commits to base repo 'refs/pull/34/head'
2017/04/03 15:01:22 [T] DeliverHooks [repo_id: 7]
2017/04/03 15:01:23 [T] E-mails sent [XXXX@XXX]: Subject: [XXX] XXXXXX (#34), issue comment
2017/04/03 15:01:24 [T] Pull request created: 7/45
2017/04/03 15:01:24 [D] Session ID: 69b660ffed886882
2017/04/03 15:01:24 [D] CSRF Token: eiBhxp_6MTDX7ZVZMbbfWgZRKP46MTQ5MTIwMTMyODYxMTk1NDgwMA==
2017/04/03 15:01:24 [...user/notification.go:33 GetNotificationCount()] [E] GetNotificationCount: database table is locked: notification
2017/04/03 15:01:24 [D] Template: status/500

Description

Creating a new PR will sometimes result in a 500 error. The PR seems to be alright and can be merged. It seems that updating notifications table does not work.

@plessbd
Copy link

plessbd commented Apr 3, 2017

duplicate of #1372 ?

@lunny
Copy link
Member

lunny commented Apr 3, 2017

seems not.

@lunny lunny added the type/bug label Apr 3, 2017
@lunny lunny added this to the 1.2.0 milestone Apr 3, 2017
@plessbd
Copy link

plessbd commented Apr 3, 2017

You are right, didnt fully see the end of error line.
Seem related though, both are because of a table lock.
This one on notification, #1372 because of repository

@bkcsoft
Copy link
Member

bkcsoft commented Jun 15, 2017

Could you test if this fixes the issue? #1961 (applying the diff and building on v1.1.2 should be straight forward)

@lunny
Copy link
Member

lunny commented Jun 20, 2017

@bkcsoft I think #1961 didn't resolve the database lock problem. And move all database lock issues to v1.3.

@lunny
Copy link
Member

lunny commented Jun 23, 2017

continue via #2040

@lunny lunny closed this as completed Jun 23, 2017
@lunny lunny removed this from the 1.3.0 milestone Jun 23, 2017
@go-gitea go-gitea locked and limited conversation to collaborators Nov 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

4 participants