-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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.
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
[APM] Use ML capabilities canAccessML
to enable anomaly detection settings
#73998
Comments
Pinging @elastic/apm-ui (Team:apm) |
ogupte
added a commit
to ogupte/kibana
that referenced
this issue
Jul 31, 2020
…I to enable anomaly detection settings in APM.
ogupte
added a commit
that referenced
this issue
Jul 31, 2020
…73999) enable anomaly detection settings in APM.
kibanamachine
pushed a commit
that referenced
this issue
Jul 31, 2020
…73999) enable anomaly detection settings in APM.
kibanamachine
pushed a commit
that referenced
this issue
Jul 31, 2020
…73999) enable anomaly detection settings in APM.
gmmorris
added a commit
to gmmorris/kibana
that referenced
this issue
Aug 2, 2020
* master: (39 commits) [Canvas][tech-debt] Rename __examples__ to __stories__ (elastic#73853) [Canvas] Storybook Redux Addon (elastic#73227) Use "Apply_filter_trigger" in "explore underlying data" action (elastic#71445) [maps] convert top nav config to TS (elastic#73851) [maps] fix fit to bounds for ES document layers with joins (elastic#73985) [Canvas][tech-debt] Refactor Toolbar (completes Kill Recompose.pure) (elastic#73309) [CI] In-progress Slack notifications (elastic#74012) [SIEM][Detection Engine] Fixes tags to accept characters such as AND, OR, (, ), ", * (elastic#74003) [SECURITY_SOLUTION][ENDPOINT] Fix host list Configuration Status cell link loosing list page/size state (elastic#73989) Tweak injected metadata (elastic#73990) Closes elastic#73998 by using `canAccessML` in the ML capabilities API to (elastic#73999) [SIEM] Fixes toaster errors when siemDefault index is an empty or empty spaces (elastic#73991) [Security Solution] Fix timeline pin event callback (elastic#73981) [Security Solution] Fix unexpected redirect (elastic#73969) [Metrics UI] Fix Metrics Explorer TSVB link to use workaround pattern (elastic#73986) [APM] docs: Update machine learning integration (elastic#73597) [Ingest Manager] Fix limited concurrency helper (elastic#73976) [build/sysv] fix missing env variable rename (elastic#73977) Fix a typo. (elastic#73948) [Ingest Manager] Revert fleet config concurrency rollout to rate limit (elastic#73940) ...
ogupte
added a commit
that referenced
this issue
Aug 3, 2020
…73999) (#74022) enable anomaly detection settings in APM. Co-authored-by: Oliver Gupte <[email protected]> Co-authored-by: Elastic Machine <[email protected]>
ogupte
added a commit
that referenced
this issue
Aug 3, 2020
…73999) (#74021) enable anomaly detection settings in APM. Co-authored-by: Oliver Gupte <[email protected]> Co-authored-by: Elastic Machine <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
As per the feedback (https://github.com/elastic/kibana/pull/73638/files#r463479399), we should be checking the value of
canAccessML
instead of only checking for the existance of the ML plugin. This will ensure that the user isn't able to use Anomaly detection settings in cases where they don't have ML enabled in the cluster or otherwise don't have access to ML in kibana.The text was updated successfully, but these errors were encountered: