Diffusion Scap 7367ab8ae26e

Scap config diff

Authored by mmodell on Tue, Apr 18, 1:53 AM.

Description

Scap config diff

Summary:
This changes the config_deploy method in deploy-local as follows:

Write configs to a temp location, then output the diff of the new
config against the pre-existing config, recursively.

This only happens if you are deploying a config for a revision that
already existed.

I can pretty easily extend this to support diffing against the
previously deployed revision. I can also provide a separate 'stage'
to do the config diff and prompt prior to continuing.

Not sure how this should be handled. @mobrovac: what do you think?

Test Plan:
Deploy a revision, then change config and deploy again, using
--force to make the config deploy happen.
You should see the diff in the verbose output of deploy-log

Reviewers: thcipriani, demon, mobrovac, Release-Engineering-Team

Subscribers: mobrovac

Tags: Release-Engineering-Team

Differential Revision: https://phabricator.wikimedia.org/D620

Details

Committed
mmodellThu, Apr 20, 6:55 PM
Pushed
mmodellThu, Apr 20, 6:55 PM
Parents
rMSCA0af3aa38208d: Announce environment along with commit
Branches
Unknown
Tags
Unknown
ChangeId
None
This commit has been deleted in the repository: it is no longer reachable from any branch, tag, or ref.