feat: add data migration for 8.8 #4090
Merged
Conversation
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
github-actions
bot
added
version/8.8
panagiotisgts
requested changes
Sep 4, 2025
charts/camunda-platform-8.8/templates/orchestration/data-migration-configmap.yaml
Outdated
Show resolved
Hide resolved
| migration: | ||
| process: | ||
| ## @param orchestration.migration.process.enabled if true, the process migration occurs. | ||
| data: |
Contributor
There was a problem hiding this comment.
Can this nesting be removed? It might be confusing, since the property would be CAMUNDA_MIGRATION_PROCESS_* whereas in the values it would be CAMUNDA_MIGRATION_DATA_PROCESS_*. Or is that something that we are doing in other places as well?
Contributor
Author
There was a problem hiding this comment.
The values.yaml does not conform to the application config so it is okay for the values.yaml to have a different structure.
Contributor
Author
There was a problem hiding this comment.
One of the main goals of having Helm is to have an abstraction layer on top of the application to not worry about the application config.
charts/camunda-platform-8.8/templates/orchestration/data-migration-configmap.yaml
Outdated
Show resolved
Hide resolved
aabouzaid
reviewed
Sep 5, 2025
charts/camunda-platform-8.8/templates/orchestration/data-migration-configmap.yaml
Outdated
Show resolved
Hide resolved
aabouzaid
reviewed
Sep 5, 2025
charts/camunda-platform-8.8/templates/orchestration/data-migration-job.yaml
Show resolved
Hide resolved
github-actions
bot
added
the
version:13.0.0-alpha8
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Which problem does the PR fix?
related: https://github.com/camunda/team-distribution/issues/415
What's in this PR?
The data migration includes the following migrations:
NOTE:
This feature is not tested. It is not possible to test this feature currently since 8.8-SNAPSHOT is in the process of being created.
context: https://camunda.slack.com/archives/C07UA4C7STY/p1756965922109239?thread_ts=1756918699.759809&cid=C07UA4C7STY
We will have to wait for the image to be available. For this reason, a new issue has been created on the QA side to test out this feature: https://github.com/camunda/product-hub/issues/3002
Checklist
Please make sure to follow our Contributing Guide.
Before opening the PR:
make go.update-golden-only.After opening the PR: