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

Unexpected behavior when using arbitrary elasticsearch visibility index #609

Open
phuongdnguyen opened this issue Nov 5, 2024 · 2 comments
Assignees

Comments

@phuongdnguyen
Copy link

phuongdnguyen commented Nov 5, 2024

Expected Behavior

Current temporal helm chart expose elasticsearch visibility index name as a configurable option, see.
Expected behavior for user is they can specify arbitrary index name.

Actual Behavior

This setting only works if visibility index starts with temporal_visibility_v1_dev, as shown here

Steps to Reproduce the Problem

  1. Use random visibilityIndex name in elasticsearch block when starting temporal server
  2. Start the temporal server along with its dependencies
  3. Start some random workflow
  4. Temporal ui workflow list page is empty

Specifications

  • Version: v1.25.1
  • Platform: darwin arm64
@bergundy
Copy link
Member

Looks like the code wants temporal_visibility_v1*. This is by design, we should enforce this in the helm chart.
There's a way to validate chart values as shown here: https://www.arthurkoziel.com/validate-helm-chart-values-with-json-schemas/.

@bergundy bergundy transferred this issue from temporalio/temporal Nov 14, 2024
@phuongdnguyen
Copy link
Author

phuongdnguyen commented Nov 15, 2024

Yes i also think that the helm chart is the go to place for deployment value validation. This will resolve this temporalio/temporal#5475 too, i think.

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

No branches or pull requests

3 participants