Skip to content
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.

Sign up for GitHub

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

Jump to bottom

codis: file /tmp/codis/data/codis3/codis-demo/topom already exists #1661

Closed
yaoyinnan opened this issue Jun 29, 2023 · 0 comments · Fixed by #1662
Closed

codis: file /tmp/codis/data/codis3/codis-demo/topom already exists #1661

yaoyinnan opened this issue Jun 29, 2023 · 0 comments · Fixed by #1662

Comments

@yaoyinnan
Copy link
Contributor

yaoyinnan commented Jun 29, 2023 •
edited
Loading

When deploying codis, the tmp file was not cleaned up due to abnormal exit, and it needs to be cleaned up manually. The path of the conflicting file is not indicated in the log. Since this file is used to mark a codis cluster, it cannot be directly overridden in code when duplicates are detected.

2023/06/29 20:04:05 main.go:171: [WARN] [0x1400012d200] dashboard online failed [13]
2023/06/29 20:04:07 fsclient.go:192: [WARN] fsclient - create /codis3/codis-demo/topom failed
2023/06/29 20:04:07 topom.go:189: [ERROR] store: acquire lock of codis-demo failed
[error]: file already exists
    4   /Users/yaoyinnan/Projects/WorkProjects/pika/codis/pkg/models/fs/fsclient.go:144
            pika/codis/v2/pkg/models/fs.(*Client).writeFile
    3   /Users/yaoyinnan/Projects/WorkProjects/pika/codis/pkg/models/fs/fsclient.go:191
            pika/codis/v2/pkg/models/fs.(*Client).Create
    2   /Users/yaoyinnan/Projects/WorkProjects/pika/codis/pkg/models/store.go:119
            pika/codis/v2/pkg/models.(*Store).Acquire
    1   /Users/yaoyinnan/Projects/WorkProjects/pika/codis/pkg/topom/topom.go:188
            pika/codis/v2/pkg/topom.(*Topom).Start
    0   /Users/yaoyinnan/Projects/WorkProjects/pika/codis/cmd/dashboard/main.go:169
            main.main
        ... ...

Modify it to the path of the exposed file in the log:

2023/06/29 20:06:52 topom.go:189: [ERROR] store: acquire lock of codis-demo failed
[error]: file /tmp/codis/data/codis3/codis-demo/topom already exists
yaoyinnan added a commit to yaoyinnan/pika that referenced this issue Jun 29, 2023
@yaoyinnan
codis: file /tmp/codis/data/codis3/codis-demo/topom already exists
ff4599a 
Modify it to the path of the exposed file in the log.

Fixes: OpenAtomFoundation#1661

Signed-off-by: yaoyinnan <[email protected]>
@yaoyinnan yaoyinnan mentioned this issue Jun 29, 2023
Merged
yaoyinnan added a commit to yaoyinnan/pika that referenced this issue Jun 29, 2023
@yaoyinnan
codis: file /tmp/codis/data/codis3/codis-demo/topom already exists
6b954e8 
Modify it to the path of the exposed file in the log.

Fixes: OpenAtomFoundation#1661

Signed-off-by: yaoyinnan <[email protected]>
yaoyinnan added a commit to yaoyinnan/pika that referenced this issue Jun 29, 2023
@yaoyinnan
codis: file /tmp/codis/data/codis3/codis-demo/topom already exists
59f310d 
Modify it to the path of the exposed file in the log.

Fixes: OpenAtomFoundation#1661

Signed-off-by: yaoyinnan <[email protected]>
yaoyinnan added a commit that referenced this issue Jun 29, 2023
@yaoyinnan
codis: file /tmp/codis/data/codis3/codis-demo/topom already exists (#…
cac6914 
…1662)

Modify it to the path of the exposed file in the log.

Fixes: #1661

Signed-off-by: yaoyinnan <[email protected]>
bigdaronlee163 pushed a commit to bigdaronlee163/pika that referenced this issue Jun 8, 2024
@yaoyinnan
codis: file /tmp/codis/data/codis3/codis-demo/topom already exists (O…
b006788 
…penAtomFoundation#1662)

Modify it to the path of the exposed file in the log.

Fixes: OpenAtomFoundation#1661

Signed-off-by: yaoyinnan <[email protected]>
cheniujh pushed a commit to cheniujh/pika that referenced this issue Sep 24, 2024
@yaoyinnan
codis: file /tmp/codis/data/codis3/codis-demo/topom already exists (O…
81d9ca0 
…penAtomFoundation#1662)

Modify it to the path of the exposed file in the log.

Fixes: OpenAtomFoundation#1661

Signed-off-by: yaoyinnan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

codis: file /tmp/codis/data/codis3/codis-demo/topom already exists yaoyinnan/pika
1 participant
@yaoyinnan