Create index alias when index is created

assigned to @zhibinmai

added 1 commit

a5b33c9b - Add unit tests

Compare with previous version

added 1 commit

6e65d7dd - Update artifact

Compare with previous version

added 1 commit

eab4a842 - Failure to create the alias should not have impact on the status of index creation

Compare with previous version

added 3 commits

eab4a842...417c93ef - 2 commits from branch master
cdc361a2 - Merge branch 'master' of https://community.opengroup.org/osdu/platform/system/indexer-service

Compare with previous version

requested review from @fhoueto.amz, @Rostislav_Dublin, @krveduru, @spencer, and @jingdongsun

@fhoueto.amz @Rostislav_Dublin @krveduru @spencer @jingdongsun Please help review this MR. The indexer creates the alias of the index (name) when the index is created

Is it possible to add an integration test that verifies that the alias exists at Elasticsearch?

@Rustam_Lotsmanenko It is fair to add integration tests though we did lots of manual tests in the dev env. The reason I didn't do is that the current integration tests (excluding my change) are not stable. I will see what I can do. Thanks.

added 7 commits

cdc361a2...1033b737 - 6 commits from branch master
b3384209 - Merge from master

Compare with previous version

Is there a passing pipeline for this. I was not able to find one

added 1 commit

20ea77c3 - Update the core common version

Compare with previous version

added 1 commit

2708ec8c - Update artifact

Compare with previous version

added 1 commit

3abb9565 - Ensure that createIndexAlias won't affect the creation of index

Compare with previous version

resolved all threads

added 1 commit

a2479b14 - Refactor the codes for creating index alias and update unit test

Compare with previous version

Given

the alias is created when the index is created in the indexer
the current indexer test framework is creating the indices directly through elastic search API instead of through the indexer API, the alias won't be created when the index is created unless we change the way on how to create the index in the integration tests.
the indexer has enough coverage on creating the alias from its unit tests
the search service has integration test that creates the alias of the index on demand. It uses same kind of elastic API to create the alias as the indexer

I suggest to skip the integration test on creating the alias in the indexer.