Skip to content
This repository has been archived by the owner on Jun 13, 2021. It is now read-only.

Auto parameters #505

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from
Draft

Conversation

simonferquel
Copy link
Contributor

- What I did

Added a range of automatically generated parameters for

  • replicas
  • limits
  • reservations

- How I did it

Overrides are materialized to CNAB parameters, whose destination is a file path mirroring the structure of the compose file.
When the run tool extract the package, it reconstructs the overrides as an "overriding compose file"

This means that we can use this to easily add new "automatic parameters" in the future

- How to verify it
//TODO

- Description for the changelog
//TODO

- A picture of a cute animal (not mandatory but encouraged)
//TODO

Signed-off-by: Simon Ferquel <simon.ferquel@docker.com>
Signed-off-by: Simon Ferquel <simon.ferquel@docker.com>
@GordonTheTurtle
Copy link

Looks like we would not be able to merge this PR because of merge conflicts. Please rebase, fix conflicts, and force push to your branch.

This makes the compose loader happy (the loader always checks that all
referenced compose files have the same version)

Signed-off-by: Simon Ferquel <simon.ferquel@docker.com>
Signed-off-by: Simon Ferquel <simon.ferquel@docker.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants