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 in Cloud SQL #2030

Closed
benglewis opened this issue May 28, 2024 · 1 comment · Fixed by #2037
Closed

Bug in Cloud SQL #2030

benglewis opened this issue May 28, 2024 · 1 comment · Fixed by #2037
Assignees
Labels
area/docs Improvements or additions to documentation kind/bug Some behavior is incorrect or out of spec resolution/fixed This issue was fixed

Comments

@benglewis
Copy link

What happened?

I was trying to use a private IP instance for GCP Cloud SQL. The docs are wrong.

Example

The docs indicate that the user must pass ServiceNetworking instance as a depends_on argument, but the code example then doesn't actually show this.

Output of pulumi about

CLI
Version 3.117.0
Go Version go1.22.3
Go Compiler gc

Plugins
KIND NAME VERSION
resource aws 6.36.0
resource aws-native 0.105.0
resource azure-native 2.41.0
resource command 0.10.0
resource gcp 7.23.0
resource kubernetes 4.11.0
resource kubernetes-cert-manager 0.0.6
resource postgresql 3.11.1
resource pulumi_ceph 4.0.0
resource pulumi_cert_manager 4.0.0
resource pulumi_keda 4.0.0
resource pulumi_rabbitmq 4.0.0
resource pulumi_rabbitmq_cluster_operator 4.0.0
language python unknown
resource random 4.16.1
resource std 1.6.2
resource time 0.0.17
resource twingate 0.0.48

Host
OS darwin
Version 14.2.1
Arch arm64

This project is written in python: executable='/Users/blewis/Programming/infrastructure/.pixi/envs/default/bin/python3' version='3.12.3'

Current Stack: benglewis/mvp/dev

Dependencies:
NAME VERSION
beautifulsoup4 4.12.3
boto3 1.34.106
Brotli 1.1.0
clearml 1.15.1
cryptography 42.0.7
importlib_resources 6.4.0
pip 24.0
pkgutil_resolve_name 1.3.10
pulumi_aws 6.36.0
pulumi_aws_native 0.105.0
pulumi_azure_native 2.41.0
pulumi_ceph 4.0.0
pulumi_cert_manager 4.0.0
pulumi_command 0.10.0
pulumi_gcp 7.23.0
pulumi_keda 4.0.0
pulumi_kubernetes_cert_manager 0.0.6
pulumi_postgresql 3.11.1
pulumi_rabbitmq 4.0.0
pulumi_rabbitmq_cluster_operator 4.0.0
pulumi_random 4.16.1
pulumi_std 1.6.2
pulumi_twingate 0.0.48
pulumiverse_time 0.0.17
PySocks 1.7.1
setuptools 69.5.1
zipp 3.17.0

Pulumi locates its logs in /var/folders/4m/d23fk4hx6_38jythgkmtprnr0000gn/T/ by default

Additional context

It would be so much more helpful if the documentation here indicated that pulumi-gcp is derived from the GCP Terraform and that you can reference the Terraform documentation if stuck with documentation issues.

Contributing

Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

@benglewis benglewis added kind/bug Some behavior is incorrect or out of spec needs-triage Needs attention from the triage team labels May 28, 2024
@VenelinMartinov
Copy link
Contributor

Thanks for taking the time to report this and sorry you had trouble here.

This issue looks like pulumi/pulumi-converter-terraform#126 which should be fixed in the newest version of the converter. I've raised #2037 to bump that.

@VenelinMartinov VenelinMartinov added area/docs Improvements or additions to documentation and removed needs-triage Needs attention from the triage team labels May 29, 2024
VenelinMartinov added a commit that referenced this issue May 30, 2024
pulls in pulumi/pulumi-converter-terraform#127

fixes #2030

---------

Co-authored-by: Ian Wahbe <ian@wahbe.com>
@pulumi-bot pulumi-bot added the resolution/fixed This issue was fixed label May 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/docs Improvements or additions to documentation kind/bug Some behavior is incorrect or out of spec resolution/fixed This issue was fixed
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants