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.

Sign up for GitHub

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

Jump to bottom

[CI] watcher/usage/10_basic/Test watcher usage stats output fails #33326

Closed
dnhatn opened this issue Aug 31, 2018 · 3 comments · Fixed by #34107
Closed

[CI] watcher/usage/10_basic/Test watcher usage stats output fails #33326

dnhatn opened this issue Aug 31, 2018 · 3 comments · Fixed by #34107
Labels
:Data Management/Watcher >test-failure Triaged test failures from CI

Comments

@dnhatn
Copy link
Member

dnhatn commented Aug 31, 2018

watcher/usage/10_basic/Test watcher usage stats output fails with an unexpected watch_count_active.

> Caused by: java.lang.AssertionError: field [watcher.count.active] is not greater than [$watch_count_active]
> Expected: a value greater than <0>
>      but: <0> was equal to <0>
>    at org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:20)

I can reproduce this locally with this command:

./gradlew :x-pack:qa:smoke-test-watcher-with-security:integTestRunner \
  -Dtests.seed=E3C29FF2DA312869 \
  -Dtests.class=org.elasticsearch.smoketest.SmokeTestWatcherWithSecurityClientYamlTestSuiteIT \
  -Dtests.method="test {yaml=watcher/usage/10_basic/Test watcher usage stats output}" \
  -Dtests.security.manager=true \
  -Dtests.locale=ru \
  -Dtests.timezone=America/Belize \
  -Dcompiler.java=10 \
  -Druntime.java=8

CI instances:
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@dnhatn dnhatn added >test-failure Triaged test failures from CI :Data Management/Watcher labels Aug 31, 2018
dnhatn added a commit that referenced this issue Aug 31, 2018
@dnhatn
Mute test watcher usage stats output
b9c45ff 
Tracked at #33326
dnhatn added a commit that referenced this issue Aug 31, 2018
@dnhatn
Mute test watcher usage stats output
ca94d05 
Tracked at #33326
spinscale added a commit to spinscale/elasticsearch that referenced this issue Sep 27, 2018
@spinscale
Watcher: Reenable watcher stats REST tests
4986417 
Due to a wrong fix, that was fixed in elastic#33360 and commit
1de2a92 the initial adding of a watch
could get lost.

Closes elastic#33326
@spinscale spinscale mentioned this issue Sep 27, 2018
Merged
spinscale added a commit that referenced this issue Sep 28, 2018
@spinscale
Watcher: Reenable watcher stats REST tests (#34107)
d846f58 
Due to a bug, that was fixed in #33360 and commit
1de2a92 the initial adding of a watch
could get lost, thus leaving the watcher stats count as zero despite adding a watch.

Closes #33326
spinscale added a commit that referenced this issue Sep 28, 2018
@spinscale
Watcher: Reenable watcher stats REST tests (#34107)
8d99be8 
Due to a bug, that was fixed in #33360 and commit
1de2a92 the initial adding of a watch
could get lost, thus leaving the watcher stats count as zero despite adding a watch.

Closes #33326
kcm pushed a commit that referenced this issue Oct 30, 2018
@spinscale @kcm
Watcher: Reenable watcher stats REST tests (#34107)
777e5c7 
Due to a bug, that was fixed in #33360 and commit
1de2a92 the initial adding of a watch
could get lost, thus leaving the watcher stats count as zero despite adding a watch.

Closes #33326
@dakrone
Copy link
Member

dakrone commented Feb 27, 2020

This failed again today with:

09:55:26 
09:55:26 org.elasticsearch.smoketest.SmokeTestWatcherWithSecurityClientYamlTestSuiteIT > test {yaml=watcher/usage/10_basic/Test watcher usage stats output} FAILED
09:55:26     java.lang.AssertionError: Failure at [watcher/usage/10_basic:48]: field [watcher.count.active] is not greater than [$watch_count_active]
09:55:26     Expected: a value greater than <1>
09:55:26          but: <1> was equal to <1>
09:55:26         at __randomizedtesting.SeedInfo.seed([3AA5E1A0040CF825:B2F1DE7AAAF095DD]:0)
09:55:26         at org.elasticsearch.test.rest.yaml.ESClientYamlSuiteTestCase.executeSection(ESClientYamlSuiteTestCase.java:405)
09:55:26         at org.elasticsearch.test.rest.yaml.ESClientYamlSuiteTestCase.test(ESClientYamlSuiteTestCase.java:382)
09:55:26         at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
09:55:26         at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)

I was unable to reproduce this on the 7.x branch.

Failure:
https://elasticsearch-ci.elastic.co/job/elastic+elasticsearch+7.x+matrix-java-periodic/ES_RUNTIME_JAVA=zulu11,nodes=general-purpose/538/consoleFull
https://gradle-enterprise.elastic.co/s/xxhruxec3jeji

@dakrone dakrone reopened this Feb 27, 2020
@martijnvg
Copy link
Member

Thanks @dakrone for triaging this. This issue is similar to #52453. Several tests have issues with producing the correct watch count via watcher stats api api. The usage api, uses the watcher stats api under the hood. I will close this issue and add this comment to #52453.

@martijnvg martijnvg mentioned this issue Feb 28, 2020
Closed
@JenniferPruitt JenniferPruitt mentioned this issue Aug 25, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
:Data Management/Watcher >test-failure Triaged test failures from CI
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Watcher: Reenable watcher stats REST tests spinscale/elasticsearch
4 participants
@dakrone @martijnvg @dnhatn @elasticmachine