Add param of withBindVolumn for EtcdClusterExtension

[liangyuanpeng]

I use remote docker for testcontainer to run unit test and i just got the error:

Caused by: com.github.dockerjava.api.exception.InternalServerErrorException: Status 500: {"message":"invalid volume specification: 'C:\\Users\\Administrator\\AppData\\Local\\Temp\\jetcd_test_etcd0_8631704060335352785:/data.etcd:rw,z'"}

So i add param of bindVolumn for this case that do not need volumn.

[lburgazzoli]

It would be nice to understand why the volume mount fails, ideally this should not be something you should be able to disable as it may be required by the jetcd container

[liangyuanpeng]

It would be nice to understand why the volume mount fails,

Run unit test on windows, and use remote docker container from linux, the voulme is C:\\Users\\Administrator\\xxx:/data.etcd:rw,z, this format can not use by docker container. So just got the error when container start.

ideally this should not be something you should be able to disable as it may be required by the jetcd container

Absolutely right, so still create volum for etcd test container by default, User need to use withBindVolumn(false) for disable volumn when they want to this.

[github-actions[bot]]

This PR is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 7 days.

[lburgazzoli]

@liangyuanpeng can you fix the findings ?

[IlyasYOY]

@lburgazzoli is this PR up to date?

I opened a #1123, I guess they are almost identical.

I have all code formatter + tests there. This feature is very important to me.

Thanks!

[lburgazzoli]

@lburgazzoli is this PR up to date?

I opened a #1123, I guess they are almost identical.

I have all code formatter + tests there. This feature is very important to me.

Thanks!

Merged, THX !

[lburgazzoli]

Done in https://github.com/etcd-io/jetcd/pull/1123