-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Ensure package generation for Go SDKs doesn't panic when encountering overly nested arrays or maps of primitive types #15680
Conversation
Changelog[uncommitted] (2024-03-14)Bug Fixes
|
8044fce
to
1f1e411
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Directory: "regress-go-15478", | ||
Description: "Regress pulumi/pulumi#15478 affecting Go ensuring SDK-gen doesn't panic", | ||
Skip: allLanguages.Except("go/any"), | ||
SkipCompileCheck: codegen.NewStringSet(golang), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we add a comment here? We're deliberately skipping right now because we don't expect it to be able to compile due to missing deeply nested container types in the core Go SDK.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added a comment ✅
} | ||
} | ||
} | ||
}, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should probably add map/map/map as well
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done ✅
…ly nested arrays or maps of primitive types
40d83a5
to
c0bb9d7
Compare
### Features - [auto/go] Support remote deployment executor image [#15697](#15697) ### Bug Fixes - [engine] Test and fix the engine filling in property dependencies to provider constructs [#15675](#15675) - [sdkgen/go] Ensure package generation for Go doesn't panic when encountering overly nested arrays or maps of primitive types [#15680](#15680)
Description
Fixes #15478 where sdk-gen should no longer panic when encountering overly nested arrays or maps of primitive types. The generated code still fails to compile but validation can be added later to cover cases where we are missing primitives in the core SDK to handle overly nested type definitions.
Checklist
make tidy
to update any new dependenciesmake lint
to verify my code passes the lint checkgofumpt
make changelog
and committed thechangelog/pending/<file>
documenting my change