Skip to content

feat(telegraf): Prepare secret store docs#7247

Open
srebhan wants to merge 3 commits into
masterfrom
secretstore
Open

feat(telegraf): Prepare secret store docs#7247
srebhan wants to merge 3 commits into
masterfrom
secretstore

Conversation

@srebhan
Copy link
Copy Markdown
Member

@srebhan srebhan commented May 18, 2026

Summary

This PR prepares the generation of secret store plugin documentation.

Checklist

@srebhan srebhan requested a review from a team as a code owner May 18, 2026 16:10
@srebhan srebhan requested review from jstirnaman and removed request for a team May 18, 2026 16:10
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 18, 2026

Vale Style Check Results

Metric Count
Errors 0
Warnings 0
Suggestions 0

Check passed

@github-actions github-actions Bot added the product:telegraf Telegraf documentation label May 18, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 18, 2026

PR Preview Action v1.4.8
🚀 Deployed preview to https://influxdata.github.io/docs-v2/pr-preview/pr-7247/
on branch gh-pages at 2026-05-18 16:37 UTC

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 18, 2026

📦 PR Preview — Preview Bot

Status Details
Result ✅ DEPLOYED
Preview View preview
Pages 1 page(s) deployed
Build time 74s
Last updated 2026-05-18 16:37:32 UTC
Pages included in this preview
  • /telegraf/v1/plugins/

Preview auto-deploys on push. Will be cleaned up when PR closes.

Copy link
Copy Markdown
Collaborator

@sanderson sanderson left a comment

Choose a reason for hiding this comment

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

Just a suggestion to the landing page text.

Comment thread data/list_filters.yml
- Output
- Aggregator
- Processor
- Secretstore
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.

I'd like this to have a space, but I have a feeling the way the filter JS is currently architected, it needs to be a single word 😄

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Unfortunately, the filter code does replace spaces with a dash but we would need to just remove the space for filtering. It's probably possible to parameterize the ListFilter function but that's beyond my skills.

Comment thread telegraf-build/templates/secretstore_index.tmpl Outdated
Co-authored-by: Scott Anderson <sanderson@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

product:telegraf Telegraf documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants