Page MenuHomeDevCentral
Differential D2640

Fix Vault certificates configuration
ClosedPublic
Actions

Authored by dereckson on Mar 26 2022, 15:49.
Tags
None
Referenced Files
F2843153: D2640.id6671.diff
Tue, Apr 23, 04:14
F2840333: D2640.id6668.diff
Mon, Apr 22, 20:03
F2840319: D2640.id6669.diff
Mon, Apr 22, 19:58
Unknown Object (File)
Sun, Apr 21, 21:38
Unknown Object (File)
Sun, Apr 21, 19:19
Unknown Object (File)
Fri, Apr 19, 15:35
Unknown Object (File)
Fri, Apr 19, 07:57
Unknown Object (File)
Fri, Apr 19, 01:31
View All Files
Subscribers
None

Details

Reviewers
dereckson
Commits
rOPS8f7fc92a914a: Fix Vault certificates configuration
Summary

salt["file.file_exists"]("/some/dir") returns False, so check a file will do.

Also, HCL defines options using the key = value syntax.

This is a follow-up for 7700b529.

Test Plan

salt-call --local state.sls roles/vault/vault test=True should be no-op on Complector.

Diff Detail

Repository
rOPS Nasqueron Operations
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

dereckson requested review of this revision.Mar 26 2022, 15:49
dereckson created this revision.
Harbormaster completed remote builds in B4139: Diff 6668.Mar 26 2022, 15:49
dereckson accepted this revision.Mar 26 2022, 15:49
This revision is now accepted and ready to land.Mar 26 2022, 15:49
dereckson updated this revision to Diff 6669.Mar 26 2022, 15:53

vault.hcl syntax error

Harbormaster completed remote builds in B4140: Diff 6669.Mar 26 2022, 15:53
dereckson accepted this revision.Mar 26 2022, 15:54
dereckson retitled this revision from Fix certificates_available for Vault configuration to Fix Vault certificates configuration.
dereckson edited the summary of this revision. (Show Details)
dereckson edited the test plan for this revision. (Show Details)
dereckson updated this revision to Diff 6670.Mar 26 2022, 15:57

Spacing

Harbormaster completed remote builds in B4141: Diff 6670.Mar 26 2022, 15:57
dereckson accepted this revision.Mar 26 2022, 15:57
Closed by commit rOPS8f7fc92a914a: Fix Vault certificates configuration (authored by dereckson). · Explain WhyMar 26 2022, 15:57
This revision was automatically updated to reflect the committed changes.
dereckson added a commit: rOPS8f7fc92a914a: Fix Vault certificates configuration.

Revision Contents
Changeset List

PathSize
roles/
vault/
vault/
files/
8 lines
2 lines

Diff 6671

View Options

roles/vault/vault/files/vault.hcl

Loading...
View Options

roles/vault/vault/init.sls

Loading...
Nasqueron DevCentral · If it had been much bigger the moon would have had a core of ice. · Powered by Phabricator