misheska / basebox-packer

Packer templates for vagrant baseboxes
286 stars 80 forks source link

Error on running "packer build -var "provisioner=chef" ubuntu1310.json" within corresponding template folder #25

Closed josefglatz closed 10 years ago

josefglatz commented 10 years ago

Everytime I want to install chef latest provisioner, it fails.

==> Some builds didn't complete successfully and had errors:
--> vmware: Script exited with non-zero exit status: 1
--> virtualbox: Script exited with non-zero exit status: 1
misheska commented 10 years ago

You also need to provide a version, so the correct command line should be: packer build -var "provisioner=chef" -var "provisioner_version=latest" ubuntu1310.json

I made a blank string the default so there would be no version in the "provisionerless" build, though I'm open to other suggestions on how to do this.

josefglatz commented 10 years ago

Ah ok, sorry. So i will test it later :+1: Thx & sorry...

misheska commented 10 years ago

Not a problem. It is not clear that they need to be used in a pair (and my documentation on this is way down at the bottom of the README). The idea is if someone wants to add, say a puppet or some other provisioner that needs a node footprint can add it there (though my only need is for Chef).

I think I will break apart my README into two files, one for the index of downloadable baseboxes and another for the actual README, as that list of baseboxes is really long and most people wouldn’t think that there is actual documentation at the end of that list ;-)

On Dec 22, 2013, at 8:07 AM, Josef Florian Glatz notifications@github.com wrote:

Ah ok, sorry. So i will test it later Thx & sorry...

— Reply to this email directly or view it on GitHub.

josefglatz commented 10 years ago

I would suggest to insert "latest" as default if no version parameter is given.

Am 22.12.2013 um 18:23 schrieb Mischa Taylor notifications@github.com:

Not a problem. It is not clear that they need to be used in a pair (and my documentation on this is way down at the bottom of the README). The idea is if someone wants to add, say a puppet or some other provisioner that needs a node footprint can add it there (though my only need is for Chef).

I think I will break apart my README into two files, one for the index of downloadable baseboxes and another for the actual README, as that list of baseboxes is really long and most people wouldn’t think that there is actual documentation at the end of that list ;-)

On Dec 22, 2013, at 8:07 AM, Josef Florian Glatz notifications@github.com wrote:

Ah ok, sorry. So i will test it later Thx & sorry...

— Reply to this email directly or view it on GitHub.

— Reply to this email directly or view it on GitHub.

misheska commented 10 years ago

I agree, but I don’t see a good way to do conditional variables in a Packer template definition. As far as I can tell, either we can have a “provisionerless” without “latest” OR “provisioner” with “latest”. That is my conundrum.

On Dec 22, 2013, at 9:35 AM, Josef Florian Glatz notifications@github.com wrote:

I would suggest to insert "latest" as default if no version parameter is given.

Am 22.12.2013 um 18:23 schrieb Mischa Taylor notifications@github.com:

Not a problem. It is not clear that they need to be used in a pair (and my documentation on this is way down at the bottom of the README). The idea is if someone wants to add, say a puppet or some other provisioner that needs a node footprint can add it there (though my only need is for Chef).

I think I will break apart my README into two files, one for the index of downloadable baseboxes and another for the actual README, as that list of baseboxes is really long and most people wouldn’t think that there is actual documentation at the end of that list ;-)

On Dec 22, 2013, at 8:07 AM, Josef Florian Glatz notifications@github.com wrote:

Ah ok, sorry. So i will test it later Thx & sorry...

— Reply to this email directly or view it on GitHub.

— Reply to this email directly or view it on GitHub.

— Reply to this email directly or view it on GitHub.