mgr/cephadm: SNMP: don't write urls manually

this just broken for non-trivial urls. Don't be a bad example

Signed-off-by: Sebastian Wagner <sewagner@redhat.com>
(cherry picked from commit 3f47c2293b9ace730d6f76c613ef2106f274ea32)

diff --git a/src/pybind/mgr/cephadm/services/monitoring.py b/src/pybind/mgr/cephadm/services/monitoring.py
index c252462c212eb04ea0e95755fabf00d26448517c..d67d3057bb80765d3ac74fe9c58b57781b2c4e41 100644 (file)
--- a/src/pybind/mgr/cephadm/services/monitoring.py
+++ b/src/pybind/mgr/cephadm/services/monitoring.py
@@ -158,7 +158,9 @@ class AlertmanagerService(CephadmService):
             assert dd.ports
             addr = dd.ip if dd.ip else self._inventory_get_addr(dd.hostname)
             deps.append(dd.name())
-            snmp_gateway_urls.append(f"http://{addr}:{dd.ports[0]}/alerts")
+
+            snmp_gateway_urls.append(build_url(scheme='http', host=addr,
+                                     port=dd.ports[0], path='/alerts'))
 
         context = {
             'dashboard_urls': dashboard_urls,