Silence NotFound when get task resource

[honnix]

Tracking issue

Closes #4387

Docs link

Describe your changes

Do not issue warning log when the error is NotFound because that is in most cases intended.

Check all the applicable boxes

• I updated the documentation accordingly.
• All new and existing tests passed.
• All commits are signed-off.

Setup Process

Screenshots

Note to reviewers

Related PRs

[codecov]

Codecov Report

Attention: 8 lines in your changes are missing coverage. Please review.

Comparison is base (c4b040b) 59.59% compared to head (2617411) 59.98%.

❗ Current head 2617411 differs from pull request most recent head 412792a. Consider uploading reports for the commit 412792a to get more accurate results

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #4388      +/-   ##
==========================================
+ Coverage   59.59%   59.98%   +0.39%     
==========================================
  Files         632      562      -70     
  Lines       53623    40291   -13332     
==========================================
- Hits        31957    24169    -7788     
+ Misses      19146    13772    -5374     
+ Partials     2520     2350     -170     

Flag	Coverage Δ
unittests	?

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

Files	Coverage Δ
flyteadmin/pkg/errors/errors.go	65.82% <100.00%> (+6.99%)	⬆️
flyteadmin/pkg/manager/impl/util/shared.go	68.02% <100.00%> (+3.46%)	⬆️
.../pkg/manager/impl/executions/quality_of_service.go	64.34% <0.00%> (+2.22%)	⬆️
flyteadmin/pkg/manager/impl/executions/queues.go	84.61% <0.00%> (-7.45%)	⬇️
flyteadmin/pkg/manager/impl/util/resources.go	91.52% <0.00%> (+0.85%)	⬆️
...g/executioncluster/impl/random_cluster_selector.go	81.25% <0.00%> (+4.13%)	⬆️
flyteadmin/pkg/manager/impl/execution_manager.go	73.08% <40.00%> (+2.54%)	⬆️

... and 563 files with indirect coverage changes

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

[honnix]

Looks like flaky tests. The CI failure is not related to this change I believe.

[hamersaw]

This is a great catch and thanks for the PR! A few nits, mostly my import organization OCD .. we should really automate this soon.