Skip to content

Conversation

@zeripath
Copy link

This PR adds another Function to iterate the objects on the storage simplifying the dump.go command.

(The exact munging of the paths in IterateObjects needs to be double checked.)

@zeripath
Copy link
Author

zeripath commented Sep 27, 2020

d8f1094 Is more of a working-towards solution than a complete solution - we probably want the storage systems to define their defaults - I'll think a bit more. I've moved this to another PR

@zeripath zeripath force-pushed the lunny/storage_config branch from d8f1094 to 99397c3 Compare September 27, 2020 20:45
@lunny
Copy link
Owner

lunny commented Sep 28, 2020

It seems there are some unrelated changes. Could you remove them?

Signed-off-by: Andrew Thornton <[email protected]>
@zeripath zeripath force-pushed the lunny/storage_config branch from 99397c3 to a28ffda Compare September 28, 2020 08:13
@zeripath
Copy link
Author

I've forced pushed to just have the IterateObjects commit on here.

@zeripath
Copy link
Author

I'll open another PR with the self-registering stuff.

@lunny lunny merged commit 38d1cc5 into lunny:lunny/storage_config Sep 28, 2020
@zeripath zeripath deleted the lunny/storage_config branch September 28, 2020 18:25
lunny added a commit that referenced this pull request Sep 19, 2023
Fix the bug on try.gitea.io

```log
2023/09/18 01:48:41 ...ations/migrations.go:635:Migrate() [I] Migration[276]: Add RemoteAddress to mirrors
2023/09/18 01:48:41 routers/common/db.go:34:InitDBEngine() [E] ORM engine initialization attempt #7/10 failed. Error: migrate: migration[276]: Add RemoteAddress to mirrors failed: exit status 128 - fatal: not a git repository (or any parent up to mount point /)
Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).
 - fatal: not a git repository (or any parent up to mount point /)
Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).
```

Caused by go-gitea#26952

---------

Co-authored-by: Jason Song <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants