Skip to content
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: fix broken helm repo alias/name support #13647

Merged
merged 1 commit into from
May 18, 2023

Conversation

alexmt
Copy link
Collaborator

@alexmt alexmt commented May 18, 2023

PR fixes bug introduced by this fix. While parsing Chart.yaml repo server should take alias into account.

Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
@codecov
Copy link

codecov bot commented May 18, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.02 🎉

Comparison is base (1e7d8ad) 49.22% compared to head (d5b420b) 49.24%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #13647      +/-   ##
==========================================
+ Coverage   49.22%   49.24%   +0.02%     
==========================================
  Files         248      248              
  Lines       42828    42838      +10     
==========================================
+ Hits        21081    21097      +16     
+ Misses      19648    19644       -4     
+ Partials     2099     2097       -2     
Impacted Files Coverage Δ
reposerver/repository/repository.go 60.87% <100.00%> (+0.35%) ⬆️

... and 1 file 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 enabled auto-merge (squash) May 18, 2023 18:22
@alexmt alexmt merged commit f61f94f into argoproj:master May 18, 2023
@alexmt alexmt deleted the broken-deps branch May 18, 2023 18:28
alexmt added a commit to alexmt/argo-cd that referenced this pull request Jul 15, 2023
Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
alexmt added a commit to alexmt/argo-cd that referenced this pull request Jul 15, 2023
Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
crenshaw-dev pushed a commit that referenced this pull request Jul 15, 2023
…elated helm charts (#14529)

* Merge pull request from GHSA-94mc-2ch7-r5r5

Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>

* fix: fix broken helm repo alias/name support (#13647)

Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>

---------

Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
crenshaw-dev pushed a commit that referenced this pull request Jul 15, 2023
…elated helm charts (#14528)

* Merge pull request from GHSA-94mc-2ch7-r5r5

Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>

* fix: fix broken helm repo alias/name support (#13647)

Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>

---------

Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>
schakrad pushed a commit to schakrad/argo-cd that referenced this pull request Jul 24, 2023
…elated helm charts (argoproj#14528)

* Merge pull request from GHSA-94mc-2ch7-r5r5

Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>

* fix: fix broken helm repo alias/name support (argoproj#13647)

Signed-off-by: Alexander Matyushentsev <AMatyushentsev@gmail.com>

---------

Signed-off-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
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
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants