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 Passed
Unit
No Test Coverage
Branch
consolidate-nginx
Build Status
Buildable 2233
Build 2481: arc lint + arc unit

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.