]> source.dussan.org Git - gitea.git/commit
handle repository.size column being NULL in migration v263 (#28336)
authorNate Levesque <nate@thenaterhood.com>
Tue, 5 Dec 2023 14:31:13 +0000 (09:31 -0500)
committerGitHub <noreply@github.com>
Tue, 5 Dec 2023 14:31:13 +0000 (14:31 +0000)
commitf891172ef4163bf75ebe00a9e76a4ab60f0d3d4a
tree933bea2a732eca411f3b68203a15b063f3251198
parent49b98e45bc6301b74181282a410aa02d8e0b8f30
handle repository.size column being NULL in migration v263 (#28336)

This resolves a problem I encountered while updating gitea from 1.20.4
to 1.21. For some reason (correct or otherwise) there are some values in
`repository.size` that are NULL in my gitea database which cause this
migration to fail due to the NOT NULL constraints.

Log snippet (excuse the escape characters)
```
ESC[36mgitea                |ESC[0m 2023-12-04T03:52:28.573122395Z 2023/12/04 03:52:28 ...ations/migrations.go:641:Migrate() [I] Migration[263]: Add git_size and lfs_size columns to repository table
ESC[36mgitea                |ESC[0m 2023-12-04T03:52:28.608705544Z 2023/12/04 03:52:28 routers/common/db.go:36:InitDBEngine() [E] ORM engine initialization attempt #3/10 failed. Error: migrate: migration[263]: Add git_size and lfs_size columns to repository table failed: NOT NULL constraint failed: repository.git_size
```

I assume this should be reasonably safe since `repository.git_size` has
a default value of 0 but I don't know if that value being 0 in the odd
situation where `repository.size == NULL` has any problematic
consequences.
models/migrations/v1_21/v263.go