1
0
Эх сурвалжийг харах

Don't set the hostname to the service name with networking.

Signed-off-by: Daniel Nephin <[email protected]>
Daniel Nephin 10 жил өмнө
parent
commit
ba61a6c5fb

+ 0 - 3
compose/service.py

@@ -605,9 +605,6 @@ class Service(object):
             container_options['hostname'] = parts[0]
             container_options['hostname'] = parts[0]
             container_options['domainname'] = parts[2]
             container_options['domainname'] = parts[2]
 
 
-        if 'hostname' not in container_options and self.use_networking:
-            container_options['hostname'] = self.name
-
         if 'ports' in container_options or 'expose' in self.options:
         if 'ports' in container_options or 'expose' in self.options:
             ports = []
             ports = []
             all_ports = container_options.get('ports', []) + self.options.get('expose', [])
             all_ports = container_options.get('ports', []) + self.options.get('expose', [])

+ 0 - 1
tests/integration/cli_test.py

@@ -226,7 +226,6 @@ class CLITestCase(DockerClientTestCase):
             containers = service.containers()
             containers = service.containers()
             self.assertEqual(len(containers), 1)
             self.assertEqual(len(containers), 1)
             self.assertIn(containers[0].id, network['Containers'])
             self.assertIn(containers[0].id, network['Containers'])
-            self.assertEqual(containers[0].get('Config.Hostname'), service.name)
 
 
         web_container = self.project.get_service('web').containers()[0]
         web_container = self.project.get_service('web').containers()[0]
         self.assertFalse(web_container.get('HostConfig.Links'))
         self.assertFalse(web_container.get('HostConfig.Links'))

+ 0 - 10
tests/unit/service_test.py

@@ -213,16 +213,6 @@ class ServiceTest(unittest.TestCase):
         opts = service._get_container_create_options({'image': 'foo'}, 1)
         opts = service._get_container_create_options({'image': 'foo'}, 1)
         self.assertIsNone(opts.get('hostname'))
         self.assertIsNone(opts.get('hostname'))
 
 
-    def test_hostname_defaults_to_service_name_when_using_networking(self):
-        service = Service(
-            'foo',
-            image='foo',
-            use_networking=True,
-            client=self.mock_client,
-        )
-        opts = service._get_container_create_options({'image': 'foo'}, 1)
-        self.assertEqual(opts['hostname'], 'foo')
-
     def test_get_container_create_options_with_name_option(self):
     def test_get_container_create_options_with_name_option(self):
         service = Service(
         service = Service(
             'foo',
             'foo',