heptapod.py 24.9 KB
Newer Older
1
import contextlib
2
import json
3
4
from io import BytesIO
import logging
5
import os
6
7
8
from pathlib import Path
import requests
import selenium.webdriver
9
import shutil
10
11
12
import subprocess
import sys
import tarfile
13
import tempfile
14
15
16
17
18
19
import time
from urllib.parse import urlparse

from tests.utils.user import User
from tests.utils import docker
from tests.utils import session
20
from .constants import DATA_DIR
21
22
23

logger = logging.getLogger(__name__)

24
INITIAL_TIMEOUT = 600  # seconds
25
26
27
28
29
30
31
32
33

BRANCH_PROTECTIONS = dict(none=0,
                          dev_can_push=1,
                          full=2,
                          dev_can_merge=3,
                          )


class Heptapod:
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
    """Base class and minimum common control of Heptapod server.

    This is used directly in case `--heptapod-remote` is selected.
    """

    fs_access = False
    """True if manipulation of files is possible.

    Implies :attr:`repositories_root` not to be None.
    """

    shell_access = False
    """True if running arbitrary commands as in a shell is possible."""

    repositories_root = None
    """Path to the repositories, from the Heptapod server point of view.

    This is potentially not meaningful on the system that runs the test,
    see :class:`DockerHeptapod` for an example.
    """
54

55
56
57
58
    reverse_call_host = None
    """Network address for the system running these tests, seen from Heptapod.
    """

59
60
61
    webdriver_remote_url = None
    """URL for a Selenium RemoteWebDriver."""

62
63
64
65
    wait_after_first_response = 0
    """Time to wait after we've had a first non error HTTP response.
    """

66
67
    hg_executable = 'hg'

68
69
    chrome_driver_args = ()

70
    def __init__(self, url, ssh_user, ssh_port,
71
                 hg_native=False,
72
                 reverse_call_host=None,
73
                 wait_after_first_response=0,
74
                 webdriver_remote_url=None):
75
76
77
        self.parsed_url = urlparse(url)
        self.url = url
        self.ssh_port = ssh_port
78
        self.ssh_user = ssh_user
79
        self.hg_native = hg_native
80
        self.users = {}
81
82
        if reverse_call_host is not None:
            self.reverse_call_host = reverse_call_host
83
        self.dead = None
84
        self.webdriver_remote_url = webdriver_remote_url
85
        self.wait_after_first_response = wait_after_first_response
86
        self.settings = {}
87

88
89
90
91
    @property
    def heptapod(self):
        return self

92
93
    @property
    def ssh_url(self):
94
        return 'ssh://{user}@{host}:{port}'.format(
95
            host=self.host,
96
            user=self.ssh_user,
97
98
99
            port=self.ssh_port,
        )

100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
    @property
    def host(self):
        return self.parsed_url.netloc.rsplit(':', 1)[0]

    @property
    def api_url(self):
        return '/'.join((self.url, 'api', 'v4'))

    @property
    def root_token_headers(self):
        return {'Private-Token': self.users['root']['token']}

    @property
    def basic_user_token_headers(self):
        return {'Private-Token': self.users['test_basic']['token']}

116
117
118
119
120
121
122
123
    def run_shell(self, command, **kw):
        exit_code, output = self.execute(command, **kw)
        if exit_code != 0:
            raise RuntimeError(
                ('Heptapod command {command} returned a non-zero '
                 'exit code {exit_code}').format(
                     command=command,
                     exit_code=exit_code,
124
                ))
125
126
        return output

127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
    def get_user(self, name):
        """Return a :class:`User` instance, or `None`."""
        info = self.users.get(name)
        if info is None:
            return None
        user_id = info.get('id')
        password = info.get('password')

        if user_id is None:
            logger.info(
                "Searching for known user %r because its id is unknown.",
                name)
            user = User.search(self, name)
            user.password = password
            user.store_in_heptapod()
            return user

        return User(heptapod=self, id=user_id, name=name, password=password)

146
147
    def new_webdriver(self):
        options = selenium.webdriver.ChromeOptions()
148
149
        for arg in self.chrome_driver_args:
            options.add_argument(arg)
150
151
152
153
154
155
156
157
158
        options.add_argument('--headless')
        if self.webdriver_remote_url:
            return selenium.webdriver.Remote(
                command_executor=self.webdriver_remote_url,
                options=options
            )
        else:
            return selenium.webdriver.Chrome(options=options)

159
    def get_user_webdriver(self, user_name):
160
161
162
163
164
165
166
167
        info = self.users[user_name]
        driver = info.get('webdriver')
        if driver is not None:
            return driver

        logger.info("Initializing a signed-in webdriver for user %r",
                    user_name)

168
169
170
171
172
173
174
175
176
177
178
        if user_name == 'root':
            driver = self.new_webdriver()
            # guaranteeing driver to be available for teardown
            # as soon as created
            info['webdriver'] = driver
            session.login_as_root(driver, self, info['password'])
        else:
            # TODO should init webdriver from here and store it before login
            # attempt as well
            driver = session.make_webdriver(self, user_name, info['password'])
            info['webdriver'] = driver
179
        return driver
180

181
182
183
184
185
186
187
188
189
190
191
    def wait_startup(self, first_response_timeout=INITIAL_TIMEOUT,
                     wait_after_first_response=None):
        """Wait for Heptapod to be ready after startup.

        We have to take into account that the server may have just started
        (that's frequent in development and it's annoying for a human to
        wait) or could even be starting from scratch, configuring itself,
        creating the DB schema etc. (typical of CI).
        In that latter case, an amount of extra wait after the first successful
        HTTP response is often needed.
        """
192
        logger.info("Waiting for Heptapod to answer requests")
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
        dead_msg = ("Heptapod server did not "
                    "respond in %s seconds" % first_response_timeout)
        start = time.time()
        while time.time() < start + first_response_timeout:
            try:
                resp = requests.get(self.url, allow_redirects=False)
            except IOError:
                resp = None

            if resp is None:
                logger.debug("Couldn't reach Heptapod")
            elif resp.status_code >= 400:
                logger.debug("Heptapod response code %r", resp.status_code)
            else:
                logger.info("Heptapod is up")
                self.dead = False
                if wait_after_first_response:
                    logger.info("Waiting additional %d seconds "
                                "after first successful HTTP call",
                                wait_after_first_response)
                    time.sleep(wait_after_first_response)
                return

            duration = 1
            logger.debug("Retrying in %.1f seconds", duration)
            time.sleep(duration)

        self.dead = True
        raise AssertionError(dead_msg)

223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
    def instance_cache_file(self):
        return os.path.join(DATA_DIR, 'instance.cache')

    def load_instance_cache(self):
        path = self.instance_cache_file()
        try:
            with open(path) as cachef:
                cached = json.load(cachef)

        except Exception:
            logger.info("Cache file %r not available or not readable. "
                        "Heptapod instance info will be retrieved "
                        "or initialized", path)
        else:
            for name, info in cached['users'].items():
                self.users[name] = dict(name=name,
                                        token=info['token'])

    def update_instance_cache(self):
        users = {name: dict(token=info['token'])
                 for name, info in self.users.items()}
        with open(self.instance_cache_file(), 'w') as cachef:
            json.dump(dict(users=users), cachef)

247
248
249
250
251
252
253
254
255
256
257
    def prepare(self, root_password):
        """Make all preparations for the Heptapod instance to be testable.

        This currently amounts to

        - defining the root password
        - activating and retrieving a root API token
        - creating a persistent `test_basic` user
        - activating and retrieving an API token for `test_basic`
        - keeping a logged in webdriver for each persistent user
        """
258
        assert not self.dead, "Heptapod server marked dead by previous test."
259
260

        if self.dead is None:
261
262
            self.wait_startup(
                wait_after_first_response=self.wait_after_first_response)
263
264
        self.load_instance_cache()

265
        logger.info("Preparing root user.")
266
        start = time.time()
267
268
        root = User.init_root(self, root_password)
        root.ensure_private_token()
269

270
        logger.info("Preparing application settings.")
271
272
273
274
        # necessary if we want to listen to web hooks from these tests
        # in GitLab v12.2.0, this is deprecated for ...from_web_hooks...
        self.set_application_settings(
            allow_local_requests_from_hooks_and_services="true")
275

276
277
278
279
280
        # Mercurial native projects (vcs_types='hg') arent allowed by
        # default
        if self.hg_native and 'hg' not in self.settings.get('vcs_types', ()):
            self.set_vcs_types_settings(['hg', 'git', 'hg_git'])

281
        logger.info("Preparing basic user.")
282
283
284
        session.ensure_user(self, 'test_basic',
                            fullname='Bäsîc Test',
                            password='test_basic')
285
286

        logger.info("Uploading users SSH keys.")
287
288
289
290
291
292
        self.load_ssh_keys()
        self.upload_ssh_pub_keys()
        subprocess.call((
            'ssh-keygen',
            '-R', '[{host}]:{port}'.format(
                host=self.host, port=self.ssh_port)))
293
        self.update_instance_cache()
294
295
        logger.info("All preparations done in %.2f seconds. "
                    "Proceeding with tests.", time.time() - start)
296
297

    def set_application_settings(self, **settings):
298
        """Change GitLab application settings and update :attr:`settings`."""
299
300
301
302
303
304
        resp = requests.put(
            self.api_url + '/application/settings',
            headers=self.root_token_headers,
            data=settings,
        )
        assert resp.status_code == 200
305
306
307
308
309
310
311
312
313
314
        self.settings = resp.json()

    def sync_application_settings(self):
        """Update :attr:`settings` for current values"""
        resp = requests.get(
            self.api_url + '/application/settings',
            headers=self.root_token_headers,
        )
        assert resp.status_code == 200
        self.settings = resp.json()
315

316
317
318
319
320
321
322
323
324
325
326
327
328
329
330
331
332
333
334
335
336
337
338
339
340
341
342
343
344
345
346
347
348
349
350
    def api_request(self, method, user=None, subpath='', **kwargs):
        """Perform a simple API HTTP request

        `method` is the HTTP method to use, same as in `requests.request`.

        The full URL is made of the API URL of the instance, together with
        the given subpath (example 'snippets/42').

        Appropriate authentication headers are added on the fly.

        :param user: the :class:`User` to run the request as. If not specified,
                     the request is sent as the root user.

        All other kwargs are passed to `requests.request()`
        """
        headers = kwargs.pop('headers', {})
        token = self.owner_token if user is None else user.token
        headers['Private-Token'] = token
        return requests.request(method,
                                '/'.join((self.api_url, subpath)),
                                headers=headers,
                                **kwargs)

    def api_get(self, **kwargs):
        return self.api_request('GET', **kwargs)

    def api_post(self, **kwargs):
        return self.api_request('POST', **kwargs)

    def api_put(self, **kwargs):
        return self.api_request('PUT', **kwargs)

    def api_delete(self, **kwargs):
        return self.api_request('DELETE', **kwargs)

351
352
353
354
355
356
357
358
359
360
361
362
363
364
365
366
367
368
369
370
371
372
    def load_ssh_keys(self):
        """Load client-side information to use SSH keys

        Also makes sure the keys are actually usable (perms)
        """
        ssh_dir = Path(__file__).parent.parent / 'data' / 'ssh'
        for name, info in self.users.items():
            base_fname = 'id_rsa_heptapod_' + name
            priv = ssh_dir / base_fname
            pub = ssh_dir / (base_fname + '.pub')

            # VCSes tend not to preserve non-executable perm bits
            priv.chmod(0o600)
            info['ssh'] = dict(priv=str(priv), pub=pub.read_text())

    def upload_ssh_pub_keys(self):
        """Upload SSH public keys for all users to Heptapod."""
        # it's really time to put the actual user object in our `self.users`
        for name, info in self.users.items():
            user = User.search(self, name)
            user.ensure_ssh_pub_key(info['ssh']['pub'])

373
    def close_webdrivers(self):
374
        for user in self.users.values():
375
            driver = user.pop('webdriver', None)
376
            if driver is not None:
377
                logger.info("Closing webdriver for user %r", user['name'])
378
379
                driver.close()

380
381
382
383
384
    def close(self):
        if self.dead is not False:
            return
        self.close_webdrivers()

385
386
    def execute(self, command, **kw):
        raise NotImplementedError('execute')
387
388
389
390
391
392

    def force_remove_route(self, route_path, source_type='Project'):
        logger.error("Attempt to force-remove route %r, not implemented "
                     "for %r", route_path, self.__class__)
        raise NotImplementedError('force_remove_route')

393
394
395
396
397
398
399
400
401
402
403
404
    def gitlab_ctl(self, command, services=None):
        """Apply service management command.

        'command' would typically be 'start', 'stop', etc.

        :param services: an iterable of service names (who can themselves
                         be different depending on the concrete subclass).
                         If supplied, the command will apply only to those
                         services.
        """
        raise NotImplementedError('gitlab_ctl')

405
406
407
    def rake(self, *args):
        """Call GitLab Rake"""
        raise NotImplementedError('rake')
408

409
410
411
412
413
414
415
416
417
418
419
420
421
422
423
424
425
    def remove_all_backups(self):
        """Remove all existing backups with no impediment for new backups.
        """
        raise NotImplementedError('remove_all_backups')

    def backup_create(self, clean_previous=True):
        """Create a new backup

        :param bool clean_previous: if true, any previously existing backups
            are removed. This is handy so that the restore rake task knows
            which one to restore without any need to tell it.
        """
        if clean_previous:
            self.remove_all_backups()
        self.rake('gitlab:backup:create')

    @contextlib.contextmanager
426
    def backup_restore(self):
427
428
429
430
431
432
433
434
        """Context manager for backups restoration.

        This is a context manager as a way to provide resuming of the
        tests session on errors, in a convenient way for the caller.
        That means ensuring as much as possible that the server is running,
        maybe wait again for it, reinitialize passwords and tokens…
        """
        try:
435
            self.gitlab_ctl('stop', services=self.RAILS_SERVICES)
436
            self.rake('gitlab:backup:restore', 'force=yes')
437
            self.gitlab_ctl('start', services=self.RAILS_SERVICES)
438
439
440
441
442
443

            self.wait_startup()
            yield

        except Exception:
            # these are idempotent
444
            self.gitlab_ctl('start', services=self.RAILS_SERVICES)
445
446
447
448
449

            # Worst case scenario, we lost all our data. We need to
            # reprepare the server for subsequent tests
            self.prepare(self.users['root']['password'])
            raise
450

451
452
453
    def set_vcs_types_settings(self, vcs_types):
        self.set_application_settings(vcs_types=','.join(vcs_types))

454
455
456
457
458
459
460
461
462
463
464
465
466
467
    def apply_hashed_storage_setting(self, hashed_storage):
        # TODO it would be tempting not to restart if the setting is already
        # at the wished value, but this currently cannot take into account
        # rollbacked values that aren't followed by a restart. This will
        # be more complicated and take more time than we can afford right now
        # to really make work.
        self.set_application_settings(hashed_storage_enabled=hashed_storage)

        # let's be sure that redis is restarted when the Rails services
        # start
        self.gitlab_ctl('stop', self.RAILS_SERVICES)
        self.gitlab_ctl('stop', ['redis'])
        # we restart everything in case a service would depend on Redis
        # and would fail to reconnect automatically
468
469
470

        # closing all webdrivers, because restart of Redis will kill sessions
        self.close_webdrivers()
471
472
473
474
475
476
477
        self.gitlab_ctl('restart')
        self.wait_startup()

        # recheck that the setting is applied
        self.sync_application_settings()
        assert self.settings['hashed_storage_enabled'] is hashed_storage

478

479
class OmnibusHeptapod(Heptapod):
480

481
482
483
484
    fs_access = True

    shell_access = True

485
486
    repositories_root = '/var/opt/gitlab/git-data/repositories'

487
488
    backups_dir = '/var/opt/gitlab/backups'

489
    RAILS_SERVICES = ('puma', 'sidekiq')
490

491
492
493
494
495
496
    gitlab_ctl_command = ('sudo', 'gitlab-ctl')

    reverse_call_host = 'localhost'

    ssh_url = 'ssh://git@localhost'

497
    # TODO read from conf for logical dependency loosening.
498
499
    # Not an immediate priority, since we're not concerned about Python 2
    # any more (see heptapod#353)
500
501
    hg_executable = '/opt/gitlab/embedded/bin/hg'

502
503
    chrome_driver_args = ('--no-sandbox', )

504
505
506
507
508
509
510
511
512
513
514
515
516
517
518
519
520
521
522
523
524
525
526
527
528
529
530
531
532
533
534
535
536
537
538
539
540
541
542
543
544
545
546
547
548
549
550
551
552
553
554
555
556
557
558
559
560
561
562
563
564
565
    def execute(self, command, user='git'):
        if user != 'git':
            raise NotImplementedError(
                "On Omnibus Heptapod, only 'git' user is allowed")

        logger.debug("OmnibusHeptapod: executing command %r", command)
        process = subprocess.Popen(command,
                                   stdout=subprocess.PIPE,
                                   stderr=subprocess.PIPE)
        out, err = [o.decode() for o in process.communicate()]
        retcode = process.poll()

        if out:
            sys.stdout.write(out)
        if err:
            sys.stderr.write(err)
        return retcode, out

    def put_archive(self, dest, path, owner='git'):
        # this assumes owner has write access to parent directory,
        # but that should be enough, and avoids some processing as root
        self.run_shell(['tar', '-xf', path, '-C', dest], user=owner)

    def put_archive_bin(self, dest, fobj):
        with tempfile.NamedTemporaryFile() as tempf:
            tempf.write(fobj.read())
            self.put_archive(dest, tempf.name)

    def put_file_lines(self, path, lines, uid=998, gid=998):
        with open(path, 'w') as fobj:
            fobj.writelines(lines)
        os.chown(path, uid, gid)

    def get_file_lines(self, path):
        with open(path) as fobj:
            return fobj.readlines()

    def remove_all_backups(self):
        # using find in order not to rely on shell expansion for *.tar
        self.run_shell(('find', '/var/opt/gitlab/backups',
                        '-name', '*.tar',
                        '-delete'))

    def rake(self, *args):
        cmd = ['gitlab-rake']
        cmd.extend(args)
        code, out = self.execute(cmd, user='git')
        return out.encode()  # Consistency with HDK (returns bytes)

    def gitlab_ctl(self, command, services=None):
        base_cmd = self.gitlab_ctl_command + (command, )
        if services is None:
            self.run_shell(base_cmd)
        else:
            for service in services:
                self.run_shell(base_cmd + (service, ))


class DockerHeptapod(OmnibusHeptapod):

    gitlab_ctl_command = ('gitlab-ctl', )

566
567
    git_executable = 'git'

568
569
    chrome_driver_args = ()

570
571
572
    def __init__(self, docker_container, **kw):
        super(DockerHeptapod, self).__init__(**kw)
        self.docker_container = docker_container
573
574
        if self.reverse_call_host is None:
            self.reverse_call_host = docker.host_address(docker_container)
575
576
577
578

    def execute(self, command, user='root'):
        return docker.heptapod_exec(self.docker_container, command, user=user)

579
580
    def run_shell(self, command, **kw):
        return docker.heptapod_run_shell(self.docker_container, command, **kw)
581

582
583
584
585
    @property
    def ssh_url(self):
        return super(OmnibusHeptapod, self).ssh_url

586
587
588
589
590
591
592
593
594
595
596
597
598
599
600
601
602
603
604
605
606
607
608
609
610
611
612
613
614
615
616
617
618
619
620
621
    def put_archive(self, dest, path, owner='git'):
        res = docker.heptapod_put_archive(self.docker_container, dest, path)
        self.run_shell(['chown', '-R', 'git:root', dest])
        return res

    def put_archive_bin(self, dest, fobj):
        return docker.heptapod_put_archive_bin(
            self.docker_container, dest, fobj)

    def get_archive(self, path, tarf):
        return docker.heptapod_get_archive(self.docker_container, path, tarf)

    def put_file_lines(self, path, lines, uid=998, gid=998):
        dirpath, filename = path.rsplit('/', 1)
        tar_buf = BytesIO()
        tarf = tarfile.open(mode='w:', fileobj=tar_buf)

        tinfo = tarfile.TarInfo(name='hgrc')
        contents_buf = BytesIO()
        contents_buf.writelines(l.encode() for l in lines)
        tinfo.size = contents_buf.tell()
        tinfo.uid, tinfo.gid = uid, gid
        contents_buf.seek(0)
        tarf.addfile(tinfo, fileobj=contents_buf)

        tar_buf.seek(0)
        self.put_archive_bin(dirpath, tar_buf)

    def get_file_lines(self, path):
        dirname, filename = path.rsplit('/', 1)
        buf = BytesIO()
        self.get_archive(path, buf)
        buf.seek(0)
        tarf = tarfile.open(mode='r:', fileobj=buf)
        return [l.decode() for l in tarf.extractfile(filename).readlines()]

622
623
624
625
626
627
628
629
630
631
632
633
634
    def force_remove_route(self, route_path, source_type='Project'):
        """Delete a route from the database.

        Sometimes GitLab fails to clean Project routes after failed tests.
        """
        logger.warn("Cleaning up leftover route at %r", route_path)
        self.run_shell([
            'gitlab-psql', 'gitlabhq_production',
            '-c', "DELETE FROM routes "
            "WHERE source_type='%s' "
            "  AND path='%s'" % (source_type, route_path)
        ])

635
636

class SourceHeptapod(Heptapod):
637
638
639
640
641
642
643
644
645
    """An Heptapod server installed from source on the same system.

    Same system means without using any container technology (Docker or
    otherwise) that would insulate the tests from the server.
    """

    fs_access = True

    shell_access = True
646

647
648
    reverse_call_host = 'localhost'

649
650
651
652
653
654
655
656
657
658
659
660
661
662
663
664
    @property
    def ssh_url(self):
        return 'ssh://{host}:{port}'.format(
            host=self.host,
            port=self.ssh_port,
        )

    def __init__(self, repositories_root, **kw):
        super(SourceHeptapod, self).__init__(**kw)
        self.repositories_root = repositories_root

    def execute(self, command, user='git'):
        if user != 'git':
            raise NotImplementedError(
                "On source Heptapod, only same user as for Rails and HgServe "
                "is allowed")
665
        logger.debug("SourceHeptapod: executing command %r", command)
666
667
668
669
670
671
672
673
674
675
676
677
678
679
680
681
682
683
684
685
686
687
688
689
690
691
        process = subprocess.Popen(command,
                                   stdout=subprocess.PIPE,
                                   stderr=subprocess.PIPE)
        out, err = [o.decode() for o in process.communicate()]
        retcode = process.poll()

        if out:
            sys.stdout.write(out)
        if err:
            sys.stderr.write(err)
        return retcode, out

    def put_archive(self, dest, path, owner='git'):
        if owner != 'git':
            raise NotImplementedError(
                "On source Heptapod, only same owner as for Rails and HgServe "
                "is allowed")
        subprocess.check_call(['tar', 'xf', path], cwd=dest)

    def get_file_lines(self, path):
        with open(path, 'r') as fobj:
            return fobj.readlines()

    def put_file_lines(self, path, lines):
        with open(path, 'w') as fobj:
            fobj.writelines(lines)
692
693
694
695
696
697
698
699
700
701
702
703
704
705
706
707
708
709
710
711
712
713
714
715
716
717
718
719
720
721
722
723
724
725


class GdkHeptapod(SourceHeptapod):
    """An Heptapod server running with the GDK.
    """

    fs_access = True

    shell_access = True

    reverse_call_host = 'localhost'

    RAILS_SERVICES = ('rails-web', 'rails-background-jobs')

    def __init__(self, gdk_root, **kw):
        self.gdk_root = gdk_root
        self.rails_root = os.path.join(gdk_root, 'gitlab')
        super(GdkHeptapod, self).__init__(
            repositories_root=os.path.join(gdk_root, 'repositories'),
            **kw)

    @property
    def backups_dir(self):
        return os.path.join(self.rails_root, 'tmp', 'backups')

    def remove_all_backups(self):
        if os.path.exists(self.backups_dir):
            shutil.rmtree(self.backups_dir)
        # as of GitLab 12.10, parent dir is always present
        os.mkdir(self.backups_dir)

    def rake(self, *args):
        cmd = ['bundle', 'exec', 'rake']
        cmd.extend(args)
726
        logger.debug("GdkHeptapod: calling %r", cmd)
727
        return subprocess.check_output(cmd, cwd=self.rails_root)
728

729
730
731
732
    def gitlab_ctl(self, command, services=None):
        base_cmd = ('gdk', command)

        def do_command(*opt_args):
733
734
735
            cmd = base_cmd + opt_args
            logger.debug("GdkHeptapod: calling %r", cmd)
            subprocess.check_call(cmd, cwd=self.rails_root)
736
737
738
739
740
741

        if services is None:
            do_command()
        else:
            for service in services:
                do_command(service)