CurrentSyncStatus updates but it doesn't sync values.yaml

[vrodomista]

Checklist:

• I've searched in the docs and FAQ for my answer: https://bit.ly/argocd-faq.

Describe the bug

I have a medium sized monorepo with about 25 apps inside of it. Recently, after upgrading to 2.0.1, I noticed that apps are simply not auto-syncing

I monitor the application repo server, and see the cache misses for all of the applications. After the cache miss, the Current Sync Status of all applications is updated to show the latest commit. Then the application controller goes through its auto sync process, but decides to "skip sync" for every application. This is where we encounter the issue. Last sync result still shows a commit from a day ago and never updates until I actually hit "sync"

As a note, everything works fine when I'm updating the templates directly - it's when there is a change to values.yaml that I see issues.

---- EDIT ----

I've actually narrowed this down further - this issue only happens to resources that have been marked with a Hook - both the PreSync job and PostSync deployment were not being used to decide if resources were out of sync

To Reproduce

Create large monorepo with dozens of applications, try to auto sync one of them

Expected behavior

I expect to make a change in the GitHub repo and have those changes reflected in the cluster resources ArgoCD manages
Screenshots

Version

2.0.1