worker/instance: Refactor worker systemd initialization
ClosedPublic
Actions

Authored by ardumont on Feb 19 2019, 9:14 PM.

Details

Reviewers

olasd
douardda
vlorentz

Group Reviewers

Reviewers

Commits

rSPSITE31239c3a6889: indexer: Fix indexer content key names
rSPSITE7dd3e4478a7e: worker/instance: Refactor worker systemd initialization

Summary

Related T1531
Depends on D1154

Test Plan

bin/octocatalog ... uffizi	archiver
bin/octocatalog ... worker01	loader/lister
bin/octocatalog ... worker01.euwest.azure	indexer
bin/octocatalog ... worker11.euwest.azure	cooker

all went fine, as is, it wants to do the same things as in D1153/D1154
(depending on the targets)

Diff Detail

Repository

rSPSITE puppet-swh-site

Branch

update_worker_configuration

Lint

No Linters Available

Unit

No Unit Test Coverage

Build Status

Buildable 4312
Build 5693: arc lint + arc unit

Event Timeline

ardumont created this revision.Feb 19 2019, 9:14 PM

Harbormaster completed remote builds in B4312: Diff 3635.Feb 19 2019, 9:14 PM

Looks fine.

There's some inconsistency in the package declarations ('present'/'latest') that we may want to fix; I'm not sure which way though.

This revision is now accepted and ready to land.Feb 20 2019, 8:55 AM

Prepare merge

Harbormaster completed remote builds in B4325: Diff 3648.Feb 20 2019, 11:15 AM

There's some inconsistency in the package declarations ('present'/'latest') that we may want to fix; I'm not sure which way though.

We exchanged already on this matter a little while back.
We concluded to stop having services upgrade itself for now.
So that means, everything should be fixed to 'present'.
I'll do it after merging this.

Thanks for the heads up.

Landed (tip: dd71506)

ardumont mentioned this in D1158: manifest: Latest refactoring/cleanup .Feb 20 2019, 12:12 PM

ardumont mentioned this in rSPSITE0651de220ab5: services: Stop requiring the latest version for now.Feb 20 2019, 6:08 PM