Skip to content

fix: revert ES_VERSION to v7 for v1.28.x compatibility#317

Merged
rkannan82 merged 1 commit intorelease/v1.28.xfrom
fix/es-version-v7-v1.28.x
Mar 18, 2026
Merged

fix: revert ES_VERSION to v7 for v1.28.x compatibility#317
rkannan82 merged 1 commit intorelease/v1.28.xfrom
fix/es-version-v7-v1.28.x

Conversation

@rkannan82
Copy link
Copy Markdown

Summary

Reverts ES_VERSION from v8 back to v7 in docker-compose.yml.

PR #315 set ES_VERSION=v8 (mirroring the v1.29.x fix), but v1.28.x auto-setup only contains v7 schema files (cluster_settings_v7.json, index_template_v7.json). With ES_VERSION=v8, auto-setup tries to load cluster_settings_v8.json which doesn't exist, causing:

curl: Failed to open
curl: /etc/temporal/schema/elasticsearch/visibility/cluster_settings_v8.json

The correct config for v1.28.x is:

This is blocking Docker image promotion for v1.28.3.

Test plan

  • CI "Ensure images work" step passes

Made with Cursor

The v1.28.x auto-setup image only contains v7 ES schema files
(cluster_settings_v7.json, index_template_v7.json). Setting ES_VERSION=v8
causes auto-setup to look for cluster_settings_v8.json which doesn't exist.

ES 8.5.0 is backwards-compatible with v7 API calls, so using ES_VERSION=v7
with ELASTICSEARCH_VERSION=8.5.0 is the correct configuration for v1.28.x.

Made-with: Cursor
@rkannan82 rkannan82 requested a review from a team as a code owner March 18, 2026 22:43
@rkannan82 rkannan82 closed this Mar 18, 2026
@rkannan82 rkannan82 reopened this Mar 18, 2026
@rkannan82 rkannan82 requested a review from prathyushpv March 18, 2026 23:04
@rkannan82 rkannan82 merged commit ca8f662 into release/v1.28.x Mar 18, 2026
7 of 9 checks passed
@rkannan82 rkannan82 deleted the fix/es-version-v7-v1.28.x branch March 18, 2026 23:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants