-
Notifications
You must be signed in to change notification settings - Fork 5.8k
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
fix: Argo CD doesn't detect the repo type when repository is scoped #11959
Conversation
Codecov ReportBase: 47.28% // Head: 47.29% // Increases project coverage by
Additional details and impacted files@@ Coverage Diff @@
## master #11959 +/- ##
==========================================
+ Coverage 47.28% 47.29% +0.01%
==========================================
Files 245 245
Lines 41669 41670 +1
==========================================
+ Hits 19702 19707 +5
+ Misses 19981 19978 -3
+ Partials 1986 1985 -1
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. ☔ View full report at Codecov. |
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.
LGTM
Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
83c6a9a
to
6c59b98
Compare
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.
LGTM
…11959) Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
…rgoproj#11959) Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com> Signed-off-by: emirot <emirot.nolan@gmail.com>
…rgoproj#11959) Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
…rgoproj#11959) Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com> Signed-off-by: schakrad <chakradari.sindhu@gmail.com>
Signed-off-by: Alexander Matyushentsev AMatyushentsev@gmail.com
Closes #9580
The #8558 updated repository API to verify that repo is permitted in a project before returning list of applications/application details for given repository. The check loads project directly using project informer and does not take into account project scoped repositories. This PR fixes the issue.