saltstack/salt
1
0
Fork 0
mirror of https://github.com/saltstack/salt.git synced 2025-04-17 10:10:20 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

Pull missing images when calling dockerng.running

Browse source 
fixes #29727
This commit is contained in:
Nicolas Delaby 2016-02-19 15:43:52 +01:00
parent 98a14f8090
commit 8c86eeb4dc
2 changed files with 21 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
salt/states/dockerng.py
View file

@ -1457,6 +1457,20 @@ def running(name,
except TypeError:
image = ':'.join(_get_repo_tag(str(image)))
if image not in __salt__['dockerng.list_tags']():
try:
# Pull image
pull_result = __salt__['dockerng.pull'](
image,
client_timeout=client_timeout,
)
except Exception as exc:
comments = ['Failed to pull {0}: {1}'.format(image, exc)]
ret['comment'] = _format_comments(comments)
return ret
else:
ret['changes']['image'] = pull_result
image_id = __salt__['dockerng.inspect_image'](image)['Id']
if name not in __salt__['dockerng.list_containers'](all=True):

11
tests/unit/states/dockerng_test.py
View file

@ -376,7 +376,8 @@ class DockerngTestCase(TestCase):
__salt__ = {'dockerng.list_containers': dockerng_list_containers,
'dockerng.inspect_container': dockerng_inspect_container,
'dockerng.inspect_image': dockerng_inspect_image,
'dockerng.list_tags': MagicMock(),
'dockerng.list_tags': MagicMock(
return_value=['image:latest']),
'dockerng.pull': MagicMock(),
'dockerng.state': MagicMock(side_effect=['stopped',
'running']),
@ -466,7 +467,7 @@ class DockerngTestCase(TestCase):
'dockerng.inspect_container': dockerng_inspect_container,
'dockerng.inspect_image': dockerng_inspect_image,
'dockerng.list_tags': MagicMock(),
'dockerng.pull': MagicMock(),
'dockerng.pull': MagicMock(return_value=True),
'dockerng.state': MagicMock(side_effect=['stopped',
'running']),
'dockerng.create': dockerng_create,
@ -482,7 +483,8 @@ class DockerngTestCase(TestCase):
self.assertEqual(ret, {'name': 'cont',
'comment': "Container 'cont' changed state.",
'changes': {'state': {'new': 'running',
'old': 'stopped'}},
'old': 'stopped'},
'image': True},
'result': True,
})
@ -508,7 +510,8 @@ class DockerngTestCase(TestCase):
- KEY: "1"
'''
__salt__ = {'dockerng.list_containers': MagicMock(),
'dockerng.list_tags': MagicMock(),
'dockerng.list_tags': MagicMock(
return_value=['image:latest']),
'dockerng.inspect_image': MagicMock(),
'dockerng.pull': MagicMock(),
'dockerng.state': MagicMock(),