DEV-72 - Add Helm chart support for transform.js #75
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
When we need to use custom JS transform in Replicator, we specify
transform:
type: js
config: ./transform.js
in replicator configuration.
Problem is that this configuration references an external file
that is currently not in Helm chart. Helm does not allow to
mount external files into pod at the time of the deployment,
see Helm issue <helm/helm#3276>.
This change adds a `transform.js` file into Helm chart that
we can reference in our configuration during deployment. the content
if this file is set from chart values, and we can overwrite this
during installation.
Bu default, the transform file is not used and the content of
`transform.js` is empty.
To use the transform, we need to overwrite `transform` section
in application configuration, and overwrite `transform.js` file
itself:
helm install replicator-with-transform \
es-replicator/es-replicator \
--values values.yml \
--set-file transformJs=./local-transform.js
`values.yml` should overwrite `transform` section:
transform:
type: js
config: ./transform.js
Local file `local-transform.js` should define transformations / filters
as per <https://replicator.eventstore.org/docs/features/transforms/js/>.
alexeyzimarev
approved these changes
Apr 24, 2023
Member
|
I need to check why the UI fails building. |
alexeyzimarev
changed the title
josephcummings
changed the title
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 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.
When we need to use custom JS transform in Replicator, we specify
in replicator configuration.
Problem is that this configuration references an external file that is currently not in Helm chart. Helm does not allow to mount external files into pod at the time of the deployment, see Helm issue helm/helm#3276.
This change adds a
transform.jsfile into Helm chart that we can reference in our configuration during deployment. the content of this file is set from chart values, and we can overwrite this during installation.By default, the transform file is not used and the content of
transform.jsis empty.To use the transform, we need to overwrite
transformsection in application configuration, and overwritetransform.jsfile content:values.ymlshould overwritetransformsection:Local file
local-transform.jsshould define transformations / filters as per https://replicator.eventstore.org/docs/features/transforms/js/.Refs: #74