Closed sherine-k closed 3 weeks ago
@sherine-k: This pull request references Jira Issue OCPBUGS-34020, which is valid. The bug has been moved to the POST state.
Requesting review from QA contact: /cc @zhouying7780
The bug has been updated to refer to the pull request using the external bug tracker.
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: sherine-k
The full list of commands accepted by this bot can be found here.
The pull request process is described here
@sherine-k: This pull request references Jira Issue OCPBUGS-34020, which is valid.
Requesting review from QA contact: /cc @zhouying7780
The bug has been updated to refer to the pull request using the external bug tracker.
/retest
/cherrypick release-4.16
@sherine-k: once the present PR merges, I will cherry-pick it on top of release-4.16 in a new PR and assign it to you.
/retest-required
/retest
/lgtm
/label qe-approved
@sherine-k: Jira Issue OCPBUGS-34020: All pull requests linked via external trackers have merged:
Jira Issue OCPBUGS-34020 has been moved to the MODIFIED state.
@sherine-k: new pull request created: #879
[ART PR BUILD NOTIFIER]
This PR has been included in build oc-mirror-plugin-container-v4.17.0-202406131343.p0.g157eb08.assembly.stream.el9 for distgit oc-mirror-plugin. All builds following this will include this PR.
Description
This PR contains:
o.Mirror.Run
, the batch now evaluates the image type:SafeError
to caller. The executor will carry on generating cluster resources.Worker
function is typed :UnsafeError
vsSafeError
. TheExecutor
orDelete
will resume activities if the error is of typeSafeError
isFailSafe
is commented: this function was originally used to determine if errors wereUnsafeError
orSafeError
. Its use is deferred to later (see OCPBUGS-34020 comments for decision details).isRetryable
: all errors that are retryable are considered unsafe (fail fast).Fixes OCPBUGS-34020
Type of change
Please delete options that are not relevant.
How Has This Been Tested?
With the following imagesetconfig, a mirror to mirror was performed.
Expected Outcome
Although the second image is in error (unauthorized), the mirroring continues. ITMS is generated. No IDMS, no CatalogSource. ⚠️ Please note that exit code is 0 in the end.