[Elastic Synthetics Integration] Deprecate Legacy Elastic Synthetics Integration edit flow

[dominiqueclarke]

Issue updated to reflect the latest discussions/decisions.

#150287 raised to add stronger messaging in 8.7 of the upcoming deprecation in 8.8.

This issue is to remove the Integration capabilities of Zip URL and Inline monitors from the Synthetics Integration (internally, the Integration is still needed as it’s used for delivering Private Locations).

ACs:

• Stop monitors configured through the Elastic Synthetics Integration from running (Zip URL and Inline)
  • This will now be done in Heartbeat [Heartbeat] Don’t run legacy Synthetics Integration monitors beats#34506
• Remove the ability to add, view, edit or delete them
• Update the Overview for the Synthetics Integration to describe it as for being used internally by Private Locations (i.e. there is no need for users to add a Synthetics Integration as it’s done behind the scenes when working with Private Locations)
  • Direct users to adding monitors through Project Monitors, or via the Synthetics UI, referring to the main Synthetics documentation
  • Being done in [Synthetics] Update description of the Synthetics Integration overview integrations#5258

Original description

Prerequisites:

• no longer doing this => [Synthetics] Migrate legacy Elastic Synthetics Integration integration policies to Synthetics Monitor Management #137494
• [Elastic Synthetics Integration] Deprecate Legacy Synthetics Integration Add Monitor Flow #137501

Disable the ability to edit Elastic Synthetics integrations outside Uptime/Synthetics monitor management, encouring users to migrate to UI Monitor Management.

This can be done either through a button directing users to Uptime or Synthetics for the migration or by immediately redirecting users to Uptime or Synthetics for the migration.

This assumes that a migration path is available for moving legacy Synthetics integration policies into UI Monitor Management via #137494

Edit monitor flow for legacy integrations

Disables the ability to edit the monitor, displaying a prompt to migrate the monitor to Uptime/Synthetics monitor management in order to edit

Edit monitor flow for migrated integrations

Once the monitor is fully migrated, the edit flow will follow the UX described in elastic/uptime#475

[elasticmachine]

Pinging @elastic/uptime (Team:uptime)

[paulb-elastic]

Further to a team discussion the other day, here's a clarification of the current behaviour:

8.4 (Tested 8.4.3)

• Can add Synthetics Integrations for all types of monitor
  • Lightweight
  • Browser
    • ZIP URL
    • Inline

8.5 (Tested 8.5.3)

• UI allows Synthetics Integrations to be added as with 8.4
• However does call out that ZIP URLs are being deprecated
  
• ⚠️ ZIP URLs seem to be broken, in that when they are created/saved, the URL doesn't actually save
  • This also affects upgrades - see below
• Upgrading from 8.4.3 to 8.5.3
  • All Synthetics Integration policies are still there
    
  • ⚠️ As with ZIP URLs created in 8.5.3, those created in 8.4.3 and migrated to 8.5.3 no longer have a URL defined
    
  • All other types (Browser Inline and Lightweight) still have the Script/URL/host set

8.6 (Tested 8.6.1)

• Synthetics Integration not shown unless you toggle “Display beta integrations” to on
• Even if set, cannot add new Synthetics Integration (monitors) but get a banner to use Uptime Monitor Management instead
  
• Upgrading from 8.5.3 to 8.6.1
  • Existing Synthetics Integrations are still there (ZIP URLs still missing URL as in 8.5.3)

8.7.0-SNAPSHOT (Tested build as ok 01/Feb/2023)

• Same as 8.6.1
  • Can't add new Synthetics Integrations
  • Existing Synthetics Integrations are still there (ZIP URLs still missing URL)

Summary

• ZIP URLs are not likely to be valid/working from 8.5 onwards
• Users haven't been able to add new Synthetics Integration monitors from 8.6
• Suggestion:
  • In 8.7
    • If there are any Synthetics Integrations defined, add a notification banner in Uptime (see below)
    • In the edit flow of the existing Integration Policy, add a notification (see below)
  • In 8.8
    • Stop any Synthetics Integration monitors from running
    • Update the Synthetics Integration UI to clarify what it is (internal, not for use)
    • Remove the notification banner added in 8.7
  • The notification/banner should clarify:
    • Synthetics Integration monitors will stop running in 8.8 and their configuration will be inaccessible from 8.8 onwards
    • Migrate to monitors configured through the UI (ideally Synthetics UI rather than Uptime Monitor Management) or Project Monitors
    • Include details in the release notes

[paulb-elastic]

Description updated based on recent discussions

[paulb-elastic]

Updated the description that the stopping of the monitors will be done in Beats elastic/beats#34506 (legacy integration policies won't be deleted).

[dominiqueclarke]

Post FF testing LGTM on a snapshot from May 10th