-
Notifications
You must be signed in to change notification settings - Fork 279
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
Move docker image builds to Travis CI #288
Conversation
- Faster - Multi-Arch works correctly with Docker CI 19.03
Codecov Report
@@ Coverage Diff @@
## master #288 +/- ##
======================================
Coverage 86.2% 86.2%
======================================
Files 49 49
Lines 2894 2894
======================================
Hits 2495 2495
Misses 308 308
Partials 91 91 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Awesome work on this. Few comments, and questions mostly for my own understanding.
Code Climate has analyzed commit b2d3b1e and detected 0 issues on this pull request. The test coverage on the diff in this pull request is 100.0% (50% is the threshold). This pull request will bring the total coverage in the repository to 86.5% (0.0% change). View more on Code Climate. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
@travisgroth would this change the docs for docker at all? https://www.pomerium.io/docs/releases.html#docker |
It should not unless I missed something. The workflow should match existing semantics for the moment. |
- Faster - Multi-Arch works correctly with Docker CI 19.03
Related: #284
It seems like Docker Hub automated builds really don't handle ARM builds correctly. This PR moves image testing as well as publishing into Travis. This should set the stage for moving to multi-arch as well.
Checklist: