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

Add TKS End-to-end test workflow template #89

Merged
merged 2 commits into from
Jun 13, 2022

Conversation

zugwan
Copy link
Contributor

@zugwan zugwan commented Jun 7, 2022

아래와 같이 TKS 서비스 URL을 입력받아 실행합니다.

argo submit --from wftmpl/tks-e2e-test -p tks_info_url=tks-info.svc:9999 -p tks_contract_url=tks-contract.svc:9999 -p tks_cluster-lcm_url=tks-cluster-lcm.svc:999 -n arg

E2E 테스트에서 sonobuy 실행하는 템플릿을 호출하기 위해 워크플로우 파라미터를 바로 쓰지 않고 템플릿 파라미터를 사용하도록 변경하였습니다.

this make run-sonobuy template be called by other workflow templates.
@Jaesang
Copy link
Contributor

Jaesang commented Jun 10, 2022

위 명령 그대로 실행하면 되나요? 어제 실행해보았는데, 무한루프를 도네요

argo watch -n argo tks-e2e-test-l9gcp

arguments:
parameters:
- name: cluster_id
value: "011b88fa-4d53-439f-9336-67845f994051"
- name: sonobuoy_mode
value: "quick"
templates:
- name: main
steps:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

특별히 step을 추가하신 이유가 궁금합니다. 향후 확장성을 위해서인가요?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

run-sonobuoy 템플릿을 정의가 되어있는 tks-validate-usercluster 가 아닌 다른 워크플로우, 워크플로우 템플릿에서 호출할 때 파라미터를 지정해서 호출하기 위해서 입니다. 기존과 같이 템플릿에서 바로 워크플로우 파라미터를 참조하게 되면 run-sonobuoy를 호출하는 외부 워크플로우에 run-sonobuoy가 사용하는 이름의 파라미터가 존재해야 합니다.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

input parameter 전달을 하려다보니 step도 같이 추가가 된거였군요. 저도 그런 이유로 최대한 input param을 사용하려고 했던 것 같습니다. 근데 참 애매한 것이, run-sonobuoy 같은 템플릿을 호출하는 외부 워크플로우의 실행시점에 사용자가 특정 값을 override해야 하는 케이스도 있다보니 다시 workflow param으로 바꾸게 되는 경우도 간혹 생기더라구요. 하지만 이 경우는 그럴 필요는 없으니 작성하신 방향에 동의합니다.

tests/tks-e2e.yaml Outdated Show resolved Hide resolved
tests/tks-e2e.yaml Outdated Show resolved Hide resolved
@zugwan
Copy link
Contributor Author

zugwan commented Jun 10, 2022

위 명령 그대로 실행하면 되나요? 어제 실행해보았는데, 무한루프를 도네요

argo watch -n argo tks-e2e-test-l9gcp

tks 서비스들 url은 환경에 맞게 지정해 주어야 합니다. CICD 클러스터에서 kubectl get svc -n tks 의 external-ip 를 사용했었습니다.
argo watch -n argo tks-e2e-test-7bw7r 로 확인하시면 됩니다.

@zugwan zugwan merged commit c887dd7 into openinfradev:main Jun 13, 2022
@zugwan zugwan deleted the e2e-tests branch June 13, 2022 01:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants