Skip to content

chore: Go fix#200

Merged
o-ga09 merged 2 commits intomainfrom
go-fix
Feb 28, 2026
Merged

chore: Go fix#200
o-ga09 merged 2 commits intomainfrom
go-fix

Conversation

@o-ga09
Copy link
Owner

@o-ga09 o-ga09 commented Feb 28, 2026

close #

実装内容

動作確認

エビデンス

テスト結果

Copilot AI review requested due to automatic review settings February 28, 2026 02:10
@github-actions
Copy link

Code Metrics Report

Coverage Code to Test Ratio Test Execution Time
29.8% 1:0.4 1m24s

Code coverage of files in pull request scope (10.8%)

Files Coverage
cmd/mcp/main.go 0.0%
internal/database/mysql/db_connect.go 4.3%
internal/database/mysql/sentry.go 0.0%
internal/database/mysql/testHelper.go 79.3%
pkg/testutil/golden.go 0.0%

Reported by octocov

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot reviewed 6 out of 6 changed files in this pull request and generated 1 comment.

Comment on lines +8 to +10
push:
tags:
- "v*"
Copy link

Copilot AI Feb 28, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pushtags: v* トリガー追加により、タグ作成/プッシュ時にもこのワークフローが実行され、Terraform Apply(github.event_name != 'pull_request')まで自動で走ります。タグは main 以外のコミットにも付けられるため、未マージのコード/設定で本番相当の Terraform Apply が走る運用事故リスクがあります。意図したデプロイ契機でない場合は、(1) Apply を workflow_dispatch 等に限定する、(2) 環境保護(Required reviewers)を通す、(3) main 上のタグに限定するための追加ガード(例: 事前に main への到達可能性を検証するステップ)などで制御してください。

Suggested change
push:
tags:
- "v*"

Copilot uses AI. Check for mistakes.
@github-actions
Copy link

Terraform Format and Style failure

Terraform Initialization ⚙️success

Terraform Validation success

Validation Output ```

Success! The configuration is valid.

</details>
#### Terraform Plan `success`
<details><summary>Show Plan</summary>

terraform
google_iam_workload_identity_pool.terraform-pool: Refreshing state... [id=projects/mh-api-389212/locations/global/workloadIdentityPools/terraform-pool]
data.google_iam_policy.auth: Reading...
data.google_iam_policy.mcp_auth: Reading...
google_project_service.enable_api["serviceusage.googleapis.com"]: Refreshing state... [id=mh-api-389212/serviceusage.googleapis.com]
google_project_iam_member.cloud_run_trace_agent: Refreshing state... [id=mh-api-389212/roles/cloudtrace.agent/serviceAccount:terraform@mh-api-389212.iam.gserviceaccount.com]
data.google_service_account.terraform_sa: Reading...
google_project_service.enable_api["artifactregistry.googleapis.com"]: Refreshing state... [id=mh-api-389212/artifactregistry.googleapis.com]
google_project_service.enable_api["secretmanager.googleapis.com"]: Refreshing state... [id=mh-api-389212/secretmanager.googleapis.com]
data.google_iam_policy.mcp_auth: Read complete after 0s [id=2054599854]
data.google_iam_policy.auth: Read complete after 0s [id=3450855414]
google_project_service.enable_api["logging.googleapis.com"]: Refreshing state... [id=mh-api-389212/logging.googleapis.com]
google_project_service.enable_api["run.googleapis.com"]: Refreshing state... [id=mh-api-389212/run.googleapis.com]
google_project_service.enable_api["iamcredentials.googleapis.com"]: Refreshing state... [id=mh-api-389212/iamcredentials.googleapis.com]
google_cloud_run_service.stg-mh-api: Refreshing state... [id=locations/asia-northeast1/namespaces/mh-api-389212/services/stg-mh-api]
data.google_service_account.terraform_sa: Read complete after 0s [id=projects/mh-api-389212/serviceAccounts/terraform@mh-api-389212.iam.gserviceaccount.com]
google_project_service.enable_api["cloudresourcemanager.googleapis.com"]: Refreshing state... [id=mh-api-389212/cloudresourcemanager.googleapis.com]
google_project_service.enable_api["iam.googleapis.com"]: Refreshing state... [id=mh-api-389212/iam.googleapis.com]
google_project_iam_member.logging_writer: Refreshing state... [id=mh-api-389212/roles/logging.logWriter/serviceAccount:terraform@mh-api-389212.iam.gserviceaccount.com]
data.google_iam_policy.agent_auth: Reading...
data.google_iam_policy.agent_auth: Read complete after 0s [id=2054599854]
google_cloud_run_service.stg-mh-mcp: Refreshing state... [id=locations/asia-northeast1/namespaces/mh-api-389212/services/stg-mh-mcp]
google_project_iam_member.service_account_user: Refreshing state... [id=mh-api-389212/roles/iam.serviceAccountUser/serviceAccount:terraform@mh-api-389212.iam.gserviceaccount.com]
google_project_iam_member.secret_manager_secret_accessor: Refreshing state... [id=mh-api-389212/roles/secretmanager.secretAccessor/serviceAccount:terraform@mh-api-389212.iam.gserviceaccount.com]
google_cloud_run_service.stg-mh-agent: Refreshing state... [id=locations/asia-northeast1/namespaces/mh-api-389212/services/stg-mh-agent]
google_iam_workload_identity_pool_provider.terraform-provider: Refreshing state... [id=projects/mh-api-389212/locations/global/workloadIdentityPools/terraform-pool/providers/terraform-provider]
google_service_account_iam_member.terraform_sa: Refreshing state... [id=projects/mh-api-389212/serviceAccounts/terraform@mh-api-389212.iam.gserviceaccount.com/roles/iam.workloadIdentityUser/principalSet://iam.googleapis.com/projects/650836814988/locations/global/workloadIdentityPools/terraform-pool/attribute.repository/o-ga09/MH-API]
google_cloud_run_service_iam_policy.auth: Refreshing state... [id=v1/projects/mh-api-389212/locations/asia-northeast1/services/stg-mh-api]
google_cloud_run_service_iam_policy.mcp_auth: Refreshing state... [id=v1/projects/mh-api-389212/locations/asia-northeast1/services/stg-mh-mcp]
google_cloud_run_service_iam_policy.agent_auth: Refreshing state... [id=v1/projects/mh-api-389212/locations/asia-northeast1/services/stg-mh-agent]

Terraform used the selected providers to generate the following execution
plan. Resource actions are indicated with the following symbols:
~ update in-place

Terraform will perform the following actions:

google_cloud_run_service.stg-mh-agent will be updated in-place

~ resource "google_cloud_run_service" "stg-mh-agent" {
id = "locations/asia-northeast1/namespaces/mh-api-389212/services/stg-mh-agent"
name = "stg-mh-agent"
# (4 unchanged attributes hidden)

  ~ template {
      ~ spec {
            # (5 unchanged attributes hidden)

          ~ containers {
              ~ image       = "asia-northeast1-docker.pkg.dev/mh-api-389212/mh-api/mh-agent:b14afcd585003c2fc40f6fde51cfc7d76cd75783" -> "asia-northeast1-docker.pkg.dev/mh-api-389212/mh-api/mh-agent:7bbfa848948e765f78b7f0d0a206d4e0fbfcb924"
                name        = null
                # (3 unchanged attributes hidden)

                # (11 unchanged blocks hidden)
            }
        }

        # (1 unchanged block hidden)
    }

    # (2 unchanged blocks hidden)
}

google_cloud_run_service.stg-mh-api will be updated in-place

~ resource "google_cloud_run_service" "stg-mh-api" {
id = "locations/asia-northeast1/namespaces/mh-api-389212/services/stg-mh-api"
name = "stg-mh-api"
# (4 unchanged attributes hidden)

  ~ template {
      ~ spec {
            # (5 unchanged attributes hidden)

          ~ containers {
              ~ image       = "asia-northeast1-docker.pkg.dev/mh-api-389212/mh-api/mh-api:b14afcd585003c2fc40f6fde51cfc7d76cd75783" -> "asia-northeast1-docker.pkg.dev/mh-api-389212/mh-api/mh-api:7bbfa848948e765f78b7f0d0a206d4e0fbfcb924"
                name        = null
                # (3 unchanged attributes hidden)

                # (10 unchanged blocks hidden)
            }
        }

        # (1 unchanged block hidden)
    }

    # (2 unchanged blocks hidden)
}

google_cloud_run_service.stg-mh-mcp will be updated in-place

~ resource "google_cloud_run_service" "stg-mh-mcp" {
id = "locations/asia-northeast1/namespaces/mh-api-389212/services/stg-mh-mcp"
name = "stg-mh-mcp"
# (4 unchanged attributes hidden)

  ~ template {
      ~ spec {
            # (5 unchanged attributes hidden)

          ~ containers {
              ~ image       = "asia-northeast1-docker.pkg.dev/mh-api-389212/mh-api/mh-mcp:b14afcd585003c2fc40f6fde51cfc7d76cd75783" -> "asia-northeast1-docker.pkg.dev/mh-api-389212/mh-api/mh-mcp:7bbfa848948e765f78b7f0d0a206d4e0fbfcb924"
                name        = null
                # (3 unchanged attributes hidden)

                # (10 unchanged blocks hidden)
            }
        }

        # (1 unchanged block hidden)
    }

    # (2 unchanged blocks hidden)
}

Plan: 0 to add, 3 to change, 0 to destroy.

─────────────────────────────────────────────────────────────────────────────

Note: You didn't use the -out option to save this plan, so Terraform can't
guarantee to take exactly these actions if you run "terraform apply" now.

</details>
*Pusher: @o-ga09, Action: `pull_request`, Working Directory: ``, Workflow: `terraform`*

@o-ga09 o-ga09 merged commit 4759952 into main Feb 28, 2026
9 checks passed
@o-ga09 o-ga09 deleted the go-fix branch February 28, 2026 02:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants