Project

General

Profile

Actions

Bug #27072

closed

pre-migration step in foreman maintain asks for permission twice.

Added by Kavita Gaikwad almost 5 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Normal
Category:
-
Target version:
-
Fixed in Releases:
Found in Releases:

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:

Actions #1

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
Actions #2

Updated by Amit Upadhye almost 5 years ago

  • Assignee changed from Anurag Patel to Amit Upadhye
Actions #3

Updated by Amit Upadhye over 4 years ago

  • Status changed from Ready For Testing to Closed
Actions #4

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

Actions #5

Updated by Martin Bacovsky over 4 years ago

  • Status changed from Ready For Testing to Closed
Actions

Also available in: Atom PDF