10272
39%

Ran 22 Aug 2019 03:07PM UTC

Jobs 1

Files 438

Run time 31s

Badge

Embed ▾

Committed 22 Aug 2019 02:40PM UTC coverage: 44.063% (+0.007%) from 44.056%

Build # 10272

Build Type

push

travis-ci-com

Committed by

brb

Commit Message

helm: Allow to specify k8s api-server host and port via env vars

Currently, there are three ways how k8s api-server address can be
passed to cilium-agent:

1. Default: k8s passes it to cilium-agent Pod via KUBERNETES_SERVICE_{HOST,PORT}
   env vars. Usually, it's a ClusterIP which is translated to the master
   node IP address via iptables rules installed by kube-proxy.
2. Via the `--k8s-kubeconfig-path` cilium-agent param. This way seems to be
   unnecessary too complicated because the kubeconfig has to be distributed
   among all cilium nodes, and cilium-agent needs only IP addr + port of
   the api service, as the service account credentials are already
   mounted into cilium-agent Pod [1].
3. Via the `--k8s-api-server` cilium-agent param. Unfortunately, this way
   provides http-only access to the api-server, as pkg/k8s/config.go
   does not try to set certificate authority and token when
   `--k8s-api-server` is set.

This commit introduces a fourth way to pass the address to the api-server
by overriding KUBERNETES_SERVICE_{HOST,PORT} env vars with helm. This is
a temporary workaround needed for the kube-proxy free getting started
guide until the third way has been fixed.

[1]: https://kubernetes.io/docs/tasks/access-application-cluster/access-cluster/#accessing-the-api-from-a-pod

Signed-off-by: Martynas Pumputis <m@lambda.lt>

Coverage Stats

25438 of 57731 relevant lines covered (44.06%)

1181.86 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
1	10272.1	22 Aug 2019 03:07PM UTC	0	44.06	Travis Job 10272.1

Source Files on build 10272

Detailed source file information is not available for this build.

cilium / cilium / 10272
39%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 10272

cilium / cilium / 10272 39%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 10272

cilium / cilium / 10272
39%

README BADGES
x