Page MenuHomeDevCentral

Check docker_container app_port/host
ClosedPublic

Authored by dereckson on Nov 11 2018, 14:15.

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
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

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