Currently, the tool does not update the custom facts generated during the analysis execution. This prevents us from collecting metrics in Insights when a user wants to execute a pre analysis through the web UI (and also, it will prevent metrics being collected when the user runs convert2rhel in the terminal as well).

Acceptance Criteria

• convert2rhel calls sub-man to update the rhsm custom facts under these conditions
  1. after the analysis execution is done or when the analysis execution fails
  2. when sub-man is installed and should_subscribe() returns False (should_subscribe() returning True means that the system would get unregistered during the rollback and with that all facts are removed from the rhsm servers so there's no point in uploading the facts)
• Once released in downstream, rhn-engineering-lstejska is notified so he can remove the workaround of calling the sub-man facts refresh from the template https://github.com/theforeman/foreman_remote_execution/blob/master/app/views/templates/script/convert2rhel_analyze.erb#L35
  • The workaround was introduced under RHELC-1280