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.

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

New get_alias tests failing on 6.x branch #30806

Closed
ywelsch opened this issue May 23, 2018 · 3 comments
Closed

New get_alias tests failing on 6.x branch #30806

ywelsch opened this issue May 23, 2018 · 3 comments
Assignees
Labels
:Data Management/Indices APIs APIs to create and manage indices and templates >test-failure Triaged test failures from CI v6.4.0

Comments

@ywelsch
Copy link
Contributor

ywelsch commented May 23, 2018

New REST tests for get_alias have been added in #29513
They fail in mixed-cluster tests on the 6.x, see for example https://elasticsearch-ci.elastic.co/job/elastic+elasticsearch+6.x+multijob-unix-compatibility/os=centos/1076/console
I've reverted the commit on the 6.x branch (see ad43231)

Log output of test failures

"TEST_FAILURE",
 {
  "description": "ID#test {p0=indices.get_alias/10_basic/Get aliases via /_alias/_all}(org.elasticsearch.backwards.MixedClusterClientYamlTestSuiteIT)",
  "message": "Failure at [indices.get_alias/10_basic:51]: field [test_index_3] has a true value but it shouldn't\nExpected: (\"\" or equalToIgnoringCase(\"false\") or \"0\")\n     but: was \"{aliases={}}\"",
  "trace": "java.lang.AssertionError: Failure at [indices.get_alias/10_basic:51]: field [test_index_3] has a true value but it shouldn't\nExpected: (\"\" or equalToIgnoringCase(\"false\") or \"0\")\n     but: was \"{aliases={}}\"\n\tat __randomizedtesting.SeedInfo.seed([5BD30D4D76C681C2:D3873297D83AEC3A]:0)\n\tat org.elasticsearch.test.rest.yaml.ESClientYamlSuiteTestCase.executeSection(ESClientYamlSuiteTestCase.java:356)\n\tat org.elasticsearch.test.rest.yaml.ESClientYamlSuiteTestCase.test(ESClientYamlSuiteTestCase.java:333)\n\tat jdk.internal.reflect.GeneratedMethodAccessor15.invoke(Unknown Source)\n\tat java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)\n\tat java.base/java.lang.reflect.Method.invoke(Method.java:564)\n\tat com.carrotsearch.randomizedtesting.RandomizedRunner.invoke(RandomizedRunner.java:1713)\n\tat com.carrotsearch.randomizedtesting.RandomizedRunner$8.evaluate(RandomizedRunner.java:907)\n\tat com.carrotsearch.randomizedtesting.RandomizedRunner$9.evaluate(RandomizedRunner.java:943)\n\tat com.carrotsearch.randomizedtesting.RandomizedRunner$10.evaluate(RandomizedRunner.java:957)\n\tat com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)\n\tat org.apache.lucene.util.TestRuleSetupTeardownChained$1.evaluate(TestRuleSetupTeardownChained.java:49)\n\tat org.apache.lucene.util.AbstractBeforeAfterRule$1.evaluate(AbstractBeforeAfterRule.java:45)\n\tat org.apache.lucene.util.TestRuleThreadAndTestName$1.evaluate(TestRuleThreadAndTestName.java:48)\n\tat org.apache.lucene.util.TestRuleIgnoreAfterMaxFailures$1.evaluate(TestRuleIgnoreAfterMaxFailures.java:64)\n\tat org.apache.lucene.util.TestRuleMarkFailure$1.evaluate(TestRuleMarkFailure.java:47)\n\tat com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)\n\tat com.carrotsearch.randomizedtesting.ThreadLeakControl$StatementRunner.run(ThreadLeakControl.java:368)\n\tat com.carrotsearch.randomizedtesting.ThreadLeakControl.forkTimeoutingTask(ThreadLeakControl.java:817)\n\tat com.carrotsearch.randomizedtesting.ThreadLeakControl$3.evaluate(ThreadLeakControl.java:468)\n\tat com.carrotsearch.randomizedtesting.RandomizedRunner.runSingleTest(RandomizedRunner.java:916)\n\tat com.carrotsearch.randomizedtesting.RandomizedRunner$5.evaluate(RandomizedRunner.java:802)\n\tat com.carrotsearch.randomizedtesting.RandomizedRunner$6.evaluate(RandomizedRunner.java:852)\n\tat com.carrotsearch.randomizedtesting.RandomizedRunner$7.evaluate(RandomizedRunner.java:863)\n\tat org.apache.lucene.util.AbstractBeforeAfterRule$1.evaluate(AbstractBeforeAfterRule.java:45)\n\tat com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)\n\tat org.apache.lucene.util.TestRuleStoreClassName$1.evaluate(TestRuleStoreClassName.java:41)\n\tat com.carrotsearch.randomizedtesting.rules.NoShadowingOrOverridesOnMethodsRule$1.evaluate(NoShadowingOrOverridesOnMethodsRule.java:40)\n\tat com.carrotsearch.randomizedtesting.rules.NoShadowingOrOverridesOnMethodsRule$1.evaluate(NoShadowingOrOverridesOnMethodsRule.java:40)\n\tat com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)\n\tat com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)\n\tat org.apache.lucene.util.TestRuleAssertionsRequired$1.evaluate(TestRuleAssertionsRequired.java:53)\n\tat org.apache.lucene.util.TestRuleMarkFailure$1.evaluate(TestRuleMarkFailure.java:47)\n\tat org.apache.lucene.util.TestRuleIgnoreAfterMaxFailures$1.evaluate(TestRuleIgnoreAfterMaxFailures.java:64)\n\tat org.apache.lucene.util.TestRuleIgnoreTestSuites$1.evaluate(TestRuleIgnoreTestSuites.java:54)\n\tat com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)\n\tat com.carrotsearch.randomizedtesting.ThreadLeakControl$StatementRunner.run(ThreadLeakControl.java:368)\n\tat java.base/java.lang.Thread.run(Thread.java:844)\nCaused by: java.lang.AssertionError: field [test_index_3] has a true value but it shouldn't\nExpected: (\"\" or equalToIgnoringCase(\"false\") or \"0\")\n     but: was \"{aliases={}}\"\n\tat org.hamcrest.MatcherAssert.assertThat(MatcherAssert.java:20)\n\tat org.junit.Assert.assertThat(Assert.java:956)\n\tat org.elasticsearch.test.rest.yaml.section.IsFalseAssertion.doAssert(IsFalseAssertion.java:59)\n\tat org.elasticsearch.test.rest.yaml.section.Assertion.execute(Assertion.java:76)\n\tat org.elasticsearch.test.rest.yaml.ESClientYamlSuiteTestCase.executeSection(ESClientYamlSuiteTestCase.java:349)\n\t... 36 more\n",
  "throwableString": "java.lang.AssertionError: Failure at [indices.get_alias/10_basic:51]: field [test_index_3] has a true value but it shouldn't\nExpected: (\"\" or equalToIgnoringCase(\"false\") or \"0\")\n     but: was \"{aliases={}}\"",
  "throwableClass": "java.lang.AssertionError",
  "assertionViolation": true,
  "assumptionViolation": false
 }
]
@ywelsch ywelsch added >test-failure Triaged test failures from CI :Data Management/Indices APIs APIs to create and manage indices and templates v6.4.0 labels May 23, 2018
@colings86 colings86 added :Data Management/Indices APIs APIs to create and manage indices and templates and removed :Data Management/Indices APIs APIs to create and manage indices and templates labels May 23, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra

javanna added a commit to javanna/elasticsearch that referenced this issue May 23, 2018
These additional tests were previously reverted in 6.x due to failing bwc tests.
They should be skipped against 5.6 nodes, bwc tests fail when the node hit by the request is on 5.6.

Relates to elastic#29513
Relates to elastic#25114
Closes elastic#30806
javanna added a commit that referenced this issue May 24, 2018
These additional tests were previously reverted in 6.x due to failing bwc tests.
They should be skipped against 5.6 nodes, bwc tests fail when the node hit by the request is on 5.6.

Relates to #29513
Relates to #25114
Closes #30806
@jasontedor
Copy link
Member

Closed by #30814

@javanna
Copy link
Member

javanna commented May 25, 2018

thanks @jasontedor for closing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Data Management/Indices APIs APIs to create and manage indices and templates >test-failure Triaged test failures from CI v6.4.0
Projects
None yet
Development

No branches or pull requests

5 participants