Differential D1477

Allow a container not to be linked to a nginx configuraiton
ClosedPublic
Actions

Authored by dereckson on Mar 23 2018, 00:30.

Details

Reviewers

dereckson

Commits

rOPS86d6148c6a3a: Allow a container not to be linked to a nginx configuraiton

Summary

Change 4aaa13576f3b was based on the fact every Docker container
is associated to a nginx configuration.

This situation was modified with change ae11316531f9, with Jenkins
slaves not matching a web server.

This change fixes the nginx unit to only try to populate a file
when the host parameter is defined in the pillar container entry.

Test Plan

Tested on Equatower

Diff Detail

Repository

rOPS Nasqueron Operations

Lint

Lint Passed

Unit

No Test Coverage

Branch

fix-containers-nginx

Build Status

Buildable 2332
Build 2580: arc lint + arc unit

Event Timeline

dereckson requested review of this revision.Mar 23 2018, 00:30

dereckson created this revision.

Harbormaster completed remote builds in B2332: Diff 3785.Mar 23 2018, 00:31

dereckson accepted this revision.Mar 23 2018, 00:31

dereckson retitled this revision from Change 4aaa13576f3b was based on the fact every Docker container is associated to a nginx configuration. to Allow a container not to be linked to a nginx configuraiton.

dereckson edited the summary of this revision. (Show Details)

This revision is now accepted and ready to land.Mar 23 2018, 00:31

Closed by commit rOPS86d6148c6a3a: Allow a container not to be linked to a nginx configuraiton (authored by dereckson). · Explain WhyMar 23 2018, 00:31

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

roles/

paas-docker/

nginx/

config.sls

2 lines

Diff 3785

View Options

roles/paas-docker/nginx/config.sls

Allow a container not to be linked to a nginx configuraitonClosedPublicActions