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

[v12] Declare device resource CRUD methods #24398

Closed
wants to merge 2 commits into from

Conversation

codingllama
Copy link
Contributor

Declare device resource CRUD methods, to be used by Terraform integration.

gravitational/teleport.e#939

Backports #23913 to branch/v12.

@codingllama codingllama changed the title Declare device resource CRUD methods [v12] Declare device resource CRUD methods Apr 11, 2023
@codingllama
Copy link
Contributor Author

Backporting for Terraform integration. FYI @sshahcodes.

@codingllama codingllama force-pushed the codingllama/backport/23913-v12 branch from 60e2b38 to 5237ee7 Compare April 11, 2023 21:27
@codingllama
Copy link
Contributor Author

Fixed issues with UpsertDeviceResource by returning "not implemented". Devices have no update-able fields in v12.

@public-teleport-github-review-bot public-teleport-github-review-bot bot removed the request for review from kimlisa April 11, 2023 22:00
@codingllama
Copy link
Contributor Author

Thanks, everyone.

I'll keep this around while we double-check that we are landing Terraform/Device support for v12, but I'll hit merge right after.

@codingllama
Copy link
Contributor Author

We lack device update capabilities on v12, which impacts Terraform, so closing this for now.

@zmb3 zmb3 deleted the codingllama/backport/23913-v12 branch April 26, 2023 22:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants