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

Documentation ip_reverse separtor #380

Closed
pierregeekme opened this issue Feb 15, 2023 · 4 comments
Closed

Documentation ip_reverse separtor #380

pierregeekme opened this issue Feb 15, 2023 · 4 comments
Labels
Product: Network Status: Coming Soon The feature will hit GA usually in less than 2 months Type: Documentation

Comments

@pierregeekme
Copy link

Hi,

I create this issue for documentation https://registry.terraform.io/providers/ovh/ovh/latest/docs/resources/ip_reverse
In section Import the separator is | but the correct character is :

Terraform Version

Affected Resource(s)

  • ovh_ip_reverse

Terraform Configuration Files

resource "ovh_ip_reverse" "myappli_mydomain_fr" {
  ip = "${openstack_compute_instance_v2.myappli.access_ip_v4}/32"
  ip_reverse = openstack_compute_instance_v2.myappli.access_ip_v4
  reverse = "myappli.mydomain.fr."
}

Debug Output

ovh_ip_reverse.myappli_mydomain_fr: Importing from ID "XX.XX.XX.XXX/32|XX.XX.XX.XX"...

│ Error: Import Id is not ip:ip_reverse formatted

Panic Output

Expected Behavior

In documentation :

The resource can be imported using the ip, ip_reverse of the address, separated by ":" E.g.,

$ terraform import ovh_ip_reverse.my_reverse '2001:0db8:c0ff:ee::/64:2001:0db8:c0ff:ee::42'

Actual Behavior

In documentation :

The resource can be imported using the ip, ip_reverse of the address, separated by "|" E.g.,

$ terraform import ovh_ip_reverse.my_reverse '2001:0db8:c0ff:ee::/64|2001:0db8:c0ff:ee::42'

Steps to Reproduce

Please list the steps required to reproduce the issue, for example:

  1. terraform import ovh_ip_reverse.myappli_mydomain_fr "XX.XX.XX.XX/32|XX.XX.XX.XX"

Important Factoids

Get API Token OVH with right on /ip/*/reverse/*

References

@yomovh yomovh added Type: Documentation Status: Coming Soon The feature will hit GA usually in less than 2 months labels Feb 16, 2023
yomovh added a commit that referenced this issue Feb 16, 2023
@yomovh yomovh linked a pull request Feb 16, 2023 that will close this issue
@pierregeekme
Copy link
Author

Hi,

thank you.

This morning i have imported many ressource "ovh_ip_reverse" with the separator ":".
But, now (5 minutes ago), i have this message :

ovh_ip_reverse.myapplu_mydomain.fr Importing from ID "xx.xx.xx.x/32:xx.xx.xx.xx"...
╷
│ Error: Import Id is not ip|ip_reverse formatted
│ 
│ 

With the separator "|" it's good. We have deploy a new version of provider ovh ?

@scraly
Copy link
Collaborator

scraly commented Feb 22, 2023

Hi,

since v0.26.0 the separator has changed, in order to be compliant with IPv6 format :)

You can find all the changelog of the OVH Terraform provider v0.26.0:
https://github.com/ovh/terraform-provider-ovh/releases/tag/v0.26.0

Maybe since a new terraform init command you have downloaded the latest version :)

@scraly
Copy link
Collaborator

scraly commented Feb 22, 2023

To know when a new release have been published, you can take a look to the officiel registry:
https://registry.terraform.io/providers/ovh/ovh/latest
image

To be notified when a new release have been published, just click in "Watch" button in this repository:
image
And then click on "Custom" and then select the checkbox "Releases" and Apply the changes :)
image

@scraly scraly closed this as completed Feb 22, 2023
@pierregeekme
Copy link
Author

Hi,

since v0.26.0 the separator has changed, in order to be compliant with IPv6 format :)

You can find all the changelog of the OVH Terraform provider v0.26.0: https://github.com/ovh/terraform-provider-ovh/releases/tag/v0.26.0

Maybe since a new terraform init command you have downloaded the latest version :)

Thank you.
Sorry, I hadn't thought to update the provider.

yomovh added a commit that referenced this issue Mar 7, 2023
… the separator as pointed by issue #380"

This reverts commit 3bc1912.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Product: Network Status: Coming Soon The feature will hit GA usually in less than 2 months Type: Documentation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants