Bug #15538

Update component .service files

Added by Eric Biagiotti 11 months ago. Updated 6 months ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
-
Target version:
Start date:
11/12/2019
Due date:
% Done:

100%

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

Description

Component .service files (e.g. arvados-ws.service) have AssertPathExists, pointing to legacy config files. Update these to point to the cluster config, since we recommend deleting legacy config files.


Subtasks

Task #15806: Review 15538 No BranchResolvedLucas Di Pentima


Related issues

Related to Arvados - Story #13648: [Epic] Use one cluster configuration file for all componentsResolved

Associated revisions

Revision 7e23ed42 (diff)
Added by Eric Biagiotti 8 months ago

Removes arvados-docker-cleaner commented out service file assert

refs #15538

Arvados-DCO-1.1-Signed-off-by: Eric Biagiotti <>

History

#1 Updated by Eric Biagiotti 11 months ago

  • Related to Story #13648: [Epic] Use one cluster configuration file for all components added

#2 Updated by Tom Clegg 8 months ago

  • Target version set to 2019-11-20 Sprint

#3 Updated by Eric Biagiotti 8 months ago

  • Assigned To set to Eric Biagiotti

#4 Updated by Eric Biagiotti 8 months ago

  • Status changed from New to In Progress

Just searched the repo for "AssertPathExists" and I get the following:

  • arvados-controller.service, arvados-dispatch-cloud.service, and arvados-health.service all assert that the new cluster config exists at /etc/arvados/config.yml. This is correct since these only work with the cluster config.
  • arvados-node-manager.service - Checks for /etc/arvados-node-manager/config.ini. This is also correct since node-manager is going away and isn't in the cluster config.

Subsequently, no changes are needed.

#5 Updated by Lucas Di Pentima 8 months ago

Nit: you could remove the commented AssertPathExists on services/dockercleaner/arvados-docker-cleaner.service.

Other than that, it seems that this issue was gradually fixed from other related stories, thanks!

#6 Updated by Eric Biagiotti 8 months ago

Lucas Di Pentima wrote:

Nit: you could remove the commented AssertPathExists on services/dockercleaner/arvados-docker-cleaner.service.

Other than that, it seems that this issue was gradually fixed from other related stories, thanks!

Removed @ 7e23ed42b6e4eab1dd5e39ece5b0830b740bab1e

#7 Updated by Eric Biagiotti 8 months ago

  • Status changed from In Progress to Resolved

#8 Updated by Peter Amstutz 6 months ago

  • Release set to 22

Also available in: Atom PDF