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: optional-complex-config test is broken #479

Open
Tracked by #13503
abhinav opened this issue Aug 1, 2023 · 0 comments
Open
Tracked by #13503

codegen: optional-complex-config test is broken #479

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

In upgrading to Pulumi v3.76 (#474), we discovered that the
optional-complex-config test is broken for this repository:

--- FAIL: TestGenerateProgram (0.11s)
    --- FAIL: TestGenerateProgram/optional-complex-config (0.11s)
        gen_program_test.go:217:
                Error Trace:    /Users/abg/src/pulumi/yaml/pkg/pulumiyaml/codegen/gen_program_test.go:217
                                                        /Users/abg/go/pkg/mod/github.com/pulumi/pulumi/pkg/v3@v3.76.1/codegen/testing/test/program_driver.go:635
                Error:          Received unexpected error:
                                1 error occurred:
                                        *
                                -error: ../testing/test/testdata/optional-complex-config-pp/yaml/optional-complex-config.yaml:4,14-18: unknown type;
                                -error: ../testing/test/testdata/optional-complex-config-pp/yaml/optional-complex-config.yaml:7,14-18: unknown type;
                                -error: ../testing/test/testdata/optional-complex-config-pp/yaml/optional-complex-config.yaml:10,14-18: unknown type;
                                -error: ../testing/test/testdata/optional-complex-config-pp/yaml/optional-complex-config.yaml:13,14-18: unknown type;
                                -error: ../testing/test/testdata/optional-complex-config-pp/yaml/optional-complex-config.yaml:16,14-18: unknown type;
                                -error: ../testing/test/testdata/optional-complex-config-pp/yaml/optional-complex-config.yaml:19,14-18: unknown type;

                Test:           TestGenerateProgram/optional-complex-config
FAIL
exit status 1
FAIL    github.com/pulumi/pulumi-yaml/pkg/pulumiyaml/codegen    0.155s

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