Skip to content

Conversation

hippogr
Copy link
Contributor

@hippogr hippogr commented Aug 22, 2025

Add webportal-dind to run webportal in a container

@hippogr hippogr requested review from yukirora and zhogu August 22, 2025 04:58
@hippogr hippogr deleted the branch dev September 5, 2025 02:11
@hippogr hippogr closed this Sep 5, 2025
@hippogr hippogr reopened this Sep 5, 2025
@Copilot Copilot AI review requested due to automatic review settings September 5, 2025 05:24
@hippogr hippogr force-pushed the ruigao/webportal-dind branch from 505a7b8 to 0496a20 Compare September 5, 2025 05:24
Copy link

@Copilot 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 introduces webportal-dind (Docker-in-Docker) functionality to run the webportal service inside a containerized environment. This enables deployment of the webportal as a containerized service rather than directly on the host.

Key changes:

  • Adds containerized webportal deployment using Docker-in-Docker pattern
  • Creates Kubernetes DaemonSet configuration for webportal-dind deployment
  • Implements deployment scripts for lifecycle management (start, stop, delete, refresh)

Reviewed Changes

Copilot reviewed 8 out of 9 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
webportal-dind.yaml.template Kubernetes DaemonSet configuration for deploying webportal in containers
install.sh Docker installation and webportal container setup script
webportal-dind.common.dockerfile Base Dockerfile for the webportal-dind image
start.sh, stop.sh, delete.sh, refresh.sh Deployment lifecycle management scripts
service.yaml Service configuration defining deployment rules and scripts

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@hippogr hippogr merged commit 3991054 into dev Sep 11, 2025
1 check passed
This was referenced Sep 15, 2025
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.

2 participants