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

Bug: timescale_peering_connection cannot be imported #207

Open
mattk-ce opened this issue Jun 12, 2024 · 2 comments
Open

Bug: timescale_peering_connection cannot be imported #207

mattk-ce opened this issue Jun 12, 2024 · 2 comments

Comments

@mattk-ce
Copy link

mattk-ce commented Jun 12, 2024

Docs say "Import can be done with timescale_vpc_id,peer_account_id,peer_region_code,peer_vpc_id format", but it does not work.

With the following import block (real IDs replaced by same-length and format random IDs):

import {
  to = timescale_peering_connection.pc
  id = "vpc-12345677890123450,123456789012,us-east-1,vpc-98765432109876543"
}

I receive

| Error: Parse Error
|
│ could not parse peering connection ID

In looking at the code, based on this line, it seems that it might be expecting the first parameter to actually be an integer. But even if it managed to parse that, it doesn't seem to ever do anything with it other than output diagnostics. Is it safe to say importing this rather critical resource is non-functional?

@mattk-ce
Copy link
Author

mattk-ce commented Jun 12, 2024

Ah, I figured it out. It looks like it expects an internal ID rather than the AWS VPC ID. I think it would be helpful to explain this in the docs, or make it more obvious with the naming of the format string, something like:

"Import can be done with timescale_internal_vpc_id,peer_account_id,peer_region_code,peer_aws_vpc_id format"

Also, the only reason I knew the internal ID of the VPC was because I also imported the VPC resource and was able to see it in the resource's properties. The Timescale control plane UI doesn't expose this ID anywhere, at least nowhere that I could find.

EDIT - reopened issue for visibility on documentation issue.

@mattk-ce mattk-ce reopened this Jun 12, 2024
@Khyme
Copy link
Contributor

Khyme commented Jun 13, 2024

Thank you for your feedback @mattk-ce , will update to make it clearer.

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

No branches or pull requests

2 participants