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

Number of commits not counted correctly #4424

Closed
2 of 7 tasks
bkroll opened this issue Jul 11, 2018 · 28 comments
Closed
2 of 7 tasks

Number of commits not counted correctly #4424

bkroll opened this issue Jul 11, 2018 · 28 comments
Labels
issue/needs-feedback For bugs, we need more details. For features, the feature must be described in more detail issue/stale

Comments

@bkroll
Copy link
Contributor

bkroll commented Jul 11, 2018

  • Gitea version (or commit ref): 1.4.3 built with: bindata, sqlite
  • Git version: git version 2.12.3
  • Operating system: Linux 4.4.138-94.39-default Rename import paths: "github.com/gogits/gogs" -> "github.com/go-gitea/gitea" #1 SMP Mon Jun 18 13:27:26 UTC 2018 (baa07f9) x86_64 x86_64 x86_64 GNU/Linux
  • 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: no error is logged when accessing the commit count or list

Description

The number of commits is not counted correctly. as it can be seen from the screenshot. There are three commits, but it says "1 Commits" on the repository pages.

Also, the uncounted commits are not shown on the user's public activity list on their profile.

What could be done to have the repositories show the correct commit counts?

Screenshots

gitea not counting correctly

@johanhugg
Copy link

johanhugg commented Jul 11, 2018

Do you have a git repository we can test on? Have you tried using the latest version of Gitea? I just compiled the version you have and tested it, I can't seem to reproduce.

@lafriks
Copy link
Member

lafriks commented Jul 11, 2018

Can you give steps on how did you pushed these commits? (ssh or https etc).

Do you have any warnings/errors in gitea.log at that time.

Commit count are cached for 24 hours if I remember correctly but cache should invalidate on git push/pr merge etc.

@lafriks lafriks added the issue/needs-feedback For bugs, we need more details. For features, the feature must be described in more detail label Jul 11, 2018
@bkroll
Copy link
Contributor Author

bkroll commented Jul 16, 2018

@johanhugg I have used the release at https://github.com/go-gitea/gitea/releases/latest as binary, but not the v1.5.0-rc1 pre-relase, as we are using gitea in production. Sorry, I can't make that gitea install available online due to our firewall.

@lafriks The commits where pushed via ssh using password authentication. No warnings or errors are written to gitea.log when pushing or when viewing the commit count.

I noticed that only some commits are not counted - in other repos on that install, there are ten commits but the counter shows only seven. However, all commits where pushed from the same remote.

The partition is not mounted with "noexec", so #1231 does not seem to apply here.

@bkroll
Copy link
Contributor Author

bkroll commented Jul 16, 2018

@lafriks Now I see that the commit counter at the repo pictured in the screenshot has updated to three, which is correct. There have been more than 24 hours since the commits where made and pushed. No further pushes have been made to the repo in the meantime.

Could there be a problem clearing the cache on push/merge?

@ghost
Copy link

ghost commented Jul 17, 2018

I ran into the same issue as well. Same setup (gitea on sqlite, version 1.4.3, amd64-linux); repository being https://neueland.iserlohn-fortress.net/gitea/aqua/smolbote. The repository is a converted mercurial repo.

Rerunning the gitea instance fixed the commit count.

Creating an empty repository however, and pushing a couple of commits to it properly updates the commit count. (This was done after rerunning the instance.)

Edit: Pushing again to the original repository doesn't update the commit count (gitea says 384; git rev-list --count --all says 385), but pushing to the test repo properly updates (2 -> 3 commits),

Edit: I also noticed that this repository wasn't getting its pushes listed in the dashboard, and no size listed in the user settings. I had manually reset it previously (deleted it, then git init --bare), which I suppose has broken its gitea integration. Remaking the repository from the gitea UI and pushing it back up fixed the issue.

@bkroll
Copy link
Contributor Author

bkroll commented Jul 23, 2018

Thanks @xiannox, rerunning the gitea instance fixed the commit counter for our problem as well, as far at least for now.

If there is no other solution, setting cronjob to regularly restart the instance could be an option, but I still hope someone can help finding another way.

@lafriks
Copy link
Member

lafriks commented Jul 23, 2018

@bkroll no need to restart. It is cached for 20hours so it will show correct after that anyway. You can also set smaller cache time in app.ini

@bkroll
Copy link
Contributor Author

bkroll commented Jul 23, 2018

Thanks @lafriks, I have set ITEM_TTL = 0 to deactivate caching, which provides a workaround to this problem. But now I have noticed that closing or referencing an issue from the commit message does not work either. Could those two problems be related? (I have checked that the noexec problem #1231 is not the case here.)

Is it normal that on ssh push to a gitea repo, no log is written to gitea.log with Debug log level? I'm not sure the hooks are running properly. Also, I have updated the instance to 1.5.0-rc2, but that did not solve the problems.

@lafriks
Copy link
Member

lafriks commented Jul 23, 2018

Check logs if there are any warnings or errors

@bkroll
Copy link
Contributor Author

bkroll commented Jul 23, 2018

Thanks @lafriks I watched gitea.log, http.log and xorm.log while I did the following:

  • pushed first file into empty repo:
    • "quick guide" still appears after pushing file
    • nothing logged
  • created new repo with "initialize repository" option activated, pushed with commit message close #1to close issue:
    • pushed changes appear in gitea
    • #1 in commit message is linked to issue page
    • no reference is displayed on the issue page itself
    • issue is not closed
    • nothing is logged.

Commits are now counted correctly, since I deactivated caching.

machine:/etc/gitea # cat app.ini | grep LEVEL -B2
[log]
MODE      = file
LEVEL     = Debug

I tried the same steps on try.gitea.org. There, everything worked as expected. On my machine, I am running SLES 12 SP3 with git 2.12.3 on sqlite db.

@lafriks
Copy link
Member

lafriks commented Jul 23, 2018

Do you have SELinux enabled in enforced mode?

@lafriks
Copy link
Member

lafriks commented Jul 23, 2018

Check /var/log/audit.log if there is anything about gitea 3000 port

@bkroll
Copy link
Contributor Author

bkroll commented Jul 24, 2018

SELinux is not enabled, /var/log/audit/ is empty. I looked into /var/log/messages, there are only Macaron log lines in there, but no errors.

@SoulSeekkor
Copy link

I can also confirm, fresh install of 1.5.0 of Gitea. The only thing I did was create a mirrored repo from GitHub, after the automatic sync has pulled in a few commits I noticed that the commit count is never updated. A restart of the Gitea instance fixed the count.

@adelowo
Copy link
Member

adelowo commented Aug 21, 2018

@SoulSeekkor There is a fix for mirrored repos already at #4720
@bkroll is this issue on a mirrored repo too or a regular one ?

@SoulSeekkor
Copy link

@adelowo Perfect, thank you!

@bkroll
Copy link
Contributor Author

bkroll commented Sep 16, 2018

@adelowo The issue occured in a regular repo, not a mirrored one.

@bjoe2k4
Copy link

bjoe2k4 commented Sep 16, 2018

I have only observed this kind of behavior when pushing via read/write deploy key, which is already reported a while ago in #3795

@theAkito
Copy link

theAkito commented Dec 4, 2018

This happens to me whenever I push from terminal. Doesn't matter if over SSH or HTTPS. It looks like the commit never happened. The only way to see the new commit is to go to the correct branch and check commits.

@lafriks
Copy link
Member

lafriks commented Dec 4, 2018

@Akito13 what version are you using?

@theAkito
Copy link

theAkito commented Dec 5, 2018

@Akito13 what version are you using?

1.6.0+rc1-123-g3379141d8

Though, I have to clarify: I actually don't care if the commits are counted incorrectly. That's not my problem. And I didn't notice any incorrect counts. I didn't really check it for all repositories, just one and there the count was correct. So I ignored that. But it bothers me, that I can't see pushes on the Dashboard or Public Activity if they are executed from a client in the terminal.

Also, the uncounted commits are not shown on the user's public activity list on their profile.

@stale
Copy link

stale bot commented Feb 3, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 weeks. Thank you for your contributions.

@stale stale bot added the issue/stale label Feb 3, 2019
@Exagone313
Copy link
Contributor

This bug is still not resolved, please do not close.

@stale stale bot removed the issue/stale label Feb 4, 2019
@adelowo
Copy link
Member

adelowo commented Feb 4, 2019

I still cannot replicate this on a regular repository through

@Exagone313
Copy link
Contributor

You can check the steps to reproduce of #5058, it's easy.

@zeripath
Copy link
Contributor

zeripath commented Feb 4, 2019

The issue with deploy keys should be fixed by #5935 and its backport to v1.7 #5944. Is there anything else causing this?

@stale
Copy link

stale bot commented Apr 5, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs during the next 2 weeks. Thank you for your contributions.

@stale stale bot added the issue/stale label Apr 5, 2019
@zeripath
Copy link
Contributor

zeripath commented Apr 7, 2019

Ok I'm gonna close this as fixed by #5935

@zeripath zeripath closed this as completed Apr 7, 2019
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
issue/needs-feedback For bugs, we need more details. For features, the feature must be described in more detail issue/stale
Projects
None yet
Development

No branches or pull requests

9 participants