Add dual visibility support#803
Closed
mihaelabalas84 wants to merge 2 commits intotemporalio:mainfrom
Closed
Conversation
|
|
Contributor
|
We have just merged the v1 branch to main which has simplified a lot of the helpers and templates. If you're able to adjust the PR to the new structure (which I think will be much cleaner than the previous) then we'd love to get this feature merged. |
This was referenced Dec 18, 2025
ignaciogonzalezsomoza
added a commit
to ignaciogonzalezsomoza/helm-charts
that referenced
this pull request
Jan 22, 2026
Add support for secondaryVisibilityStore configuration to enable dual visibility feature in Temporal server. This allows writing to two visibility stores simultaneously for migration scenarios. Changes: - Add secondaryVisibilityStore support in temporal.persistence.filterConfig - Include secondaryVisibility in temporal.persistence.eachStore iteration - Add TEMPORAL_SECONDARY_VISIBILITY_STORE_PASSWORD env var to server pods - Automatically process secondary visibility store in schema jobs This implementation adapts PR temporalio#803 to the V1 chart architecture, leveraging the centralized helper functions introduced in V1 to avoid code duplication. Resolves: temporalio#803
ignaciogonzalezsomoza
added a commit
to ignaciogonzalezsomoza/helm-charts
that referenced
this pull request
Jan 22, 2026
Add support for secondaryVisibilityStore configuration to enable dual visibility feature in Temporal server. This allows writing to two visibility stores simultaneously for migration scenarios. Changes: - Add secondaryVisibilityStore support in temporal.persistence.filterConfig - Include secondaryVisibility in temporal.persistence.eachStore iteration - Add TEMPORAL_SECONDARY_VISIBILITY_STORE_PASSWORD env var to server pods - Automatically process secondary visibility store in schema jobs This implementation adapts PR temporalio#803 to the V1 chart architecture, leveraging the centralized helper functions introduced in V1 to avoid code duplication. Resolves: temporalio#803
6 tasks
ignaciogonzalezsomoza
added a commit
to ignaciogonzalezsomoza/helm-charts
that referenced
this pull request
Jan 23, 2026
Add support for secondaryVisibilityStore configuration to enable dual visibility feature in Temporal server. This allows writing to two visibility stores simultaneously for migration scenarios. Changes: - Add secondaryVisibilityStore support in temporal.persistence.filterConfig - Include secondaryVisibility in temporal.persistence.eachStore iteration - Add TEMPORAL_SECONDARY_VISIBILITY_STORE_PASSWORD env var to server pods - Automatically process secondary visibility store in schema jobs This implementation adapts PR temporalio#803 to the V1 chart architecture, leveraging the centralized helper functions introduced in V1 to avoid code duplication. Resolves: temporalio#803
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What was changed
Why?
Checklist
Closes
How was this tested: