Skip to content

fix: 'argocd-server-tls' Secret should be loaded from informer#14522

Merged
crenshaw-dev merged 1 commit intoargoproj:masterfrom
alexmt:external-tls-secret-from-informer
Jul 17, 2023
Merged

fix: 'argocd-server-tls' Secret should be loaded from informer#14522
crenshaw-dev merged 1 commit intoargoproj:masterfrom
alexmt:external-tls-secret-from-informer

Conversation

@alexmt
Copy link
Collaborator

@alexmt alexmt commented Jul 15, 2023

Argo CD is loading argocd-server-tls Secret directly from Kubernetes although it is available in informer. This results in thousands of unnecessary k8s API calls and most likelly slows down reconciliation.

Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
@alexmt alexmt requested a review from jannfis July 15, 2023 09:35
@alexmt alexmt added cherry-pick/2.5 Candidate for cherry picking into the 2.5 release branch cherry-pick/2.6 Candidate for cherry picking into the 2.6 release branch cherry-pick/2.7 Candidate for cherry picking into the 2.7 release branch labels Jul 15, 2023
@codecov
Copy link

codecov bot commented Jul 15, 2023

Codecov Report

Patch coverage has no change and project coverage change: +0.17 🎉

Comparison is base (1c7eed1) 49.61% compared to head (f41c760) 49.79%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #14522      +/-   ##
==========================================
+ Coverage   49.61%   49.79%   +0.17%     
==========================================
  Files         256      261       +5     
  Lines       43800    44751     +951     
==========================================
+ Hits        21733    22283     +550     
- Misses      19934    20274     +340     
- Partials     2133     2194      +61     
Impacted Files Coverage Δ
util/settings/settings.go 51.09% <0.00%> (+1.09%) ⬆️

... and 44 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@alexmt alexmt requested a review from crenshaw-dev July 15, 2023 10:22
@kencochrane
Copy link

Thank you for fixing that @alexmt

@crenshaw-dev crenshaw-dev merged commit ff92e60 into argoproj:master Jul 17, 2023
@crenshaw-dev
Copy link
Member

/cherry-pick release-2.8

gcp-cherry-pick-bot bot pushed a commit that referenced this pull request Jul 17, 2023
Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
@crenshaw-dev
Copy link
Member

/cherry-pick release-2.7

gcp-cherry-pick-bot bot pushed a commit that referenced this pull request Jul 17, 2023
Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
@crenshaw-dev
Copy link
Member

/cherry-pick release-2.6

@crenshaw-dev
Copy link
Member

/cherry-pick release-2.5

gcp-cherry-pick-bot bot pushed a commit that referenced this pull request Jul 17, 2023
Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
gcp-cherry-pick-bot bot pushed a commit that referenced this pull request Jul 17, 2023
Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
crenshaw-dev pushed a commit that referenced this pull request Jul 17, 2023
… (#14546)

Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
Co-authored-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
crenshaw-dev pushed a commit that referenced this pull request Jul 17, 2023
… (#14547)

Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
Co-authored-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
crenshaw-dev pushed a commit that referenced this pull request Jul 17, 2023
… (#14548)

Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
Co-authored-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
crenshaw-dev pushed a commit that referenced this pull request Jul 17, 2023
… (#14549)

Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
Co-authored-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
Jneville0815 pushed a commit to radiusmethod/argo-cd that referenced this pull request Jul 18, 2023
…roj#14522)

Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
Signed-off-by: Jimmy Neville <jimmyeneville@gmail.com>
@alexmt alexmt deleted the external-tls-secret-from-informer branch July 21, 2023 10:19
schakrad pushed a commit to schakrad/argo-cd that referenced this pull request Jul 24, 2023
…roj#14522) (argoproj#14547)

Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
Co-authored-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
Signed-off-by: schakrad <58915923+schakrad@users.noreply.github.com>
yyzxw pushed a commit to yyzxw/argo-cd that referenced this pull request Aug 9, 2023
…roj#14522)

Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
@suzaku suzaku mentioned this pull request Aug 15, 2023
tesla59 pushed a commit to tesla59/argo-cd that referenced this pull request Dec 16, 2023
…roj#14522)

Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cherry-pick/2.5 Candidate for cherry picking into the 2.5 release branch cherry-pick/2.6 Candidate for cherry picking into the 2.6 release branch cherry-pick/2.7 Candidate for cherry picking into the 2.7 release branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants