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

[AUTOCUT] Integration Test failed for opensearch-system-templates: 2.17.0 #21

Closed
gaiksaya opened this issue Sep 6, 2024 · 7 comments
Closed
Labels
autocut v2.17.0 Issues targeting release v2.17.0

Comments

@gaiksaya
Copy link
Member

gaiksaya commented Sep 6, 2024

The integration test failed at distribution level for component opensearch-system-templates
Version: 2.17.0
Distribution: tar
Architecture: x64
Platform: linux

Please check the logs: https://build.ci.opensearch.org/job/integ-test/8630/display/redirect

* Test-report manifest:*
- https://ci.opensearch.org/ci/dbc/integ-test/2.17.0/10271/linux/x64/tar/test-results/8630/integ-test/test-report.yml

Note: Steps to reproduce, additional logs and other files can be found within the above test-report manifest.
Instructions of this test-report manifest can be found here.

@gaiksaya gaiksaya added autocut untriaged v2.17.0 Issues targeting release v2.17.0 labels Sep 6, 2024
@gaiksaya
Copy link
Member Author

gaiksaya commented Sep 6, 2024

Hi @mohitamg
Seeing below error in integration tests:

    {"error":{"root_cause":[{"type":"invalid_index_template_exception","reason":"index_template [my-metrics-template] invalid, cause [index template [my-metrics-template] specifies a context which cannot be used without enabling: cluster.application_templates.enabled]"}],"type":"invalid_index_template_exception","reason":"index_template [my-metrics-template] invalid, cause [index template [my-metrics-template] specifies a context which cannot be used without enabling: cluster.application_templates.enabled]"},"status":400}

        at __randomizedtesting.SeedInfo.seed([CEEB7C3E9FD3604E:40C1544C45A8D424]:0)

        at app//org.opensearch.client.RestClient.convertResponse(RestClient.java:479)

        at app//org.opensearch.client.RestClient.performRequest(RestClient.java:371)

        at app//org.opensearch.client.RestClient.performRequest(RestClient.java:346)

        at app//org.opensearch.system.applicationtemplates.CreateIndexTemplateWithContextTemplateIT.testCreateIndexWithContextBasedTemplate(CreateIndexTemplateWithContextTemplateIT.java:59)

  2> NOTE: leaving temporary files on disk at: /tmp/tmppekdv1i0/opensearch-system-templates/build/testrun/integTest/temp/org.opensearch.system.applicationtemplates.CreateIndexTemplateWithContextTemplateIT_CEEB7C3E9FD3604E-001

  2> NOTE: test params are: codec=Asserting(Lucene99): {}, docValues:{}, maxPointsInLeafNode=177, maxMBSortInHeap=7.269244417062377, sim=Asserting(RandomSimilarity(queryNorm=true): {}), locale=be-BY, timezone=ECT

  2> NOTE: Linux 6.1.102-108.177.amzn2023.x86_64 amd64/Eclipse Adoptium 21.0.1 (64-bit)/cpus=4,threads=2,free=316366808,total=536870912

  2> NOTE: All tests run in this JVM: [CreateIndexTemplateWithContextTemplateIT]

Do we need to add any additional cluster config?

@mgodwan
Copy link
Member

mgodwan commented Sep 6, 2024

Yes, the tests require 2 settings to be set.

We do set this in build.gradle

systemProperty 'opensearch.experimental.feature.application_templates.enabled', 'true'
setting 'cluster.application_templates.enabled', 'true'

Are these not honored in CI run or is the testClusters.integTest being overridden?

@gaiksaya
Copy link
Member Author

gaiksaya commented Sep 6, 2024

No! Since we are testing against the remote cluster deployed in localhost not the internal JVM spun cluster. See the command used here: https://github.com/opensearch-project/opensearch-build/blob/main/scripts/default/integtest.sh#L105

@gaiksaya
Copy link
Member Author

gaiksaya commented Sep 6, 2024

We would need to add something like this in the test manifest https://github.com/opensearch-project/opensearch-build/blob/main/manifests/2.17.0/opensearch-2.17.0-test.yml#L137-L139
Can you create a PR for this component?

@mgodwan
Copy link
Member

mgodwan commented Sep 6, 2024

Got it! Thanks for the pointer
Let me raise a PR to unblock this

@mgodwan
Copy link
Member

mgodwan commented Sep 6, 2024

Raised the PR : opensearch-project/opensearch-build#5002

@gaiksaya
Copy link
Member Author

gaiksaya commented Sep 6, 2024

Closing the issue as tests are passing now. https://build.ci.opensearch.org/blue/organizations/jenkins/integ-test/detail/integ-test/8631/pipeline/
Should re-open in case it fails for other platforms/archs.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autocut v2.17.0 Issues targeting release v2.17.0
Projects
None yet
Development

No branches or pull requests

2 participants