check flow now uses flag for project and reads env var

Jakob Külzer · Jakob Külzer · commit 4b8a64a081c0 · 2018-06-05T19:34:16.000-04:00
diff --git a/cli/cmd/check/check.go b/cli/cmd/check/check.go
@@ -27,7 +27,7 @@ func ConfigureCommand(app *kingpin.Application, ctx *azkaban.Context) {
 	}
 	cmd = app.Command("check", "")
 	checkFlowCmd = cmd.Command("flow", "checks a flow").Action(s.checkFlow)
-	project = checkFlowCmd.Arg("project", "Project").Required().String()
+	project = checkFlowCmd.Flag("project", "Project").Envar("HARBORMASTER_PROJECT").Required().String()
 	flow = checkFlowCmd.Arg("flow", "Flow").HintAction(s.suggestFlow).Required().String()
 	checkCountFlag = checkFlowCmd.Flag("execution-count", "number of executions to check").Short('n').Default("20").Int()
 	detailCountFlag = checkFlowCmd.Flag("detail-count", "number of execution details").Short('d').Default("5").Int()

Original file line number	Diff line number	Diff line change
`@@ -27,7 +27,7 @@ func ConfigureCommand(app kingpin.Application, ctx azkaban.Context) {`
`27`	`27`	`}`
`28`	`28`	`cmd = app.Command("check", "")`
`29`	`29`	`checkFlowCmd = cmd.Command("flow", "checks a flow").Action(s.checkFlow)`
`30`		`- project = checkFlowCmd.Arg("project", "Project").Required().String()`
	`30`	`+ project = checkFlowCmd.Flag("project", "Project").Envar("HARBORMASTER_PROJECT").Required().String()`
`31`	`31`	`flow = checkFlowCmd.Arg("flow", "Flow").HintAction(s.suggestFlow).Required().String()`
`32`	`32`	`checkCountFlag = checkFlowCmd.Flag("execution-count", "number of executions to check").Short('n').Default("20").Int()`
`33`	`33`	`detailCountFlag = checkFlowCmd.Flag("detail-count", "number of execution details").Short('d').Default("5").Int()`