Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Support for Docker run command arguments. Fixes #1967 #1968

Merged
merged 1 commit into from Sep 25, 2018
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
23 changes: 23 additions & 0 deletions src/rockstor/storageadmin/migrations/0006_dcontainerargs.py
@@ -0,0 +1,23 @@
# -*- coding: utf-8 -*-
from __future__ import unicode_literals

from django.db import migrations, models


class Migration(migrations.Migration):

dependencies = [
('storageadmin', '0005_auto_20180913_0923'),
]

operations = [
migrations.CreateModel(
name='DContainerArgs',
fields=[
('id', models.AutoField(verbose_name='ID', serialize=False, auto_created=True, primary_key=True)),
('name', models.CharField(max_length=1024)),
('val', models.CharField(max_length=1024, blank=True)),
('container', models.ForeignKey(to='storageadmin.DContainer')),
],
),
]
2 changes: 1 addition & 1 deletion src/rockstor/storageadmin/models/__init__.py
Expand Up @@ -45,7 +45,7 @@
from tls_certificate import TLSCertificate # noqa E501
from rockon import (RockOn, DImage, DContainer, DPort, DVolume, # noqa E501
ContainerOption, DCustomConfig, DContainerLink, # noqa E501
DContainerEnv, DContainerDevice) # noqa E501
DContainerEnv, DContainerDevice, DContainerArgs) # noqa E501
from smart import (SMARTAttribute, SMARTCapability, SMARTErrorLog, # noqa E501
SMARTErrorLogSummary, SMARTTestLog, SMARTTestLogDetail, # noqa E501
SMARTIdentity, SMARTInfo) # noqa E501
Expand Down
9 changes: 9 additions & 0 deletions src/rockstor/storageadmin/models/rockon.py
Expand Up @@ -126,6 +126,15 @@ class Meta:
app_label = 'storageadmin'


class DContainerArgs(models.Model):
container = models.ForeignKey(DContainer)
name = models.CharField(max_length=1024)
val = models.CharField(max_length=1024, blank=True)

class Meta:
app_label = 'storageadmin'


class DCustomConfig(models.Model):
rockon = models.ForeignKey(RockOn)
key = models.CharField(max_length=1024)
Expand Down
18 changes: 17 additions & 1 deletion src/rockstor/storageadmin/views/rockon.py
Expand Up @@ -24,7 +24,7 @@
from storageadmin.models import (RockOn, DImage, DContainer, DPort, DVolume,
ContainerOption, DCustomConfig,
DContainerLink, DContainerEnv,
DContainerDevice)
DContainerDevice, DContainerArgs)
from storageadmin.serializers import RockOnSerializer
from storageadmin.util import handle_exception
import rest_framework_custom as rfc
Expand Down Expand Up @@ -320,6 +320,22 @@ def _create_update_meta(self, name, r_d):
if (oo.id not in id_l):
oo.delete()

cmd_args = containers[c].get('cmd_arguments', [])
id_l = []
for ca in cmd_args:
# there are no unique constraints on this model, so we need
# this bandaid.
if (DContainerArgs.objects.filter(
container=co, name=ca[0], val=ca[1]).count() > 1):
DContainerArgs.objects.filter(
container=co, name=ca[0], val=ca[1]).delete()
cao, created = DContainerArgs.objects.get_or_create(
container=co, name=ca[0], val=ca[1])
id_l.append(cao.id)
for cao in DContainerArgs.objects.filter(container=co):
if (cao.id not in id_l):
cao.delete()

l_d = r_d.get('container_links', {})
for cname in l_d:
ll = l_d[cname]
Expand Down
12 changes: 11 additions & 1 deletion src/rockstor/storageadmin/views/rockon_helpers.py
Expand Up @@ -26,7 +26,7 @@
from storageadmin.models import (RockOn, DContainer, DVolume, DPort,
DCustomConfig, DContainerLink,
ContainerOption, DContainerEnv,
DContainerDevice)
DContainerDevice, DContainerArgs)
from fs.btrfs import mount_share
from rockon_utils import container_status
import logging
Expand Down Expand Up @@ -206,6 +206,15 @@ def vol_owner_uid(container):
return os.stat(share_mnt).st_uid


def cargs(container):
cargs_list = []
for c in DContainerArgs.objects.filter(container=container):
cargs_list.append(c.name)
if (len(c.val.strip()) > 0):
cargs_list.append(c.val)
return cargs_list


def envars(container):
var_list = []
for e in DContainerEnv.objects.filter(container=container):
Expand Down Expand Up @@ -233,6 +242,7 @@ def generic_install(rockon):
cmd.extend(container_ops(c))
cmd.extend(envars(c))
cmd.append(c.dimage.name)
cmd.extend(cargs(c))
run_command(cmd)


Expand Down