feat: Retire split-testing (#6542)

Author: khvn26

.github/workflows/platform-docker-build-test-publish.yml

@@ -50,18 +50,6 @@ jobs:
       secrets: |
         github_private_cloud_token=${{ secrets.GH_PRIVATE_ACCESS_TOKEN }}
 
-  docker-build-split-testing:
-    name: Build Split Testing Image
-    uses: ./.github/workflows/.reusable-docker-build.yml
-    with:
-      target: private-cloud-unified
-      image-name: flagsmith-private-cloud-split-testing
-      build-args: |
-        WITH=saml,auth-controller,ldap,workflows,licensing,split-testing
-    secrets:
-      secrets: |
-        github_private_cloud_token=${{ secrets.GH_PRIVATE_ACCESS_TOKEN }}
-
   docker-build-api-test:
     name: Build API Test Image
     uses: ./.github/workflows/.reusable-docker-build.yml
@@ -149,7 +137,7 @@ jobs:
   # Publish to Quay.io
 
   docker-publish-quay-enterprise:
-    needs: [ docker-build-private-cloud, run-e2e-tests ]
+    needs: [docker-build-private-cloud, run-e2e-tests]
     uses: ./.github/workflows/.reusable-docker-publish.yml
     if: github.event_name == 'release'
     with:
@@ -161,7 +149,7 @@ jobs:
     secrets: inherit
 
   docker-publish-quay-enterprise-api:
-    needs: [ docker-build-private-cloud-api, run-e2e-tests ]
+    needs: [docker-build-private-cloud-api, run-e2e-tests]
     uses: ./.github/workflows/.reusable-docker-publish.yml
     if: github.event_name == 'release'
     with:
@@ -173,7 +161,7 @@ jobs:
     secrets: inherit
 
   docker-publish-quay-frontend:
-    needs: [ docker-build-frontend, run-e2e-tests ]
+    needs: [docker-build-frontend, run-e2e-tests]
     uses: ./.github/workflows/.reusable-docker-publish.yml
     if: github.event_name == 'release'
     with:
@@ -184,18 +172,6 @@ jobs:
       target-images: quay.io/${{ vars.QUAY_ORGANISATION_NAME }}/flagsmith-frontend
     secrets: inherit
 
-  docker-publish-quay-split-testing:
-    needs: [ docker-build-split-testing, run-e2e-tests ]
-    uses: ./.github/workflows/.reusable-docker-publish.yml
-    if: github.event_name == 'release'
-    with:
-      target-registry-url: quay.io
-      docker-username: ${{ vars.QUAY_PUBLISH_USERNAME }}
-      docker-password-secret-name: QUAY_PUBLISH_PASSWORD
-      source-images: ${{ needs.docker-build-split-testing.outputs.image }}
-      target-images: quay.io/${{ vars.QUAY_ORGANISATION_NAME }}/flagsmith-split-testing
-    secrets: inherit
-
   update-charts:
     needs: [docker-publish-api, docker-publish-frontend, docker-publish-unified]
     runs-on: depot-ubuntu-latest
@@ -215,7 +191,7 @@ jobs:
                 state: "open",
               }
             );
-      
+
             for (const pr of prs) {
               const labels = pr.labels.map(label => label.name);
               if (labels.includes("yaml-updates")) {
@@ -248,20 +224,20 @@ jobs:
       - name: Open a PR bumping Flagsmith to ${{ github.event.release.tag_name }}
         uses: fjogeleit/yaml-update-action@main
         env:
-          COMMIT_MESSAGE: 'deps: bump Flagsmith from ${{ steps.chart-yaml.outputs.appVersion }} to ${{ steps.version-trim.outputs.version }}'
+          COMMIT_MESSAGE: "deps: bump Flagsmith from ${{ steps.chart-yaml.outputs.appVersion }} to ${{ steps.version-trim.outputs.version }}"
         with:
           token: ${{ secrets.FLAGSMITH_CHARTS_GITHUB_TOKEN }}
           repository: flagsmith/flagsmith-charts
           workDir: chart
-          masterBranchName: 'main'
-          targetBranch: 'main'
+          masterBranchName: "main"
+          targetBranch: "main"
           branch: deps/bump-flagsmith-${{ steps.version-trim.outputs.version }}
           commitChange: true
           createPR: true
           message: ${{ env.COMMIT_MESSAGE }}
           title: ${{ env.COMMIT_MESSAGE }}
-          description: 'Automated PR generated by Flagsmith release [${{ github.event.release.tag_name }}](${{ github.event.release.url }}).'
-          valueFile: 'charts/flagsmith/Chart.yaml'
+          description: "Automated PR generated by Flagsmith release [${{ github.event.release.tag_name }}](${{ github.event.release.url }})."
+          valueFile: "charts/flagsmith/Chart.yaml"
           changes: |
             {
               "appVersion": "${{ steps.version-trim.outputs.version }}"

.github/workflows/platform-pull-request.yml

@@ -129,20 +129,6 @@ jobs:
       secrets: |
         github_private_cloud_token=${{ secrets.GH_PRIVATE_ACCESS_TOKEN }}
 
-  docker-build-split-testing:
-    if: github.event.pull_request.draft == false && needs.permissions-check.outputs.can-write == 'true'
-    needs: [permissions-check, docker-prepare-report-comment]
-    name: Build Split Testing Image
-    uses: ./.github/workflows/.reusable-docker-build.yml
-    with:
-      target: private-cloud-unified
-      image-name: flagsmith-private-cloud-split-testing
-      build-args: |
-        WITH=saml,auth-controller,ldap,workflows,licensing,split-testing
-    secrets:
-      secrets: |
-        github_private_cloud_token=${{ secrets.GH_PRIVATE_ACCESS_TOKEN }}
-
   run-e2e-tests:
     if: github.event.pull_request.draft == false && !cancelled()
     needs: [permissions-check, docker-build-api, docker-build-e2e]

api/api/urls/v1.py

@@ -97,29 +97,3 @@
     re_path(r"^webhooks/", include("webhooks.urls", namespace="webhooks")),
     path("", include("projects.code_references.urls", namespace="code_references")),
 ]
-
-if settings.SPLIT_TESTING_INSTALLED:
-    from split_testing.views import (  # type: ignore[import-not-found]
-        ConversionEventTypeView,
-        CreateConversionEventView,
-        SplitTestViewSet,
-    )
-
-    split_testing_router = routers.DefaultRouter()
-    split_testing_router.register(r"", SplitTestViewSet, basename="split-tests")
-
-    urlpatterns += [
-        re_path(
-            r"^split-testing/", include(split_testing_router.urls), name="split-testing"
-        ),
-        re_path(
-            r"^split-testing/conversion-events/",
-            CreateConversionEventView.as_view(),
-            name="conversion-events",
-        ),
-        path(
-            "conversion-event-types/",
-            ConversionEventTypeView.as_view(),
-            name="conversion-event-types",
-        ),
-    ]

api/app/settings/common.py

@@ -1403,10 +1403,6 @@
 WEBHOOK_BACKOFF_BASE = env.int("WEBHOOK_BACKOFF_BASE", default=2)
 WEBHOOK_BACKOFF_RETRIES = env.int("WEBHOOK_BACKOFF_RETRIES", default=3)
 
-# Split Testing settings
-SPLIT_TESTING_INSTALLED = importlib.util.find_spec("split_testing")
-if SPLIT_TESTING_INSTALLED:
-    INSTALLED_APPS += ("split_testing",)
 
 ENABLE_API_USAGE_ALERTING = env.bool("ENABLE_API_USAGE_ALERTING", default=False)