]> git-server-git.apps.pok.os.sepia.ceph.com Git - ceph.git/commitdiff
mgr/cephadm: fix typo with vrrp_interfaces in keepalive setup 61663/head
authorAdam King <adking@redhat.com>
Wed, 5 Feb 2025 22:00:06 +0000 (17:00 -0500)
committerAdam King <adking@redhat.com>
Wed, 5 Feb 2025 22:09:17 +0000 (17:09 -0500)
This was intended to be vrrp_interfaces, the variable actually
used later in the code. Instead, due to a typo, it was setting
a variable that is unused other than in the log line right after
it is set. Issue was introduced in
https://github.com/ceph/ceph/commit/58ddc4e20f7cead1f2594241450f4beb5230c746

Signed-off-by: Adam King <adking@redhat.com>
src/pybind/mgr/cephadm/services/ingress.py

index 60fc586da85c65582b3330e7f9c3f49492ee4805..14d3a01b7753b26b364213448ca81df060fc799a 100644 (file)
@@ -394,10 +394,10 @@ class IngressService(CephService):
         else:
             for subnet, ifaces in self.mgr.cache.networks.get(host, {}).items():
                 if subnet == spec.vrrp_interface_network:
-                    vrrp_interface = [list(ifaces.keys())[0]] * len(interfaces)
+                    vrrp_interfaces = [list(ifaces.keys())[0]] * len(interfaces)
                     logger.info(
                         f'vrrp will be configured on {host} interface '
-                        f'{vrrp_interface} (which is in subnet {subnet})'
+                        f'{vrrp_interfaces} (which is in subnet {subnet})'
                     )
                     break
             else: