Guard Knative readiness predicates against failed reads

[kahirokunn]

Changes

• Return API read errors before accessing KnativeServing or KnativeEventing status.

Release Note

NONE

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 65.84%. Comparing base (71eefe9) to head (b2d8d06).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2348   +/-   ##
=======================================
  Coverage   65.84%   65.84%           
=======================================
  Files          56       56           
  Lines        2673     2673           
=======================================
  Hits         1760     1760           
  Misses        777      777           
  Partials      136      136           

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[kahirokunn]

@dsimansk Hi ✋ PTAL. Thank you 🙏

[dsimansk]

@kahirokunn can you rebase pls to get verify codegen passing?

[kahirokunn]

@kahirokunn can you rebase pls to get verify codegen passing?

Thank you. I fixed it 🙏

[dsimansk]

/approve
/lgtm

[knative-prow]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dsimansk, kahirokunn

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details

Needs approval from an approver in each of these files:

• OWNERS [dsimansk]

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