cec9f3c4d7
Fixes #801. The solution is complicated by the fact that the Selector field of Deployments is immutable. It would have been easy to just make the Selectors of the main Concierge Deployment, the Kube cert agent Deployment, and the various Services use more specific labels, but that would break upgrades. Instead, we make the Pod template labels and the Service selectors more specific, because those not immutable, and then handle the Deployment selectors in a special way. For the main Concierge and Supervisor Deployments, we cannot change their selectors, so they remain "app: app_name", and we make other changes to ensure that only the intended pods are selected. We keep the original "app" label on those pods and remove the "app" label from the pods of the Kube cert agent Deployment. By removing it from the Kube cert agent pods, there is no longer any chance that they will accidentally get selected by the main Concierge Deployment. For the Kube cert agent Deployment, we can change the immutable selector by deleting and recreating the Deployment. The new selector uses only the unique label that has always been applied to the pods of that deployment. Upon recreation, these pods no longer have the "app" label, so they will not be selected by the main Concierge Deployment's selector. The selector of all Services have been updated to use new labels to more specifically target the intended pods. For the Concierge Services, this will prevent them from accidentally including the Kube cert agent pods. For the Supervisor Services, we follow the same convention just to be consistent and to help future-proof the Supervisor app in case it ever has a second Deployment added to it. The selector of the auto-created impersonation proxy Service was also previously using the "app" label. There is no change to this Service because that label will now select the correct pods, since the Kube cert agent pods no longer have that label. It would be possible to update that selector to use the new more specific label, but then we would need to invent a way to pass that label into the controller, so it seemed like more work than was justified.
47 lines
1.1 KiB
YAML
47 lines
1.1 KiB
YAML
#! Copyright 2020-2021 the Pinniped contributors. All Rights Reserved.
|
|
#! SPDX-License-Identifier: Apache-2.0
|
|
|
|
#@ load("@ytt:data", "data")
|
|
#@ load("@ytt:template", "template")
|
|
|
|
#@ def defaultResourceName():
|
|
#@ return data.values.app_name
|
|
#@ end
|
|
|
|
#@ def defaultResourceNameWithSuffix(suffix):
|
|
#@ return data.values.app_name + "-" + suffix
|
|
#@ end
|
|
|
|
#@ def pinnipedDevAPIGroupWithPrefix(prefix):
|
|
#@ return prefix + "." + data.values.api_group_suffix
|
|
#@ end
|
|
|
|
#@ def namespace():
|
|
#@ if data.values.into_namespace:
|
|
#@ return data.values.into_namespace
|
|
#@ else:
|
|
#@ return data.values.namespace
|
|
#@ end
|
|
#@ end
|
|
|
|
#@ def defaultLabel():
|
|
app: #@ data.values.app_name
|
|
#@ end
|
|
|
|
#@ def deploymentPodLabel():
|
|
deployment.pinniped.dev: supervisor
|
|
#@ end
|
|
|
|
#@ def labels():
|
|
_: #@ template.replace(defaultLabel())
|
|
_: #@ template.replace(data.values.custom_labels)
|
|
#@ end
|
|
|
|
#@ def getAndValidateLogLevel():
|
|
#@ log_level = data.values.log_level
|
|
#@ if log_level != "info" and log_level != "debug" and log_level != "trace" and log_level != "all":
|
|
#@ fail("log_level '" + log_level + "' is invalid")
|
|
#@ end
|
|
#@ return log_level
|
|
#@ end
|