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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pass through additional secret outputs from the provider schema #526

Merged
merged 4 commits into from
Nov 16, 2023

Conversation

tgummerer
Copy link
Contributor

Currently when the provider marks any outputs as secrets, the yaml language provider does not forward that information to the engine.

Read the secret outputs from the schema and append them as additional secret outputs to tell the engine what needs to be kept secret.

Fixes #521

Currently when the provider marks any outputs as secrets, the yaml
language provider does not forward that information to the engine.

Read the secret outputs from the schema and append them as additional
secret outputs to tell the engine what needs to be kept secret.
@tgummerer tgummerer marked this pull request as ready for review November 16, 2023 14:34
@tgummerer tgummerer requested a review from a team November 16, 2023 14:34
@tgummerer tgummerer merged commit 2e209b5 into main Nov 16, 2023
7 checks passed
@tgummerer tgummerer deleted the tg/additional-secret-outputs branch November 16, 2023 15:15
github-merge-queue bot pushed a commit to pulumi/pulumi that referenced this pull request Nov 17, 2023
# Description

Upgrade pulumi-yaml to the latest version, including
pulumi/pulumi-yaml#526.


## Checklist

- [x] I have run `make tidy` to update any new dependencies
- [ ] I have run `make lint` to verify my code passes the lint check
  - [ ] I have formatted my code using `gofumpt`

<!--- Please provide details if the checkbox below is to be left
unchecked. -->
- [ ] I have added tests that prove my fix is effective or that my
feature works
<!--- 
User-facing changes require a CHANGELOG entry.
-->
- [ ] I have run `make changelog` and committed the
`changelog/pending/<file>` documenting my change
<!--
If the change(s) in this PR is a modification of an existing call to the
Pulumi Cloud,
then the service should honor older versions of the CLI where this
change would not exist.
You must then bump the API version in
/pkg/backend/httpstate/client/api.go, as well as add
it to the service.
-->
- [ ] Yes, there are changes in this PR that warrants bumping the Pulumi
Cloud API version
<!-- @pulumi employees: If yes, you must submit corresponding changes in
the service repo. -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

YAML SDK does not honor schema-set Secrets
2 participants