You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2022/08/26 19:34:42 UTC
[GitHub] [accumulo] dlmarion opened a new pull request, #2899: Added property to enable Monitor web server to bind to all interfaces
dlmarion opened a new pull request, #2899:
URL: https://github.com/apache/accumulo/pull/2899
Currently the Monitor uses either the supplied address (-a arg) or the resolved
address for the hostname for advertising it's location in ZooKeeper and the
bind address for the embedded Jetty web server. This commit introduces a new
property that will set the bind address to "0.0.0.0" for the Jetty web server
but will leave the Monitor's ZooKeeper address unchanged. This change is being
made because of some constraints in containerized environments
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo] dlmarion closed pull request #2899: Added property to enable Monitor web server to bind to all interfaces
Posted by GitBox <gi...@apache.org>.
dlmarion closed pull request #2899: Added property to enable Monitor web server to bind to all interfaces
URL: https://github.com/apache/accumulo/pull/2899
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo] dlmarion commented on pull request #2899: Added property to enable Monitor web server to bind to all interfaces
Posted by GitBox <gi...@apache.org>.
dlmarion commented on PR #2899:
URL: https://github.com/apache/accumulo/pull/2899#issuecomment-1230424921
Confirmed that using `-a 0.0.0.0` does work. ZK contained:
```
get /accumulo/836a122d-320f-47e9-afbb-58aaefd4310c/monitor/http_addr
http://accumulo-monitor-744588d856-6dh7b:9995/
```
So, it's not using the IP address of the hostname, it's using the hostname.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo] dlmarion commented on pull request #2899: Added property to enable Monitor web server to bind to all interfaces
Posted by GitBox <gi...@apache.org>.
dlmarion commented on PR #2899:
URL: https://github.com/apache/accumulo/pull/2899#issuecomment-1230334957
> Isn't this something that should be done in the `/etc/hosts` file?
I'm not sure I understand how that solves this issue. If you start a server with an ip address (`-a 1.2.3.4` for example) or the `-a` argument is not supplied and the ip address is resolved from the hostname (`1.2.3.4` for example), then the server will bind to ip address `1.2.3.4` *only*.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo] dlmarion commented on pull request #2899: Added property to enable Monitor web server to bind to all interfaces
Posted by GitBox <gi...@apache.org>.
dlmarion commented on PR #2899:
URL: https://github.com/apache/accumulo/pull/2899#issuecomment-1230404437
Ok, looking at the code again, I see that I had the order for starting the Jetty web server and the insert of the node into ZK backwards.
If I use `-a 0.0.0.0`, then the web server will bind to `0.0.0.0` and the ZK entry will reflect the hostname's resolved address.
If I use `-a 1.2.3.4`, then the web server will bind to `1.2.3.4` and the ZK entry will be `1.2.3.4`
The latter won't work; the issue I ran into was [this](https://serverfault.com/questions/1057081/why-does-kubectl-port-forward-require-the-destination-service-to-be-bound-to-loc) one. I used `-a 1.2.3.4` when starting the Monitor and both the Jetty web server and ZK address used that address. I could not reach the monitor in this case.
I still think this property, or one that allows the user to specify the monitor bind address, would be useful. In a situation where you have the Monitor running on a host with multiple NIC's, and you have traffic separated.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo] milleruntime commented on pull request #2899: Added property to enable Monitor web server to bind to all interfaces
Posted by GitBox <gi...@apache.org>.
milleruntime commented on PR #2899:
URL: https://github.com/apache/accumulo/pull/2899#issuecomment-1230327201
Isn't this something that should be done in the `/etc/hosts` file?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo] milleruntime commented on pull request #2899: Added property to enable Monitor web server to bind to all interfaces
Posted by GitBox <gi...@apache.org>.
milleruntime commented on PR #2899:
URL: https://github.com/apache/accumulo/pull/2899#issuecomment-1230352787
> > Isn't this something that should be done in the `/etc/hosts` file?
>
> I'm not sure I understand how that solves this issue. If you start a server with an ip address (`-a 1.2.3.4` for example) or the `-a` argument is not supplied and the ip address is resolved from the hostname (`1.2.3.4` for example), then the server will bind to ip address `1.2.3.4` _only_.
>
> Edit: It was also put the address `1.2.3.4` in ZooKeeper as the location for that server.
Couldn't you just start the Monitor using `-a 0.0.0.0`? I guess I don't understand what problem this is trying to solve. Can you explain what you mean by "some constraints in containerized environments"?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [accumulo] dlmarion commented on pull request #2899: Added property to enable Monitor web server to bind to all interfaces
Posted by GitBox <gi...@apache.org>.
dlmarion commented on PR #2899:
URL: https://github.com/apache/accumulo/pull/2899#issuecomment-1230456991
I'm closing this PR. Setting the address for the Monitor process to 0.0.0.0 resolves this issue.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org