Skip to content

Commit

Permalink
Upgrade google-protobuf to 3.21.4 from inherited 3.21.2 (#1262)
Browse files Browse the repository at this point in the history
<!--Thanks for your contribution. See [CONTRIBUTING](CONTRIBUTING.md)
    for Pulumi's contribution guidelines.

    Help us merge your changes more quickly by adding more details such
    as labels, milestones, and reviewers.-->

### Proposed changes

<!--Give us a brief description of what you've done and what it solves.
-->

Upgrade google-protobuf from 3.21.2 inherited from @pulumi/pulumi to
3.21.4 (the latest version). This affects both the provider and the Node
SDK that are coupled together in this provider.

Fixes spontaneous failure of TestAccAwsCluster and many other tests.
Typical failure:

```
 Resources:
      + 1 to create
      command.go:98: Invoke '/usr/local/bin/pulumi preview --non-interactive --diff' failed: exit status 255
      command.go:121: Command completed without output
      program.go:1066: [/usr/local/bin/pulumi preview --non-interactive --diff] failed: exit status 255; retrying...
      command.go:46: **** Invoke '/usr/local/bin/pulumi preview --non-interactive --diff' in '/tmp/p-it-fv-az692-2-aws-profil-d12a0b7a-1252542546'
  Previewing update (p-it-fv-az692-2-aws-profil-d12a0b7a):
  warning: using pulumi-resource-eks from $PATH at /home/runner/work/pulumi-eks/pulumi-eks/bin/pulumi-resource-eks
  + pulumi:pulumi:Stack: (create)
      [urn=urn:pulumi:p-it-fv-az692-2-aws-profil-d12a0b7a::example-aws-profile::pulumi:pulumi:Stack::example-aws-profile-p-it-fv-az692-2-aws-profil-d12a0b7a]
  error: Running program '/tmp/p-it-fv-az692-2-aws-profil-d12a0b7a-1252542546/' failed with an unhandled exception:
  <ref *1> TypeError: c.g is not a function
      at Function.N.setWrapperField (/tmp/p-it-fv-az692-2-aws-profil-d12a0b7a-1252542546/node_modules/google-protobuf/google-protobuf.js:91:66)
      at proto.pulumirpc.RegisterResourceRequest.setObject (/tmp/p-it-fv-az692-2-aws-profil-d12a0b7a-1252542546/node_modules/@pulumi/pulumi/proto/resource_pb.js:2511:23)
      at Object.<anonymous> (/tmp/p-it-fv-az692-2-aws-profil-d12a0b7a-1252542546/node_modules/@pulumi/runtime/resource.ts:486:17)
      at Generator.next (<anonymous>)
      at fulfilled (/tmp/p-it-fv-az692-2-aws-profil-d12a0b7a-1252542546/node_modules/@pulumi/pulumi/runtime/resource.js:18:58) {
    promise: Promise { <rejected> [Circular *1] }
  }
```

### Related issues (optional)

Partially addresses #1258

<!--Refer to related PRs or issues: #1234, or 'Fixes #1234' or 'Closes
#1234'.
Or link to full URLs to issues or pull requests in other GitHub
repositories. -->
  • Loading branch information
t0yv0 authored Jul 17, 2024
1 parent 8357a88 commit b83e6e4
Show file tree
Hide file tree
Showing 2 changed files with 2,143 additions and 1,624 deletions.
7 changes: 6 additions & 1 deletion nodejs/eks/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,11 @@
"tslint-config-prettier": "^1.18.0",
"typescript": "^4.6.2"
},
"overrides": {
"@pulumi/pulumi": {
"google-protobuf": "3.21.4"
}
},
"pulumi": {
"resource": true
},
Expand All @@ -61,4 +66,4 @@
"/bin/"
]
}
}
}
Loading

0 comments on commit b83e6e4

Please sign in to comment.