Closed sparrc closed 3 years ago
Omit the docker dependency. This is mainly because we have less control over docker outside of AL. it can come from "docker-ce", "docker" or "docker-ee", or even not from deb. so i omit this dependency in packaging;
The debian packaging format allows optional dependencies. You should specify a Docker dependency using something like:
Depends: docker-ce | docker-ee | docker.io | docker
This obviously won't cover the case where it's not installed via a package, but for people going that route, they will have other issues, and other ways of satisfying the dependency.
We should not be leaving critical dependencies unspecified
Summary
Add generic deb packaging (one that is not specific to platform).
Implementation details
Some changes from the ubuntu-trusty packaging:
--lintian-opts --suppress-tags bad-distribution-in-changes-file
to suppress "bad distribution" linter error. This is because "stable" is not a valid distribution name for Ubuntu.docker-ce (>= 17.12.0) | docker-engine (>= 1.6.0) | docker-ee | docker.io
added a post-install script to enable the ecs serviceoverride_dh_auto_install
rule to add the bundled agent container tarball to the cache directoryarm64
as a valid architecture to the control filedh_installsystemd --no-start --no-enable --name=ecs
Testing
Built the deb and tested it on:
Verified that the service runs fine on these platforms.
Description for the changelog
N/A
Licensing
This contribution is under the terms of the Apache 2.0 License: