Too much warning "cannot get cache context when getting data"

[adlternative]

Description

A large number of cache warnings appear after pushing a large number of tags (about 2,000)

Gitea Version

1.20.0+dev (90a7bba)

Can you reproduce the bug on the Gitea demo site?

No

Log Gist

No response

Screenshots

Git Version

2.39.0.198

Operating System

archlinux

How are you running Gitea?

a command-line

gitea web

Database

MySQL

[lunny]

I think this has been fixed in #23054

[adlternative]

I think this has been fixed in #23054

Well, I think it still triggered warns with Gitea latest version "1.20.0+dev-34-gb9a9b8e46 built with GNU Make 4.4, go1.20.1 : bindata"

[yp05327]

In #23054,

getting the system settings will not try from the cache but will read from the database directly.

It seems that cache will read from database directly when server starts. So is it possible that this context data is just not existed in the database?

[adlternative]

In #23054,

getting the system settings will not try from the cache but will read from the database directly.

It seems that cache will read from database directly when server starts. So is it possible that this context data is just not existed in the database?

Agree, maybe so.

[lunny]

In #23054,

getting the system settings will not try from the cache but will read from the database directly.

It seems that cache will read from database directly when server starts. So is it possible that this context data is just not existed in the database?

The warning is not because of the record is not exist in database but cannot get the context cache key. I think #23186 will fix this.

[adlternative]

Thanks, I think it has been fixed!

[karolyi]

In what version should this have been fixed? I'm still getting them with 1.19.0:

2023/03/28 16:21:22 ...les/cache/context.go:71:SetContextData() [W] [6422f3fa-46] cannot get cache context when setting data: &{context.Background.WithCancel.WithCancel.WithValue(type pprof.labelContextKey, val {"graceful-lifecycle":"with-hammer"}).WithCancel.WithValue(type pprof.labelContextKey, val {"graceful-lifecycle":"with-hammer", "pid":"6422f3fa-46", "ppid":"", "process-description":"Incoming Email", "process-type":"system"}) 6422f3fa-46}
2023/03/28 16:21:22 ...les/cache/context.go:62:GetContextData() [W] [6422f3fa-46] cannot get cache context when getting data: &{context.Background.WithCancel.WithCancel.WithValue(type pprof.labelContextKey, val {"graceful-lifecycle":"with-hammer"}).WithCancel.WithValue(type pprof.labelContextKey, val {"graceful-lifecycle":"with-hammer", "pid":"6422f3fa-46", "ppid":"", "process-description":"Incoming Email", "process-type":"system"}) 6422f3fa-46}
2023/03/28 16:21:22 ...les/cache/context.go:71:SetContextData() [W] [6422f3fa-46] cannot get cache context when setting data: &{context.Background.WithCancel.WithCancel.WithValue(type pprof.labelContextKey, val {"graceful-lifecycle":"with-hammer"}).WithCancel.WithValue(type pprof.labelContextKey, val {"graceful-lifecycle":"with-hammer", "pid":"6422f3fa-46", "ppid":"", "process-description":"Incoming Email", "process-type":"system"}) 6422f3fa-46}
2023/03/28 16:21:22 ...les/cache/context.go:62:GetContextData() [W] [6422f3fa-46] cannot get cache context when getting data: &{context.Background.WithCancel.WithCancel.WithValue(type pprof.labelContextKey, val {"graceful-lifecycle":"with-hammer"}).WithCancel.WithValue(type pprof.labelContextKey, val {"graceful-lifecycle":"with-hammer", "pid":"6422f3fa-46", "ppid":"", "process-description":"Incoming Email", "process-type":"system"}) 6422f3fa-46}
2023/03/28 16:21:22 ...les/cache/context.go:71:SetContextData() [W] [6422f3fa-46] cannot get cache context when setting data: &{context.Background.WithCancel.WithCancel.WithValue(type pprof.labelContextKey, val {"graceful-lifecycle":"with-hammer"}).WithCancel.WithValue(type pprof.labelContextKey, val {"graceful-lifecycle":"with-hammer", "pid":"6422f3fa-46", "ppid":"", "process-description":"Incoming Email", "process-type":"system"}) 6422f3fa-46}

[karolyi]

Oh, 1.20.0, supposedly. Let's wait a bit more then.