Field containerLogs in docker:index/container:Container is incorrectly set as required

[andrzejressel]

What happened?

containerLogs is set as required in schema

pulumi-docker/provider/cmd/pulumi-resource-docker/schema.json

Lines 2406 to 2409 in aeabb7d

	"required": [
	"bridge",
	"command",
	"containerLogs",

while the description is "The logs of the container if its execution is done (attach must be disabled)." - and it indeed is not returned in persistent containers:

https://github.com/kreuzwerker/terraform-provider-docker/blob/54685a5fb9b375961620d2c555c7f3925327112c/internal/provider/resource_docker_container_funcs.go#L535-L581

This field is not required in original provider: https://github.com/kreuzwerker/terraform-provider-docker/blob/54685a5fb9b375961620d2c555c7f3925327112c/internal/provider/resource_docker_container.go#L123-L127

Example

N/A

Output of pulumi about

N/A

Additional context

No response

Contributing

Vote on this issue by adding a 👍 reaction.
To contribute a fix for this issue, leave a comment (and link to your pull request, if you've opened one already).

[andrzejressel]

There is also issue with healthcheck:

https://github.com/kreuzwerker/terraform-provider-docker/blob/54685a5fb9b375961620d2c555c7f3925327112c/internal/provider/resource_docker_container_funcs.go#L739-L749

https://github.com/pulumi/pulumi-docker/blob/master/provider/cmd/pulumi-resource-docker/schema.json#L2413

[blampe]

This is a known issue with bridged providers which is unfortunately not easy to fix in a backwards-compatible way pulumi/pulumi-terraform-bridge#1239

I've put up a possible fix in #1059. As you can see it is not backwards-compatible with existing SDKs, so existing users could see their programs fail to compile by upgrading and we want to avoid that as much as possible.

It would be helpful to understand more about your use case -- is there anything unusual you're doing to trigger the bug?