Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Docker compose 파일에서 Redis 제거 (#109) #110

Merged
merged 1 commit into from
Feb 19, 2024

Conversation

sungjindev
Copy link
Contributor

⭐️ Issue Number

🚩 Summary

최근에 검색어 자동 완성 기능에 대한 요구 사항이 변경되면서 기존에 사용하던 Redis 환경을 모두 제거하는 작업이 있었습니다. 그 과정에서 일부 Docker compose 파일에 반영이 안되어 있는 오류를 확인하였고, 이번 PR을 통해 이를 해결하려고 합니다.

  • docker-compose.yml에서 Redis 의존성 제거
  • docker-compose-dev.yml에서 Redis 의존성 제거

📋 To Do

  • 정상적으로 Dev환경에 CD가 되는지 테스트

@sungjindev sungjindev added the 🛠️ Fix 오류, 버그 수정 label Feb 19, 2024
@sungjindev sungjindev self-assigned this Feb 19, 2024
Copy link

Test Results

16 tests  ±0   16 ✅ ±0   15s ⏱️ ±0s
10 suites ±0    0 💤 ±0 
10 files   ±0    0 ❌ ±0 

Results for commit 87138a4. ± Comparison against base commit 7fd7d89.

Copy link
Contributor

@jinhoon227 jinhoon227 left a comment

Choose a reason for hiding this comment

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

승인합니다!

@sungjindev sungjindev merged commit aa99dc0 into develop Feb 19, 2024
3 checks passed
@sungjindev sungjindev deleted the fix/remove-redis-from-docker(#109) branch February 19, 2024 07:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🛠️ Fix 오류, 버그 수정
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants