Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Elastic Synthetics Integration] Deprecate Legacy Synthetics Integration Add Monitor Flow #137501

Closed
Tracked by #137505
, #137508
...
dominiqueclarke opened this issue Jul 28, 2022 · 7 comments · Fixed by #146369
Closed
Tracked by #137505
, #137508
...

[Elastic Synthetics Integration] Deprecate Legacy Synthetics Integration Add Monitor Flow #137501

dominiqueclarke opened this issue Jul 28, 2022 · 7 comments · Fixed by #146369
Labels
bug Fixes for quality problems that affect the customer experience enhancement New value added to drive a business result Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability

Comments

@dominiqueclarke
Copy link
Contributor

Disable the ability to Add new Elastic Synthetics integrations outside Uptime/Synthetics monitor management, by redirecting users to Uptime to create monitors and private locations.

This can be done either through a button directing users to Uptime or Synthetics, or by immediately redirecting users to Uptime.
@dominiqueclarke dominiqueclarke added enhancement New value added to drive a business result Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability labels Jul 28, 2022
@elasticmachine
Copy link
Contributor

Pinging @elastic/uptime (Team:uptime)

@dominiqueclarke
Copy link
Contributor Author

It might be a bit confusing for existing users of Elastic Synthetics Integration if they were immediately redirected to Uptime or Synthetics, unless we added some type of callout to the top of the page when redirected from the Integration explaining the context.

This was referenced Jul 28, 2022
Closed
Closed
@paulb-elastic
Copy link
Contributor

Agree @dominiqueclarke we should add some explanation for why we've directed them over.

Also, we should carefully consider where we direct them to. Ideally there'd be enough of the Synthetics UI flow available to direct them there (I’m not particularly keen on directing them to Uptime, which is not the desired end flow for this).

@paulb-elastic
Copy link
Contributor

As mentioned here it would be better to direct them to the proper flow in Synthetics UI than the temporary Uptime flow (which will then need to be changed again).

@dominiqueclarke
Copy link
Contributor Author

@paulb-elastic If we redirect to Synthetics UI, it'll be best to wait till 8.6 or GA, whenever we enable the app by default.

The reason is that users who do not have the advanced options switch for synthetics on will get an app not found error.

@paulb-elastic
Copy link
Contributor

Thanks @dominiqueclarke, good point

Keen to hear from @hbharding and @drewpost too, we'll just need to make sure to track these for changing flows when Synthetics UI is available for everyone.

@paulb-elastic paulb-elastic mentioned this issue Oct 11, 2022
Closed
@paulb-elastic paulb-elastic added the bug Fixes for quality problems that affect the customer experience label Nov 16, 2022
@shahzad31 shahzad31 self-assigned this Nov 28, 2022
@shahzad31 shahzad31 mentioned this issue Nov 28, 2022
Merged
shahzad31 added a commit that referenced this issue Nov 29, 2022
@shahzad31
[Synthetics] Deprecate adding synthetics integrations from fleet (#14…
d136668 
…6369)

Co-authored-by: florent-leborgne <[email protected]>
Fixes #137501
kibanamachine pushed a commit to kibanamachine/kibana that referenced this issue Nov 29, 2022
@shahzad31
[Synthetics] Deprecate adding synthetics integrations from fleet (ela…
0675d81 
…stic#146369)

Co-authored-by: florent-leborgne <[email protected]>
Fixes elastic#137501

(cherry picked from commit d136668)
@shahzad31 shahzad31 removed their assignment Nov 29, 2022
kibanamachine referenced this issue Nov 30, 2022
@kibanamachine @shahzad31
[8.6] [Synthetics] Deprecate adding synthetics integrations from fleet (
c9fe5ac 
#146369) (#146629)

# Backport

This will backport the following commits from `main` to `8.6`:
- [[Synthetics] Deprecate adding synthetics integrations from fleet
(#146369)](#146369)

<!--- Backport version: 8.9.7 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT
[{"author":{"name":"Shahzad","email":"[email protected]"},"sourceCommit":{"committedDate":"2022-11-29T20:29:26Z","message":"[Synthetics]
Deprecate adding synthetics integrations from fleet
(#146369)\n\nCo-authored-by: florent-leborgne
<[email protected]>\r\nFixes
https://github.com/elastic/kibana/issues/137501","sha":"d13666885469c354bdb55c7bb84f127d23b1b83f","branchLabelMapping":{"^v8.7.0$":"main","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["Team:uptime","release_note:skip","Team:Fleet","v8.6.0","v8.7.0"],"number":146369,"url":"https://github.com/elastic/kibana/pull/146369","mergeCommit":{"message":"[Synthetics]
Deprecate adding synthetics integrations from fleet
(#146369)\n\nCo-authored-by: florent-leborgne
<[email protected]>\r\nFixes
https://github.com/elastic/kibana/issues/137501","sha":"d13666885469c354bdb55c7bb84f127d23b1b83f"}},"sourceBranch":"main","suggestedTargetBranches":["8.6"],"targetPullRequestStates":[{"branch":"8.6","label":"v8.6.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"main","label":"v8.7.0","labelRegex":"^v8.7.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/146369","number":146369,"mergeCommit":{"message":"[Synthetics]
Deprecate adding synthetics integrations from fleet
(#146369)\n\nCo-authored-by: florent-leborgne
<[email protected]>\r\nFixes
https://github.com/elastic/kibana/issues/137501","sha":"d13666885469c354bdb55c7bb84f127d23b1b83f"}}]}]
BACKPORT-->

Co-authored-by: Shahzad <[email protected]>
@shahzad31
Copy link
Contributor

POST FF Testing, tested on cloud, and it works as expected !!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Fixes for quality problems that affect the customer experience enhancement New value added to drive a business result Team:Uptime - DEPRECATED Synthetics & RUM sub-team of Application Observability
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Synthetics] Deprecate adding synthetics integrations from fleet shahzad31/kibana
4 participants
@shahzad31 @dominiqueclarke @elasticmachine @paulb-elastic