Failed to synchronize tags to release in SyncMirror with error: "pushUpdateAddTag: GetTag: Length must be 40:"

[somera]

Feature Description

In gitea.log I see sometimes:

==> gitea.log <==
2022/03/09 12:06:37 ...irror/mirror_pull.go:261:runSync() [E] Failed to synchronize tags to releases for repository: pushUpdateAddTag: GetTag: Length must be 40:
        /source/services/mirror/mirror_pull.go:261 (0x1cdf3b8)
        /source/services/mirror/mirror_pull.go:376 (0x1ce10b8)
        /source/services/mirror/mirror.go:42 (0x1cdb344)
        /source/services/mirror/mirror.go:144 (0x1cdbe31)
        /source/modules/queue/unique_queue_channel.go:76 (0x14954b6)
        /source/modules/queue/workerpool.go:437 (0x149e1c5)
        /source/modules/queue/workerpool.go:280 (0x149d096)
        /usr/local/go/src/runtime/asm_amd64.s:1581 (0x471520)

But you can't see:

• which project
• which tag?
• how long is it?

Screenshots

No response

[zeripath]

You have to tell us what version of Gitea you're reporting from.

Assuming you're using 1.16 - you can add a trace logger for the services/mirror as I have previously explained which will add significant amounts of tracing.

How to add trace logging for services/mirror

You can adjust your ini like this:

[log]
MODE=console, traceconsole
...

[log.traceconsole]
MODE=console
LEVEL=trace
EXPRESSION=services/mirror

Or you can run gitea manager logging add console -n traceconsole -l trace -e "services/mirror"

You can use a file logger too. Look at gitea manager logging add --help gitea manager logging add file --help

This is an interesting case. The error is coming from within:

gitea/modules/repository/repo.go

Lines 293 to 295 in 2448c5a

	// PushUpdateAddTag must be called for any push actions to add tag
	func PushUpdateAddTag(repo *repo_model.Repository, gitRepo *git.Repository, tagName string) error {
	tag, err := gitRepo.GetTag(tagName)

The error string indicates that the ID it is trying to look for is actually an empty string but that doesn't make any sense at all.

I've created another PR to further improve the logging here, but if you add the tracelogger as above we can find out what is special about the repo that is causing this.

[somera]

I'm using 1.16.3.

[somera]

@zeripath some log examples

Repo-URL: https://github.com/ballerina-platform/module-ballerina-cache

2022/03/10 08:50:08 ...irror/mirror_pull.go:356:SyncPullMirror() [T] SyncMirrors [repo_id: 6354]
2022/03/10 08:50:08 ...irror/mirror_pull.go:375:SyncPullMirror() [T] SyncMirrors [repo: 6354:ballerina-platform/module-ballerina-cache]: Running Sync
2022/03/10 08:50:08 ...irror/mirror_pull.go:190:runSync() [T] SyncMirrors [repo: 6354:ballerina-platform/module-ballerina-cache]: running git remote update...

==> gitea.log <==
2022/03/10 08:50:09 ...age_stats_nogogit.go:86:func2() [E] Unable to open checker for 11d0f5d34c7e8287afa5724c0e4b907a00ebe573. Error: failed to run attr-check. Error: context canceled
        Stderr:
        /source/modules/git/repo_language_stats_nogogit.go:86 (0xaeef8f)
        /usr/local/go/src/runtime/asm_amd64.s:1581 (0x471520)

Repo-URL: https://github.com/hendisantika/spring-boot-paypal

2022/03/10 08:49:53 ...irror/mirror_pull.go:356:SyncPullMirror() [T] SyncMirrors [repo_id: 6417]
2022/03/10 08:49:53 ...irror/mirror_pull.go:375:SyncPullMirror() [T] SyncMirrors [repo: 6417:hendisantika/spring-boot-paypal]: Running Sync
2022/03/10 08:49:53 ...irror/mirror_pull.go:190:runSync() [T] SyncMirrors [repo: 6417:hendisantika/spring-boot-paypal]: running git remote update...

==> gitea.log <==
2022/03/10 08:49:53 ...age_stats_nogogit.go:86:func2() [E] Unable to open checker for cf898a8c2e397c699e5e65b8ffa060883532eb1a. Error: failed to run attr-check. Error: context canceled
        Stderr:
        /source/modules/git/repo_language_stats_nogogit.go:86 (0xaeef8f)
        /usr/local/go/src/runtime/asm_amd64.s:1581 (0x471520)

Repo-URL: https://github.com/ballerina-platform/ballerina-standard-library

2022/03/10 08:49:53 ...irror/mirror_pull.go:356:SyncPullMirror() [T] SyncMirrors [repo_id: 6330]
2022/03/10 08:49:53 ...irror/mirror_pull.go:375:SyncPullMirror() [T] SyncMirrors [repo: 6330:ballerina-platform/ballerina-standard-library]: Running Sync
2022/03/10 08:49:53 ...irror/mirror_pull.go:190:runSync() [T] SyncMirrors [repo: 6330:ballerina-platform/ballerina-standard-library]: running git remote update...

==> gitea.log <==
2022/03/10 08:49:53 ...age_stats_nogogit.go:86:func2() [E] Unable to open checker for a5f1ab1c2af1bb0866438c793c390062bf0e39ec. Error: failed to run attr-check. Error: context canceled
        Stderr:
        /source/modules/git/repo_language_stats_nogogit.go:86 (0xaeef8f)
        /usr/local/go/src/runtime/asm_amd64.s:1581 (0x471520)

[zeripath]

Unable to open checker for a5f1ab1c2af1bb0866438c793c390062bf0e39ec. Error: failed to run attr-check. Error: context canceled

Ignore these. They are nothing to do with your initial report. They are incorrectly being reported as an error when they're normal. PRs to fix them have already been merged and I recently closed a duplicate issue you opened about these.

[zeripath]

We need to find cases where:

Failed to synchronize tags to releases for repository: pushUpdateAddTag: GetTag: Length must be 40:

Is reported. Not the check-attr false positive.

[somera]

Unable to open checker for a5f1ab1c2af1bb0866438c793c390062bf0e39ec. Error: failed to run attr-check. Error: context canceled

Ignore these. They are nothing to do with your initial report. They are incorrectly being reported as an error when they're normal. PRs to fix them have already been merged and I recently closed a duplicate issue you opened about these.

Sorry. Wrong post.

[somera]

I got this error on run "Garbage-Collection" cron. I will start it again.