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

deploy: use anchor to reuse common env variables. #12

Merged
merged 1 commit into from
Mar 13, 2024

Conversation

henriquesimoes
Copy link
Collaborator

@henriquesimoes henriquesimoes commented Mar 8, 2024

Reduce the amount of information replicated in the compose file. Merging the reusable mappings must be done explicitly, using YAML merge feature, so it is not a limitation of the docker-compose we use in deploy.

Fixes: 655c72f ("deploy: Add docker-compose to launch IOCs")

Reduce the amount of information replicated in the compose file. Merging
the reusable mappings must be done explicitly, using YAML merge feature
[1], so it is not a limitation of the docker-compose we use in deploy.

[1]: https://yaml.org/type/merge.html

Fixes: 655c72f ("deploy: Add docker-compose to launch IOCs")
@henriquesimoes henriquesimoes requested a review from ericonr March 8, 2024 18:01
Copy link
Member

@ericonr ericonr left a comment

Choose a reason for hiding this comment

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

Did you manage to test this on our test crate?

@henriquesimoes
Copy link
Collaborator Author

Did you manage to test this on our test crate?

Yes, but not with RFFE IOC itself. I ran a Debian container defining the variables the same way we do in the compose file (including the indirect definition of CRATE_NUMBER).

@ericonr
Copy link
Member

ericonr commented Mar 12, 2024

Feel free to merge, then :)

@henriquesimoes henriquesimoes merged commit 286286f into main Mar 13, 2024
1 check passed
@henriquesimoes henriquesimoes deleted the simplify-yml branch March 13, 2024 12:29
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