Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: Pass AppNamespace to UpdateSpec api call #687

Merged

Conversation

mwijngaard
Copy link
Contributor

Adds support for applications in other namespace than default argocd namespace (when using the argocd api).

@codecov-commenter
Copy link

codecov-commenter commented Mar 12, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 66.29%. Comparing base (7d93c7a) to head (133a3da).

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #687      +/-   ##
==========================================
+ Coverage   66.27%   66.29%   +0.01%     
==========================================
  Files          22       22              
  Lines        2150     2151       +1     
==========================================
+ Hits         1425     1426       +1     
  Misses        591      591              
  Partials      134      134              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@mwijngaard mwijngaard force-pushed the add-appnamespace-to-update-spec branch from 2a08935 to 8211270 Compare March 12, 2024 15:18
…other namespaces than argocd itself (when using the argocd api).

Signed-off-by: Merijn Wijngaard <mwijngaard@gmail.com>
@mwijngaard mwijngaard force-pushed the add-appnamespace-to-update-spec branch from 8211270 to 133a3da Compare March 12, 2024 15:22
@mwijngaard
Copy link
Contributor Author

Any chance this could get merged?

Copy link
Collaborator

@chengfang chengfang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jannfis jannfis changed the title Pass AppNamespace to UpdateSpec api call fix: Pass AppNamespace to UpdateSpec api call Jun 7, 2024
@jannfis jannfis merged commit f8b345b into argoproj-labs:master Jun 7, 2024
10 checks passed
@mwijngaard mwijngaard deleted the add-appnamespace-to-update-spec branch June 8, 2024 19:00
sribiere-jellysmack pushed a commit to sribiere-jellysmack/argocd-image-updater that referenced this pull request Aug 13, 2024
Signed-off-by: Merijn Wijngaard <mwijngaard@gmail.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants