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

Upgraded istio to v1.19.10 #2743

Merged
merged 2 commits into from
Jun 4, 2024

Conversation

biswajit-9776
Copy link
Contributor

Pull Request Template for Kubeflow manifests Issues

  • Please include a summary of changes and the related issue.
  • List any dependencies that are required for this change.
  • Please delete the options that are not relevant.
  • The following checklist will help you to satisfy the requirements.

✏️ A brief description of the changes

I upgraded istio from version 1.18.7 to 1.19.10

📦 List any dependencies that are required for this change

My PR depends on #

🐛 If this PR is related to an issue, please put the link of the issue here.

The following issues are related, because ...

✅ Unit Test Checklist

  • 🛠️ Make sure you have installed kustomize == 5.2.1+
  • ✍️ Have you written new tests for your core changes, as applicable?
  • 🔄 Have you successfully run existing tests with your changes ?
  • 🚀 Have you successfully run existing and new tests with your changes ?

✅ Contributor checklist


You can join our slack channel wg-manifests here. This link also contains our meeting schedule.

Signed-off-by: biswajit-9776 <biswajitpatt139@gmail.com>
@juliusvonkohout
Copy link
Member

Please also fix the Readme based on your experience with the upgrade for istio and istio-cni in this PR.

@juliusvonkohout
Copy link
Member

/ok-to-test

@biswajit-9776
Copy link
Contributor Author

Let me upgrade istio-cni and push it in this same PR.

Signed-off-by: biswajit-9776 <biswajitpatt139@gmail.com>
@biswajit-9776
Copy link
Contributor Author

I pushed this PR with two scripts which I think could push after the v1.22.0 upgradation.

@juliusvonkohout
Copy link
Member

I pushed this PR with two scripts which I think could push after the v1.22.0 upgradation.

I do not see the two scripts in /hack
Please add them in the next PR

/lgtm
/approve

@google-oss-prow google-oss-prow bot added the lgtm label Jun 4, 2024
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: juliusvonkohout

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@google-oss-prow google-oss-prow bot merged commit de39a52 into kubeflow:master Jun 4, 2024
22 checks passed
@biswajit-9776
Copy link
Contributor Author

Yes I will push the two scripts in the upcoming PRs.

@biswajit-9776 biswajit-9776 deleted the istio-1.19.10 branch June 5, 2024 11:11
doncorsean pushed a commit to doncorsean/kubeflow-manifests that referenced this pull request Jul 18, 2024
* Upgraded istio to v1.19.10

Signed-off-by: biswajit-9776 <biswajitpatt139@gmail.com>

* Upgraded istio-cni to v1.19.10

Signed-off-by: biswajit-9776 <biswajitpatt139@gmail.com>

---------

Signed-off-by: biswajit-9776 <biswajitpatt139@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants