Feature #23729
closed
Provide better hostname generator examples
Description
Our discovery rule hostname example with rand() method do not work: abc-<%= @host.facts['bios_vendor'] + '-' + rand(99999) %>
Errors out with: TypeError (no implicit conversion of Integer into String) and "to_s" call must be added. Let's add that to the example screen.
We should also provide some new helpers and allow them in safemode:
- rand_hex(n) - returns SecureRandom.hex(n)
- rand_name - returns NameGenerator.new.next_random_name
- mac_name(mac_address) - returns NameGenerator.new.next_mac_name(mac_address)
Add those to the examples. Note NameGenerator is part of Foreman Core.
Updated by Lukas Zapletal almost 6 years ago
- Related to Feature #16330: Give reasonable names to discovered hosts via deacon added
Updated by Aditi Puntambekar over 5 years ago
I think the discovery rule hostname example with rand() method has already been taken care of in [[https://github.com/theforeman/foreman_discovery/commit/fe346a413d3a69fc9e878e768caea1119d055d10]].
So will this issue focus only on new helpers for name generation ? How do we plan to go ahead ?
Updated by Lukas Zapletal over 5 years ago
Yes, test the rest and add it to the doc text please.
Updated by Aditi Puntambekar over 5 years ago
- Related to Feature #25679: Provide better hostname generator examples with safemode enabled added
Updated by Aditi Puntambekar over 5 years ago
This issue has been fixed in Foreman core as [[https://projects.theforeman.org/issues/25679]]. Can this be closed then ?
Updated by Aditi Puntambekar almost 5 years ago
- Status changed from New to Closed