eventarc.v1.Trigger is auto-named incorrectly

[mikhailshilkov]

The eventarc.v1.Trigger resource has two properties related to its name: a required triggerId:

https://github.com/pulumi/pulumi-google-native/blob/master/sdk/nodejs/eventarc/v1/trigger.ts#L201

and an optional name:

https://github.com/pulumi/pulumi-google-native/blob/master/sdk/nodejs/eventarc/v1/trigger.ts#L188

I suppose they represent the same thing, just with a different format. I cannot omit the triggerId, so I always have to set an explicit resource name. If I omit name, it seems to generate an auto-name with a random suffix, however it gets ignored and the eventual resource in Google Cloud will have the same name as my specified triggerId.

Digging into Google's specs (AIP-133):

Create RPCs may support this behavior by providing a string {resource}_id field on the request message.

and later

The name field on the resource must be ignored.

Therefore, I think the correct behavior would be:

1. Do not project the name property into resource inputs.
2. triggerId should be an optional property. The provider should auto-name it if user doesn't specify an explicit value.