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

Istio charts for 1.19.9 and 1.20.5 are incorrectly sourced. istio-ingress-* and istio-egress-* charts are considered "Legacy Gateway Helm charts" #47

Open
prasad-shirodkar opened this issue Nov 12, 2024 · 0 comments · May be fixed by #48
Assignees

Comments

@prasad-shirodkar
Copy link
Contributor

prasad-shirodkar commented Nov 12, 2024

Istio charts for 1.19.9 and 1.20.5 are incorrectly sourced. Example: refer https://github.com/oracle-cne/catalog/blob/release/2.0/charts/istio-base-1.19.9/Chart.yaml#L3C1-L4C31

# This version is never actually shipped. istio/release-builder will replace it at build-time
# with the appropriate version

istio-ingress-* and istio-egress-* charts are considered "Legacy Gateway Helm charts"

In catalog repo for istio 1.19.9 and 1.20.5, the chart directories (under charts) represented by istio-ingress-* and istio-egress-* are considered by upstream as Legacy Gateway Helm charts

Istio historically offered two different charts - manifests/charts/gateways/istio-ingress and manifests/charts/gateways/istio-egress. These are replaced by this chart. While not required, it is recommended all new users use this chart, and existing users migrate when possible.

This chart has the following benefits and differences:

  • Designed with Helm best practices in mind (standardized values options, values schema, values are not all nested under gateways.istio-ingressgateway.*, release name and namespace taken into account, etc).
  • Utilizes Gateway injection, simplifying upgrades, allowing gateways to run in any namespace, and avoiding repeating config for sidecars and gateways.
  • Published to official Istio Helm repository.
  • Single chart for all gateways (Ingress, Egress, East West).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant