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

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

Merged
merged 1 commit into from
Sep 21, 2020

Conversation

ChrsMark
Copy link
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
Collaborator

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
Collaborator

💚 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