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

fix/INT-5482: Allow string interpolation for the 'endpoints' property in docker compose #319

Conversation

khoaHyh
Copy link
Contributor

@khoaHyh khoaHyh commented Aug 30, 2023

Passing in interpolated variables doesn't work for the entrypoints property in tb-registry's services.yml file. This PR is to expand all the values passed in the entrypoints array.

@khoaHyh khoaHyh requested a review from jbinto August 30, 2023 01:50
@khoaHyh khoaHyh marked this pull request as ready for review August 31, 2023 14:12
@@ -387,6 +387,9 @@ func readServices(op errors.Op, r Registry, opts readServicesOptions) (serviceGl
for i, volume := range s.Remote.Volumes {
s.Remote.Volumes[i].Value = ve.expand(volume.Value, "remote.volumes")
}
for i, value := range s.Entrypoint {
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

todo: make test

@khoaHyh khoaHyh merged commit 1b994ab into master Aug 31, 2023
@khoaHyh khoaHyh deleted the fix/INT-5482_allow-interpolation-for-entrypoints-property-docker-compose branch August 31, 2023 19:12
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