Skip to content

Commit 155dd98

Browse files
committed
refactor: utilsスクリプトでデフォルトパラメータを活用
- すべてのutilsスクリプトからハードコーディングされた値を削除 - SakuraServerUserAgentのデフォルト値を活用 - 7ファイルで重複していたパラメータ設定を削除 - scripts/initialize_server.rb - scripts/utils/check_server_status.rb - scripts/utils/check_startup_scripts.rb - scripts/utils/find_resources.rb - scripts/utils/start_server.rb - scripts/utils/test_with_packet_filter.rb - scripts/utils/test_with_startup_script.rb - DRY原則の徹底により保守性が向上
1 parent 40520b2 commit 155dd98

File tree

7 files changed

+14
-52
lines changed

7 files changed

+14
-52
lines changed

scripts/initialize_server.rb

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -55,13 +55,8 @@ def initialize(input, options = {})
5555
@dry_run = options[:dry_run] || false
5656
@force = options[:force] || false
5757

58-
# さくらのクラウドAPIクライアント初期化(石狩第二ゾーン)
59-
@ssua = SakuraServerUserAgent.new(
60-
zone: "31002",
61-
zone_id: "is1b",
62-
packet_filter_id: nil,
63-
verbose: @verbose
64-
)
58+
# さくらのクラウドAPIクライアント初期化(デフォルト値使用)
59+
@ssua = SakuraServerUserAgent.new(verbose: @verbose)
6560
end
6661

6762
def run

scripts/utils/check_server_status.rb

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -11,13 +11,8 @@
1111

1212
search_term = ARGV[0]
1313

14-
params = {
15-
zone: "31002",
16-
zone_id: "is1b",
17-
packet_filter_id: nil
18-
}
19-
20-
ssua = SakuraServerUserAgent.new(**params)
14+
# デフォルト値を使用(石狩第二ゾーン)
15+
ssua = SakuraServerUserAgent.new
2116

2217
puts "=== サーバー詳細情報 ==="
2318
puts "検索条件: #{search_term}"

scripts/utils/check_startup_scripts.rb

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -5,12 +5,6 @@
55
require 'dotenv/load'
66
require_relative '../sakura_server_user_agent.rb'
77

8-
params = {
9-
zone: "31002",
10-
zone_id: "is1b",
11-
packet_filter_id: nil
12-
}
13-
148
puts "=== スタートアップスクリプト(Note)一覧 ==="
159
puts "時刻: #{Time.now}"
1610
puts ""
@@ -23,7 +17,8 @@ def get_notes
2317
end
2418
end
2519

26-
ssua = StartupScriptChecker.new(**params)
20+
# デフォルト値を使用(石狩第二ゾーン)
21+
ssua = StartupScriptChecker.new
2722

2823
# Noteリソースの一覧を取得
2924
response = ssua.get_notes()

scripts/utils/find_resources.rb

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -19,19 +19,13 @@
1919
# 検索文字列(引数がなければ全て表示)
2020
search_term = ARGV[0] || ""
2121

22-
# 石狩第二ゾーンのパラメータ(本番環境)
23-
params = {
24-
zone: "31002",
25-
zone_id: "is1b",
26-
packet_filter_id: nil
27-
}
28-
2922
puts "=== さくらのクラウド リソース検索 ==="
3023
puts "検索条件: #{search_term.empty? ? '全て' : search_term}"
3124
puts "時刻: #{Time.now.strftime('%Y-%m-%d %H:%M:%S')}"
3225
puts ""
3326

34-
ssua = SakuraServerUserAgent.new(**params)
27+
# デフォルト値を使用(石狩第二ゾーン)
28+
ssua = SakuraServerUserAgent.new
3529

3630
# サーバーの確認
3731
puts "【サーバー一覧】"

scripts/utils/start_server.rb

Lines changed: 2 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -15,13 +15,8 @@
1515

1616
server_id = ARGV[0]
1717

18-
params = {
19-
zone: "31002",
20-
zone_id: "is1b",
21-
packet_filter_id: nil
22-
}
23-
24-
ssua = SakuraServerUserAgent.new(**params)
18+
# デフォルト値を使用(石狩第二ゾーン)
19+
ssua = SakuraServerUserAgent.new
2520

2621
puts "サーバー #{server_id} を起動中..."
2722

scripts/utils/test_with_packet_filter.rb

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -20,21 +20,15 @@
2020
puts "サーバー名: #{server_name}"
2121
puts ""
2222

23-
# 本番環境と同じパラメータ
24-
params = {
25-
zone: "31002", # 石狩第二
26-
zone_id: "is1b", # 石狩第二
27-
packet_filter_id: '112900922505', # 本番環境のパケットフィルターID
28-
}
29-
3023
puts "📋 設定内容:"
3124
puts " - ゾーン: 石狩第二 (is1b)"
3225
puts " - パケットフィルターID: 112900922505"
3326
puts " - スタートアップスクリプトID: #{SakuraServerUserAgent::STARTUP_SCRIPT_ID}"
3427
puts ""
3528

3629
begin
37-
ssua = SakuraServerUserAgent.new(**params)
30+
# デフォルト値を使用(パケットフィルター含む)
31+
ssua = SakuraServerUserAgent.new
3832

3933
# 通常版Ubuntu 24.04を検索
4034
archives = ssua.get_archives()['Archives']

scripts/utils/test_with_startup_script.rb

Lines changed: 2 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -21,15 +21,9 @@
2121
puts "スタートアップスクリプトID: #{SakuraServerUserAgent::STARTUP_SCRIPT_ID}"
2222
puts ""
2323

24-
# サーバー作成パラメータ(シンプル)
25-
params = {
26-
zone: "31002",
27-
zone_id: "is1b",
28-
packet_filter_id: nil, # テスト用にパケットフィルター無効
29-
}
30-
3124
begin
32-
ssua = SakuraServerUserAgent.new(**params)
25+
# デフォルト値を使用(テスト用にパケットフィルター無効)
26+
ssua = SakuraServerUserAgent.new(packet_filter_id: nil)
3327

3428
# 通常版Ubuntu 24.04を検索
3529
archives = ssua.get_archives()['Archives']

0 commit comments

Comments
 (0)