Skip to content
This repository has been archived by the owner on Nov 8, 2022. It is now read-only.

Commit

Permalink
Merge pull request #334 from tiffanyfj/cover-control
Browse files Browse the repository at this point in the history
Made it so all missing metrics are shown to fix issue #333
  • Loading branch information
tiffanyfay committed Sep 5, 2015
2 parents 315901a + 904f021 commit 06e3d4d
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 3 deletions.
12 changes: 10 additions & 2 deletions cmd/pulsectl/task.go
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,11 @@ func createTaskUsingTaskManifest(ctx *cli.Context) {
r := pClient.CreateTask(t.Schedule, t.Workflow, t.Name, !ctx.IsSet("no-start"))

if r.Err != nil {
fmt.Printf("Error creating task:\n%v\n", r.Err)
errors := strings.Split(r.Err.Error(), " -- ")
fmt.Println("Error creating task:")
for _, err := range errors {
fmt.Printf("%v\n", err)
}
os.Exit(1)
}
fmt.Println("Task created")
Expand Down Expand Up @@ -199,7 +203,11 @@ func createTaskUsingWFManifest(ctx *cli.Context) {
// Create task
r := pClient.CreateTask(sch, wf, name, !ctx.IsSet("no-start"))
if r.Err != nil {
fmt.Printf("Error creating task:\n%v\n", r.Err)
errors := strings.Split(r.Err.Error(), " -- ")
fmt.Println("Error creating task:")
for _, err := range errors {
fmt.Printf("%v\n", err)
}
os.Exit(1)
}
fmt.Println("Task created")
Expand Down
4 changes: 3 additions & 1 deletion control/control.go
Original file line number Diff line number Diff line change
Expand Up @@ -285,9 +285,11 @@ func (p *pluginControl) ValidateDeps(mts []core.Metric, plugins []core.Subscribe
_, errs := p.validateMetricTypeSubscription(mt, mt.Config())
if len(errs) > 0 {
perrs = append(perrs, errs...)
return perrs
}
}
if len(perrs) > 0 {
return perrs
}

//validate plugins
for _, plg := range plugins {
Expand Down

0 comments on commit 06e3d4d

Please sign in to comment.