Improve logging.

Correct behavior for undefined recipe list.
Correct package name to upstream repo.
This commit is contained in:
James Van Dyke 2013-07-11 12:22:15 -04:00
parent 1fadd03837
commit a6f3bb3bb2
2 changed files with 13 additions and 8 deletions

View File

@ -1,8 +1,8 @@
package main package main
import ( import (
"github.com/jvandyke/packer/provisioner/chef-solo"
"github.com/mitchellh/packer/packer/plugin" "github.com/mitchellh/packer/packer/plugin"
"github.com/mitchellh/packer/provisioner/chef-solo"
) )
func main() { func main() {

View File

@ -149,8 +149,6 @@ func (p *Provisioner) Provision(ui packer.Ui, comm packer.Communicator) error {
return fmt.Errorf("Error running Chef Solo: %s", err) return fmt.Errorf("Error running Chef Solo: %s", err)
} }
// return fmt.Errorf("Die")
return nil return nil
} }
@ -178,6 +176,7 @@ func UploadLocalDirectory(localDir string, comm packer.Communicator) (err error)
return return
} }
log.Printf("Uploading directory %s", localDir)
err = filepath.Walk(localDir, visitPath) err = filepath.Walk(localDir, visitPath)
if err != nil { if err != nil {
return fmt.Errorf("Error uploading cookbook %s: %s", localDir, err) return fmt.Errorf("Error uploading cookbook %s: %s", localDir, err)
@ -187,7 +186,8 @@ func UploadLocalDirectory(localDir string, comm packer.Communicator) (err error)
} }
func CreateRemoteDirectory(path string, comm packer.Communicator) (err error) { func CreateRemoteDirectory(path string, comm packer.Communicator) (err error) {
//Ui.Say(fmt.Sprintf("Creating directory: %s", path)) log.Printf("Creating remote directory: %s ", path)
var copyCommand = []string{"mkdir -p", path} var copyCommand = []string{"mkdir -p", path}
var cmd packer.RemoteCmd var cmd packer.RemoteCmd
@ -211,6 +211,7 @@ func CreateSoloRb(cookbooksPaths []string, comm packer.Communicator) (str string
Ui.Say(fmt.Sprintf("Creating Chef configuration file...")) Ui.Say(fmt.Sprintf("Creating Chef configuration file..."))
remotePath := RemoteStagingPath + "/solo.rb" remotePath := RemoteStagingPath + "/solo.rb"
tf, err := ioutil.TempFile("", "packer-chef-solo-rb") tf, err := ioutil.TempFile("", "packer-chef-solo-rb")
if err != nil { if err != nil {
return "", fmt.Errorf("Error preparing Chef solo.rb: %s", err) return "", fmt.Errorf("Error preparing Chef solo.rb: %s", err)
@ -234,12 +235,13 @@ func CreateSoloRb(cookbooksPaths []string, comm packer.Communicator) (str string
name := tf.Name() name := tf.Name()
tf.Close() tf.Close()
f, err := os.Open(name) f, err := os.Open(name)
comm.Upload(remotePath, f)
defer os.Remove(name) defer os.Remove(name)
// Upload the Chef Solo configuration file to the cookbook directory. log.Printf("Chef configuration file contents: %s", contents)
// Upload the Chef Solo configuration file to the cookbook directory.
log.Printf("Uploading chef configuration file to %s", remotePath)
err = comm.Upload(remotePath, f)
if err != nil { if err != nil {
return "", fmt.Errorf("Error uploading Chef Solo configuration file: %s", err) return "", fmt.Errorf("Error uploading Chef Solo configuration file: %s", err)
} }
@ -257,7 +259,10 @@ func CreateAttributesJson(jsonAttrs map[string]interface{}, recipes []string, co
} }
// Add Recipes to JSON // Add Recipes to JSON
jsonAttrs["run_list"] = formattedRecipes if len(formattedRecipes) > 0 {
log.Printf("Overriding node run list: %s", strings.Join(formattedRecipes, ", "))
jsonAttrs["run_list"] = formattedRecipes
}
// Convert to JSON string // Convert to JSON string
jsonString, err := json.MarshalIndent(jsonAttrs, "", " ") jsonString, err := json.MarshalIndent(jsonAttrs, "", " ")