qontract-reconcile 0.10.2.dev183__py3-none-any.whl → 0.10.2.dev184__py3-none-any.whl

{qontract_reconcile-0.10.2.dev183.dist-info → qontract_reconcile-0.10.2.dev184.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: qontract-reconcile
-Version: 0.10.2.dev183
+Version: 0.10.2.dev184
 Summary: Collection of tools to reconcile services with their desired state as defined in the app-interface DB.
 Project-URL: homepage, https://github.com/app-sre/qontract-reconcile
 Project-URL: repository, https://github.com/app-sre/qontract-reconcile

{qontract_reconcile-0.10.2.dev183.dist-info → qontract_reconcile-0.10.2.dev184.dist-info}/RECORD

@@ -15,7 +15,7 @@ reconcile/closedbox_endpoint_monitoring_base.py,sha256=al7m8EgnnYx90rY1REryW3byN
 reconcile/cluster_deployment_mapper.py,sha256=5gumAaRCcFXsabUJ1dnuUy9WrP_FEEM5JnOnE8ch9sE,2326
 reconcile/dashdotdb_base.py,sha256=83ZWIf5JJk3P_D69y2TmXRcQr6ELJGlv10OM0h7fJVs,4767
 reconcile/dashdotdb_cso.py,sha256=QRK0YfIqO4rehs8btD3l_GXIO2ZIycTQEKEthBdB0xA,3639
-reconcile/dashdotdb_dora.py,sha256=hMh0Eq6ze5llWGoDV3l4OGcWY5OeJdS6Xy4b1Y4C_MY,17855
+reconcile/dashdotdb_dora.py,sha256=juZvr7R45f3TLbIZy290Oa1OtsSnqX-Ez71ppYAUjXw,17780
 reconcile/dashdotdb_dvo.py,sha256=lCkZ0iby6HrNQb-3kYb6xrt8wCjVUZYxKzz9SiStfHU,8946
 reconcile/dashdotdb_slo.py,sha256=TvKdMOtUZcZP9QydcUJMKh0zURHgOMN_RTpQpCkD1Z8,3960
 reconcile/database_access_manager.py,sha256=Z3aAmw2LsmMIIor-bOGzziVZdVNC82Gmw8oHBUAFf-8,25577
@@ -26,7 +26,7 @@ reconcile/gcp_image_mirror.py,sha256=1ThuUff_04ZdF6uxcLoDuHhoNA3OIw0V-z0-CwdPE2w
 reconcile/github_org.py,sha256=Wc5cZamatuWsW2ZJT2ib5ps8l3iY3RXHwNUxVJerqz0,14173
 reconcile/github_owners.py,sha256=viE1KJ-zaTxuZ5yItg2C263J0brn-Q-3hR_DkYDMbhY,3122
 reconcile/github_repo_invites.py,sha256=U9UCzNVwrZ7MqODtFah8ogH0NNY-XjBin7G9gqHtCUY,2690
-reconcile/github_repo_permissions_validator.py,sha256=ske7cHJ-41jnaywgRlDnR2z9E1yLDSJ0tivq6bhhZdc,1827
+reconcile/github_repo_permissions_validator.py,sha256=PNqL4dqa2OaNBy-NmLVN-t1HZa6eS6HgSYmfOunYqtA,1798
 reconcile/github_users.py,sha256=QdX164LZrm8sqggMj-0beCzWofpS6OEBfzKNrWPrfj0,3934
 reconcile/github_validator.py,sha256=-j17tn3csFVjPMSPL3te48iWVkPZCncRXdeKeLdGjjQ,931
 reconcile/gitlab_fork_compliance.py,sha256=RbHckzLnE9zkOFHJANzoejEMMbMAivmqJVs3Suvp9lU,4591
@@ -102,10 +102,10 @@ reconcile/resource_template_tester.py,sha256=DsKvBuNLPxm4Fa-e1YHHySnhThm5i_j-nF3
 reconcile/run_integration.py,sha256=8kc-lMKF9n2bvyrItJ-3nOgQMqK7lh1UAubXQrxQDPY,9711
 reconcile/saas_file_validator.py,sha256=tyvFYU6lnkfDYIkAIr5pWqSvO5Yc6TagZ-quJYD2dtI,2547
 reconcile/sendgrid_teammates.py,sha256=oO8QbLb4s1o8A6CGiCagN9CmS05BSS_WLztuY0Ym9D8,4773
-reconcile/service_dependencies.py,sha256=S8DOputtxAX-MXvWuTLEOsgLlVkXJHALODPSnWOpvGk,4500
+reconcile/service_dependencies.py,sha256=G2qCuYFc8wQLpRxkdhmibxSAl3nUM3hcan4x50W_mCA,4335
 reconcile/signalfx_endpoint_monitoring.py,sha256=Nqgsg1cflSd2nNnm89y_e8c--7xLUqTrKOHkDs-qADE,2868
 reconcile/slack_base.py,sha256=I-msunWxfgu5bSwXYulGbtLjxUB_tRmTCAUCU-3nabI,3484
-reconcile/slack_usergroups.py,sha256=vMifpbnrQDLeckGtUmpIg7sVvlhpaJz8HZH_loA7fpY,30221
+reconcile/slack_usergroups.py,sha256=xFkVe67RXSUj8JvpfSFEiRdQzB0TnJJEHW_b5PEwLng,30213
 reconcile/sql_query.py,sha256=OEzEZaqgv-kzG3GR2x9w3uMIfSFXP6EdhlW4u5mc1Dg,25895
 reconcile/status.py,sha256=cY4IJFXemhxptRJqR4qaaOWqei9e4jgLXuVSGajMsjg,544
 reconcile/status_board.py,sha256=kJ0bus_wdyX3zsFJuUPrH4n9BNG_jhDbiQ3waOLVRBE,8538
@@ -616,7 +616,7 @@ reconcile/utils/external_resources.py,sha256=YzTb0xAcNdmKO326mGQy7BmST56CZcdru4l
 reconcile/utils/filtering.py,sha256=S4PbMHuFr3ED0P2Q_ea5CAaB7FimI62B-F5YTaKrphA,402
 reconcile/utils/git.py,sha256=o4p9m8jlzCJDcutl2HErvGLhL6sZ1NB4Aw3zGcQIzso,2427
 reconcile/utils/github_api.py,sha256=o4J0ZU1ZSr9808uoorKHv19iae-eLo85yrCZX67p2kw,2822
-reconcile/utils/gitlab_api.py,sha256=qQMN9JwaUvbirbLBy1BImkeJ4iFR-OCfWExphrfkmXs,28610
+reconcile/utils/gitlab_api.py,sha256=U7l26fM1lQ4g-O06D0ZawvOXF_yzzH5NdR5iDx2ZJFA,28616
 reconcile/utils/gpg.py,sha256=EKG7_fdMv8BMlV5yUdPiqoTx-KrzmVSEAl2sLkaKwWI,1123
 reconcile/utils/gql.py,sha256=C0thIm_k9MBldfqwHzyqtYZk9sIvMdm9IbbnXLGwjD8,14158
 reconcile/utils/grouping.py,sha256=vr9SFHZ7bqmHYrvYcEZt-Er3-yQYfAAdq5sHLZVmXPY,456
@@ -667,12 +667,12 @@ reconcile/utils/state.py,sha256=az4tBmZ0EdbFcAGiBVUxs3cr2-BVWsuDQiNTvjjQq8s,1637
 reconcile/utils/structs.py,sha256=LcbLEg8WxfRqM6nW7NhcWN0YeqF7SQzxOgntmLs1SgY,352
 reconcile/utils/template.py,sha256=wTvRU4AnAV_o042tD4Mwls2dwWMuk7MKnde3MaCjaYg,331
 reconcile/utils/terraform_client.py,sha256=IDlrNvGEc2i6ElZIL_fzaJEad1nRC3DkP9_VXhJXmU0,37329
-reconcile/utils/terrascript_aws_client.py,sha256=-FNQXltzl4OYGO7eMyvH7-wYUMHNFEVhiNwCvhN6WrI,292307
+reconcile/utils/terrascript_aws_client.py,sha256=RkiYjRietHFNXtfA1-WEZ1lZbJFBA_XCtTOsZUij5VM,292360
 reconcile/utils/three_way_diff_strategy.py,sha256=oQcHXd9LVhirJfoaOBoHUYuZVGfyL2voKr6KVI34zZE,4833
 reconcile/utils/throughput.py,sha256=iP4UWAe2LVhDo69mPPmgo9nQ7RxHD6_GS8MZe-aSiuM,344
 reconcile/utils/vault.py,sha256=aSA8l9cJlPUHpChFGl27nSY-Mpq9FMjBo7Dcgb1BVfM,15036
 reconcile/utils/vaultsecretref.py,sha256=0KUSzuvTRxPyKY919TO3-B_eYg4_76fzKvMF8j5s1G0,911
-reconcile/utils/vcs.py,sha256=_S_QZ5UYXOnhNDOBQ-jNVOB6VpYh7Y96YLsM5T3ivug,8836
+reconcile/utils/vcs.py,sha256=AK35vIjx9bXYclKmvNekpaG_OETt-ZybibwV-m123xc,10186
 reconcile/utils/acs/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJWZG3hSuFU,0
 reconcile/utils/acs/base.py,sha256=4UsDrCpAOuddL3PKNuIQYoJP1BtZQNNB8_KEX0lXneg,2532
 reconcile/utils/acs/notifiers.py,sha256=DlzTDM9arWQlBSiDy70y5Mf38OKVs9V0FzFe2LfOKXA,5046
@@ -761,7 +761,7 @@ reconcile/utils/runtime/sharding.py,sha256=r0ieUtNed7NvknSw6qQrCkKpVXE1shuHGnfFc
 reconcile/utils/saasherder/__init__.py,sha256=3U8plqMAPRE1kjwZ5YnIsYsggTf4_gS7flRUEuXVBAs,343
 reconcile/utils/saasherder/interfaces.py,sha256=NEYQspYfyWQhBeJyNCqSFbixi1A4wRVGB7FeNM5BDCk,9141
 reconcile/utils/saasherder/models.py,sha256=JaOz_DEtudJZhiDe90kaBlJkppFufn81V92oK9PHYx0,10208
-reconcile/utils/saasherder/saasherder.py,sha256=PjwJ36Eki5X0Wqu7-sE1slApMh6th3NojAWfZpOz_OQ,87111
+reconcile/utils/saasherder/saasherder.py,sha256=V8DOaMvY1POYd_EquIlU2sI62l64C-PjtaTCuzwVbn4,87306
 reconcile/utils/terraform/__init__.py,sha256=zNbiyTWo35AT1sFTElL2j_AA0jJ_yWE_bfFn-nD2xik,250
 reconcile/utils/terraform/config.py,sha256=5UVrd563TMcvi4ooa5JvWVDW1I3bIWg484u79evfV_8,164
 reconcile/utils/terraform/config_client.py,sha256=gRL1rQ0AqvShei_rcGqC3HDYGskOFKE1nPrJyJE9yno,4676
@@ -807,7 +807,7 @@ tools/saas_promotion_state/__init__.py,sha256=47DEQpj8HBSa-_TImW-5JCeuQeRkm5NMpJ
 tools/saas_promotion_state/saas_promotion_state.py,sha256=UfwwRLS5Ya4_Nh1w5n1dvoYtchQvYE9yj1VANt2IKqI,3925
 tools/sre_checkpoints/__init__.py,sha256=CDaDaywJnmRCLyl_NCcvxi-Zc0hTi_3OdwKiFOyS39I,145
 tools/sre_checkpoints/util.py,sha256=zEDbGr18ZeHNQwW8pUsr2JRjuXIPz--WAGJxZo9sv_Y,894
-qontract_reconcile-0.10.2.dev183.dist-info/METADATA,sha256=2wN6rP5VfQjaP3NLAKUMYzPS5K4iHPrhgKKruSonpk4,24627
-qontract_reconcile-0.10.2.dev183.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
-qontract_reconcile-0.10.2.dev183.dist-info/entry_points.txt,sha256=5i9l54La3vQrDLAdwDKQWC0iG4sV9RRfOb1BpvzOWLc,698
-qontract_reconcile-0.10.2.dev183.dist-info/RECORD,,
+qontract_reconcile-0.10.2.dev184.dist-info/METADATA,sha256=4cOL2ogbG9Fe9GL834sDkyleO4EK2cgWR-AoP0ZIuls,24627
+qontract_reconcile-0.10.2.dev184.dist-info/WHEEL,sha256=qtCwoSJWgHk21S1Kb4ihdzI2rlJ1ZKaIurTj_ngOhyQ,87
+qontract_reconcile-0.10.2.dev184.dist-info/entry_points.txt,sha256=5i9l54La3vQrDLAdwDKQWC0iG4sV9RRfOb1BpvzOWLc,698
+qontract_reconcile-0.10.2.dev184.dist-info/RECORD,,

reconcile/dashdotdb_dora.py

@@ -34,7 +34,7 @@ from reconcile.typed_queries.saas_files import get_saas_files
 from reconcile.utils.github_api import GithubRepositoryApi
 from reconcile.utils.gitlab_api import GitLabApi
 from reconcile.utils.secret_reader import create_secret_reader
-from reconcile.utils.vcs import GITHUB_BASE_URL
+from reconcile.utils.vcs import VCS
 
 QONTRACT_INTEGRATION = "dashdotdb-dora"
 
@@ -422,19 +422,21 @@ class DashdotdbDORA(DashdotdbBase):
             return rc, []
 
         LOG.info("Fetching commits %s", rc)
-        if rc.repo_url.startswith(GITHUB_BASE_URL):
-            try:
-                commits = self._github_compare_commits(rc)
-            except GithubException as e:
-                if e.status == 404:
-                    LOG.info(
-                        f"Ignoring RepoChanges for {rc} because could not calculate them: {e.data['message']}"
-                    )
-                    return rc, []
-        elif rc.repo_url.startswith(self.gl.server):
-            commits = self._gitlab_compare_commits(rc)
-        else:
-            raise Exception(f"Unknown git hosting {rc.repo_url}")
+        repo_info = VCS.parse_repo_url(rc.repo_url)
+        match repo_info.platform:
+            case "github":
+                try:
+                    commits = self._github_compare_commits(rc, repo_info.name)
+                except GithubException as e:
+                    if e.status == 404:
+                        LOG.info(
+                            f"Ignoring RepoChanges for {rc} because could not calculate them: {e.data['message']}"
+                        )
+                        return rc, []
+            case "gitlab":
+                commits = self._gitlab_compare_commits(rc, repo_info.name)
+            case _:
+                raise Exception(f"Unknown git hosting {rc.repo_url}")
 
         return rc, commits
 
@@ -455,13 +457,10 @@ class DashdotdbDORA(DashdotdbBase):
 
         return app_env, datetime.fromisoformat(response.json()["finish_timestamp"])
 
-    def _gitlab_compare_commits(self, rc: RepoChanges) -> list[Commit]:
+    def _gitlab_compare_commits(self, rc: RepoChanges, repo: str) -> list[Commit]:
         if not rc.repo_url or not rc.ref_from or not rc.ref_to:
             return []
 
-        prefix = "https://gitlab.cee.redhat.com/"
-        repo = rc.repo_url[rc.repo_url.startswith(prefix) and len(prefix) :]
-
         commits = self.gl.repository_compare(repo, rc.ref_from, rc.ref_to)
 
         return [
@@ -473,12 +472,10 @@ class DashdotdbDORA(DashdotdbBase):
             for commit in commits
         ]
 
-    def _github_compare_commits(self, rc: RepoChanges) -> list[Commit]:
+    def _github_compare_commits(self, rc: RepoChanges, repo: str) -> list[Commit]:
         if not rc.repo_url:
             return []
 
-        repo = rc.repo_url.removeprefix(GITHUB_BASE_URL).rstrip("/")
-
         return [
             Commit(rc.repo_url, commit.sha, commit.commit.committer.date)
             for commit in self.gh_api(repo).compare(rc.ref_from, rc.ref_to)

reconcile/github_repo_permissions_validator.py

@@ -11,7 +11,7 @@ from reconcile.jenkins_job_builder import init_jjb
 from reconcile.utils.jjb_client import JJB
 from reconcile.utils.secret_reader import SecretReader
 from reconcile.utils.semver_helper import make_semver
-from reconcile.utils.vcs import GITHUB_BASE_URL
+from reconcile.utils.vcs import VCS
 
 QONTRACT_INTEGRATION = "github-repo-permissions-validator"
 QONTRACT_INTEGRATION_VERSION = make_semver(0, 1, 0)
@@ -46,8 +46,8 @@ def run(dry_run: bool, instance_name: str) -> None:
     error = False
     for job in pr_check_jobs:
         repo_url = jjb.get_repo_url(job)
-        repo_name = repo_url.removeprefix(GITHUB_BASE_URL).rstrip("/")
-        repo = gh.get_repo(repo_name)
+        repo_info = VCS.parse_repo_url(repo_url)
+        repo = gh.get_repo(repo_info.name)
         permissions = repo.permissions
         if not permissions.push and repo_url not in invitations:
             logging.error(f"missing write permissions for bot in repo {repo_url}")

reconcile/service_dependencies.py

@@ -16,7 +16,7 @@ from reconcile.gql_definitions.service_dependencies.service_dependencies import
     SaasResourceTemplateV2,
 )
 from reconcile.utils import gql
-from reconcile.utils.vcs import GITHUB_BASE_URL
+from reconcile.utils.vcs import VCS
 
 QONTRACT_INTEGRATION = "service-dependencies"
 
@@ -32,15 +32,13 @@ def get_desired_dependency_names(
     required_dep_names = set()
 
     code_components: list[AppCodeComponentsV1] = app.code_components or []
-    if code_components:
-        gitlab_urls = [cc for cc in code_components if "gitlab" in cc.url]
-        if gitlab_urls:
-            required_dep_names.update(get_dependency_names(dependency_map, "gitlab"))
-        github_urls = [
-            cc for cc in code_components if cc.url.startswith(GITHUB_BASE_URL)
-        ]
-        if github_urls:
-            required_dep_names.update(get_dependency_names(dependency_map, "github"))
+    code_component_platforms = {
+        platform
+        for cc in code_components
+        if (platform := VCS.parse_repo_url(cc.url).platform)
+    }
+    for platform in code_component_platforms:
+        required_dep_names.update(get_dependency_names(dependency_map, platform))
 
     jenkins_configs: list[JenkinsConfigV1] = app.jenkins_configs or []
     if jenkins_configs:

reconcile/slack_usergroups.py

@@ -10,7 +10,6 @@ from typing import (
     Any,
     TypedDict,
 )
-from urllib.parse import urlparse
 
 from github.GithubException import UnknownObjectException
 from pydantic import BaseModel
@@ -67,6 +66,7 @@ from reconcile.utils.slack_api import (
     SlackApiError,
     UsergroupNotFoundException,
 )
+from reconcile.utils.vcs import VCS
 
 DATE_FORMAT = "%Y-%m-%d %H:%M"
 QONTRACT_INTEGRATION = "slack-usergroups"
@@ -77,25 +77,23 @@ error_occurred = False
 
 def get_git_api(url: str) -> GithubRepositoryApi | GitLabApi:
     """Return GitHub/GitLab API based on url."""
-    parsed_url = urlparse(url)
-
-    if parsed_url.hostname:
-        if "github" in parsed_url.hostname:
+    repo_info = VCS.parse_repo_url(url)
+    match repo_info.platform:
+        case "github":
             vault_settings = get_app_interface_vault_settings()
             secret_reader = create_secret_reader(use_vault=vault_settings.vault)
             instance = queries.get_github_instance()
             token = secret_reader.read(instance["token"])
-
             return GithubRepositoryApi(
                 repo_url=url,
                 token=token,
             )
-        if "gitlab" in parsed_url.hostname:
+        case "gitlab":
             settings = queries.get_app_interface_settings()
             instance = queries.get_gitlab_instance()
             return GitLabApi(instance=instance, project_url=url, settings=settings)
-
-    raise ValueError(f"Unable to handle URL: {url}")
+        case _:
+            raise ValueError(f"Unable to handle URL: {url}")
 
 
 class SlackObject(BaseModel):

reconcile/utils/gitlab_api.py

@@ -388,7 +388,7 @@ class GitLabApi:
 
     @retry()
     def get_project(self, repo_url: str) -> Project | None:
-        repo = repo_url.replace(self.server + "/", "")
+        repo = repo_url.removeprefix(self.server).strip("/")
         try:
             project = self.gl.projects.get(repo)
         except GitlabGetError:

reconcile/utils/saasherder/saasherder.py

@@ -20,7 +20,6 @@ from contextlib import suppress
 from datetime import UTC, datetime, timedelta
 from types import TracebackType
 from typing import Any
-from urllib.parse import urlparse
 
 import yaml
 from github import (
@@ -83,7 +82,7 @@ from reconcile.utils.saasherder.models import (
 )
 from reconcile.utils.secret_reader import SecretReaderBase
 from reconcile.utils.state import State
-from reconcile.utils.vcs import GITHUB_BASE_URL
+from reconcile.utils.vcs import VCS
 
 TARGET_CONFIG_HASH = "target_config_hash"
 
@@ -741,17 +740,21 @@ class SaasHerder:  # pylint: disable=too-many-public-methods
         trigger_reason: str,
     ) -> tuple[str, str]:
         [url, sha] = trigger_reason.split(" ")[0].split("/commit/")
-        repo_name = urlparse(url).path.strip("/")
+        repo_info = VCS.parse_repo_url(url)
+        repo_name = repo_info.name
         file_name = f"{repo_name.replace('/', '-')}-{sha}.tar.gz"
-        if url.startswith(GITHUB_BASE_URL):
-            github = self._initiate_github(saas_file, base_url="https://api.github.com")
-            repo = github.get_repo(repo_name)
-            # get_archive_link get redirect url form header, it does not work with github-mirror
-            archive_url = repo.get_archive_link("tarball", ref=sha)
-        elif "gitlab" in url:
-            archive_url = f"{url}/-/archive/{sha}/{file_name}"
-        else:
-            raise Exception(f"Only GitHub and GitLab are supported: {url}")
+        match repo_info.platform:
+            case "github":
+                github = self._initiate_github(
+                    saas_file, base_url="https://api.github.com"
+                )
+                repo = github.get_repo(repo_name)
+                # get_archive_link get redirect url form header, it does not work with github-mirror
+                archive_url = repo.get_archive_link("tarball", ref=sha)
+            case "gitlab":
+                archive_url = f"{url}/-/archive/{sha}/{file_name}"
+            case _:
+                raise Exception(f"Only GitHub and GitLab are supported: {url}")
 
         return file_name, archive_url
 
@@ -761,18 +764,19 @@ class SaasHerder:  # pylint: disable=too-many-public-methods
     ) -> tuple[Any, str]:
         commit_sha = self._get_commit_sha(url, ref, github)
 
-        if url.startswith(GITHUB_BASE_URL):
-            repo_name = url.removeprefix(GITHUB_BASE_URL).rstrip("/")
-            repo = github.get_repo(repo_name)
-            content = self._get_file_contents_github(repo, path, commit_sha)
-        elif "gitlab" in url:
-            if not self.gitlab:
-                raise Exception("gitlab is not initialized")
-            project = self.gitlab.get_project(url)
-            f = project.files.get(file_path=path.lstrip("/"), ref=commit_sha)
-            content = f.decode()
-        else:
-            raise Exception(f"Only GitHub and GitLab are supported: {url}")
+        repo_info = VCS.parse_repo_url(url)
+        match repo_info.platform:
+            case "github":
+                repo = github.get_repo(repo_info.name)
+                content = self._get_file_contents_github(repo, path, commit_sha)
+            case "gitlab":
+                if not self.gitlab:
+                    raise Exception("gitlab is not initialized")
+                project = self.gitlab.get_project(url)
+                f = project.files.get(file_path=path.lstrip("/"), ref=commit_sha)
+                content = f.decode()
+            case _:
+                raise Exception(f"Only GitHub and GitLab are supported: {url}")
 
         return yaml.safe_load(content), commit_sha
 
@@ -782,52 +786,53 @@ class SaasHerder:  # pylint: disable=too-many-public-methods
     ) -> tuple[list[Any], str]:
         commit_sha = self._get_commit_sha(url, ref, github)
         resources: list[Any] = []
-        if url.startswith(GITHUB_BASE_URL):
-            repo_name = url.removeprefix(GITHUB_BASE_URL).rstrip("/")
-            repo = github.get_repo(repo_name)
-            directory = repo.get_contents(path, commit_sha)
-            if isinstance(directory, ContentFile):
-                raise Exception(f"Path {path} and sha {commit_sha} is a file!")
-            for f in directory:
-                file_path = os.path.join(path, f.name)
-                file_contents_decoded = self._get_file_contents_github(
-                    repo, file_path, commit_sha
+        repo_info = VCS.parse_repo_url(url)
+        match repo_info.platform:
+            case "github":
+                repo = github.get_repo(repo_info.name)
+                directory = repo.get_contents(path, commit_sha)
+                if isinstance(directory, ContentFile):
+                    raise Exception(f"Path {path} and sha {commit_sha} is a file!")
+                for f in directory:
+                    file_path = os.path.join(path, f.name)
+                    file_contents_decoded = self._get_file_contents_github(
+                        repo, file_path, commit_sha
+                    )
+                    result_resources = yaml.safe_load_all(file_contents_decoded)
+                    resources.extend(result_resources)
+            case "gitlab":
+                if not self.gitlab:
+                    raise Exception("gitlab is not initialized")
+                project = self.gitlab.get_project(url)
+                dir_contents = self.gitlab.get_directory_contents(
+                    project,
+                    ref=commit_sha,
+                    path=path,
                 )
-                result_resources = yaml.safe_load_all(file_contents_decoded)
-                resources.extend(result_resources)
-        elif "gitlab" in url:
-            if not self.gitlab:
-                raise Exception("gitlab is not initialized")
-            project = self.gitlab.get_project(url)
-            dir_contents = self.gitlab.get_directory_contents(
-                project,
-                ref=commit_sha,
-                path=path,
-            )
-            for content in dir_contents.values():
-                result_resources = yaml.safe_load_all(content)
-                resources.extend(result_resources)
-        else:
-            raise Exception(f"Only GitHub and GitLab are supported: {url}")
+                for content in dir_contents.values():
+                    result_resources = yaml.safe_load_all(content)
+                    resources.extend(result_resources)
+            case _:
+                raise Exception(f"Only GitHub and GitLab are supported: {url}")
 
         return resources, commit_sha
 
     @retry()
     def _get_commit_sha(self, url: str, ref: str, github: Github) -> str:
-        commit_sha = ""
-        if url.startswith(GITHUB_BASE_URL):
-            repo_name = url.removeprefix(GITHUB_BASE_URL).rstrip("/")
-            repo = github.get_repo(repo_name)
-            commit = repo.get_commit(sha=ref)
-            commit_sha = commit.sha
-        elif "gitlab" in url:
-            if not self.gitlab:
-                raise Exception("gitlab is not initialized")
-            project = self.gitlab.get_project(url)
-            commits = project.commits.list(ref_name=ref, per_page=1, page=1)
-            commit_sha = commits[0].id
-
-        return commit_sha
+        repo_info = VCS.parse_repo_url(url)
+        match repo_info.platform:
+            case "github":
+                repo = github.get_repo(repo_info.name)
+                commit = repo.get_commit(sha=ref)
+                return commit.sha
+            case "gitlab":
+                if not self.gitlab:
+                    raise Exception("gitlab is not initialized")
+                project = self.gitlab.get_project(url)
+                commits = project.commits.list(ref_name=ref, per_page=1, page=1)
+                return commits[0].id
+            case _:
+                return ""
 
     @staticmethod
     def _additional_resource_process(resources: Resources, html_url: str) -> None:

reconcile/utils/terrascript_aws_client.py

@@ -186,7 +186,7 @@ from reconcile.utils.password_validator import (
 )
 from reconcile.utils.secret_reader import SecretReader, SecretReaderBase
 from reconcile.utils.terraform import safe_resource_id
-from reconcile.utils.vcs import GITHUB_BASE_URL
+from reconcile.utils.vcs import VCS
 
 GH_BASE_URL = os.environ.get("GITHUB_API", "https://api.github.com")
 LOGTOES_RELEASE = "repos/app-sre/logs-to-elasticsearch-lambda/releases/latest"
@@ -5686,20 +5686,20 @@ class TerrascriptClient:  # pylint: disable=too-many-public-methods
             return ref
 
         # get commit_sha from branch
-        if url.startswith(GITHUB_BASE_URL):
-            github = self.init_github()
-            repo_name = url.removeprefix(GITHUB_BASE_URL).rstrip("/")
-            repo = github.get_repo(repo_name)
-            commit = repo.get_commit(sha=ref)
-            return commit.sha
-
-        if "gitlab" in url:
-            gitlab = self.init_gitlab()
-            project = gitlab.get_project(url)
-            commits = project.commits.list(ref_name=ref, per_page=1, page=1)
-            return commits[0].id
-
-        return ""
+        repo_url_info = VCS.parse_repo_url(url)
+        match repo_url_info.platform:
+            case "github":
+                github = self.init_github()
+                repo = github.get_repo(repo_url_info.name)
+                commit = repo.get_commit(sha=ref)
+                return commit.sha
+            case "gitlab":
+                gitlab = self.init_gitlab()
+                project = gitlab.get_project(url)
+                commits = project.commits.list(ref_name=ref, per_page=1, page=1)
+                return commits[0].id
+            case _:
+                return ""
 
     def get_asg_image_id(
         self, filters: Iterable[Mapping[str, Any]], account: str, region: str

reconcile/utils/vcs.py

@@ -6,6 +6,8 @@ from collections.abc import Iterable
 from dataclasses import dataclass
 from datetime import datetime
 from enum import Enum
+from typing import Literal
+from urllib.parse import urlparse
 
 from gitlab.v4.objects import ProjectMergeRequest
 
@@ -24,6 +26,16 @@ from reconcile.utils.secret_reader import (
 
 GITHUB_BASE_URL = "https://github.com/"
 
+Platform = Literal["github", "gitlab"]
+
+SUPPORTED_PLATFORMS: list[Platform] = ["github", "gitlab"]
+
+
+@dataclass(frozen=True)
+class RepoInfo:
+    platform: Platform | None
+    name: str
+
 
 class MRCheckStatus(Enum):
     NONE = 0
@@ -155,16 +167,40 @@ class VCS:
                 # Lets assume all other states as non-present
                 return MRCheckStatus.NONE
 
+    @staticmethod
+    def parse_repo_url(url: str) -> RepoInfo:
+        """
+        Parse a repository URL and return a RepoInfo object.
+        `platform` can be 'github', 'gitlab' or None if not recognized,
+        it's inferred from the URL host.
+        `name` is the path part of the URL, stripped of leading and trailing slashes.
+        """
+        parsed_url = urlparse(url)
+        platform = next(
+            (
+                p
+                for p in SUPPORTED_PLATFORMS
+                if (hostname := parsed_url.hostname) and p in hostname
+            ),
+            None,
+        )
+        name = parsed_url.path.strip("/").removesuffix(".git")
+        return RepoInfo(platform=platform, name=name)
+
     def get_commit_sha(
         self, repo_url: str, ref: str, auth_code: HasSecret | None
     ) -> str:
         if bool(self._is_commit_sha_regex.search(ref)):
             return ref
-        if repo_url.startswith(GITHUB_BASE_URL):
-            github = self._init_github(repo_url=repo_url, auth_code=auth_code)
-            return github.get_commit_sha(ref=ref)
-        # assume gitlab by default
-        return self._gitlab_instance.get_commit_sha(ref=ref, repo_url=repo_url)
+        repo_info = self.parse_repo_url(repo_url)
+        match repo_info.platform:
+            case "github":
+                github = self._init_github(repo_url=repo_url, auth_code=auth_code)
+                return github.get_commit_sha(ref=ref)
+            case "gitlab":
+                return self._gitlab_instance.get_commit_sha(ref=ref, repo_url=repo_url)
+            case _:
+                raise ValueError(f"Unsupported repository URL: {repo_url}")
 
     def get_commits_between(
         self,
@@ -177,30 +213,33 @@ class VCS:
         Return a list of commits between two commits.
         Note, that the commit_to is included in the result list, whereas commit_from is not included.
         """
-        if repo_url.startswith(GITHUB_BASE_URL):
-            github = self._init_github(repo_url=repo_url, auth_code=auth_code)
-            data = github.compare(commit_from=commit_from, commit_to=commit_to)
-            return [
-                Commit(
-                    repo=repo_url,
-                    sha=gh_commit.sha,
-                    date=gh_commit.commit.committer.date,
+        repo_info = self.parse_repo_url(repo_url)
+        match repo_info.platform:
+            case "github":
+                github = self._init_github(repo_url=repo_url, auth_code=auth_code)
+                data = github.compare(commit_from=commit_from, commit_to=commit_to)
+                return [
+                    Commit(
+                        repo=repo_url,
+                        sha=gh_commit.sha,
+                        date=gh_commit.commit.committer.date,
+                    )
+                    for gh_commit in data
+                ]
+            case "gitlab":
+                data = self._gitlab_instance.repository_compare(
+                    repo_url=repo_url, ref_from=commit_from, ref_to=commit_to
                 )
-                for gh_commit in data
-            ]
-        # assume gitlab by default
-        else:
-            data = self._gitlab_instance.repository_compare(
-                repo_url=repo_url, ref_from=commit_from, ref_to=commit_to
-            )
-            return [
-                Commit(
-                    repo=repo_url,
-                    sha=gl_commit["id"],
-                    date=datetime.fromisoformat(gl_commit["committed_date"]),
-                )
-                for gl_commit in data
-            ]
+                return [
+                    Commit(
+                        repo=repo_url,
+                        sha=gl_commit["id"],
+                        date=datetime.fromisoformat(gl_commit["committed_date"]),
+                    )
+                    for gl_commit in data
+                ]
+            case _:
+                raise ValueError(f"Unsupported repository URL: {repo_url}")
 
     def close_app_interface_mr(self, mr: ProjectMergeRequest, comment: str) -> None:
         if not self._allow_deleting_mrs: