Skip to content

feat: upgrade istio to 1.20 #1590

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

Merged
merged 2 commits into from
Apr 18, 2024
Merged

feat: upgrade istio to 1.20 #1590

merged 2 commits into from
Apr 18, 2024

Conversation

merll
Copy link
Contributor

@merll merll commented Apr 16, 2024

This upgrades the helm chart for Istio to the latest 1.20 release.

Checklist

  • Architecture Design Records have been added as adr/*.md and appended to list in adr/_index.md, if applicable.
  • The values-schema.yaml file and test/** fixtures have been updated to reflect code changes, if applicable.
  • The OpenApi Schema from redkubes/otomi-api project is compatible with definitions from values-schema.yaml file, if applicable.
  • Helm releases are meeting otomi's baseline security policies, if applicable.
  • Helm chart and helmfile changes are tested against upgrade scenario, if applicable.

Sorry, something went wrong.

@srodenhuis
Copy link
Contributor

Does the upgrade affect the Istio tracing configuration?

@merll
Copy link
Contributor Author

merll commented Apr 17, 2024

Does the upgrade affect the Istio tracing configuration?

Not from what I could see. I can check a bit more on the configuration specifically, but just from testing workloads using Otel, traces work fine with the upgrade. I tested with mixed components upgraded (istio-proxy not restarted), and also with a cluster initially deployed on this branch.

Copy link
Contributor

@j-zimnowoda j-zimnowoda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Does the upgrade affect compatibility with knative?
I will check it.

@j-zimnowoda
Copy link
Contributor

Does the upgrade affect compatibility with knative?
I will check it.

I think we need to upgrade knative -serving to v.1.13: https://github.com/knative/serving/releases/tag/knative-v1.13.1

@j-zimnowoda
Copy link
Contributor

We are going to provide knative upgrade as a separate PR

@j-zimnowoda j-zimnowoda self-requested a review April 18, 2024 14:27
@j-zimnowoda j-zimnowoda merged commit 337a405 into main Apr 18, 2024
19 checks passed
@j-zimnowoda j-zimnowoda deleted the upgrade-istio-1-20 branch April 18, 2024 14:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants