openzipkin-attic / docker-zipkin

Docker images for OpenZipkin
Apache License 2.0
688 stars 329 forks source link

Support starting with STORAGE_TYPE=mem #139

Closed radhus closed 7 years ago

radhus commented 7 years ago

Fix starting with the in-memory storage backend by specifying STORAGE_TYPE=mem by creating an empty .mem_profile file.

While it's the default storage backend, manually setting the environment variable didn't previously work. This hinders deployments where the variable is always set but to a dynamic value, hence 'mem' needs to be supported there as well.

codefromthecrypt commented 7 years ago

sgtm thanks. repushing 1.23.0 tag which will include this