Capture reason for global resync

[pierDipi]

As per title, helps with debugging flakes

[pierDipi]

/cc @Cali0707

[codecov]

Codecov Report

Attention: Patch coverage is 55.55556% with 8 lines in your changes are missing coverage. Please review.

Project coverage is 54.17%. Comparing base (551691c) to head (f5e1aad).
Report is 27 commits behind head on main.

Files	Patch %	Lines
...ol-plane/pkg/reconciler/trigger/v2/controllerv2.go	20.00%	3 Missing and 1 partial ⚠️
...l-plane/pkg/reconciler/consumergroup/controller.go	57.14%	2 Missing and 1 partial ⚠️
control-plane/pkg/reconciler/source/controller.go	83.33%	1 Missing ⚠️

Additional details and impacted files

@@              Coverage Diff              @@
##               main    #3818       +/-   ##
=============================================
- Coverage     73.90%   54.17%   -19.74%     
  Complexity      876      876               
=============================================
  Files           100      346      +246     
  Lines          3407    18199    +14792     
  Branches        288      288               
=============================================
+ Hits           2518     9859     +7341     
- Misses          716     7416     +6700     
- Partials        173      924      +751     

Flag	Coverage Δ
java-unittests	73.93% <ø> (+0.02%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[pierDipi]

/retest-required

[pierDipi]

/test reconciler-tests-namespaced-broker

[pierDipi]

/test reconciler-tests-namespaced-broker

[Cali0707]

/retest

[Cali0707]

/test all

[pierDipi]

/retest-required

[pierDipi]

/retest-required

[pierDipi]

/retest-required

[pierDipi]

/retest-required

[matzew]

/lgtm
/approve

[knative-prow]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: matzew, pierDipi

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:

• OWNERS [matzew,pierDipi]

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

[pierDipi]

/retest

[pierDipi]

/retest-required

[pierDipi]

/retest-required

[knative-prow]

@pierDipi: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name	Commit	Details	Required	Rerun command
reconciler-tests-namespaced-broker_eventing-kafka-broker_main	f5e1aad	link	true	/test reconciler-tests-namespaced-broker
reconciler-tests_eventing-kafka-broker_main	f5e1aad	link	true	/test reconciler-tests

Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[knative-prow-robot]

PR needs rebase.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[Cali0707]

@pierDipi do we still need this PR with the new flake fixes we merged?