SUSE / caasp-salt

A collection of salt states used to provision a kubernetes cluster
Apache License 2.0
64 stars 29 forks source link

[3.0] Generate the /etc/hosts file with a custom state #691

Closed inercia closed 5 years ago

inercia commented 5 years ago

(Backport of #616)

Generate the /etc/hosts file from a state, merging our entries with previously found entries.

(Users are instructed with a comment in /etc/hosts to add their custom entries in /etc/caasp/hosts)

See https://bugzilla.suse.com/show_bug.cgi?id=1098334

bsc#1098334

mjura commented 5 years ago

I have re-ran CI, we have to merge this PR, it is needed also to fix http://bugzilla.suse.com/show_bug.cgi?id=1097294

bergmannf commented 5 years ago

I currently don't have enough RAM free on my machine to try this backport, as I am already working on another update-related bug.

Can someone verify that this backport does not add an admin alias to one of the other nodes in the /etc/hosts file - I remember seeing this before.

mjura commented 5 years ago

CI finished successfully, we need this change!