Bug 1618804 - [v2v] Schedule Unschedule Migration does not seem to work correctly
Summary: [v2v] Schedule Unschedule Migration does not seem to work correctly
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: V2V
Version: 5.9.4
Hardware: Unspecified
OS: Unspecified
high
medium
Target Milestone: GA
: 5.10.0
Assignee: Michael Ro
QA Contact: Kedar Kulkarni
Red Hat CloudForms Documentation
URL:
Whiteboard: v2v
Depends On:
Blocks: 1619654
TreeView+ depends on / blocked
 
Reported: 2018-08-17 16:09 UTC by Kedar Kulkarni
Modified: 2019-02-12 16:49 UTC (History)
11 users (show)

Fixed In Version: 5.10.0.21
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
: 1619654 (view as bug list)
Environment:
Last Closed: 2019-02-12 16:49:32 UTC
Category: ---
Cloudforms Team: V2V
Target Upstream Version:
Embargoed:
kkulkarn: automate_bug+


Attachments (Terms of Use)
ScreenGIF1 (3.49 MB, image/gif)
2018-08-17 16:09 UTC, Kedar Kulkarni
no flags Details
Screenshot (136.38 KB, image/png)
2018-08-17 18:05 UTC, Kedar Kulkarni
no flags Details
ScreenGIF2 (65.75 KB, image/png)
2018-09-07 19:24 UTC, Kedar Kulkarni
no flags Details
ScreenGIF4 (2.27 MB, video/mp4)
2018-09-07 19:25 UTC, Kedar Kulkarni
no flags Details

Description Kedar Kulkarni 2018-08-17 16:09:29 UTC
Created attachment 1476647 [details]
ScreenGIF1

Description of problem:
Schedule Unschedule Migration does not seem to work correctly , see attached screenGIF

Version-Release number of selected component (if applicable):
5.9.4.4

How reproducible:
Believe 100%

Steps to Reproduce:
1.See attached GIF
2.
3.

Actual results:
Unschedule does not seem to work well. 

Expected results:

Once you schedule migration, button should change to Unschedule, and let you unschedule migration and after unscheduling, it should change to Schedule again.


Additional info:

Comment 3 Kedar Kulkarni 2018-08-17 18:05:08 UTC
Created attachment 1476682 [details]
Screenshot

I also found 500 API error. See screenshot.

Comment 4 Kedar Kulkarni 2018-08-17 18:06:45 UTC
Please note, in some cases Unschedule button does show up correctly but throws Internal Server Error 500 as shown in attached screenshot. I believe thorough investigation of this feature is needed.

Comment 8 Allen W 2018-08-20 23:28:58 UTC
Merged!

Comment 10 Satoe Imaishi 2018-08-23 12:09:02 UTC
https://github.com/ManageIQ/manageiq-v2v/pull/602

Comment 11 Kedar Kulkarni 2018-09-07 19:24:10 UTC
On latest 5.10.0.14 build,  Just before it starts running at scheduled time, "Unschedule" button changes to "Schedule" and then it starts running. So there is a brief window where :
1) User may think it is not scheduled any more.
2) User may try to Schedule it again. 

We should fix it.
See attached Video.

Comment 12 Kedar Kulkarni 2018-09-07 19:24:59 UTC
Created attachment 1481657 [details]
ScreenGIF2

Happening on Plans Completed Page.

Comment 13 Kedar Kulkarni 2018-09-07 19:25:46 UTC
Created attachment 1481658 [details]
ScreenGIF4

Happening on Plans Not started Page.

Comment 19 Allen W 2018-10-16 19:06:28 UTC
and merged!

Comment 20 CFME Bot 2018-10-17 13:47:48 UTC
New commit detected on ManageIQ/manageiq-v2v/hammer:

https://github.com/ManageIQ/manageiq-v2v/commit/43f99c2968faf92fbd0ee3afde8c50b6a72694e9
commit 43f99c2968faf92fbd0ee3afde8c50b6a72694e9
Author:     Michael Ro <mikerodev>
AuthorDate: Tue Oct 16 14:24:57 2018 -0400
Commit:     Michael Ro <mikerodev>
CommitDate: Tue Oct 16 14:24:57 2018 -0400

    Merge pull request #711 from mturley/710-no-confusing-schedule-button

    [#710] Don't show Schedule button when scheduled migration is about to start

    (cherry picked from commit a561e5e20170af108c62e709e916d8917bd43d73)

    https://bugzilla.redhat.com/show_bug.cgi?id=1618804

 app/javascript/react/screens/App/Overview/components/Migrations/MigrationsCompletedList.js | 20 +-
 app/javascript/react/screens/App/Overview/components/Migrations/MigrationsNotStartedList.js | 25 +-
 app/javascript/react/screens/App/Overview/components/Migrations/ScheduleMigrationButton.js | 14 +-
 app/javascript/react/screens/App/Overview/components/Migrations/helpers/getPlanScheduleInfo.js | 12 +
 4 files changed, 50 insertions(+), 21 deletions(-)

Comment 21 Kedar Kulkarni 2018-10-26 14:40:52 UTC
In 5.10.0.21 it does work correctly.


Note You need to log in before you can comment on or make changes to this bug.