Skip to content

Telegraf agent manage params#233

Open
m0dular wants to merge 9 commits into
puppetlabs:mainfrom
m0dular:telegraf_agent_manage_params
Open

Telegraf agent manage params#233
m0dular wants to merge 9 commits into
puppetlabs:mainfrom
m0dular:telegraf_agent_manage_params

Conversation

@m0dular

@m0dular m0dular commented Oct 24, 2025

Copy link
Copy Markdown
Contributor

This commit adds the following new features:

  • Add new management params
  • Add support for InfluxDB 1 in Telegraf outputs
  • Make file-sync metrics configurable
  • Add parameter for extra options to Telegraf inputs

These params control managing the Telegraf class, service,
and outputs.  It also adds one to manage token authentication.
This is still supported with enterprise InfluxDB
This commit adds $include_pe_metrics to the defined types and templates
to allow for configuring whether we want to include file sync metrics.
This commit adds a new parameter for passing additional hash elements to
the options parameter of the Telegraf inputs used in this module.
@m0dular m0dular requested review from a team and bastelfreak as code owners October 24, 2025 17:26
Comment thread CHANGELOG.md

All notable changes to this project will be documented in this file. The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/) and this project adheres to [Semantic Versioning](http://semver.org).

## [v2.4.0](https://github.com/puppetlabs/puppet_operational_dashboards/tree/v2.3.0)

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the changelog is generated automatically, please don't do it by hand

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is when the release workflow works, which it currently doesn't. I thought I'd have to manually do it at the moment?

Comment thread README.md Outdated
@MartyEwings

Copy link
Copy Markdown
Collaborator

Im unable to perform acceptance pipeline on this at the moment as work is being carried out on the provisioner backend

@m0dular

m0dular commented Oct 24, 2025

Copy link
Copy Markdown
Contributor Author

Is there an estimate for when that will be working again? It's been non-functional for about a year now. Alternatively, could there be a release without the acceptance tests?

This commit fixes the hardcoded database name and removed the version
string.  This is not needed anymore on newer postgresql versions
@MartyEwings

Copy link
Copy Markdown
Collaborator

@m0dular unit tests should at least run now, have a go

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants