[fix][ci] succeed only if all needs pass

[tisonkun]

This OR logic has been introduced by #17614.

I don't know why it's added in the first place - we shouldn't have an OR here. But anyway since pulsar-client-cpp moved out, it should be reasonable to fix the workflow.

Otherwise, if system tests passed but other tests failed or canceled, the required status can still pass.

Documentation

• doc-required
  (Your PR needs to update docs and you will update later)

• doc-not-needed
  (Please explain why)

• doc
  (Your PR contains doc changes)

• doc-complete
  (Docs have been already added)

Matching PR in forked repository

PR in forked repository: tisonkun#5

[tisonkun]

cc @lhotari @nicoloboschi

[lhotari]

I don't know why it's added in the first place - we shouldn't have an OR here. But anyway since pulsar-client-cpp moved out, it should be reasonable to fix the workflow.

The point of it was that when cpp build still existed, and there was a cpp only change, the unit & integration tests would have been skipped. the logic was for that purpose. The problem was that it wasn't properly removed in #17881 .