Page MenuHomeDevCentral

Check docker_container app_port/host
ClosedPublic

Authored by dereckson on Nov 11 2018, 14:15.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Mar 27, 02:23
Unknown Object (File)
Sat, Mar 23, 21:50
Unknown Object (File)
Fri, Mar 22, 03:35
Unknown Object (File)
Thu, Mar 21, 05:33
Unknown Object (File)
Thu, Mar 21, 05:32
Unknown Object (File)
Mon, Mar 18, 18:46
Unknown Object (File)
Sun, Mar 17, 00:35
Unknown Object (File)
Tue, Mar 12, 23:08
Subscribers
None

Details

Summary

If an entry under docker_containers has host defined, it needs also
to have app_port defined. The host key is used by nginx to determine
if a web configuration should be generated for this container.

This change offers an unit test to validate the pillar configuration,
and fixes the issue for exim container introduced in 3c52ac6a.

Test Plan

New config test provided

Diff Detail

Repository
rOPS Nasqueron Operations
Lint
Lint Passed
Unit
No Test Coverage
Branch
cleanup-docker-paas-app-port
Build Status
Buildable 3121
Build 3369: arc lint + arc unit

Event Timeline

dereckson created this revision.
This revision is now accepted and ready to land.Nov 11 2018, 14:52
This revision was automatically updated to reflect the committed changes.