Refactor and consolidate Zemke-Rhyne unit
ClosedPublic
Actions

Authored by dereckson on Sep 8 2018, 17:57.

Details

Reviewers

dereckson

Maniphest Tasks

T1412: Improve Zemke-Rhyne deployment on Salt master

Commits

rOPS81312f759ab2: Refactor and consolidate Zemke-Rhyne unit

Summary

Strangely, Zemke-Rhyne was associated to the webserver-legacy role.

Actually, Zemke-Rhyne is mainly used to provision secrets
to Docker containers. So it's more logic to deploy it to
the Saltmaster role, as it's an integral part of deployment.

The .ssh/authorized_keys file will be reprovisonned each time
configuration (and so data/servers.json) changes.

Ref T1412

Test Plan

salt --local state.apply roles/saltmaster/zemke-rhyne

Diff Detail

Repository

rOPS Nasqueron Operations

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

dereckson requested review of this revision.Sep 8 2018, 17:57

dereckson created this revision.

Harbormaster completed remote builds in B2639: Diff 4298.Sep 8 2018, 17:57

dereckson edited the summary of this revision. (Show Details)Sep 8 2018, 17:57

dereckson added a task: T1412: Improve Zemke-Rhyne deployment on Salt master.

Tweak requisites

Harbormaster completed remote builds in B2641: Diff 4301.Sep 8 2018, 18:59

OK, but got a non repro 255 error code from PHP (ie, a PHP fatal error)

      ID: zr_authorized_keys
Function: cmd.run
    Name: make clean all
  Result: False
 Comment: Command "make clean all" run
 Started: 18:18:31.265016
Duration: 479.128 ms
 Changes:   
          ----------
          pid:
              65731
          retcode:
              1
          stderr:
          stdout:
              rm -f .ssh/authorized_keys
              mkdir -p -m 0700 .ssh
              php bin/getpublickeys > .ssh/authorized_keys
              *** Error code 255

              Stop.
              make: stopped in /usr/home/zr

This revision is now accepted and ready to land.Sep 8 2018, 19:04

Closed by commit rOPS81312f759ab2: Refactor and consolidate Zemke-Rhyne unit (authored by dereckson). · Explain WhySep 8 2018, 19:04

This revision was automatically updated to reflect the committed changes.