-
Notifications
You must be signed in to change notification settings - Fork 47
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: Change naming convention for RepoCred secret created in Argo CD namespace. #658
Conversation
c02fa6a
to
d3b17e9
Compare
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## main #658 +/- ##
==========================================
- Coverage 60.94% 60.75% -0.20%
==========================================
Files 99 99
Lines 17865 17867 +2
==========================================
- Hits 10888 10855 -33
- Misses 5755 5784 +29
- Partials 1222 1228 +6
☔ View full report in Codecov by Sentry. |
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, thanks @jparsai! One small ask, can you also ensure we have E2E test that verifies that the Argo CD repository secret is created, similar to what the unit test is doing in the PR?
1936016
to
ed68909
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, thanks again @jparsai
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jgwest, jparsai The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Description:
This PR is to change naming convention for Secrets created for Repository Credential in Argo CD namespace.
JIRA Ticket:
https://issues.redhat.com/browse/GITOPSRVCE-755