Form conversion: Compute/Infrastructure/Providers Discover items

[himdel]

Convert form to use DDF/React and API - parent issue #6825

Location: Compute/Infrastructure/Providers
Form: Discover items

current type: Angular
API state:
Need info

PM Input: 2 - Medium

imported status: NOT ASSIGNED

Missing components:

• ip address ranges

[skateman]

I am not entirely sure if this form is useful at all, it can only discover providers in a /24 or smaller IPv4 subnet:

@Fryguy @agrare is this something we could get rid of? Or should it be more generic and with IPv6 support?
@miq-bot add_label question

[chessbyte]

I vote to remove this feature.

[agrare]

I don't know how often this is used...but we could get rid of a bunch of port-scanning discovery code if we dropped this 👍

[chessbyte]

this is literally scanning the network for virtual infrastructure (vSphere, RHV, OpenStack, and SCVMM) endpoints. Do we really have customers that do not know where these are in their datacenter??

[Fryguy]

We did in the past...port scanning a network was a great way to find rogue vspheres. The backend code can scan any subnet range so that /24 limitation is UI-only...that is, we could open it up if we wanted and I don't know why the UI is limited in that way.

I kind of like the feature and don't think we should drop it, personally, but perhaps maybe it should be changed in some way? That is, instead of creating EMSes, perhaps it's just a report that returns the IP addresses it discovered. Maybe it just becomes a low-level tool that a power-user could use without the UI?

[skateman]

I'd be happy to ✂️ delete this from the UI

[chessbyte]

from a UX perspective, if the admin does not know about the vSpheres, they likely won't have the credentials - so adding the EMS sounds like overkill. For that reason, I do like the report idea - but not sure on level of effort for that.

[skateman]

So what's the decision here? To discover or not to discover from the UI?

[Fryguy]

Would like @agrare's opinion as well.