Hello I noticed an AWS managed policy was missing and added

Question

Hello I noticed an AWS managed policy was missing and added it here <https github com pulumi pulumi aws pull 1833> Is this the correct process for contributing

billowy-army-68599 · Answer

< sparse gold 10561> sort of

sparse-gold-10561 · Answer

disappointed

billowy-army-68599 · Answer

it actually needs to be added here <https github com pulumi pulumi aws blob master provider resources go L3802>

billowy-army-68599 · Answer

< sparse gold 10561> i edited my comment i was wrong

sparse-gold-10561 · Answer

oh so we only need it once here <https github com pulumi pulumi aws pull 1833 files diff 34c57e622183cb0d8dd0d3f9eaa0861b3340120e9b2ad811bac7ac7be4cea4b1R3793>

billowy-army-68599 · Answer

you ll need to add it `resources go` and then regenerate the schema using `make`

sparse-gold-10561 · Answer

ok let me mess with it and i ll submit the updated changes

sparse-gold-10561 · Answer

okay like this <https github com pulumi pulumi aws pull 1833 files>

billowy-army-68599 · Answer

< sparse gold 10561> you missed the step to regen the schemas I pushed a commit to your branch i hope you don t mind

billowy-army-68599 · Answer

billowy-army-68599 · Answer

if you want to regen yourself just run `make` from within the repo if not close your PR and your commit is in my branch

sparse-gold-10561 · Answer

I dont think my pulumi development environment is ready i ll close my PR

sparse-gold-10561 · Answer

It was throwing some errors building unrelated to this change

sparse-gold-10561 · Answer

ohwell thanks

stocky-restaurant-98004 · Answer

< sparse gold 10561> For future reference the minimal steps after changing resources go are ```make tfgen make build sdks``` Then you can push your branch If you hit any additional hurdles please let me know If we can make our docs clearer and I m sure there s room for improvement this is valuable feedback

sparse-gold-10561 · Answer

For me its building the entire development environment within a Windows Subsystem for Linux environment It seems like I need both pulumi and pulumi aws built before I can run the make commands I was having problems getting them all to build

stocky-restaurant-98004 · Answer

Pulumi itself would just need to be on your `PATH` `brew install pulumi` or maybe there s some sort of sync between WSL and your PS prompt not sure as I haven t used WSL

sparse-gold-10561 · Answer

i was planning to try it again this week and see if i can setup a full environment if so ill make notes and see about creating a doc to help others replicate the environment

stocky-restaurant-98004 · Answer

< sparse gold 10561> Thanks so much I ll be sure to include your feedback in a GH issue for our team since enabling the community eco system is a major ongoing theme for us

sparse-gold-10561 · Answer

I was able to get WSL to fully build pulumi aws however when i run make it keeps trying to switch all the go files to v4 from v5 I did the go mod tidy but that doesnt seem to help any ideas

sparse-gold-10561 · Answer

``` import gt gt ```