-
Notifications
You must be signed in to change notification settings - Fork 140
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
Upgrade istio dependencies to 1.18.1 #385
Conversation
zhaohuabing
commented
Aug 18, 2023
•
edited
Loading
edited
- Change the Istio version from 1.18.2 to 1.18.1 since 1.18.2 envoy commit doesn't exist
- Upgrade all the Istio dependencies to 1.18.1
Signed-off-by: huabing zhao <[email protected]>
Signed-off-by: huabing zhao <[email protected]>
Signed-off-by: huabing zhao <[email protected]>
Signed-off-by: huabing zhao <[email protected]>
c2ab686
to
b45dc56
Compare
Signed-off-by: huabing zhao <[email protected]>
Signed-off-by: huabing zhao <[email protected]>
25c3b54
to
b2fd61f
Compare
@@ -36,7 +36,7 @@ jobs: | |||
- name: Setup Go | |||
uses: actions/setup-go@v1 | |||
with: | |||
go-version: 1.19 | |||
go-version: '1.20' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is recommended to merge the commit, there are too many commit id.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All the commits will be squashed into one when this PR is merged into the master.
Signed-off-by: huabing zhao <[email protected]>
c992105
to
d893b29
Compare
Signed-off-by: huabing zhao <[email protected]>
d893b29
to
97c8292
Compare
Signed-off-by: huabing zhao <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
good Job! thanks.