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

add import documentation for zone #486

Merged
merged 1 commit into from
Nov 16, 2023
Merged

Conversation

yomovh
Copy link
Contributor

@yomovh yomovh commented Nov 3, 2023

Description

Add documentation on how to import domain_zone resource
Fixes #481

Type of change

  • [X ] Documentation update

How Has This Been Tested?

Test Configuration:

  • Terraform version: terraform version: Terraform v1.6.2
  • Existing HCL configuration you used:
data "ovh_me" "myaccount" {}

data "ovh_order_cart" "mycart" {
  ovh_subsidiary = data.ovh_me.myaccount.ovh_subsidiary
}

data "ovh_order_cart_product_plan" "zone" {
  cart_id        = data.ovh_order_cart.mycart.id
  price_capacity = "renew"
  product        = "dns"
  plan_code      = "zone"
}

resource "ovh_domain_zone" "main" {
  ovh_subsidiary = data.ovh_order_cart.mycart.ovh_subsidiary

  plan {
    duration     = data.ovh_order_cart_product_plan.zone.selected_price.0.duration
    plan_code    = data.ovh_order_cart_product_plan.zone.plan_code
    pricing_mode = data.ovh_order_cart_product_plan.zone.selected_price.0.pricing_mode
  }
}
  • terraform import ovh_domain_zone.main XXXXXX

Checklist:

N/A

@yomovh yomovh linked an issue Nov 3, 2023 that may be closed by this pull request
@yomovh yomovh requested a review from rbeuque74 November 3, 2023 15:52
@ovh-cds
Copy link
Collaborator

ovh-cds commented Nov 5, 2023

CDS Report terraform-provider-ovh-testacc#2122.0 ✘
*

  • checks ✘

@ovh-cds
Copy link
Collaborator

ovh-cds commented Nov 5, 2023

CDS Report terraform-provider-ovh-testacc#2123.0 ✘
*

  • checks ✘

@yomovh yomovh requested review from amstuta and removed request for rbeuque74 November 16, 2023 09:23
@amstuta amstuta merged commit 13ea072 into master Nov 16, 2023
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.

[BUG] Importing Domain Zone
3 participants