Skip to content

FIX: Fix memcached-init script files#802

Open
cheesecrust wants to merge 1 commit intonaver:developfrom
cheesecrust:internal/scripts
Open

FIX: Fix memcached-init script files#802
cheesecrust wants to merge 1 commit intonaver:developfrom
cheesecrust:internal/scripts

Conversation

@cheesecrust
Copy link
Contributor

@cheesecrust cheesecrust commented Nov 8, 2024

🔗 Related Issue

⌨️ What I did

  • scripts 파일 실행에 있어 현재 arcus-memcached server를 상대로 실행 할 수 있도록 하였습니다.

    • start-stop-daemon의 경우 linux Debian 계열에 있는 명령어로 centOs 또는 Mac 환경에는 존재하지 않아 초경량 배포판을 제외한 모든 배포판에 있는 nohup 명령을 사용하여 변경하였습니다.

    • 이에 따라 사용하지 않게 된 pid 파일 관련 구현을 삭제하였습니다.
      (start-stop-deamon의 경우 실행시 실행 pid를 pid 파일에 기록한 후 stop 시킬때 이를 활용하였는데, 이 pr 에서는 조회 한 후 grep 하여 정지시키도록 하였습니다.)

    • memcached.conf 파일은 memcached-init start 실행하여 memcached 서버를 구동할때 줄 옵션들을 적는 파일 입니다. 한 줄에 옵션 하니씩을 설정하게 되면 memcached server start 시에 해당 옵션들과 함께 구동됩니다.

*추가적으로 script를 실행하기 위해서는 script 파일들 안의 path들을 로컬 머신에 맞게 설정해야합니다.

@cheesecrust cheesecrust force-pushed the internal/scripts branch 3 times, most recently from 50bec26 to 249fdc5 Compare November 11, 2024 06:01
@cheesecrust cheesecrust changed the title FIX: Fix script files FIX: Fix memcached-init script files Nov 11, 2024
@namsic namsic removed their request for review November 12, 2024 01:05
@jhpark816
Copy link
Collaborator

@ing-eoking
본 PR을 대신하여 마무리하시죠.

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