Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

when chef version not set terraform fails #2568

Closed
ramonskie opened this issue Jun 30, 2015 · 3 comments
Closed

when chef version not set terraform fails #2568

ramonskie opened this issue Jun 30, 2015 · 3 comments

Comments

@ramonskie
Copy link

when using the chef provisioner
and not provide the version as stated in
https://www.terraform.io/docs/provisioners/chef.html#version_string_
it should use the latest

but in stead it just fails because it still provides the -v option

* Command "sudo bash ./install.sh -v " exited with non-zero exit status: 1

see https://github.com/hashicorp/terraform/blob/master/builtin/provisioners/chef/linux_provisioner.go#L36

@svanharmelen
Copy link
Contributor

Thanks for the report @ramonskie! Just fixed this one in PR #2570.

@ramonskie
Copy link
Author

👍

adamenger added a commit to reverbdotcom/terraform that referenced this issue Jun 30, 2015
…g-rds-snapshots

* 'master' of github.com:reverbdotcom/terraform: (524 commits)
  docs: tweaks to RELEASING
  Minor change to docs
  Update CHANGELOG.md
  Update DynamoDB example docs to remove non-key attributes; update test to remove non-key attribute from attribute set to prevent infinite planning loops
  Update CHANGELOG.md
  use /usr/bin/env bash
  provider/aws: fix go vet
  provider/aws: ignore providers with Meta nil
  update CHANGELOG
  provider/aws: Code cleanups for Spot Requests
  provider/aws: fix db_subnet acc test
  Fixing the tests
  Fixes issue hashicorp#2568
  Update CHANGELOG.md
  Update CHANGELOG.md
  fixes typo
  Fixed void Azure network config bug.
  provider/aws: ecs task definition is deregistered correctly
  provider/azure: fixup storage service test
  provider/docker: [tests] change images
  ...
@ghost
Copy link

ghost commented May 1, 2020

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you have found a problem that seems similar to this, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@ghost ghost locked and limited conversation to collaborators May 1, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants