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-addons: change metrics to check abnormal status #226

Merged
merged 2 commits into from
Feb 5, 2024
Merged

Conversation

intelliguy
Copy link
Contributor

ready가 아닌 모든 컨테이너를 비정상(abnormal)로 판단하던 로직으로 혼선을 가져왔고 이에 따라 적절한 내역으로 변경함

  • pod의 비정상 판단
    • 기존: kube_pod_status_ready{condition="true"} == 0
    • 변경: kube_pod_status_reason != 0
    • pod의 상태가 ready가 아니면 비정상 에서 상태변경에 대한 이유가 있는 파드로 대치, 이유는 다음확인
      Evicted, NodeAffinity, NodeLost, Shutdown, UnexpectedAdmissionError
  • 컨테이너의 비정상 판단
    • 기존: kube_pod_container_status_ready == 0.
    • 변경: kube_pod_container_status_terminated_reason{reason!='Completed'} != 0
    • 컨테이너의 상태가 ready가 아니면 비정상 에서 완료상태가 Completed가 아닌 상황에서 종료한 컨테이너의 수로 변경

이에따라 부가적으로 제공하던 비정상 컨테이너 리스트로 정지상태의 컨테이너와 정지 이유를 전달하는 방식으로 변경

@bluejayA
Copy link
Contributor

이 내용이 아래 티켓과 연관이 되는건가요?
https://github.com/openinfradev/tks-issues/issues/814

@intelliguy
Copy link
Contributor Author

intelliguy commented Jan 26, 2024

https://github.com/openinfradev/tks-issues/issues/855 연관입니다. 링크했습니다,.

Copy link

This PR is stale because it has been open 3 days with no activity. Remove stale label or comment or this will be closed in 3 days.

@github-actions github-actions bot added the Stale There has been no activity on this label Jan 29, 2024
@intelliguy intelliguy removed the Stale There has been no activity on this label Jan 31, 2024
@ktkfree
Copy link
Contributor

ktkfree commented Feb 1, 2024

기존 환경에 영향이 있는 PR 인데, version 을 올려야하지 않을까 합니다.

@intelliguy
Copy link
Contributor Author

기존 환경에 영향이 있는 PR 인데, version 을 올려야하지 않을까 합니다.

의견반영해 패치버전 하나 올렸습니다. 이에따라 base에도 적용해야 최종 반영됩니다. 기타 변경 완료후 진행예정

@intelliguy intelliguy merged commit e98deae into main Feb 5, 2024
2 of 4 checks passed
@intelliguy intelliguy deleted the lma-addons branch February 5, 2024 09:38
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.

3 participants