Istio chart should set minimum cpu/memory request

[cbron]

The v1 istio chart has a minimum cpu and memory requirement for the cluster. V2 should do this also with chart annotations, see the helm chart reqs doc. We can use the most common istio installation as the case.

[cbron]

Parent mem/cpu issue: rancher/dashboard#1849

[Ellenqs]

This is validated on Rancher v2.5-head (df89971).
Istio version: v1.8.100-rc01

Steps

• Deploy rancher single install.
• Create a downstream cluster (1 control plane, 1 etcd, 3 worker).
• Go to cluster explorer, install monitoring, then install istio.
• Go to App & Marketplace, find rancher-istio in Installed Apps, click view YAML, check chart annotations in the file.

Result:
Istio is installed successfully and the cpu and memory requests are shown in annotation.

[cbron]

@Ellenqs is there a warning when you don't have enough resources ? Or is that not built yet.

[bmdepesa]

@cbron that's covered in this issue (see the linked comment): rancher/dashboard#1849 (comment) which is in Next Up. We'll test that part of the functionality with that issue.