-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Git repositories added to config.toml
do not appear to be indexed
#3692
Comments
I did some more digging and found that ignoring the names is intentional (commit 71a7230). Disregard my previous note that the names are not parsed |
Hi, thanks for the quick response! I tried your suggestion of adding a local repository and that worked properly - I was able to add the repo to the config and access it in the code browser. However, when I added a remote repository, the issue reappeared. To check whether the issue was related to the hourly interval, I left the server running for about 9 hours. At the end of that period, there were no files (including local repo) in the code browser. The console logs are in the attached file and my
|
Thanks for detailed information, this is indeed a bug, fixing in #3703 |
Just pulled the branch associated with the bug-fix, and it works on my machine! |
Overview
Describe the bug
Git repositories added to
config.toml
do not appear to be indexed. They are read properly from the config file, but they are not added to the database or indexed. This seems related to #3163.Information about your version
Most testing was done with a local build of branch r0.22
I also tested other versions. I built locally using commit 5aa27b5
I also built locally using branch r0.23
Information about your GPU
Apple M2 Max
Details
Detailed description
When repositories are added to
config.toml
, they are not added to the database. They do not seem to be indexed either. When I went through the source, I could not find anywhere in where the config repositories are added.Looking at previous versions of Tabby (like 0.11), it seems like this process was handled by the scheduler in the past (see commit da02d47) . However, I can't find where that functionality went in newer releases.
It also seems like names in the
config.toml
are not parsed anywhere. TheRepositoryConfig
struct only has one field:Steps to reproduce
The outputs shown here were done with version r0.24.0-dev.0, but the results are similar on other versions.
~/.tabby
directorycargo build
target/debug/tabby serve
config.toml
, as listed belowtarget/debug/tabby serve
10.1. Use a SQL viewer to confirm that there are no entries in
dev-db.sqlite
in therepositories
table10.2. Use a SQL viewer to confirm that there are no relevant entries in the database in the
job_runs
table10.2. Check that there are no repositories or index data stored in
~/.tabby
12.1. Use a SQL viewer to confirm the second repo was added to the database.
12.2. Console messages confirm the job was successful
config.toml
This is the entire config.toml I used.
The text was updated successfully, but these errors were encountered: