Merge pull request #10978 from hashicorp/azr_invalid_prov_pause_before
Fix Invalid provprovisioner pause_before panic
This commit is contained in:
commit
69705f3e40
|
@ -0,0 +1,12 @@
|
||||||
|
source "null" "example1" {
|
||||||
|
communicator = "none"
|
||||||
|
}
|
||||||
|
|
||||||
|
build {
|
||||||
|
sources = ["source.null.example1"]
|
||||||
|
|
||||||
|
provisioner "shell-local" {
|
||||||
|
pause_before = "5"
|
||||||
|
inline = ["echo Did I wait a bit?"]
|
||||||
|
}
|
||||||
|
}
|
|
@ -22,6 +22,8 @@ func TestValidateCommand(t *testing.T) {
|
||||||
{path: filepath.Join(testFixture("validate"), "null_var.json"), 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: 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
|
// wrong version fails
|
||||||
{path: filepath.Join(testFixture("version_req", "base_failure")), exitCode: 1},
|
{path: filepath.Join(testFixture("version_req", "base_failure")), exitCode: 1},
|
||||||
{path: filepath.Join(testFixture("version_req", "base_success")), exitCode: 0},
|
{path: filepath.Join(testFixture("version_req", "base_success")), exitCode: 0},
|
||||||
|
|
|
@ -120,6 +120,7 @@ func (p *Parser) decodeProvisioner(block *hcl.Block, cfg *PackerConfig) (*Provis
|
||||||
if err != nil {
|
if err != nil {
|
||||||
return nil, append(diags, &hcl.Diagnostic{
|
return nil, append(diags, &hcl.Diagnostic{
|
||||||
Summary: "Failed to parse pause_before duration",
|
Summary: "Failed to parse pause_before duration",
|
||||||
|
Severity: hcl.DiagError,
|
||||||
Detail: err.Error(),
|
Detail: err.Error(),
|
||||||
})
|
})
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue