Debian docker repo has docker-engine package

[taylorbyte]

Changed pkgs for Debian from docker-io to docker-engine. its the only package available in https://apt.dockerproject.org/repo/ for Debian.

[fpytloun]

Hello,
correct community repository is deb [arch=amd64] https://download.docker.com/linux/debian/ stretch stable
and package name is docker-ce with latest version 17.12.0~ce-0~debian.

Default behavior here is to use docker.io which is Debian upstream package (https://tracker.debian.org/pkg/docker.io) unless overriden from pillar.

[fpytloun]

See comment above

[taylorbyte]

Sorry about that.
I don't mean to be rude I just want to help; i can't seem to find anywhere in the formula that sets the repo you referred to as default, this to me means the formula is incomplete or broken without directions on how to set the repo via a pillar, the only thing i can find about setting a repo is an example given here: https://github.com/salt-formulas/salt-formula-docker/blob/master/tests/pillar/repo_docker.sls which uses the repo i was referring to. It seems that some documentation is missing which should tell people which repo to use and how to set it using the "salt-formula-linux" formula which seems to be a prerequisite for this formula to work.

[fpytloun]

Formulas are not supposed to do what they are not supposed to. Eg. to setup repositories properly, there's salt-formula-linux.
But you are right that documentation should mention this + testing pillar should be fixed.
Will you update this pull request?