aci: pylint
This commit is contained in:
parent
a531ce199d
commit
3e148556e6
1 changed files with 12 additions and 16 deletions
|
@ -85,7 +85,6 @@ class ACIImage(TarImageFormatter):
|
||||||
|
|
||||||
return isolators
|
return isolators
|
||||||
|
|
||||||
|
|
||||||
@staticmethod
|
@staticmethod
|
||||||
def _get_docker_config_value(docker_config, key, default_value):
|
def _get_docker_config_value(docker_config, key, default_value):
|
||||||
# Try the key itself.
|
# Try the key itself.
|
||||||
|
@ -100,7 +99,6 @@ class ACIImage(TarImageFormatter):
|
||||||
|
|
||||||
return default_value
|
return default_value
|
||||||
|
|
||||||
|
|
||||||
@staticmethod
|
@staticmethod
|
||||||
def _build_ports(docker_config):
|
def _build_ports(docker_config):
|
||||||
""" Builds the ports definitions for the ACI. """
|
""" Builds the ports definitions for the ACI. """
|
||||||
|
@ -143,7 +141,6 @@ class ACIImage(TarImageFormatter):
|
||||||
def _build_volumes(docker_config):
|
def _build_volumes(docker_config):
|
||||||
""" Builds the volumes definitions for the ACI. """
|
""" Builds the volumes definitions for the ACI. """
|
||||||
volumes = []
|
volumes = []
|
||||||
names = set()
|
|
||||||
|
|
||||||
def get_name(docker_volume_path):
|
def get_name(docker_volume_path):
|
||||||
return "volume-%s" % ACIImage._ac_name(docker_volume_path)
|
return "volume-%s" % ACIImage._ac_name(docker_volume_path)
|
||||||
|
@ -159,7 +156,6 @@ class ACIImage(TarImageFormatter):
|
||||||
})
|
})
|
||||||
return volumes
|
return volumes
|
||||||
|
|
||||||
|
|
||||||
@staticmethod
|
@staticmethod
|
||||||
def _build_manifest(namespace, repository, tag, docker_layer_data, synthetic_image_id):
|
def _build_manifest(namespace, repository, tag, docker_layer_data, synthetic_image_id):
|
||||||
""" Builds an ACI manifest from the docker layer data. """
|
""" Builds an ACI manifest from the docker layer data. """
|
||||||
|
@ -186,18 +182,18 @@ class ACIImage(TarImageFormatter):
|
||||||
"acVersion": "0.6.1",
|
"acVersion": "0.6.1",
|
||||||
"name": '%s/%s/%s' % (hostname.lower(), namespace.lower(), repository.lower()),
|
"name": '%s/%s/%s' % (hostname.lower(), namespace.lower(), repository.lower()),
|
||||||
"labels": [
|
"labels": [
|
||||||
{
|
{
|
||||||
"name": "version",
|
"name": "version",
|
||||||
"value": tag,
|
"value": tag,
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
"name": "arch",
|
"name": "arch",
|
||||||
"value": docker_layer_data.get('architecture', 'amd64')
|
"value": docker_layer_data.get('architecture', 'amd64')
|
||||||
},
|
},
|
||||||
{
|
{
|
||||||
"name": "os",
|
"name": "os",
|
||||||
"value": docker_layer_data.get('os', 'linux')
|
"value": docker_layer_data.get('os', 'linux')
|
||||||
}
|
}
|
||||||
],
|
],
|
||||||
"app": {
|
"app": {
|
||||||
"exec": exec_path,
|
"exec": exec_path,
|
||||||
|
|
Reference in a new issue