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

Update arguments for codefresh_api_key resource #63

Merged
merged 2 commits into from
Sep 29, 2021

Conversation

YakShavingCatHerder
Copy link
Contributor

user_id is a required parameter to use this resource; considering the diffs between on-prem & SaaS, I didn't feel the need to complicate the examples anymore by adding data call for user, but that is important.

It should probably be noted that the majority of the [currently documented resources|https://registry.terraform.io/providers/codefresh-io/codefresh/latest/docs/resources/user] do not play well with self-hosted deployments. I'm testing getting this resource working with a self-hosted deployment and will update as needed if I'm able to get a working model going.

`user_id` is a required parameter to use this resource; considering the diffs between on-prem & SaaS, I didn't feel the need to complicate the examples anymore by adding data call for user, but that is important. 

It should probably be noted that the majority of the [currently documented resources|https://registry.terraform.io/providers/codefresh-io/codefresh/latest/docs/resources/user] do not play well with self-hosted deployments.  I'm testing getting this resource working with a self-hosted deployment and will update as needed if I'm able to get a working model going.
@sharon-codefresh sharon-codefresh merged commit a5ba5c3 into codefresh-io:master Sep 29, 2021
@YakShavingCatHerder YakShavingCatHerder deleted the patch-1 branch March 20, 2022 04:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants