mirror of
https://github.com/hashicorp/packer.git
synced 2026-09-18 22:11:39 -04:00
* hcl2template: pass user variable values to plugins as packer_user_variables (#13686) HCL2 builds do not send the packer_user_variables config key to the builder, provisioner and post-processor plugins. Legacy JSON builds send this key (through CoreBuild.packerConfig()). Without the key, the plugin SDK keeps interpolate.Context.UserVariables nil. Then the template function {{ user "name" }} fails with 'error calling user: test'. This failure occurs in each string that a plugin interpolates at run time, for example the contents of the vagrant post-processor's vagrantfile_template. Add the function PackerConfig.userVariableValues(). This function converts the input variable values to strings, equivalent to the legacy user variables. The core sends the map at each plugin handoff point: builder, provisioner, post-processor, and enforced provisioner. The map does not contain the sensitive variables. A sensitive value goes to a plugin only if the template refers to it explicitly. The map does not contain values that are not primitive (lists, maps, objects). Legacy user variables were always strings. Co-authored-by: Claude Fable 5 <[email protected]> * Dependency upgrade (#13689) * go.mod version upgrade * actions upgrade * plugin-getter: install plugins from non-GitHub HTTP sources (#13691) Packer can only install plugins from github.com, with releases.hashicorp.com consulted first for HashiCorp-published plugins. This has been a long-standing gap for air-gapped and policy-restricted environments (#11164): the source address parser already accepts any hostname, but both existing getters reject non-github.com sources at install time. Add a remote plugin getter that installs plugins from the host named in a required_plugins source address. The host serves the directory structure of releases.hashicorp.com under the source's path: an index.json listing versions, and per version a SHA256SUMS file, the zips it lists, and - when the zip names carry no plugin protocol version - the version's manifest.json. A plugin published on releases.hashicorp.com is therefore mirrored as a verbatim copy of its tree, with every checksum file and signature upstream-authored. A plugin published as GitHub release assets is mirrored by copying each release's assets into a version directory, renaming their SHA256SUMS file to the unprefixed convention with content unchanged, and writing an index.json listing the versions. Both kinds of content can be served side by side by one host. Getter selection happens per source address: github.com sources keep the release and github getters unchanged, while any other host is served by the remote getter over HTTPS. Sources with three or more components are supported, up to the existing 16-component limit, so nested artifact-repository paths and hosts that embed the upstream origin in their path all resolve. Version discovery, constraint solving, checksum verification, and the binary naming rules match the existing getters. The installed filename is rebuilt from validated checksum-file fields and never taken from the server's response, checksum entries matching neither known naming shape are rejected rather than guessed at, and nothing the remote metadata supplies is used to fetch from another origin or path. index.json parsing is covered by fixtures captured from the live releases API, so a format change there fails tests rather than user installs. Closes #11164 * fix: update link to CONTRIBUTING.md for consistency with main branch * Merge pull request #13701 from hashicorp/sanya-hashicorp/fix-vuln Module Dependency update * build(deps): bump google.golang.org/grpc (#13704) Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.83.1 to 1.83.2. - [Release notes](https://github.com/grpc/grpc-go/releases) - [Commits](https://github.com/grpc/grpc-go/compare/v1.83.1...v1.83.2) --- updated-dependencies: - dependency-name: google.golang.org/grpc dependency-version: 1.83.2 dependency-type: indirect ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * [COMPLIANCE] Add/Update Copyright Headers (#13699) * [COMPLIANCE] Add/Update Copyright Headers * make generate update * make generate --------- Co-authored-by: hashicorp-copywrite[bot] <110428419+hashicorp-copywrite[bot]@users.noreply.github.com> Co-authored-by: sanya <[email protected]> Co-authored-by: Sanya <[email protected]> * Packer release 1.16.1 (#13705) * Module upgrade (#13707) --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: Erik Berg <[email protected]> Co-authored-by: Claude Fable 5 <[email protected]> Co-authored-by: Benjamin Holmes <[email protected]> Co-authored-by: elomito <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: hashicorp-copywrite[bot] <110428419+hashicorp-copywrite[bot]@users.noreply.github.com>
390 lines
11 KiB
Go
390 lines
11 KiB
Go
// Copyright IBM Corp. 2024, 2026
|
|
// SPDX-License-Identifier: BUSL-1.1
|
|
|
|
package command
|
|
|
|
import (
|
|
"fmt"
|
|
"path/filepath"
|
|
"testing"
|
|
|
|
"github.com/google/go-cmp/cmp"
|
|
packersdk "github.com/hashicorp/packer-plugin-sdk/packer"
|
|
"github.com/hashicorp/packer/packer"
|
|
)
|
|
|
|
func TestValidateCommand(t *testing.T) {
|
|
tt := []struct {
|
|
path string
|
|
exitCode int
|
|
extraArgs []string
|
|
}{
|
|
{path: filepath.Join(testFixture("validate"), "build.json")},
|
|
{path: filepath.Join(testFixture("validate"), "build.pkr.hcl")},
|
|
{path: filepath.Join(testFixture("validate"), "build_with_vars.pkr.hcl")},
|
|
{path: filepath.Join(testFixture("validate-invalid"), "bad_provisioner.json"), exitCode: 1},
|
|
{path: filepath.Join(testFixture("validate-invalid"), "missing_build_block.pkr.hcl"), exitCode: 1},
|
|
{path: filepath.Join(testFixture("validate"), "null_var.json"), exitCode: 1},
|
|
{path: filepath.Join(testFixture("validate"), "var_foo_with_no_default.pkr.hcl"), exitCode: 1},
|
|
|
|
{path: testFixture("hcl", "validation", "wrong_pause_before.pkr.hcl"), exitCode: 1},
|
|
|
|
// wrong version fails
|
|
{path: filepath.Join(testFixture("version_req", "base_failure")), exitCode: 1},
|
|
{path: filepath.Join(testFixture("version_req", "base_success")), exitCode: 0},
|
|
|
|
// wrong version field
|
|
{path: filepath.Join(testFixture("version_req", "wrong_field_name")), exitCode: 1},
|
|
|
|
// wrong packer block type
|
|
{path: filepath.Join(testFixture("validate", "invalid_block_type.pkr.hcl")), exitCode: 1},
|
|
|
|
// wrong packer block
|
|
{path: filepath.Join(testFixture("validate", "invalid_packer_block.pkr.hcl")), exitCode: 1},
|
|
|
|
// Should return multiple errors,
|
|
{path: filepath.Join(testFixture("validate", "circular_error.pkr.hcl")), exitCode: 1},
|
|
|
|
// datasource could be unknown at that moment
|
|
{path: filepath.Join(testFixture("hcl", "data-source-validation.pkr.hcl")), exitCode: 0},
|
|
|
|
// datasource unknown at validation-time without datasource evaluation -> fail on provisioner
|
|
{path: filepath.Join(testFixture("hcl", "local-ds-validate.pkr.hcl")), exitCode: 1},
|
|
// datasource unknown at validation-time with datasource evaluation -> success
|
|
{path: filepath.Join(testFixture("hcl", "local-ds-validate.pkr.hcl")), exitCode: 0, extraArgs: []string{"--evaluate-datasources"}},
|
|
}
|
|
|
|
for _, tc := range tt {
|
|
t.Run(tc.path, func(t *testing.T) {
|
|
c := &ValidateCommand{
|
|
Meta: TestMetaFile(t),
|
|
}
|
|
tc := tc
|
|
args := tc.extraArgs
|
|
args = append(args, tc.path)
|
|
if code := c.Run(args); code != tc.exitCode {
|
|
fatalCommand(t, c.Meta)
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
func TestValidateCommand_SkipDatasourceExecution(t *testing.T) {
|
|
datasourceMock := &packersdk.MockDatasource{}
|
|
meta := TestMetaFile(t)
|
|
meta.CoreConfig.Components.PluginConfig.DataSources = packer.MapOfDatasource{
|
|
"mock": func() (packersdk.Datasource, error) {
|
|
return datasourceMock, nil
|
|
},
|
|
}
|
|
c := &ValidateCommand{
|
|
Meta: meta,
|
|
}
|
|
args := []string{filepath.Join(testFixture("validate"), "datasource.pkr.hcl")}
|
|
if code := c.Run(args); code != 0 {
|
|
fatalCommand(t, c.Meta)
|
|
}
|
|
if datasourceMock.ExecuteCalled {
|
|
t.Fatalf("Datasource should not be executed on validation")
|
|
}
|
|
if !datasourceMock.OutputSpecCalled {
|
|
t.Fatalf("Datasource OutPutSpec should be called on validation")
|
|
}
|
|
}
|
|
|
|
func TestValidateCommand_SyntaxOnly(t *testing.T) {
|
|
tt := []struct {
|
|
path string
|
|
exitCode int
|
|
}{
|
|
{path: filepath.Join(testFixture("validate"), "build.json")},
|
|
{path: filepath.Join(testFixture("validate"), "build.pkr.hcl")},
|
|
{path: filepath.Join(testFixture("validate"), "build_with_vars.pkr.hcl")},
|
|
{path: filepath.Join(testFixture("validate-invalid"), "bad_provisioner.json")},
|
|
{path: filepath.Join(testFixture("validate-invalid"), "missing_build_block.pkr.hcl")},
|
|
{path: filepath.Join(testFixture("validate-invalid"), "broken.json"), exitCode: 1},
|
|
{path: filepath.Join(testFixture("validate"), "null_var.json")},
|
|
{path: filepath.Join(testFixture("validate"), "var_foo_with_no_default.pkr.hcl")},
|
|
}
|
|
|
|
for _, tc := range tt {
|
|
t.Run(tc.path, func(t *testing.T) {
|
|
c := &ValidateCommand{
|
|
Meta: TestMetaFile(t),
|
|
}
|
|
c.CoreConfig.Version = "102.0.0"
|
|
tc := tc
|
|
args := []string{"-syntax-only", tc.path}
|
|
if code := c.Run(args); code != tc.exitCode {
|
|
fatalCommand(t, c.Meta)
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
func TestValidateCommandOKVersion(t *testing.T) {
|
|
c := &ValidateCommand{
|
|
Meta: TestMetaFile(t),
|
|
}
|
|
args := []string{
|
|
filepath.Join(testFixture("validate"), "template.json"),
|
|
}
|
|
|
|
// This should pass with a valid configuration version
|
|
c.CoreConfig.Version = "102.0.0"
|
|
if code := c.Run(args); code != 0 {
|
|
fatalCommand(t, c.Meta)
|
|
}
|
|
}
|
|
|
|
func TestValidateCommandBadVersion(t *testing.T) {
|
|
c := &ValidateCommand{
|
|
Meta: TestMetaFile(t),
|
|
}
|
|
args := []string{
|
|
filepath.Join(testFixture("validate"), "template.json"),
|
|
}
|
|
|
|
// This should fail with an invalid configuration version
|
|
c.CoreConfig.Version = "100.0.0"
|
|
if code := c.Run(args); code != 1 {
|
|
t.Errorf("Expected exit code 1")
|
|
}
|
|
|
|
stdout, stderr := GetStdoutAndErrFromTestMeta(t, c.Meta)
|
|
expected := `Error:
|
|
|
|
This template requires Packer version 101.0.0 or higher; using 100.0.0
|
|
|
|
|
|
`
|
|
|
|
if diff := cmp.Diff(expected, stderr); diff != "" {
|
|
t.Errorf("Unexpected output: %s", diff)
|
|
}
|
|
t.Log(stdout)
|
|
}
|
|
|
|
func TestValidateCommandExcept(t *testing.T) {
|
|
tt := []struct {
|
|
name string
|
|
args []string
|
|
exitCode int
|
|
}{
|
|
{
|
|
name: "JSON: validate except build and post-processor",
|
|
args: []string{
|
|
"-except=vanilla,pear",
|
|
filepath.Join(testFixture("validate"), "validate_except.json"),
|
|
},
|
|
},
|
|
{
|
|
name: "JSON: fail validate except build and post-processor",
|
|
args: []string{
|
|
"-except=chocolate,apple",
|
|
filepath.Join(testFixture("validate"), "validate_except.json"),
|
|
},
|
|
exitCode: 1,
|
|
},
|
|
{
|
|
name: "HCL2: validate except build and post-processor",
|
|
args: []string{
|
|
"-except=file.vanilla,pear",
|
|
filepath.Join(testFixture("validate"), "validate_except.pkr.hcl"),
|
|
},
|
|
},
|
|
{
|
|
name: "HCL2: fail validation except build and post-processor",
|
|
args: []string{
|
|
"-except=file.chocolate,apple",
|
|
filepath.Join(testFixture("validate"), "validate_except.pkr.hcl"),
|
|
},
|
|
exitCode: 1,
|
|
},
|
|
}
|
|
|
|
c := &ValidateCommand{
|
|
Meta: TestMetaFile(t),
|
|
}
|
|
c.CoreConfig.Version = "102.0.0"
|
|
|
|
for _, tc := range tt {
|
|
t.Run(tc.name, func(t *testing.T) {
|
|
defer cleanup()
|
|
|
|
tc := tc
|
|
if code := c.Run(tc.args); code != tc.exitCode {
|
|
fatalCommand(t, c.Meta)
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
func TestValidateCommand_VarFiles(t *testing.T) {
|
|
tt := []struct {
|
|
name string
|
|
path string
|
|
varfile string
|
|
exitCode int
|
|
}{
|
|
{name: "with basic HCL var-file definition",
|
|
path: filepath.Join(testFixture(filepath.Join("validate", "var-file-tests")), "basic.pkr.hcl"),
|
|
varfile: filepath.Join(testFixture(filepath.Join("validate", "var-file-tests")), "basic.pkrvars.hcl"),
|
|
exitCode: 0,
|
|
},
|
|
{name: "with unused variable in var-file definition",
|
|
path: filepath.Join(testFixture(filepath.Join("validate", "var-file-tests")), "basic.pkr.hcl"),
|
|
varfile: filepath.Join(testFixture(filepath.Join("validate", "var-file-tests")), "undeclared.pkrvars.hcl"),
|
|
exitCode: 0,
|
|
},
|
|
{name: "with unused variable in JSON var-file definition",
|
|
path: filepath.Join(testFixture(filepath.Join("validate", "var-file-tests")), "basic.pkr.hcl"),
|
|
varfile: filepath.Join(testFixture(filepath.Join("validate", "var-file-tests")), "undeclared.json"),
|
|
exitCode: 0,
|
|
},
|
|
}
|
|
for _, tc := range tt {
|
|
t.Run(tc.path, func(t *testing.T) {
|
|
c := &ValidateCommand{
|
|
Meta: TestMetaFile(t),
|
|
}
|
|
tc := tc
|
|
args := []string{"-var-file", tc.varfile, tc.path}
|
|
if code := c.Run(args); code != tc.exitCode {
|
|
fatalCommand(t, c.Meta)
|
|
}
|
|
})
|
|
}
|
|
}
|
|
|
|
func TestValidateCommand_VarFilesWarnOnUndeclared(t *testing.T) {
|
|
tt := []struct {
|
|
name string
|
|
path string
|
|
varfile string
|
|
exitCode int
|
|
}{
|
|
{name: "default warning with unused variable in HCL var-file definition",
|
|
path: filepath.Join(testFixture(filepath.Join("validate", "var-file-tests")), "basic.pkr.hcl"),
|
|
varfile: filepath.Join(testFixture(filepath.Join("validate", "var-file-tests")), "undeclared.pkrvars.hcl"),
|
|
exitCode: 0,
|
|
},
|
|
{name: "default warning with unused variable in JSON var-file definition",
|
|
path: filepath.Join(testFixture(filepath.Join("validate", "var-file-tests")), "basic.pkr.hcl"),
|
|
varfile: filepath.Join(testFixture(filepath.Join("validate", "var-file-tests")), "undeclared.json"),
|
|
exitCode: 0,
|
|
},
|
|
}
|
|
for _, tc := range tt {
|
|
t.Run(tc.path, func(t *testing.T) {
|
|
c := &ValidateCommand{
|
|
Meta: TestMetaFile(t),
|
|
}
|
|
tc := tc
|
|
args := []string{"-var-file", tc.varfile, tc.path}
|
|
if code := c.Run(args); code != tc.exitCode {
|
|
fatalCommand(t, c.Meta)
|
|
}
|
|
|
|
stdout, stderr := GetStdoutAndErrFromTestMeta(t, c.Meta)
|
|
expected := `Warning: Undefined variable
|
|
|
|
The variable "unused" was set but was not declared as an input variable.
|
|
To declare variable "unused" place this block in one of your .pkr.hcl files,
|
|
such as variables.pkr.hcl
|
|
|
|
variable "unused" {
|
|
type = string
|
|
default = null
|
|
}
|
|
|
|
|
|
The configuration is valid.
|
|
`
|
|
if diff := cmp.Diff(expected, stdout); diff != "" {
|
|
t.Errorf("Unexpected output: %s", diff)
|
|
}
|
|
t.Log(stderr)
|
|
})
|
|
}
|
|
}
|
|
|
|
func TestValidateCommand_VarFilesDisableWarnOnUndeclared(t *testing.T) {
|
|
tt := []struct {
|
|
name string
|
|
path string
|
|
varfile string
|
|
exitCode int
|
|
}{
|
|
{name: "no-warn-undeclared-var with unused variable in HCL var-file definition",
|
|
path: filepath.Join(testFixture(filepath.Join("validate", "var-file-tests")), "basic.pkr.hcl"),
|
|
varfile: filepath.Join(testFixture(filepath.Join("validate", "var-file-tests")), "undeclared.pkrvars.hcl"),
|
|
exitCode: 0,
|
|
},
|
|
{name: "no-warn-undeclared-var with unused variable in JSON var-file definition",
|
|
path: filepath.Join(testFixture(filepath.Join("validate", "var-file-tests")), "basic.pkr.hcl"),
|
|
varfile: filepath.Join(testFixture(filepath.Join("validate", "var-file-tests")), "undeclared.json"),
|
|
exitCode: 0,
|
|
},
|
|
}
|
|
for _, tc := range tt {
|
|
t.Run(tc.path, func(t *testing.T) {
|
|
c := &ValidateCommand{
|
|
Meta: TestMetaFile(t),
|
|
}
|
|
tc := tc
|
|
args := []string{"-no-warn-undeclared-var", "-var-file", tc.varfile, tc.path}
|
|
if code := c.Run(args); code != tc.exitCode {
|
|
fatalCommand(t, c.Meta)
|
|
}
|
|
|
|
stdout, stderr := GetStdoutAndErrFromTestMeta(t, c.Meta)
|
|
expected := `The configuration is valid.
|
|
`
|
|
if diff := cmp.Diff(expected, stdout); diff != "" {
|
|
t.Errorf("Unexpected output: %s", diff)
|
|
}
|
|
t.Log(stderr)
|
|
})
|
|
}
|
|
}
|
|
|
|
func TestValidateCommand_ShowLineNumForMissing(t *testing.T) {
|
|
tt := []struct {
|
|
path string
|
|
exitCode int
|
|
extraArgs []string
|
|
}{
|
|
{path: filepath.Join(testFixture("validate-invalid"), "missing_build_block.pkr.hcl"), exitCode: 1},
|
|
}
|
|
|
|
for _, tc := range tt {
|
|
t.Run(tc.path, func(t *testing.T) {
|
|
c := &ValidateCommand{
|
|
Meta: TestMetaFile(t),
|
|
}
|
|
tc := tc
|
|
args := tc.extraArgs
|
|
args = append(args, tc.path)
|
|
if code := c.Run(args); code != tc.exitCode {
|
|
fatalCommand(t, c.Meta)
|
|
}
|
|
|
|
stdout, stderr := GetStdoutAndErrFromTestMeta(t, c.Meta)
|
|
expected := fmt.Sprintf(`Error: Unknown source file.cho
|
|
|
|
on %s line 6:
|
|
(source code not available)
|
|
|
|
Known: [file.chocolate]
|
|
|
|
|
|
`, tc.path)
|
|
if diff := cmp.Diff(expected, stderr); diff != "" {
|
|
t.Errorf("Unexpected output: %s", diff)
|
|
}
|
|
t.Log(stdout)
|
|
})
|
|
}
|
|
}
|