Closed pierDipi closed 10 months ago
/cc @skonto @houshengbo @matzew @creydr
/cherry-pick release-1.11
@pierDipi: once the present PR merges, I will cherry-pick it on top of release-1.11 in a new PR and assign it to you.
/cherry-pick release-1.10
@pierDipi: once the present PR merges, I will cherry-pick it on top of release-1.10 in a new PR and assign it to you.
Attention: 2 lines
in your changes are missing coverage. Please review.
Comparison is base (
8412d98
) 79.40% compared to head (a9e0019
) 79.40%. Report is 1 commits behind head on main.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: creydr, pierDipi
The full list of commands accepted by this bot can be found here.
The pull request process is described here
Cherry pick done manually in https://github.com/knative/operator/pull/1591
/retest-required
/test eventing-upgrade-tests /lgtm
@pierDipi: failed to push cherry-picked changes in GitHub: pushToNamedFork is not implemented in the v2 client
@pierDipi: failed to push cherry-picked changes in GitHub: pushToNamedFork is not implemented in the v2 client
It might happen that not all resources were created successfully so we should not fail finalization when resources are not found, we can just ignore not found errors.