operator
operator copied to clipboard
Revert "Propagate manifests paths earlier to operator status (#2106)"
This reverts commit 2ac3a1eaad81a89738cd7c4d13b6374c0d198b3f.
Per my last comment on #2106. Let's make sure we can revert if needed. Otherwise I'll close based on outcomes of discussion.
/hold
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: dsimansk
The full list of commands accepted by this bot can be found here.
The pull request process is described here
- ~~OWNERS~~ [dsimansk]
Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment
Codecov Report
:white_check_mark: All modified and coverable lines are covered by tests.
:white_check_mark: Project coverage is 62.16%. Comparing base (2ac3a1e) to head (cd1a2e2).
:warning: Report is 31 commits behind head on main.
Additional details and impacted files
@@ Coverage Diff @@
## main #2107 +/- ##
=======================================
Coverage 62.16% 62.16%
=======================================
Files 49 49
Lines 2318 2318
=======================================
Hits 1441 1441
Misses 785 785
Partials 92 92
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
:rocket: New features to boost your workflow:
- :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
@dsimansk I so far did not see anything negatively influencing the operator with the patch merged before. Let's hold on this revert for now, before we find out anything else.
@dsimansk @houshengbo is there any follow up here?
@dsimansk @houshengbo is there any follow up here?