Skip to content
This repository has been archived by the owner on May 16, 2023. It is now read-only.

Workaround helm template bug when testing for APIVersions #1684

Open
wants to merge 1 commit into
base: 7.17
Choose a base branch
from

Conversation

mfrechePgest
Copy link

What this PR does / why we need it:

#1420 kinda broke chart deployment with ArgoCD on older K8S versions ( < 1.21 )

Issue is related to argoproj/argo-cd#7291

I hope they'll find a fix there but a workaround suggested in the linked issue was to set up more precise APIVersions.Has in helm charts.
That's the point of this PR

It mirror this PR 7a54fb1 ; adding the Kind to APIVersions.Has function

Thx

@elasticmachine
Copy link
Collaborator

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

@cla-checker-service
Copy link

cla-checker-service bot commented Aug 24, 2022

💚 CLA has been signed

@botelastic
Copy link

botelastic bot commented Feb 21, 2023

This PR 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. To track this PR (even if closed), please open a corresponding issue if one does not already exist.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants