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.

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

tidb-drainer cannot be deployed successfully out-of-box #792

Closed
sokada1221 opened this issue Aug 20, 2019 · 4 comments
Closed

tidb-drainer cannot be deployed successfully out-of-box #792

sokada1221 opened this issue Aug 20, 2019 · 4 comments

Comments

@sokada1221
Copy link
Contributor

Bug Report

What version of Kubernetes are you using?
1.13

What version of TiDB Operator are you using?
1.0.0-Release1

What storage classes exist in the Kubernetes cluster and what are used for PD/TiKV pods?
N/A

What's the status of the TiDB cluster pods?
tidb-drainer pod crashes right after container start

What did you do?
Install TiDB cluster v2.1.16 using GA helm charts (v1.0.0)

What did you expect to see?
Successful drainer deployment out-of-box

What did you see instead?
Error due to incompatible changes made in drainer from v2.1.15 to v2.1.16

2019/08/20 13:38:10 main.go:26: [fatal] verifying flags error, See 'drainer --help'. component drainer's config file /etc/drainer/drainer.toml contained unknown configuration options: syncer.detect-interval, syncer.to.compression
github.com/pingcap/tidb-binlog/pkg/util.StrictDecodeFile
    /home/jenkins/workspace/release_tidb_2.1/go/src/github.com/pingcap/tidb-binlog/pkg/util/util.go:159
github.com/pingcap/tidb-binlog/drainer.(*Config).configFromFile
    /home/jenkins/workspace/release_tidb_2.1/go/src/github.com/pingcap/tidb-binlog/drainer/config.go:211
github.com/pingcap/tidb-binlog/drainer.(*Config).Parse
    /home/jenkins/workspace/release_tidb_2.1/go/src/github.com/pingcap/tidb-binlog/drainer/config.go:156
main.main
    /home/jenkins/workspace/release_tidb_2.1/go/src/github.com/pingcap/tidb-binlog/cmd/drainer/main.go:25
runtime.main
    /usr/local/go/src/runtime/proc.go:200
runtime.goexit
    /usr/local/go/src/runtime/asm_amd64.s:1337
@aylei
Copy link
Contributor

aylei commented Aug 20, 2019

What's your installation command?

@sokada1221
Copy link
Contributor Author

@aylei it was like this:
helm install http://download.pingcap.org/tidb-drainer-latest.tgz --name tidb-drainer --namespace ${CLUSTER_NAME}

Sorry, I had a fix already but had to run to get dinner 😆I made a PR just now - please check. It was due to incompatible config format changes from drainer.

@aylei
Copy link
Contributor

aylei commented Aug 20, 2019

@shinnosuke-okada Thanks!

@aylei
Copy link
Contributor

aylei commented Aug 22, 2019

closed via #793

@aylei aylei closed this as completed Aug 22, 2019
yahonda pushed a commit that referenced this issue Dec 27, 2021
* zh: add dmcluster usage document

* add the document

* fix a link

* improve language

* Update enable-tls-for-dm.md

* improve language

* Apply suggestions from code review

Co-authored-by: Ran <huangran@pingcap.com>

* Update deploy-tidb-dm.md

* Update deploy-tidb-dm.md

* Apply suggestions from code review

Co-authored-by: Ran <huangran@pingcap.com>

* Apply suggestions from code review

Co-authored-by: Ran <huangran@pingcap.com>

* Apply suggestions from code review

Co-authored-by: Chunzhu Li <lichunzhu@pingcap.com>

* Apply suggestions from code review

Co-authored-by: Chunzhu Li <lichunzhu@pingcap.com>

* Update enable-tls-for-dm.md

Co-authored-by: Ran <huangran@pingcap.com>
Co-authored-by: Chunzhu Li <lichunzhu@pingcap.com>
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

No branches or pull requests

2 participants