Skip to content
This repository has been archived by the owner on Jun 4, 2024. It is now read-only.

Pin Teleport Terraform Provider to Teleport major version #937

Merged
merged 2 commits into from
Oct 4, 2023

Conversation

tigrato
Copy link
Contributor

@tigrato tigrato commented Sep 29, 2023

The previous constrain >= (=teleport.version=) allows Terraform to pick any version bigger than the specified version. This is a problem because >= 12.0.0 can actually select 14.0.0 which breaks our client guarantees when connecting to Auth server.

This PR forces Terraform to pick any version >=12.0.0 and < 13.0.0. The pessimist version constrain prevents Terraform provider to use versions outside of the specified major version.

@marcoandredinis
Copy link
Contributor

You should also run make terraform-gen-tfschema so that we can generate the file which then should be copied to teleport repo.

The previous constrain `>= (=teleport.version=)` allows Terraform to pick any version bigger than the specified version. This is a problem because `>= 12.0.0` can actually select `14.0.0` which breaks our client guarantees when connecting to Auth server.

This PR forces Terraform to pick any version `>=12.0.0` and `< 13.0.0`. The pessimist version constrain prevents Terraform provider to use versions outside of the specified major version.

Signed-off-by: Tiago Silva <[email protected]>
@tigrato tigrato force-pushed the tigrato/fix-terraform-pinned-version branch from 363d40e to 7705d16 Compare September 29, 2023 09:46
@tigrato tigrato enabled auto-merge (squash) October 4, 2023 13:57
@tigrato tigrato merged commit 7373912 into master Oct 4, 2023
10 checks passed
@tigrato tigrato deleted the tigrato/fix-terraform-pinned-version branch October 4, 2023 14:02
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 this pull request may close these issues.

4 participants