Merge pull request #1862 from mrfuxi/costom-container-name-in-logs

Use custom container name in logs
This commit is contained in:
Aanand Prasad 2015-09-08 14:36:28 -07:00
commit 85709d85d2
2 changed files with 13 additions and 1 deletions

View File

@ -6,6 +6,7 @@ from functools import reduce
import six
from .const import LABEL_CONTAINER_NUMBER
from .const import LABEL_PROJECT
from .const import LABEL_SERVICE
@ -70,7 +71,12 @@ class Container(object):
@property
def name_without_project(self):
return '{0}_{1}'.format(self.service, self.number)
project = self.labels.get(LABEL_PROJECT)
if self.name.startswith('{0}_{1}'.format(project, self.service)):
return '{0}_{1}'.format(self.service, self.number)
else:
return self.name
@property
def number(self):

View File

@ -83,9 +83,15 @@ class ContainerTest(unittest.TestCase):
self.assertEqual(container.name, "composetest_db_1")
def test_name_without_project(self):
self.container_dict['Name'] = "/composetest_web_7"
container = Container(None, self.container_dict, has_been_inspected=True)
self.assertEqual(container.name_without_project, "web_7")
def test_name_without_project_custom_container_name(self):
self.container_dict['Name'] = "/custom_name_of_container"
container = Container(None, self.container_dict, has_been_inspected=True)
self.assertEqual(container.name_without_project, "custom_name_of_container")
def test_inspect_if_not_inspected(self):
mock_client = mock.create_autospec(docker.Client)
container = Container(mock_client, dict(Id="the_id"))