fix: Make pretty_to_bytes return IP addresses as is, instead of throw…

…ing ValueError (#422)

Co-authored-by: Sverre H. Huseby <sverre.huseby@statnett.no>

changelogs/fragments/0-postgresql_set_avoid_handle_ip_addresses.yml

@@ -0,0 +1,2 @@
+bugfixes:
+- postgresql_set - avoid throwing ValueError for IP addresses and other values that may look like a number, but which are not (https://github.com/ansible-collections/community.postgresql/pull/422).

plugins/modules/postgresql_set.py

@@ -269,7 +269,11 @@ def pretty_to_bytes(pretty_val):
             pretty_val = int(pretty_val)
 
         except ValueError:
-            pretty_val = float(pretty_val)
+            try:
+                pretty_val = float(pretty_val)
+
+            except ValueError:
+                return pretty_val
 
         return pretty_val
 

tests/unit/plugins/modules/test_postgresql_set.py

@@ -21,6 +21,7 @@
     ('100MB', 104857600),
     ('1GB', 1073741824),
     ('10GB', 10737418240),
+    ('127.0.0.1', '127.0.0.1')
 ]
 )
 def test_pretty_to_bytes(input_, expected):