Join GitHub today
GitHub is home to over 28 million developers working together to host and review code, manage projects, and build software together.
Sign upbind-address parameter issue on 0.9 #2468
Comments
beckettsean
added
the
category/clustering
label
May 1, 2015
beckettsean
added this to the 0.9.0 milestone
May 1, 2015
jwilder
self-assigned this
May 1, 2015
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
easyrasta
May 2, 2015
Strange, I would open issue to say opposite side. I am trying to do a docker image, but If you want an image with broker and data node it little bit difficult. because broker need to announce his hostame to register, and datanode trying to register to this broker with hostname. So to fix it I need to se hostame in /etc/hosts to redirect to localhost, and give a hostname of docker machine.
This is mainly for first node starting of cluster
easyrasta
commented
May 2, 2015
|
Strange, I would open issue to say opposite side. I am trying to do a docker image, but If you want an image with broker and data node it little bit difficult. because broker need to announce his hostame to register, and datanode trying to register to this broker with hostname. So to fix it I need to se hostame in /etc/hosts to redirect to localhost, and give a hostname of docker machine. This is mainly for first node starting of cluster |
toddboom
modified the milestones:
0.9.0,
0.9.1
May 8, 2015
corylanou
assigned
corylanou
and unassigned
jwilder
May 28, 2015
corylanou
added
the
2 - Working
label
May 28, 2015
toddboom
modified the milestones:
0.9.1,
0.9.2
Jun 5, 2015
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
ckmaresca
Jun 6, 2015
I am having a similar issue after changing the bind address in rc31. Changing the bind address re-binds all the daemons to the correct port, but the broker is still looking for the data node at localhost:
[[srvr] 2015/06/06 11:33:38 broker cq: no data nodes currently available.
[messaging] 2015/06/06 11:33:37 reconnecting to broker: url={http <nil> localhost:8086 /messaging/messages index=3&streaming=true&topicID=0 }, err=Get http://localhost:8086/messaging/messages?index=3&streaming=true&topicID=0: dial tcp 127.0.0.1:8086: connection refused
I re-installed influx several times to see if I was missing something and changed the bind address before first launch, but that didn't work at all. Perhaps I'm missing something in the configs as I an incomplete understanding of influx configuration options....
I'm guessing that waiting for r32 is the thing to do, but I thought I'd log this anyway.
ckmaresca
commented
Jun 6, 2015
|
I am having a similar issue after changing the bind address in rc31. Changing the bind address re-binds all the daemons to the correct port, but the broker is still looking for the data node at
I re-installed influx several times to see if I was missing something and changed the bind address before first launch, but that didn't work at all. Perhaps I'm missing something in the configs as I an incomplete understanding of influx configuration options.... I'm guessing that waiting for r32 is the thing to do, but I thought I'd log this anyway. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
noroute
Jul 2, 2015
Is there a schedule for 0.9.2? This issue is quite ugly as there is no simple workaround (or is there?)....
noroute
commented
Jul 2, 2015
|
Is there a schedule for 0.9.2? This issue is quite ugly as there is no simple workaround (or is there?).... |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
pauldix
Jul 2, 2015
Member
@noroute: 0.9.2 will be released on July 23rd. In the meantime, when this issue is fixed and merged into master, we have nightly builds that you can test with that can be found here: https://influxdb.com/download/index.html
|
@noroute: 0.9.2 will be released on July 23rd. In the meantime, when this issue is fixed and merged into master, we have nightly builds that you can test with that can be found here: https://influxdb.com/download/index.html |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
beckettsean
Jul 15, 2015
Contributor
@corylanou what's the status of this issue? Are you working on a fix like Paul seems to think?
|
@corylanou what's the status of this issue? Are you working on a fix like Paul seems to think? |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
beckettsean
Jul 16, 2015
Contributor
@pauldix @toddboom it turns out @corylanou is otherwise occupied leading into the 0.9.2 code freeze. Who else do you nominate to fix this?
|
@pauldix @toddboom it turns out @corylanou is otherwise occupied leading into the 0.9.2 code freeze. Who else do you nominate to fix this? |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
marcosnils
Jul 16, 2015
Contributor
@beckettsean @toddboom @pauldix I remember fixing this issue in 0.9rc some time ago. If you still agree with the solution proposed in this PR #2479 I can refloat, but please merge this time
|
@beckettsean @toddboom @pauldix I remember fixing this issue in 0.9rc some time ago. If you still agree with the solution proposed in this PR #2479 I can refloat, but please merge this time |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
beckettsean
Jul 16, 2015
Contributor
@marcosnils thanks for the reminder! I'm not qualified to
|
@marcosnils thanks for the reminder! I'm not qualified to |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
pauldix
Jul 16, 2015
Member
@marcosnils if you can redo it, we'd gladly merge. Sorry about not getting it in last time. I don't think there will be another big refactor that would cause a problem.
Although, looping @jwilder in to make sure his work isn't changing how server startup/binding works.
|
@marcosnils if you can redo it, we'd gladly merge. Sorry about not getting it in last time. I don't think there will be another big refactor that would cause a problem. Although, looping @jwilder in to make sure his work isn't changing how server startup/binding works. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
marcosnils
Jul 16, 2015
Contributor
@pauldix no problem, I'll submit a new PR tomorrow with the updated code. I took special care in trying not to break backwards compatibility in it, so i'd be great if you can confirm.
|
@pauldix no problem, I'll submit a new PR tomorrow with the updated code. I took special care in trying not to break backwards compatibility in it, so i'd be great if you can confirm. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
jwilder
Jul 16, 2015
Collaborator
This might interfere with my changes but not sure off-hand.
Also, I'm not sure that changing the hostname would fix the original issue. I believe the original issue was that the old broker/meta-store saved the IP addresses of existing nodes and tried to reconnect to them on startup using its existing state. Changing the hostname was not supported because you would need to remove the node from the meta-store/raft and re-add it. We probably have a similar with 0.9.1 because our new raft still needs to save the raft peers and node addresses in the meta-store.
I think we'd have to detect hostname changes for a node and propagate the changes through the cluster.
|
This might interfere with my changes but not sure off-hand. Also, I'm not sure that changing the hostname would fix the original issue. I believe the original issue was that the old broker/meta-store saved the IP addresses of existing nodes and tried to reconnect to them on startup using its existing state. Changing the hostname was not supported because you would need to remove the node from the meta-store/raft and re-add it. We probably have a similar with 0.9.1 because our new raft still needs to save the raft peers and node addresses in the meta-store. I think we'd have to detect hostname changes for a node and propagate the changes through the cluster. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
pauldix
Jul 16, 2015
Member
ok, based on what @jwilder said I think this is going to be more involved. Probably best to hold off for now @marcosnils.
We definitely have a thing where we need to be able to update the IP/hostname of the server. People running in Docker containers commonly have this problem.
|
ok, based on what @jwilder said I think this is going to be more involved. Probably best to hold off for now @marcosnils. We definitely have a thing where we need to be able to update the IP/hostname of the server. People running in Docker containers commonly have this problem. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
marcosnils
Jul 16, 2015
Contributor
@jwilder @pauldix I thought the main issue arose when trying to start influxdb the first time with the hostname parameter set. That is what my PR actually fixes, and what the test I've included asserts.
There's also what you're mentioning about changing the configuration parameter once the cluster has been initialized which involves change propagation.
The solution for this last situation fixes the first one, but requires thorough testing and handling complex situations. Maybe in the meantime applying my simple fix will allow users to setup a cluster using the hostname parameter.
|
@jwilder @pauldix I thought the main issue arose when trying to start influxdb the first time with the hostname parameter set. That is what my PR actually fixes, and what the test I've included asserts. There's also what you're mentioning about changing the configuration parameter once the cluster has been initialized which involves change propagation. The solution for this last situation fixes the first one, but requires thorough testing and handling complex situations. Maybe in the meantime applying my simple fix will allow users to setup a cluster using the hostname parameter. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
pauldix
Jul 21, 2015
Member
@marcosnils sorry for the delayed response, If your fix is just for setting the hostname initially, I think it's worth doing it. I've created #3421 to track the issue of updating the hostname/IP of a server in a cluster.
|
@marcosnils sorry for the delayed response, If your fix is just for setting the hostname initially, I think it's worth doing it. I've created #3421 to track the issue of updating the hostname/IP of a server in a cluster. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
pauldix
Jul 22, 2015
Member
@marcosnils yeah, he still has some stuff in development that would probably make your code different. Don't want to have you go to the trouble of doing it and then clobber your stuff with an ugly rebase :)
|
@marcosnils yeah, he still has some stuff in development that would probably make your code different. Don't want to have you go to the trouble of doing it and then clobber your stuff with an ugly rebase :) |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
jwilder
Jul 22, 2015
Collaborator
@marcosnils Sorry for the delay. If you wouldn't mind waiting for #3372 to land that would be great. Hoping to get merge that in the next day or two.
|
@marcosnils Sorry for the delay. If you wouldn't mind waiting for #3372 to land that would be great. Hoping to get merge that in the next day or two. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
jwilder
Jul 23, 2015
Collaborator
@marcosnils #3372 has landed. If you want to rebase, we should be able to merge the fix in now.
|
@marcosnils #3372 has landed. If you want to rebase, we should be able to merge the fix in now. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
marcosnils
Jul 24, 2015
Contributor
@beckettsean @jwilder @pauldix this is no longer an issue. Just tried it with master branch code and works. I guess this can be closed when next release is available.
|
@beckettsean @jwilder @pauldix this is no longer an issue. Just tried it with master branch code and works. I guess this can be closed when next release is available. |
jwilder
closed this
Jul 24, 2015
jwilder
removed
the
2 - Working
label
Jul 24, 2015
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
sagetrey
Dec 15, 2015
It is still an issue. when connecting to localhost:8086 I get a connection refused. Doesn't happen when I connect to it from outside.
sagetrey
commented
Dec 15, 2015
|
It is still an issue. when connecting to localhost:8086 I get a connection refused. Doesn't happen when I connect to it from outside. |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
marcosnils
Dec 15, 2015
Contributor
@sagetrey please provide more information about which influx version you're running, the specific error you're getting (log trace) and under which address service parameter you changed. It'd be also good to know if you are configuring a new cluster or an already existing one.
|
@sagetrey please provide more information about which influx version you're running, the specific error you're getting (log trace) and under which |
This comment has been minimized.
Show comment
Hide comment
This comment has been minimized.
kolargol
Sep 20, 2018
Reopening this as this is still issue. Influx when configured with bind-address = ":123" listen only on ipv6 interface and there is no way forcing it to use ipv4.
kolargol
commented
Sep 20, 2018
|
Reopening this as this is still issue. Influx when configured with bind-address = ":123" listen only on ipv6 interface and there is no way forcing it to use ipv4. |
beckettsean commentedMay 1, 2015
To issue from mailing list: https://groups.google.com/d/msgid/influxdb/455762cc-9669-4aa6-83ba-3df4e331dace%40googlegroups.com
I have issue after change "bind-address" parameter from "0.0.0.0" to particular IP (not localhost).
But unfortunatelly, influxdb can't run.
I don't know why influxdb still contact localhost not IP from bind-address
Regards,
--Rizal