Skip to content

feat: Add docker build and helm chart#18

Merged
dealako merged 6 commits intomainfrom
asherman/helm-chart
Jul 31, 2025
Merged

feat: Add docker build and helm chart#18
dealako merged 6 commits intomainfrom
asherman/helm-chart

Conversation

@AlanSherman
Copy link
Contributor

@AlanSherman AlanSherman commented Jul 30, 2025

  • Add GitHub Actions workflows for Docker image builds:
    • docker-build-main.yml: builds on pushes to main branch
    • docker-build-tag.yml: builds on tag creation for releases
  • Add complete Helm chart for Kubernetes deployment:
    • Chart.yaml with metadata and version info
    • Comprehensive values.yaml with environment configuration
    • Template files for all Kubernetes resources:
      • Deployment with configurable replicas and environment variables
      • Service with ClusterIP type and port configuration
      • Ingress with TLS support and path-based routing
      • ServiceAccount with RBAC configuration
    • Helper templates for consistent naming and labeling
  • Configure environment variables for Auth0, Supabase, and PCC services
  • Set up proper resource limits and health checks
  • Include comprehensive README with deployment instructions

This enables containerized deployment to Kubernetes environments with
proper CI/CD integration for automated image builds.

Copilot AI review requested due to automatic review settings July 30, 2025 15:12
@AlanSherman AlanSherman requested a review from jordane as a code owner July 30, 2025 15:12

This comment was marked as outdated.

@AlanSherman AlanSherman changed the title Add docker build and helm chart WIP: Add docker build and helm chart Jul 30, 2025
@AlanSherman AlanSherman force-pushed the asherman/helm-chart branch from 33c0a5a to 3c7b611 Compare July 30, 2025 15:43
@AlanSherman AlanSherman force-pushed the asherman/helm-chart branch 17 times, most recently from bd59c9a to 06c5b5b Compare July 31, 2025 21:20
- Add GitHub Actions workflows for Docker image builds:
  - docker-build-main.yml: builds on pushes to main branch
  - docker-build-tag.yml: builds on tag creation for releases
- Add complete Helm chart for Kubernetes deployment:
  - Chart.yaml with metadata and version info
  - Comprehensive values.yaml with environment configuration
  - Template files for all Kubernetes resources:
    - Deployment with configurable replicas and environment variables
    - Service with ClusterIP type and port configuration
    - Ingress with TLS support and path-based routing
    - ServiceAccount with RBAC configuration
  - Helper templates for consistent naming and labeling
- Configure environment variables for Auth0, Supabase, and PCC services
- Set up proper resource limits and health checks
- Include comprehensive README with deployment instructions

This enables containerized deployment to Kubernetes environments with
proper CI/CD integration for automated image builds.

Signed-off-by: Alan Sherman <asherman@linuxfoundation.org>
@AlanSherman AlanSherman force-pushed the asherman/helm-chart branch from 06c5b5b to 39b98d9 Compare July 31, 2025 21:21
@AlanSherman AlanSherman changed the title WIP: Add docker build and helm chart feat: Add docker build and helm chart Jul 31, 2025
@AlanSherman AlanSherman requested a review from Copilot July 31, 2025 21:24

This comment was marked as outdated.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Alan Sherman <alan@alansherman.org>
@AlanSherman AlanSherman requested a review from Copilot July 31, 2025 21:32

This comment was marked as outdated.

Signed-off-by: Alan Sherman <asherman@linuxfoundation.org>
@AlanSherman AlanSherman force-pushed the asherman/helm-chart branch from b837afd to e9e6e73 Compare July 31, 2025 21:34
@AlanSherman AlanSherman requested a review from Copilot July 31, 2025 21:35
Copy link
Contributor

Copilot AI left a comment

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 adds complete containerization and Kubernetes deployment infrastructure for the LFX V2 PCC UI application by introducing Docker build workflows and a comprehensive Helm chart for Kubernetes deployment.

  • Adds GitHub Actions workflows for automated Docker image builds on main branch pushes and release tags
  • Creates a complete Helm chart with configurable deployment templates for all Kubernetes resources
  • Configures environment variables for Auth0, Supabase, and PCC services with both direct values and secret references

Reviewed Changes

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

Show a summary per file
File Description
.github/workflows/docker-build-main.yml GitHub Actions workflow for building Docker images on main branch pushes
.github/workflows/docker-build-tag.yml GitHub Actions workflow for building and releasing Docker images and Helm charts on tag creation
charts/lfx-v2-pcc-ui/Chart.yaml Helm chart metadata and configuration
charts/lfx-v2-pcc-ui/values.yaml Default values and configuration for the Helm chart
charts/lfx-v2-pcc-ui/templates/_helpers.tpl Helm template helper functions for consistent naming and labeling
charts/lfx-v2-pcc-ui/templates/deployment.yaml Kubernetes Deployment template with environment variable configuration
charts/lfx-v2-pcc-ui/templates/service.yaml Kubernetes Service template for exposing the application
charts/lfx-v2-pcc-ui/templates/serviceaccount.yaml Kubernetes ServiceAccount template with RBAC configuration
charts/lfx-v2-pcc-ui/templates/ingress.yaml Kubernetes Ingress template for external access and TLS support
charts/lfx-v2-pcc-ui/README.md Documentation for chart configuration and deployment

AlanSherman and others added 2 commits July 31, 2025 14:38
Signed-off-by: Alan Sherman <asherman@linuxfoundation.org>
@dealako dealako merged commit 4d549ab into main Jul 31, 2025
5 checks passed
@dealako dealako deleted the asherman/helm-chart branch July 31, 2025 21:57
@github-actions
Copy link

✅ E2E Tests Passed

Browser: chromium
Status: passed

All E2E tests passed successfully.

Test Configuration

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.

4 participants