You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Thank you very much for your library, it's very convenient and I love it!
I used it in bitbucket pipelines and successfully run Redis passing "SkipReaper" via ContainerOptions (bitbucket has strict limitations for containers in the pipeline). But I wasn't able to do the same for Kafka, because the options struct is merged only with a Kafka container. So, I've ended up copying some code from your library to be able to set the needed options.
I kindly ask you to add a way to pass options for network and zookeeper containers. Maybe by merging options to every container, maybe, but adding several options variables, is up to you to choose.
Thank you again for creating such an amazing library, I was very glad to find it.
P.S.: In case you are too busy to add it, but you are ok with me creating a PR, write here a few words about how you want it to be. Maybe I'll find some time to contribute.
The text was updated successfully, but these errors were encountered:
Thank you very much for your library, it's very convenient and I love it!
I used it in bitbucket pipelines and successfully run Redis passing "SkipReaper" via ContainerOptions (bitbucket has strict limitations for containers in the pipeline). But I wasn't able to do the same for Kafka, because the options struct is merged only with a Kafka container. So, I've ended up copying some code from your library to be able to set the needed options.
I kindly ask you to add a way to pass options for network and zookeeper containers. Maybe by merging options to every container, maybe, but adding several options variables, is up to you to choose.
Thank you again for creating such an amazing library, I was very glad to find it.
P.S.: In case you are too busy to add it, but you are ok with me creating a PR, write here a few words about how you want it to be. Maybe I'll find some time to contribute.
The text was updated successfully, but these errors were encountered: