Skip to content

Add/eks version exporter - #2420

Merged
samidbb merged 3 commits into
masterfrom
add/eks-version-exporter
Jul 10, 2026
Merged

Add/eks version exporter#2420
samidbb merged 3 commits into
masterfrom
add/eks-version-exporter

Conversation

@samidbb

@samidbb samidbb commented Jul 9, 2026

Copy link
Copy Markdown
Contributor

Describe your changes

This pull request introduces a new Terraform module to automate deployment of the eks-version-exporter using GitOps practices with Flux CD which is done here dfds/platform-apps#600. The deployment is conditional, only occurring when Grafana is also deployed.

The most important changes are:

New Terraform module for EKS Version Exporter:

  • Added a new eks_version_exporter module in compute/k8s-services/main.tf, which is only instantiated when Grafana is enabled. This module automates the deployment of the EKS version exporter by managing manifests via GitHub and Flux CD.

Module implementation and configuration:

  • Defined required Terraform and provider versions in versions.tf to ensure compatibility, specifically with the GitHub provider.
  • Declared input variables in vars.tf for cluster and repository configuration, allowing flexible use across environments.
  • Added a data source in dependencies.tf to fetch the GitHub repository’s default branch and set up local variables for consistent file paths and naming.

GitOps manifest management:

  • Implemented resources in main.tf to generate and manage Flux CD-compatible YAML files (app-config.yaml and kustomization.yaml) in the appropriate GitHub repo paths, enabling automated deployment and updates.
  • Created template files for the necessary Kubernetes and Flux CD resources, including RBAC and Kustomization definitions, to be rendered and committed by Terraform. (values/app-config.yaml and values/kustomization.yaml) [1] [2]

Issue ticket number and link

Checklist before requesting a review

Is it a new release?

  • Apply a release tag release:(major|minor|patch), following semantic versioning in this guide or norelease if there is no changes to the Terraform code

@DFDS-Snyk

DFDS-Snyk commented Jul 9, 2026

Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues
Code Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@samidbb samidbb added the release:minor Triggers a minor release label Jul 9, 2026
@samidbb
samidbb marked this pull request as ready for review July 9, 2026 13:58
@samidbb
samidbb requested a review from a team as a code owner July 9, 2026 13:58

@avnes avnes left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This looks correct to me

@samidbb
samidbb merged commit bdaa880 into master Jul 10, 2026
36 of 38 checks passed
@samidbb
samidbb deleted the add/eks-version-exporter branch July 10, 2026 07:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release:minor Triggers a minor release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants