-
Notifications
You must be signed in to change notification settings - Fork 17
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
ApplicationSet found, but no diff rendered #64
Comments
Hi @juliankock, Thank you so much! Is your repository private? I hope this helps! If not, let me know, and we can investigate further. |
I wonder if this only affects deploying a new directory (" Adding a new directory doesn't cause a diff in the PR Comment:
Removing an existing directory actually caused an error: Modifying an existing directory had the desired behavior (diff rendered): ApplicationSet is here: https://github.com/seanturner026/argocd-applicationset/blob/8b2ad93e8c3cf920f073f4c79520d39251559d86/clusters/staging/infra-appset.yaml And is deployed by this Application: https://github.com/seanturner026/argocd-applicationset/blob/8b2ad93e8c3cf920f073f4c79520d39251559d86/clusters/staging/argocd-app.yaml Seems to be the case on 0.0.18 and 0.0.24? Please let me know if I should make separate Issues here. Also having a look on my end. |
Thank you for the examples and investigation, @seanturner026. |
No inconvenience at all! |
Hi @juliankock, This issue should now be fixed in version Please let me know if you’re still encountering this issue 🙏🏻 It’s possible that the behavior you’re seeing is caused by something else |
Hi @juliankock, Just wanted to follow up on this - have you had a chance to check out the new version? :) The issue you experienced might also be related to: issues 85 |
It seems like it now finds the ApplicationSets. However, since I use SSH AUTH reference to the repository, I now get the following failing GitHub action:
I do mount the secrets as specified here: https://dag-andersen.github.io/argocd-diff-preview/github-actions-workflow/ So maybe I need to add an SSH key to the container? This is only a local constraint for our running ArgoCD instance. So if possible, the library here could just change to an HTTP repoURL. |
Hey!
Thanks for a wonderful initiative.
I'm having trouble rendering my
ApplicationSets
. It seems like they are found, but not used in diff generation, or loaded to ArgoCD. See here:Here is an example of an ApplicationSet:
I did look into the codebase, and I do not immediately see any overwrite of
spec.template.spec.destination.namespace
. May that be it?update: tried to remove
spec.template.spec.destination.namespace
didn't fix it.The text was updated successfully, but these errors were encountered: