TestSearchRepos is broken #3347

Closed
opened 2025-11-02 05:09:26 -06:00 by GiteaMirror · 5 comments
Owner

Originally created by @zeripath on GitHub (May 17, 2019).

  • Gitea version (or commit ref):1.9.0+dev-262-g96b412bb8

Description

TAGS="bindata sqlite sqlite_unlock_notify" make build test-sqlite\#TestSearchRepo fails

Originally created by @zeripath on GitHub (May 17, 2019). - Gitea version (or commit ref):1.9.0+dev-262-g96b412bb8 ## Description `TAGS="bindata sqlite sqlite_unlock_notify" make build test-sqlite\#TestSearchRepo` fails
GiteaMirror added the type/testingtype/bug labels 2025-11-02 05:09:26 -06:00
Author
Owner

@zeripath commented on GitHub (May 17, 2019):

This has been broken since at least v1.7.0-dev and likely earlier.

@zeripath commented on GitHub (May 17, 2019): This has been broken since at least v1.7.0-dev and likely earlier.
Author
Owner

@zeripath commented on GitHub (May 17, 2019):

I've just spent all evening trying to figure this out and have failed.

The closest I came to actually getting user2/repo1 indexed was after forcibly updating the repo I manage to get to:

6fb58a8cdc/models/repo_indexer.go (L155)

But I never got past this before the search occurred.

Likely there is a timing issue or locking issue her but I don't understand enough to make this work.

@lunny do you have any idea how to solve this? @kolaente is this the issue you're experiencing on golang-ci?

@zeripath commented on GitHub (May 17, 2019): I've just spent all evening trying to figure this out and have failed. The closest I came to actually getting user2/repo1 indexed was after forcibly updating the repo I manage to get to: https://github.com/sanjid133/gitea/blob/6fb58a8cdcd76aa45902e50da8f2b450fe9d3d35/models/repo_indexer.go#L155 But I never got past this before the search occurred. Likely there is a timing issue or locking issue her but I don't understand enough to make this work. @lunny do you have any idea how to solve this? @kolaente is this the issue you're experiencing on golang-ci?
Author
Owner

@zeripath commented on GitHub (May 17, 2019):

This issue is preventing #6961 from passing ci

@zeripath commented on GitHub (May 17, 2019): This issue is preventing #6961 from passing ci
Author
Owner

@lunny commented on GitHub (May 18, 2019):

I know this issue and will also spend some time to investigate it.

@lunny commented on GitHub (May 18, 2019): I know this issue and will also spend some time to investigate it.
Author
Owner

@kolaente commented on GitHub (May 18, 2019):

@zeripath It could be related, but the tests which are failing in #6418 are TestBleveSearchIssues, unit tests, not integration tests.

@kolaente commented on GitHub (May 18, 2019): @zeripath It could be related, but the tests which are failing in #6418 are `TestBleveSearchIssues`, unit tests, not integration tests.
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: github-starred/gitea#3347