Remove Fideslog

noxfiles/ci_nox.py

@@ -258,7 +258,14 @@ def load_tests(session: nox.Session) -> None:
     https://github.com/fcsonline/drill
     """
     session.run(
-        "drill", "-b", "noxfiles/drill.yml", "--quiet", "--stats", external=True
+        "drill",
+        "-b",
+        "noxfiles/drill.yml",
+        "--quiet",
+        "--stats",
+        "--timeout",
+        "10",  # Our server runs more slowly in CI
+        external=True,
     )
 
 

noxfiles/drill.yml

@@ -4,7 +4,7 @@
 concurrency: 5 # How many instances of the `plan` get run concurrently
 base: 'http://localhost:8080'
 iterations: 20 # How many times to run the plan
-rampup: 1 # The number of seconds between starting concurrent runs
+rampup: 2 # The number of seconds between starting concurrent runs
 
 # This plan is run in its entirety, sequentially, for each iteration
 plan:

requirements.txt

@@ -13,7 +13,6 @@ expandvars==0.9.0
 fastapi[all]==0.89.1
 fastapi-pagination[sqlalchemy]==0.11.4
 fideslang==2.2.0
-fideslog==1.2.10
 firebase-admin==5.3.0
 GitPython==3.1.35
 httpx==0.23.1

src/fides/api/alembic/migrations/versions/4e9c4168bf18_drop_userregistration_table.py

@@ -0,0 +1,55 @@
+"""drop userregistration table
+
+Revision ID: 4e9c4168bf18
+Revises: ddec29f24945
+Create Date: 2023-10-26 12:15:06.316303
+
+"""
+import sqlalchemy as sa
+from alembic import op
+from sqlalchemy.dialects import postgresql
+
+# revision identifiers, used by Alembic.
+revision = "4e9c4168bf18"
+down_revision = "ddec29f24945"
+branch_labels = None
+depends_on = None
+
+
+def upgrade():
+    # ### commands auto generated by Alembic - please adjust! ###
+    op.drop_index("ix_userregistration_id", table_name="userregistration")
+    op.drop_table("userregistration")
+    # ### end Alembic commands ###
+
+
+def downgrade():
+    # ### commands auto generated by Alembic - please adjust! ###
+    op.create_table(
+        "userregistration",
+        sa.Column("id", sa.VARCHAR(length=255), autoincrement=False, nullable=False),
+        sa.Column(
+            "created_at",
+            postgresql.TIMESTAMP(timezone=True),
+            server_default=sa.text("now()"),
+            autoincrement=False,
+            nullable=True,
+        ),
+        sa.Column(
+            "updated_at",
+            postgresql.TIMESTAMP(timezone=True),
+            server_default=sa.text("now()"),
+            autoincrement=False,
+            nullable=True,
+        ),
+        sa.Column("user_email", sa.VARCHAR(), autoincrement=False, nullable=True),
+        sa.Column(
+            "user_organization", sa.VARCHAR(), autoincrement=False, nullable=True
+        ),
+        sa.Column("analytics_id", sa.VARCHAR(), autoincrement=False, nullable=False),
+        sa.Column("opt_in", sa.BOOLEAN(), autoincrement=False, nullable=False),
+        sa.PrimaryKeyConstraint("id", name="userregistration_pkey"),
+        sa.UniqueConstraint("analytics_id", name="userregistration_analytics_id_key"),
+    )
+    op.create_index("ix_userregistration_id", "userregistration", ["id"], unique=False)
+    # ### end Alembic commands ###

src/fides/api/api/v1/api.py

@@ -18,7 +18,6 @@
     privacy_notice_endpoints,
     privacy_preference_endpoints,
     privacy_request_endpoints,
-    registration_endpoints,
     saas_config_endpoints,
     served_notice_endpoints,
     storage_endpoints,
@@ -51,5 +50,4 @@
 api_router.include_router(user_endpoints.router)
 api_router.include_router(user_permission_endpoints.router)
 api_router.include_router(manual_webhook_endpoints.router)
-api_router.include_router(registration_endpoints.router)
 api_router.include_router(served_notice_endpoints.router)

src/fides/api/db/base.py

@@ -34,7 +34,6 @@
     ServedNoticeHistory,
 )
 from fides.api.models.privacy_request import PrivacyRequest
-from fides.api.models.registration import UserRegistration
 from fides.api.models.storage import StorageConfig
 from fides.api.models.system_history import SystemHistory
 from fides.api.models.system_manager import SystemManager

src/fides/api/graph/graph_differences.py

@@ -57,7 +57,7 @@ def format_graph_for_caching(
 
 
 class GraphDiff(FidesSchema):
-    """A more detailed description about how two graphs differ. Do not send these details to FidesLog."""
+    """A more detailed description about how two graphs differ."""
 
     previous_collections: List[str] = []
     current_collections: List[str] = []
@@ -71,7 +71,7 @@ class GraphDiff(FidesSchema):
 
 
 class GraphDiffSummary(FidesSchema):
-    """A summary about how two graphs have changed. This can be sent to FidesLog."""
+    """A summary about how two graphs have changed."""
 
     prev_collection_count: int = 0
     curr_collection_count: int = 0