Page MenuHomeDevCentral

Consolidate nginx installation and common config files
ClosedPublic

Authored by dereckson on Mar 13 2018, 01:16.
Tags
None
Referenced Files
F2742865: D1415.id3608.diff
Thu, Mar 28, 06:40
F2741753: D1415.id3635.diff
Thu, Mar 28, 02:04
F2740306: D1415.id3628.diff
Wed, Mar 27, 21:17
F2740292: D1415.id3632.diff
Wed, Mar 27, 21:15
Unknown Object (File)
Wed, Mar 27, 02:35
Unknown Object (File)
Sat, Mar 23, 02:29
Unknown Object (File)
Fri, Mar 22, 23:51
Unknown Object (File)
Fri, Mar 22, 01:59
Subscribers
None

Details

Summary

Both shellserver and webserver-legacy roles contains states
useful for every server with nginx. Those states are moved to
the webserver-core role.

Let's encrypt certbot client doesn't need to be installed for
Docker engines, as it runs then as a container.

Test Plan
  • salt equatower state.apply roles/webserver-core (Docker, should provision)
  • salt eglide state.apply roles/webserver-core test=True (non Docker, should be no op)

Diff Detail

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

Event Timeline

dereckson created this revision.

When it's not broken, don't fix it?

$ salt equatower state.apply roles/webserver-core test=True
equatower:
----------
          ID: nginx
    Function: pkg.installed
      Result: None
     Comment: The following packages would be installed/updated: nginx
     Started: 02:19:08.583014
    Duration: 935.315 ms
     Changes:
----------
          ID: /usr/bin/list-nginx-vhosts-conf
    Function: file.managed
      Result: None
     Comment: The file /usr/bin/list-nginx-vhosts-conf is set to be changed
     Started: 02:19:09.521603
    Duration: 34.917 ms
     Changes:
----------
          ID: /usr/bin/autochmod
    Function: file.managed
      Result: None
     Comment: The file /usr/bin/autochmod is set to be changed
     Started: 02:19:09.556725
    Duration: 12.39 ms
     Changes:

Summary for equatower
------------
Succeeded: 3 (unchanged=3)
Failed:    0
------------
Total states run:     3
Total run time: 982.622 ms
This revision is now accepted and ready to land.Mar 15 2018, 21:19
This revision was automatically updated to reflect the committed changes.