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

docs: add docs for private endpoint connections #197

Merged
merged 1 commit into from
Apr 4, 2024

Conversation

carloruiz
Copy link
Contributor

Adds more detailed docs for the private_endpoint_connection resource.

Commit checklist

  • Changelog
  • Doc gen (make generate)
  • Integration test(s)
  • Acceptance test(s)
  • Example(s)

@carloruiz carloruiz force-pushed the add-docs-to-private-connect branch 3 times, most recently from 9385819 to 5cb8b4d Compare April 1, 2024 17:38
## Example Usage
Private endpoint connections allow customer applications to connect to a CockroachDB Cloud cluster without traversing the public internet. All application-database traffic remains within the cloud-provider network.

**IMPORTANT:**You must have the following role to successfully handle the resource:
Copy link

Choose a reason for hiding this comment

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

Suggested change
**IMPORTANT:**You must have the following role to successfully handle the resource:
**IMPORTANT:** You must have the following role to successfully handle the resource:

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@carloruiz carloruiz force-pushed the add-docs-to-private-connect branch from 5cb8b4d to 3cf6248 Compare April 2, 2024 02:06
@carloruiz carloruiz requested a review from fantapop April 3, 2024 18:20
Copy link

@caobill caobill left a comment

Choose a reason for hiding this comment

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

lgtm

# Establish a connection between the endpoint and the service.
resource "cockroach_private_endpoint_connection" "connection" {
cluster_id = cockroach_cluster.my_cluster.id
endpoint_id = "6133183410995353"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Could this just be:
endpoint_id = azurerm_private_endpoint.my_endpoint.id

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This particular declaration is for GCP, which doesn't have a TF resource for private endpoint connections.

Copy link
Collaborator

@fantapop fantapop left a comment

Choose a reason for hiding this comment

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

I left one suggestion but whatever your response there, this seems good to me.

@carloruiz carloruiz force-pushed the add-docs-to-private-connect branch from cc03cdf to 50b33af Compare April 4, 2024 18:42
@carloruiz carloruiz merged commit 9c26a15 into main Apr 4, 2024
3 checks passed
@carloruiz carloruiz deleted the add-docs-to-private-connect branch April 4, 2024 18:45
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.

4 participants