Closed lmzuccarelli closed 3 months ago
@lmzuccarelli: This pull request references Jira Issue OCPBUGS-33121, 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: lmzuccarelli
The full list of commands accepted by this bot can be found here.
The pull request process is described here
@lmzuccarelli: This pull request references Jira Issue OCPBUGS-33121, which is valid.
Requesting review from QA contact: /cc @zhouying7780
/retest
Usually before removing flags on GA version of oc-mirror like v1, it is necessary to deprecate them and let the customers know in the docs they need to remove the flag from their automatons / scripts. It gives them time to change their side.
Removing flags directly without deprecating could cause breaking automation which uses the flag removed.
Suggestion:
-Based on the apiVersion of the image set configuration (v2alpha1 for example) it would be possible to identify that the customer wants to use v2 and print a message the correct flag is --v2 and not -v2
and stop the flow so the customer can fix the mistake.
/test unit
@lmzuccarelli: all tests passed!
Full PR test history. Your PR dashboard.
/lgtm
@lmzuccarelli: Jira Issue OCPBUGS-33121: All pull requests linked via external trackers have merged:
Jira Issue OCPBUGS-33121 has been moved to the MODIFIED state.
[ART PR BUILD NOTIFIER]
This PR has been included in build oc-mirror-plugin-container-v4.17.0-202406052200.p0.ge164546.assembly.stream.el9 for distgit oc-mirror-plugin. All builds following this will include this PR.
Description
This fix addresses the issue faced in using the --v2 (used to invoke v2 of oc mirror) and accidentally using -v2 (used for verbose logging)
This fix will only be for 4.17 onwards until we deprecate v1
Fixes # OCPBUGS-33121
Type of change
How Has This Been Tested?
Locally and unit tests
Expected Outcome
console message should show