Skip to content

feat(splunk-o11y): add Forge impact dynamic variables#389

Merged
edersonbrilhante merged 1 commit into
mainfrom
add-forge-impact-dynamic-variables
Jul 3, 2026
Merged

feat(splunk-o11y): add Forge impact dynamic variables#389
edersonbrilhante merged 1 commit into
mainfrom
add-forge-impact-dynamic-variables

Conversation

@edersonbrilhante

Copy link
Copy Markdown
Member

Description

Adds dynamic dashboard variable support to the ForgeCICD Impact dashboard in the shared Splunk Observability configuration module.

  • Adds the dynamic_variables input to the forge_impact dashboard submodule.
  • Renders each provided dynamic variable on the Signalfx dashboard.
  • Adds optional parent dashboard_variables.forge_impact support while falling back to the existing runner K8S tenant names for current callers.
  • Regenerates the shared module README.

Type of Change

  • Feature
  • Bug Fix
  • Documentation
  • Refactor
  • Other: ________

Testing

  • terraform fmt -check modules/integrations/splunk_o11y_conf_shared/variables.tf modules/integrations/splunk_o11y_conf_shared/dashboards.tf modules/integrations/splunk_o11y_conf_shared/dashboards/forge_impact/main.tf modules/integrations/splunk_o11y_conf_shared/dashboards/forge_impact/variables.tf
  • git diff --check
  • Commit pre-commit hooks passed, including Terraform/Tofu fmt and validate hooks.

@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@edersonbrilhante edersonbrilhante marked this pull request as ready for review July 3, 2026 12:35
@edersonbrilhante edersonbrilhante merged commit ee1bb79 into main Jul 3, 2026
9 checks passed
@edersonbrilhante edersonbrilhante deleted the add-forge-impact-dynamic-variables branch July 3, 2026 12:41
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.

1 participant