Bug #16996

[crunch-dispatch-local] package missing service file

Added by Javier Bértoli about 1 year ago. Updated 9 months ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
Crunch
Target version:
Start date:
10/26/2020
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
-
Release relationship:
Auto

Description

The package crunch-dispatch-local has no service file, which is a known issue

I manually got it running with the following service file

# Copyright (C) The Arvados Authors. All rights reserved.
#
# SPDX-License-Identifier: AGPL-3.0

[Unit]
Description=Arvados Local Crunch Dispatcher Service
Documentation=https://doc.arvados.org/
After=network.target

# systemd==229 (ubuntu:xenial) obeys StartLimitInterval in the [Unit] section
StartLimitInterval=0

# systemd>=230 (debian:9) obeys StartLimitIntervalSec in the [Unit] section
StartLimitIntervalSec=0

[Service]
Type=notify
EnvironmentFile=-/etc/arvados/environment
ExecStart=/usr/bin/crunch-dispatch-local -poll-interval=1 -crunch-run-command=/usr/local/bin/crunch-run.sh
# Set a reasonable default for the open file limit
LimitNOFILE=65536
Restart=always
RestartSec=1
LimitNOFILE=1000000

# systemd<=219 (centos:7, debian:8, ubuntu:trusty) obeys StartLimitInterval in the [Service] section
StartLimitInterval=0

[Install]
WantedBy=multi-user.target

and a shell script /usr/local/bin/crunch-run.sh

#!/bin/sh
exec /usr/bin/crunch-run -container-enable-networking=default -container-network-mode=host $@

This can surely be improved but, at least, both these files should be added to the package.

Update - the shell script is unnecessary, crunch-run can be invoked directly, the default arguments for crunch-run are fine.


Subtasks

Task #17026: ReviewResolvedWard Vandewege


Related issues

Related to Arvados - Story #9952: crunch-dispatch-local config file and systemd unitClosed

Related to Arvados - Task #16545: Review 16471-install-using-saltResolved10/09/2020

Associated revisions

Revision e22b49be (diff)
Added by Javier Bértoli about 1 year ago

fix(crunch-dispatch-local): add missing service file

refs #16996
Arvados-DCO-1.1-Signed-off-by: Javier Bértoli <>

Revision e94e9b92 (diff)
Added by Javier Bértoli about 1 year ago

fix(crunch-dispatch-local): add missing service file

refs #16996
Arvados-DCO-1.1-Signed-off-by: Javier Bértoli <>

Revision 397981da
Added by Javier Bértoli about 1 year ago

Merge branch '16996-add-dispatch-local-service-file' into master

closes #16996
Arvados-DCO-1.1-Signed-off-by: Javier Bértoli <>

Revision 6be4c965 (diff)
Added by Javier Bértoli 9 months ago

fix(crunch-dispatch-local): add missing service file

refs #16996
Arvados-DCO-1.1-Signed-off-by: Javier Bértoli <>

History

#1 Updated by Javier Bértoli about 1 year ago

  • Related to Story #9952: crunch-dispatch-local config file and systemd unit added

#2 Updated by Javier Bértoli about 1 year ago

  • Description updated (diff)

#3 Updated by Peter Amstutz about 1 year ago

  • Target version changed from 2020-10-21 Sprint to 2020-11-04 Sprint

#4 Updated by Javier Bértoli about 1 year ago

  • Related to Task #16545: Review 16471-install-using-salt added

#5 Updated by Peter Amstutz about 1 year ago

  • Assigned To set to Javier Bértoli
  • Status changed from New to In Progress

#6 Updated by Javier Bértoli about 1 year ago

  • Status changed from In Progress to Feedback

Done in commit e94e9b924@arvados

#7 Updated by Ward Vandewege about 1 year ago

  • Description updated (diff)

#8 Updated by Ward Vandewege about 1 year ago

Javier Bértoli wrote:

Done in commit e94e9b924@arvados

LGTM thanks.

#9 Updated by Javier Bértoli about 1 year ago

  • Status changed from Feedback to Resolved

Merged

#10 Updated by Peter Amstutz 9 months ago

  • Release set to 37

Also available in: Atom PDF