Skip to content
This repository has been archived by the owner on Sep 17, 2024. It is now read-only.

fix: remove preconfigured policies from 7.15 #1774

Merged
merged 1 commit into from
Nov 4, 2021

Conversation

mdelapenya
Copy link
Contributor

What does this PR do?

As per @nchaulet's comment in #1680 (comment), preconfigured policies should not be present in 7.15

This PR is removing the examples we created as placeholders for that feature in master, that were backported as part of #1644

Why is it important?

Those scenarios should not be run on CI, which is causing noise in the nightly scheduled build for 7.15

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works
  • I have run the Unit tests (make unit-test), and they are passing locally
  • I have run the End-2-End tests for the suite I'm working on, and they are passing locally
  • I have noticed new Go dependencies (run make notice in the proper directory)

How to test this PR locally

Running this command should return 0 scenarios:

TAGS="@policies" TIMEOUT_FACTOR=3 LOG_LEVEL=TRACE DEVELOPER_MODE=true ELASTIC_APM_ACTIVE=false PROVIDER=elastic-package  make -C e2e/_suites/fleet functional-test

Related issues

@mdelapenya mdelapenya added Team:Automation Label for the Observability productivity team backport-skip Skip notification from the automated backport with mergify labels Nov 4, 2021
@mdelapenya mdelapenya self-assigned this Nov 4, 2021
@mdelapenya mdelapenya requested a review from a team November 4, 2021 06:49
@elasticmachine
Copy link
Contributor

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2021-11-04T06:50:02.354+0000

  • Duration: 44 min 37 sec

  • Commit: 54b0353

Test stats 🧪

Test Results
Failed 0
Passed 237
Skipped 0
Total 237

💚 Flaky test report

Tests succeeded.

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

@mdelapenya mdelapenya merged commit 6494df7 into elastic:7.15 Nov 4, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
backport-skip Skip notification from the automated backport with mergify Team:Automation Label for the Observability productivity team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants