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

Fix remote_write flaky test #21173

Merged
merged 2 commits into from
Sep 21, 2020
Merged

Conversation

ChrsMark
Copy link
Member

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

Signed-off-by: chrismark <chrismarkou92@gmail.com>
@ChrsMark ChrsMark added bug Team:Platforms Label for the Integrations - Platforms team labels Sep 18, 2020
@ChrsMark ChrsMark self-assigned this Sep 18, 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 18, 2020
Signed-off-by: chrismark <chrismarkou92@gmail.com>
@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: [Pull request #21173 opened]

  • Start Time: 2020-09-18T13:51:17.377+0000

  • Duration: 72 min 49 sec

Test stats 🧪

Test Results
Failed 0
Passed 3177
Skipped 689
Total 3866

Copy link
Contributor

@kaiyan-sheng kaiyan-sheng left a comment

Choose a reason for hiding this comment

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

CI seems to be happy.

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.

Thanks for fixing the test!

@ChrsMark ChrsMark merged commit e770f32 into elastic:master Sep 21, 2020
ChrsMark added a commit to ChrsMark/beats that referenced this pull request Sep 21, 2020
ChrsMark added a commit that referenced this pull request Sep 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Team:Platforms Label for the Integrations - Platforms team v7.10.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Flaky prometheus remote write system test
4 participants