dduk-ddak / coding-night-live

Web-based Communication Application for Codelabs
Other
62 stars 11 forks source link

Issue #80 #84

Closed punkyoon closed 7 years ago

punkyoon commented 7 years ago

python3 deploy_helper.py실행 시, pip 설치부터 nginx 설치, DB migration 등 서버 배포에 필요한 모든 과정을 자동으로 실행. 사용자에게 입력 받는 부분은 OAuth 설정 관련한 것 이외에는 없음.

python3 undeploy_helper.py실행 시, 서비스 종료(redis&nginx) 및 프로세스 종료 실행. 서버 종료에 필요한 모든 과정을 자동으로 실행.

minhoryang commented 7 years ago

코딩스타일, posix에러코드, 템플릿 추출, 설정보관 등등 여러가지 리뷰항목이 있는데요.

이 파일의 예상 사용자를 누구로 잡았는지가 궁금합니다. (디버깅/테스트용인지, 개인 배포 자동화용인지, 외부인 배포 자동화용인지, ...)

juice500ml commented 7 years ago

60 은 어떤 식으로 적용되었나요?

minhoryang commented 7 years ago

@juice500ml #60은 DEBUG=false 로 다 끝난거 아니었나요?

punkyoon commented 7 years ago

@juice500ml #60 은 이제 nginx에서 admin에 접속 가능한 ip를 별도로 설정해주면 해결 가능한 부분입니다. 이에 더해서 urls.py를 수정하여 /admin이 아닌 다른 주소로 admin page에 접속하는 것을 권장하는 메뉴얼을 만들어줘야할 것 같습니다.

minhoryang commented 7 years ago

제가 커밋을 하나 작성해 보았는데요. 한번 확인해 주실 수 있나요? https://github.com/minhoryang/coding-night-live/commit/a135e31fb5bc0c87d2e94023e04a7435b96bd6fd

punkyoon commented 7 years ago

여러분 제가 다 해결하고 오겠습니다...뚝..딱..뚝..딱

punkyoon commented 7 years ago

@minhoryang @juice500ml 리뷰받은 부분에 대해서는 거의 해결해두었습니다.

minhoryang commented 7 years ago

https://docs.djangoproject.com/en/1.10/howto/custom-management-commands/#django.core.management.BaseCommand.can_import_settings

minhoryang commented 7 years ago

제가 nginx/local_nginx.conf 를 생성하는 코드를 짜보았는데요. https://github.com/minhoryang/coding-night-live/commit/935553461f995e0476e1daa27ab156fb5b5c9315

한번 리뷰해주시겠어요?

minhoryang commented 7 years ago

daphne가 killall로 죽지않아요 ㅠㅠ

minhoryang commented 7 years ago

제가 makefile를 조금 더 건드려봤는데, 매우 구질구질하네요 https://github.com/minhoryang/coding-night-live/commit/9c6cb686b1b3af2f237635fb8abf3fe3e9b5c9d5 만약 싫으시면 무시하시고, https://github.com/dduk-ddak/coding-night-live/pull/84/commits/be68a4c21093ecce12d5cd9e5539a21515a0e065 만 revert 시키면 될 것 같아요.

punkyoon commented 7 years ago

daphne가 killall로 안죽다니..! 처음 접해보는 문제군요!

minhoryang commented 7 years ago

@punkyoon OS X 에서는 daphne가 python뒤에 붙어서 켜지더라구욧.

punkyoon commented 7 years ago

제가 너무 리눅스에 맞춰서 개발한 탓이네요..ㅠ_ㅠ

minhoryang commented 7 years ago

@punkyoon 아니에요아니에요!!!! 리눅스에서 잘 되면 다행인거죠! 무슨 포인트가 달랐는지 알아보고있어요!

juice500ml commented 7 years ago

python locale도 잡아줘야 할까요? 뭔가 도를 넘은 것 같기도..

punkyoon commented 7 years ago

@juice500ml 아직은 안잡아줘도 될 거 같아요!

minhoryang commented 7 years ago

Makefile 테스트만 한번 더 해주시겠어요?

minhoryang commented 7 years ago

제가 해보았습니다!