From c6dddb75cfa74ea1678213ea89efe9a5228aec05 Mon Sep 17 00:00:00 2001 From: Megan Marsh Date: Fri, 31 Jan 2020 07:45:18 -0800 Subject: [PATCH] clarify artifice postprocessor wording to make it clear that it will not clean up old artifacts (#8676) fix #5451 --- post-processor/artifice/post-processor.go | 2 +- website/source/docs/post-processors/artifice.html.md | 4 ++++ 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/post-processor/artifice/post-processor.go b/post-processor/artifice/post-processor.go index 3ab566dea..361214e1b 100644 --- a/post-processor/artifice/post-processor.go +++ b/post-processor/artifice/post-processor.go @@ -56,7 +56,7 @@ func (p *PostProcessor) Configure(raws ...interface{}) error { func (p *PostProcessor) PostProcess(ctx context.Context, ui packer.Ui, artifact packer.Artifact) (packer.Artifact, bool, bool, error) { if len(artifact.Files()) > 0 { - ui.Say(fmt.Sprintf("Discarding artifact files: %s", strings.Join(artifact.Files(), ", "))) + ui.Say(fmt.Sprintf("Discarding files from artifact: %s", strings.Join(artifact.Files(), ", "))) } artifact, err := NewArtifact(p.config.Files) diff --git a/website/source/docs/post-processors/artifice.html.md b/website/source/docs/post-processors/artifice.html.md index e6bd8e89d..5ceba7e65 100644 --- a/website/source/docs/post-processors/artifice.html.md +++ b/website/source/docs/post-processors/artifice.html.md @@ -33,6 +33,10 @@ stateless build environment for each build on the infrastructure of your choosing. You can use this to build just about anything: buildpacks, containers, jars, binaries, tarballs, msi installers, and more. +Please note that the artifice post-processor will _not_ delete your old artifact +files, even if it removes them from the artifact. If you want to delete the +old artifact files, you can use the shell-local post-processor to do so. + ## Workflow Artifice helps you tie together a few other packer features: