Skip to content

fix: standardize component naming to hyphenated format - #421

Merged
DerekRoberts merged 2 commits into
mainfrom
copilot/fix-inconsistent-component-naming
Jan 27, 2026
Merged

DerekRoberts merged 2 commits into
mainfrom
copilot/fix-inconsistent-component-naming

Conversation

Copilot AI commented Dec 18, 2025 •

Copy link
Copy Markdown
Contributor

Description

Component names were inconsistent across the codebase: directories and images used backend-py/backend-java (correct), while Helm values, templates, and workflows used backendPy/backendJava/backendGo (incorrect) or backendpy/backendjava (incorrect).

Standardized all references to hyphenated format: backend-py, backend-java, backend-go.

Changes

Helm Charts (charts/app/)

  • Renamed template directories: backendPy/ → backend-py/, backendJava/ → backend-java/
  • Renamed template files: backendPy-*.yaml → backend-py-*.yaml, backendJava-*.yaml → backend-java-*.yaml
  • Updated values.yaml keys: backendPy: → backend-py:, backendJava: → backend-java:, backendGo: → backend-go:
  • Updated helper template functions and Values references to use (index .Values "backend-py") syntax
  • Updated Kubernetes labels: backendpy → backend-py, backendjava → backend-java

GitHub Workflows (.github/workflows/)

  • pentests.yml: Updated matrix names to use hyphenated format
  • merge.yml: Updated --set parameters and integration test URLs
  • pr-validate.yml: Updated markdown link URLs

Type of change

  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Breaking Changes

⚠️ Route hostnames and resource names will change:

  • Before: {release}-backendpy, {release}-backendjava
  • After: {release}-backend-py, {release}-backend-java

Existing deployments using Helm values with old naming (backendPy, backendJava, backendGo) must update to new naming (backend-py, backend-java, backend-go).

How Has This Been Tested?

  • Helm template syntax validation (helm template with test values)
  • Verified route hostnames render with hyphenated format
  • Verified Kubernetes labels use hyphenated format
  • CodeQL security scan passed
  • Code review completed

Checklist

  • I have read the CONTRIBUTING doc
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have already been accepted and merged

Further comments

27 files changed, 208 insertions(+), 208 deletions(-)—purely refactoring with no functional additions. All naming now consistent across directories, images, Helm charts, Kubernetes resources, and CI/CD workflows.

Original prompt

This section details on the original issue you should resolve

<issue_title>fix: inconsistent component naming</issue_title>
<issue_description>Our components do not follow consistent naming across templates, deployments, folders, ZAP scans and more. Make them consistent without breaking everything!

Good names:

  • backend-py
  • backend-java

Bad names:

  • backendpy
  • backendPy
  • backendjava
  • backendjava</issue_description>

Comments on the Issue (you are @copilot in this section)


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.


Thanks for the PR!

Deployments, as required, will be available below:

Please create PRs in draft mode. Mark as ready to enable:

After merge, new images are deployed in:

Copilot AI changed the title [WIP] Fix inconsistent component naming across templates and folders fix: standardize component naming to hyphenated format Dec 18, 2025
Copilot AI requested a review from DerekRoberts December 18, 2025 20:42
@DerekRoberts DerekRoberts moved this from New to Active in DevOps (NR) Dec 19, 2025
@DerekRoberts
DerekRoberts marked this pull request as ready for review January 15, 2026 18:00
Copilot AI review requested due to automatic review settings January 15, 2026 18:00

Copilot AI 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.

Pull request overview

This PR standardizes component naming across the codebase from inconsistent camelCase/lowercase formats (backendPy, backendJava, backendGo, backendpy, backendjava) to a consistent hyphenated format (backend-py, backend-java, backend-go).

Changes:

  • Renamed Helm template directories and files from backendPy/ and backendJava/ to backend-py/ and backend-java/
  • Updated values.yaml keys to use hyphenated format for all three backends
  • Modified GitHub workflow files to reference hyphenated component names in deployment parameters, test matrices, and URLs
  • Updated Kubernetes labels and network policies to use hyphenated naming

Reviewed changes

Copilot reviewed 25 out of 27 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
charts/app/values.yaml Renamed top-level keys from camelCase to hyphenated format
charts/app/templates/backend-py/* Created new hyphenated template directory with updated helper functions and value references using (index .Values "backend-py") syntax
charts/app/templates/backend-java/* Created new hyphenated template directory with updated helper functions and value references using (index .Values "backend-java") syntax
charts/app/templates/backendPy/* Removed old camelCase template files
charts/app/templates/backendJava/* Removed old camelCase template files
charts/app/templates/*-knp.yaml Updated Kubernetes Network Policy label selectors to use hyphenated format
charts/app/templates/*-secret.yaml Added new secret templates for hyphenated component names
.github/workflows/pr-validate.yml Updated deployment URLs to use hyphenated format
.github/workflows/pentests.yml Updated matrix strategy to use hyphenated component names
.github/workflows/merge.yml Updated Helm set parameters and integration test URLs to use hyphenated format

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread charts/app/values.yaml Outdated
Comment thread .github/workflows/merge.yml Outdated
Comment thread .github/workflows/pentests.yml Outdated
Copilot AI and others added 2 commits January 27, 2026 13:30
…nated format

Co-authored-by: DerekRoberts <4391600+DerekRoberts@users.noreply.github.com>
@DerekRoberts
DerekRoberts force-pushed the copilot/fix-inconsistent-component-naming branch from 8dab9f3 to 1f8c675 Compare January 27, 2026 21:32
@DerekRoberts
DerekRoberts merged commit d79939e into main Jan 27, 2026
17 checks passed
@DerekRoberts
DerekRoberts deleted the copilot/fix-inconsistent-component-naming branch January 27, 2026 21:45
@github-project-automation github-project-automation Bot moved this from Active to Done in DevOps (NR) Jan 27, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

fix: inconsistent component naming

3 participants