diff --git a/provisioner/salt-masterless/provisioner.go b/provisioner/salt-masterless/provisioner.go index fc4732c60..c31a78631 100644 --- a/provisioner/salt-masterless/provisioner.go +++ b/provisioner/salt-masterless/provisioner.go @@ -27,8 +27,8 @@ type Config struct { // Local path to the salt state tree LocalStateTree string `mapstructure:"local_state_tree"` - // Local path to the salt pillar tree - LocalPillarTree string `mapstructure:"local_pillar_tree"` + // Local path to the salt pillar roots + LocalPillarRoots string `mapstructure:"local_pillar_roots"` // Where files will be copied before moving to the /srv/salt directory TempConfigDir string `mapstructure:"temp_config_dir"` @@ -63,7 +63,7 @@ func (p *Provisioner) Prepare(raws ...interface{}) error { "bootstrap_args": &p.config.BootstrapArgs, "minion_config": &p.config.MinionConfig, "local_state_tree": &p.config.LocalStateTree, - "local_pillar_tree": &p.config.LocalPillarTree, + "local_pillar_roots": &p.config.LocalPillarRoots, "temp_config_dir": &p.config.TempConfigDir, } @@ -83,10 +83,10 @@ func (p *Provisioner) Prepare(raws ...interface{}) error { } } - if p.config.LocalPillarTree != "" { - if _, err := os.Stat(p.config.LocalPillarTree); err != nil { + if p.config.LocalPillarRoots != "" { + if _, err := os.Stat(p.config.LocalPillarRoots); err != nil { errs = packer.MultiErrorAppend(errs, - errors.New("local_pillar_tree must exist and be accessible")) + errors.New("local_pillar_roots must exist and be accessible")) } } @@ -155,7 +155,7 @@ func (p *Provisioner) Provision(ui packer.Ui, comm packer.Communicator) error { return fmt.Errorf("Unable to move %s/states to /srv/salt: %d", p.config.TempConfigDir, err) } - if p.config.LocalPillarTree != "" { + if p.config.LocalPillarRoots != "" { ui.Message(fmt.Sprintf("Creating remote pillar directory: %s/pillar", p.config.TempConfigDir)) cmd := &packer.RemoteCmd{Command: fmt.Sprintf("mkdir -p %s/pillar", p.config.TempConfigDir)} @@ -167,9 +167,9 @@ func (p *Provisioner) Provision(ui packer.Ui, comm packer.Communicator) error { return fmt.Errorf("Error creating remote pillar directory: %s", err) } - ui.Message(fmt.Sprintf("Uploading local pillar tree: %s", p.config.LocalPillarTree)) - if err = UploadLocalDirectory(p.config.LocalPillarTree, fmt.Sprintf("%s/pillar", p.config.TempConfigDir), comm, ui); err != nil { - return fmt.Errorf("Error uploading local pillar tree to remote: %s", err) + ui.Message(fmt.Sprintf("Uploading local pillar roots: %s", p.config.LocalPillarRoots)) + if err = UploadLocalDirectory(p.config.LocalPillarRoots, fmt.Sprintf("%s/pillar", p.config.TempConfigDir), comm, ui); err != nil { + return fmt.Errorf("Error uploading local pillar roots to remote: %s", err) } ui.Message(fmt.Sprintf("Moving %s/pillar to /srv/pillar", p.config.TempConfigDir)) diff --git a/provisioner/salt-masterless/provisioner_test.go b/provisioner/salt-masterless/provisioner_test.go index 69c2065a2..c59ee9463 100644 --- a/provisioner/salt-masterless/provisioner_test.go +++ b/provisioner/salt-masterless/provisioner_test.go @@ -87,17 +87,17 @@ func TestProvisionerPrepare_LocalStateTree(t *testing.T) { } } -func TestProvisionerPrepare_LocalPillarTree(t *testing.T) { +func TestProvisionerPrepare_LocalPillarRoots(t *testing.T) { var p Provisioner config := testConfig() - config["local_pillar_tree"] = "/i/dont/exist/i/think" + config["local_pillar_roots"] = "/i/dont/exist/i/think" err := p.Prepare(config) if err == nil { t.Fatal("should have error") } - config["local_pillar_tree"] = os.TempDir() + config["local_pillar_roots"] = os.TempDir() err = p.Prepare(config) if err != nil { t.Fatalf("err: %s", err) diff --git a/website/source/docs/provisioners/salt-masterless.html.markdown b/website/source/docs/provisioners/salt-masterless.html.markdown index 29009f671..c5d099d5d 100644 --- a/website/source/docs/provisioners/salt-masterless.html.markdown +++ b/website/source/docs/provisioners/salt-masterless.html.markdown @@ -30,6 +30,10 @@ Optional: [state tree](http://docs.saltstack.com/ref/states/highstate.html#the-salt-state-tree). This will be uploaded to the `/srv/salt` on the remote. +* `local_pillar_roots` (string) - The path to your local + [pillar roots](http://docs.saltstack.com/ref/configuration/master.html#pillar-configuration). + This will be uploaded to the `/srv/pillar` on the remote. + * `skip_bootstrap` (boolean) - By default the salt provisioner runs [salt bootstrap](https://github.com/saltstack/salt-bootstrap) to install salt. Set this to true to skip this step.