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

disable autosync of all argocd apps #77

Merged
merged 2 commits into from
Apr 6, 2022

Conversation

robertchoi80
Copy link
Contributor

@robertchoi80 robertchoi80 commented Apr 5, 2022

금일 테스트 시 논의된 대로 모든 argocd app의 sync를 비활성화하는 부분 추가하였습니다.
admin-v2 에서 테스트하려 하니 최근 변경된 proto 때문인지 제대로 테스트가 불가하여, 내일 decapod10에서 다같이 테스트해보고 잘 되면 머지하면 될것 같습니다.

@ktkfree
Copy link
Contributor

ktkfree commented Apr 5, 2022

AppGroup label 에 따른 별도 처리를 하지 않는다면 그냥 전체 App 을 grep 해도 되지 않을까요?
argocd app list -o name | grep ${app_prefix}

@robertchoi80
Copy link
Contributor Author

AppGroup label 에 따른 별도 처리를 하지 않는다면 그냥 전체 App 을 grep 해도 되지 않을까요? argocd app list -o name | grep ${app_prefix}

맞네요. 첨에 이 코드를 decapod-flow 쪽에서 따오다보니 app_prefix가 존재하지 않는 경우도 고려된 코드였는데, 이 경우엔 prefix가 항상 존재하니 그렇게 해도 될것 같습니다. 수정해서 올렸습니다!

@robertchoi80 robertchoi80 changed the title [WIP] disable autosync of all argocd apps disable autosync of all argocd apps Apr 6, 2022
@ktkfree ktkfree merged commit 69acf5b into openinfradev:release-v2 Apr 6, 2022
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