Bug 1696423 - [v2v] [RFE] Hidden UI changes to support Conversion Hosts settings page
Summary: [v2v] [RFE] Hidden UI changes to support Conversion Hosts settings page
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat CloudForms Management Engine
Classification: Red Hat
Component: V2V
Version: 5.10.2
Hardware: All
OS: All
medium
medium
Target Milestone: GA
: 5.10.3
Assignee: Fabien Dupont
QA Contact: Shveta
Red Hat CloudForms Documentation
URL:
Whiteboard: V2V
Depends On: 1693339
Blocks:
TreeView+ depends on / blocked
 
Reported: 2019-04-04 19:58 UTC by Satoe Imaishi
Modified: 2019-04-30 20:32 UTC (History)
3 users (show)

Fixed In Version: 5.10.3.1
Doc Type: If docs needed, set a value
Doc Text:
Clone Of: 1693339
Environment:
Last Closed: 2019-04-30 20:32:48 UTC
Category: ---
Cloudforms Team: V2V
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Comment 2 CFME Bot 2019-04-05 00:13:30 UTC
New commits detected on ManageIQ/manageiq-v2v/hammer:

https://github.com/ManageIQ/manageiq-v2v/commit/75686ef4ca01d74f2c7af03c4c5bd2ef7d5d2b3b
commit 75686ef4ca01d74f2c7af03c4c5bd2ef7d5d2b3b
Author:     Michael Ro <mikerodev>
AuthorDate: Thu Jan 24 15:31:21 2019 -0500
Commit:     Michael Ro <mikerodev>
CommitDate: Thu Jan 24 15:31:21 2019 -0500

    Merge pull request #851 from mturley/settings-tabs

    Add routed tabs to the Settings page to prepare for Conversion Hosts settings

    (cherry picked from commit 339f3bf847c95c693f45af5abc107a0369f912ca)

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

 app/javascript/react/config/config.js | 5 +
 app/javascript/react/screens/App/Settings/Settings.js | 113 +-
 app/javascript/react/screens/App/Settings/__tests__/Settings.test.js | 36 -
 app/javascript/react/screens/App/Settings/__tests__/__snapshots__/Settings.test.js.snap | 97 -
 app/javascript/react/screens/App/Settings/index.js | 16 +-
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/ConversionHostsSettings.js | 5 +
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/index.js | 3 +
 app/javascript/react/screens/App/Settings/screens/GeneralSettings/GeneralSettings.js | 83 +
 app/javascript/react/screens/App/Settings/screens/GeneralSettings/__tests__/GeneralSettings.test.js | 36 +
 app/javascript/react/screens/App/Settings/screens/GeneralSettings/__tests__/__snapshots__/GeneralSettings.test.js.snap | 73 +
 app/javascript/react/screens/App/Settings/screens/GeneralSettings/index.js | 22 +
 11 files changed, 260 insertions(+), 229 deletions(-)


https://github.com/ManageIQ/manageiq-v2v/commit/8904746e3589dae8b3dcee23dc398e4022d38d7f
commit 8904746e3589dae8b3dcee23dc398e4022d38d7f
Author:     Michael Ro <mikerodev>
AuthorDate: Fri Jan 25 16:11:30 2019 -0500
Commit:     Michael Ro <mikerodev>
CommitDate: Fri Jan 25 16:11:30 2019 -0500

    Merge pull request #858 from mturley/857-conversion-hosts-settings-skeleton

    [#857] Add skeleton for Conversion Hosts Settings components

    (cherry picked from commit 6b37e5474f5abd1cbacfa99f7fbb1da379d4064f)

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

 app/javascript/react/screens/App/Settings/Settings.js | 8 +-
 app/javascript/react/screens/App/Settings/SettingsActions.js | 24 +-
 app/javascript/react/screens/App/Settings/SettingsConstants.js | 3 +
 app/javascript/react/screens/App/Settings/SettingsReducer.js | 39 +-
 app/javascript/react/screens/App/Settings/__tests__/Settings.test.js | 10 +
 app/javascript/react/screens/App/Settings/__tests__/SettingsActions.test.js | 34 +
 app/javascript/react/screens/App/Settings/__tests__/SettingsReducer.test.js | 55 +-
 app/javascript/react/screens/App/Settings/__tests__/__snapshots__/Settings.test.js.snap | 51 +
 app/javascript/react/screens/App/Settings/__tests__/__snapshots__/SettingsActions.test.js.snap | 24 +
 app/javascript/react/screens/App/Settings/__tests__/__snapshots__/SettingsReducer.test.js.snap | 112 +
 app/javascript/react/screens/App/Settings/index.js | 4 +-
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/ConversionHostsSettings.js | 64 +-
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/__tests__/ConversionHostsSettings.test.js | 34 +
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard.js | 16 +
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostsEmptyState.js | 16 +
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostsList.js | 17 +
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/index.js | 16 +-
 17 files changed, 515 insertions(+), 12 deletions(-)


https://github.com/ManageIQ/manageiq-v2v/commit/04e0bea269cec47673ff43fc499e88bb819c0f2b
commit 04e0bea269cec47673ff43fc499e88bb819c0f2b
Author:     Michael Ro <mikerodev>
AuthorDate: Tue Jan 29 16:39:31 2019 -0500
Commit:     Michael Ro <mikerodev>
CommitDate: Tue Jan 29 16:39:31 2019 -0500

    Merge pull request #863 from mturley/conversion-host-wizard-skeleton

    Add skeleton for Configure Conversion Host wizard steps

    (cherry picked from commit f6ecbbcd880d27cba767aea70531509b8d138cf8)

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

 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard.js | 16 -
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizard.js | 117 +
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizardAuthenticationStep/ConversionHostWizardAuthenticationStep.js | 10 +
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizardAuthenticationStep/index.js | 3 +
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizardBody.js | 34 +
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizardConstants.js | 6 +
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizardHostsStep/ConversionHostWizardHostsStep.js | 10 +
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizardHostsStep/index.js | 3 +
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizardLocationStep/ConversionHostWizardLocationStep.js | 10 +
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizardLocationStep/index.js | 3 +
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizardResultsStep/ConversionHostWizardResultsStep.js | 10 +
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizardResultsStep/index.js | 3 +
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/index.js | 3 +
 13 files changed, 212 insertions(+), 16 deletions(-)


https://github.com/ManageIQ/manageiq-v2v/commit/8b3ee76eaee27376c5f18ff3f7cc325b3cd23cc9
commit 8b3ee76eaee27376c5f18ff3f7cc325b3cd23cc9
Author:     Mike Turley <mike.turley.umass.edu>
AuthorDate: Tue Jan 29 18:11:46 2019 -0500
Commit:     Mike Turley <mike.turley.umass.edu>
CommitDate: Tue Jan 29 18:11:46 2019 -0500

    Merge pull request #864 from mzazrivec/add_empty_state_for_coversion_hosts

    Conversion Hosts: add initial button for empty state screen

    (cherry picked from commit eab39c449eceec38f621bc053ddf8642b0f8d2ab)

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

 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostsEmptyState.js | 13 +-
 1 file changed, 10 insertions(+), 3 deletions(-)


https://github.com/ManageIQ/manageiq-v2v/commit/5d8c17ae03a55c9c6316cddbd35f7cbc3cd8883c
commit 5d8c17ae03a55c9c6316cddbd35f7cbc3cd8883c
Author:     Mike Turley <mike.turley.umass.edu>
AuthorDate: Tue Feb  5 15:54:52 2019 -0500
Commit:     Mike Turley <mike.turley.umass.edu>
CommitDate: Tue Feb  5 15:54:52 2019 -0500

    Merge pull request #871 from mturley/conv-host-wizard-step1

    Conversion Host Configuration Wizard Step #1 (Location), and introducing common reducers

    (cherry picked from commit a0ee080c47502453d76ab6d7f0629471438c184f)

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

 app/javascript/common/constants.js | 16 +
 app/javascript/react/screens/App/Mappings/Mappings.js | 8 +-
 app/javascript/react/screens/App/Mappings/MappingsActions.js | 2 +-
 app/javascript/react/screens/App/Mappings/index.js | 15 +-
 app/javascript/react/screens/App/Mappings/screens/MappingWizard/MappingWizardConstants.js | 11 +-
 app/javascript/react/screens/App/Mappings/screens/MappingWizard/__tests__/helpers.test.js | 6 +-
 app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/MappingWizardClustersStep.js | 3 +-
 app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/MappingWizardClustersStepActions.js | 17 +-
 app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/MappingWizardClustersStepConstants.js | 8 -
 app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/MappingWizardClustersStepReducer.js | 25 +-
 app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/__tests__/MappingWizardClusterStepActions.test.js | 27 +-
 app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/__tests__/MappingWizardClustersStep.test.js | 2 +-
 app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/__tests__/MappingWizardClustersStepReducers.test.js | 74 +-
 app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/__tests__/__snapshots__/MappingWizardClusterStepActions.test.js.snap | 24 -
 app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/__tests__/__snapshots__/MappingWizardClustersStepReducers.test.js.snap | 121 -
 app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/__tests__/__snapshots__/index.test.js.snap | 4 -
 app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/__tests__/index.test.js | 8 +-
 app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/components/ClustersStepForm/clustersStepForm.fixtures.js | 3 +-
 app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/index.js | 15 +-
 app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/mappingWizardClustersStep.fixtures.js | 158 +-
 app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardDatastoresStep/__tests__/MappingWizardDatastoresStep.test.js | 3 +-
 app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardDatastoresStep/__tests__/MappingWizardDatastoresStepActions.test.js | 5 +-
 app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardNetworksStep/__tests__/MappingWizardNetworksStep.test.js | 3 +-
 app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardNetworksStep/__tests__/MappingWizardNetworksStepActions.test.js | 5 +-
 app/javascript/react/screens/App/Overview/Overview.js | 8 +-
 app/javascript/react/screens/App/Overview/OverviewActions.js | 8 -
 app/javascript/react/screens/App/Overview/OverviewConstants.js | 8 -
 app/javascript/react/screens/App/Overview/OverviewReducer.js | 34 +-
 app/javascript/react/screens/App/Overview/OverviewValidators.js | 5 -
 app/javascript/react/screens/App/Overview/__test__/__snapshots__/index.test.js.snap | 5 +
 app/javascript/react/screens/App/Overview/__test__/helpers.test.js | 18 +-
 app/javascript/react/screens/App/Overview/__test__/index.test.js | 8 +-
 app/javascript/react/screens/App/Overview/helpers.js | 15 -
 app/javascript/react/screens/App/Overview/index.js | 8 +-
 app/javascript/react/screens/App/Settings/SettingsActions.js | 17 +-
 app/javascript/react/screens/App/Settings/SettingsConstants.js | 1 +
 app/javascript/react/screens/App/Settings/SettingsReducer.js | 8 +-
 app/javascript/react/screens/App/Settings/__tests__/SettingsActions.test.js | 4 +-
 app/javascript/react/screens/App/Settings/__tests__/__snapshots__/SettingsReducer.test.js.snap | 12 +
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/ConversionHostsSettings.js | 79 +-
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/__tests__/ConversionHostsSettings.test.js | 24 +-
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizard.js | 39 +-
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizardLocationStep/ConversionHostWizardLocationStep.js | 119 +-
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizardLocationStep/index.js | 24 +-
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizardSelectors.js | 10 +
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/index.js | 18 +-
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostsEmptyState.js | 6 +-
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/index.js | 6 +-
 app/javascript/react/screens/App/common/forms/BootstrapSelect.js | 21 +-
 app/javascript/redux/common/providers/__tests__/providersActions.test.js | 49 +
 app/javascript/redux/common/providers/__tests__/providersHelpers.test.js | 17 +
 app/javascript/redux/common/providers/__tests__/providersReducer.test.js | 82 +
 app/javascript/redux/common/providers/providersActions.js | 15 +
 app/javascript/redux/common/providers/providersConstants.js | 8 +
 app/javascript/redux/common/providers/providersHelpers.js | 15 +
 app/javascript/redux/common/providers/providersReducer.js | 48 +
 app/javascript/redux/common/providers/providersValidators.js | 6 +
 app/javascript/redux/common/targetResources/__tests__/__snapshots__/targetResourcesActions.test.js.snap | 25 +
 app/javascript/redux/common/targetResources/__tests__/__snapshots__/targetResourcesReducer.test.js.snap | 99 +
 app/javascript/redux/common/targetResources/__tests__/targetResourcesActions.test.js | 39 +
 app/javascript/redux/common/targetResources/__tests__/targetResourcesReducer.test.js | 76 +
 app/javascript/redux/common/targetResources/targetResources.fixtures.js | 152 +
 app/javascript/redux/common/targetResources/targetResourcesActions.js | 15 +
 app/javascript/redux/common/targetResources/targetResourcesConstants.js | 11 +
 app/javascript/redux/common/targetResources/targetResourcesReducer.js | 35 +
 app/javascript/redux/reducers/index.js | 6 +-
 66 files changed, 1121 insertions(+), 645 deletions(-)


https://github.com/ManageIQ/manageiq-v2v/commit/96dcc9ebe2c93272cd82d7bc831dfab17c11beb4
commit 96dcc9ebe2c93272cd82d7bc831dfab17c11beb4
Author:     Mike Turley <mike.turley.umass.edu>
AuthorDate: Tue Feb  5 18:33:44 2019 -0500
Commit:     Mike Turley <mike.turley.umass.edu>
CommitDate: Tue Feb  5 18:33:44 2019 -0500

    Merge pull request #874 from mzazrivec/conversion_hosts_list_view_phase_1

    Conversion Hosts: list view phase 1

    (cherry picked from commit e379015fc4764ce0ab70e323313115e30443df67)

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

 app/javascript/react/screens/App/Settings/Settings.scss | 4 +
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostsList.js | 56 +-
 2 files changed, 54 insertions(+), 6 deletions(-)


https://github.com/ManageIQ/manageiq-v2v/commit/d4f7d9fed8c4ac815305353e79573d447bf6eace
commit d4f7d9fed8c4ac815305353e79573d447bf6eace
Author:     Milan Zázrivec <mzazrivec>
AuthorDate: Mon Feb 25 08:38:42 2019 -0500
Commit:     Milan Zázrivec <mzazrivec>
CommitDate: Mon Feb 25 08:38:42 2019 -0500

    Merge pull request #880 from mturley/conv-host-wizard-step3

    Conversion Host Configuration Wizard - Step #3 - Authentication

    (cherry picked from commit c097fb87593a3cfa013bdb7824d569b896d21b41)

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

 app/javascript/react/screens/App/App.scss | 5 +
 app/javascript/react/screens/App/Overview/Overview.scss | 4 +-
 app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardVMStep/components/CSVDropzoneField.js | 51 +-
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizard.js | 2 +-
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizardAuthenticationStep/ConversionHostWizardAuthenticationStep.js | 137 +-
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizardAuthenticationStep/index.js | 22 +-
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizardConstants.js | 3 +
 app/javascript/react/screens/App/common/forms/BootstrapSelect.js | 2 +-
 app/javascript/react/screens/App/common/forms/FormField.js | 41 +-
 app/javascript/react/screens/App/common/forms/TextFileInput/TextFileInput.js | 64 +
 app/javascript/react/screens/App/common/forms/TextFileInput/TextFileInput.scss | 8 +
 app/javascript/react/screens/App/common/forms/TextFileInput/index.js | 3 +
 package.json | 2 +-
 yarn.lock | 11 +-
 14 files changed, 299 insertions(+), 56 deletions(-)


https://github.com/ManageIQ/manageiq-v2v/commit/e328552343e8178a634cb36fbba15cbb04a01d92
commit e328552343e8178a634cb36fbba15cbb04a01d92
Author:     Milan Zázrivec <mzazrivec>
AuthorDate: Mon Mar  4 08:34:33 2019 -0500
Commit:     Milan Zázrivec <mzazrivec>
CommitDate: Mon Mar  4 08:34:33 2019 -0500

    Merge pull request #883 from mturley/conv-host-wizard-step4

    Conversion Host Configuration Wizard - Step #4 - Results (without passing SSH keys)

    (cherry picked from commit 200c02265bb70421487574e81a92b70bb7c54615)

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

 app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardResultsStep/MappingWizardResultsStep.js | 46 +-
 app/javascript/react/screens/App/Overview/Overview.scss | 1 -
 app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardResultsStep/PlanWizardResultsStep.js | 56 +-
 app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardResultsStep/PlanWizardResultsStep.scss | 6 -
 app/javascript/react/screens/App/Overview/screens/PlanWizard/components/PlanWizardResultsStep/__tests__/__snapshots__/PlanWizardResultsStep.test.js.snap | 4 +-
 app/javascript/react/screens/App/Settings/SettingsActions.js | 12 +-
 app/javascript/react/screens/App/Settings/SettingsConstants.js | 1 +
 app/javascript/react/screens/App/Settings/SettingsReducer.js | 26 +-
 app/javascript/react/screens/App/Settings/__tests__/__snapshots__/SettingsReducer.test.js.snap | 48 +
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizard.js | 6 +-
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizardResultsStep/ConversionHostWizardResultsStep.js | 61 +-
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizardResultsStep/helpers.js | 22 +
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizardResultsStep/index.js | 29 +-
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/index.js | 5 +-
 app/javascript/react/screens/App/common/ModalWizard/ModalWizard.scss | 6 +
 app/javascript/react/screens/App/common/WizardErrorState.js | 29 +
 app/javascript/react/screens/App/common/WizardLoadingState.js | 22 +
 17 files changed, 303 insertions(+), 77 deletions(-)

Comment 3 CFME Bot 2019-04-05 00:18:28 UTC
New commit detected on ManageIQ/manageiq-v2v/hammer:

https://github.com/ManageIQ/manageiq-v2v/commit/19084c8d3c1334d7bb83ea5b43263cb15b33d186
commit 19084c8d3c1334d7bb83ea5b43263cb15b33d186
Author:     Milan Zázrivec <mzazrivec>
AuthorDate: Tue Feb 12 10:43:34 2019 -0500
Commit:     Milan Zázrivec <mzazrivec>
CommitDate: Tue Feb 12 10:43:34 2019 -0500

    Merge pull request #876 from mturley/conv-host-wizard-step2

    Conversion Host Configuration Wizard - Step #2 - Host(s)

    (cherry picked from commit e64122f30b4a92d36d9bddf297674d5036583674)

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

 app/assets/stylesheets/manageiq-v2v/base.scss | 3 +-
 app/assets/stylesheets/manageiq-v2v/type-ahead-select.scss | 58 +
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizardHostsStep/ConversionHostWizardHostsStep.js | 55 +-
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizardHostsStep/index.js | 21 +-
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizardLocationStep/ConversionHostWizardLocationStep.js | 67 +-
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizardLocationStep/index.js | 20 +-
 app/javascript/react/screens/App/common/forms/TypeAheadSelectField.js | 18 +
 app/javascript/redux/common/targetResources/targetResourcesConstants.js | 2 +-
 8 files changed, 190 insertions(+), 54 deletions(-)

Comment 4 CFME Bot 2019-04-05 17:18:38 UTC
New commits detected on ManageIQ/manageiq-v2v/hammer:

https://github.com/ManageIQ/manageiq-v2v/commit/fcfef595e41bee8cee8db9dc993821900905ee28
commit fcfef595e41bee8cee8db9dc993821900905ee28
Author:     Mike Turley <mike.turley.umass.edu>
AuthorDate: Mon Mar  4 11:58:19 2019 -0500
Commit:     Mike Turley <mike.turley.umass.edu>
CommitDate: Mon Mar  4 11:58:19 2019 -0500

    Merge pull request #882 from mzazrivec/conversion_hosts_list_view_phase_2

    Conversion Host: list view phase 2

    (cherry picked from commit 1daec12682ad9885a71aadf88193a2e81d7c0c7e)

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

 app/javascript/react/screens/App/Settings/SettingsActions.js | 36 +-
 app/javascript/react/screens/App/Settings/SettingsConstants.js | 4 +
 app/javascript/react/screens/App/Settings/SettingsReducer.js | 61 +-
 app/javascript/react/screens/App/Settings/__tests__/__snapshots__/SettingsReducer.test.js.snap | 60 +
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/ConversionHostsSettings.js | 34 +-
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostRemoveButton.js | 26 +
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostsList.js | 121 +-
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/DeleteConversionHostConfirmationModal.js | 66 +
 8 files changed, 355 insertions(+), 53 deletions(-)


https://github.com/ManageIQ/manageiq-v2v/commit/e2c0d2098c557459814f4a9d448402066ce99457
commit e2c0d2098c557459814f4a9d448402066ce99457
Author:     Mike Turley <mike.turley.umass.edu>
AuthorDate: Tue Mar  5 13:35:16 2019 -0500
Commit:     Mike Turley <mike.turley.umass.edu>
CommitDate: Tue Mar  5 13:35:16 2019 -0500

    Merge pull request #894 from mturley/conv-host-wizard-cloud-user

    Conversion Host Configuration Wizard - use correct default cloud-user (with a dash) instead of cloud_user

    (cherry picked from commit 36a087ebf61c32937fbe5378ba087fcdb2f5beba)

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

 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizardAuthenticationStep/ConversionHostWizardAuthenticationStep.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


https://github.com/ManageIQ/manageiq-v2v/commit/4f06604d6511d4fe36cf6668e11b14c4e33ae3ec
commit 4f06604d6511d4fe36cf6668e11b14c4e33ae3ec
Author:     Milan Zázrivec <mzazrivec>
AuthorDate: Tue Mar 12 07:36:01 2019 -0400
Commit:     Milan Zázrivec <mzazrivec>
CommitDate: Tue Mar 12 07:36:01 2019 -0400

    Merge pull request #889 from mturley/conv-host-tasks-polling

    Conversion Host Configuration - List view #3 - Polling for tasks and rendering enable/disable status

    (cherry picked from commit d4ea6f601f4e6b8af24e6e997e626d92b4d0b616)

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

 app/javascript/react/screens/App/Settings/Settings.js | 2 +-
 app/javascript/react/screens/App/Settings/Settings.scss | 9 +
 app/javascript/react/screens/App/Settings/SettingsActions.js | 12 +
 app/javascript/react/screens/App/Settings/SettingsConstants.js | 1 +
 app/javascript/react/screens/App/Settings/SettingsReducer.js | 63 +-
 app/javascript/react/screens/App/Settings/__tests__/SettingsReducer.test.js | 6 +-
 app/javascript/react/screens/App/Settings/__tests__/__snapshots__/Settings.test.js.snap | 1 +
 app/javascript/react/screens/App/Settings/__tests__/__snapshots__/SettingsReducer.test.js.snap | 132 +-
 app/javascript/react/screens/App/Settings/helpers.js | 77 +
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/ConversionHostsSettings.js | 82 +-
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/ConversionHostsSettingsConstants.js | 10 +
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/__tests__/ConversionHostsSettings.test.js | 20 +-
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostRemoveButton.js | 3 +-
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizardHostsStep/ConversionHostWizardHostsStep.js | 26 +-
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizardHostsStep/index.js | 10 +-
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostsList.js | 64 +-
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostsListItem.js | 126 +
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/DeleteConversionHostConfirmationModal.js | 31 +-
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/index.js | 54 +-
 19 files changed, 583 insertions(+), 146 deletions(-)


https://github.com/ManageIQ/manageiq-v2v/commit/d474c1ce19d48bf6c7e4fdf5a1cd257b6780e43b
commit d474c1ce19d48bf6c7e4fdf5a1cd257b6780e43b
Author:     Milan Zázrivec <mzazrivec>
AuthorDate: Thu Mar 14 05:30:56 2019 -0400
Commit:     Milan Zázrivec <mzazrivec>
CommitDate: Thu Mar 14 05:30:56 2019 -0400

    Merge pull request #907 from mturley/conv-host-tasks-meta-fix

    Conversion Host Settings page: Ensure task.meta will always be defined

    (cherry picked from commit dc8eb8ba5c87b14b1284d9c66d6efe827f09edef)

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

 app/javascript/react/screens/App/Settings/helpers.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


https://github.com/ManageIQ/manageiq-v2v/commit/0951eb76f87d1ac17ef7429c14c08ac59de946a5
commit 0951eb76f87d1ac17ef7429c14c08ac59de946a5
Author:     Milan Zázrivec <mzazrivec>
AuthorDate: Thu Mar 14 05:45:07 2019 -0400
Commit:     Milan Zázrivec <mzazrivec>
CommitDate: Thu Mar 14 05:45:07 2019 -0400

    Merge pull request #908 from mturley/conv-host-wizard-ssh-keys

    Conversion Host Configuration Wizard - Properly pass SSH keys and VDDK path to API

    (cherry picked from commit e65d65ea6dd511b62bae5121ea526f0077f4e368)

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

 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizardAuthenticationStep/ConversionHostWizardAuthenticationStep.js | 4 +-
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizardResultsStep/helpers.js | 12 +-
 2 files changed, 7 insertions(+), 9 deletions(-)


https://github.com/ManageIQ/manageiq-v2v/commit/adae39a1b02cdd8c4148d514eb70f300d22ca3fe
commit adae39a1b02cdd8c4148d514eb70f300d22ca3fe
Author:     Mike Turley <mike.turley.umass.edu>
AuthorDate: Thu Mar 14 11:36:51 2019 -0400
Commit:     Mike Turley <mike.turley.umass.edu>
CommitDate: Thu Mar 14 11:36:51 2019 -0400

    Merge pull request #909 from mzazrivec/fix_conversion_host_removal_modal

    Fix conversion host removal modal

    (cherry picked from commit fda0a672389ad4280ba9b07be511b74736e1a253)

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

 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/DeleteConversionHostConfirmationModal.js | 12 +-
 1 file changed, 8 insertions(+), 4 deletions(-)

Comment 5 CFME Bot 2019-04-05 17:36:07 UTC
New commits detected on ManageIQ/manageiq-v2v/hammer:

https://github.com/ManageIQ/manageiq-v2v/commit/b8f39b73264982abffe87faeaf6ff461bc3453cc
commit b8f39b73264982abffe87faeaf6ff461bc3453cc
Author:     Milan Zázrivec <mzazrivec>
AuthorDate: Wed Mar 20 04:56:56 2019 -0400
Commit:     Milan Zázrivec <mzazrivec>
CommitDate: Wed Mar 20 04:56:56 2019 -0400

    Merge pull request #913 from mturley/conv-host-wizard-ssh-field-name

    Conversion Host Configuration Wizard - rename API field from ssh_key to conversion_host_ssh_private_key

    (cherry picked from commit 3e28cca810f0b6cae0164249bea0a218f93f9d0e)

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

 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizardResultsStep/helpers.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)


https://github.com/ManageIQ/manageiq-v2v/commit/e5b7742b2506071b8b902e3c8320fd6c596c5d6c
commit e5b7742b2506071b8b902e3c8320fd6c596c5d6c
Author:     Milan Zázrivec <mzazrivec>
AuthorDate: Tue Mar 26 12:53:41 2019 -0400
Commit:     Milan Zázrivec <mzazrivec>
CommitDate: Tue Mar 26 12:53:41 2019 -0400

    Merge pull request #916 from mturley/901-conv-host-retry

    [#901] Conversion Host Enablement - List View - Add Retry button with modal for re-entering SSH keys

    (cherry picked from commit 4d7e37cea2aaf83cc5ce8a230c1f556e9b33134f)

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

 app/javascript/common/constants.js | 5 +
 app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/MappingWizardClustersStepConstants.js | 5 -
 app/javascript/react/screens/App/Mappings/screens/MappingWizard/components/MappingWizardClustersStep/MappingWizardClustersStepSelectors.js | 2 +-
 app/javascript/react/screens/App/Settings/SettingsActions.js | 23 +-
 app/javascript/react/screens/App/Settings/SettingsConstants.js | 4 +
 app/javascript/react/screens/App/Settings/SettingsReducer.js | 18 +-
 app/javascript/react/screens/App/Settings/__tests__/__snapshots__/SettingsReducer.test.js.snap | 36 +
 app/javascript/react/screens/App/Settings/helpers.js | 11 +
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/ConversionHostsSettings.js | 22 +-
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostRetryButton.js | 32 +
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizardAuthenticationStep/ConversionHostWizardAuthenticationStep.js | 47 +-
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostsList.js | 21 +-
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostsListItem.js | 37 +-
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/RetryConversionHostConfirmationModal/RetryConversionHostConfirmationModal.js | 117 +
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/RetryConversionHostConfirmationModal/index.js | 34 +
 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/index.js | 14 +-
 app/javascript/react/screens/App/common/forms/SshKeyField.js | 30 +
 17 files changed, 392 insertions(+), 66 deletions(-)


https://github.com/ManageIQ/manageiq-v2v/commit/cdbe65f1155c9c4bead6c302261aeb3987a80d5d
commit cdbe65f1155c9c4bead6c302261aeb3987a80d5d
Author:     Milan Zázrivec <mzazrivec>
AuthorDate: Wed Apr  3 05:17:19 2019 -0400
Commit:     Milan Zázrivec <mzazrivec>
CommitDate: Wed Apr  3 05:17:19 2019 -0400

    Merge pull request #924 from mturley/auth-user-root-for-rhv

    Conversion Host Wizard: set the auth_user to 'root' for RHV hosts

    (cherry picked from commit 28a3302324d630bfd1420203234e651131c65dde)

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

 app/javascript/react/screens/App/Settings/screens/ConversionHostsSettings/components/ConversionHostWizard/ConversionHostWizardResultsStep/helpers.js | 4 +-
 1 file changed, 1 insertion(+), 3 deletions(-)


https://github.com/ManageIQ/manageiq-v2v/commit/11a670309d45854610682787dccf372b1cff3b42
commit 11a670309d45854610682787dccf372b1cff3b42
Author:     Milan Zázrivec <mzazrivec>
AuthorDate: Fri Apr  5 04:19:52 2019 -0400
Commit:     Milan Zázrivec <mzazrivec>
CommitDate: Fri Apr  5 04:19:52 2019 -0400

    Merge pull request #928 from mturley/settings-helper-tests

    Conversion Host Settings - Unit tests for helpers, prevent corner case of reappearing failures

    (cherry picked from commit f6dfc222053d10d30de268c253176aeea32dabbc)

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

 app/javascript/react/screens/App/Settings/__tests__/helpers.test.js | 137 +
 app/javascript/react/screens/App/Settings/helpers.js | 28 +-
 2 files changed, 151 insertions(+), 14 deletions(-)

Comment 6 Shveta 2019-04-10 17:41:25 UTC
Since this PR is to add conversion host page changes but the tab will be hidden until PR https://github.com/ManageIQ/manageiq-v2v/pull/922 gets merged.
Nothing to verify in UI yet.
Verifying based on above PR's.


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