Remove deprecated toolserver CRD from kagent-crds helm chart

[antweiss]

As the ToolServer api has been removed - the CRD is redundant and is just causing confusion

[EItanya]

Good catch!

I forgot it also needs to be removed from go/config/crd/bases

[EItanya]

Thanks so much for the PR, and a great catch. Unfortunately this will not work today. These CRDs are autogenerated from the go types, which still exist. go/controller/api/v1alpha1. You'll need to figure out how to get it to stop generating this CR

[antweiss]

Thanks! I believe I know how to do it. Will look into it tomorrow. Anton Weiss http://otomato.io

…

On Wed, Aug 20, 2025, 10:53 PM Eitan Yarmush ***@***.***> wrote: *EItanya* left a comment (kagent-dev/kagent#783) <#783 (comment)> Thanks so much for the PR, and a great catch. Unfortunately this will not work today. These CRDs are autogenerated from the go types, which still exist. go/controller/api/v1alpha1. You'll need to figure out how to get it to stop generating this CR — Reply to this email directly, view it on GitHub <#783 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAY7P3UQTY7PSX2NE7HWUAL3OTG27AVCNFSM6AAAAACEMM2DIWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZTEMBXHA4DAMBVHE> . You are receiving this because you authored the thread.Message ID: ***@***.***>

[antweiss]

So, there's a way to avoid generating all CRs for v1alpha1 by adding the // +kubebuilder:skip directive on the v1alpha1 package. This will also remove the deprecated Memories CR. The question is - do we need older versions?

[antweiss]

@EItanya I updated the PR so that all v1alpha1 CRs are skipped while generating. I believe this is ok at this stage of the project but I could be wrong. LMKWYT

[EItanya]

It should be fine, however there is one situation I am worried about. Do upgrades from 0.5.x -> this commit work properly after this change. I remember seeing issues upgrading CRDs when removing versions like this, but I may have been doing it wrong. I think if it fails we should wait until 0.7.x to remove them completely.

What do you think?

[antweiss]

I think I'll give it a try. Will install CRDs from the old chart and then upgrade to this.