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] Frequent failure of WatcherRestIT painless/50_update_scripts/Test condition scripts are updated on execution #45582

Closed
mark-vieira opened this issue Aug 14, 2019 · 3 comments · Fixed by #47690
Closed

[CI] Frequent failure of WatcherRestIT painless/50_update_scripts/Test condition scripts are updated on execution #45582

mark-vieira opened this issue Aug 14, 2019 · 3 comments · Fixed by #47690
Labels
:Core/Infra/Scripting Scripting abstractions, Painless, and Mustache :Data Management/Watcher >test-failure Triaged test failures from CI

Comments

@mark-vieira
Copy link
Contributor

We are getting frequent failures of painless/50_update_scripts/Test condition scripts are updated on execution} in CI.

https://build-stats.elastic.co/app/kibana#/discover?_g=(refreshInterval:(pause:!t,value:0),time:(from:now-7d,mode:quick,to:now))&_a=(columns:!(branch,test),index:e58bf320-7efd-11e8-bf69-63c8ef516157,interval:auto,query:(language:kuery,query:'class%20:%20"org.elasticsearch.smoketest.WatcherRestIT"%20and%20test:%20"test%20%7Byaml%3Dpainless%2F50_update_scripts%2FTest%20condition%20scripts%20are%20updated%20on%20execution%7D"'),sort:!(time,desc))

Example: https://gradle-enterprise.elastic.co/s/zc2v7np5oofi6/tests/6ej7h6toy4af6-qkrsuwjuipy72
@mark-vieira mark-vieira added :Core/Infra/Scripting Scripting abstractions, Painless, and Mustache >test-failure Triaged test failures from CI :Data Management/Watcher labels Aug 14, 2019
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-features

@mark-vieira
Copy link
Contributor Author

Muted on master in 8b92ca2.

jakelandis added a commit to jakelandis/elasticsearch that referenced this issue Oct 7, 2019
@jakelandis
Watcher - renable update_scripts tests
55894d1 
These tests are believed to be fixed by elastic#43939

closes elastic#45582 and elastic#43975

These tests are currently only muted master.

Jul 23 - muted in master (Test transform scripts are updated on execution)
Aug 14 - muted in master (Test condition scripts are updated on execution)
Aug 15 - last recorded failure (from either test on any branch)
Aug 16 - elastic#43939 commited to master
Aug 22 - elastic#43939 backported to 7.x
@jakelandis jakelandis mentioned this issue Oct 7, 2019
Merged
jakelandis added a commit that referenced this issue Oct 7, 2019
@jakelandis
Re-enable Watcher rest tests (#47690)
07ed3d5 
These tests are believed to be fixed by #43939

closes #45582 and #43975
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
:Core/Infra/Scripting Scripting abstractions, Painless, and Mustache :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.

Re-enable Watcher rest tests jakelandis/elasticsearch
2 participants
@mark-vieira @elasticmachine