Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 29370632: NoIssue - Refactor ensure_*state functions (Closed)

Created:
Jan. 3, 2017, 2:25 a.m. by f.lopez
Modified:
Jan. 4, 2017, 9:08 p.m.
Reviewers:
Fred, f.nicolaisen
CC:
mathias
Visibility:
Public.

Description

NoIssue - Refactor ensure_*state functions Please push this as matze since he is on vacation and this has been tested several times over the last weeks!!

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+81 lines, -14 lines) Patch
A modules/adblockplus/lib/puppet/parser/functions/ensure_directory_state.rb View 1 chunk +13 lines, -0 lines 0 comments Download
A modules/adblockplus/lib/puppet/parser/functions/ensure_file_state.rb View 1 chunk +13 lines, -0 lines 0 comments Download
A modules/adblockplus/lib/puppet/parser/functions/ensure_state.rb View 1 chunk +37 lines, -0 lines 0 comments Download
A modules/adblockplus/lib/puppet/parser/functions/ensure_symlink_state.rb View 1 chunk +13 lines, -0 lines 0 comments Download
M modules/adblockplus/manifests/host.pp View 1 chunk +1 line, -1 line 0 comments Download
M modules/adblockplus/manifests/puppet.pp View 2 chunks +2 lines, -11 lines 0 comments Download
M modules/adblockplus/manifests/puppet/agent.pp View 1 chunk +1 line, -1 line 0 comments Download
M modules/adblockplus/manifests/puppet/master.pp View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 3
f.lopez
Jan. 3, 2017, 2:25 a.m. (2017-01-03 02:25:46 UTC) #1
Fred
LGTM
Jan. 3, 2017, 4:31 p.m. (2017-01-03 16:31:43 UTC) #2
f.nicolaisen
Jan. 4, 2017, 5:28 p.m. (2017-01-04 17:28:45 UTC) #3
LGTM

Powered by Google App Engine
This is Rietveld