Closed houshengbo closed 8 months ago
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: houshengbo
The full list of commands accepted by this bot can be found here.
The pull request process is described here
Attention: 4 lines
in your changes are missing coverage. Please review.
Comparison is base (
7f4be55
) 79.40% compared to head (6d3ba94
) 79.37%. Report is 9 commits behind head on main.
Files | Patch % | Lines |
---|---|---|
pkg/reconciler/knativeserving/ingress/istio.go | 75.00% | 2 Missing and 2 partials :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
/lgtm
I opened the follow-up issue - https://github.com/knative/operator/issues/1646
/test eventing-upgrade-tests
/test eventing-upgrade-tests
/cherry-pick release-1.12
@houshengbo: new pull request created: #1647
/cherry-pick release-1.11
/cherry-pick release-1.10
@houshengbo: #1628 failed to apply on top of branch "release-1.11":
Applying: Update istio gateway from v1alpha3 to v1beta1
Using index info to reconstruct a base tree...
M pkg/apis/operator/base/ingressconfiguration.go
M pkg/reconciler/knativeserving/ingress/istio.go
Falling back to patching base and 3-way merge...
Auto-merging pkg/reconciler/knativeserving/ingress/istio.go
CONFLICT (content): Merge conflict in pkg/reconciler/knativeserving/ingress/istio.go
Auto-merging pkg/apis/operator/base/ingressconfiguration.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Update istio gateway from v1alpha3 to v1beta1
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".
@houshengbo: #1628 failed to apply on top of branch "release-1.10":
Applying: Update istio gateway from v1alpha3 to v1beta1
Using index info to reconstruct a base tree...
M pkg/apis/operator/base/ingressconfiguration.go
M pkg/reconciler/knativeserving/ingress/istio.go
Falling back to patching base and 3-way merge...
Auto-merging pkg/reconciler/knativeserving/ingress/istio.go
CONFLICT (content): Merge conflict in pkg/reconciler/knativeserving/ingress/istio.go
Auto-merging pkg/apis/operator/base/ingressconfiguration.go
error: Failed to merge in the changes.
hint: Use 'git am --show-current-patch=diff' to see the failed patch
Patch failed at 0001 Update istio gateway from v1alpha3 to v1beta1
When you have resolved this problem, run "git am --continue".
If you prefer to skip this patch, run "git am --skip" instead.
To restore the original branch and stop patching, run "git am --abort".
Fixes #1469
Proposed Changes
Release Note