Declare missing shellserver role units to top state
ClosedPublic
Actions

Authored by dereckson on Oct 29 2017, 16:22.

Details

Reviewers

dereckson

Commits

rOPS2ed01c9ec1c6: Declare missing shellserver role units to top state

Summary

Database and Bonjour chaton units were missing
and ignored by high state.

Test Plan

Check the two states are currently fine:
salt eglide state.apply roles/shellserver/database test=True
salt eglide state.apply roles/shellserver/bonjour-chaton test=True

Diff Detail

Repository

rOPS Nasqueron Operations

Lint

Lint Passed

Unit

No Test Coverage

Branch

bonjour-chaton

Build Status

Buildable 1774
Build 2022: arc lint + arc unit

Event Timeline

dereckson created this revision.Oct 29 2017, 16:22

Harbormaster completed remote builds in B1774: Diff 2878.Oct 29 2017, 16:22

When highstate runs, it propagates the new SSL certificate for Bonjour chaton:

     ID: bonjour_chaton_certificates_public                               
Function: cmd.run                  
    Name: cp /etc/letsencrypt/live/robot.paysannerebelle.com/fullchain.pem /opt/bonjour-chaton/certs/cert.pem                                              
  Result: True                     
 Comment: Command "cp /etc/letsencrypt/live/robot.paysannerebelle.com/fullchain.pem /opt/bonjour-chaton/certs/cert.pem" run                                     Started: 16:23:09.349093          
Duration: 58.266 ms                
 Changes:                          
          ----------               
          pid:                     
              1333                 
          retcode:                 
              0                    
          stderr:                  
          stdout:

This is a desirable behavior to ensure the certificate is always accurate (if not, the bot would use expired one).

The remaining states are up to date.

This revision is now accepted and ready to land.Oct 29 2017, 16:24

Closed by commit rOPS2ed01c9ec1c6: Declare missing shellserver role units to top state (authored by dereckson). · Explain WhyOct 29 2017, 16:24

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

top.sls

2 lines

Diff 2878

View Options

Declare missing shellserver role units to top stateClosedPublicActions