-
Notifications
You must be signed in to change notification settings - Fork 3k
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
hubble-relay: deprecate peer svc through local unix domain socket #23407
hubble-relay: deprecate peer svc through local unix domain socket #23407
Conversation
cade90c
to
1bea450
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Given v1.13 is not out, let's backport this documentation change/deprecation notice to v1.13 so that we can remove the unix domain socket support in Relay quicker (approve offline by @aanm).
1bea450
to
8a2f4d2
Compare
8a2f4d2
to
cbc6f60
Compare
Travis-CI hit #23314 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✔️
Hubble server has been integrated into the Cilium agent since v1.9, released Nov 10th 2020. Some doc fixes and improvements on the way. Signed-off-by: Alexandre Perrin <alex@isovalent.com>
Since Cilium v1.12 Hubble Relay has been connecting to the Hubble Peer service through the hubble-peer Kubernetes Service (i.e. TCP) by default. The ability to query the Peer Service through the local UNIX domain socket for Relay has been kept for compatibility reasons and as a fallback solution, but we should move away from sharing socket through Pods as it has potential security implications. This patch deprecate disabling the hubble-peer Kubernetes Service as it will be the only option once the local UNIX domain support for the Peer Service in Hubble Relay is removed. Signed-off-by: Alexandre Perrin <alex@isovalent.com>
cbc6f60
to
7af198e
Compare
Doc change only, no need for a full CI run so marking this one as |
As suggested here, requested by @aanm.