Commit 3cce3df
authored
Remove rds.db.Proxy revert patch (#5915)
This patch was added a while ago to fix a panic in the bridge. We
accidentally broke it [a couple of releases
ago](https://github.com/pulumi/pulumi-aws/pull/5872/files#diff-621a91eb03679b13105bec390bcc509e1358fd96caff45b2437709e3d526c775).
While fixing the patch, I went ahead and tested what would happen if we
removed the patch altogether and it worked.
It looks like the reason we didn't catch this regression in tests is
that our test only ran in preview. This must have been good enough to
catch the original panic, but wasn't good enough to catch this
regression. I've updated the tests to perform actual deployments and
added another test case that was mentioned in the original issue.
closes #3968
fixes #59011 parent c785a6f commit 3cce3df
File tree
11 files changed
+20
-494
lines changed- examples
- test-programs/regress-3835
- patches
11 files changed
+20
-494
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1055 | 1055 | | |
1056 | 1056 | | |
1057 | 1057 | | |
1058 | | - | |
1059 | | - | |
| 1058 | + | |
| 1059 | + | |
1060 | 1060 | | |
1061 | 1061 | | |
1062 | 1062 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
30 | 30 | | |
31 | 31 | | |
32 | 32 | | |
33 | | - | |
34 | 33 | | |
35 | 34 | | |
36 | 35 | | |
37 | 36 | | |
38 | 37 | | |
39 | 38 | | |
40 | 39 | | |
| 40 | + | |
41 | 41 | | |
42 | 42 | | |
43 | | - | |
| 43 | + | |
44 | 44 | | |
45 | 45 | | |
46 | 46 | | |
47 | 47 | | |
48 | 48 | | |
49 | 49 | | |
50 | 50 | | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
51 | 66 | | |
| 67 | + | |
0 commit comments