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

Tokenized Variables in strings are not fully resolving #286

Open
1 task
joekiller opened this issue Sep 6, 2024 · 0 comments
Open
1 task

Tokenized Variables in strings are not fully resolving #286

joekiller opened this issue Sep 6, 2024 · 0 comments
Labels
bug Something isn't working

Comments

@joekiller
Copy link

Description

As a user I expect a Terraform Function in CDK, such as tonumber, to resolve when utilized within a construct rendered by the CDK8sProvider.

Actual behavior: the Token is resolved from a float to a variable however no further resolution takes place leaving the symbol in an un-interpolated ${} HCL interpolation syntax format, Interpolation.

Screenshot 2024-09-06 at 11 21 53 AM

I created a test demonstrating the defect here: joekiller@c4c6612

Versions

cdktf debug
language: typescript
cdktf-cli: 0.20.8
node: v20.11.1
cdktf: 0.20.8
constructs: 10.3.0
jsii: null
terraform: 1.9.4
arch: arm64
os: darwin 23.6.0
providers
random@ ~> 3.6.2 (LOCAL)
terraform provider version: 3.6.2
@cdktf/provider-google-beta (PREBUILT)
terraform provider version: 5.38.0
prebuilt provider version: 13.20.0
cdktf version: ^0.20.0
@cdktf/provider-google (PREBUILT)
terraform provider version: 5.38.0
prebuilt provider version: 13.27.0
cdktf version: ^0.20.0
@cdktf/provider-helm (PREBUILT)
terraform provider version: 2.14.1
prebuilt provider version: 10.2.1
cdktf version: ^0.20.0
@cdktf/provider-kubernetes (PREBUILT)
terraform provider version: 2.31.0
prebuilt provider version: 11.6.0
cdktf version: ^0.20.0

running npm list

dexcom-cdk@1.0.0 /Users/jl0918/src/github/dexcom-inc/dexcom-cdk
├── @cdktf/provider-google-beta@13.20.0
├── @cdktf/provider-google@13.27.0
├── @cdktf/provider-helm@10.2.1
├── @cdktf/provider-kubernetes@11.6.0
├── @eslint-community/eslint-utils@4.4.0
├── @eslint/compat@1.1.1
├── @eslint/js@9.6.0
├── @google-cloud/secret-manager@5.6.0
├── @types/eslint__js@8.42.3
├── @types/jest@29.5.12
├── @types/node@20.14.10
├── cdk8s-plus-29@2.5.4
├── cdk8s@2.68.97
├── cdktf-cdk8s@0.10.34
├── cdktf-cli@0.19.2
├── cdktf@0.20.8
├── constructs@10.3.0
├── eslint-config-prettier@9.1.0
├── globals@15.8.0
├── husky@9.0.11
├── jest@29.7.0
├── jsii-pacmak@1.102.0
├── jsii@5.4.31
├── keychain@1.5.0
├── prettier@3.3.3
├── ts-jest@29.2.2
├── ts-node@10.9.2
├── typescript-eslint@7.16.0
├── typescript@5.5.3
└── utility-types@3.11.0

Providers

───────────┬─────────────────────┬─────────┬─────────────────────┬─────────────┐
─── ─── ─ ── ─────── ───
Provider │ Provider CDKTF │ │ Package Name │ Package │
Name Version Constraint Version
───────────┼─────────────────────┼─────────┼─────────────────────┼─────────────┤
─── ─── ─ ── ─────── ───
random │ 3.6.2 │ ~> 3.6.│ │ │

───────────┼─────────────────────┼─────────┼─────────────────────┼─────────────┤
─── ─── ─ ── ─────── ───
google-bet│ 5.38.0 ^0.20.│ │ @cdktf/provider-goog│ 13.20.0 │
e-beta

───────────┼─────────────────────┼─────────┼─────────────────────┼─────────────┤
─── ─── ─ ── ─────── ───
google │ 5.38.0 ^0.20.│ │ │ 13.27.0 │
@cdktf/provider-google

───────────┼─────────────────────┼─────────┼─────────────────────┼─────────────┤
─── ─── ─ ── ─────── ───
helm │ 2.14.1 ^0.20.│ │ @cdktf/provider-helm│ 10.2.1 │

───────────┼─────────────────────┼─────────┼─────────────────────┼─────────────┤
─── ─── ─ ── ─────── ───
kubernetes│ 2.31.0 ^0.20.│ │ @cdktf/provider-kube│ 11.6.0 │
netes

───────────┴─────────────────────┴─────────┴─────────────────────┴─────────────┘
─── ─── ─ ── ─────── ───

Gist

No response

Possible Solutions

I wonder if the Only run once clause of the aspect is blocking the final run of resolution.

Workarounds

No response

Anything Else?

No response

References

Help Wanted

  • I'm interested in contributing a fix myself

Community Note

  • Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
  • Please do not leave "+1" or other comments that do not add relevant new information or questions, they generate extra noise for issue followers and do not help prioritize the request
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment
@joekiller joekiller added the bug Something isn't working label Sep 6, 2024
@joekiller joekiller changed the title Tokenized FunVariables in strings are not fully resolving Tokenized Variables in strings are not fully resolving Sep 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant