search

saltstack / salt

Software to automate the management and configuration of any infrastructure or application at scale. Get access to the Salt software package repository here:
https://repo.saltproject.io/
Apache License 2.0
14.14k stars 5.47k forks source link

augeas.change doesn't support setting empty values #24661

Closed kartiksubbarao closed 9 years ago

kartiksubbarao commented 9 years ago

Currently, augeas.change doesn't work when setting parameters to empty values. For example, this state definition doesn't work:

/etc/ntp.conf:
  augeas.change:
    - lens: ntp.lns
    - context: /files/etc/ntp.conf
    - changes:
      - set server[1]/iburst ""

The problem is the regexp on line 133 of modules/augeas_cfg.py. I'll submit a pull request that fixes this.

jfindlay commented 9 years ago

@kartiksubbarao, feel free to close this issue if you think it is resolved.