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

Support scanning for updates to Helm-backed ArgoCD Application CRDs #45

Open
jstewart612 opened this issue Jun 19, 2021 · 11 comments
Open
Labels
enhancement Adding additional functionality or improvements pinned Prevents stalebot from removing priority: could Future work depending on bandwidth and availability

Comments

@jstewart612
Copy link

Is your feature request related to a problem? Please describe.
ArgoCD has the concept of Application CRDs. These can source from Helm charts themselves, but do not install helm in the traditional manner. Rather, it renders the templates, and then applies those to the cluster. As such, you do not see helm installs with the helm cli, and as such, Nova seems not to be able to find them and assess upgradable candidates.

This feature request is to add this functionality to Nova.

Describe the solution you'd like
Add a flag to Nova to allow it to also scan ArgoCD Application CRD instances on your cluster and check them for updates if they are based on Helm charts.

Describe alternatives you've considered
I am unaware that any exist.

Additional context
I don't see that any is required here but let me know if you do.

@jstewart612
Copy link
Author

If it is helpful for implementation, the fields you'd be looking to scan of that CRD are:

.spec.source.helm.chart: the chart name
.spec.source.helm.targetRevision: A Semantic Release compatible version value

@rbren
Copy link
Contributor

rbren commented Jun 21, 2021

Thanks for the request! This is a great idea

@stale
Copy link

stale bot commented Jul 21, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale Marked as stale by stalebot label Jul 21, 2021
@rbren rbren added the pinned Prevents stalebot from removing label Jul 21, 2021
@stale stale bot removed the stale Marked as stale by stalebot label Jul 21, 2021
@jstewart612
Copy link
Author

still needed

@sudermanjr sudermanjr added enhancement Adding additional functionality or improvements priority: could Future work depending on bandwidth and availability labels Oct 27, 2021
@jstewart612
Copy link
Author

A colleague actually did work on this locally and I have recommended submission here for wider adoption of the feature.

@hungran
Copy link

hungran commented Nov 15, 2022

+1 on this feature, we have a lot of external applications implement by gitops approach - argocd then we are finding a way to check if whether they was update or release.

@jstewart612
Copy link
Author

@lucasreed is it safe to say no maintainer of this repository will accept the last PR submitted to address this issue while it requires an additional external binary? That you compel a native library be used for the purpose? This is not to voice disagreement: merely to establish the terms.

@lucasreed
Copy link
Contributor

Hey @jstewart612, I'm afraid I am no longer a maintainer here, so I'll let someone else answer that.

@sudermanjr
Copy link
Member

Catching up on this now, and I would say that is correct @jstewart612. Adding another third-party binary introduces maintenance cost that we are not willing to incur at this time. If we were to implement a dynamic client instead, that could be covered by unit tests and much easier/safer to maintain. Additionally, it would not require a new method of keeping up on dependencies beyond dependabot.

@andreyolv
Copy link

Flux is a great gitops tool and widely used.

@voidlily
Copy link

I originally was asking about this in this argocd issue, but they said it might be better to ask if nova is able to do this instead, and I then found the issue here, so here's my 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Adding additional functionality or improvements pinned Prevents stalebot from removing priority: could Future work depending on bandwidth and availability
Projects
Status: No status
Development

Successfully merging a pull request may close this issue.

7 participants