This message was deleted.
# package-authorings
sparse-intern-71089
09/29/2023, 10:09 AMThis message was deleted.
t
thankful-match-46035
09/29/2023, 11:57 AMI don’t understand what is the issue? 🙂
l
limited-rainbow-51650
09/29/2023, 12:15 PMI had some time for a quick check. In the TF provider, the
ovh_cloud_projecturnurnovh_cloud_project.urnurnresources.got
thankful-match-46035
09/29/2023, 12:16 PM(thanks ringo now I am understanding 🙂 )
q
quaint-spring-93350
09/29/2023, 12:16 PMyes, precisely, but the problem is in the schema itself here: https://github.com/ovh/pulumi-ovh/blob/main/provider/cmd/pulumi-resource-ovh/schema.json
quaint-spring-93350
09/29/2023, 12:16 PMsearch for
"urn"quaint-spring-93350
09/29/2023, 12:16 PMthere should be none fields with that name
l
limited-rainbow-51650
09/29/2023, 12:17 PM@quaint-spring-93350 the
schema.jsonresources.goresources.goschema.json👍 1
q
quaint-spring-93350
09/29/2023, 12:17 PMah, cool!
t
thankful-match-46035
09/29/2023, 12:51 PM@limited-rainbow-51650 if you can help me where i can change the “urn” field in `resources.go`🙏
l
limited-rainbow-51650
09/29/2023, 12:53 PM@thankful-match-46035 see earlier up where I asked input myself from @enough-garden-22763. After you have a fix, you should rebuild everything in the Pulumi provider and create a new release.
e
enough-garden-22763
09/29/2023, 12:54 PM Copy code
"aws_api_gateway_authorizer": {
Tok: awsResource(apigatewayMod, "Authorizer"),
Fields: map[string]*tfbridge.SchemaInfo{
"rest_api_id": {
Name: "restApi",
Type: "string",
AltTypes: []tokens.Type{awsTypeDefaultFile(apigatewayMod, "RestApi")},
},
},
},enough-garden-22763
09/29/2023, 12:55 PMhttps://github.com/pulumi/pulumi-aws/blob/6741aee58a981a6b2bcad6eeb497944d5f5b1523/provider/resources.go#L790-#L790 here's a quick example from pulumi-aws that tweaks the name of a field. I think the same could be used here to avoid the conflict on urn. Let me know if this helps.
t
thankful-match-46035
09/29/2023, 1:14 PMso the field “urn” should be renamed to “URN” or the text “urn” is forbidden in lowercase and uppercase? 🙂
e
enough-garden-22763
09/29/2023, 1:16 PMHm, yes, "URN" would be good to try. It's difficult to reason through without trying because I suspect some language projections like C# might reject some casings. I think last time we had something like this we opted for "urn_".
q
quaint-spring-93350
09/29/2023, 1:16 PMsame here, we add underscore suffix to reserved field names
quaint-spring-93350
09/29/2023, 1:17 PMalternatively, for readability, it could be
ovhUrnquaint-spring-93350
09/29/2023, 1:17 PMas that's an OVH-specific identifier
t
thankful-match-46035
09/29/2023, 1:20 PMit’s not an ovh urn
it’s a cloud project urn 😉
thankful-match-46035
09/29/2023, 1:21 PMSo if i understand correctly, i need to do this modification in the
resource.go Copy code
"ovh_cloud_project": {
Tok: ovhResource(cloudProjectMod, "Project"),
Fields: map[string]*tfbridge.SchemaInfo{
"urn": {
Name: "ProjectURN",
},
},
},👍 1
thankful-match-46035
09/29/2023, 1:36 PMAnd, question, should I fix in all resources containing urn field or only coud_project?
thankful-match-46035
09/29/2023, 2:24 PMv0.34.3 with the fix in progress 🤞
👀 1
thankful-match-46035
09/29/2023, 3:06 PM3 Views