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

ipv6 addresses not automatically wrapped #1944

Closed
wombat opened this Issue Sep 5, 2016 · 5 comments

Comments

Projects
None yet
3 participants
@wombat
Copy link

wombat commented Sep 5, 2016

What did you do?
We installed Kubernetes and Prometheus on top of Rackspace and seeing sometimes ipv4, sometimes ipv6 addresses for nodes, service and pod ips.

What did you expect to see?
All endpoints either declared explicitly or via autodiscovery should be in state UP

What did you see instead? Under which circumstances?
Endpoints with a ipv6 addresses have a state DOWN and the error says Get http://xxxx:xxxx:xxxx:xxx:xxxx:xxxx:xxxx:xxx:9100/metrics: dial tcp: too many colons in address xxxx:xxxx:xxxx:xxx:xxxx:xxxx:xxxx:xxx:9100

Environment

  • System information:
    Linux 4.7.0-coreos x86_64
  • Prometheus version:
    prometheus, version 1.1.0 (branch: master, revision: a60378c)
    build user: root@583368332c21
    build date: 20160903-19:35:49
    go version: go1.6.3
  • Alertmanager version:

none

  • Prometheus configuration file:

https://raw.githubusercontent.com/prometheus/prometheus/v1.1.0/documentation/examples/prometheus-kubernetes.yml

  • Alertmanager configuration file:
    none
  • Logs:
    none
  • Screens:
    bildschirmfoto 2016-09-05 um 12 18 47
@fabxc

This comment has been minimized.

Copy link
Member

fabxc commented Sep 5, 2016

I see where we are rendering them wrong in the UI. However, for scraping we are using URL.String(), which handles ipv6 addresses correctly.

@jimmidyson

This comment has been minimized.

Copy link
Member

jimmidyson commented Sep 5, 2016

@fabxc Aren't we using IP.String() which doesn't handle it properly? It appears it doesn't... https://play.golang.org/p/Q8ikNMh4aF Haven't looked properly though...

@fabxc

This comment has been minimized.

Copy link
Member

fabxc commented Sep 5, 2016

Ah, yes. Just walked to the Prometheus code. Quite a few places where unexpected things could happen with ipv6. Will work on a fix.

@fabxc fabxc referenced this issue Sep 5, 2016

Merged

Fix IPv6 scraping #1946

@fabxc fabxc closed this in #1946 Sep 6, 2016

@lock

This comment has been minimized.

Copy link

lock bot commented Mar 24, 2019

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked and limited conversation to collaborators Mar 24, 2019

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
You can’t perform that action at this time.