Enable IPv6 proxy when settings.server.is_ipv6=True in IPv6 runs (#16795

) * Enable IPv6 proxy for puppet tests in IPv6 runs Signed-off-by: Gaurav Talreja <[email protected]> * Remove use of enable_proxy option and enable proxy as per setting for IPv6 Signed-off-by: Gaurav Talreja <[email protected]> --------- Signed-off-by: Gaurav Talreja <[email protected]> (cherry picked from commit a79da44)
SatelliteQE · Nov 7, 2024 · b249aea · b249aea
1 parent 1e1b6b7
commit b249aea
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 22 deletions.
diff --git a/pytest_fixtures/core/sat_cap_factory.py b/pytest_fixtures/core/sat_cap_factory.py
@@ -351,7 +351,7 @@ def installer_satellite(request):
     sat.setup_firewall()
 
     # register to cdn (also enables rhel repos from cdn)
-    sat.register_to_cdn(enable_proxy=True)
+    sat.register_to_cdn()
 
     # setup source repositories
     if settings.server.version.source == "ga":

diff --git a/robottelo/hosts.py b/robottelo/hosts.py
@@ -739,7 +739,6 @@ def register_contenthost(
         auto_attach=False,
         serverurl=None,
         baseurl=None,
-        enable_proxy=False,
     ):
         """Registers content host on foreman server either by specifying
         organization name and activation key name or by specifying organization
@@ -1428,15 +1427,21 @@ def install_tracer(self):
             raise ContentHostError('There was an error installing katello-host-tools-tracer')
         self.execute('katello-tracer-upload')
 
-    def register_to_cdn(self, pool_ids=None, enable_proxy=False):
+    def register_to_cdn(self, pool_ids=None):
         """Subscribe satellite to CDN"""
         self.reset_rhsm()
+
+        # Enabling proxy for IPv6
+        if settings.server.is_ipv6:
+            url = urlparse(settings.server.http_proxy_ipv6_url)
+            self.enable_rhsm_proxy(url.hostname, url.port)
+            self.enable_dnf_proxy(url.hostname, url.scheme, url.port)
+
         cmd_result = self.register_contenthost(
             org=None,
             lce=None,
             username=settings.subscription.rhn_username,
             password=settings.subscription.rhn_password,
-            enable_proxy=enable_proxy,
         )
         if cmd_result.status != 0:
             raise ContentHostError(
@@ -1602,6 +1607,7 @@ def enable_ipv6_http_proxy(self):
         if settings.server.is_ipv6:
             url = urlparse(settings.server.http_proxy_ipv6_url)
             self.enable_rhsm_proxy(url.hostname, url.port)
+            self.enable_dnf_proxy(url.hostname, url.scheme, url.port)
             self.ipv6 = settings.server.is_ipv6
 
     def disable_ipv6_http_proxy(self):
@@ -2351,24 +2357,6 @@ def sync_inventory_status(self, org):
         )
         return inventory_sync
 
-    def register_contenthost(
-        self,
-        org='Default_Organization',
-        lce='Library',
-        username=settings.server.admin_username,
-        password=settings.server.admin_password,
-        enable_proxy=False,
-    ):
-        """Satellite Registration to CDN"""
-        # Enabling proxy for IPv6
-        if enable_proxy and settings.server.is_ipv6:
-            url = urlparse(settings.server.http_proxy_ipv6_url)
-            self.enable_rhsm_proxy(url.hostname, url.port)
-            self.enable_dnf_proxy(url.hostname, url.scheme, url.port)
-        return super().register_contenthost(
-            org=org, lce=lce, username=username, password=password, enable_proxy=enable_proxy
-        )
-
     def run_orphan_cleanup(self, smart_proxy_id=None):
         """Run orphan cleanup task for all or given smart proxy."""
         timestamp = datetime.utcnow().replace(microsecond=0)