Confd doesn't replace telemetryd config in etc

Description

Telemetryd config on minion is usually placed in org.opennms.features.telemetry.listeners.*.cfg in etc folder.

Any updates through karaf shell gets updated in the config as well.

Confd takes a different route and configures telemetryd config through karaf features deployed in deploy folder.

Ideally it would be good just to replace the existing config in etc folder. 

 

Acceptance / Success Criteria

None

Lucidchart Diagrams

Activity

Show:

Stefan Wachter July 2, 2021 at 5:37 PM

Stefan Wachter July 1, 2021 at 1:51 PM

Maybe it is easier to just add the flow configuration that will automatically be part of the rpm and deb packages. When building the minion image that file would be excluded.

Benjamin Reed June 30, 2021 at 2:25 PM

I would say just put the "recommended" config in opennms-assemblies/minion/src/main/filtered/etc/ but add .example to the end of the filenames. Then we can make the Debian and RPM packages just rename those files from .example in their setup.

Chandra Gorantla June 23, 2021 at 1:57 PM

 

Note that we use etc/telemetryd-configuration.xml only on OpenNMS.
This problem is relevant only on Minion since we use .cfg files to configure listeners only on Minion.

Goal for confd is to emit config that is compatible with whenever user is going to change.
Currently, it installs configuration through features. It works as long as user doesn't attempt to change anything at runtime.
May be users doesn't usually change configuration at runtime for containers.

Updating flow listener configurations just on RPM distribution is not a bad idea. Let's check with .

Stefan Wachter June 23, 2021 at 6:25 AM

I wonder if the RPM distribution of Minion could simply contain default flow listener configurations (in form of files etc/org.opennms.features.telemetry.listeners-*.cfg)  whereas in the Minion image no such files are included.

Fixed

Details

Assignee

Reporter

HB Backlog Status

Internal Priority

Low High

Components

Sprint

Fix versions

Affects versions

Priority

PagerDuty

Created April 27, 2021 at 3:53 PM
Updated July 6, 2021 at 2:45 PM
Resolved July 6, 2021 at 1:57 PM

Flag notifications