fix: add proper migration, start service account api

Author: hamir-suspect

github_hooks/db/migrate/20250718124232_create_service_accounts.rb

@@ -0,0 +1,12 @@
+class CreateServiceAccounts < ActiveRecord::Migration[6.1]
+  def change
+    create_table :service_accounts, id: false do |t|
+      t.uuid :id, primary_key: true, null: false
+      t.string :description
+      t.uuid :creator_id, null: false
+    end
+
+    add_foreign_key :service_accounts, :users, column: :id, on_delete: :cascade
+    add_foreign_key :service_accounts, :users, column: :creator_id, on_delete: :nullify
+  end
+end

guard/Makefile

@@ -141,4 +141,6 @@ endif
 		/home/protoc/source/encryptor.proto
 	docker run --rm -v $(PWD):/home/protoc/code -v $(TMP_INTERNAL_REPO_DIR):/home/protoc/source renderedtext/protoc:$(RT_PROTOC_IMG_VSN) protoc -I /home/protoc/source -I /home/protoc/source/include --elixir_out=plugins=grpc:$(RELATIVE_INTERNAL_PB_OUTPUT_DIR) --plugin=/root/.mix/escripts/protoc-gen-elixir \
 		/home/protoc/source/instance_config.proto
+	docker run --rm -v $(PWD):/home/protoc/code -v $(TMP_INTERNAL_REPO_DIR):/home/protoc/source renderedtext/protoc:$(RT_PROTOC_IMG_VSN) protoc -I /home/protoc/source -I /home/protoc/source/include --elixir_out=plugins=grpc:$(RELATIVE_INTERNAL_PB_OUTPUT_DIR) --plugin=/root/.mix/escripts/protoc-gen-elixir \
+		/home/protoc/source/service_account.proto
 	rm -rf $(TMP_INTERNAL_REPO_DIR)

guard/docker-compose.yml

@@ -50,6 +50,7 @@ services:
       START_GPRC_GUARD_API: "true"
       START_GRPC_AUTH_API: "true"
       START_GRPC_USER_API: "true"
+      START_GRPC_SERVICE_ACCOUNT_API: "true"
       START_GRPC_ORGANIZATION_API: "true"
       START_GRPC_INSTANCE_CONFIG_API: "true"
       INSTANCE_CONFIG_API: "true"

guard/helm/templates/service.yaml

@@ -100,6 +100,25 @@ spec:
 ---
 apiVersion: v1
 kind: Service
+metadata:
+  name: "{{ .Chart.Name }}-service-account-api"
+  namespace: {{ .Release.Namespace }}
+spec:
+  type: NodePort
+  selector:
+    {{- if .Values.global.development.minimalDeployment }}
+    app: "{{ .Chart.Name }}"
+    {{- else }}
+    app: "{{ .Chart.Name }}-user-api"
+    {{- end }}
+  ports:
+    - name: grpc
+      port: 50051
+      targetPort: 50051
+      protocol: TCP
+---
+apiVersion: v1
+kind: Service
 metadata:
   name: "{{ .Chart.Name }}-organization-api"
   namespace: {{ .Release.Namespace }}

guard/helm/templates/user-api-dpl.yaml

@@ -89,6 +89,8 @@ spec:
               value: "false"
             - name: START_GRPC_USER_API
               value: "true"
+            - name: START_GRPC_SERVICE_ACCOUNT_API
+              value: "true"
             - name: START_GPRC_HEALTH_CHECK
               value: "true"
             - name: RABBIT_CONSUMER

guard/priv/front_repo/migrations/20250716010415_create_service_accounts.exs

@@ -0,0 +1,13 @@
+defmodule Guard.FrontRepo.Migrations.CreateServiceAccounts do
+  use Ecto.Migration
+
+  def change do
+    create table(:service_accounts, primary_key: false) do
+      add :id, references(:users, type: :binary_id, on_delete: :delete_all), 
+          primary_key: true, null: false
+      add :description, :string
+      add :creator_id, references(:users, type: :binary_id, on_delete: :nilify_all), 
+          null: false
+    end
+  end
+end

guard/test/guard/front_repo/service_account_test.exs

@@ -159,21 +159,6 @@ defmodule Guard.FrontRepo.ServiceAccountTest do
       assert {"has already been taken", _} = changeset.errors[:user_id]
     end
 
-    test "sets timestamps on creation" do
-      user = create_test_user()
-
-      attrs = %{
-        user_id: user.id,
-        description: "Test description"
-      }
-
-      changeset = ServiceAccount.changeset(%ServiceAccount{}, attrs)
-      {:ok, service_account} = FrontRepo.insert(changeset)
-
-      assert service_account.created_at != nil
-      assert service_account.updated_at != nil
-      assert service_account.created_at == service_account.updated_at
-    end
   end
 
   describe "update_changeset/2" do