diff options
| author | diogo464 <[email protected]> | 2025-10-30 20:03:37 +0000 |
|---|---|---|
| committer | diogo464 <[email protected]> | 2025-10-30 20:03:37 +0000 |
| commit | 2bfd274bf31ed74f690bb5d4087c6a433ab83a06 (patch) | |
| tree | 6fb707941aac04aa9737b127bd6c666197713326 /src/main.rs | |
| parent | 6e4b2e481305efdd7caf096ecca99a7060a9c0d1 (diff) | |
fix: dont panic when unable to obtain hostname
if we are unable to obtain the hostname of the local machine from the
/etc/hostname file or HOSTNAME env var then an empty string is used as
the hostname and a warning is shown instead of a panic. since the
hostname is just used to determine if we are executing on the cluster,
and the cluster machines all have their hostnames set, then using the
empty string should not be a problem.
Diffstat (limited to 'src/main.rs')
| -rw-r--r-- | src/main.rs | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/main.rs b/src/main.rs index da113c1..6e12d6f 100644 --- a/src/main.rs +++ b/src/main.rs | |||
| @@ -1033,7 +1033,8 @@ fn machine_generate_configs( | |||
| 1033 | AddressAllocationPolicy::Total(n) => { | 1033 | AddressAllocationPolicy::Total(n) => { |
| 1034 | let mut counter = 0; | 1034 | let mut counter = 0; |
| 1035 | while counter < *n { | 1035 | while counter < *n { |
| 1036 | let machine = machines[(counter as usize) % machines.len()]; | 1036 | let machine = machines[(counter as usize) % machines.len()]; // TODO: proper error |
| 1037 | // message for panic here | ||
| 1037 | let address = machine_address_for_idx(machine, counter / (machines.len() as u32)); | 1038 | let address = machine_address_for_idx(machine, counter / (machines.len() as u32)); |
| 1038 | addresses.push(address); | 1039 | addresses.push(address); |
| 1039 | counter += 1; | 1040 | counter += 1; |
