Make pod_name and pod_namespace optional

[mneverov]

Currently, POD_NAME and POD_NAMESPACE environment variables are mandatory. As a user I don't want to specify them but just run the controller.
Investigate if

• we need to use resource lock
• it is possible to deduct name/namespace from os.host or by other means

Update readme with both variables, descriptions and default values from the investigation.

[aojea]

actually those need to be passed as parameters

node-ipam-controller/pkg/leaderelection/leaderelection.go

Lines 16 to 22 in 0a1e0fd

	type Config struct {
	LeaseDuration time.Duration
	RenewDeadline time.Duration
	RetryPeriod time.Duration
	ResourceLock string
	ResourceName string
	}

you can use some defaults , for namespace "default, and for id is commont to use os.Hostname()

[mneverov]

actually those need to be passed as parameters

The biggest problem at the moment that they are not specified and by default the controller runs in leader election mode, but without these variables it fails to start. Would be nice to address it

[aojea]

yeah, we can define some default values , pod_name = os.Hostname() and pod_namespace = "default" or "kube-system", whatever we use to install our controller, I can not remember now

[mneverov]

maybe also can be used staging/src/k8s.io/component-helpers/node/util/hostname.go:28

[aojea]

yeah, that works too

[k8s-triage-robot]

The Kubernetes project currently lacks enough contributors to adequately respond to all issues.

This bot triages un-triaged issues according to the following rules:

• After 90d of inactivity, lifecycle/stale is applied
• After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
• After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

• Mark this issue as fresh with /remove-lifecycle stale
• Close this issue with /close
• Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle stale