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

지정된 저장소를 사용하는지 CI단계에서 체크하는 로직 추가 #189

Merged
merged 2 commits into from
Feb 21, 2023

Conversation

intelliguy
Copy link
Contributor

@intelliguy intelliguy commented Feb 9, 2023

포함내역

  • 지정된 저장소 사용여부를 CI에서 체크
    • helm chart 저장소 체크와 정의된 값을 기반으로 사용되는 이미지의 저장소 체크, 2단계로 구성되며
  • 기존에 정확하게 사용되지 않던 validation 제거 (과거 *스더님이 추가한 것으로 보여지고, lma 한정으로 뭔가 체크했던 듯)

관련되어 개발자가 알아야 할 내역

  • 본 매커니즘은 시스템적으로 챠트나 이미지에 지정된 부분에 대한 처리를 수행
  • 예외적 상황을 기술하여 해당 케이스의 테스트를 패시하도록 만들어줄 필요가 있음
    • 필요에 따라 의도적으로 helm chart 저장소를 예외처리: .
      github/workflows/check_chart_repo.sh 파일을 수정하여 EXCEPTION_LIST에 예외 저장소를 추가
    • 필요에 따라 의도적으로 이미지 저장소를 예외처리: .
      github/workflows/check_image_repo.sh 파일을 수정하여 EXCEPTION_LIST에 예외 저장소를 추가
  • 예외 사항기술을 위한 구체적인 예시는 해당 변수(EXCEPTION_LIST)하단의 내역을 참조
    • 예시내역은 현재의 모든 사용자원임
    • 따라서 주석을 풀고 붙여주면 본 체크는 모두 예외처리를 통해 통과하게됨

@intelliguy intelliguy force-pushed the repo_validation branch 2 times, most recently from ee2e48b to d9e5e04 Compare February 9, 2023 05:58
@intelliguy intelliguy changed the title valication.. test 지정된 저장소를 사용하는지 CI단계에서 체크하는 로직 추가 Feb 9, 2023
@intelliguy intelliguy self-assigned this Feb 9, 2023
@intelliguy intelliguy requested a review from zugwan February 13, 2023 04:17
@github-actions
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 Feb 16, 2023
@zugwan zugwan removed the Stale There has been no activity on this label Feb 17, 2023
@github-actions
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 Feb 20, 2023
@ktkfree
Copy link
Contributor

ktkfree commented Feb 20, 2023

생각해보니 여긴 base 군요.
decapod-site 에서 동일한 작업을 하고 있는데, 여기에도 추가하신 의도를 설명 부탁 드립니다~
양쪽에서 모두 하는 것도 좋으나, 작업 프로세스상 exception 리스트 관리가 복잡함이 있을 수 있을것 같아서요.

@github-actions github-actions bot removed the Stale There has been no activity on this label Feb 20, 2023
@ktkfree ktkfree merged commit 6bf3ea4 into main Feb 21, 2023
@ktkfree ktkfree deleted the repo_validation branch February 21, 2023 06:00
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