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

All Languages - update Template S3 Terraform Backend - pages/public_c… #5431

Merged
merged 18 commits into from
Dec 6, 2023

Conversation

Mx7ca
Copy link
Contributor

@Mx7ca Mx7ca commented Nov 25, 2023

…loud/compute/use_object_storage_terraform_backend_state

This PR fixes #5430 in the documentation of all existing languages.

Copy link
Contributor

@scraly scraly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Top, thank you!

@scraly
Copy link
Contributor

scraly commented Nov 27, 2023

Thanks for the PR, I will update it to add the information for old terraform version

Copy link
Contributor

@scraly scraly left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add compatibility with old version

@BrenKt BrenKt self-assigned this Nov 27, 2023
@scraly
Copy link
Contributor

scraly commented Nov 27, 2023

The modification will only work for new version, not old version.

Please add this change:

Before terraform version 1.6.0:

terraform {
    backend "s3" {
      bucket = "terraform-state-hp"
      key    = "terraform.tfstate"
      region = "gra"
 #or sbg or any activated high performance storage region
      endpoint = "s3.gra.perf.cloud.ovh.net"
      skip_credentials_validation = true
      skip_region_validation = true
    }
}

After terraform version 1.6.0:

terraform {
    backend "s3" {
      bucket = "terraform-state-hp"
      key    = "terraform.tfstate"
      region = "gra"
      # sbg or any activated high performance storage region
      endpoints = {
        s3 = "https://s3.gra.perf.cloud.ovh.net/"
      }
      skip_credentials_validation = true
      skip_region_validation      = true
      skip_requesting_account_id  = true
    }
}

@scraly
Copy link
Contributor

scraly commented Nov 27, 2023

FYI the information about the terraform version is in the hashicorp doc:
https://developer.hashicorp.com/terraform/language/upgrade-guides#s3-backend

@BrenKt BrenKt assigned Mx7ca and unassigned BrenKt Nov 29, 2023
@BrenKt BrenKt self-requested a review November 29, 2023 08:50
@Mx7ca
Copy link
Contributor Author

Mx7ca commented Nov 30, 2023

@scraly thanks for the link to OVH's documentation. I've updated the version restriction to all files and also updated the "updated" date again.
Is it now okay?

@scraly
Copy link
Contributor

scraly commented Dec 4, 2023

Thanks, it's good for me :-)

The only thing to change is the date of the modification :)

@OVH-Team-Guides OVH-Team-Guides merged commit f5084e7 into ovh:develop Dec 6, 2023
@OVH-Team-Guides
Copy link
Collaborator

Thank you for your contribution @Mx7ca

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.

documentation for Terraform S3 Backend outdated
4 participants