Uploaded image for project: 'RHEL Conversions'
  1. RHEL Conversions
  2. RHELC-970

Add `convert2rhel analysis` as a new CLI subcommand

XMLWordPrintable

    • 5
    • 1.4 GA, 2023-Q2
    • Testable

      With the introduction of the new experimental environment variable to trigger the pre conversion analysis workflow, we need to iterate on the solution once more to make it official in the Convert2RHEL workflow, and the way to go with that is to add a subcommand structure to our CLI that will allow us to have better control of this workflow (and possible, new ones in the future)

      Reference

      • gist that represents a WIP patch for those subcommands.

      Acceptance Criteria

      • A new subcommand is added called analyze¹ that will take over the experimental environment variable added in the tech preview
        • The environment variable can be found here
      • A default subcommand is added called convert to mimic keep the default behavior of running Convert2RHEL without any subcommands²
      • Adapt the code to accept both the experimental environment variable for now, and, the new subcommand.
        • We need to warn the user that the variable will go away in the next release, but for backward compatibility, we should keep it for one more release.

      Footnotes

      [1]: Voting that happened in our private group chat to decide the name of the new subcommand https://chat.google.com/room/AAAAj41tKcs/CUC9USbCBI8
      [2]: There is no way to say that a single subcommand is optional (not required), so we need to introduce the convert subcommand and make it the default one when nothing else is provided. You can see this gist that has a minimal code draft for this feature. The gist may be too much and a better solution may come out after some discussion, so take this gist as a base for understanding the problem, not necessarily the same code needs to be applied.

              prwatson@redhat.com Preston Watson
              rolivier@redhat.com Rodolfo Olivieri
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Created:
                Updated:
                Resolved: