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

[Backport 2.16] [Remote Routing Table] Add integration tests for Remote Routing Table #14865

Closed
wants to merge 1 commit into from

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport b9d5804 from #14631.

Signed-off-by: Shailendra Singh <[email protected]>
(cherry picked from commit b9d5804)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Copy link
Contributor

❌ Gradle check result for db2f322: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Copy link
Member

@dbwiddis dbwiddis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Task :server:compileInternalClusterTestJava
/home/runner/work/OpenSearch/OpenSearch/server/src/internalClusterTest/java/org/opensearch/gateway/remote/RemoteRoutingTableServiceIT.java:60: error: package RemoteRoutingTableBlobStore does not exist
RemoteRoutingTableBlobStore.REMOTE_ROUTING_TABLE_PATH_TYPE_SETTING.getKey(),
^
/home/runner/work/OpenSearch/OpenSearch/server/src/internalClusterTest/java/org/opensearch/gateway/remote/RemoteRoutingTableServiceIT.java:181: error: cannot find symbol
indexRoutingPath = getIndexRoutingPath(baseMetadataPath.add(INDEX_ROUTING_TABLE), indexRoutingTables.get(0).getIndex().getUUID());
^
symbol: variable INDEX_ROUTING_TABLE
location: class RemoteRoutingTableServiceIT

Copy link
Contributor

❌ Gradle check result for db2f322: null

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Copy link
Contributor

❌ Gradle check result for db2f322: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@dblock
Copy link
Member

dblock commented Jul 22, 2024

@shailendra0811 this is not expected I believe, can you please take a look?

@shailendra0811
Copy link
Contributor

This backport PR is failing as PR is not backported yet.

Copy link
Contributor

❌ Gradle check result for db2f322: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@dblock
Copy link
Member

dblock commented Jul 23, 2024

Looks like #14862 was merged. I think you'll need to manually backport this one because it's out of order?

@dbwiddis
Copy link
Member

@dbwiddis dbwiddis closed this Jul 23, 2024
@dbwiddis dbwiddis deleted the backport/backport-14631-to-2.16 branch July 24, 2024 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants