Skip to content

Conversation

@sgh002400
Copy link
Contributor

변경사항

  • 홈서버 배포 자동화

상세내용

  • GitHub Actions에서 Docker Hub로 애플리케이션 이미지를 빌드 및 업로드
  • 홈서버에 SSH 접속
  • Docker Swarm을 통해 해당 서비스 롤링 업데이트 실행

연관 이슈

@sgh002400 sgh002400 requested a review from Copilot August 16, 2025 07:32
@sgh002400 sgh002400 self-assigned this Aug 16, 2025
Copy link

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 migrates the deployment strategy from AWS Elastic Beanstalk to a home server deployment using Docker Swarm with automated CI/CD through GitHub Actions.

  • Replaced AWS ECR/Beanstalk deployment with Docker Hub and home server SSH deployment
  • Updated Java runtime from JDK 17 to JDK 21 across build configurations
  • Removed local development Docker Compose for Kafka infrastructure
  • Modified API endpoints and security configuration for simplified testing

Reviewed Changes

Copilot reviewed 11 out of 12 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
.github/workflows/cd.yml Complete rewrite of deployment workflow to use Docker Hub and SSH deployment to home server
.github/workflows/ci.yml Updated JDK version from 17 to 21
Dockerfile Modernized with JDK 21 and updated build path for modular structure
docker-compose.yml Removed local Kafka development setup
Dockerrun.aws.json Removed AWS Beanstalk configuration file
Tradin-Api/src/main/resources/application-dev.yml Changed server port from 8081 to 8080 and fixed Redis configuration indentation
Tradin-Api/src/main/java/com/tradin/api/history/HistoryController.java Simplified endpoint to remove path parameter dependency
Tradin-Core/src/main/java/com/tradin/core/history/service/HistoryFacadeService.java Hardcoded strategy ID to 1L for testing purposes
Tradin-Core/src/main/java/com/tradin/core/strategy/service/StrategyService.java Updated test data with non-zero values
Security configuration files Added /v1/histories endpoint to permitAll lists

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

@Trading-Insight Trading-Insight deleted a comment from Copilot AI Aug 16, 2025
@Trading-Insight Trading-Insight deleted a comment from Copilot AI Aug 16, 2025
@Trading-Insight Trading-Insight deleted a comment from Copilot AI Aug 16, 2025
@Trading-Insight Trading-Insight deleted a comment from Copilot AI Aug 16, 2025
@Trading-Insight Trading-Insight deleted a comment from Copilot AI Aug 16, 2025
@sgh002400 sgh002400 marked this pull request as ready for review August 16, 2025 10:06
@sgh002400 sgh002400 merged commit 10f7818 into dev Aug 16, 2025
1 check failed
@sgh002400 sgh002400 deleted the feat/#37-auto-deploy branch August 16, 2025 10:06
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.

[FEAT] 홈서버 배포 자동화

2 participants