Bug #27072
closedpre-migration step in foreman maintain asks for permission twice.
Description
Cloned from https://bugzilla.redhat.com/show_bug.cgi?id=1710305
Description of problem:
pre-migration step in foreman maintain asks for permission twice.
Version-Release number of selected component (if applicable):
@satellite 6.5.0
@rubygem-foreman_maintain-0.3.3-1.el7sat.noarch
How reproducible:
always
Steps to Reproduce:
1. Run #foreman-maintain upgrade run --target-version 6.5
2. on "Procedures before migrating to Satellite 6.5" it asks twice for permission to continue.
Actual results:
Continue with [Procedures before migrating to Satellite 6.5], [y(yes), n(no), q(quit)] y
Continue with [Procedures before migrating to Satellite 6.5], [y(yes), n(no), q(quit)] y
Running Procedures before migrating to Satellite 6.5
================================================================================
disable active sync plans:
- Total 0 sync plans are now disabled. [OK]
--------------------------------------------------------------------------------
Expected results:
Continue with [Procedures before migrating to Satellite 6.5], [y(yes), n(no), q(quit)] y
Running Procedures before migrating to Satellite 6.5
================================================================================
disable active sync plans:
- Total 0 sync plans are now disabled. [OK]
--------------------------------------------------------------------------------
Additional info:
Updated by The Foreman Bot almost 5 years ago
- Status changed from New to Ready For Testing
- Pull request https://github.com/theforeman/foreman_maintain/pull/274 added
Updated by Amit Upadhye almost 5 years ago
- Assignee changed from Anurag Patel to Amit Upadhye
Updated by Amit Upadhye over 4 years ago
- Status changed from Ready For Testing to Closed
Applied in changeset foreman_maintain|0014c2e853f85d8e4f4633b50c4413a099efdd7c.
Updated by Martin Bacovsky over 4 years ago
- Status changed from Closed to Ready For Testing
- Assignee changed from Amit Upadhye to Martin Bacovsky
- Pull request https://github.com/theforeman/foreman_maintain/pull/278 added
The previous fix caused regression on quitting after failed preparation steps. Another proposal sent as https://github.com/theforeman/foreman_maintain/pull/278
Updated by Martin Bacovsky over 4 years ago
- Status changed from Ready For Testing to Closed
Applied in changeset foreman_maintain|7c070f7e3781af41a1d384f5e6f8faa9a8e7f3a6.