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

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.

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

No branches or pull requests

2 participants