Skip to content

Fix path after renaming alerts to detections#71202

Merged
patrykkopycinski merged 1 commit intoelastic:masterfrom
patrykkopycinski:fix/security-detections-rename-fix
Jul 9, 2020
Merged

Fix path after renaming alerts to detections#71202
patrykkopycinski merged 1 commit intoelastic:masterfrom
patrykkopycinski:fix/security-detections-rename-fix

Conversation

@patrykkopycinski
Copy link
Copy Markdown
Contributor

@patrykkopycinski patrykkopycinski commented Jul 9, 2020 •
edited
Loading

Summary

Fixes path after renaming alerts to detections

@patrykkopycinski patrykkopycinski added bug Fixes for quality problems that affect the customer experience v8.0.0 release_note:skip Skip the PR/issue when compiling release notes v7.9.0 labels Jul 9, 2020
@patrykkopycinski patrykkopycinski requested review from a team as code owners July 9, 2020 06:45
@patrykkopycinski patrykkopycinski self-assigned this Jul 9, 2020
angorayc
angorayc approved these changes Jul 9, 2020
View reviewed changes
Copy link
Copy Markdown
Contributor

@angorayc angorayc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍 👍 👍

@kibanamachine
Copy link
Copy Markdown
Contributor

kibanamachine commented Jul 9, 2020

💚 Build Succeeded

Build metrics

@kbn/optimizer bundle module count

id value diff baseline
alerts 22 +22 -
apm 651 +651 -
beats_management 175 +175 -
canvas 799 +799 -
cloud 4 +4 -
crossClusterReplication 60 +60 -
dashboardEnhanced 6 +6 -
dashboardMode 5 +5 -
dataEnhanced 5 +5 -
discoverEnhanced 2 +2 -
embeddableEnhanced 3 +3 -
features 6 +6 -
fileUpload 14 +14 -
globalSearch 16 +16 -
globalSearchProviders 3 +3 -
graph 96 +96 -
grokdebugger 11 +11 -
indexLifecycleManagement 92 +92 -
indexManagement 172 +172 -
infra 606 +606 -
ingestManager 358 +358 -
ingestPipelines 258 +258 -
lens 124 +124 -
licenseManagement 95 +95 -
licensing 8 +8 -
logstash 149 +149 -
maps 458 +458 -
mapsLegacyLicensing 2 +2 -
ml 386 +386 -
monitoring 202 +202 -
observability 33 +33 -
painlessLab 18 +18 -
remoteClusters 66 +66 -
reporting 19 +19 -
rollup 69 +69 -
searchprofiler 29 +29 -
security 264 +264 -
securitySolution 812 +812 -
snapshotRestore 33 +33 -
spaces 118 +118 -
transform 158 +158 -
triggers_actions_ui 151 +151 -
uiActionsEnhanced 28 +28 -
upgradeAssistant 26 +26 -
uptime 250 +250 -
watcher 49 +49 -
total - +6911 -

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

@patrykkopycinski patrykkopycinski merged commit 822e6cf into elastic:master Jul 9, 2020
@patrykkopycinski patrykkopycinski deleted the fix/security-detections-rename-fix branch July 9, 2020 08:37
@tylersmalley tylersmalley mentioned this pull request Jul 9, 2020
Merged
@kibanamachine
Copy link
Copy Markdown
Contributor

kibanamachine commented Jul 13, 2020

Friendly reminder: Looks like this PR hasn’t been backported yet.
To create backports run node scripts/backport --pr 71202 or prevent reminders by adding the backport:skip label.

@kibanamachine kibanamachine added the backport missing Added to PRs automatically when the are determined to be missing a backport. label Jul 13, 2020
@patrykkopycinski patrykkopycinski added backport:skip This PR does not require backporting and removed backport missing Added to PRs automatically when the are determined to be missing a backport. labels Jul 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@angorayc angorayc angorayc approved these changes

Assignees

@patrykkopycinski patrykkopycinski

Labels

backport:skip This PR does not require backporting bug Fixes for quality problems that affect the customer experience release_note:skip Skip the PR/issue when compiling release notes v7.9.0 v8.0.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@patrykkopycinski @kibanamachine @angorayc