Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature: diff support for package managed files #21769

Merged
merged 11 commits into from
Mar 18, 2015

Conversation

isbm
Copy link
Contributor

@isbm isbm commented Mar 18, 2015

This feature allows to find out the following:

  1. File(s) is unchanged and is as same as vendor supported (in original, latest available package).
  2. File(s) is not managed by anything (does not belongs to any package)
  3. Requested file(s) is not installed on current system.
  4. File(s) is changed and diff from the original file is provided.

An example:

# salt '*' pkg.diff /etc/nsswitch.conf /etc/passwd /etc/sudoers
e162.suse.de:
    ----------
    /etc/nsswitch.conf:
        Unchanged
    /etc/passwd:
        Not managed
    /etc/sudoers:
        --- A /etc/sudoers
        +++ B /etc/sudoers
        @@ -71,7 +71,7 @@
         ## User privilege specification
         ##
         root ALL=(ALL) ALL
        -
        +bofh ALL=(ALL) NOPASSWD: ALL
         ## Uncomment to allow members of group wheel to execute any command
         # %wheel ALL=(ALL) ALL

@thatch45, YUM is coming soon, I don't have CentOS at the moment under my hand. Their YUM needs plugins installed, so there should be additional checks for this.

@thatch45
Copy link
Contributor

Thanks @isbm ! Sorry I have not replied to your email about this yet. I am buried right now :)

thatch45 added a commit that referenced this pull request Mar 18, 2015
Feature: diff support for package managed files
@thatch45 thatch45 merged commit 77a9fcb into saltstack:develop Mar 18, 2015
thatch45 added a commit that referenced this pull request Mar 18, 2015
@isbm isbm deleted the isbm-zypper-diff branch March 19, 2015 17:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants