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

Detect-files not finding helm chart when path is the root of the chart #431

Closed
2 tasks done
ndlanier opened this issue Jan 11, 2023 · 2 comments
Closed
2 tasks done
Labels
bug Something isn't working triage This bug needs triage

Comments

@ndlanier
Copy link

What happened?

Running pluto detect-files -d ./chart-repo/myexamplechart/ sees no deprecated or removed apiVersions. But it sees it when we run it from the root directory of the repo. I have manually specified the ingress template for the chart uses v1beta which is removed in kube 1.22

pluto detect-files -d ./
NAME                        KIND      VERSION              REPLACEMENT            REMOVED   DEPRECATED  
client-portal-web-ing       Ingress   extensions/v1beta1   networking.k8s.io/v1   true      true        
client-portal-web-com-ing   Ingress   extensions/v1beta1   networking.k8s.io/v1   true      true        
client-portal-web-ing       Ingress   extensions/v1beta1   networking.k8s.io/v1   true      true        
client-portal-web-com-ing   Ingress   extensions/v1beta1   networking.k8s.io/v1   true      true       

This is happening on the linux amd64 version and the darwin arm64 version (installed via brew for the darwin install).

What did you expect to happen?

I expected pluto to output that the ingress component is using a removed api version

How can we reproduce this?

Create a repository for multiple helm charts (not packaged) and try running the pluto commands.
Explanation of how the detect-files command works would assist in troubleshooting.

Version

Latest

Search

  • I did search for other open and closed issues before opening this.

Code of Conduct

  • I agree to follow this project's Code of Conduct

Additional context

No response

@ndlanier ndlanier added bug Something isn't working triage This bug needs triage labels Jan 11, 2023
@sudermanjr
Copy link
Member

detect-files won't necessarily detect helm templates. You're better off using helm template | pluto detect.

@ndlanier
Copy link
Author

We had a typo with the api version we were testing. Apologies.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working triage This bug needs triage
Projects
None yet
Development

No branches or pull requests

2 participants