Skip to content

Commit

Permalink
Mute watcher rolling upgrade yaml tests (#81935)
Browse files Browse the repository at this point in the history
Relates to #81110
  • Loading branch information
martijnvg committed Dec 21, 2021
1 parent d46ccb0 commit bbd08e2
Show file tree
Hide file tree
Showing 4 changed files with 27 additions and 10 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,6 @@

import org.apache.lucene.util.TimeUnits;
import org.elasticsearch.Version;
import org.elasticsearch.client.Request;
import org.elasticsearch.client.Response;
import org.elasticsearch.common.settings.Settings;
import org.elasticsearch.common.util.concurrent.ThreadContext;
import org.elasticsearch.test.rest.ESRestTestCase;
Expand All @@ -24,12 +22,6 @@

import java.nio.charset.StandardCharsets;
import java.util.Base64;
import java.util.List;
import java.util.Map;
import java.util.concurrent.TimeUnit;

import static org.hamcrest.Matchers.equalTo;
import static org.hamcrest.Matchers.greaterThanOrEqualTo;

@TimeoutSuite(millis = 5 * TimeUnits.MINUTE) // to account for slow as hell VMs
public class UpgradeClusterClientYamlTestSuiteIT extends ESClientYamlSuiteTestCase {
Expand All @@ -55,7 +47,8 @@ public void waitForTemplates() throws Exception {
}
}

@Before
/*@Before
// AwaitFix=https://github.com/elastic/elasticsearch/issues/81110
public void waitForWatcher() throws Exception {
// Wait for watcher to be in started state in order to avoid errors due
// to manually executing watches prior for watcher to be ready:
Expand All @@ -73,7 +66,7 @@ public void waitForWatcher() throws Exception {
} catch (AssertionError e) {
throw new AssertionError("Failure in test setup: Failed to initialize at least 3 watcher nodes", e);
}
}
}*/

@Override
protected boolean preserveIndicesUponCompletion() {
Expand Down
Original file line number Diff line number Diff line change
@@ -1,5 +1,8 @@
---
"CRUD watch APIs":
- skip:
version: "all"
reason: "https://github.com/elastic/elasticsearch/issues/81110"
# no need to put watch, exists already
- do:
watcher.get_watch:
Expand Down Expand Up @@ -72,3 +75,8 @@
watcher.stats: {}
- match: { "manually_stopped": false }
- match: { "stats.0.watcher_state": "started" }

---
"Test do something":
- do: { ping: { } }
- is_true: ''
Original file line number Diff line number Diff line change
@@ -1,5 +1,8 @@
---
"CRUD watch APIs":
- skip:
version: "all"
reason: "https://github.com/elastic/elasticsearch/issues/81110"
- do:
watcher.put_watch:
id: "my_watch"
Expand Down Expand Up @@ -101,3 +104,8 @@
watcher.stats: {}
- match: { "manually_stopped": false }
- match: { "stats.0.watcher_state": "started" }

---
"Test do something":
- do: { ping: { } }
- is_true: ''
Original file line number Diff line number Diff line change
@@ -1,5 +1,8 @@
---
"CRUD watch APIs":
- skip:
version: "all"
reason: "https://github.com/elastic/elasticsearch/issues/81110"
# no need to put watch, exists already
- do:
watcher.get_watch:
Expand Down Expand Up @@ -73,3 +76,8 @@
watcher.stats: {}
- match: { "manually_stopped": false }
- match: { "stats.0.watcher_state": "started" }

---
"Test do something":
- do: { ping: { } }
- is_true: ''

0 comments on commit bbd08e2

Please sign in to comment.