Page MenuHomeSoftware Heritage

defaults: Move gpg/certificate blocks to a dedicated config file
ClosedPublic

Authored by ardumont on Aug 5 2019, 6:54 PM.

Details

Summary

To:

  • avoid further errors when touching needed blank spaces
  • reduce the defaults.yaml's already too big size
Test Plan

bin/octocatalog <node>
Nothing new

Also tested live in staging environment and same. Nothing to complain.

Diff Detail

Repository
rSPSITE puppet-swh-site
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

data/defaults.yaml
1

I don't like that name, i'll rename it to defaults_security.yaml.
We are dealing with public key/cert block, this is security related.
We are specifying that we trust those.

Rename defaults_gpg.yaml to defaults_security.yaml

I've checked back (after the new modification) and bin/octocatalog is still happy.
It still does not want to touch anything.

data/defaults.yaml
1

that name

was defaults_gpg.yaml...

This revision is now accepted and ready to land.Aug 6 2019, 10:49 AM

as @anlambert mentioned to me, maybe puppet could load those blocks from files, but let's check that later ;)

Looks good to me. This is helpful as I can't count the number of times the auto-trimming feature of my text editor removed those needed spaces.

I am also wondering if each GPG key / certificate could be put in a separate file then read by puppet and integrated in the YAML file.

This revision was automatically updated to reflect the committed changes.