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

inspec exec format json backtrace #614

Closed
alexpop opened this issue Apr 5, 2016 · 2 comments
Closed

inspec exec format json backtrace #614

alexpop opened this issue Apr 5, 2016 · 2 comments
Labels
Type: Enhancement Improves an existing feature

Comments

@alexpop
Copy link
Contributor

alexpop commented Apr 5, 2016

I've executed the following control:

control "one-control" do
  impact 1.0
  describe mount("/tmp") do
    it { should be_mounted  }
  end
end

with the --format json options and received the following output: output.json.txt

This is the character count of the content with and without the backtrace content:

$ wc -m /tmp/output_w_backtrace.json
    4981 /tmp/output_w_backtrace.json

$ wc -m /tmp/output_wo_backtrace.json
     636 /tmp/output_wo_backtrace.json

This enhancement request is to explore when to provide the backtrace output. For example have it in --format fulljson, but not have it in --format json

@alexpop alexpop added the Type: Enhancement Improves an existing feature label Apr 5, 2016
@arlimus
Copy link
Contributor

arlimus commented Apr 6, 2016

👍 interesting, with --controls and that whole fulljson piece merged, we are starting to see more of these filter-that-json problems, or also: jq-soluble 😁

I'm not sure yet about the correct way...

that --format json formatter is still 100% rspec-oriented, with just 2 extra fields that don't hurt. but in the end: fulljson is the more complete one. idea: we might opt to rebrand our json to rspecjson and make sure we create a real minimal json as the alternative; something that 100% complements our json profiles without any overhead. we would then leave rspecjson as it is (i.e. vanilla) and just implement a full and minimal json.

@chris-rock
Copy link
Contributor

solved with #671

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Enhancement Improves an existing feature
Projects
None yet
Development

No branches or pull requests

3 participants