202 Commits

Author SHA1 Message Date
Matthew Hooker
fa3e70ef75 builder/amazon/chroot:
Delete files at destination before copy. This should help with the dangling
symbolic link issue we've been seeing with ubuntu.

fixes GH-500
2013-10-17 22:50:02 +00:00
Mitchell Hashimoto
3fad717ba4 builder/amazon,builder/openstack: remove more cgl libs 2013-10-16 16:29:53 -10:00
Mitchell Hashimoto
697c91b0b0 Remove dependency on identifier package, use time ordered UUID [GH-541] 2013-10-16 16:21:14 -10:00
Mitchell Hashimoto
54e1f70a3c Merge pull request #522 from patricklucas/fix_instance_not_found_race
builder/amzon: fix race condition after launching EC2 instance
2013-10-14 01:45:55 -07:00
Matthew Hooker
739fbc8962 common/config: config filter function [GH-521]
Fixes #521
2013-10-13 11:01:12 -07:00
Patrick Lucas
aeb0e8bc06 Fix race condition after launching EC2 instance
It is possible for an instance to not immediately exist after it is
launched. Previously, InstanceStateRefreshFunc would crash if this race
condition were realized.

This change takes the exact same approach of the function above,
AMIStateRefreshFunc, treating 'InvalidInstanceID.NotFound' as if there
were an empty result.
2013-10-11 14:25:18 -07:00
Matthew Hooker
2af19060ce Fix the way we run commands in chroot.
* Single quotes around chroot command cause it to fail.
* The chroot command also gets passed through the CommandWrapper template, so having sudo would clash with however the user wants to run it.
* Fix spelling mistake.
2013-10-01 09:35:55 +00:00
Mitchell Hashimoto
a793e03fb6 builder/amazon/chroot: remove unused comment 2013-09-30 09:35:58 -07:00
Mitchell Hashimoto
c84dd13539 builder/amazon/chroot: CommandWrapper
/cc @mwhooker - I changed the interface up a bit to return an error,
since things should return errors in Go (the ui.Error bit was kind of
ghetto because it had no way to bubble that error up except through the
UI).

Using this, I made it so that the communicator uses both a
CommandWrapper and ShellCommand with chroot so that the chroot commannd
is also wrapped (it wasn't before).

I think the functionality of all this is the same but I'd love if you
could look it over and make sure.
2013-09-30 09:33:57 -07:00
Mitchell Hashimoto
5b7c745c3c builder/amazon/chroot: alphebatize the configs
/cc @mwhooker
2013-09-30 09:32:20 -07:00
Matthew Hooker
6f7aad23f0 comment, formatting. 2013-09-29 08:04:57 +00:00
Matthew Hooker
a84c74318c WIP copying files. 2013-09-28 01:10:33 +00:00
Matthew Hooker
837e641802 move wrapper definitions around. 2013-09-27 22:08:15 +00:00
Matthew Hooker
8857358830 building but there's an exec error. 2013-09-27 20:47:44 +00:00
Matthew Hooker
40e15c84ef fixing up types. 2013-09-27 11:55:19 +00:00
Matthew Hooker
299ee6efd2 replace command config with wrapper command. 2013-09-27 10:54:53 +00:00
Matthew Hooker
93c2de0af3 fix chroot Upload command. 2013-09-27 02:11:28 +00:00
Matthew Hooker
ccd9b56f9c wip 2013-09-26 18:34:01 -07:00
Matthew Hooker
6d158eaa29 bugfixes, wip 2013-09-27 01:28:06 +00:00
Matthew Hooker
308b148bb4 get chroot command from proper place. 2013-09-26 16:39:37 -07:00
Matthew Hooker
6d4b834f61 Merge remote-tracking branch 'origin/master' into 430 2013-09-26 16:24:28 -07:00
Matthew Hooker
75d2d769a9 We're not using this log package. 2013-09-26 02:31:10 -07:00
Matthew Hooker
7e2b9de778 Unit tests. 2013-09-26 02:25:57 -07:00
Matthew Hooker
74adaa534e bugfix. 2013-09-26 01:35:29 -07:00
Matthew Hooker
da467b7d67 using blocking cp method. 2013-09-26 01:32:53 -07:00
Matthew Hooker
0659ccdea0 Revert "notes/reorg."
This reverts commit 73c5aec24d0a504ecfbae038b4b7effb54adb929.

Conflicts:
	builder/amazon/chroot/step_copy_files.go
2013-09-26 01:16:51 -07:00
Matthew Hooker
95f5cea285 wip 2013-09-26 01:08:06 -07:00
Matthew Hooker
03994f052d Merge branch 'master' into 430 2013-09-26 01:04:42 -07:00
Matthew Hooker
a03807f8e8 wip 2013-09-26 00:58:25 -07:00
Matthew Hooker
73c5aec24d notes/reorg. 2013-09-26 00:31:07 -07:00
Matthew Hooker
cd0cae207c pass chroot command to /bin/sh
this allows us to specify an arbitrary chroot command, i.e. for sudo
2013-09-25 03:20:22 -07:00
Bob Potter
db84c9f547 Update builder-amazon-chroot to work with WaitForState changes in 7c56148f 2013-09-24 23:44:03 -05:00
Mitchell Hashimoto
f8ff737b0a builder/amazon/common: save access/secret key from env [GH-434] 2013-09-18 13:59:23 -07:00
Jesse Nelson
0fadb8d02f exclude everything in /tmp, but keep /tmp itself
There is IMO a bug with image builder that it removes /tmp, in the
current setup. This patch makes the image bundle ignore everyting in
/tmp, but keeps /tmp on the box.
2013-09-18 13:42:15 -07:00
Matthew Hooker
bac3d8ffed Allow user to override chroot command. 2013-09-17 05:58:16 -07:00
Mitchell Hashimoto
7c56148f95 builder/amazon/common: error if WaitForState can't find resource
repeatably
2013-09-12 20:37:14 -07:00
Mitchell Hashimoto
f9feeac355 builder/amazon/common: rename instance.go to state.go, makes more sense 2013-09-12 20:33:58 -07:00
Mitchell Hashimoto
6a8e4f1439 builder/amazon/*: use WaitForState for AMIs 2013-09-12 20:33:32 -07:00
Mark Peek
9035b51165 builder/amazon: add clean_ami_name template function
Add a clean_ami_name template function which will translate illegal
characters in an AMI name to '-'. Example usage would be:
    "ami_name": "Ubuntu 12.04 {{isotime | clean_ami_name}}"
2013-09-07 18:42:50 -07:00
Mitchell Hashimoto
8c30dc78ac builder/amazon/common: don't need squash here 2013-09-06 11:48:38 -07:00
Mitchell Hashimoto
c4cf8553c8 builder/amazon/common: support NoDevice [GH-396] 2013-09-06 11:44:57 -07:00
Mitchell Hashimoto
67c71aa836 builder/amazon/common: if instance query returns none, ignore 2013-09-05 17:19:23 -07:00
Mitchell Hashimoto
5736dbd3aa Merge pull request #388 from jmassara/ami_region_copy
builder/amazon/all: AMI region copy fixes
2013-09-05 17:14:16 -07:00
Mitchell Hashimoto
6746ffc133 builder/amazon/all: TemporaryKeyPairName
/cc @whostolebenfrog - I actually simplified things quite a bit. I added
a "uuid" global template function so it just uses that now. I renamed it
so that it is clear it is a temporary keypair.
2013-09-05 12:23:08 -07:00
Ben Griffiths
0058929186 Allow customization of SSH Key pair pattern
Defined in a template using ssh_keypair_pattern. Defaults to "packer %s"
2013-09-05 13:28:31 +01:00
James Massara
952b2c921f Also deregister AMI if it was copied to other regions 2013-09-04 17:48:00 -07:00
James Massara
a613446205 builder/amazon: Re-ordered steps for AMI region copying so tags and
launch permissions are also applied to copied AMIs
2013-09-04 16:06:06 -07:00
Mitchell Hashimoto
8e7c2796fc builder/amazon/instance: new multistep API 2013-08-31 13:03:13 -07:00
Mitchell Hashimoto
b5606af9e2 builder/amazon/ebs: new multistep API 2013-08-31 13:00:43 -07:00
Mitchell Hashimoto
1b8646c963 builder/amazon/chroot: new multistep API 2013-08-31 12:58:55 -07:00