camunda / camunda

Process Orchestration Framework
https://camunda.com/platform/
3.34k stars 612 forks source link

Make sure configurations on OpenSearch files are pulled from OpenSearch config #24355

Open grlimacan opened 2 weeks ago

grlimacan commented 2 weeks ago

Engineering DRI: @grlimacan

Context

In these OpenSearch files, we are accidentally pulling configurations for ElasticSearch. SnapshotRepositoryOS BackupReaderOS AbstractProcessDistributedByModelElementInterpreterOS DecisionGroupByMatchedRuleInterpreterOS GroupByIncidentFlowNodeInterpreterOS AbstractProcessGroupByIdentityInterpreterOS DecisionViewRawDataInterpreterOS

Acceptance Criteria

All OpenSearch configurations are being pulled properly for C7 and C8

Testing Notes

From QA the only files that can be tested are the ones concerning backup. So please test C7 + C8 that the property opensearch.backup.repositoryName is being used correctly

Links

Breakdown

### Pull Requests
- [ ] https://github.com/camunda/camunda-optimize/pull/14627
- [ ] https://github.com/camunda/camunda/pull/24373

Task Lifecycle

For managing the issue lifecycle, please use the workflow commands. You can see the available commands by writing /help as a comment on this issue.

Review

Engineering Review

QA Review

Completion

grlimacan commented 2 weeks ago

/assign @grlimacan

grlimacan commented 2 weeks ago

/eng review @oleksandr-kriuchenko-lohika

oleksandr-kriuchenko-lohika commented 2 weeks ago

/eng approve