I think it might be better if we utilize the equivalent types from the docker create command: https://github.com/docker/cli/blob/0e70f1b7b831565336006298b9443b015c3c87a5/cli/command/container/opts.go#L34-L132

This plugin also has docker/cli as a dependency so we should be able to utilize the github.com/docker/cli/opts package which I think we should use instead so the options here are consistent with the equivalent options.

An alternative is to utilize the method that compose does to parse the options, but it's a bit more adhoc. The underlying implementation seems to be the same, but we may not want to reuse that section of code mostly because it is not easily reusable outside of a YAML context.

Agree, let's avoid adding unnecessary dep. opts.MemBytes can be used similar to

@jsternberg @crazy-max done, the types have been aligned to github.com/docker/cli/opts, and the unnecessary deps have been removed

Matching with my recommendation above, I think we can do this:

if d.memory != "" {
    var memory opts.MemBytes
    if err := memory.Set(d.memory); err != nil {
        return err
    }
    hc.Resources.Memory = int64(memory)
}

done

I'm hesitant to add the dependencies that are added here since the functionality included in this PR is stuff that's also been done in other docker code. My preference is going to be that we utilize those methods rather than add a new way to parse the units.

The same applies to the other two dependencies here.

If there's a compelling reason why this is brand new functionality for docker and these dependencies make implementing that functionality easier, these dependencies seem solid. My preference is just going to be on not adding dependencies if there's an existing alternative.

I think strconv.ParseInt makes more sense here and then we don't need a dependency. The container create command uses the cli flags and that uses strconv.ParseInt internally.

@jsternberg done