Merge pull request #5 from iwashi623/feature/add_jobsettingfile_option

Feature/add jobsettingfile option

README.md

@@ -7,7 +7,7 @@ Glue Job にのみ関心を持つツールで、Glue のスクリプトや設定
 ## Use gjobctl
 
 ### 準備
-
+GlueJobのリポジトリ例
 ```bash
 sample-job
 ├── gjobctl.yml  ## <- gjobctl設定ファイル
@@ -21,6 +21,7 @@ sample-job
 ```yml:gjobctl.yml
 region: ap-northeast-1
 job_name: sample-job
+job_setting_file: sample-job.json
 ```
 
 json の Glue Job 定義ファイルは下記する`gjobctl get`コマンドを使うと簡単に手に入ります。

create.go

@@ -18,10 +18,12 @@ type CreateOption struct {
 func (app *App) Create(opt *CreateOption) error {
 	// JSONファイルからGlue Jobの設定を読み込む
 	var settingFileName string
-	if opt.JobSettingFile == nil {
-		settingFileName = app.config.JobName + ".json"
-	} else {
+	if opt.JobSettingFile != nil {
 		settingFileName = *opt.JobSettingFile
+	} else if app.config.JobSettingFile != "" {
+		settingFileName = app.config.JobSettingFile
+	} else {
+		settingFileName = app.config.JobName + ".json"
 	}
 	f, err := os.Open(settingFileName)
 	if err != nil {

gjobctl.go

@@ -40,6 +40,7 @@ func (app *App) setup() error {
 }
 
 type AppConfig struct {
-	Region  string `yaml:"region"`
-	JobName string `yaml:"job_name"`
+	Region         string `yaml:"region"`
+	JobName        string `yaml:"job_name"`
+	JobSettingFile string `yaml:"job_setting_file"`
 }

script_deploy.go

@@ -20,10 +20,12 @@ type ScriptDeployOption struct {
 func (app *App) ScriptDeploy(opt *ScriptDeployOption) error {
 	// JSONファイルからGlue Jobの設定を読み込む
 	var settingFileName string
-	if opt.JobSettingFile == nil {
-		settingFileName = app.config.JobName + ".json"
-	} else {
+	if opt.JobSettingFile != nil {
 		settingFileName = *opt.JobSettingFile
+	} else if app.config.JobSettingFile != "" {
+		settingFileName = app.config.JobSettingFile
+	} else {
+		settingFileName = app.config.JobName + ".json"
 	}
 	f, err := os.Open(settingFileName)
 	if err != nil {

update.go

@@ -18,10 +18,12 @@ type UpdateOption struct {
 func (app *App) Update(opt *UpdateOption) error {
 	// JSONファイルからGlue Jobの設定を読み込む
 	var settingFileName string
-	if opt.JobSettingFile == nil {
-		settingFileName = app.config.JobName + ".json"
-	} else {
+	if opt.JobSettingFile != nil {
 		settingFileName = *opt.JobSettingFile
+	} else if app.config.JobSettingFile != "" {
+		settingFileName = app.config.JobSettingFile
+	} else {
+		settingFileName = app.config.JobName + ".json"
 	}
 	f, err := os.Open(settingFileName)
 	if err != nil {