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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

codegen: Components not implemented yet #476

Open
abhinav opened this issue Aug 1, 2023 · 0 comments
Open

codegen: Components not implemented yet #476

abhinav opened this issue Aug 1, 2023 · 0 comments
Labels
area/codegen `pulumi convert` kind/enhancement Improvements or new features

Comments

@abhinav
Copy link
Contributor

abhinav commented Aug 1, 2023

Hello!

  • Vote on this issue by adding a 馃憤 reaction
  • If you want to implement this feature, comment to let us know (we'll work with you on design, scheduling, etc.)

Issue details

TestGenerateProgram/components currently fails on this repository with:

% PULUMI_ACCEPT=true go test -run=TestGenerate
--- FAIL: TestGenerateProgram (0.78s)
    --- FAIL: TestGenerateProgram/components (0.00s)
panic: Not implemented yet [recovered]
        panic: Not implemented yet

goroutine 329 [running]:
testing.tRunner.func1.2({0x101d09e40, 0x10210b7d8})
        /opt/homebrew/Cellar/go/1.20.6/libexec/src/testing/testing.go:1526 +0x1c8
testing.tRunner.func1()
        /opt/homebrew/Cellar/go/1.20.6/libexec/src/testing/testing.go:1529 +0x384
panic({0x101d09e40, 0x10210b7d8})
        /opt/homebrew/Cellar/go/1.20.6/libexec/src/runtime/panic.go:884 +0x204
github.com/pulumi/pulumi-yaml/pkg/pulumiyaml/codegen.(*generator).genNode(0x101dd95c0?, {0x1021368b0?, 0x140010964d0?})
        /Users/abg/src/pulumi/yaml/pkg/pulumiyaml/codegen/gen_program.go:162 +0xb0
github.com/pulumi/pulumi-yaml/pkg/pulumiyaml/codegen.GenerateProgram(0x14001c77700)
        /Users/abg/src/pulumi/yaml/pkg/pulumiyaml/codegen/gen_program.go:46 +0x250
github.com/pulumi/pulumi/pkg/v3/codegen/testing/test.TestProgramCodegen.func1(0x14000dd04e0)
        /Users/abg/go/pkg/mod/github.com/pulumi/pulumi/pkg/v3@v3.76.1/codegen/testing/test/program_driver.go:589 +0xee8
testing.tRunner(0x14000dd04e0, 0x140001cea50)
        /opt/homebrew/Cellar/go/1.20.6/libexec/src/testing/testing.go:1576 +0x10c
created by testing.(*T).Run
        /opt/homebrew/Cellar/go/1.20.6/libexec/src/testing/testing.go:1629 +0x368
exit status 2
FAIL    github.com/pulumi/pulumi-yaml/pkg/pulumiyaml/codegen    0.825s

This tracks fixing the issue.

Feel free to close this issue if this functionality is not intended to be added
for pulumi-yaml.

Discovered in #474

Affected area/feature

@abhinav abhinav added kind/enhancement Improvements or new features needs-triage Needs attention from the triage team labels Aug 1, 2023
abhinav added a commit that referenced this issue Aug 1, 2023
Gets the latest testdata with `make get_testdata`,
updates `get_schemas.sh` with schemas referenced by these tests,
and runs codegen to regenerate all cases.

Broken tests were opted out, with the following issues created for them.

- #476
- #477
- #478
- #479
- #480
abhinav added a commit that referenced this issue Aug 1, 2023
The version of Pulumi used by pulumi-yaml
has not been updated in a while.

Update to the latest release,
pull the latest testdata with `make get_testdata`,
update `get_schemas.sh` with schemas referenced by these tests,
and run codegen to regenerate all cases.

Broken tests were opted out, with the following issues created for them.

- #476
- #477
- #478
- #479
- #480
@dixler dixler added area/codegen `pulumi convert` and removed needs-triage Needs attention from the triage team labels Aug 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/codegen `pulumi convert` kind/enhancement Improvements or new features
Projects
None yet
Development

No branches or pull requests

2 participants