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

[Security Solution] [Detections] Replace 'partial failure' with 'warning' for rule statuses #91167

Merged
merged 5 commits into from
Feb 17, 2021
Merged

[Security Solution] [Detections] Replace 'partial failure' with 'warning' for rule statuses #91167

merged 5 commits into from
Feb 17, 2021

Commits on Feb 16, 2021

  1. removes usage of 'partial failure' status and replaces with a 'warnin… 

    …g' status, also adds some logic to be backwards compatible with 'partial failure' statuses
    @dhurley14
    dhurley14 committed Feb 16, 2021
    Configuration menu
    Copy the full SHA
    2f6bd37 View commit details
    Browse the repository at this point in the history

  2. update integration tests from 'partial failure' to 'warning'

    @dhurley14
    dhurley14 committed Feb 16, 2021
    Configuration menu
    Copy the full SHA
    0832d9c View commit details
    Browse the repository at this point in the history

  3. fix integration test to warn and not error when no index patterns mat… 

    …ch concrete indices
    @dhurley14
    dhurley14 committed Feb 16, 2021
    Configuration menu
    Copy the full SHA
    e0ae733 View commit details
    Browse the repository at this point in the history

  4. fix integration test

    @dhurley14
    dhurley14 committed Feb 16, 2021
    Configuration menu
    Copy the full SHA
    bfade53 View commit details
    Browse the repository at this point in the history

  5. removes outdated comments from the create_rules e2e test

    @dhurley14
    dhurley14 committed Feb 16, 2021
    Configuration menu
    Copy the full SHA
    6d3cb93 View commit details
    Browse the repository at this point in the history