Skip to content

Cherry-pick #21173 to 7.x: Fix remote_write flaky test#21193

Merged
ChrsMark merged 1 commit intoelastic:7.xfrom
ChrsMark:backport_21173_7.x
Sep 21, 2020
Merged

Cherry-pick #21173 to 7.x: Fix remote_write flaky test#21193
ChrsMark merged 1 commit intoelastic:7.xfrom
ChrsMark:backport_21173_7.x

Conversation

@ChrsMark
Copy link
Copy Markdown
Member

@ChrsMark ChrsMark commented Sep 21, 2020

Cherry-pick of PR #21173 to 7.x branch. Original message:

What does this PR do?

This PR fixes the flaky remote_write system test. The issue should be due to docker networking since the http://host.docker.internal:9201/write endpoint seems to not be working properly and we can stick with http://0.0.0.0:9201/write only...

I tested locally (on macOS) with MODULE=prometheus mage -v pythonIntegTest and seems to work now. Let's see if CI is happy too.

Why is it important?

To enable back the system test for the remote_write metricset.

Related issues

@ChrsMark ChrsMark added [zube]: In Review backport Team:Platforms Label for the Integrations - Platforms team labels Sep 21, 2020
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/integrations-platforms (Team:Platforms)

@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 21, 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 #21193 opened]

  • Start Time: 2020-09-21T08:32:27.100+0000

  • Duration: 77 min 10 sec

Test stats 🧪

Test Results
Failed 0
Passed 3176
Skipped 687
Total 3863

@ChrsMark ChrsMark merged commit e2102e3 into elastic:7.x Sep 21, 2020
@zube zube bot removed the [zube]: Done label Dec 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport Team:Platforms Label for the Integrations - Platforms team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants