Skip to content

Commit ee10460

Browse files
committed
refactor: YAGNI原則に基づきサーバーID表示を削除
- ユーザー向け出力からサーバーID表示を削除(4箇所) - 現在のワークフローではIPアドレスとサーバー名のみ必要 - 内部処理では引き続きサーバーIDを使用(API呼び出し用) - GitHub Actionsのコメント表示を改善(コードブロック追加) - よりシンプルで分かりやすい出力に改善
1 parent 6ebaa73 commit ee10460

File tree

2 files changed

+11
-12
lines changed

2 files changed

+11
-12
lines changed

.github/workflows/auto_respond_initialize.yml

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -65,18 +65,18 @@ jobs:
6565
6666
if [ -z "$EXIT_CODE" ] || [ "$EXIT_CODE" = "0" ]; then
6767
echo "server_found=true" >> $GITHUB_OUTPUT
68-
echo "server_info<<EOF" >> $GITHUB_OUTPUT
69-
echo "$OUTPUT" >> $GITHUB_OUTPUT
70-
echo "EOF" >> $GITHUB_OUTPUT
68+
echo "server_info<<EOF" >> $GITHUB_OUTPUT
69+
echo "$OUTPUT" >> $GITHUB_OUTPUT
70+
echo "EOF" >> $GITHUB_OUTPUT
7171
7272
# 削除準備タスクも実行(インクリメンタル実行用)
7373
# IPアドレスを明示的にパラメータとして渡す
7474
bundle exec rake "server:prepare_deletion[${{ steps.extract_ip.outputs.ip_address }}]" 2>&1 || true
7575
else
7676
echo "server_found=false" >> $GITHUB_OUTPUT
7777
echo "error_message<<EOF" >> $GITHUB_OUTPUT
78-
echo "$OUTPUT" >> $GITHUB_OUTPUT
79-
echo "EOF" >> $GITHUB_OUTPUT
78+
echo "$OUTPUT" >> $GITHUB_OUTPUT
79+
echo "EOF" >> $GITHUB_OUTPUT
8080
fi
8181
8282
- name: Issueにコメント - サーバーが見つかった場合
@@ -99,8 +99,10 @@ jobs:
9999
const serverInfo = `${{ steps.find_server.outputs.server_info }}`.replace(/%0A/g, '\n').replace(/%0D/g, '\r').replace(/%25/g, '%');
100100
101101
const comment = `## 🔍 サーバー情報を確認しました
102-
102+
103+
\`\`\`
103104
${serverInfo}
105+
\`\`\`
104106
105107
---
106108

scripts/initialize_server.rb

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -379,7 +379,6 @@ def display_server_details_for_deletion(server)
379379
puts "=" * 60
380380
puts ""
381381
puts " サーバー名: #{server['Name']}"
382-
puts " サーバーID: #{server['ID']}"
383382
puts " IPアドレス: #{@input}"
384383
puts " 説明: #{server['Description']}"
385384
puts " タグ: #{server['Tags'].join(', ')}"
@@ -603,7 +602,7 @@ def execute_deletion(server, disk_ids)
603602
puts "=" * 60
604603
puts ""
605604
puts "削除される予定のリソース:"
606-
puts " - サーバー: #{server['Name']} (ID: #{server_id})"
605+
puts " - サーバー: #{server['Name']}"
607606
puts " - ディスク数: #{disk_ids.length}個"
608607
puts ""
609608
puts "⚠️ これはドライランです。実際には何も削除されていません。"
@@ -613,7 +612,7 @@ def execute_deletion(server, disk_ids)
613612
puts "=" * 60
614613
puts ""
615614
puts "削除されたリソース:"
616-
puts " - サーバー: #{server['Name']} (ID: #{server_id})"
615+
puts " - サーバー: #{server['Name']}"
617616
puts " - ディスク数: #{disk_ids.length}個"
618617
end
619618
puts ""
@@ -714,7 +713,6 @@ def verify_server_match(dojo_name, server_info)
714713
def display_server_info(server)
715714
puts "🖥️ サーバー情報:"
716715
puts " - サーバー名: #{server['Name']}"
717-
puts " - サーバーID: #{server['ID']}"
718716
puts " - 説明: #{server['Description']}"
719717
puts " - タグ: #{server['Tags'].join(', ')}"
720718
puts " - ステータス: #{server['Instance']['Status']}"
@@ -747,7 +745,6 @@ def display_deletion_plan(server_info, ip_address, dojo_name)
747745
puts "以下のサーバーを初期化(削除して再作成)します:"
748746
puts ""
749747
puts " サーバー名: #{server_info['Name']}"
750-
puts " サーバーID: #{server_info['ID']}"
751748
puts " IPアドレス: #{ip_address}"
752749
puts " CoderDojo: #{dojo_name || '(自動判定)'}"
753750
puts ""
@@ -757,7 +754,7 @@ def display_deletion_plan(server_info, ip_address, dojo_name)
757754
puts "1. さくらのクラウドコントロールパネルにログイン"
758755
puts " https://secure.sakura.ad.jp/cloud/"
759756
puts ""
760-
puts "2. サーバーID: #{server_info['ID']} を検索"
757+
puts "2. サーバー「#{server_info['Name']}を検索"
761758
puts ""
762759
puts "3. サーバーを停止してから削除(ディスクも含む)"
763760
puts ""

0 commit comments

Comments
 (0)