Page MenuHomeDevCentral
Differential D1270

Remove core.hostnames pillar
ClosedPublic
Actions

Authored by dereckson on Dec 1 2017, 12:11.
Tags
None
Referenced Files
F10329307: D1270.diff
Sun, Jun 29, 03:26
Unknown Object (File)
Mon, Jun 23, 11:10
Unknown Object (File)
Thu, Jun 19, 03:54
Unknown Object (File)
Tue, Jun 17, 18:18
Unknown Object (File)
Tue, Jun 17, 17:52
Unknown Object (File)
Tue, Jun 17, 16:52
Unknown Object (File)
Tue, Jun 17, 10:13
Unknown Object (File)
Fri, Jun 13, 18:29
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