-
Notifications
You must be signed in to change notification settings - Fork 63
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
use annotation tracking by default #125
Conversation
Signed-off-by: Manabu McCloskey <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you also mention within the README file that idpbuilder's argocd set by default the property application.resourceTrackingMethod: annotation
, point to the argocd doc and explain why ?
Signed-off-by: Manabu Mccloskey <[email protected]>
@cmoulliard Good idea. Reflected it in the README. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a spelling typo and LGTM
README.md
Outdated
ArgoCD manifests are generated using a bash script available [here](./hack/argo-cd/generate-manifests.sh). | ||
This script runs kustomize to modify the basic installation manifests provided by ArgoCD. Modifications include: | ||
|
||
1. Prevent notificaiton and dex pods from running. This is done to keep the number of pods running low by default. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/notificaition/notification/
Signed-off-by: Manabu Mccloskey <[email protected]>
8c4ba83
to
6d76d4c
Compare
Annotation tracking method is a must when using Crossplane with ArgoCD. This PR enables it.
https://argo-cd.readthedocs.io/en/stable/user-guide/resource_tracking/#additional-tracking-methods-via-an-annotation