Fix Salt file.managed state without content replacement
ClosedPublic
Actions

Authored by dereckson on Feb 16 2018, 20:51.

Details

Reviewers

dereckson

Commits

rOPS1ab6a542345e: Fix Salt file.managed state without content replacement

Summary

Suppress the following warning:

[WARNING ] State for file: /srv/api/data/servers-log-all.json - Neither
'source' nor 'contents' nor 'contents_pillar' nor 'contents_grains'
was defined, yet 'replace' was set to 'True'. As there is no source to
replace the file with, 'replace' has been set to 'False' to avoid reading
the file unnecessarily.

This is a follow-up for 87580901de7f.

Test Plan

salt-call --local state.apply roles/webserver-content/org/nasqueron/api test=True

Diff Detail

Repository

rOPS Nasqueron Operations

Lint

Lint Passed

Unit

No Test Coverage

Branch

fix-D1310 (branched from master)

Build Status

Buildable 2078
Build 2326: arc lint + arc unit

Event Timeline

dereckson requested review of this revision.Feb 16 2018, 20:51

dereckson created this revision.

dereckson accepted this revision.Feb 16 2018, 20:57

dereckson edited the summary of this revision. (Show Details)

dereckson edited the test plan for this revision. (Show Details)

This revision is now accepted and ready to land.Feb 16 2018, 20:57

Closed by commit rOPS1ab6a542345e: Fix Salt file.managed state without content replacement (authored by dereckson). · Explain WhyFeb 16 2018, 21:00

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

roles/

webserver-content/

org/

nasqueron/

api.sls

1 line

Diff 3372

View Options

roles/webserver-content/org/nasqueron/api.sls

Fix Salt file.managed state without content replacementClosedPublicActions