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

Data node fail to join cluster in 0.9.0rc25 #2348

Closed
PierreF opened this issue Apr 20, 2015 · 0 comments · Fixed by #2353
Closed

Data node fail to join cluster in 0.9.0rc25 #2348

PierreF opened this issue Apr 20, 2015 · 0 comments · Fixed by #2353
Assignees

Comments

@PierreF
Copy link
Contributor

PierreF commented Apr 20, 2015

I can't have additional data node joining a cluster with influxdb 0.9.0rc25 (or git master).

With the following configuration:

server1.toml:

port = 8086
reporting-disabled = true
[broker]
dir="/tmp/influxdb/8086/broker"
[data]
dir="/tmp/influxdb/8086/data"

server2.toml:

port = 8087
reporting-disabled = true
[broker]
dir="/tmp/influxdb/8087/broker"
[data]
dir="/tmp/influxdb/8087/data"

I start the influxDB server with:

$ rm -fr /tmp/influxdb/; influxd -config server1.toml

And, in another terminal

$ influxd -config server2.toml -join http://localhost:8086

After that, writing (with replication factor = 2) on either localhost:8086 or localhost:8087, only the first got the write.

In addition, curl on /data/data_nodes return only one URL

$ curl -v http://localhost:8086/data/data_nodes
[...]
[{"id":1,"url":"http://vanaheim:8086"}]

Following the exact same step with rc24, the last curl return two nodes:

[{"id":1,"url":"http://vanaheim:8086"},{"id":2,"url":"http://vanaheim:8087"}]

I've done a git bisect to find the commit which broke my setup, it's c5bdb5a.

More precisely, commenting the following line fix my issue : https://github.com/influxdb/influxdb/blob/3332f09c5b6c0d8c01750c87112f9eea663d7d9c/cmd/influxd/run.go#L643

Here is the paste with console output from influxdb : http://pastebin.com/zpCs9ZVd

@jwilder jwilder self-assigned this Apr 20, 2015
jwilder added a commit that referenced this issue Apr 21, 2015
jwilder added a commit that referenced this issue Apr 21, 2015
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

Successfully merging a pull request may close this issue.

2 participants