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

triton provider key_material does not accept a file as the docs say #6211

Closed
jhorwit2 opened this issue Apr 17, 2016 · 3 comments · Fixed by #6255
Closed

triton provider key_material does not accept a file as the docs say #6211

jhorwit2 opened this issue Apr 17, 2016 · 3 comments · Fixed by #6255

Comments

@jhorwit2
Copy link

Version: current stable

As you can see in this PR key_file was changed in favor of key_material; however, it was does not work with a file. The private key value is passed to create a new auth object here, but since it's a file name it breaks.

@jhorwit2 jhorwit2 changed the title trion povider key_material does not accept a file as the docs say triton povider key_material does not accept a file as the docs say Apr 17, 2016
@jhorwit2 jhorwit2 changed the title triton povider key_material does not accept a file as the docs say triton provider key_material does not accept a file as the docs say Apr 17, 2016
@phinze
Copy link
Contributor

phinze commented Apr 18, 2016

Hi @jhorwit2 - the file() interpolation function is the intended way to lift file contents into config. Will get the docs fixed up on this!

Should be:

provider "triton" {
  account      = "AccountName"
  key_material = "${file("~/.ssh/id_rsa")}"
  key_id       = "25:d4:a9:fe:ef:e6:c0:bf:b4:4b:4b:d4:a8:8f:01:0f"
  # ...
}

(edit: actually made the change to the pasted config 😝 )

@jhorwit2
Copy link
Author

@phinze just submitted a PR #6255 for the documentation a few lines down that still references the key path.

phinze added a commit that referenced this issue Apr 20, 2016
provider/triton: fixed triton key_material documentation
phinze added a commit that referenced this issue Apr 20, 2016
[origin/master] Merge pull request #6255 from jhorwit2/jah/#6211
483d081
phinze added a commit that referenced this issue Apr 20, 2016
[origin/master] Merge pull request #6255 from jhorwit2/jah/#6211
483d081
phinze added a commit that referenced this issue Apr 20, 2016
[origin/master] Merge pull request #6255 from jhorwit2/jah/#6211
483d081
phinze added a commit that referenced this issue Apr 20, 2016
[origin/master] Merge pull request #6255 from jhorwit2/jah/#6211
483d081
terraformbot pushed a commit that referenced this issue Apr 20, 2016
[origin/master] Merge pull request #6255 from jhorwit2/jah/#6211
483d081
terraformbot pushed a commit that referenced this issue Apr 20, 2016
[origin/master] Merge pull request #6255 from jhorwit2/jah/#6211
483d081
terraformbot pushed a commit that referenced this issue Apr 20, 2016
[origin/master] Merge pull request #6255 from jhorwit2/jah/#6211
483d081
terraformbot pushed a commit that referenced this issue Apr 20, 2016
[origin/master] Merge pull request #6255 from jhorwit2/jah/#6211
483d081
terraformbot pushed a commit that referenced this issue Apr 20, 2016
[origin/master] Merge pull request #6255 from jhorwit2/jah/#6211
483d081
terraformbot pushed a commit that referenced this issue Apr 20, 2016
[origin/master] Merge pull request #6255 from jhorwit2/jah/#6211
483d081
terraformbot pushed a commit that referenced this issue Apr 20, 2016
[origin/master] Merge pull request #6255 from jhorwit2/jah/#6211
483d081
terraformbot pushed a commit that referenced this issue Apr 20, 2016
[origin/master] Merge pull request #6255 from jhorwit2/jah/#6211
483d081
terraformbot pushed a commit that referenced this issue Apr 20, 2016
[origin/master] Merge pull request #6255 from jhorwit2/jah/#6211
483d081
terraformbot pushed a commit that referenced this issue Apr 20, 2016
[origin/master] Merge pull request #6255 from jhorwit2/jah/#6211
483d081
terraformbot pushed a commit that referenced this issue Apr 20, 2016
[origin/master] Merge pull request #6255 from jhorwit2/jah/#6211
483d081
terraformbot pushed a commit that referenced this issue Apr 20, 2016
[origin/master] Merge pull request #6255 from jhorwit2/jah/#6211
483d081
terraformbot pushed a commit that referenced this issue Apr 20, 2016
[origin/master] Merge pull request #6255 from jhorwit2/jah/#6211
483d081
@ghost
Copy link

ghost commented Apr 26, 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 Apr 26, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants