search

saltstack-formulas / openssh-formula

http://docs.saltstack.com/en/latest/topics/development/conventions/formulas.html
Other
90 stars 297 forks source link

Fixed supporting multiline banner_string defined in the pillar. #112

Closed tibold closed 7 years ago

tibold commented 7 years ago

Fixed rendering openssh.banner state with multiline banner_string defined in the pillar.

The pillar:

 openssh:                                                             
   banner_string: |                                                   
     Welcome to {{ grains['id'] }}.                                   
     This machine is managed using SaltStack.                         
     Do NOT modify any system settings.                               
     Changes will either get lost or interfere with the machine setup.

The rendering error fixed in this pull request:

salt-master:
    Data failed to compile:
----------
    Rendering SLS 'base:openssh.banner' failed: could not found expected ':'; line 13

---
[...]
    - name: /etc/ssh/banner

    - contents: |
        Welcome to salt-master.
This machine is managed using SaltStack.
Do NOT modify any system settings.    <======================
Changes will either get lost or interfere with the machine setup.
aboe76 commented 7 years ago

thanks for the fix @tibold