INTERNAL: Do not extern arcus_zk_shutdown

[namsic]

🔗 Related Issue

• jam2in/arcus-works#519
• jam2in/arcus-works#536

⌨️ What I did

• arcus_zk_shutdown flag를 static으로 변경합니다.

• zk_final, hb_final 시점을 앞당깁니다.

  • 기존: main thread event loop 종료 후
  • 변경: shutdown api 호출 시점

• arcus_zk_final() 함수 안에서 arcus_zk_shutdown 값을 변경합니다.

  • 기존: sm thread 종료(arcus_zk_shutdown = 1)와 zk 연결 종료(arcus_zk_final())를 구분 변경: arcus_zk_final() 호출하면 두 동작을 이어서 수행

[jhpark816]

@namsic 혹시 종료 테스트는 해 보았나요?

[namsic]

혹시 종료 테스트는 해 보았나요?

./memcached -E .libs/default_engine.so -z ncp-2c4-002:40000 -v -p 40010

위와 같은 명령으로 구동하고 kill 명령 실행하는 정도만 해 보았습니다.

Memcached is watching Arcus cache cloud for "/arcus/cache_list/sample"
update cluster config. (count=2)
server[0] = 10.33.111.24:40010-ncp-2c4-001
server[1] = 10.34.108.181:40010-ncp-2c4-002
...
memcached shutdown by signal(SIGTERM)
Main thread is now terminating from clock handler.
Initiating arcus memcached shutdown...
arcus_zk_final(graceful shutdown)
zk threads terminated
2024-04-17 21:11:30,955:631006(0x7fb307f94740):ZOO_INFO@zookeeper_close@3356: Closing zookeeper sessionId=0x206bfd5daea0018 to [10.34.108.181:40000]

2024-04-17 21:11:30,958:631006(0x7fb307f94740):ZOO_INFO@zookeeper_close@3378: Freeing zookeeper resources for sessionId=0x206bfd5daea0018

zk connection closed
Listen sockets closed.
Worker thread[0] is now terminating from libevent process.
Worker thread[1] is now terminating from libevent process.
Worker thread[2] is now terminating from libevent process.
Worker thread[3] is now terminating from libevent process.
Worker threads terminated.
ITEM base module destroyed.
ITEM list module destroyed.
ITEM set module destroyed.
ITEM map module destroyed.
ITEM btree module destroyed.
ITEM change log module destroyed.
ITEM module destroyed.
SLABS module destroyed.
ASSOC module destroyed.
PREFIX module destroyed.
Memcached engine destroyed.
arcus_zk_destroy
Arcus memcached terminated.

아직은 shutdown 명령이나 종료 전 대기 로직이 추가되지는 않아서 큰 차이 없을 것으로 생각했는데, 문제가 되거나 더 확인할 만한 시나리오가 있을까요?

[jhpark816]

@namsic 최소한의 테스트로, 위에 보인 정도로 확인하였는 지를 물어 본 것이며, 확인한 상태라서 바로 merge 하겠습니다.