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

Add support for remote profiles #51

Merged
merged 3 commits into from
Feb 29, 2016
Merged

Conversation

chris-rock
Copy link
Collaborator

This adds supports for remote profiles in Kitchen-InSpec. Just add the inspec_tests section to .kitchen.yml and those profiles will be executed in addition to the tests located in test/integration

suites:
  - name: profile
    run_list:
      - recipe[os_prepare]
    inspec_tests:
      - https://github.com/nathenharvey/tmp_compliance_profile

@chris-rock chris-rock force-pushed the chris-rock/remote-inspec-tests branch from c5b0025 to 0b841b9 Compare February 18, 2016 21:28
@chris-rock chris-rock changed the title add support for remote profiles WIP: add support for remote profiles Feb 19, 2016
@chris-rock chris-rock force-pushed the chris-rock/remote-inspec-tests branch from 0b841b9 to e60b7ca Compare February 28, 2016 17:45
@chris-rock chris-rock changed the title WIP: add support for remote profiles Add support for remote profiles Feb 28, 2016
@chris-rock
Copy link
Collaborator Author

This is the proposed final implementation for allowing users to run remote profiles as part of a test-kitchen run. To achieve this, this PRs manipulates test-kitchen to be able to parse the inspec_tests from the suites yaml options. It prepends DataMunger::verifier_data_for with the goal to parse the data properly.

@chris-rock
Copy link
Collaborator Author

Instead of the Ruby 2.0 way to patch a method, I stored the referenced to the old method and patched the full method, due to the fact that this approach works with Ruby 1.9.3

@chris-rock chris-rock force-pushed the chris-rock/remote-inspec-tests branch from 7ad4795 to f4a82dc Compare February 29, 2016 10:46
@arlimus
Copy link
Collaborator

arlimus commented Feb 29, 2016

Great workaround on top of an awesome feature 😁
+1

arlimus added a commit that referenced this pull request Feb 29, 2016
@arlimus arlimus merged commit a95888f into master Feb 29, 2016
@arlimus arlimus deleted the chris-rock/remote-inspec-tests branch February 29, 2016 11:35
@tas50 tas50 added Type: Enhancement Adds new functionality. and removed enhancement labels Jan 14, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Enhancement Adds new functionality.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants