bookkeeper icon indicating copy to clipboard operation
bookkeeper copied to clipboard

[improve] Enable hostname specification as a script argument for topology script

Open shustsud opened this issue 2 months ago • 1 comments

Motivation

Currently, when using ScriptBasedMapping for network location resolution via the topology script, only IP addresses are passed to the script.
In some situations it is preferable to pass hostnames instead of IPs.
This PR introduces a configuration option to enable that behavior.

Changes

  • Added a new configuration key:
    networkTopologyScriptUseHostNameArgs
    • default: false
    • when set to true, ScriptBasedMapping passes hostnames instead of IP addresses to the topology script.
  • Updated CachedDNSToSwitchMapping and ScriptBasedMapping.RawScriptBasedMapping to respect this setting.
  • Added corresponding unit test:
    TestRackawareEnsemblePlacementPolicyUsingScript.testUseHostnameArgsOption

shustsud avatar Oct 03 '25 00:10 shustsud

rerun failure checks

shustsud avatar Nov 17 '25 07:11 shustsud