Describe the bug
In PR #2388 upgrade tests have been introduced. They fail due to unresolved #2665. WG decided to merge them, but instead of failing warning is used if FailOnMissingEvents configuration is set to false. It's okay for now, but in that case the TestProbe test should have status of Skipped instead of Pass.
Expected behavior
TestProbe should be skipped if issues are found and FailOnMissingEvents==false
To Reproduce
Run eventing upgrade tests and observe Silended FAIL messages and state of TestProbe.
Knative release version
master
Describe the bug
In PR #2388 upgrade tests have been introduced. They fail due to unresolved #2665. WG decided to merge them, but instead of failing warning is used if
FailOnMissingEventsconfiguration is set to false. It's okay for now, but in that case theTestProbetest should have status of Skipped instead of Pass.Expected behavior
TestProbeshould be skipped if issues are found andFailOnMissingEvents==falseTo Reproduce
Run eventing upgrade tests and observe
Silended FAILmessages and state ofTestProbe.Knative release version
master