Skip to content

Conversation

@pantierra
Copy link
Contributor

@pantierra pantierra commented Nov 17, 2025

This PR reorganizes the Helm chart templates from a flat structure into logical modules for better maintainability:

  • Backward compatibility maintained. No changes to values.yaml or deployment configurations.
  • All tests passing (lint, unit tests, template generation)
  • Documentation integrated into chart's own README.md

New Structure:

  • /core - ServiceAccount, RBAC, Knative resources
  • /database - PostgreSQL and pgstac bootstrap templates
  • /networking - Ingress and middleware configurations
  • /monitoring - Observability stack (Prometheus, Grafana)
  • /services - API service deployments (unchanged)
  • /_helpers - Template helpers split by domain (core, database, services, validation)

@pantierra pantierra force-pushed the feature/reorganize-templates branch from 2d44ae7 to 41c0820 Compare November 17, 2025 12:23
@pantierra pantierra changed the title refactor: Reorganized template files. refactor: Reorganize template files. Nov 17, 2025
@pantierra pantierra marked this pull request as ready for review November 17, 2025 12:30
@pantierra pantierra force-pushed the feature/reorganize-templates branch from 41c0820 to 7b45f36 Compare November 17, 2025 12:32
@pantierra pantierra self-assigned this Nov 17, 2025
@pantierra pantierra force-pushed the feature/reorganize-templates branch from 7b45f36 to 0556334 Compare November 17, 2025 13:03
Copy link
Contributor

@emmanuelmathot emmanuelmathot left a comment

Choose a reason for hiding this comment

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

Love the new org. I can tell from the tests that it's all ok. Great job!

@pantierra pantierra merged commit 8f2e780 into main Nov 18, 2025
4 checks passed
@pantierra pantierra deleted the feature/reorganize-templates branch November 18, 2025 11:12
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