lostluck commented on PR #31010: URL: https://github.com/apache/beam/pull/31010#issuecomment-2091067510
Internal is currently on Moby 24, not 25, which has `ContainerStart(ctx context.Context, container string, options types.ContainerStartOptions) error` but not yet `StartOptions` in a container package. That's probably the break we saw. (https://pkg.go.dev/github.com/docker/[email protected]+incompatible/api/types/container#StartOptions doesn't yet exist essentially). If you have a pressing need to migrate the Beam Go SDK to the new types (or to push us to Moby 26), I could do something to the internal beam fork to basically locally apply the revert. I think the original PR was largely a (otherwise, very welcome) clean up PR? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
