From a93ce3e0aabf639967bba1c0e5b7314b9edf6993 Mon Sep 17 00:00:00 2001 From: Kaustubh Dhokte Date: Fri, 21 Feb 2025 23:50:45 +0000 Subject: [PATCH] Debian: correct etcd conf file path used in collectd This change corrects etcd config file path in /etc/collect.d/collect_containerization. The file still uses etcd config file path used on centos i.e. /etc/etcd/etcd.conf. The same on Debian (new StarlingX base OS) is /etc/default/etcd. With this correction, now collectd can correctly dump etcd database. Test Plan: AIO-SX: PASS: Run 'collect --all', etcd database dumped correctly under var/extra. Manually check the contents of var/extra/etcd_database.dump. Closes-Bug: 2071866 Change-Id: I91cb7a6dac503c4eee034a068245d2da61267f54 Signed-off-by: Kaustubh Dhokte --- tools/collector/debian-scripts/collect_containerization.sh | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/collector/debian-scripts/collect_containerization.sh b/tools/collector/debian-scripts/collect_containerization.sh index a3585928..44704b1c 100755 --- a/tools/collector/debian-scripts/collect_containerization.sh +++ b/tools/collector/debian-scripts/collect_containerization.sh @@ -156,7 +156,7 @@ if [ "$nodetype" = "controller" -a "${ACTIVE}" = true ] ; then CMD="cp -r /opt/platform/helm_charts ${HELM_DIR}/" run_command "${CMD}" "${LOGFILE}" - export $(grep '^ETCD_LISTEN_CLIENT_URLS=' /etc/etcd/etcd.conf | tr -d '"') + export $(grep '^ETCD_LISTEN_CLIENT_URLS=' /etc/default/etcd | tr -d '"') CMD="sudo ETCDCTL_API=3 etcdctl \ --endpoints=$ETCD_LISTEN_CLIENT_URLS get / --prefix"