-
Notifications
You must be signed in to change notification settings - Fork 27
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
Consider adding support for Read
#432
Comments
Perhaps this should be modelled as "refresh" |
The README points to #20 and says `Update` isn't implemented. [Diff](#433) and [Read](#432) are still not done, but [Update is implemented](https://github.com/pulumi/pulumi-command/blob/e7e39fd913708c3e5255d8399c87a736553fec43/sdk/python/pulumi_command/remote/command.py#L196) The example code could be updated (hah!) to include an `update` (maybe with Luke's `color`/`description` example), but this PR just removes information that misled me.
Hi do we have any update regarding this issue ? A lot of my resources depend on the output of one If you could give some pointers, I would love to give this feature a try by myself. Pulumi is perfect for me with this feature :D |
Hi @vnghia, I'm not quite sure how a custom Read would solve your problem. Could you elaborate? Unless You might consider changing your command to modify the output, for instance, by piping it to another program. You could also use ignoreChanges if the output doesn't matter. |
I see. Probably something off in my configuration :/ Thank you very much! |
The
Read
operation is defined in the Pulumi resource model.Allowing users to define a specific command to run on
Read
could open up new use cases. If you have such a case, please comment below!This issue is a follow-up from #20.
The text was updated successfully, but these errors were encountered: