Enhance kubeconfig loading

[bouskaJ]

Enable using HOME environment variable to load kubernetes config.

Previous code was causing issues when running client in docker container wit random userId.

See

panic: user: unknown userid 1102770000

goroutine 1 [running]:
github.com/apache/camel-k/pkg/client.initialize(0x0, 0x0)
	/home/jenkins/workspace/fuse-camel-k-install/pkg/client/client.go:147 +0x123
github.com/apache/camel-k/pkg/client.NewOutOfClusterClient(0x0, 0x0, 0x1ad7100, 0x15, 0xc00012c58b, 0x3)
	/home/jenkins/workspace/fuse-camel-k-install/pkg/client/client.go:74 +0x35
github.com/apache/camel-k/e2e.newTestClient(...)
	/home/jenkins/workspace/fuse-camel-k-install/e2e/test_support.go:82
github.com/apache/camel-k/e2e.init.1()
	/home/jenkins/workspace/fuse-camel-k-install/e2e/test_support.go:72 +0xfd
FAIL	github.com/apache/camel-k/e2e	0.268s
FAIL

[bouskaJ]

This could break https://github.com/apache/camel-k/blob/master/pkg/client/client.go#L213. DON'T MERGE

[nicolaferraro]

@bouskaJ are you experiencing issues inside a container? I though that was fixed in 1.0.0-M4, including case of random user ID.

[bouskaJ]

Hi @nicolaferraro. I have a special case. I am in OpenShift container and I would like to run Camel-K against different OpenShift. So I am "in Cluster" but I would like to use "out of cluster" initialization.

What I did.

1. I have set KUBECONF ENV
   -> shouldUseContainerMode returns false (

   camel-k/pkg/client/client.go

   Line 207 in 21eb85b

   return false, nil

   )
   -> then I continue to

   camel-k/pkg/client/client.go

   Line 145 in 21eb85b

   kubeconfig, err = getDefaultKubeConfigFile()

   
   and here we are:

panic: user: unknown userid 1102770000

goroutine 1 [running]:
github.com/apache/camel-k/pkg/client.initialize(0x0, 0x0)
	/home/jenkins/workspace/fuse-camel-k-install/pkg/client/client.go:147 +0x123
github.com/apache/camel-k/pkg/client.NewOutOfClusterClient(0x0, 0x0, 0x1ad7100, 0x15, 0xc00012c58b, 0x3)
	/home/jenkins/workspace/fuse-camel-k-install/pkg/client/client.go:74 +0x35
github.com/apache/camel-k/e2e.newTestClient(...)
	/home/jenkins/workspace/fuse-camel-k-install/e2e/test_support.go:82
github.com/apache/camel-k/e2e.init.1()
	/home/jenkins/workspace/fuse-camel-k-install/e2e/test_support.go:72 +0xfd
FAIL	github.com/apache/camel-k/e2e	0.268s
FAIL

Maybe bouskaJ@104f8cd should fix it WDYT?

[bouskaJ]

Moreover we should use KUBECONF variable provided by user to load configuration.

[nicolaferraro]

Moreover we should use KUBECONF variable provided by user to load configuration.

Yes, that is what I expected.. if there's a KUBECONF that should take precedence and the code should not even look into the home directory

[bouskaJ]

Thank you for your opinion. Here is my new PR #1076