aicers / giganto

Raw-Event Storage System for AICE
Apache License 2.0
6 stars 3 forks source link

Use remote configuration directly #863

Closed BLYKIM closed 2 weeks ago

BLYKIM commented 1 month ago

Closes: #854 Closes: #866 Closes: #870

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 80.00000% with 27 lines in your changes missing coverage. Please review.

Project coverage is 76.98%. Comparing base (5f71aaa) to head (af7d8a0). Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
src/settings.rs 40.00% 12 Missing :warning:
src/main.rs 0.00% 6 Missing :warning:
src/peer.rs 64.28% 5 Missing :warning:
src/graphql/status.rs 95.06% 4 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #863 +/- ## ========================================== + Coverage 76.75% 76.98% +0.23% ========================================== Files 31 31 Lines 24907 24875 -32 ========================================== + Hits 19117 19151 +34 + Misses 5790 5724 -66 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

sophie-cluml commented 1 month ago

@BLYKIM 이거 그런데 -c 옵션 없이 remote conf 로만 돌렸을 때 mac OS 기준으로

Error: Path not found "/Users/pita/Library/Application Support/logs/apps"

이런 에러 뜨지 않으시던가요? 저 경로에 mkdir 해놓고 돌리셨던 것이죠? /Users/pita 는 저의 $HOME 입니다.

현재 PR의 수정사항은 이 지점 말고는 다른 이견이 있는 부분은 딱히 없네요. 요 부분만 cross check & 앞으로 방향 sync를 하면 좋을것 같습니다.

BLYKIM commented 1 month ago

@sophie-cluml -c 옵션없이 돌렸을 때 logs/apps는 디렉토리를 생성해두었어서 문제는 없었습니다. 다만 -c 옵션 없이 ( local configuration 파일 없이) 돌릴 때 간단한 sync가 필요한 부분들이 있습니다. 어제 출근해서는 바빠보이셔서 못했고 오늘 planning에서 시간 잡을 수 있을까요?

sophie-cluml commented 3 weeks ago

https://github.com/aicers/giganto/pull/863#issuecomment-2429185258 에 대해서는 #876 가 생성되었습니다.