Bug 2050710 - [Doc] Remove "SHA-1 fingerprint of a vCenter host" prerequisite starting MTV-2.3
Summary: [Doc] Remove "SHA-1 fingerprint of a vCenter host" prerequisite starting MTV-2.3
Keywords:
Status: CLOSED NEXTRELEASE
Alias: None
Product: Migration Toolkit for Virtualization
Classification: Red Hat
Component: Documentation
Version: 2.2.0
Hardware: Unspecified
OS: Unspecified
medium
medium
Target Milestone: ---
: 2.3.0
Assignee: Sasha Beskin
QA Contact: Ilanit Stein
Steve Goodman
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2022-02-04 13:52 UTC by Ilanit Stein
Modified: 2022-03-16 13:36 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2022-03-16 13:36:47 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)

Description Ilanit Stein 2022-02-04 13:52:31 UTC
Description of problem:

In MTV-2.3 web console, in Add provider window this fingerprint is fetched automatically in UI. 
In MTV-2.2 users were expected to fetch it.

According to this change Please update MTV-2.3 doc as follows:

1) Remove:
Section: “2.6.2. Obtaining the SHA-1 fingerprint of a vCenter host”
This command is mentioned:

$ openssl s_client \
    -connect <www.example.com>:443 \ 
    < /dev/null 2>/dev/null \
    | openssl x509 -fingerprint -noout -in /dev/stdin \
    | cut -d '=' -f 2

2) In section: "4.1.1. Adding a VMware source provider"
the "vCenter SHA-1 fingerprint." Prerequisite can be removed.


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

Additional info:
This change is for MTV-2.3+ Document.

Comment 2 Sasha Beskin 2022-02-23 13:27:31 UTC
@istein
Should the whole section on obtaining the SHA-1 fingerprint be removed?

Comment 5 Ilanit Stein 2022-03-01 05:14:51 UTC
Sasha,

I think we need to document what's in the new "Add provider" in MTV-2.3.
Instead of asking for  Fingerprint, the user is asked to verify the certificate.
Please see the last 2 "Add provider" screenshots for reference.

Comment 6 Sasha Beskin 2022-03-01 15:25:46 UTC
Ilanit,

Should I make these changes within the scope of this bug? 
And my initial question still stands - looks like the whole section on obtaining the SHA-1 fingerprint is not needed anymore. Is it correct?

Comment 7 Ilanit Stein 2022-03-07 16:45:41 UTC
In the scope of this bug these changes are required:


"4.1.1. Adding a VMware source provider" section needs to change as follows:
- Remove prerequisite:
vCenter SHA-1 fingerprint.

- In the procedure, need to change:
from:
SHA-1 fingerprint: vCenter SHA-1 fingerprint
To:
click the "I trust the authenticity of this certificate"
(Check attached "Add VMware provider second screenshot " to see how it looks like in MTV-2.3). 


In section "5.1. Migrating virtual machines"
Procedure, Step 2 about creating the secret:
<vcenter_fingerprint> is required.
Therefore we still need to keep in MTV-2.3 document this section: 
"2.6.2. Obtaining the SHA-1 fingerprint of a vCenter host"

Comment 8 Sasha Beskin 2022-03-08 16:00:23 UTC
Hi Ilanit,

You wrote:

- In the procedure, need to change:
from:
SHA-1 fingerprint: vCenter SHA-1 fingerprint
To:
click the "I trust the authenticity of this certificate"

Do I understand correctly that first the user should click 'Verify certificate'?

Another question:

In section Migrating virtual machines, should obtaining the SHA-1 fingerprint of a vCenter host be added as a VMware-only prerequisite?

Comment 9 Sasha Beskin 2022-03-09 11:14:55 UTC
PR:
https://github.com/konveyor/forklift-documentation/pull/287

Ready for QE review

Comment 10 Ilanit Stein 2022-03-09 12:18:46 UTC
In reply to comment #8:

Do I understand correctly that first the user should click 'Verify certificate'?
>Yes

In section Migrating virtual machines, should obtaining the SHA-1 fingerprint of a vCenter host be added as a VMware-only prerequisite?
This prerequisite is VMware only, and also only if the VMware provider is added via CLI.

Comment 11 Ilanit Stein 2022-03-10 14:43:19 UTC
Moving to Verified as the PR mentioned in comment #9 looks good.


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