Skip to content

Commit

Permalink
Update jaeger all-in-one image to 1.14
Browse files Browse the repository at this point in the history
Version 1.12 for me tried to allocate all the ram which would result in the machine no longer reacting.
With 1.14 I no longer saw this behavior.

Also use the new tmp volume (see jaegertracing/jaeger#1571)
  • Loading branch information
lukedirtwalker committed Sep 10, 2019
1 parent 17cc7b5 commit 298ab8c
Showing 1 changed file with 2 additions and 5 deletions.
7 changes: 2 additions & 5 deletions python/topology/jaeger.py
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ def _generate_dc(self):
'version': '2',
'services': {
'jaeger': {
'image': 'jaegertracing/all-in-one:1.12.0',
'image': 'jaegertracing/all-in-one:1.14.0',
'container_name': name,
'user': '%s:%s' % (str(os.getuid()), str(os.getgid())),
'ports': [
Expand All @@ -57,13 +57,10 @@ def _generate_dc(self):
],
'environment': [
'SPAN_STORAGE_TYPE=badger',
'BADGER_EPHEMERAL=false',
'BADGER_DIRECTORY_VALUE=/badger/data',
'BADGER_DIRECTORY_KEY=/badger/key',
'BADGER_CONSISTENCY=true',
],
'volumes': [
'%s:/badger' % self.docker_jaeger_dir,
'%s:/tmp' % self.docker_jaeger_dir,
],
}
}
Expand Down

0 comments on commit 298ab8c

Please sign in to comment.