Skip to content

Commit 3ac763c

Browse files
authored
Merge pull request kubernetes#94056 from tosi3k/logexporter-per-zone
Enable per-zone logexporter pods creation
2 parents 3a7d710 + e453a1f commit 3ac763c

File tree

2 files changed

+6
-0
lines changed

2 files changed

+6
-0
lines changed

cluster/log-dump/log-dump.sh

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -503,13 +503,17 @@ function dump_nodes_with_logexporter() {
503503
local -r cloud_provider="${KUBERNETES_PROVIDER}"
504504
local -r enable_hollow_node_logs="${ENABLE_HOLLOW_NODE_LOGS:-false}"
505505
local -r logexport_sleep_seconds="$(( 90 + NUM_NODES / 3 ))"
506+
if [[ -z "${ZONE_NODE_SELECTOR_DISABLED:-}" ]]; then
507+
local -r node_selector="${ZONE_NODE_SELECTOR_LABEL:-topology.kubernetes.io/zone}: ${ZONE}"
508+
fi
506509

507510
# Fill in the parameters in the logexporter daemonset template.
508511
local -r tmp="${KUBE_TEMP}/logexporter"
509512
local -r manifest_yaml="${tmp}/logexporter-daemonset.yaml"
510513
mkdir -p "${tmp}"
511514
cp "${KUBE_ROOT}/cluster/log-dump/logexporter-daemonset.yaml" "${manifest_yaml}"
512515

516+
sed -i'' -e "s@{{.NodeSelector}}@${node_selector:-}@g" "${manifest_yaml}"
513517
sed -i'' -e "s@{{.LogexporterNamespace}}@${logexporter_namespace}@g" "${manifest_yaml}"
514518
sed -i'' -e "s@{{.ServiceAccountCredentials}}@${service_account_credentials}@g" "${manifest_yaml}"
515519
sed -i'' -e "s@{{.CloudProvider}}@${cloud_provider}@g" "${manifest_yaml}"

cluster/log-dump/logexporter-daemonset.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -67,6 +67,8 @@ spec:
6767
requests:
6868
cpu: 10m
6969
memory: 10Mi
70+
nodeSelector:
71+
{{.NodeSelector}}
7072
volumes:
7173
- name: service
7274
secret:

0 commit comments

Comments
 (0)