Skip to content

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

Merged
v1v merged 1 commit intoelastic:masterfrom
v1v:flaky/disable-test-aws
Sep 3, 2020
Merged

[Metricbeat][test] Disable ec2 flaky test#20959
v1v merged 1 commit intoelastic:masterfrom
v1v:flaky/disable-test-aws

Conversation

@v1v
Copy link
Copy Markdown
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
Copy Markdown
Contributor

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
Copy Markdown
Contributor

💚 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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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