From ab1e9c200f43c32bb5617aa916d994b743b6dbbe Mon Sep 17 00:00:00 2001 From: "Madhusudan.C.S" Date: Fri, 12 Feb 2016 01:04:24 -0800 Subject: [PATCH] Graduate DaemonSet to v1beta1 API, enable it by default. --- cluster/aws/config-test.sh | 3 --- cluster/common.sh | 14 ++++---------- cluster/gce/config-test.sh | 1 - cluster/kubemark/config-default.sh | 3 --- docs/api.md | 2 +- pkg/master/master.go | 2 +- 6 files changed, 6 insertions(+), 19 deletions(-) diff --git a/cluster/aws/config-test.sh b/cluster/aws/config-test.sh index 770f777f27..050469ea9d 100755 --- a/cluster/aws/config-test.sh +++ b/cluster/aws/config-test.sh @@ -82,9 +82,6 @@ MASTER_IP_RANGE="${MASTER_IP_RANGE:-10.246.0.0/24}" MASTER_RESERVED_IP="${MASTER_RESERVED_IP:-}" RUNTIME_CONFIG="${KUBE_RUNTIME_CONFIG:-}" -# Enable various v1beta1 features -ENABLE_DAEMONSETS="${KUBE_ENABLE_DAEMONSETS:-true}" - # Optional: Cluster monitoring to setup as part of the cluster bring up: # none - No cluster monitoring setup # influxdb - Heapster, InfluxDB, and Grafana diff --git a/cluster/common.sh b/cluster/common.sh index 842ae363b5..6662a0f3f4 100755 --- a/cluster/common.sh +++ b/cluster/common.sh @@ -420,17 +420,11 @@ function yaml-quote { echo "'$(echo "${@}" | sed -e "s/'/''/g")'" } -# Builds the RUNTIME_CONFIG var from other feature enable options +# Builds the RUNTIME_CONFIG var from other feature enable options (such as +# features in alpha) function build-runtime-config() { - if [[ "${ENABLE_DAEMONSETS}" == "true" ]]; then - if [[ -z "${RUNTIME_CONFIG}" ]]; then - RUNTIME_CONFIG="extensions/v1beta1/daemonsets=true" - else - if echo "${RUNTIME_CONFIG}" | grep -q -v "extensions/v1beta1/daemonsets=true"; then - RUNTIME_CONFIG="${RUNTIME_CONFIG},extensions/v1beta1/daemonsets=true" - fi - fi - fi + # There is nothing to do here for now. Just using this function as a placeholder. + : } function write-master-env { diff --git a/cluster/gce/config-test.sh b/cluster/gce/config-test.sh index c85d949db4..c5980f33c5 100755 --- a/cluster/gce/config-test.sh +++ b/cluster/gce/config-test.sh @@ -51,7 +51,6 @@ RUNTIME_CONFIG="${KUBE_RUNTIME_CONFIG:-}" TERMINATED_POD_GC_THRESHOLD=${TERMINATED_POD_GC_THRESHOLD:-100} # Optional: enable v1beta1 related features -ENABLE_DAEMONSETS="${KUBE_ENABLE_DAEMONSETS:-true}" ENABLE_REPLICASETS="${KUBE_ENABLE_REPLICASETS:-true}" # Increase the sleep interval value if concerned about API rate limits. 3, in seconds, is the default. diff --git a/cluster/kubemark/config-default.sh b/cluster/kubemark/config-default.sh index 43fe163758..d0a4b39124 100644 --- a/cluster/kubemark/config-default.sh +++ b/cluster/kubemark/config-default.sh @@ -41,9 +41,6 @@ CLUSTER_IP_RANGE="${CLUSTER_IP_RANGE:-10.244.0.0/16}" RUNTIME_CONFIG="${KUBE_RUNTIME_CONFIG:-}" TERMINATED_POD_GC_THRESHOLD=${TERMINATED_POD_GC_THRESHOLD:-100} -# Optional: enable v1beta1 related features -ENABLE_DAEMONSETS="${KUBE_ENABLE_DAEMONSETS:-true}" - TEST_CLUSTER_LOG_LEVEL="${TEST_CLUSTER_LOG_LEVEL:---v=2}" TEST_CLUSTER_RESYNC_PERIOD="${TEST_CLUSTER_RESYNC_PERIOD:---min-resync-period=12h}" diff --git a/docs/api.md b/docs/api.md index d2f7c9aa81..7b42608109 100644 --- a/docs/api.md +++ b/docs/api.md @@ -113,7 +113,7 @@ create their own API groups](design/extending-api.md), and to avoid naming colli ## Enabling resources in the extensions group -HorizontalPodAutoscalers, Jobs, Ingress, Deployments and ReplicaSets are enabled by default. +DaemonSets, Deployments, HorizontalPodAutoscalers, Ingress, Jobs and ReplicaSets are enabled by default. Other extensions resources can be enabled by setting runtime-config on apiserver. runtime-config accepts comma separated values. For ex: to disable deployments and jobs, set `--runtime-config=extensions/v1beta1/deployments=false,extensions/v1beta1/jobs=false` diff --git a/pkg/master/master.go b/pkg/master/master.go index 5abfbd2023..49dd09b388 100644 --- a/pkg/master/master.go +++ b/pkg/master/master.go @@ -570,7 +570,7 @@ func (m *Master) thirdpartyapi(group, kind, version string) *apiserver.APIGroupV // getExperimentalResources returns the resources for extensions api func (m *Master) getExtensionResources(c *Config) map[string]rest.Storage { // All resources except these are disabled by default. - enabledResources := sets.NewString("horizontalpodautoscalers", "ingresses", "jobs", "replicasets", "deployments") + enabledResources := sets.NewString("daemonsets", "deployments", "horizontalpodautoscalers", "ingresses", "jobs", "replicasets") resourceOverrides := m.ApiGroupVersionOverrides["extensions/v1beta1"].ResourceOverrides isEnabled := func(resource string) bool { // Check if the resource has been overriden.