mirror of https://github.com/pulumi/pulumi.git
f43f549d12
11471: Improve `bors` merge reliability: Windows smoke tests are advisory and run with reduced parallelism r=AaronFriel a=AaronFriel The `continue-on-error` flag is set to true whenever the matrix platform contains "windows" for smoke tests. This does mean these jobs will allow failures and proceed, however this should substantially reduce the false negatives due to the behavior of the CI platform. Per the response from GitHub support ticket on my account: > I've confirmed that our engineering team has an open backlog item focused around "protecting" the provisioning service in cases of resource starvation. I believe this will address your exact concerns so that high resource consumption does not result in an abandoned job. > > In the meantime, there unfortunately is not much capability exposed for affecting priority on the VM. Until improvements are implemented, our team still recommends trying to reduce load on the CPU at the point(s) in the workflow where the job consistently is getting abandoned. I hope that setting this option, reducing Windows test parallelism, and setting timeouts appropriately will help improve CI reliability. Our CI jobs typically complete in 35 minutes, any longer indicates high likelihood that the runner has failed. 11559: fix(sdk/python): Allow for duplicate output values in python programs r=kpitzen a=kpitzen I used the node SDK as inspiration for this change - there were a few things we were doing differently in python (specifically handling all cases in the outer layer of `massage` rather than allowing for more fine-grained control in a `massage_complex` function like in node. We also take the stack concept from the node SDK to resolve the immediate issue of duplicate outputs. <!--- Thanks so much for your contribution! If this is your first time contributing, please ensure that you have read the [CONTRIBUTING](https://github.com/pulumi/pulumi/blob/master/CONTRIBUTING.md) documentation. --> # Description <!--- Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. --> Fixes #11133 ## Checklist <!--- Please provide details if the checkbox below is to be left unchecked. --> - [x] I have added tests that prove my fix is effective or that my feature works <!--- User-facing changes require a CHANGELOG entry. --> - [x] 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 Service, 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 Service API version <!-- `@Pulumi` employees: If yes, you must submit corresponding changes in the service repo. --> Co-authored-by: Aaron Friel <mayreply@aaronfriel.com> Co-authored-by: Kyle Pitzen <kylepitzen@Kyles-MBP.fios-router.home> |
||
|---|---|---|
| .. | ||
| pending | ||
| config.yaml | ||