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 require_chef_omnibus driver option #4

Merged
merged 1 commit into from
Feb 19, 2013

Conversation

schisamo
Copy link
Contributor

This option works just like it does in kitchen-ec2. It ensures the
desired version of Chef is installed via the official Opscode Omnibus
client package.

The main reason for adding this is to allow projects to use the official
Canonical published Vagrant base boxes:

http://uec-images.ubuntu.com/vagrant/

These base boxes do not ship with Chef pre-installed.

This change has been verified as working with the following .kitchen.yml:

driver_plugin: vagrant
platforms:
- name: ubuntu-12.10
  driver_config:
    box: canonical-ubuntu-12.10
    box_url: http://uec-images.ubuntu.com/vagrant/quantal/current/quantal-server-cloudimg-amd64-vagrant-disk1.box
    require_chef_omnibus: 11.4.0
  run_list:
  - recipe[apt]
- name: ubuntu-12.04
  driver_config:
    box: canonical-ubuntu-12.04
    box_url: http://uec-images.ubuntu.com/vagrant/precise/current/precise-server-cloudimg-amd64-vagrant-disk1.box
    require_chef_omnibus: 11.4.0
  run_list:
  - recipe[apt]
suites:
- name: default
  run_list:
  - recipe[hubot]
  attributes: {}

This option works just like it does in `kitchen-ec2`. It ensures the 
desired version of Chef is installed via the official Opscode Omnibus 
client package. 

The main reason for adding this is to allow projects to use the official 
Canonical published Vagrant base boxes: 

http://uec-images.ubuntu.com/vagrant/

These base boxes do not ship with Chef pre-installed.
fnichol added a commit that referenced this pull request Feb 19, 2013
Add support for `require_chef_omnibus` driver option
@fnichol fnichol merged commit a23888b into test-kitchen:master Feb 19, 2013
@fnichol
Copy link
Contributor

fnichol commented Feb 19, 2013

Ah, totally right! I've got a commit inbound for test-kitchen to address test-kitchen/test-kitchen#32 as well

@lxndrp
Copy link

lxndrp commented Jul 30, 2013

It seems that this feature is gone again; I couldn't find any reference to it on the master tree. This is somewhat a pity, because it badly interacts with the current box default which points to provisionerless VMs.

@ghost ghost mentioned this pull request Aug 1, 2013
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

3 participants