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

Computed defaults #30

Merged
merged 2 commits into from
Jul 23, 2013
Merged

Computed defaults #30

merged 2 commits into from
Jul 23, 2013

Conversation

fnichol
Copy link
Contributor

@fnichol fnichol commented Jul 23, 2013

Allows for computed defaults for a select list of pre-determined
platforms.

Currently a .kitchen.yml file of the following would work:

---
platforms:
  - name: ubuntu-10.04
  - name: ubuntu-12.04
  - name: centos-5.9
  - name: centos-6.4
  - name: debian-7.1.0

suites:
  - name: default
    run_list:
      - recipe[something]

Allows for computed defaults for a select list of pre-determined
platforms.

Currently a .kitchen.yml file of the following would work:

    ---
    platforms:
      - name: ubuntu-10.04
      - name: ubuntu-12.04
      - name: centos-5.9
      - name: centos-6.4
      - name: debian-7.1.0

    suites:
      - name: default
        run_list:
          - recipe[something]
fnichol added a commit that referenced this pull request Jul 23, 2013
@fnichol fnichol merged commit b914e7e into master Jul 23, 2013
@fnichol fnichol deleted the compupted-defaults branch July 23, 2013 03:10
@sethvargo
Copy link
Contributor

@fnichol This is so awesome! Is it feasible to remove the name key completely or would that require too much data manipulation/guessing?

@fnichol
Copy link
Contributor Author

fnichol commented Jul 23, 2013

@sethvargo At the moment that's more complicated as this "Array-of-Hashes" behavior is Test Kitchen core and most other drivers don't (or can't) have good defaults.

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.

None yet

2 participants