Page MenuHomeDevCentral

Check docker_container app_port/host
ClosedPublic

Authored by dereckson on Nov 11 2018, 14:15.
Tags
None
Referenced Files
F3786345: D2007.diff
Tue, Nov 26, 14:20
Unknown Object (File)
Wed, Nov 13, 06:02
Unknown Object (File)
Sun, Nov 3, 10:13
Unknown Object (File)
Sun, Nov 3, 10:13
Unknown Object (File)
Fri, Nov 1, 14:20
Unknown Object (File)
Thu, Oct 31, 12:36
Unknown Object (File)
Oct 24 2024, 17:40
Unknown Object (File)
Sep 26 2024, 07:39
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.