starlingx/monitoring
Code Issues Proposed changes

Implement collectd startup in manifest apply post stage

Browse Source 
Starting collectd too early in the manifest apply is seen
to occasionally fail due to a dependency configuration on
hostname resolution in FQDNLookup not being complete.

Since influxdb is used by collectd and is a controller
only service this update moves it to the manifest apply
post stage as well and is filtered out from non
controller load types.

This issue is fixed by the following multi-git changes.

stx-metal:
   Filter influxdb out of storage and compute only loads.
   No real inter git merge dependency

stx-integ: This update.
   Add startup Before=pmond dependency

stx-config:
   Move collectd config and startup to manifest apply post stage
   Move influxdb config and startup to manifest apply post stage

Test Plan:
PASS: Build iso
PASS: verify install storage system and collectd startup
PASS: Verify Storage system DOR
PASS: Verify influxdb and extensions excluded in non-controller loads
PASS: Verify collectd starts properly on all nodes (CC,DOR,UNLOCK)
PASS: Verify influxdb starts properly on controller nodes (CC,DOR,UNLOCK)
PASS: Verify collectd pmond process monitoring and recovery
PASS: Verify influxdb pmond process monitoring and recovery

PEND: Verify collectd statistics storage and fetch to/from influxdb
PEND: Install AIO DX and verify collectd and influxdb startup

Change-Id: I47d70b05bdbdd22f8fce2f56fcc287fac7371ace
Closes-Bug: 1797909
Signed-off-by: Eric MacDonald <eric.macdonald@windriver.com>
This commit is contained in:
Eric MacDonald 2019-01-02 10:21:06 -05:00
parent eb8ba2c3ab
commit 44b74e4066
4 changed files with 6 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
collectd-extensions/centos/build_srpm.data
View File

@ -16,4 +16,4 @@ COPY_LIST="$PKG_BASE/src/LICENSE \
$PKG_BASE/src/example.py \ $PKG_BASE/src/example.py \
$PKG_BASE/src/example.conf" $PKG_BASE/src/example.conf"
TIS_PATCH_VER=3 TIS_PATCH_VER=4

1
collectd-extensions/src/collectd.service
View File

@ -1,6 +1,7 @@
[Unit] [Unit]
Description=Collectd statistics daemon and extension services Description=Collectd statistics daemon and extension services
Documentation=man:collectd(1) man:collectd.conf(5) Documentation=man:collectd(1) man:collectd.conf(5)
Before=pmon.service
After=local-fs.target network-online.target After=local-fs.target network-online.target
Requires=local-fs.target network-online.target Requires=local-fs.target network-online.target

2
influxdb-extensions/centos/build_srpm.data
View File

@ -4,4 +4,4 @@ COPY_LIST="$PKG_BASE/src/LICENSE \
$PKG_BASE/src/influxdb.conf.pmon \ $PKG_BASE/src/influxdb.conf.pmon \
$PKG_BASE/src/influxdb.service" $PKG_BASE/src/influxdb.service"
TIS_PATCH_VER=1 TIS_PATCH_VER=2

4
influxdb-extensions/src/influxdb.service
View File

@ -1,8 +1,10 @@
[Unit] [Unit]
Description=InfluxDB open-source, distributed, time series database Description=InfluxDB open-source, distributed, time series database
Documentation=https://influxdb.com/docs/ Documentation=https://influxdb.com/docs/
After=local-fs.target network.target
Before=collectd.service Before=collectd.service
Before=pmon.service
After=local-fs.target network-online.target
Requires=local-fs.target network-online.target
[Service] [Service]
User=influxdb User=influxdb