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

OSSM-5690: Sync with istio 1.20 #49

Merged
merged 7 commits into from
Mar 18, 2024

Conversation

yannuil
Copy link
Contributor

@yannuil yannuil commented Mar 15, 2024

No description provided.

Signed-off-by: Yann Liu <yannliu@redhat.com>
Signed-off-by: Yann Liu <yannliu@redhat.com>
Signed-off-by: Yann Liu <yannliu@redhat.com>
Signed-off-by: Yann Liu <yannliu@redhat.com>
Signed-off-by: Yann Liu <yannliu@redhat.com>
Signed-off-by: Yann Liu <yannliu@redhat.com>
k8s.io/gengo v0.0.0-20220902162205-c0856e24416d
k8s.io/klog/v2 v2.90.1
sigs.k8s.io/gateway-api v0.6.2
istio.io/client-go v1.20.0-rc.0
Copy link
Member

Choose a reason for hiding this comment

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

Shouldn't it be istio.io/client-go v1.20.2-0.20231213021015-d9f0411d8b67, like in maistra/istio?

Copy link
Contributor Author

@yannuil yannuil Mar 15, 2024

Choose a reason for hiding this comment

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

I want to make it the same as the istio commit 1.20 I am rebasing on, I can eliminate the extra changes in the go.mod commit.

For details, please check out this [PR](golangci/golangci-lint#4227).

Signed-off-by: Yann Liu <yannliu@redhat.com>
@yannuil
Copy link
Contributor Author

yannuil commented Mar 18, 2024

/retest-required

1 similar comment
@yannuil
Copy link
Contributor Author

yannuil commented Mar 18, 2024

/retest-required

@jwendell
Copy link
Member

/retest

1 similar comment
@jwendell
Copy link
Member

/retest

@openshift-merge-bot openshift-merge-bot bot merged commit 8fc1aab into maistra:maistra-2.6 Mar 18, 2024
4 checks passed
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.

3 participants