Skip to content

TestSearchRepos is broken #6977

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

Closed
zeripath opened this issue May 17, 2019 · 5 comments · Fixed by #7004
Closed

TestSearchRepos is broken #6977

zeripath opened this issue May 17, 2019 · 5 comments · Fixed by #7004

Comments

@zeripath
Copy link
Contributor

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

Description

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

@zeripath
Copy link
Contributor Author

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

@zeripath
Copy link
Contributor Author

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?

@zeripath
Copy link
Contributor Author

This issue is preventing #6961 from passing ci

@lunny
Copy link
Member

lunny commented May 18, 2019

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

@kolaente
Copy link
Member

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

zeripath added a commit to zeripath/gitea that referenced this issue May 21, 2019
…b.com:zeripath/gitea into fix-go-gitea#6977-Force-update-index-and-wait
@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.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants