Page MenuHomeDevCentral

Standardize containers pillar
ClosedPublic

Authored by dereckson on Sep 11 2018, 13:07.
Tags
None
Referenced Files
F2844034: D1720.id4376.diff
Tue, Apr 23, 12:04
F2843921: D1720.id4374.diff
Tue, Apr 23, 10:42
Unknown Object (File)
Tue, Apr 23, 02:13
Unknown Object (File)
Mon, Apr 22, 22:41
Unknown Object (File)
Mon, Apr 22, 22:05
Unknown Object (File)
Mon, Apr 22, 22:01
Unknown Object (File)
Sun, Apr 21, 21:33
Unknown Object (File)
Fri, Apr 19, 12:50
Subscribers
None

Details

Summary

The docker_containers dictionnary contains two kind of items:

  • containers, with several instance, one key per instance
  • unique container with one assocaited instance

While it currently works, this setup makes complex to parse the entry
without knowledge of what key contains what.

An example of an issue is how to provide nginx configuration for
the containers with several instances? That would require two for loop
one to find containers with Host key at first level, a second to seek
Host key at second level (and parse wastefully properties of unique
containers).

This change standardizes the entries to use the hierarchy model
and provides documentation about the entry structure and how it
matches roles/ states and files.

A drawback is it repeats the service name as instance name for
containers deployed only once like Cachet, but that also provides
more flexibility if in the future we need to run another instance
of the same service for someone else.

Ref T1422

Test Plan

Apply nginx and containers configuration to Equatower

Diff Detail

Repository
rOPS Nasqueron Operations
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dereckson created this revision.

Refactor nginx configuration states for the new topology. Fix cachet instance name.

Disambig double containers variable use

This revision is now accepted and ready to land.Sep 11 2018, 13:40
This revision was automatically updated to reflect the committed changes.