Skip to content

Conversation

yasulab
Copy link
Member

@yasulab yasulab commented Sep 5, 2025

Fix #256

概要

スタートアップスクリプトの実行問題を解決するため、Ubuntuアーカイブ選択ロジックを修正し、デバッグ機能とRakeタスクを追加しました。

✅ SSH接続成功!

問題の原因

間違ったUbuntuアーカイブが選択されていた:

  • ❌ 選択されていた: CData Sync v25.1(Ubuntu 24.04.2 LTS 64bit)
  • ✅ 正しい: Ubuntu Server 24.04.2 LTS 64bit

正規表現が不適切で、Ubuntuを含む全てのアーカイブがマッチしていたため、最後にマッチした「CData Sync」が選択されていました。

修正内容

1. アーカイブ選択ロジックの修正

  • 正規表現を ^Ubuntu Server に変更して、Ubuntu Serverで始まるものだけを対象に
  • break を追加して最初にマッチしたアーカイブを使用

2. デバッグ機能の追加

  • VERBOSE=true 環境変数でデバッグログを有効化
  • disk/config APIの詳細ログを追加してNotesパラメータの設定を確認可能に

3. Rakeタスク追加

  • rake server:status[サーバー名] タスクを追加
  • scripts/utils/check_server_status.rb をRake統合

テスト結果

$ nc -vz 133.242.227.219 22
Connection to 133.242.227.219 port 22 [tcp/ssh] succeeded!

変更ファイル

  • Rakefile - server:statusタスク追加
  • scripts/deploy.rb - アーカイブ選択ロジック修正、VERBOSE対応
  • scripts/sakura_server_user_agent.rb - デバッグログ追加

マージ準備完了

SSH接続が確認できたため、マージ可能です。

問題:
- 間違ったアーカイブ(CData Sync)が選択されSSH接続できなかった
- 正規表現が不適切でUbuntuを含む全てのアーカイブがマッチしていた

修正内容:
1. アーカイブ選択ロジックの修正
   - 正規表現を '^Ubuntu Server' に変更
   - breakを追加して最初にマッチしたアーカイブを使用

2. デバッグ機能の追加
   - VERBOSE環境変数でデバッグログを有効化
   - disk/config APIの詳細ログを追加

3. Rakeタスク追加
   - rake server:status[サーバー名] でサーバー状態確認

テスト結果:
- SSH接続成功を確認(nc -vz 133.242.227.219 22)
@yasulab yasulab force-pushed the test-ssh-connection-after-startup-script-fix branch from d93f87c to 53ab67f Compare September 5, 2025 07:13
@yasulab yasulab changed the title fix: スタートアップスクリプト修正後のSSH接続確認とserver:statusタスク追加 fix: 間違ったサーバーイメージを選択してしまうロジックを修正と、デバッグ用ユーティリティの追加 Sep 5, 2025
@yasulab yasulab changed the title fix: 間違ったサーバーイメージを選択してしまうロジックを修正と、デバッグ用ユーティリティの追加 fix: 間違ったサーバーイメージを選択してしまうロジックの修正と、デバッグ用ユーティリティの追加 Sep 5, 2025
@yasulab yasulab changed the title fix: 間違ったサーバーイメージを選択してしまうロジックの修正と、デバッグ用ユーティリティの追加 fix: 間違った Ubuntu イメージを選択してしまうロジックの修正と、デバッグ用ユーティリティの追加 Sep 5, 2025
@yasulab yasulab changed the title fix: 間違った Ubuntu イメージを選択してしまうロジックの修正と、デバッグ用ユーティリティの追加 fix: 間違った Ubuntu イメージが選択されてしまうロジックの修正と、デバッグ用ユーティリティの追加 Sep 5, 2025
@yasulab yasulab merged commit 22d4c6f into main Sep 5, 2025
4 checks passed
@yasulab yasulab deleted the test-ssh-connection-after-startup-script-fix branch September 5, 2025 07:23
github-actions bot pushed a commit that referenced this pull request Sep 5, 2025
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.

1 participant