skip profile schemas when checking for schema migrations (#5446)

* skip profile schemas when checking for schema migrations

* remove dead code

* invert the conditional

backend/infrahub/core/validators/models/violation.py

@@ -1,17 +1,4 @@
-from typing import Union
-
-from pydantic import BaseModel, Field
-
-from infrahub.core.branch import Branch
-from infrahub.core.path import SchemaPath
-from infrahub.core.schema import GenericSchema, NodeSchema
-
-
-class SchemaConstraintValidatorRequest(BaseModel):
-    branch: Branch = Field(..., description="The name of the branch to target")
-    constraint_name: str = Field(..., description="The name of the constraint to validate")
-    node_schema: Union[NodeSchema, GenericSchema] = Field(..., description="Schema of Node or Generic to validate")
-    schema_path: SchemaPath = Field(..., description="SchemaPath to the element of the schema to validate")
+from pydantic import BaseModel
 
 
 class SchemaViolation(BaseModel):

backend/infrahub/core/validators/tasks.py

@@ -32,11 +32,14 @@ async def schema_validate_migrations(message: SchemaValidateMigrationData) -> li
     log.info(f"{len(message.constraints)} constraint(s) to validate")
     # NOTE this task is a good candidate to add a progress bar
     for constraint in message.constraints:
+        schema = message.schema_branch.get(name=constraint.path.schema_kind)
+        if not isinstance(schema, (GenericSchema, NodeSchema)):
+            continue
         batch.add(
             task=schema_path_validate,
             branch=message.branch,
             constraint_name=constraint.constraint_name,
-            node_schema=message.schema_branch.get(name=constraint.path.schema_kind),
+            node_schema=schema,
             schema_path=constraint.path,
         )