Skip to content

Cherry-pick #23480 to 7.11: Fixing flaky tests for windows 7 32bit#23559

Merged
narph merged 1 commit intoelastic:7.11from
narph:backport_23480_7.11
Jan 19, 2021
Merged

Cherry-pick #23480 to 7.11: Fixing flaky tests for windows 7 32bit#23559
narph merged 1 commit intoelastic:7.11from
narph:backport_23480_7.11

Conversation

@narph
Copy link
Copy Markdown
Contributor

@narph narph commented Jan 19, 2021 •
edited by zube bot
Loading

Cherry-pick of PR #23480 to 7.11 branch. Original message:

Will update soon

What does this PR do?

After investigation we see 2 main issues:

  1. constant .... overflows int

Because the int value is overflowed, the fix is to specify the int64 type instead.

  1. Tests failing consistently most likely due to expfmt package. This happens in function https://github.com/elastic/beats/blob/master/metricbeat/helper/prometheus/prometheus.go#L72
    When trying to read and parse the data, lines for example:
etcd_disk_wal_fsync_duration_seconds_bucket{le="0.002"} 0	
etcd_disk_wal_fsync_duration_seconds_bucket{le="0.004"} 0

are not considered.

Why is it important?

Should fix/skip the following:
(etcd failing tests were previously fixed)
image

image

Python tests

image

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 added an entry in CHANGELOG.next.asciidoc or CHANGELOG-developer.next.asciidoc.

Related issues

@narph narph added [zube]: In Review backport Team:Integrations Label for the Integrations team labels Jan 19, 2021
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Jan 19, 2021
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Jan 19, 2021

Pinging @elastic/integrations (Team:Integrations)

@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Jan 19, 2021
@narph @v1v
Fixing flaky tests for windows 7 32bit (elastic#23480)
66beb16 
* enabling testing on windows 32 bit

* update test file

* fix

* [CI] Optional stage instead the mandatory one

windows stage is mandatory, but optional stages are not mandatory for PRs

* fix int issue

* ignore 386 architecture

* skip tests

* skip test

* function redeclared

* re write

* add windows constraint

* add tag

* add comments

Co-authored-by: Victor Martinez <VictorMartinezRubio@gmail.com>
(cherry picked from commit 1b43637)
@narph narph self-assigned this Jan 19, 2021
@narph narph added :Windows windows-10 Enable builds in the CI for windows-10 windows-2008 Enable builds in the CI for windows-2008 windows-2012 Enable builds in the CI for windows-2012 windows-2016 Enable builds in the CI for windows-2016 windows-7 Enable builds in the CI for windows-7 windows-7-32 Enable builds in the CI for windows-7 32 bits windows-8 Enable builds in the CI for windows-8 labels Jan 19, 2021
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Jan 19, 2021 •
edited by jenkins-beats-ci bot
Loading

💚 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

Expand to view the summary

Build stats

  • Build Cause: narph commented: /test

    • Start Time: 2021-01-19T10:39:04.856+0000

  • Duration: 44 min 50 sec

  • Commit: 66beb16

Test stats 🧪

Test Results
Failed 0
Passed 10072
Skipped 2466
Total 12538

💚 Flaky test report

Tests succeeded.

Expand to view the summary

Test stats 🧪

Test Results
Failed 0
Passed 10072
Skipped 2466
Total 12538

@narph
Copy link
Copy Markdown
Contributor Author

narph commented Jan 19, 2021

/test

@narph narph merged commit 7009b4a into elastic:7.11 Jan 19, 2021
@narph narph deleted the backport_23480_7.11 branch January 19, 2021 16:04
@zube zube bot removed the [zube]: Done label Apr 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@kaiyan-sheng kaiyan-sheng kaiyan-sheng approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@narph narph

Labels

backport Team:Integrations Label for the Integrations team :Windows windows-7-32 Enable builds in the CI for windows-7 32 bits windows-7 Enable builds in the CI for windows-7 windows-8 Enable builds in the CI for windows-8 windows-10 Enable builds in the CI for windows-10 windows-2008 Enable builds in the CI for windows-2008 windows-2012 Enable builds in the CI for windows-2012 windows-2016 Enable builds in the CI for windows-2016

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@narph @elasticmachine @kaiyan-sheng