`ContainerInspectParameters.IncludeSize` is defined but never used

[mt-akar]

Problem

I am trying to get the disk usage size of a container using docker inspect. InspectContainerAsync call returns a ContainerInspectResponse object that contains SizeRw and SizeRootFs fields. However, these always return null.

This should have a simple solution where we pass a ContainerInspectParameters with IncludeSize property to InspectContainerAsync to tell the SDK to populate those values. However, inspect operations do not accept any parameters object. This results in an inability to specify the IncludeSize flag while inspecting containers. ContainerInspectParameters very much feels like it is designed for this purpose, but it is never used. More interestingly, in the ContainerOperations, almost all operations take a parameters object, except the inspect operations.

Oversight

I believe there is an oversight in the implementation of this SDK in this regard. Could you please look at this?

[HofmeisterAn]

We already generate the query type ContainerInspectParameters, we just need to pass it to the container inspect API. This repository is somewhat inactive, but if you create a PR here: https://github.com/testcontainers/Docker.DotNet, I can review and merge it.

[mt-akar]

@HofmeisterAn I see that you are maintaining a forked version of this repo. How wonderful. 🌟 I am grateful to see that, and I would love to contribute. 🚀🚀🚀

I just do not know how to pass that down to the container inspect API. I will have to solve this on my own anyways. If you point me in the right direction, I would love to contribute my updates to your repo.

You can find me on discord with username viraleffect

[HofmeisterAn]

You will need to extend the IContainerOperations interface and add the type as an argument to the method. Then, implement it like the other methods, for example, this.