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

The content_diff and show_resource_diff options can't be turned off #22

Open
natemccurdy opened this issue Oct 14, 2015 · 3 comments
Open

Comments

@natemccurdy
Copy link

It seems that --content_diff and --show_resource_diff can't be turned off.

The flags are available to set, but they aren't used in the code anywhere.

This causes problems when creating diff reports with a huge amount of changes as the report can be extremely large and un-processable

@alex-harvey-z3q
Copy link

For what it's worth the --show_resource_diff option works fine for me.

@natemccurdy
Copy link
Author

@alexharv074 Do you mean that, when you don't use the --show_resource_diff flag, you correctly don't see any resource differences?

My issue is not that the flags don't work, but that they can't be turned off.

@alex-harvey-z3q
Copy link

@natemccurdy If I run a diff without this option I'll see something like:

Differences in old:
        Logstash::package::install[logstash]:

        logstash::package::install{"logstash":
             package_url => "http://packages.mydomain.com/prod1/nexus/thirdparty/linux/logstash-1.4.1-1_bd507eb.noarch.rpm"
        }

If I add the --show_resource_diff option this output changes to:

Differences as diff:
        Logstash::package::install[logstash]
        logstash::package::install{"logstash":
-            package_url => "http://packages.mydomain.com/uat1/nexus/thirdparty/linux/logstash-1.4.1-1_bd507eb.noarch.rpm"
+            package_url => "http://packages.mydomain.com/prod1/nexus/thirdparty/linux/logstash-1.4.1-1_bd507eb.noarch.rpm"
        }

If you also add --debug you'll get more information about how it is calling diff -u with --show_resource_diff turned on.

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

No branches or pull requests

2 participants