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

lma-primary: update logic for object store #387

Merged
merged 1 commit into from
Sep 10, 2024
Merged

lma-primary: update logic for object store #387

merged 1 commit into from
Sep 10, 2024

Conversation

intelliguy
Copy link
Contributor

@intelliguy intelliguy commented Sep 9, 2024

object store에 대한 수정

  1. 근본적으로 이 플로우에서 object store에 대한 조작을 진행함
  2. 영향을 받는 부분은
    • loki의 백엔드로서
    • prometheus의 장기저장소로서
  3. loki 부분은 primary에서만 잘 하면됨
  4. prometheus 부분은 맴버 클러스터에서도 어디에 연결되어야 할지 알아야 함

현재 구조는

  • default는 s3로 하고 있고
  • 사용자가 minio로 지정한 경우 이 설정을 수정하도록 하는 부분이 본 workflow

따라서, 명확하게 minio라고 지정하지 않으면 prometheus 연결에서 문제가 발생할수 있음

다만, 현재의 상황 (primary=aws, member=byoh)에서는 현 pr로 해소됨
(primary=byoh, member=aws)라면 문제가 발생할 것임
현재 byoh가 사용된 케이스(BTV)에서는 항상 endpoint-secret가 존재했으므로 본 로직을 통과하지 않기도 함

object store에 대한 수정
@ktkfree ktkfree merged commit 58269f0 into develop Sep 10, 2024
3 checks passed
@ktkfree ktkfree deleted the object-store branch September 10, 2024 04:05
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.

2 participants