Skip to content

Fix flaky couchbase test#7791

Merged
exekias merged 1 commit intoelastic:masterfrom
ruflin:flaky-coudbase-tests
Jul 30, 2018
Merged

Fix flaky couchbase test#7791
exekias merged 1 commit intoelastic:masterfrom
ruflin:flaky-coudbase-tests

Conversation

@ruflin
Copy link
Copy Markdown
Contributor

@ruflin ruflin commented Jul 30, 2018

If curl is run with -f it also returns 0 in case of a 401 it seems. This change checking for 200 should fix the healthcheck.

Closes #7683

If curl is run with -f it also returns 0 in case of a 401 it seems. This change checking for 200 should fix the healthcheck.

Closes elastic#7683
@ruflin ruflin added module Metricbeat Metricbeat needs_backport PR is waiting to be backported to other branches. :Testing labels Jul 30, 2018
@ruflin
Copy link
Copy Markdown
Contributor Author

ruflin commented Jul 30, 2018

jenkins, test this

@exekias exekias merged commit 93400c0 into elastic:master Jul 30, 2018
@ruflin ruflin deleted the flaky-coudbase-tests branch August 2, 2018 09:05
ruflin added a commit to ruflin/beats that referenced this pull request Aug 27, 2018
If curl is run with -f it also returns 0 in case of a 401 it seems. This change checking for 200 should fix the healthcheck.

Closes elastic#7683

(cherry picked from commit 93400c0)
@ruflin ruflin added v6.5.0 and removed needs_backport PR is waiting to be backported to other branches. labels Aug 27, 2018
ruflin added a commit to ruflin/beats that referenced this pull request Aug 27, 2018
If curl is run with -f it also returns 0 in case of a 401 it seems. This change checking for 200 should fix the healthcheck.

Closes elastic#7683

(cherry picked from commit 93400c0)
@ruflin ruflin added the v6.4.1 label Aug 27, 2018
jsoriano pushed a commit that referenced this pull request Aug 27, 2018
If curl is run with -f it also returns 0 in case of a 401 it seems. This change checking for 200 should fix the healthcheck.

Closes #7683

(cherry picked from commit 93400c0)
jsoriano pushed a commit that referenced this pull request Aug 27, 2018
If curl is run with -f it also returns 0 in case of a 401 it seems. This change checking for 200 should fix the healthcheck.

Closes #7683

(cherry picked from commit 93400c0)
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
If curl is run with -f it also returns 0 in case of a 401 it seems. This change checking for 200 should fix the healthcheck.

Closes elastic#7683

(cherry picked from commit 6f2729d)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Failing metricbeat test: couchbase

2 participants