Page MenuHomeDevCentral
Differential D1270

Remove core.hostnames pillar
ClosedPublic
Actions

Authored by dereckson on Dec 1 2017, 12:11.
Tags
None
Referenced Files
F7695160: D1270.id3265.diff
Thu, May 1, 05:42
Unknown Object (File)
Tue, Apr 29, 18:43
Unknown Object (File)
Mon, Apr 28, 15:19
Unknown Object (File)
Sun, Apr 27, 19:14
Unknown Object (File)
Fri, Apr 25, 10:24
Unknown Object (File)
Tue, Apr 22, 13:12
Unknown Object (File)
Tue, Apr 22, 11:53
Unknown Object (File)
Tue, Apr 22, 05:07
View All Files
Subscribers
None

Details

Reviewers
dereckson
Commits
rOPS336e12bd0071: Remove core.hostnames pillar
Summary

b112ea76 introduced nodes definition, including hostnames,
so it's possible to directly get the hostname from the node
entry instead of a dedicated pillar entry.

This change partially reverts b88a308fec2d.
This change reverts 866436e489cd.

Test Plan

salt * state.apply roles/core/hostname test=True

Diff Detail

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

Event Timeline

dereckson created this revision.Dec 1 2017, 12:11
Harbormaster completed remote builds in B2019: Diff 3265.Dec 1 2017, 12:11
dereckson accepted this revision.Dec 1 2017, 12:12
dereckson edited the summary of this revision. (Show Details)
dereckson edited the summary of this revision. (Show Details)
This revision is now accepted and ready to land.Dec 1 2017, 12:12
Closed by commit rOPS336e12bd0071: Remove core.hostnames pillar (authored by dereckson). · Explain WhyDec 1 2017, 12:13
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
pillar/
core/
1 line
roles/
core/
hostname/
2 lines

Diff 3266

View Options

pillar/core/hostnames.sls

Loading...
View Options

pillar/top.sls

Loading...
View Options

roles/core/hostname/init.sls

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