Skip to content

Conversation

@spinscale
Copy link
Contributor

Due to a bug, that was fixed in #33360 and commit
1de2a92 the initial adding of a watch
could get lost.

Closes #33326

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 added >test Issues or PRs that are addressing/adding tests v7.0.0 :Data Management/Watcher v6.5.0 labels Sep 27, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@spinscale spinscale merged commit d846f58 into elastic:master Sep 28, 2018
spinscale added a commit that referenced this pull request Sep 28, 2018
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
jasontedor added a commit to jasontedor/elasticsearch that referenced this pull request Sep 28, 2018
* master:
  Use more precise does S3 bucket exist method (elastic#34123)
  LLREST: Introduce a strict mode (elastic#33708)
  [CCR] Adjust list retryable errors (elastic#33985)
  Fix AggregationFactories.Builder equality and hash regarding order (elastic#34005)
  MINOR: Remove some deadcode in NodeEnv and Related (elastic#34133)
  Rest-Api-Spec: Correct spelling in filter_path description (elastic#33154)
  Core: Don't rely on java time for epoch seconds formatting (elastic#34086)
  Retry errors when fetching follower global checkpoint. (elastic#34019)
  Watcher: Reenable watcher stats REST tests (elastic#34107)
  Remove special-casing of Synonym filters in AnalysisRegistry (elastic#34034)
  Rename CCR APIs (elastic#34027)
  Fixed CCR stats api serialization issues and (elastic#33983)
  Support 'string'-style queries on metadata fields when reasonable. (elastic#34089)
  Logging: Drop Settings from security logger get calls (elastic#33940)
  SQL: Internal refactoring of operators as functions (elastic#34097)
kcm pushed a commit that referenced this pull request Oct 30, 2018
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Data Management/Watcher >test Issues or PRs that are addressing/adding tests v6.5.0 v7.0.0-beta1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

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

3 participants