diff --git a/builder/osc/bsusurrogate/builder.go b/builder/osc/bsusurrogate/builder.go index 68f9cd0da..886d728aa 100644 --- a/builder/osc/bsusurrogate/builder.go +++ b/builder/osc/bsusurrogate/builder.go @@ -1,12 +1,15 @@ // The bsusurrogate package contains a packer.Builder implementation that -// builds a new EBS-backed AMI using an ephemeral instance. +// builds a new EBS-backed OMI using an ephemeral instance. package bsusurrogate import ( + "errors" + "fmt" "log" osccommon "github.com/hashicorp/packer/builder/osc/common" "github.com/hashicorp/packer/common" + "github.com/hashicorp/packer/helper/config" "github.com/hashicorp/packer/helper/multistep" "github.com/hashicorp/packer/packer" "github.com/hashicorp/packer/template/interpolate" @@ -33,8 +36,69 @@ type Builder struct { } func (b *Builder) Prepare(raws ...interface{}) ([]string, error) { - log.Println("Preparing Outscale Builder...") + b.config.ctx.Funcs = osccommon.TemplateFuncs + + err := config.Decode(&b.config, &config.DecodeOpts{ + Interpolate: true, + InterpolateContext: &b.config.ctx, + InterpolateFilter: &interpolate.RenderFilter{ + Exclude: []string{ + "omi_description", + "run_tags", + "run_volume_tags", + "snapshot_tags", + "spot_tags", + "tags", + }, + }, + }, raws...) + if err != nil { + return nil, err + } + + if b.config.PackerConfig.PackerForce { + b.config.OMIForceDeregister = true + } + + // Accumulate any errors + var errs *packer.MultiError + errs = packer.MultiErrorAppend(errs, b.config.AccessConfig.Prepare(&b.config.ctx)...) + errs = packer.MultiErrorAppend(errs, b.config.RunConfig.Prepare(&b.config.ctx)...) + errs = packer.MultiErrorAppend(errs, + b.config.OMIConfig.Prepare(&b.config.AccessConfig, &b.config.ctx)...) + errs = packer.MultiErrorAppend(errs, b.config.BlockDevices.Prepare(&b.config.ctx)...) + errs = packer.MultiErrorAppend(errs, b.config.RootDevice.Prepare(&b.config.ctx)...) + + if b.config.OMIVirtType == "" { + errs = packer.MultiErrorAppend(errs, errors.New("ami_virtualization_type is required.")) + } + + foundRootVolume := false + for _, launchDevice := range b.config.BlockDevices.LaunchMappings { + if launchDevice.DeviceName == b.config.RootDevice.SourceDeviceName { + foundRootVolume = true + } + } + + if !foundRootVolume { + errs = packer.MultiErrorAppend(errs, fmt.Errorf("no volume with name '%s' is found", b.config.RootDevice.SourceDeviceName)) + } + + //TODO: Chek if this is necessary + if b.config.IsSpotVm() && ((b.config.OMIENASupport != nil && *b.config.OMIENASupport) || b.config.OMISriovNetSupport) { + errs = packer.MultiErrorAppend(errs, + fmt.Errorf("Spot instances do not support modification, which is required "+ + "when either `ena_support` or `sriov_support` are set. Please ensure "+ + "you use an OMI that already has either SR-IOV or ENA enabled.")) + } + + if errs != nil && len(errs.Errors) > 0 { + return nil, errs + } + + packer.LogSecretFilter.Set(b.config.AccessKey, b.config.SecretKey, b.config.Token) return nil, nil + } func (b *Builder) Run(ui packer.Ui, hook packer.Hook, cache packer.Cache) (packer.Artifact, error) { diff --git a/builder/osc/common/template_funcs.go b/builder/osc/common/template_funcs.go index 1f63f661f..1b574f75b 100644 --- a/builder/osc/common/template_funcs.go +++ b/builder/osc/common/template_funcs.go @@ -1,6 +1,9 @@ package common -import "bytes" +import ( + "bytes" + "html/template" +) func isalphanumeric(b byte) bool { if '0' <= b && b <= '9' { @@ -28,3 +31,7 @@ func templateCleanOMIName(s string) string { } return string(newb[:]) } + +var TemplateFuncs = template.FuncMap{ + "clean_omi_name": templateCleanOMIName, +}