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.

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

[Metricbeat][test] Disable ec2 flaky test #20959

Merged
merged 1 commit into from
Sep 3, 2020

Conversation

v1v
Copy link
Member

@v1v v1v commented Sep 3, 2020

What does this PR do?

Disable the test since it's failing in the CI

Related issues

Relates #20951

@v1v v1v added flaky-test Unstable or unreliable test cases. Team:Integrations Label for the Integrations team labels Sep 3, 2020
@v1v v1v requested a review from kaiyan-sheng September 3, 2020 11:27
@elasticmachine
Copy link
Collaborator

Pinging @elastic/integrations (Team:Integrations)

@botelastic botelastic bot added needs_team Indicates that the issue/PR needs a Team:* label and removed needs_team Indicates that the issue/PR needs a Team:* label labels Sep 3, 2020
@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: [Pull request #20959 opened]

  • Start Time: 2020-09-03T11:28:57.668+0000

  • Duration: 49 min 3 sec

Test stats 🧪

Test Results
Failed 0
Passed 836
Skipped 72
Total 908

@jsoriano jsoriano self-requested a review September 3, 2020 13:19
@jsoriano jsoriano removed their assignment Sep 3, 2020
Copy link
Member

@jsoriano jsoriano left a comment

Choose a reason for hiding this comment

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

LGTM, do you want to skip in this PR the S3 test that was also failing?

@v1v
Copy link
Member Author

v1v commented Sep 3, 2020

LGTM, do you want to skip in this PR the S3 test that was also failing?

That particular test failed only once in the 2.0 pipeline but no in the 1.0, so I'd say it might be a temporary issue, so if I can reproduce it again, I'd say to raise another issue and PR with the skip

@v1v v1v merged commit e7f5553 into elastic:master Sep 3, 2020
@v1v v1v deleted the flaky/disable-test-aws branch September 3, 2020 13:59
v1v added a commit to v1v/beats that referenced this pull request Sep 3, 2020
…ne-2.0

* upstream/master:
  [Metricbeat][test] Disable ec2 flaky test (elastic#20959)
  Check if tracer is active before starting a transaction (elastic#20852)
  [Elastic Agent] Add support for variable replacement from providers (elastic#20839)
  Only request wildcard expansion for hidden indices if supported (elastic#20938)
  [Ingest Manager] New agent structure (symlinks) (elastic#20400)
  [Ingest Manager] Print a message confirming shutdown (elastic#20948)
  Skip flaky test on unix input (elastic#20942)
  [Ingest Manager] Align introspect-inspect naming in code (elastic#20952)
  [Filebeat][zeek] Map new x509 fields for ssl module (elastic#20927)
  [CI] fix regression with variable name (elastic#20930)
  [Autodiscover] Handle input-not-finished errors in config reload (elastic#20915)
  [Ingest Manager] Remove Success from fleet contract (elastic#20449)
v1v added a commit to v1v/beats that referenced this pull request Sep 3, 2020
…-faster

* upstream/master:
  [Metricbeat][test] Disable ec2 flaky test (elastic#20959)
  Check if tracer is active before starting a transaction (elastic#20852)
  [Elastic Agent] Add support for variable replacement from providers (elastic#20839)
  Only request wildcard expansion for hidden indices if supported (elastic#20938)
  [Ingest Manager] New agent structure (symlinks) (elastic#20400)
  [Ingest Manager] Print a message confirming shutdown (elastic#20948)
  Skip flaky test on unix input (elastic#20942)
  [Ingest Manager] Align introspect-inspect naming in code (elastic#20952)
  [Filebeat][zeek] Map new x509 fields for ssl module (elastic#20927)
@jsoriano
Copy link
Member

jsoriano commented Oct 1, 2020

@v1v could you please backport this change to 7.x? the test is also failing there. Thanks!

@jsoriano jsoriano added needs_backport PR is waiting to be backported to other branches. v7.10.0 labels Oct 1, 2020
v1v added a commit to v1v/beats that referenced this pull request Oct 1, 2020
v1v added a commit that referenced this pull request Oct 1, 2020
v1v added a commit to v1v/beats that referenced this pull request Oct 8, 2020
v1v added a commit that referenced this pull request Oct 8, 2020
melchiormoulin pushed a commit to melchiormoulin/beats that referenced this pull request Oct 14, 2020
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
flaky-test Unstable or unreliable test cases. needs_backport PR is waiting to be backported to other branches. Team:Integrations Label for the Integrations team v7.10.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants